Messages in this thread | ![/](/images/icornerl.gif) | | Date | Tue, 27 Sep 2005 05:22:18 -0700 | From | Greg KH <> | Subject | Re: [vendor-sec] Re: [BUG/PATCH/RFC] Oops while completing async USB via usbdevio |
| |
On Tue, Sep 27, 2005 at 01:03:19PM +0200, Martin Pitt wrote: > Hi! > > Greg KH [2005-09-27 2:13 -0700]: > > On Tue, Sep 27, 2005 at 01:04:15AM -0700, Greg KH wrote: > > > First off, thanks for providing a patch for this problem, it is real, > > > and has been known for a while (thanks to your debugging :) > > > > > > On Sun, Sep 25, 2005 at 05:13:30PM +0200, Harald Welte wrote: > > > > > > > > I suggest this (or any other) fix to be applied to both 2.6.14 final and > > > > the stable series. I didn't yet investigate 2.4.x, but I think it is > > > > likely to have the same problem. > > > > > > I agree, but I think we need an EXPORT_SYMBOL_GPL() for your newly > > > exported symbol, otherwise the kernel will not build if you have USB > > > built as a module. > > > > Hm, it's even messier. With your patch, we get: > > *** Warning: "__send_sig_info" [drivers/usb/core/usbcore.ko] undefined! > > *** Warning: "__put_task_struct" [drivers/usb/core/usbcore.ko] undefined! > > when the USB core is a module. > > > > I can't pass judgement if we want to export both of these functions to > > modules... Anyone else know? > > FWIW, our kernel maintainer just added > > EXPORT_SYMBOL_GPL(__send_sig_info); > > and it worked fine (we modularize as much as possible).
Yes, that would work, that's not an issue. The issue is if this is the best solution or not (generally exporting functions that start with "__" is not a good idea...)
thanks,
greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |