On Mon, Jun 18, 2018 at 01:06:16PM +0200, David Sterba wrote: > On Fri, Jun 15, 2018 at 05:19:07PM +0100, Filipe Manana wrote: > > On Fri, Jun 15, 2018 at 4:54 PM, David Sterba <dste...@suse.cz> wrote: > > > On Mon, Jun 11, 2018 at 07:24:28PM +0100, fdman...@kernel.org wrote: > > >> From: Filipe Manana <fdman...@suse.com> > > >> Fixes: 12fcfd22fe5b ("Btrfs: tree logging unlink/rename fixes") > > >> Reported-by: Vijay Chidambaram <vvija...@gmail.com> > > >> Signed-off-by: Filipe Manana <fdman...@suse.com> > > > > > > There are some warnings and possible lock up caused by this patch, the > > > 1/2 alone is ok but 1/2 + 2/2 leads to the following warnings. I checked > > > twice, the patch base was the pull request ie. without any other 4.18 > > > stuff. > > > > Are you sure it's this patch? > > On top of for-4.18 it didn't cause any problems here, plus the trace > > below has nothing to do with renames, hard links or fsync at all - > > everything seems stuck on waiting for IO from dev replace. > > It was a false alert, sorry. Strange that the warnings appeared only in > the VM running both patches and not otherwise. > > Though the test did not directly use rename, the possible error scenario > I had in mind was some leftover from locking, error handling or state > that blocked umount of 011.
Dave, are you sending this in for 4.19? I don't see it in your first pull request.