[PATCH v2] xfsdump: call mlog_exit in content_stream_restore

Alex Elder aelder at sgi.com
Wed Sep 21 15:04:17 CDT 2011


On Wed, 2011-09-21 at 14:34 -0500, Bill Kendall wrote:
> This patch adds mlog_exit() calls to all the return paths in
> content_stream_restore(). mlog_exit() is supposed to be called before
> returning from content_stream_dump() and content_stream_restore(), but the
> latter did not do so, allowing for the stream exit status to be unset.
> 
> Signed-off-by: Bill Kendall <wkendall at sgi.com>

I did not give this as thorough a review as I did last time.
You have already implemented several of the suggestions I
made, and this now looks better than before to me.

Reviewed-by: Alex Elder <aelder at sgi.com>





More information about the xfs mailing list