On Tue, 9 Jan 2007, Phil Endecott wrote:
> Gadgetfs had a mode in which endpoint descriptors were written by
> the user program before connection. This mode had some bugs, and
> may never have been used. This patch removes that mode. The only
> mode of operation now is for the user program to wait for endpoint
> 0 to report a SET_CONFIGURATION, and then write the endpoint
> descriptors.
>
> With this change, testusb works equally well as before for me: test
> 10 fails. I am now wondering if this is a host problem, and would
> appreciate feedback about whether testusb works for other people
> after applying this patch.
Phil:
I haven't tried testing your patch. But I have tried using the existing
gadgetfs with net2280 and uhci-hcd. There was no trouble running test #10
with the usb program or with g_zero instead of gadgetfs. I can't imagine
what's going wrong on your system.
On the other hand, I did find (another) bug in the async I/O handling in
inode.c. If you aren't using the AIO code in usb.c then it shouldn't
affect you, but you might want to give it a try anyway.
Alan Stern
Index: usb-2.6/drivers/usb/gadget/inode.c
===================================================================
--- usb-2.6.orig/drivers/usb/gadget/inode.c
+++ usb-2.6/drivers/usb/gadget/inode.c
@@ -586,7 +586,6 @@ static ssize_t ep_aio_read_retry(struct
}
kfree(priv->buf);
kfree(priv);
- aio_put_req(iocb);
return len;
}
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
