Re: [PATCH 12/60] xfs: Use inode create transaction

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 12/60] xfs: Use inode create transaction
From:	Mark Tinguely <tinguely@xxxxxxx>
Date:	Mon, 24 Jun 2013 13:55:52 -0500
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<1371617468-32559-13-git-send-email-david@xxxxxxxxxxxxx>
References:	<1371617468-32559-1-git-send-email-david@xxxxxxxxxxxxx> <1371617468-32559-13-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:

Replace the use of buffer based logging of inode initialisation,
uses the new logical form to describe the range to be initialised
in recovery. We continue to "log" the inode buffers to push them
into the AIL and ensure that the inode create transaction is not
removed from the log before the inode buffers are written to disk.

Update the transaction identifier and reservations to match the
changed implementation.

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


Looks good. Nice feature.

Reviewed-by: Mark Tinguely <tinguely@xxxxxxx>

<Prev in Thread]	Current Thread	[Next in Thread>
Re: [PATCH 08/60] xfs: Introduce an ordered buffer item, (continued) 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 [PATCH 21/60] xfs: introduce xfs_sb.c for sharing with libxfs, Dave Chinner [PATCH 18/60] xfs: split out xfs inode operations into separate file, Dave Chinner

Previous by Date:	Bug#713994: XFS repair, Wes Deviers
Next by Date:	Re: [PATCH 59/60] xfs: Add xfs_log_rlimit.c, Mark Tinguely
Previous by Thread:	[PATCH 12/60] xfs: Use inode create transaction, Dave Chinner
Next by Thread:	[PATCH 17/60] xfs: split out attribute fork truncation code into separate file, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]