From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH 22/24] sysfs: Make sysfs_rename_link atomic Date: Sat, 30 May 2009 19:48:26 +0900 Message-ID: <4A210EFA.5080103@kernel.org> References: <1243551665-23596-22-git-send-email-ebiederm@xmission.com> <4A1FA7F4.9000708@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Andrew Morton , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Cornelia Huck , linux-fsdevel@vger.kernel.org, Kay Sievers , Greg KH , "Eric W. Biederman" To: "Eric W. Biederman" Return-path: Received: from hera.kernel.org ([140.211.167.34]:57032 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755276AbZE3Ksg (ORCPT ); Sat, 30 May 2009 06:48:36 -0400 In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: Eric W. Biederman wrote: > Tejun Heo writes: > >> 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. While I am at add additional sanity >>> checking to ensure it is a symlink I am renaming. >>> >>> Acked-by: Kay Sievers >>> Signed-off-by: Eric W. Biederman >> It would be really nice to merge or group this together with the first >> three patches. Other than that, > > Perfection is the enemy of the good on that one. That just convolutes > things unnecessarily, makes the patches harder to review, and requires > additional testing. > > I much prefer to work in a tree without rewinding. Everything is a matter of degree and I don't think my bar here was too high. Unless patches are in some exported tree and when the content of the patchset changes (you're changing the behavior of the same code two times), reordering patches is what's usually done. When the end results stay the same, reshuffling isn't that much of work, is it? Well, I'm not the maintainer and things like this are mostly upto the maintainer, so let's leave it to be decided by Greg. Thanks. -- tejun