On Wed, Jul 08, 2015 at 01:36:04PM +0300, Sagi Grimberg wrote: > We don't have any other information today, but sector is not the only > information that is requires a descriptor format, so maybe it will be a > bit awkward to condition the descriptor format on the sector info? The only reason why you'd want to support descriptor type sense data is because you need to add a second descriptor. If we have another case that needs descriptor sense data it'll also need to add that additional descriptor. So we'll need a conditional for it in the sense data generation anyway. -- 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