Re: [PATCH V2] xfs: libxfs: move xfs_perag_put late

2018-11-27 Thread Darrick J. Wong
On Tue, Nov 27, 2018 at 10:16:02AM +0100, Carlos Maiolino wrote: > On Tue, Nov 27, 2018 at 08:33:38AM +0800, Pan Bian wrote: > > The function xfs_alloc_get_freelist calls xfs_perag_put to drop the > > reference. However, pag->pagf_btreeblks is read and write after the > > put operation. This patch

Re: [PATCH V2] xfs: libxfs: move xfs_perag_put late

2018-11-27 Thread Carlos Maiolino
On Tue, Nov 27, 2018 at 08:33:38AM +0800, Pan Bian wrote: > The function xfs_alloc_get_freelist calls xfs_perag_put to drop the > reference. However, pag->pagf_btreeblks is read and write after the > put operation. This patch moves the put operation late. I'm not a native English speaker too, but

[PATCH V2] xfs: libxfs: move xfs_perag_put late

2018-11-26 Thread Pan Bian
The function xfs_alloc_get_freelist calls xfs_perag_put to drop the reference. However, pag->pagf_btreeblks is read and write after the put operation. This patch moves the put operation late. Signed-off-by: Pan Bian --- V2: correct the commit log --- fs/xfs/libxfs/xfs_alloc.c | 2 +- 1 file cha