On Fri, Jul 25, 2014 at 04:34:21PM -0400, Martin K. Petersen wrote: > + return sectors >> (ilog2(bi->interval) - 9); I was going to say this should be 'SECTOR_SHIFT', but that constant or variants of it are only defined locally in various subsystems.. Looks good, Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html