On Fri, 15 Aug 2008 13:27:00 +0200 Kay Sievers wrote:

quoted text
> On Thu, Aug 14, 2008 at 17:38, Randy Dunlap <randy.dunlap@oracle.com> wrote:
> > calling  param_sysfs_init+0x0/0x1e7
> > ------------[ cut here ]------------
> > WARNING: at fs/sysfs/dir.c:463 sysfs_add_one+0x35/0x3d()
> > sysfs: duplicate filename 'acpi' can not be created
> 
> Hi Randy,
> care to add a printk to the module sysfs setup, and post dmesg again?
> 
> --- a/kernel/params.c
> +++ b/kernel/params.c
> @@ -603,6 +603,8 @@ static void __init param_sysfs_builtin(void)
>  		}
>  		name_len = dot - kp->name;
> 
> +		printk("XXX adding modparam:'%s' %i (%p)\n", kp->name, i, kp);
> +
>   		/* new kbuild_modname? */
>  		if (strlen(modname) != name_len
>  		    || strncmp(modname, kp->name, name_len) != 0) {
> 
> I might show the order of registering the /sys/modules/ directory, and
> if there is possibly another "acpi" section, which tries to add
> parameter names.

Done.  Full boot log is attached.

Extract is:

XXX adding modparam:'acpi.power_nocheck' 34 (ffffffff806a4cf0)
XXX adding modparam:'processor.latency_factor' 35 (ffffffff806a4d18)
XXX adding modparam:'processor.nocst' 36 (ffffffff806a4d40)
XXX adding modparam:'processor.max_cstate' 37 (ffffffff806a4d68)
XXX adding modparam:'processor.ignore_ppc' 38 (ffffffff806a4d90)
XXX adding modparam:'thermal.psv' 39 (ffffffff806a4db8)
XXX adding modparam:'thermal.off' 40 (ffffffff806a4de0)
XXX adding modparam:'thermal.nocrt' 41 (ffffffff806a4e08)
XXX adding modparam:'thermal.tzp' 42 (ffffffff806a4e30)
XXX adding modparam:'thermal.crt' 43 (ffffffff806a4e58)
XXX adding modparam:'thermal.act' 44 (ffffffff806a4e80)
XXX adding modparam:'acpi.acpica_version' 45 (ffffffff806a4ea8)
XXX adding modparam:'keyboard.brl_nbchords' 46 (ffffffff806a4ed0)
------------[ cut here ]------------
WARNING: at fs/sysfs/dir.c:463 sysfs_add_one+0x35/0x3d()
sysfs: duplicate filename 'acpi' can not be created


---
~Randy
Linux Plumbers Conference, 17-19 September 2008, Portland, Oregon USA
http://linuxplumbersconf.org/