Am Dienstag, 24. August 2004 22:41 schrieb Pete Zaitcev:
> On Tue, 24 Aug 2004 21:10:38 +0200
> Oliver Neukum <[EMAIL PROTECTED]> wrote:
> 
> > Am Dienstag, 24. August 2004 20:33 schrieb Pete Zaitcev:
> > > +�������sc->work_timer.expires = jiffies + UB_URB_TIMEOUT;
> > > +�������add_timer(&sc->work_timer);
> > > +
> > > ��������if ((rc = usb_submit_urb(&sc->work_urb, GFP_ATOMIC)) != 0) {
> > 
> > You're doing it the wrong way round.
> > You can not know how long after the timer is added the URB will
> > really go on the wire. You need the add the timer after you've
> > submitted the URB.
> 
> Not a bad idea, although the reasoning is obviously faulty,
> as there's no difference in behaviour (see that GFP_ATOMIC?)

You can still take a lengthy irq or system management call in between.
Depending on this never be the case is very dangerous.

        Regards
                Oliver


-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to