From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kay Sievers Subject: Re: [PATCH 22/20] sysfs: Make sysfs_rename_link atomic Date: Sun, 24 May 2009 15:03:31 +0200 Message-ID: References: <1243109591-21611-2-git-send-email-ebiederm@xmission.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Andrew Morton , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Tejun Heo , Cornelia Huck , linux-fsdevel@vger.kernel.org, "Eric W. Biederman" To: "Eric W. Biederman" Return-path: Received: from mail-bw0-f174.google.com ([209.85.218.174]:64204 "EHLO mail-bw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752928AbZEXNDw convert rfc822-to-8bit (ORCPT ); Sun, 24 May 2009 09:03:52 -0400 In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sun, May 24, 2009 at 01:21, Kay Sievers wrote= : > On Sat, May 23, 2009 at 23:32, Kay Sievers wro= te: >> On Sat, May 23, 2009 at 22:13, Eric W. Biederman wrote: >>> From: Eric W. Biederman >>> >>> Use the existing sysfs_rename to make sysfs_rename_link an atomic >>> operation that does less work. =C2=A0While I am at add additional s= anity >>> checking to ensure it is a symlink I am renaming. >>> >>> Signed-off-by: Eric W. Biederman >> >> Looks great, and so much better than the remove and re-create thing. > > Do you have your git tree public somewhere, or do you mind sending me > an all-in-one patch? I like to give it a try here. Looks good. It survives a heavy hotplug setup just fine, renaming works fine here, and sysfs looks right. There are a bunch of warnings for some non-empty directories, which I replied with in the patch that added the warning. Thanks, Kay -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel= " in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html