linux-kernel - Re: [PATCH] dm: verity target

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 28 Sep 2011 17:30:14 -0400
From:	Valdis.Kletnieks@...edu
To:	Will Drewry <wad@...omium.org>
Cc:	Mandeep Singh Baines <msb@...omium.org>,
	Alasdair G Kergon <agk@...hat.com>,
	Milan Broz <mbroz@...hat.com>, dm-devel@...hat.com,
	Elly Jones <ellyjones@...omium.org>,
	Olof Johansson <olofj@...omium.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dm: verity target

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.

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

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

I'll hopefully have some more comments over the weekend if I get some spare
cycles.

Content of type "application/pgp-signature" skipped