From: Dave Chinner <david@fromorbit.com>
To: Mark Tinguely <tinguely@sgi.com>
Cc: xfs@oss.sgi.com
Subject: Re: [PATCH TARBALL, DEV ONLY] xfsprogs: metadata CRC support V2
Date: Fri, 26 Apr 2013 07:40:40 +1000 [thread overview]
Message-ID: <20130425214040.GQ30622@dastard> (raw)
In-Reply-To: <51799C24.8010901@sgi.com>
On Thu, Apr 25, 2013 at 04:12:04PM -0500, Mark Tinguely wrote:
> On 04/12/13 09:55, Dave Chinner wrote:
> >Hi folks,
> >
> >I've attached a tarball containing the patch series that adds
> >metadata CRC support to xfsprogs. It is still very much a work in
> >progress, but is sufficient to start running xfstests on CRC enable
> >filesystems.
> >
> >THere is much missing functionality, especially with respect to
> >directly/attribute modifications. Only mkfs and xfs_repair have been
> >updated to explicitly support CRC on directory/sttribute blocks,
> >though all the other tools will stil run the result might be a
> >little unpredictable.
> >
> >The patch series is as follows:
> >
>
> BTW forgot about this gem, the UBER/xfsprogs-kernel-3.8-update patch
> has been kicking up duplicate defines in sles11sp2 (gcc 4.3.4), but
> fine in newer distros (gcc 4.6.2). Looks like a compile difference
> issue:
>
> xfs_dir2_data_aoff_t xfs_dir2_format.h and xfs_dir2.h
> xfs_dir2_db_t xfs_dir2_format.h and xfs_dir2.h
>
> The one below seems new to me:
> prid_t project.h and xfs_types.h
>
> I have been removing the entry in xfs_dir2_format.h in the past to test.
> I don't remember the prid_t duplicate.
Hmmmm. I'm not getting those warnings here (gcc-4.7.2). I'll have a
look into it....
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2013-04-25 21:40 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-12 14:55 [PATCH TARBALL, DEV ONLY] xfsprogs: metadata CRC support V2 Dave Chinner
2013-04-25 16:37 ` Mark Tinguely
2013-04-25 21:39 ` Dave Chinner
2013-04-25 21:12 ` Mark Tinguely
2013-04-25 21:40 ` Dave Chinner [this message]
2013-04-29 21:33 ` Mark Tinguely
2013-04-29 22:09 ` Dave Chinner
2013-04-30 12:03 ` Dave Chinner
2013-04-26 6:12 ` Christoph Hellwig
2013-04-29 1:46 ` Dave Chinner
2013-05-09 12:42 ` Rich Johnston
2013-05-09 13:22 ` Dave Chinner
2013-05-09 13:37 ` Rich Johnston
2013-05-09 13:41 ` Rich Johnston
2013-05-09 20:24 ` Rich Johnston
2013-05-09 22:35 ` Dave Chinner
2013-05-09 23:00 ` Rich Johnston
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=20130425214040.GQ30622@dastard \
--to=david@fromorbit.com \
--cc=tinguely@sgi.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