Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-29 Thread Liu Bo
On Mon, May 28, 2018 at 06:17:18PM +0200, David Sterba wrote: > On Mon, May 28, 2018 at 04:40:28PM +0200, David Sterba wrote: > > On Fri, May 18, 2018 at 11:00:18AM +0800, Liu Bo wrote: > > > Here are a collection of patches I did for btrfs_search_slot(). > > > > > > v2: more explicit commit log f

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-28 Thread David Sterba
On Mon, May 28, 2018 at 04:40:28PM +0200, David Sterba wrote: > On Fri, May 18, 2018 at 11:00:18AM +0800, Liu Bo wrote: > > Here are a collection of patches I did for btrfs_search_slot(). > > > > v2: more explicit commit log for each patch. > > > > Liu Bo (6): > > Btrfs: remove superfluous free

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-28 Thread David Sterba
On Fri, May 18, 2018 at 11:00:18AM +0800, Liu Bo wrote: > Here are a collection of patches I did for btrfs_search_slot(). > > v2: more explicit commit log for each patch. > > Liu Bo (6): > Btrfs: remove superfluous free_extent_buffer > Btrfs: use more straightforward extent_buffer_uptodate >

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-23 Thread Liu Bo
On Wed, May 23, 2018 at 02:34:40PM +0200, David Sterba wrote: > On Wed, May 23, 2018 at 10:16:55AM +0800, Su Yue wrote: > > >>> [ 47.692084] kernel BUG at fs/btrfs/locking.c:286! > > >> > > >> I saw the crash too but did not investigate the root cause. So I'll > > >> remove the branch from for-ne

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-23 Thread David Sterba
On Wed, May 23, 2018 at 10:16:55AM +0800, Su Yue wrote: > >>> [ 47.692084] kernel BUG at fs/btrfs/locking.c:286! > >> > >> I saw the crash too but did not investigate the root cause. So I'll > >> remove the branch from for-next until it's fixed. Thanks for the report. > > > > I think the problem

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-22 Thread Su Yue
On 05/22/2018 08:35 PM, Nikolay Borisov wrote: > > > On 22.05.2018 15:02, David Sterba wrote: >> On Tue, May 22, 2018 at 07:05:14PM +0800, Su Yue wrote: >>> Hi Liu and David, >>> During my local xfstests on kdave/for-next, btrfs/139 failed and >>> btrfs BUG_ON due to qgroup rescan. >>>

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-22 Thread Nikolay Borisov
On 22.05.2018 15:02, David Sterba wrote: > On Tue, May 22, 2018 at 07:05:14PM +0800, Su Yue wrote: >> Hi Liu and David, >> During my local xfstests on kdave/for-next, btrfs/139 failed and >> btrfs BUG_ON due to qgroup rescan. >> The bisect result is commit 560215eb3f32("Merge branch >>

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-22 Thread David Sterba
On Tue, May 22, 2018 at 07:05:14PM +0800, Su Yue wrote: > Hi Liu and David, > During my local xfstests on kdave/for-next, btrfs/139 failed and > btrfs BUG_ON due to qgroup rescan. > The bisect result is commit 560215eb3f32("Merge branch > 'ext/liubo/search-cleanups-wip' into for-next-ne

Re: [PATCH v2 0/6] btrfs_search_slot cleanups

2018-05-22 Thread Su Yue
Hi Liu and David, During my local xfstests on kdave/for-next, btrfs/139 failed and btrfs BUG_ON due to qgroup rescan. The bisect result is commit 560215eb3f32("Merge branch 'ext/liubo/search-cleanups-wip' into for-next-next-v4.18-20180521") which seems merged this patchset.