Am Mittwoch, 25. Februar 2009 schrieb Greg KH: > On Mon, Feb 23, 2009 at 07:29:20PM +0100, Karsten Wiese wrote: > > > > ehci-hcd uses usb_get_urb() and usb_put_urb() in an unbalanced way causing > > isochronous URB's kref.counts incrementing once per usb_submit_urb() call. > > There is no need for ehci-hcd to deal with usb_get_urb() nor usb_put_urb() > > anyhow, so patch removes their usages in ehci-hcd. > > It also removes NULL assignments that have no effect. > > > > Signed-off-by: Karsten Wiese <fzu@xxxxxxxxxxxxxxxxxxxxx> > > --- > > > > Made against 2.6.29-rc6. Suitable to go there and stable ASAP, no? > > Are you sure this is still needed? It doesn't apply to my tree as I > think some other patch fixes this. Would that other patch be usb-ehci-slow-down-itd-reuse.patch? Then no. With it there is still the unbalancing urb = NULL; past ehci_urb_done(ehci, urb, 0); in itd_complete(). I guessed, usb-ehci-slow-down-itd-reuse.patch would go in for 2.6.30, so this doesn't fit ontop of usb-ehci-slow-down-itd-reuse.patch. If there is no other patch, I see 3 sequence possibilities: - usb-ehci-slow-down-itd-reuse.patch, then $SUBJECT - $SUBJECT, then usb-ehci-slow-down-itd-reuse.patch - both patches as one Which do you prefer? Thanks, Karsten -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
