On Fri, Oct 12, 2018 at 09:41:28AM +0200, Hannes Reinecke wrote: >> +{ >> + return 0; >> +} >> #endif /* CONFIG_BLK_DEV_ZONED */ >> static inline bool rq_is_sync(struct request *rq) >> > Actually, we should be checking whether we can't blank out this attribute > via the is_visible mechanism; after all, not every block device is zoned, > and those which are not have no need of the attribute... The answer would be 0 in this case. I'm not sure blanking out is worth it as it will create a lot more code for very little gain..