| 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
| ||
|
Message-ID: <20101020173231.GP3935@agk-dp.fab.redhat.com> Date: Wed, 20 Oct 2010 18:32:31 +0100 From: Alasdair G Kergon <agk@...hat.com> To: Milan Broz <mbroz@...hat.com> Cc: Andi Kleen <ak@...ux.intel.com>, Mike Snitzer <snitzer@...hat.com>, linux-kernel@...r.kernel.org, device-mapper development <dm-devel@...hat.com>, Andi Kleen <andi@...stfloor.org>, pedrib@...il.com Subject: Re: [dm-devel] [PATCH] DM-CRYPT: Scale to multiple CPUs v3 On Wed, Oct 20, 2010 at 04:20:44PM +0200, Milan Broz wrote: > this patch introduces apparent regressions in async crypto mode. OK, thanks for performing those tests. This patch is stalled until these problems are understood properly and solutions are proposed. I was prepared to overlook the regression in stacked 'async' provided there was an inline FIXME explaining clearly how to fix it in future should we need to, but I can't support a patch that apparently breaks stacked sync encryption. One solution might be to make the proposed changes only for non-stacked configurations and leave stacks to work as they do today - either detecting that automatically or having userspace supply a target parameter to say whether or not to turn it on. (For that, would need tests to show a stack of many layers where only one is a crypt target continues to work after the change.) Alasdair -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists