From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id EF6A2C433EF for ; Thu, 20 Jan 2022 17:56:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233145AbiATR4Q (ORCPT ); Thu, 20 Jan 2022 12:56:16 -0500 Received: from netrider.rowland.org ([192.131.102.5]:33547 "HELO netrider.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S233152AbiATR4L (ORCPT ); Thu, 20 Jan 2022 12:56:11 -0500 Received: (qmail 158091 invoked by uid 1000); 20 Jan 2022 12:56:09 -0500 Date: Thu, 20 Jan 2022 12:56:09 -0500 From: Alan Stern To: Thomas =?iso-8859-1?Q?Wei=DFschuh?= Cc: linux-usb@vger.kernel.org, DocMAX Subject: Re: Issue with UAS and" VIA Labs, Inc. VL817 SATA Adaptor" Message-ID: References: <40eecdd0-93bc-40c6-b8c0-f4ad4c6ffe59@t-8ch.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org On Thu, Jan 20, 2022 at 06:39:51PM +0100, Thomas Weißschuh wrote: > Hi Alan, > > On 2022-01-20 09:42-0500, Alan Stern wrote: > > On Thu, Jan 20, 2022 at 08:28:30AM +0100, Thomas Weißschuh wrote: > > > Hi Alan, > > > > > > I hava a IcyBox IB-3740-C31 [0], this device seems to be identical to the one > > > here. > > > > When you say "described here", do you mean "described earlier in this > > email thread"? I ask because from the way you wrote that sentence, it > > looks like you mean that the device described in [0] seems to be > > identical to the one you have. > > I meant the one described in this thread. I have the one sold at [0], which > looks identical except for the logo to the one described in the thread. > > > > > It has the same USB IDs and case design. > > > It also has the serial number "4". > > > The only difference it seems is the field bcdDevice which is "1.36" and the > > > reported name is different (see the patch below). > > > > > > So I adapted the patch slightly to also match that bcdDevice. > > > I also changed the productName field but that does not seem to be used anyways. > > > > > > Using the quirk flags "fgkm" as mentioned in [1] did not help. > > > > > > FYI while there are many reports that UAS does not work with these devices, > > > there also are a few that report it working. For example [2]. > > > > That's odd. And I don't really want to change the kernel in a way that > > will cause those working devices to stop working with UAS. > > This is why I brought it up. There are however many more reports online for > those devices where UAS does not work. It's a bad situation. It would be nice if we could figure out why some of them work and others don't. Also, do these same devices work with UAS in Windows or Mac OSX? > > > diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h > > > index 29191d33c0e3..53e8249644b2 100644 > > > --- a/drivers/usb/storage/unusual_devs.h > > > +++ b/drivers/usb/storage/unusual_devs.h > > > @@ -2301,6 +2301,19 @@ UNUSUAL_DEV( 0x2027, 0xa001, 0x0000, 0x9999, > > > USB_SC_DEVICE, USB_PR_DEVICE, usb_stor_euscsi_init, > > > US_FL_SCM_MULT_TARG ), > > > > > > +UNUSUAL_DEV( 0x2109, 0x0715, 0x0000, 0x9999, > > > + "VIA Labs, Inc.", > > > + "VL817 SATA Adaptor", > > > + USB_SC_DEVICE, USB_PR_DEVICE, NULL, > > > + US_FL_IGNORE_UAS), > > > > Does the new patch fix your problem? > > Yes it does. As does disabling UAS via the usb-storage quirk parameter. All right. I will submit the revised patch after the current merge window is over. Alan Stern > > Alan Stern > > > > > This is the exact issue: > > > > > > [ 3606.231973] scsi host14: uas_eh_device_reset_handler start > > > [ 3606.232149] sd 14:0:0:0: [sdg] tag#2 uas_zap_pending 0 uas-tag 1 inflight: CMD > > > [ 3606.232154] sd 14:0:0:0: [sdg] tag#2 CDB: Write(16) 8a 00 00 00 00 00 18 0c c9 80 00 00 00 80 00 00 > > > [ 3606.306257] usb 4-4.4: reset SuperSpeed Plus Gen 2x1 USB device number 11 using xhci_hcd > > > [ 3606.328584] scsi host14: uas_eh_device_reset_handler success > > > > > > For this patch: > > > > > > Tested-by: Thomas Weißschuh > > > > > > Thomas > > > > > > [0] https://icybox.de/en/product.php?id=155 > > > [1] https://lore.kernel.org/linux-usb/c4b4aa34-12d9-7000-6398-d94a7ebffdfc@suse.com/ > > > [2] https://spod.cx/blog/enabling_trim_support_via_VL817_usb_sata_adaptor.shtml > > Thomas