xfs
[Top] [All Lists]

Re: [PATCH 01/11] xfs: disentagle EFI release from the extent count

To: Brian Foster <bfoster@xxxxxxxxxx>
Subject: Re: [PATCH 01/11] xfs: disentagle EFI release from the extent count
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Wed, 12 Aug 2015 00:15:18 -0700
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20150810123726.GA18933@xxxxxxxxxxxxxxx>
References: <1438883072-28706-1-git-send-email-bfoster@xxxxxxxxxx> <1438883072-28706-2-git-send-email-bfoster@xxxxxxxxxx> <20150809073641.GA3163@xxxxxxxxxxxxx> <20150810123726.GA18933@xxxxxxxxxxxxxxx>
User-agent: Mutt/1.5.23 (2014-03-12)
On Mon, Aug 10, 2015 at 08:37:27AM -0400, Brian Foster wrote:
> > As a follow on we should be able to remove atomic_inc_return and
> > replace it with a local iterator in xfs_bmap_finish().
> > 
> 
> I'm not sure what you mean here...

efi_next_extent is not only used is to find a 'slot' for the extent
in xfs_trans_log_efi_extent.  For that we don't need an atomic_t
in the EFI, but can have a local variable in xfs_bmap_finish.

<Prev in Thread] Current Thread [Next in Thread>