> > > There are some other messages that get printed such as:
> > > 
> > > hcd.c: usb_hcd_pci_remove 00:10.0, count != 1
> > > 
> > > but it appears from the code that this is just a soft warning that all
> > > of the references to the bus aren't freed yet. This isn't a bug.
> > 
> > It's an err() ... Minimally, you've got a memory leak.  Worst case,
> > you've got some device driver thinking it can call into a device on
> > that bus ... oops!
> 
> Oh boy, not another reference counting discussion. Look at the code.

Says JE, to someone who clearly _has_  done quite a bit more
than just look at it.   Sigh.  It'd really be useful if you were less
antagonistic in your responses, since that's a guaranteed way
to get discussions off on the wrong foot.  (Which I certainly hope
is not your goal.)


> rmmod'ing uhci-hcd is still safe, FWIW. And that's not necessarily a
> memory leak.

I can imagine some ways it'd not be a memory leak, but that'd
mean something was lying in its use counts.

Under what circumstances would a live reference to that bus still
be around?  Every module touching it has affirmed that it's done
with the bus, and with every device on it.  And none of them are
going to get any more chances to drop those refcounts; any one
of those modules could get rmmodded right away.


> I won't guarantee that there are no leaks since the code hasn't received
> extensive testing yet, but I think you're misunderstanding this
> particular scenario.

You really like those two comments:  "look at the code" (implying
all kinds of nasties, starting with my not having done that), and
"you're misunderstanding this" (going further than that).

Trying to belittle me is an unwarrantedly antagonistic response
to a constructive comment.  Although I can tell you've gotten
into the habit of doing it ... that'd be a fine habit to break, and I
know I'd not be the only one to appreciate that.

- Dave



_______________________________________________________________

Have big pipes? SourceForge.net is looking for download mirrors. We supply
the hardware. You get the recognition. Email Us: [EMAIL PROTECTED]
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to