From: Miklos Szeredi Subject: Re: [GIT PULL] ext4 changes for 3.15 Date: Fri, 4 Apr 2014 19:16:25 +0200 Message-ID: References: <20140403191558.GA8745@thunk.org> <20140404035308.GC2525@thunk.org> <20140404134429.GB26806@quack.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: "Theodore Ts'o" , Linus Torvalds , Linux Kernel Mailing List , "linux-ext4@vger.kernel.org" , Al Viro To: Jan Kara Return-path: In-Reply-To: <20140404134429.GB26806@quack.suse.cz> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org On Fri, Apr 4, 2014 at 3:44 PM, Jan Kara wrote: > On Thu 03-04-14 23:53:08, Ted Tso wrote: >> On Thu, Apr 03, 2014 at 12:39:42PM -0700, Linus Torvalds wrote: >> > Btw, since I'm planning on getting to the filesystem pulls later today >> > (or perhaps tomorrow), I wanted to check: are you ok with the ext4 >> > parts of the cross-rename patches from Miklos? >> > >> > They are currently at >> > >> > git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git cross-rename >> > >> > in case you want to refresh your memory. >> >> I've pulled in the cross-rename branch to my test branch and run a set >> of tests, and it passes. Unfortunately I don't believe Miklos >> contributed tests for renameat(2) to xfstests, so we don't have any >> on-point testing of renameat() and cross-rename, but it's at least not >> triggering any failures on the existing tests. > Actually Miklos has contributed xfstests tests. I'm not sure if they were > merged but I've seen them flying by. Yes: http://marc.info/?l=linux-kernel&m=139523745403081&w=2 The following cases are tested: 1) plain rename (flags = 0) 2) cross rename 3) noreplace rename For each the following are tested: a) same directory rename b) cross directory rename And all combinations of source and dest being: - negative - regular file - symlink - empty dir - nonempty dir After the operation the file type of both source and dest are checked (or the error value returned). Thanks, Miklos