On 9/24/17 16:59, Christoph Hellwig wrote: >> + * Zoned block device information for mq I/O schedulers. >> + * Set by low level device driver (stacking driver may not set this). >> + */ >> + unsigned int nr_zones; >> + unsigned long *seq_zones; > > Might be worth explaining the seq_zones is a bit bitmap to check > if a given zone number is sequential. > > And maybe give it a name that hints at that, like seq_zone_bitmap? > > Otherwise looks fine: > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> Thanks. Will send a V5 with your suggested modifications. -- Damien Le Moal Western Digital Research