From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761541AbXJXXwE (ORCPT ); Wed, 24 Oct 2007 19:52:04 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1761436AbXJXXvd (ORCPT ); Wed, 24 Oct 2007 19:51:33 -0400 Received: from pentafluge.infradead.org ([213.146.154.40]:44405 "EHLO pentafluge.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760647AbXJXXvc (ORCPT ); Wed, 24 Oct 2007 19:51:32 -0400 Date: Wed, 24 Oct 2007 16:43:48 -0700 From: Greg KH To: Kay Sievers Cc: David Miller , jens.axboe@oracle.com, rjw@sisk.pl, torvalds@linux-foundation.org, akpm@linux-foundation.org, htejun@gmail.com, linux-kernel@vger.kernel.org Subject: Re: linux-2.6.23-git3: Many sysfs-related warnings in dmesg Message-ID: <20071024234348.GA4909@kroah.com> References: <20071016204238.GA1093@kroah.com> <20071016205053.GI9942@kernel.dk> <20071016213730.GB21815@kroah.com> <20071016.153248.95506316.davem@davemloft.net> <20071016232350.GB24827@kroah.com> <1192616214.4300.16.camel@lov.site> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1192616214.4300.16.camel@lov.site> User-Agent: Mutt/1.5.16 (2007-06-09) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 17, 2007 at 12:16:54PM +0200, Kay Sievers wrote: > On Tue, 2007-10-16 at 16:23 -0700, Greg KH wrote: > > On Tue, Oct 16, 2007 at 03:32:48PM -0700, David Miller wrote: > > > From: Greg KH > > > Date: Tue, 16 Oct 2007 14:37:30 -0700 > > > > > > > Kay, are we doing something wrong in userspace when renaming wireless > > > > devices such that we can overlap names? > > Not udev, but SUSE 10.2's network renaming. It uses udev and calls > ifrename in the same code path. 10.3 uses the unified version from the > udev tree. > > > > It does it for all network devices, I see this ugly message on every > > > single system I have from Fedora foo to RHEL foo to ubuntu foo to > > > debian foo. > > > > > > udev simply applies the MAC address to device name rules blindly, it > > > doesn't check if the device already has the desired name already > > There is a check for the same name in udev for long. > > > Ugh :( > > > > > It's been like this forever, and since userland has been doing it for > > > so long, you can't warn on this there is too much established > > > practice. Expecting people to install "fixed" udev is not an > > > acceptable answer, the warning is a regression and therefore you'll > > > have to remove the kernel warning for this case and live with this > > > issue essentially forever. > > We should probably just add the check to kobject_rename() and print a > simple warning and then do nothing. Or just do the check in the network > ioctl, if we really don't want to see this. I agree that perhaps kobject_rename() should check for this. Let me go see if I can get that to work... thanks, greg k-h