On 6/25/25 3:18 PM, Christoph Hellwig wrote: > On Wed, Jun 25, 2025 at 03:14:51PM +0900, Damien Le Moal wrote: >> On 6/25/25 3:12 PM, Christoph Hellwig wrote: >>> On Wed, Jun 25, 2025 at 02:59:05PM +0900, Damien Le Moal wrote: >>>> +bool bio_needs_zone_write_plugging(struct bio *bio) >>> >>> Can you use this in blk_zone_plug_bio instead of duplicating the logic? >> >> I thought about doing that, but we would still need to again do the >> switch/case >> on the bio op. But the checks before that could go into a common static >> helper. >> >>> I also wonder if we should only it it, as despite looking quite complex >> >> This does not parse... > > The "only it" above should be "inline", -ENOTENOUGHCOFFEE
OK. Let me see if I can do that. -- Damien Le Moal Western Digital Research