Re: [PATCH] USB: introduce urb->core_status field

Alan Stern <[email protected]>
Newsgroups gmane.linux.usb.devel
Message-ID <[email protected]>
On Fri, 8 Jun 2007, Greg KH wrote:

> On Fri, Jun 08, 2007 at 05:12:05PM -0400, Alan Stern wrote:
> > This patch (as926) replaces urb->status with urb->core_status, in an
> > attempt to help prevent drivers from abusing the API by polling
> > urb->status.  The original field remains for use by drivers, but it
> > isn't set until an URB is about to be handed to its completion
> > routine.
> 
> Hm, is there any way to keep this from being in the urb entirely?
> Shouldn't this be an internal-to-the hcd type thing?

Yes, it could be done that way, I think.  It'll take more work than a 
simple search-and-replace.  The trickiest part is how to unlink an URB 
after it has been submitted but before it has been handed to the HCD...

> I say this as I worry people will just start using core_status instead
> of status when we remove the status field.

Maybe.  It _is_ documented as being private to usbcore -- but that 
might not stop people!

Alan Stern


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.