[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <EB5DDA28-421A-4C67-817C-74F9B9643351@dilger.ca>
Date: Thu, 18 Aug 2011 00:16:00 -0600
From: Andreas Dilger <adilger@...ger.ca>
To: "djwong@...ibm.com" <djwong@...ibm.com>
Cc: Theodore Ts'o <tytso@....edu>,
Andreas Dilger <adilger.kernel@...ger.ca>,
linux-fsdevel <linux-fsdevel@...r.kernel.org>,
linux-ext4 <linux-ext4@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
Sunil Mushran <sunil.mushran@...cle.com>,
Joel Becker <jlbec@...lplan.org>,
Mingming Cao <cmm@...ibm.com>,
Amir Goldstein <amir73il@...il.com>,
Coly Li <colyli@...il.com>, Andi Kleen <andi@...stfloor.org>
Subject: Re: [RFC] ext4 metadata checksumming design
On 2011-08-16, at 9:25 PM, "Darrick J. Wong" <djwong@...ibm.com> wrote:
> - Extended attribute blocks that are stored in the inode table -- the h_magic
> field is written by the kernel, but neither the kernel nor e2fsprogs ever
> actually read this field. The field could be reused to checksum the extra
> space since (as far as I can tell) EAs are the only user of that empty space.
I haven't had a chance to read the document you wrote, but wanted to comment on xattrs. There is a hash field for each xattr (including internal xattrs), and one for the external xattr blocks that can be used to validate the xattr value.
In addition to the hash for the in-inode xattrs, the inode hash itself would serve to validate the xattr values.
I have a patch for e2fsprogs that checks the xattr hash for in-inode xattrs (currently it is always 0).
> Please have a look at the design document and please feel free to suggest any
> changes.
Hopefully soon.
Cheers, Andreas--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists