xfs
[Top] [All Lists]

Re: [PATCH 11/47] xfs: move deferred operations into a separate file

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 11/47] xfs: move deferred operations into a separate file
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 4 Aug 2016 09:00:07 -0700
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, "Darrick J. Wong" <darrick.wong@xxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, vishal.l.verma@xxxxxxxxx, bfoster@xxxxxxxxxx, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160803225756.GW16044@dastard>
References: <146907695530.25461.3225785294902719773.stgit@xxxxxxxxxxxxxxxx> <146907703710.25461.16650495404061662831.stgit@xxxxxxxxxxxxxxxx> <20160801080223.GB30547@xxxxxxxxxxxxx> <20160802223950.GN16044@dastard> <20160803091627.GA5289@xxxxxxxxxxxxx> <20160803225756.GW16044@dastard>
User-agent: Mutt/1.6.1 (2016-04-27)
On Thu, Aug 04, 2016 at 08:57:56AM +1000, Dave Chinner wrote:
> So, please explain in more detail what the problem is and what the
> proposed solution is so I (and probably Darrick, too) have some
> understanding of the issue you see with this code.

We were doing 1 (actually 2 with the busy extent tracking) allocations
for each free extent, and now we're up to three.  We need to get this
down to 1 and not increase it for no benefit.

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