[Top] [All Lists]

Re: [PATCH 0/7] xfs: xfs_log_recover.c cleanups

To: xfs@xxxxxxxxxxx
Subject: Re: [PATCH 0/7] xfs: xfs_log_recover.c cleanups
From: Alex Elder <aelder@xxxxxxx>
Date: Fri, 09 Apr 2010 17:26:04 -0500
In-reply-to: <201003182253.o2IMrfX0001827@xxxxxxxxxxxxxxxxxxxxxx>
References: <201003182253.o2IMrfX0001827@xxxxxxxxxxxxxxxxxxxxxx>
Reply-to: aelder@xxxxxxx
On Thu, 2010-03-18 at 17:53 -0500, Alex Elder wrote:
> This series of patches contains a set of changes primarily in
> the file "fs/xfs/xfs_log_recover.c".  They're mostly cleanup
> patches, but one of them fixes a bug in two spots.

I'm re-submitting this series of patches.  A few of them
are unchanged (already reviewed by Christoph Hellwig).
I have addressed a few other comments he made:
- Only encapsulate the validity check, not the reporting
  of errors, in the 5th patch
- The 7th "odds and ends" patch has been broken into
  several smaller patches, each with its own explanation.
  A few changes have been removed for now.
- Added another patch to the series that gets rid of
  the l_sectbb_log (which holds the log-base-2 of
  the number of basic blocks in a log sector), and
  in its places uses l_sectsize (the number of basic
  blocks in a log sector).

There are now 10 patches in the series.  I've indicated
the patches that involve changes from before (possibly
pulled into a separate patch) with "PATCHv2".


<Prev in Thread] Current Thread [Next in Thread>
  • Re: [PATCH 0/7] xfs: xfs_log_recover.c cleanups, Alex Elder <=