[Top] [All Lists]

RE: [PATCH 0/10] Perag lookup rework

To: "Dave Chinner" <david@xxxxxxxxxxxxx>
Subject: RE: [PATCH 0/10] Perag lookup rework
From: "Alex Elder" <aelder@xxxxxxx>
Date: Thu, 14 Jan 2010 12:42:12 -0600
Cc: <xfs@xxxxxxxxxxx>
In-reply-to: <1263210469-22171-1-git-send-email-david@xxxxxxxxxxxxx>
Thread-index: AcqStfG+FJYDE6GcTRuPmpiaaqPUUQCkv0cg
Thread-topic: [PATCH 0/10] Perag lookup rework
Dave Chinner wrote:
> This is the previous set of patches all put together in the one series
> for the current xfs-dev tree.
> Patch 10 is a new patch - it embeds the busy extent list into the perag
> structure rather than leaving as a separate allocation.

All 10 patches look good.  I am adding the following descriptions
to these patches which lacked them:

[03/10] XFS: Convert filestreams code to use per-ag get/put routines

Use xfs_perag_get() and xfs_perag_put() in the filestreams code.

[09/10] xfs: handle ENOMEM correctly during initialisation of perag structures

Add proper error handling in case an error occurs while initializing
new perag structures for a mount point.  The mount structure is
restored to its previous state by deleting and freeing any perag
structures added during the call.

Reviewed-by: Alex Elder <aelder@xxxxxxx>

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