Re: g_mass_storage bug ?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 24, 2014 at 10:30:17AM -0500, Felipe Balbi wrote:
> On Wed, Sep 24, 2014 at 11:17:18AM -0400, Alan Stern wrote:
> > On Wed, 24 Sep 2014, Alan Stern wrote:
> > 
> > > > Case (6) is when Hi == Di, looking at my logs, I have:
> > > > 
> > > > 720 [  286.843965] SCSI CDB: 1a 00 3f 00 c0 00
> > > > 721 [  286.844000] g_mass_storage gadget: SCSI command: MODE SENSE(6); Dc=6, Di=192;  Hc=6, Hi=192
> > > > 722 [  286.844018] g_mass_storage gadget: bulk-in set halt
> > > > 723 [  286.844034] g_mass_storage gadget: sending command-failure status
> > > > 724 [  286.844045] g_mass_storage gadget:   sense data: SK x06, ASC x29, ASCQ x00;  info x0
> > > > 
> > > > Isn't it wrong to halt in this condition ?
> > > 
> > > No, it's correct.  SK = 6 and ASC = 0x29 means Unit Attention, Reset
> > > Occurred.  It occurs because this is the first command the gadget has
> > > received since starting up, which certainly is a form of reset.  In
> > > effect, this is how the device tells the host that it was just powered
> > > on.
> > 
> > Actually, looking at your log again, it seems more like this followed a 
> > genuine reset, not a power-on.  Regardless, it's still appropriate.
> 
> It seems to work better against my v3.16.1 desktop and Mac OS X then it
> does against v3.17-rc5 (running on the development board).
> 
> I'll try using a USB stick attached to the board.

alright, USB stick attached to the board behaves perfectly, but then
again I don't see any stalls from that device. Any chance I can persuade
you into running a similar test with your net2272/2280 UDC ? I'm using a
500MiB tmpfs as storage backend, then just connecting to an XHCI host
and looking at dmesg. A dd also triggers a few resets with v3.17-rc5
which don't happen on 3.16.1.

cheers

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux