Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-19 Thread Greg Kroah-Hartman
On Sun, Aug 19, 2012 at 12:23:38PM +0200, Bjørn Mork wrote: > Ming Lei writes: > > On Fri, Aug 17, 2012 at 10:42 PM, Greg Kroah-Hartman > > wrote: > >> On Fri, Aug 17, 2012 at 10:38:16AM -0400, Alan Stern wrote: > >>> On Fri, 17 Aug 2012, Ming Lei wrote: > >>> > >>> > But, if HOTPLUG is not enab

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-19 Thread Bjørn Mork
Ming Lei writes: > On Fri, Aug 17, 2012 at 10:42 PM, Greg Kroah-Hartman > wrote: >> On Fri, Aug 17, 2012 at 10:38:16AM -0400, Alan Stern wrote: >>> On Fri, 17 Aug 2012, Ming Lei wrote: >>> >>> > But, if HOTPLUG is not enabled, should device_add() trigger driver probe >>> > further after kernel i

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Ming Lei
On Fri, Aug 17, 2012 at 10:42 PM, Greg Kroah-Hartman wrote: > On Fri, Aug 17, 2012 at 10:38:16AM -0400, Alan Stern wrote: >> On Fri, 17 Aug 2012, Ming Lei wrote: >> >> > But, if HOTPLUG is not enabled, should device_add() trigger driver probe >> > further after kernel init is completed? Or even

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Greg Kroah-Hartman
On Fri, Aug 17, 2012 at 10:38:16AM -0400, Alan Stern wrote: > On Fri, 17 Aug 2012, Ming Lei wrote: > > > But, if HOTPLUG is not enabled, should device_add() trigger driver probe > > further after kernel init is completed? Or even devices should be allowed > > to add into system? > > Indeed, doe

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Alan Stern
On Fri, 17 Aug 2012, Ming Lei wrote: > But, if HOTPLUG is not enabled, should device_add() trigger driver probe > further after kernel init is completed? Or even devices should be allowed > to add into system? Indeed, does it make any sense to have USB support at all if HOTPLUG isn't enabled?

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Greg Kroah-Hartman
On Fri, Aug 17, 2012 at 08:16:15PM +0800, Ming Lei wrote: > On Fri, Aug 17, 2012 at 6:24 PM, Fengguang Wu wrote: > > On Fri, Aug 17, 2012 at 05:27:08PM +0800, Fengguang Wu wrote: > >> Hi Bjørn, > >> > >> > -static const struct usb_device_id id_table[] __devinitconst = { > >> > +static const struct

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Ming Lei
On Fri, Aug 17, 2012 at 6:24 PM, Fengguang Wu wrote: > On Fri, Aug 17, 2012 at 05:27:08PM +0800, Fengguang Wu wrote: >> Hi Bjørn, >> >> > -static const struct usb_device_id id_table[] __devinitconst = { >> > +static const struct usb_device_id id_table[] = { >> >> Good catch! It magically fixed the

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Fengguang Wu
On Fri, Aug 17, 2012 at 05:27:08PM +0800, Fengguang Wu wrote: > Hi Bjørn, > > > -static const struct usb_device_id id_table[] __devinitconst = { > > +static const struct usb_device_id id_table[] = { > > Good catch! It magically fixed the oops. So that id_table was > freed sometime with __devinitc

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Fengguang Wu
Hi Bjørn, > -static const struct usb_device_id id_table[] __devinitconst = { > +static const struct usb_device_id id_table[] = { Good catch! It magically fixed the oops. So that id_table was freed sometime with __devinitconst? There are some more "usb_device_id .* __devinitconst" users in the sy

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Bjørn Mork
Bjørn Mork writes: > Fengguang Wu writes: > >> @@ -779,6 +780,7 @@ static int usb_device_match(struct devic >> intf = to_usb_interface(dev); >> usb_drv = to_usb_driver(drv); >> >> + pr_info("%s: device %s, driver %s \n", __func__, >> dev_name(dev), drv->

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-17 Thread Bjørn Mork
Fengguang Wu writes: > @@ -779,6 +780,7 @@ static int usb_device_match(struct devic > intf = to_usb_interface(dev); > usb_drv = to_usb_driver(drv); > > + pr_info("%s: device %s, driver %s \n", __func__, > dev_name(dev), drv->name); > id =

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-16 Thread Greg Kroah-Hartman
On Fri, Aug 17, 2012 at 10:00:46AM +0800, Fengguang Wu wrote: > On Sun, Aug 05, 2012 at 09:58:26AM -0700, Greg KH wrote: > > On Sun, Aug 05, 2012 at 10:59:38AM +0800, Fengguang Wu wrote: > > > Hi all, > > > > > > This line triggers an oops in kvm boot test: > > > > > > usb_match_id(): > > > ==>

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-16 Thread Fengguang Wu
On Sun, Aug 05, 2012 at 09:58:26AM -0700, Greg KH wrote: > On Sun, Aug 05, 2012 at 10:59:38AM +0800, Fengguang Wu wrote: > > Hi all, > > > > This line triggers an oops in kvm boot test: > > > > usb_match_id(): > > ==> 748 for (; id->idVendor || id->idProduct || > > id->bDeviceClass

Re: BUG: unable to handle kernel paging request in usb_match_id()

2012-08-05 Thread Greg Kroah-Hartman
On Sun, Aug 05, 2012 at 10:59:38AM +0800, Fengguang Wu wrote: > Hi all, > > This line triggers an oops in kvm boot test: > > usb_match_id(): > ==> 748 for (; id->idVendor || id->idProduct || id->bDeviceClass > || > 749id->bInterfaceClass || id->driver_info;