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

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



On Mon, 11 Apr 2011, Timo Juhani Lindfors wrote:

> maximilian attems <max@stro.at> 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 but at least on squeeze I
> don't even have the modprobe applet:

I checked busybox source, it does support the verbose -v switch.
so changing your function in the patch, looks good.

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
 
-- 
maks



Reply to: