Messages in this thread | | | Date | Wed, 4 Feb 2009 15:59:49 -0800 | From | Greg KH <> | Subject | Re: [PATCH 1/2] usb-serial : fix the nousb oops |
| |
On Wed, Feb 04, 2009 at 09:10:20AM +0800, Dave Young wrote: > remove wgreathouse@smva.com in cc-list because this mail will be > rejected by the smva.com email server. > > On Wed, Feb 4, 2009 at 7:58 AM, Greg KH <greg@kroah.com> wrote: > > On Mon, Feb 02, 2009 at 09:43:04AM +0800, Dave Young wrote: > >> On Sun, Feb 1, 2009 at 11:51 PM, Greg KH <greg@kroah.com> wrote: > >> > On Sun, Feb 01, 2009 at 06:53:23PM +0800, Dave Young wrote: > >> >> > >> >> In case with "nousb" booting, serial drivers will trigger kernel oops. > >> > > >> > They will? What is the oops message? > >> > >> Yes, they will, at lease I saw usb_serial/aircable/ftdi_sio oops. I > >> will manage to copy the oops message later. > >> > >> > > >> >> Here add usb_disabled() check in usb_serial_init and usb_serial_register > >> > > >> > This is odd, is it something new? Why is usb-serial the only driver > >> > that needs such a thing? > >> > >> No, it is not new thing. usb serial drivers normally call > >> usb_serial_register first, then usb_register, usb_register will fail > >> due to "nousb" checking, but usb_serial_register no. > > > > But as usb_register will fail, shouldn't they properly back out and no > > oopses happen? This sounds like a simple fix to the drivers that are > > failing here, not changing the usb-serial core. > > Hi greg > > I think all usb serial drivers which call usb_serial_register will > oops, so IMHO usb core should be fixed. > > I think maybe you missed my another reply, copy the content here again.
No, I got it.
I'm more interested in why usb-serial is oopsing, as it should't be.
> --- > hand copy part of the oops message:
Can you take a picture of the oops?
thanks,
greg k-h
| |