Bart, > Do the benefits of this change outweigh the additional complexity > introduced by this code and the risk of breaking support for certain > devices? I'm asking this because the number of LLDs that sets > inquiry_len is small: > > $ git grep -nH '>inquiry_len[[:blank:]]*=[[:blank:]]'|grep -v scsi_scan > drivers/firewire/sbp2.c:1508: sdev->inquiry_len = 36; > drivers/staging/rts5208/rtsx.c:67: sdev->inquiry_len = 36; > drivers/usb/image/microtek.c:323: s->inquiry_len = 0x24; > drivers/usb/storage/scsiglue.c:77: sdev->inquiry_len = 36; > > Does any of these LLDs support SPC-4 devices? Can this change > e.g. break support for certain USB sticks? The intent is to enable more modern protocol features on devices attached using USB. I was concerned about blindly increasingly the inquiry length and risk breaking older devices that we know are likely to have problems in this department. At the same time the conservative clamp on the inquiry length prevents several useful features from being enabled on modern devices. There is a risk associated with any change. This patch tries to strike a reasonable balance between avoiding regressions and accommodating modern USB SSDs. -- Martin K. Petersen Oracle Linux Engineering
