Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-07-02 Thread Soewono Effendi
On Mon, 1 Jul 2002 23:45:34 -0700 "Greg KH" <[EMAIL PROTECTED]> wrote: > On Fri, Jun 28, 2002 at 04:47:57PM +0200, Soewono Effendi wrote: > > > > And I think there must be some locking to protect > > if (driver->owner) > > __MOD_INC_USE_COUNT(driver->owner); > > and > >

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-07-02 Thread Oliver Neukum
> > The goal is just to minimize the kernel locking time as much as > > possible, with the cost of size, I admit that. > > Why do you think that the BKL or any lock is needed around the test and > set functions? > > I agree module unloading is extreemly racy, but take a look at the > current thre

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-07-01 Thread Greg KH
On Fri, Jun 28, 2002 at 04:47:57PM +0200, Soewono Effendi wrote: > > And I think there must be some locking to protect > if (driver->owner) > __MOD_INC_USE_COUNT(driver->owner); > and > if (driver->owner) > __MOD_DEC_USE_COUNT(driver->owner); > > may be: >

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-07-01 Thread Greg KH
On Thu, Jun 27, 2002 at 09:21:02PM +0200, Oliver Neukum wrote: > > PS: Is it true that the IBM guys have a keyboard they make > a notch into whenever they reduce BKL usage ? Heh, yes, I think there is something like that in the main BKL squasher's cubical. I know I enjoy bugging him about it by

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-28 Thread Soewono Effendi
About the BKL stuffs... As I can see on linux-2.5.22 no BKL is needed. The idea is just to use __MOD_INC_USE_COUNT and __MOD_DEC_USE_COUNT, though I have the feeling that there should be at least a minimum locking mechanism to guide safe "driver->owner". Is this idea also applicable on 2.4 or

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-28 Thread Oliver Neukum
Am Donnerstag, 27. Juni 2002 23:41 schrieb David Brownell: > Oliver Neukum wrote: > > Am Donnerstag, 27. Juni 2002 21:33 schrieben Sie: > >>OK, I'll let you fix that BKL stuff (didn't you add it > >>in the first place? :) but this looks like the locking > > > > I did not add it. It was there for a

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread David Brownell
OK, I'll let you fix that BKL stuff (didn't you add it in the first place? :) but this looks like the locking is being done at the wrong level: > @@ -1349,7 +1351,9 @@ > usbfs_add_device(dev); > > /* find drivers willing to handle this device */ > + lock_kernel(); /* guard agai

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread Oliver Neukum
Am Donnerstag, 27. Juni 2002 20:24 schrieb David Brownell: > >>>Hotpluggings are rare events ... Besides they take BKL and thus > >>>couldn't run in concurrency. > >> > >>What was the reason they have BKL? In principle it shouldn't > >>be needed, since all the relevant USB data structures are lo

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread David Brownell
>>>Hotpluggings are rare events ... Besides they take BKL and thus >>>couldn't run in concurrency. >> >>What was the reason they have BKL? In principle it shouldn't >>be needed, since all the relevant USB data structures are locked >>correctly, and there's no comment in the code explaining why >

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread Oliver Neukum
Am Donnerstag, 27. Juni 2002 18:20 schrieb David Brownell: > Oliver Neukum wrote: > >>after looking around in source codes and searching in mailing list and > >>G*gle, I still couldn't find any hints why there exists only one > >> "khubd" in USB Stack. > > > > Hotpluggings are rare events ... Bes

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread Soewono Effendi
On Thu, 27 Jun 2002 09:15:25 -0700 "David Brownell" <[EMAIL PROTECTED]> wrote: > Periodic urbs get some special treatment, "automagic resubmission" where urbs > don't get handed back to drivers in most completion callbacks. Ownership > of the URB there is in a strange "shared between drivers" st

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread David Brownell
Oliver Neukum wrote: >>after looking around in source codes and searching in mailing list and >>G*gle, I still couldn't find any hints why there exists only one "khubd" >>in USB Stack. > > Hotpluggings are rare events ... Besides they take BKL and thus > couldn't run in concurrency. What was t

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread David Brownell
> Well, actually I wanted to know how exactly URB flows through the Linux USB Stack, >e.g. > driver allocates URB, submitted to USB Core, goes to HCD, something done there, > returned to to driver by callback function, etc. > And how does it apply for all the different transfer modes. That's th

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread Soewono Effendi
On Thu, 27 Jun 2002 10:38:37 -0400 "Johannes Erdfelt" <[EMAIL PROTECTED]> wrote: > Simplicity. [snip] I see. > > > Is there any informations about "URB lifetime" ? who allocates and deallocates it, > > when one should/shouldn't remove it, etc. > > The driver allocates and deallocates the URB

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread Oliver Neukum
Am Donnerstag, 27. Juni 2002 14:49 schrieb Soewono Effendi: > Hello there, > > after looking around in source codes and searching in mailing list and > G*gle, I still couldn't find any hints why there exists only one "khubd" > in USB Stack. Isn't it "better (performance?)" to have one "khubd" for

Re: [linux-usb-devel] Need helps to understand USB Stack under Linux

2002-06-27 Thread Johannes Erdfelt
On Thu, Jun 27, 2002, Soewono Effendi <[EMAIL PROTECTED]> wrote: > after looking around in source codes and searching in mailing list and G*gle, > I still couldn't find any hints why there exists only one "khubd" in USB Stack. > Isn't it "better (performance?)" to have one "khubd" for each HC? (t