| To: | Brian Foster <bfoster@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 4/4] xfs: only writeback and truncate pages for the freed range |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Tue, 9 Sep 2014 15:14:50 +1000 |
| Cc: | xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1410092760-3451-5-git-send-email-bfoster@xxxxxxxxxx> |
| References: | <1410092760-3451-1-git-send-email-bfoster@xxxxxxxxxx> <1410092760-3451-5-git-send-email-bfoster@xxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Sun, Sep 07, 2014 at 08:26:00AM -0400, Brian Foster wrote: > xfs_free_file_space() only affects the range of the file for which space > is being freed. It currently writes and truncates the page cache from > the start offset of the free to EOF. > > Modify xfs_free_file_space() to write back and truncate page cache of > just the range being freed. > > Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx> looks good. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 3/4] xfs: writeback and inval. file range to be shifted by collapse, Dave Chinner |
|---|---|
| Next by Date: | Re: [PATCH] xfs: xlog_cil_force_lsn doesn't always wait correctly, Brian Foster |
| Previous by Thread: | [PATCH 4/4] xfs: only writeback and truncate pages for the freed range, Brian Foster |
| Next by Thread: | [PATCH 2/4] xfs: refactor xfs_bmap_shift_extents() into multiple functions, Brian Foster |
| Indexes: | [Date] [Thread] [Top] [All Lists] |