Re: [PATCH] dm: verity target

From: John Stoffel
Date: Wed Sep 28 2011 - 21:27:20 EST


>>>>> "Valdis" == Valdis Kletnieks <Valdis.Kletnieks@xxxxxx> writes:

Valdis> On Tue, 27 Sep 2011 14:02:05 CDT, Will Drewry said:
>> I was just curious if there is any interest in pulling this change, or
>> if not, if there is any particular set of concerns, fixes, etc.

Valdis> Out of curiosity, how much of the stack does this end up eating? My root
Valdis> filesystem is already ext4 on an LVM partition that's on a LUKS/dm-crypt
Valdis> partition on a hard drive, and I'm sure somebody out there will have used xfs
Valdis> instead - and then exported it via NFS or something. Are we going to get weird
Valdis> stack overflows if people throw dm-verity into this sort of mix?

>> realize it's not a small amount of code to digest (though it is
>> smaller than the post from last year[1]). Would re-posting with an
>> added blob explaining the name be useful,

Valdis> Probably will need it to be merged, unless you set up an
Valdis> auto-reply that says "Patch rejected, 'verity' is *not* a typo
Valdis> for 'verify'" ;)

God, I've been reading this as veriFy all along. I think your name
stinks because it's too close to Verify, and too obscure otherwise.

John
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/