Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756249AbXJ3PXd (ORCPT ); Tue, 30 Oct 2007 11:23:33 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753399AbXJ3PXZ (ORCPT ); Tue, 30 Oct 2007 11:23:25 -0400 Received: from ns.suse.de ([195.135.220.2]:42038 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753395AbXJ3PXY (ORCPT ); Tue, 30 Oct 2007 11:23:24 -0400 Date: Tue, 30 Oct 2007 14:12:11 +0100 Message-ID: From: Takashi Iwai To: Kamalesh Babulal Cc: Greg KH , alsa-devel@alsa-project.org, Amit Shah , linux-kernel@vger.kernel.org Subject: Re: [alsa-devel] sysfs: WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() - with ALSA In-Reply-To: References: <877aabc40710260014v220ab804o5de5dbc36b870bd1@mail.gmail.com> <20071026091722.GA21274@linux.vnet.ibm.com> <20071026155202.GD23016@kroah.com> <472220BA.3060201@linux.vnet.ibm.com> User-Agent: Wanderlust/2.15.5 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 MULE XEmacs/21.5 (beta28) (fuki) (+CVS-20070806) (i386-suse-linux) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4314 Lines: 133 At Mon, 29 Oct 2007 08:30:50 +0100, I wrote: > > At Fri, 26 Oct 2007 22:45:38 +0530, > Kamalesh Babulal wrote: > > > > Takashi Iwai wrote: > > > At Fri, 26 Oct 2007 08:52:02 -0700, > > > Greg KH wrote: > > >> On Fri, Oct 26, 2007 at 02:47:22PM +0530, Kamalesh Babulal wrote: > > >>> I get similar warning, while loading the alsa driver > > >> This is a different problem, not a rename issue like the network one > > >> was. > > >> > > >>> [ 191.933548] Advanced Linux Sound Architecture Driver Version 1.0.15 > > >>> (Tue Oct 23 06:09:18 2007 UTC). > > >>> [ 191.996323] ALSA sound/core/seq/oss/seq_oss.c:232: can't register > > >>> device seq > > >>> [ 192.046004] sysfs: duplicate filename 'audio' can not be created > > >>> [ 192.086048] WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() > > >> This is an error in the alsa core, perhaps the alsa developers can help > > >> figure out why they are registering the same file twice? > > > > > > ALSA core shouldn't do so. Maybe a built-in OSS driver conflict with > > > ALSA? We'd need to check .config, then. > > > > > > > > > Takashi > > > - > > Hi Takashi, > > > > Attaching the .config file, > > You're using allyesconfig, and it's a missing runtime configuration to > avoid the duplication. Some ALSA and OSS drivers can be loaded > without real hardwares. With allyesconfig, ALSA and OSS drivers may > be registered to the same device unless explicitly set up. Meanwhile, it's not sexy to have such warnings from sysfs core, not from the caller side. The (untested) patch below may fix the problem. Now the standard register_sound_*() is used for old OSS code. Please give it a try. Takashi [PATCH] Use register_sound_*() in OSS core code Use the standard register_sound_*() functions instead of calling device_create() directly in the old OSS core code. Signed-off-by: Takashi Iwai --- diff --git a/sound/oss/soundcard.c b/sound/oss/soundcard.c index a9c23b2..2c20e70 100644 --- a/sound/oss/soundcard.c +++ b/sound/oss/soundcard.c @@ -523,15 +523,11 @@ bad: */ static const struct { unsigned short minor; - char *name; - umode_t mode; int *num; } dev_list[] = { /* list of minor devices */ /* seems to be some confusion here -- this device is not in the device list */ - {SND_DEV_DSP16, "dspW", S_IWUGO | S_IRUSR | S_IRGRP, - &num_audiodevs}, - {SND_DEV_AUDIO, "audio", S_IWUGO | S_IRUSR | S_IRGRP, - &num_audiodevs}, + {SND_DEV_DSP16, &num_audiodevs}, + {SND_DEV_AUDIO, &num_audiodevs}, }; static int dmabuf; @@ -560,17 +556,11 @@ static int __init oss_init(void) sound_dmap_flag = (dmabuf > 0 ? 1 : 0); for (i = 0; i < ARRAY_SIZE(dev_list); i++) { - device_create(sound_class, NULL, - MKDEV(SOUND_MAJOR, dev_list[i].minor), - "%s", dev_list[i].name); - if (!dev_list[i].num) continue; - - for (j = 1; j < *dev_list[i].num; j++) - device_create(sound_class, NULL, - MKDEV(SOUND_MAJOR, dev_list[i].minor + (j*0x10)), - "%s%d", dev_list[i].name, j); + for (j = 0; j < *dev_list[i].num; j++) + register_sound_special(&oss_sound_fops, + dev_list[i].minor + (j * 0x10)); } if (sound_nblocks >= 1024) @@ -584,11 +574,10 @@ static void __exit oss_cleanup(void) int i, j; for (i = 0; i < ARRAY_SIZE(dev_list); i++) { - device_destroy(sound_class, MKDEV(SOUND_MAJOR, dev_list[i].minor)); if (!dev_list[i].num) continue; - for (j = 1; j < *dev_list[i].num; j++) - device_destroy(sound_class, MKDEV(SOUND_MAJOR, dev_list[i].minor + (j*0x10))); + for (j = 0; j < *dev_list[i].num; j++) + unregister_sound_special(dev_list[i].minor + (j * 0x10)); } unregister_sound_special(1); diff --git a/sound/sound_core.c b/sound/sound_core.c index 46daca1..3a22848 100644 --- a/sound/sound_core.c +++ b/sound/sound_core.c @@ -259,6 +259,9 @@ int register_sound_special_device(const struct file_operations *fops, int unit, case 4: name = "audio"; break; + case 5: + name = "dspW"; + break; case 8: name = "sequencer2"; if (unit >= SOUND_STEP) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/