Re: usb/180726: XHCI umass support breaks between r248085 and r252560 on 9-STABLE

2013-07-27 Thread Alexandre Kovalenko
The following reply was made to PR usb/180726; it has been noted by GNATS.

From: Alexandre Kovalenko bsd.gai...@gmail.com
To: bug-follo...@freebsd.org,
 bsd.gai...@gmail.com
Cc:  
Subject: Re: usb/180726: XHCI umass support breaks between r248085 and r252560 
on 9-STABLE
Date: Sat, 27 Jul 2013 23:07:04 -0400

 --Apple-Mail=_14A7616C-868E-4938-8FF0-F03573AB44BF
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/plain;
charset=utf-8
 
 I can confirm that as of
 
 FreeBSD twinhead 9.2-BETA2 FreeBSD 9.2-BETA2 #13 r253716: Sat Jul 27 =
 18:26:26 EDT 2013
 
 problem is fixed and drives are properly recognized.
 
 Thank you,
 
 
 Alexandre Sunny Kovalenko (=D0=9E=D0=BB=D0=B5=D0=BA=D1=81=D0=B0=D0=BD=D0=
 =B4=D1=80 =D0=9A=D0=BE=D0=B2=D0=B0=D0=BB=D0=B5=D0=BD=D0=BA=D0=BE)
 
 
 
 
 
 --Apple-Mail=_14A7616C-868E-4938-8FF0-F03573AB44BF
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/html;
charset=utf-8
 
 htmlheadmeta http-equiv=3DContent-Type content=3Dtext/html =
 charset=3Dutf-8/headbody style=3Dword-wrap: break-word; =
 -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; I can =
 confirm that as ofdivbr/divdivFreeBSD twinhead 9.2-BETA2 FreeBSD =
 9.2-BETA2 #13 r253716: Sat Jul 27 18:26:26 EDT =
 2013/divdivbr/divdivproblem is fixed and drives are properly =
 recognized./divdivbr/divdivThank you,/divbrbrdiv
 span class=3DApple-style-span style=3Dborder-collapse: separate; =
 color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; =
 font-variant: normal; font-weight: normal; letter-spacing: normal; =
 line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: =
 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: =
 0px; -webkit-border-horizontal-spacing: 0px; =
 -webkit-border-vertical-spacing: 0px; =
 -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: =
 auto; -webkit-text-stroke-width: 0px; font-size: medium; span =
 class=3DApple-style-span style=3Dborder-collapse: separate; color: =
 rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: =
 normal; font-weight: normal; letter-spacing: normal; line-height: =
 normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; =
 text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; =
 -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: =
 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: =
 auto; -webkit-text-stroke-width: 0px; font-size: medium; div =
 style=3Dword-wrap: break-word; -webkit-nbsp-mode: space; =
 -webkit-line-break: after-white-space; divAlexandre Sunny =
 Kovalenko (=D0=9E=D0=BB=D0=B5=D0=BA=D1=81=D0=B0=D0=BD=D0=B4=D1=80 =
 =D0=9A=D0=BE=D0=B2=D0=B0=D0=BB=D0=B5=D0=BD=D0=BA=D0=BE)/divdivbr/di=
 v/div/spanbr class=3DApple-interchange-newline/spanbr =
 class=3DApple-interchange-newline
 /div
 br/body/html=
 
 --Apple-Mail=_14A7616C-868E-4938-8FF0-F03573AB44BF--
___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org


Re: usb/180726: XHCI umass support breaks between r248085 and r252560 on 9-STABLE

2013-07-21 Thread Hans Petter Selasky

On 07/21/13 21:06, Alexandre Kovalenko wrote:



Number: 180726
Category:   usb
Synopsis:   XHCI umass support breaks between r248085 and r252560 on 
9-STABLE
Confidential:   no
Severity:   non-critical
Priority:   low
Responsible:freebsd-usb
State:  open
Quarter:
Keywords:
Date-Required:
Class:  sw-bug
Submitter-Id:   current-users
Arrival-Date:   Sun Jul 21 19:10:00 UTC 2013
Closed-Date:
Last-Modified:
Originator: Alexandre Kovalenko
Release:9-STABLE
Organization:
Environment:

FreeBSD twinhead 9.2-BETA1 FreeBSD 9.2-BETA1 #11 r248085:253503: Sun Jul 21 
14:29:42 EDT 2013 root@twinhead:/usr/obj/usr/src/sys/TWINHEAD  amd64

Description:


  Three different external hard drives (Seagate, Western Digital and noname USB 
3.0 enclosure) refused to be recognized as the umass devices. Reverting 
/usr/src/sys/dev/bsd/controller to r248085, building and loading just xhci 
module makes drives appear again. Below are snippets from the log in both cases:



Can you try:

1) Attached patch.

2) Non ASMedia USB 3.0 controller

--HPS

