From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [PATCH 01/20] sysfs: Implement sysfs_rename_link Date: Wed, 27 May 2009 17:37:21 -0700 Message-ID: <20090528003721.GB2720@suse.de> References: <1242865694-2100-1-git-send-email-ebiederm@xmission.com> <20090528001418.GA2499@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Greg KH , "Eric W. Biederman" , Andrew Morton , linux-kernel@vger.kernel.org, Tejun Heo , Cornelia Huck , linux-fsdevel@vger.kernel.org, Benjamin Thery , Daniel Lezcano , "Eric W. Biederman" To: Kay Sievers Return-path: Received: from cantor2.suse.de ([195.135.220.15]:38977 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758021AbZE1Bhq (ORCPT ); Wed, 27 May 2009 21:37:46 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu, May 28, 2009 at 02:30:15AM +0200, Kay Sievers wrote: > On Thu, May 28, 2009 at 02:14, Greg KH wrote: > > > > So, there's been a lot of talk in this thread. > > > > Eric, do you have an updated set of patches for me to try out? > > I think we should get a version of the patch in that removes all files > in a directory on cleanup, but warn if a subdirectory is still there. > James has a patch to fix the one issue we've seen so far with existing > child directories. > > After that, we should work on fixing the users that leave files > behind, and can possibly stop cleaning up files, if we want to. That would be good. But note that we always "allowed" such things to happen, so odds are there are lots of places in the kernel that took advantage of this. If we make it a rule, people will complain. > > Or are there still problems, like the "fry the ext3 boot partition" that > > Kay found? > > That is unrelated to Eric's patches, They just added the dump, which I > tried to trigger. It's not entirely clear what caused the filesytem > damage, but I was definitely able to reproduce the unclean shutdown > without any of Eric's sysfs patches. Ok. But I think Eric had some updates to some of the patches along the way, so a whole new respin would be good to ensure I get the correct ones. Eric? thanks, greg k-h