public inbox for linux-usb@vger.kernel.org
 help / color / mirror / Atom feed
From: Julian Sikorski <belegdol@gmail.com>
To: "Martin K. Petersen" <martin.petersen@oracle.com>
Cc: linux-scsi@vger.kernel.org, linux-usb@vger.kernel.org
Subject: Re: [PATCH] usb-storage: Ignore UAS for LaCie Rugged FW USB3
Date: Tue, 13 Feb 2024 19:14:39 +0100	[thread overview]
Message-ID: <0a892e80-424f-4047-a885-5efbcc75af53@gmail.com> (raw)
In-Reply-To: <yq1bk8kl9th.fsf@ca-mkp.ca.oracle.com>

Am 13.02.24 um 15:28 schrieb Martin K. Petersen:
> 
> Julian,
> 
>> I am cross-posting this to scsi list, maybe someone there would be
>> able to understand what is going on.
> 
> I'm going to copy you on a patch I have in my development tree. I would
> appreciate if you could try that out.
> 
> Thanks!
> 
Hi Martin,

Thanks for responding. I tried your patch but it, unfortunately, did not 
help with the issue I and Tasos were having. Not on its own, and not 
with LaCie:Rugged FW USB3:0x2000 added to /proc/scsi/device_info.
Just to make sure we are talking about the same problem: prior to 
c92a6b5d6335, the drive would be usable pretty much immediately after 
connecting - in the logs this manifests by the following rows appearing:

Feb 13 18:59:42 kernel:  sdb: sdb1 sdb2
Feb 13 18:59:42 kernel: sd 8:0:0:0: [sdb] Attached SCSI disk

After c92a6b5d6335, one or more restarts are required (below is from 
Fedora's 6.7.4 with your patch applied on top):

Feb 13 19:01:41 kernel: sd 8:0:0:0: [sdb] tag#13 uas_eh_abort_handler 0 
uas-tag 1 inflight: IN
Feb 13 19:01:41 kernel: sd 8:0:0:0: [sdb] tag#13 CDB: Inquiry 12 01 b9 
00 ff 00
Feb 13 19:01:41 kernel: scsi host8: uas_eh_device_reset_handler start
Feb 13 19:01:41 napoleon2 kernel: usb 2-3: reset SuperSpeed USB device 
number 3 using xhci_hcd
Feb 13 19:01:41 kernel: scsi host8: uas_eh_device_reset_handler success
Feb 13 19:02:11 kernel: scsi host8: uas_eh_device_reset_handler start
Feb 13 19:02:11 kernel: sd 8:0:0:0: [sdb] tag#16 uas_zap_pending 0 
uas-tag 1 inflight:
Feb 13 19:02:11 kernel: sd 8:0:0:0: [sdb] tag#16 CDB: Inquiry 12 01 b9 
00 ff 00
Feb 13 19:02:11 kernel: usb 2-3: reset SuperSpeed USB device number 3 
using xhci_hcd
Feb 13 19:02:11 kernel: scsi host8: uas_eh_device_reset_handler success
Feb 13 19:02:41 kernel: scsi host8: uas_eh_device_reset_handler start
Feb 13 19:02:41 kernel: sd 8:0:0:0: [sdb] tag#17 uas_zap_pending 0 
uas-tag 1 inflight:
Feb 13 19:02:41 kernel: sd 8:0:0:0: [sdb] tag#17 CDB: Inquiry 12 01 b9 
00 ff 00
Feb 13 19:02:42 kernel: usb 2-3: reset SuperSpeed USB device number 3 
using xhci_hcd
Feb 13 19:02:42 kernel: scsi host8: uas_eh_device_reset_handler success
Feb 13 19:03:12 kernel: scsi host8: uas_eh_device_reset_handler start
Feb 13 19:03:12 kernel: sd 8:0:0:0: [sdb] tag#28 uas_zap_pending 0 
uas-tag 1 inflight:
Feb 13 19:03:12 kernel: sd 8:0:0:0: [sdb] tag#28 CDB: Inquiry 12 01 b9 
00 ff 00
Feb 13 19:03:12 kernel: usb 2-3: reset SuperSpeed USB device number 3 
using xhci_hcd
Feb 13 19:03:12 kernel: scsi host8: uas_eh_device_reset_handler success

Four resets are needed with the current kernel-6.7.4. With early 5.19 
kernels on and after 7fb019c46eee it needed one reset, with c92a6b5d6335 
and with e60ac0b9e445 multiple resets were needed too. As 30 seconds 
need to pass between the resets, it does take a while until the drive 
can be used.
I did not test other commits between c92a6b5d6335 and c92a6b5d6335 as I 
was bisecting and only tested the commits suggested by git.
6.7.4 I tested with bare metal Fedora 39, 5.19 with a Fedora 36 VM.
If this was already clear, please feel free to ignore.
In case it is relevant: my drive (but not Tasos') is patched to use 
US_FL_NO_REPORT_OPCODES | US_FL_NO_SAME.

Best regards,
Julian




  reply	other threads:[~2024-02-13 18:14 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-09 15:11 [PATCH] usb-storage: Ignore UAS for LaCie Rugged FW USB3 Tasos Sahanidis
2024-02-09 15:31 ` Alan Stern
2024-02-10 13:24   ` Tasos Sahanidis
2024-02-10 14:21     ` Julian Sikorski
2024-02-10 21:35       ` Julian Sikorski
2024-02-10 21:43         ` Julian Sikorski
2024-02-13 13:06           ` Julian Sikorski
2024-02-13 14:28             ` Martin K. Petersen
2024-02-13 18:14               ` Julian Sikorski [this message]
2024-02-13 18:49                 ` Martin K. Petersen
2024-02-13 14:32             ` Oliver Neukum

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=0a892e80-424f-4047-a885-5efbcc75af53@gmail.com \
    --to=belegdol@gmail.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    /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