diff --git a/sys/dev/usb/controller/xhci.c b/sys/dev/usb/controller/xhci.c
index e6d5865..7070ef7 100644
--- a/sys/dev/usb/controller/xhci.c
+++ b/sys/dev/usb/controller/xhci.c
@@ -1654,7 +1654,7 @@ restart:
 			/* fill out buffer pointers */
 
 			if (average == 0) {
-npkt = 0;
+npkt = 1;
 memset(buf_res, 0, sizeof(buf_res));
 			} else {
 usbd_get_page(temp-pc, temp-offset +
@@ -1691,9 +1691,7 @@ restart:
 
 			switch (temp-trb_type) {
 			case XHCI_TRB_TYPE_ISOCH:
-/* BEI: Interrupts are inhibited until EOT */
 dword = XHCI_TRB_3_CHAIN_BIT | XHCI_TRB_3_CYCLE_BIT |
-XHCI_TRB_3_BEI_BIT |
 XHCI_TRB_3_TBC_SET(temp-tbc) |
 XHCI_TRB_3_TLBPC_SET(temp-tlbpc);
 if (td != td_first) {
@@ -1728,10 +1726,8 @@ restart:
 	dword |= XHCI_TRB_3_DIR_IN;
 break;
 			default:	/* XHCI_TRB_TYPE_NORMAL */
-/* BEI: Interrupts are inhibited until EOT */
 dword = XHCI_TRB_3_CHAIN_BIT | XHCI_TRB_3_CYCLE_BIT |
 XHCI_TRB_3_TYPE_SET(XHCI_TRB_TYPE_NORMAL) |
-XHCI_TRB_3_BEI_BIT |
 XHCI_TRB_3_TBC_SET(temp-tbc) |
 XHCI_TRB_3_TLBPC_SET(temp-tlbpc);
 if (temp-direction == UE_DIR_IN)
___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org

Re: usb/180726: XHCI umass support breaks between r248085 and r252560 on 9-STABLE

2013-07-21 Thread Alexandre Kovalenko
On Jul 21, 2013, at 3:32 PM, Hans Petter Selasky h...@bitfrost.no wrote:

 On 07/21/13 21:06, Alexandre Kovalenko wrote:
 
 Number: 180726
 Category:   usb
 Synopsis:   XHCI umass support breaks between r248085 and r252560 on 
 9-STABLE
 Confidential:   no
 Severity:   non-critical
 Priority:   low
 Responsible:freebsd-usb
 State:  open
 Quarter:
 Keywords:
 Date-Required:
 Class:  sw-bug
 Submitter-Id:   current-users
 Arrival-Date:   Sun Jul 21 19:10:00 UTC 2013
 Closed-Date:
 Last-Modified:
 Originator: Alexandre Kovalenko
 Release:9-STABLE
 Organization:
 Environment:
 FreeBSD twinhead 9.2-BETA1 FreeBSD 9.2-BETA1 #11 r248085:253503: Sun Jul 21 
 14:29:42 EDT 2013 root@twinhead:/usr/obj/usr/src/sys/TWINHEAD  amd64
 Description:
 
  Three different external hard drives (Seagate, Western Digital and noname 
 USB 3.0 enclosure) refused to be recognized as the umass devices. Reverting 
 /usr/src/sys/dev/bsd/controller to r248085, building and loading just xhci 
 module makes drives appear again. Below are snippets from the log in both 
 cases:
 
 
 Can you try:
 
 1) Attached patch.
 
 2) Non ASMedia USB 3.0 controller
 
 --HPS
   
 xhci.diff

1) I will try to apply this patch on top of 253503. Let me know if I need to 
revert to the earlier rev -- that would take a little bit more time.

2) ASMedia is not a USB 3.0 controller -- it is one of three umass devices I 
was doing testing with (USB-to-SATA enclosure). The other two devices are made 
by Seagate and Western Digital. The USB controller is

xhci0@pci0:2:0:0:   class=0x0c0330 card=0x chip=0x00141912 rev=0x03 
hdr=0x00
vendor = 'Renesas Technology Corp.'
class  = serial bus
subclass   = USB

I can try to find different USB controller, but that would take some time.


Alexandre Sunny Kovalenko (Олександр Коваленко)

___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org

Re: usb/180726: XHCI umass support breaks between r248085 and r252560 on 9-STABLE

2013-07-21 Thread Hans Petter Selasky

On 07/21/13 21:41, Alexandre Kovalenko wrote:

1) I will try to apply this patch on top of 253503. Let me know if I need to 
revert to the earlier rev -- that would take a little bit more time.


You don't need to revert. I'm on #bsdusb on EF-net.

--HPS
___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org


Re: usb/180726: XHCI umass support breaks between r248085 and r252560 on 9-STABLE

2013-07-21 Thread Alexandre Kovalenko

