From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bob Peterson <rpeterso@redhat.com>
Date: Mon, 30 Jul 2018 16:06:57 -0400 (EDT)
Subject: [Cluster-devel] [PATCH 2/3] gfs2: Split gfs2_indirect_init
In-Reply-To: <20180730123353.15815-2-agruenba@redhat.com>
References: <20180730123353.15815-1-agruenba@redhat.com>
	<20180730123353.15815-2-agruenba@redhat.com>
Message-ID: <1145701876.55813214.1532981217591.JavaMail.zimbra@redhat.com>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

----- Original Message -----
> Function gfs2_indirect_init currently initializes a new indirect block
> and sets up the pointer to the new block in the parent indirect block.
> This is easier to understand when done in two separate calls.
> 
> Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
> ---
Hi,

Reviewed-by: Bob Peterson <rpeterso@redhat.com>

This looks right. It took me a while to convince myself though. :)

Regards,

Bob Peterson