xfs
[Top] [All Lists]

Re: [PATCH 1/2] xfs: don't truncate attribute extents if no extents exis

To: Brian Foster <bfoster@xxxxxxxxxx>
Subject: Re: [PATCH 1/2] xfs: don't truncate attribute extents if no extents exist
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Sun, 21 Jun 2015 02:22:10 -0700
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20150619154522.GH12833@xxxxxxxxxxxxxxx>
References: <1434631741-50856-1-git-send-email-bfoster@xxxxxxxxxx> <1434631741-50856-2-git-send-email-bfoster@xxxxxxxxxx> <20150619151421.GC22842@xxxxxxxxxxxxx> <20150619154522.GH12833@xxxxxxxxxxxxxxx>
User-agent: Mutt/1.5.23 (2014-03-12)
On Fri, Jun 19, 2015 at 11:45:23AM -0400, Brian Foster wrote:
> How about this?
> 
>         /*
>          * Invalidate and truncate the attribute fork extents. Make sure the
>          * fork actually has attributes as otherwise the invalidation has no
>          * blocks to read and returns an error. In this case, just do the fork
>          * removal below.
>          */

Yes, that looks good.

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