On Jul 21, 2013, at 3:32 PM, Hans Petter Selasky h...@bitfrost.no wrote:

 On 07/21/13 21:06, Alexandre Kovalenko wrote:
 
 Number: 180726
 Category:   usb
 Synopsis:   XHCI umass support breaks between r248085 and r252560 on 
 9-STABLE
 Confidential:   no
 Severity:   non-critical
 Priority:   low
 Responsible:freebsd-usb
 State:  open
 Quarter:
 Keywords:
 Date-Required:
 Class:  sw-bug
 Submitter-Id:   current-users
 Arrival-Date:   Sun Jul 21 19:10:00 UTC 2013
 Closed-Date:
 Last-Modified:
 Originator: Alexandre Kovalenko
 Release:9-STABLE
 Organization:
 Environment:
 FreeBSD twinhead 9.2-BETA1 FreeBSD 9.2-BETA1 #11 r248085:253503: Sun Jul 21 
 14:29:42 EDT 2013 root@twinhead:/usr/obj/usr/src/sys/TWINHEAD  amd64
 Description:
 
  Three different external hard drives (Seagate, Western Digital and noname 
 USB 3.0 enclosure) refused to be recognized as the umass devices. Reverting 
 /usr/src/sys/dev/bsd/controller to r248085, building and loading just xhci 
 module makes drives appear again. Below are snippets from the log in both 
 cases:
 
 
 Can you try:
 
 1) Attached patch.
 
 2) Non ASMedia USB 3.0 controller
 
 --HPS
   
 xhci.diff

After applying this patch 
(http://svnweb.freebsd.org/base/head/sys/dev/usb/controller/xhci.c?view=patchr1=253532r2=253531pathrev=253532)
 on top of r253503 the life is good -- devices are recognized properly.

Thank you very much for your help.

Alexandre Sunny Kovalenko (Олександр Коваленко)


___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org

Re: usb/180726: XHCI umass support breaks between r248085 and r252560 on 9-STABLE

2013-07-21 Thread Alexandre Kovalenko
The following reply was made to PR usb/180726; it has been noted by GNATS.

From: Alexandre Kovalenko bsd.gai...@gmail.com
To: bug-follo...@freebsd.org,
 bsd.gai...@gmail.com
Cc:  
Subject: Re: usb/180726: XHCI umass support breaks between r248085 and r252560 
on 9-STABLE
Date: Sun, 21 Jul 2013 17:47:06 -0400

 --Apple-Mail=_6F6E3510-295E-4026-89ED-4A3E44C88E2A
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/plain;
charset=utf-8
 
 The patch =
 (http://svnweb.freebsd.org/base/head/sys/dev/usb/controller/xhci.c?view=3D=
 patchr1=3D253532r2=3D253531pathrev=3D253532) fixes the issue for me, =
 so PR could be closed.
 
 Is it possible to include it into 9.2?
 
 Alexandre Sunny Kovalenko (=D0=9E=D0=BB=D0=B5=D0=BA=D1=81=D0=B0=D0=BD=D0=
 =B4=D1=80 =D0=9A=D0=BE=D0=B2=D0=B0=D0=BB=D0=B5=D0=BD=D0=BA=D0=BE)
 
 
 
 
 
 --Apple-Mail=_6F6E3510-295E-4026-89ED-4A3E44C88E2A
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/html;
charset=utf-8
 
 htmlheadmeta http-equiv=3DContent-Type content=3Dtext/html =
 charset=3Dutf-8/headbody style=3Dword-wrap: break-word; =
 -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; The =
 patch (a =
 href=3Dhttp://svnweb.freebsd.org/base/head/sys/dev/usb/controller/xhci.c?=
 view=3Dpatchamp;r1=3D253532amp;r2=3D253531amp;pathrev=3D253532http://=
 svnweb.freebsd.org/base/head/sys/dev/usb/controller/xhci.c?view=3Dpatcham=
 p;r1=3D253532amp;r2=3D253531amp;pathrev=3D253532/a) fixes the issue =
 for me, so PR could be closed.divbr/divdivIs it possible to =
 include it into 9.2?/divdivbr/divdiv
 span class=3DApple-style-span style=3Dborder-collapse: separate; =
 color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; =
 font-variant: normal; font-weight: normal; letter-spacing: normal; =
 line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: =
 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: =
 0px; -webkit-border-horizontal-spacing: 0px; =
 -webkit-border-vertical-spacing: 0px; =
 -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: =
 auto; -webkit-text-stroke-width: 0px; font-size: medium; span =
 class=3DApple-style-span style=3Dborder-collapse: separate; color: =
 rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: =
 normal; font-weight: normal; letter-spacing: normal; line-height: =
 normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; =
 text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; =
 -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: =
 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: =
 auto; -webkit-text-stroke-width: 0px; font-size: medium; div =
 style=3Dword-wrap: break-word; -webkit-nbsp-mode: space; =
 -webkit-line-break: after-white-space; divAlexandre Sunny =
 Kovalenko (=D0=9E=D0=BB=D0=B5=D0=BA=D1=81=D0=B0=D0=BD=D0=B4=D1=80 =
 =D0=9A=D0=BE=D0=B2=D0=B0=D0=BB=D0=B5=D0=BD=D0=BA=D0=BE)/divdivbr/di=
 v/div/spanbr class=3DApple-interchange-newline/spanbr =
 class=3DApple-interchange-newline
 /div
 br/body/html=
 
 --Apple-Mail=_6F6E3510-295E-4026-89ED-4A3E44C88E2A--
___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org