Re: [vendor-sec] Re: [BUG/PATCH/RFC] Oops while completing async USB via usbdevio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]
  Powered by Linux