From: Alex Elder <aelder@sgi.com>
To: Dave Chinner <david@fromorbit.com>
Cc: xfs@oss.sgi.com
Subject: Re: Updated xfsprogs 2.6.38 merge
Date: Fri, 29 Jul 2011 17:12:44 -0500 [thread overview]
Message-ID: <1311977564.2790.67.camel@doink> (raw)
In-Reply-To: <20110705024855.GA561@dastard>
On Tue, 2011-07-05 at 12:48 +1000, Dave Chinner wrote:
> Folks,
>
> I pushed out an updated 2.6.38 kernel merge to xfsprogs patchset a
> couple of days ago. I've been doing quite a bit of testing on it,
> both 32 bit and 64 bit, with 512 byte, 1k and 4k block size
> filesystems and I haven't come across any regressions. The patchset
> can be found here:
>
> git://git.kernel.org/pub/scm/fs/xfs/xfsprogs-dev kernel-2.6.38-sync
>
> It's pretty much unchanged from the last set of patches I sent,
> except for one minor fix to the radix tree code for an off by one in
> the path array size for item and tag deletes.
>
> I'm pretty much ready to commit this update so I can then move
> forward with updating it to the 3.0 kernel code base as a smaller
> incremental series.
>
> Cheers,
>
> Dave.
I looked over the changes--the third one really since
the first two already indicated I'd signed off on them.
It is a very large patch, and most of the changes are
pretty easily seen to be straightforward transformations.
So my "review" consisted of a full-but-quick scan through
it.
More importantly, you report no regressions and I can
confirm that I haven't seen any myself either so far,
except that the golden output for test 122 needs to be
updated:
- to reflect that xfs_bmbt_rec_{32,64}_t have now
been replaced by xfs_bmbt_rec_t
- to reflect that xfs_dinode_core_t no longer exists
- and that xfs_alloctype_t isn't shown any more,
because it's no longer an enum type.
It looks to me like that test could be updated so it
looks for structure definitions rather than typedef's,
and possibly review the list of ignored types.
Anyway, I really want to see this committed so we can move
forward without further ado. So I say get it in...
Signed-off-by: Alex Elder <aelder@sgi.com>
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
prev parent reply other threads:[~2011-07-29 22:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-05 2:48 Updated xfsprogs 2.6.38 merge Dave Chinner
2011-07-10 21:01 ` Christoph Hellwig
2011-07-11 0:01 ` Dave Chinner
2011-07-11 5:27 ` Christoph Hellwig
2011-07-22 15:57 ` Christoph Hellwig
2011-07-29 22:12 ` Alex Elder [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1311977564.2790.67.camel@doink \
--to=aelder@sgi.com \
--cc=david@fromorbit.com \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox