mbox series

[0/2] Do not read the IO hints VPD page from USB storage devices

Message ID 20240612165249.2671204-1-bvanassche@acm.org
Headers show
Series Do not read the IO hints VPD page from USB storage devices | expand

Message

Bart Van Assche June 12, 2024, 4:52 p.m. UTC
Hi Martin,

Recently it was reported that reading the IO hints VPD page makes at least two
USB storage devices crash. Hence this patch series that disables reading the IO
hints VPD page from USB storage devices. Please consider this patch series for
your scsi-fixes branch.

Thanks,

Bart.

Bart Van Assche (2):
  scsi: core: Introduce the BLIST_SKIP_IO_HINTS flag
  scsi: core: Do not query IO hints for USB devices

 drivers/scsi/sd.c              | 4 ++++
 drivers/usb/storage/scsiglue.c | 2 ++
 include/scsi/scsi_devinfo.h    | 4 +++-
 3 files changed, 9 insertions(+), 1 deletion(-)

Comments

Andy Shevchenko June 13, 2024, 5:44 p.m. UTC | #1
On Wed, Jun 12, 2024 at 6:53 PM Bart Van Assche <bvanassche@acm.org> wrote:
>
> Recently it was reported that the following USB storage devices are unusable
> with Linux kernel 6.9:
> * Kingston DataTraveler G2
> * Garmin FR35
>
> This is because attempting to read the IO hint VPD page causes these devices
> to reset. Hence do not read the IO hint VPD page from USB storage devices.

> Cc: Alan Stern <stern@rowland.harvard.edu>
> Cc: linux-usb@vger.kernel.org
> Cc: Joao Machado <jocrismachado@gmail.com>

> Cc: Andy Shevchenko <andy.shevchenko@gmail.com>
> Cc: Christian Heusel <christian@heusel.eu>

Besides no need to repeat these Cc's in case there are other tags for
the same emails, can you move the rest of Cc's after the --- line
below? For you it will be the same effect, for many others the Git
history won't be polluted with this noise.
Bart Van Assche June 13, 2024, 6:10 p.m. UTC | #2
On 6/13/24 10:44 AM, Andy Shevchenko wrote:
> On Wed, Jun 12, 2024 at 6:53 PM Bart Van Assche <bvanassche@acm.org> wrote:
>>
>> Recently it was reported that the following USB storage devices are unusable
>> with Linux kernel 6.9:
>> * Kingston DataTraveler G2
>> * Garmin FR35
>>
>> This is because attempting to read the IO hint VPD page causes these devices
>> to reset. Hence do not read the IO hint VPD page from USB storage devices.
> 
>> Cc: Alan Stern <stern@rowland.harvard.edu>
>> Cc: linux-usb@vger.kernel.org
>> Cc: Joao Machado <jocrismachado@gmail.com>
> 
>> Cc: Andy Shevchenko <andy.shevchenko@gmail.com>
>> Cc: Christian Heusel <christian@heusel.eu>
> 
> Besides no need to repeat these Cc's in case there are other tags for
> the same emails, can you move the rest of Cc's after the --- line
> below? For you it will be the same effect, for many others the Git
> history won't be polluted with this noise.

I will leave out the redundant Cc's but I'm surprised by the request to move
Cc tags after the --- line. There are many patches with Cc: tags in Linus' tree.
I have never before seen anyone requesting to move Cc tags after the --- line.

Thanks,

Bart.
Andy Shevchenko June 13, 2024, 7:44 p.m. UTC | #3
On Thu, Jun 13, 2024 at 8:10 PM Bart Van Assche <bvanassche@acm.org> wrote:
> On 6/13/24 10:44 AM, Andy Shevchenko wrote:
> > On Wed, Jun 12, 2024 at 6:53 PM Bart Van Assche <bvanassche@acm.org> wrote:
> >>
> >> Recently it was reported that the following USB storage devices are unusable
> >> with Linux kernel 6.9:
> >> * Kingston DataTraveler G2
> >> * Garmin FR35
> >>
> >> This is because attempting to read the IO hint VPD page causes these devices
> >> to reset. Hence do not read the IO hint VPD page from USB storage devices.
> >
> >> Cc: Alan Stern <stern@rowland.harvard.edu>
> >> Cc: linux-usb@vger.kernel.org
> >> Cc: Joao Machado <jocrismachado@gmail.com>
> >
> >> Cc: Andy Shevchenko <andy.shevchenko@gmail.com>
> >> Cc: Christian Heusel <christian@heusel.eu>
> >
> > Besides no need to repeat these Cc's in case there are other tags for
> > the same emails, can you move the rest of Cc's after the --- line
> > below? For you it will be the same effect, for many others the Git
> > history won't be polluted with this noise.
>
> I will leave out the redundant Cc's but I'm surprised by the request to move
> Cc tags after the --- line. There are many patches with Cc: tags in Linus' tree.

I know, and that's why I am asking for them to be moved away. It's
just duplicate information and since we have lore.kernel.org, we may
always get the real message with the same data, no need to repeat this
in Git history.

> I have never before seen anyone requesting to move Cc tags after the --- line.