xfs
[Top] [All Lists]

Re: iomap infrastructure and multipage writes V5

To: Eric Sandeen <sandeen@xxxxxxxxxxx>
Subject: Re: iomap infrastructure and multipage writes V5
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Mon, 13 Feb 2017 22:10:24 -0800
Cc: Dave Chinner <david@xxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, rpeterso@xxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
Dkim-signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20170209; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=bWSmX9VAxPub6XHXVhOdMS2q+0zoOOBFvPSQiy/cguQ=; b=kzx1k7EYTL5KwEWK4Bah0XAV+ Pde8VAVkQSVXyA5Ml4B51P4tUEbtymxsRJSJUZZo5ABLSbopSmHZZrMSqPeF3WXjUIJztlUmhQ5KS qqoSEEnl+y3M0q8kfr2o3RNiI6uuBEGaHvYBNrvOEF8talB1lDwXq4xZ6K5wKNKnqaWVw7uDI+w49 zLnv1Ap8UBIjIaeqSoVMmSkPce1FAKHnrG4ICW/vRsAoDX4V5pfLXhKcYZYcygwtZQxkNigPAyyUQ V7NrhB+DteSyjbivSP773gXg4vUo5RgiwNLCZ/1i4Hy71zP6ykDxaoK8KUs7/0t3R+/v0StbpoYXx rTLmkrNNg==;
In-reply-to: <75a139c8-5e49-3a89-10d1-20caeef3f69a@xxxxxxxxxxx>
References: <1464792297-13185-1-git-send-email-hch@xxxxxx> <20160628002649.GI12670@dastard> <20160630172239.GA23082@xxxxxx> <20160718111400.GC16044@dastard> <20160718111851.GD16044@dastard> <20160731191900.GA29301@xxxxxx> <20160802234247.GO16044@dastard> <75a139c8-5e49-3a89-10d1-20caeef3f69a@xxxxxxxxxxx>
User-agent: Mutt/1.7.1 (2016-10-04)
On Mon, Feb 13, 2017 at 04:31:55PM -0600, Eric Sandeen wrote:
> > This solves the problem for me, and from history appears to be the
> > right thing to do. Christoph, can you send a proper patch for this?
> 
> Did anything ever come of this?  I don't think I saw a patch, and it looks
> like it is not upstream.

"xfs: fix bogus space reservation in xfs_iomap_write_allocate" got edits
from Dave to fix this issue up in-line.  It's actually still kinda
bogus as I found out while spending more time with the allocator,
though.  Fixing the whole total vs minlen thing is rather invasive,
but I've started working on it and hope to have something for the
next merge window.

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