From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754653Ab2AaOll (ORCPT ); Tue, 31 Jan 2012 09:41:41 -0500 Received: from mail-ee0-f46.google.com ([74.125.83.46]:40532 "EHLO mail-ee0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752904Ab2AaOlk (ORCPT ); Tue, 31 Jan 2012 09:41:40 -0500 Message-ID: <4F27FDA0.6090806@suse.cz> Date: Tue, 31 Jan 2012 15:41:36 +0100 From: Jiri Slaby User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20120124 Thunderbird/10.0 MIME-Version: 1.0 To: "Eric W. Biederman" CC: Greg KH , Greg Kroah-Hartman , LKML , ML netdev , Kay Sievers Subject: Re: [PATCH] sysfs: Update the name hash when renaming sysfs entries References: <4F27120A.4040106@suse.cz> <4F27C54F.1010107@suse.cz> <4F27C9D4.3040105@suse.cz> <4F27CDBC.5020201@suse.cz> In-Reply-To: X-Enigmail-Version: 1.3.5 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/31/2012 03:40 PM, Eric W. Biederman wrote: > > This fixes a bug introduced with sysfs name hashes where renaming a > network device appears to succeed but silently makes the sysfs files for > that network device inaccessible. > > In at least one configuration this bug has stopped networking from > coming up during boot. > > Signed-off-by: Eric W. Biederman It works for me. Thanks. Tested-by: Jiri Slaby > --- > fs/sysfs/dir.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/fs/sysfs/dir.c b/fs/sysfs/dir.c > index ea64d01..dd3779c 100644 > --- a/fs/sysfs/dir.c > +++ b/fs/sysfs/dir.c > @@ -872,6 +872,7 @@ int sysfs_rename(struct sysfs_dirent *sd, > > dup_name = sd->s_name; > sd->s_name = new_name; > + sd->s_hash = sysfs_name_hash(sd->s_ns, sd->s_name); > } > > /* Move to the appropriate place in the appropriate directories rbtree. */ -- js suse labs