Re: [PATCH 11/60] xfs: Inode create item recovery

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 11/60] xfs: Inode create item recovery
From:	Mark Tinguely <tinguely@xxxxxxx>
Date:	Mon, 24 Jun 2013 09:37:42 -0500
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<1371617468-32559-12-git-send-email-david@xxxxxxxxxxxxx>
References:	<1371617468-32559-1-git-send-email-david@xxxxxxxxxxxxx> <1371617468-32559-12-git-send-email-david@xxxxxxxxxxxxx>
User-agent:	Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0

On 06/18/13 23:50, Dave Chinner wrote:

When we find a icreate transaction, we need to get and initialise
the buffers in the range that has been passed. Extract and verify
the information in the item record, then loop over the range
initialising and issuing the buffer writes delayed.

Support an arbitrary size range to initialise so that in
future when we allocate inodes in much larger chunks all kernels
that understand this transaction can still recover them.

Signed-off-by: Dave Chinner<david@xxxxxxxxxxxxx>
---



Looks good.

Reviewed-by: Mark Tinguely <tinguely@xxxxxxx>

<Prev in Thread]	Current Thread	[Next in Thread>
Re: [PATCH 07/60] xfs: Introduce ordered log vector support, (continued) Re: [PATCH 07/60] xfs: Introduce ordered log vector support, Mark Tinguely [PATCH 08/60] xfs: Introduce an ordered buffer item, Dave Chinner Re: [PATCH 08/60] xfs: Introduce an ordered buffer item, Mark Tinguely [PATCH 09/60] xfs: Inode create log items, Dave Chinner Re: [PATCH 09/60] xfs: Inode create log items, Mark Tinguely [PATCH 06/60] xfs: xfs_ifree doesn't need to modify the inode buffer, Dave Chinner Re: [PATCH 06/60] xfs: xfs_ifree doesn't need to modify the inode buffer, Mark Tinguely [PATCH 10/60] xfs: Inode create transaction reservations, Dave Chinner Re: [PATCH 10/60] xfs: Inode create transaction reservations, Mark Tinguely [PATCH 11/60] xfs: Inode create item recovery, Dave Chinner Re: [PATCH 11/60] xfs: Inode create item recovery, Mark Tinguely <= [PATCH 12/60] xfs: Use inode create transaction, Dave Chinner Re: [PATCH 12/60] xfs: Use inode create transaction, Mark Tinguely [PATCH 17/60] xfs: split out attribute fork truncation code into separate file, Dave Chinner [PATCH 13/60] xfs: remove local fork format handling from xfs_bmapi_write(), Dave Chinner [PATCH 14/60] xfs: move getdents code into it's own file, Dave Chinner [PATCH 20/60] xfs: move xfs_getbmap to xfs_extent_ops.c, Dave Chinner [PATCH 16/60] xfs: split out attribute listing code into separate file, Dave Chinner [PATCH 15/60] xfs: reshuffle dir2 definitions around for userspace, Dave Chinner [PATCH 23/60] xfs: sync minor header differences needed by userspace., Dave Chinner [PATCH 24/60] xfs: move xfs_bmap_punch_delalloc() to xfs_aops.c, Dave Chinner

Previous by Date:	[PATCH] xfstests: use xfs_io fiemap instead of filefrag V2, Josef Bacik
Next by Date:	[PATCH] xfstests: fix btrfs/265 so it actually works, Josef Bacik
Previous by Thread:	[PATCH 11/60] xfs: Inode create item recovery, Dave Chinner
Next by Thread:	[PATCH 12/60] xfs: Use inode create transaction, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]