Bug#615831: [PATCH 1/2] panic(): print name of kernel module before loading it

2011-05-31 Thread Timo Juhani Lindfors
Hi, maximilian attems writes: >> cat: /sys/class/input/input*/device/description: No such file or directory > > yep that is the case for usb keyboard, so you can check no existence of that. aha, good idea. I'll send new patches against git HEAD. I built these with git-buildpackage and tested th

Bug#615831: [PATCH 1/2] panic(): print name of kernel module before loading it

2011-05-31 Thread maximilian attems
On Tue, May 31, 2011 at 10:16:31AM +0300, Timo Juhani Lindfors wrote: > maximilian attems writes: > > now the only thing that is holding me up to merge your patch is that > > /proc/bus/input/devices usage, there should be some sysfs file, no? > > > > > > cat /sys/class/input/input*/device/descript

Bug#615831: [PATCH 1/2] panic(): print name of kernel module before loading it

2011-05-31 Thread Timo Juhani Lindfors
maximilian attems writes: > now the only thing that is holding me up to merge your patch is that > /proc/bus/input/devices usage, there should be some sysfs file, no? > > > cat /sys/class/input/input*/device/description shows a KBD Unfortunately it does not show a keyboard here: root@metunconf:~

Bug#615831: [PATCH 1/2] panic(): print name of kernel module before loading it

2011-05-16 Thread maximilian attems
On Mon, 11 Apr 2011, Timo Juhani Lindfors wrote: > maximilian attems writes: > >> +try_modprobe() > > I'd prefer the function to be named modprobe_verbose() > > Ok. > > >> + try_modprobe atkbd > > why not using "-v" from modprobe, > > the only argument against that would count is busybox suppo

Re: [PATCH 1/2] panic(): print name of kernel module before loading it

2011-04-11 Thread Timo Juhani Lindfors
maximilian attems writes: >> +try_modprobe() > I'd prefer the function to be named modprobe_verbose() Ok. >> +try_modprobe atkbd > why not using "-v" from modprobe, > the only argument against that would count is busybox support. > please check? I'm not very familiar with busybox internals

Re: [PATCH 1/2] panic(): print name of kernel module before loading it

2011-03-27 Thread maximilian attems
On Wed, 02 Mar 2011, Timo Juhani Lindfors wrote: > Loading a kernel can crash the system so it is useful that the user > sees which module might be causing trouble. ack on the statement. > Signed-off-by: Timo Juhani Lindfors > > diff --git a/scripts/functions b/scripts/functions > index 57969

[PATCH 1/2] panic(): print name of kernel module before loading it

2011-03-02 Thread Timo Juhani Lindfors
Loading a kernel can crash the system so it is useful that the user sees which module might be causing trouble. Signed-off-by: Timo Juhani Lindfors diff --git a/scripts/functions b/scripts/functions index 579696a..af49e45 100644 --- a/scripts/functions +++ b/scripts/functions @@ -31,6 +31,12 @@