Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754692AbYGGMcD (ORCPT ); Mon, 7 Jul 2008 08:32:03 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752312AbYGGMbu (ORCPT ); Mon, 7 Jul 2008 08:31:50 -0400 Received: from out01.mta.xmission.com ([166.70.13.231]:32986 "EHLO out01.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751714AbYGGMbt (ORCPT ); Mon, 7 Jul 2008 08:31:49 -0400 From: ebiederm@xmission.com (Eric W. Biederman) To: Cornelia Huck Cc: Greg Kroah-Hartman , Andrew Morton , Tejun Heo , Daniel Lezcano , linux-kernel@vger.kernel.org, Al Viro , Linux Containers , Benjamin Thery , References: <20080618170729.808539948@theryb.frec.bull.fr> <485F04E1.70204@gmail.com> <486706C9.9040303@gmail.com> <4869D314.5030403@gmail.com> <486A0751.9080602@gmail.com> <486AF4FA.8020805@gmail.com> <486B060C.7030607@gmail.com> <486C4515.1070007@gmail.com> <486CB051.5000507@fr.ibm.com> <486CF71F.5090405@gmail.com> <20080707134101.518734a9@gondolin.boeblingen.de.ibm.com> Date: Mon, 07 Jul 2008 05:22:43 -0700 In-Reply-To: <20080707134101.518734a9@gondolin.boeblingen.de.ibm.com> (Cornelia Huck's message of "Mon, 7 Jul 2008 13:41:01 +0200") Message-ID: User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SA-Exim-Connect-IP: 24.130.11.59 X-SA-Exim-Mail-From: ebiederm@xmission.com X-Spam-DCC: XMission; sa02 1397; Body=1 Fuz1=1 Fuz2=1 X-Spam-Combo: ;Cornelia Huck X-Spam-Relay-Country: X-Spam-Report: * -1.8 ALL_TRUSTED Passed through trusted hosts only via SMTP * 0.0 T_TM2_M_HEADER_IN_MSG BODY: T_TM2_M_HEADER_IN_MSG * -2.6 BAYES_00 BODY: Bayesian spam probability is 0 to 1% * [score: 0.0020] * -0.0 DCC_CHECK_NEGATIVE Not listed in DCC * [sa02 1397; Body=1 Fuz1=1 Fuz2=1] * 0.0 XM_SPF_Neutral SPF-Neutral Subject: Re: [PATCH 00/15] sysfs support for namespaces X-SA-Exim-Version: 4.2 (built Thu, 03 Mar 2005 10:44:12 +0100) X-SA-Exim-Scanned: Yes (on mgr1.xmission.com) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1535 Lines: 35 Cornelia Huck writes: > My impression was that the networking folks didn't want any warnings for > renaming failures, not just not for renaming a device to the same name. Which would be reasonable. Because all of the checks have been done before sysfs is called so if sysfs sees a problem it is a sysfs bug. >> In addition my introduction sysfs_rename_link handles this case >> cleanly by first removing the old link and then creating the new >> link. Preventing false positives when the link names are the same. > > sysfs_rename_link() looks cleaner, I agree. > >> >> So it should be safe to drop Cornelia patch without a reoccurance >> of scary errors. > > Hm, the description looks badly worded - I unfortunately left the old > text unchanged when I respun the patch :( The patch re-introduces the > warning in sysfs_add_one() which had been removed in the meanwhile and > makes device_rename() use a non-warning version. I still think we want > a warning for the general case since this is usually caused be some > problems in the calling code (and the alternative would be to add > checks to all callers.) Right. We just need to get the sysfs paths clean enough that we don't emit false positives. I think I have accomplished that for rename. Eric -- 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/