linux-usb.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Alan Stern <stern@rowland.harvard.edu>
To: Andrea Vai <andrea.vai@unipv.it>
Cc: Greg KH <gregkh@linuxfoundation.org>, <linux-usb@vger.kernel.org>
Subject: Re: Slow I/O on USB media
Date: Mon, 1 Jul 2019 14:57:00 -0400 (EDT)	[thread overview]
Message-ID: <Pine.LNX.4.44L0.1907011453340.1536-100000@iolanthe.rowland.org> (raw)
In-Reply-To: <3e7662d1391346dd11a903e66e7a8936ca83dba9.camel@unipv.it>

On Mon, 1 Jul 2019, Andrea Vai wrote:

> > At any rate, you are some commits you could try (beginning with
> > "git 
> > checkout <commit>" and then running your scripts):
> > 
> >         c76cd634eb5b
> >         b1669432b355
> >         507413a5f88a
> >         a52fb43a5faa
> >         38fabca18fc4
> >         fc2fd5f0f1aa
> > 
> > These are all between 4.20 and 5.0-rc1.
> 
> Hi,
>   these were all "good".
> 
> Then I ran another bisect (the sixth (!), more carefully, starting
> from
> 
> git bisect good c76cd634eb5b
> git bisect bad 241e39004581
> 
> ), and it seems to give some consistent result.
> 
> I found that:
> 
> f664a3cc17b7d0a2bc3b3ab96181e1029b0ec0e6 is the first bad commit
> commit f664a3cc17b7d0a2bc3b3ab96181e1029b0ec0e6
> Author: Jens Axboe <axboe@kernel.dk>
> Date:   Thu Nov 1 16:36:27 2018 -0600
> 
>     scsi: kill off the legacy IO path
>     
>     This removes the legacy (non-mq) IO path for SCSI.
>     
>     Cc: linux-scsi@vger.kernel.org
>     Acked-by: Himanshu Madhani <himanshu.madhani@cavium.com>
>     Reviewed-by: Hannes Reinecke <hare@suse.com>
>     Tested-by: Ming Lei <ming.lei@redhat.com>
>     Reviewed-by: Omar Sandoval <osandov@fb.com>
>     Acked-by: Martin K. Petersen <martin.petersen@oracle.com>
>     Signed-off-by: Jens Axboe <axboe@kernel.dk>
> 
> :040000 040000 312373927bae1c6fd1da40ded2c12dfa5e4de71c 4eccbd2c84bf83cb2eb72a81514d59ebf12866b7 M	Documentation
> :040000 040000 98de24b4fe20b82095f53f56c9193c5537d70ed0 8e2092780100205ae1c3723a598a89794a50677f M	drivers
> :040000 040000 fbc10c84d3eb6b7933598018319f96767ee3a0f3 2523940c2819e8adb32758f5093e477da481ca65 M	include
> 
> I reverted it and the test succeeded.
> Then I made a double check: "git clone" again; git checkout
> 3a7ea2c483a53fc89e336f69c6ee1d7defe00811 (the last good), and the test
> succeded. Then git checkout f664a3cc17b7d0a2bc3b3ab96181e1029b0ec0e6
> (the first bad) and the test failed; then reverted it and the test
> succeded again.
> 
> Does it make sense?

Yes, that does make sense.

What you should do next is report the problem to all the people
involved in that commit: Jens Axboe, Himanshu Madhani, and so on.  Be 
sure to CC: the USB mailing list and also linux-scsi@vger.kernel.org.

Given them a full description of the problem and explain how you
determined that this commit was the cause.  They should be able to help
figure out what's going wrong and fix it.

Alan Stern


  reply	other threads:[~2019-07-01 18:57 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-30 13:18 Slow I/O on USB media Andrea Vai
2019-05-30 13:25 ` Greg KH
2019-06-03 11:13   ` Andrea Vai
2019-06-04  5:43     ` Greg KH
2019-06-04  7:26       ` Andrea Vai
2019-06-05  7:36       ` Andrea Vai
2019-06-05 14:26         ` Alan Stern
2019-06-05 15:46           ` Andrea Vai
2019-06-05 16:11             ` Alan Stern
2019-06-05 14:55         ` Greg KH
     [not found]           ` <0c2adde7154b0a6c8b2ad7fc5258916731b78775.camel@unipv.it>
2019-06-05 16:23             ` Andrea Vai
2019-06-05 17:39               ` Greg KH
2019-06-06  8:41                 ` Andrea Vai
2019-06-06  9:03                 ` Andrea Vai
2019-06-06 14:00                 ` Andrea Vai
2019-06-06 14:30                   ` Alan Stern
2019-06-06 14:47                   ` Greg KH
2019-06-07  7:59                     ` Andrea Vai
2019-06-08  7:43                     ` Andrea Vai
2019-06-08  9:29                       ` Andrea Vai
2019-06-10 14:38                         ` Greg KH
2019-06-11  6:48                           ` Andrea Vai
2019-06-10 14:40                         ` Alan Stern
2019-06-10 14:55                           ` Andrea Vai
2019-06-10 16:20                             ` Alan Stern
2019-06-17 15:52                           ` Andrea Vai
2019-06-17 16:14                             ` Alan Stern
2019-06-17 16:34                               ` Andrea Vai
2019-06-17 17:28                                 ` Alan Stern
2019-07-01 17:52                                   ` Andrea Vai
2019-07-01 18:57                                     ` Alan Stern [this message]
2019-06-10 14:37                       ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Pine.LNX.4.44L0.1907011453340.1536-100000@iolanthe.rowland.org \
    --to=stern@rowland.harvard.edu \
    --cc=andrea.vai@unipv.it \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-usb@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).