diff mbox

powerpc/configs: Enable VMX crypto

Message ID 1468279521-9720-1-git-send-email-anton@ozlabs.org (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Anton Blanchard July 11, 2016, 11:25 p.m. UTC
From: Anton Blanchard <anton@samba.org>

We see big improvements with the VMX crypto functions (often 10x or more),
so enable it as a module.

Signed-off-by: Anton Blanchard <anton@samba.org>
---
 arch/powerpc/configs/powernv_defconfig | 2 ++
 arch/powerpc/configs/pseries_defconfig | 2 ++
 2 files changed, 4 insertions(+)

Comments

Stephen Rothwell July 11, 2016, 11:33 p.m. UTC | #1
Hi Anton,

On Tue, 12 Jul 2016 09:25:21 +1000 Anton Blanchard <anton@ozlabs.org> wrote:
>
> From: Anton Blanchard <anton@samba.org>
> 
> We see big improvements with the VMX crypto functions (often 10x or more),
> so enable it as a module.
> 
> Signed-off-by: Anton Blanchard <anton@samba.org>
> ---
>  arch/powerpc/configs/powernv_defconfig | 2 ++
>  arch/powerpc/configs/pseries_defconfig | 2 ++
>  2 files changed, 4 insertions(+)

Not in ppc64_defconfig?
Unknown via Linuxppc-dev July 11, 2016, 11:41 p.m. UTC | #2
Hi Steven,

> Not in ppc64_defconfig?

Good point. The recent addition of powernv_defconfig made me forget
about ppc64_defconfig. We could do with some rationalisation here.

pseries isn't really pseries, perhaps we should call it ibm_defconfig,
or maybe server_deconfig. ppc64_defconfig continues to be the lower end
defconfig.

Anton
Michael Ellerman July 12, 2016, 4:17 a.m. UTC | #3
Anton Blanchard <anton@samba.org> writes:

> Hi Steven,
>
>> Not in ppc64_defconfig?
>
> Good point. The recent addition of powernv_defconfig made me forget
> about ppc64_defconfig. We could do with some rationalisation here.

Understatement of the century.

> pseries isn't really pseries, perhaps we should call it ibm_defconfig,
> or maybe server_deconfig.

pseries should be just pseries, and nothing else.

> ppc64_defconfig continues to be the lower end defconfig.

ppc64_defconfig *should* be the primary config. And similarly ppc64le
when building little endian.

The reason being that 'make defconfig' will give you one of those
depending on uname (when building native).

I think the only thing disqualifying it from that role at the moment is
that it builds 4K pages.

But I think we should just change that. We can add ppc64_4K_defconfig
for anyone who wants it (G5 users mainly I would guess).

We'd also need to make sure ppc64/ppc64le have all the modern options
enabled.

cheers
diff mbox

Patch

diff --git a/arch/powerpc/configs/powernv_defconfig b/arch/powerpc/configs/powernv_defconfig
index dce352e..3f6226b 100644
--- a/arch/powerpc/configs/powernv_defconfig
+++ b/arch/powerpc/configs/powernv_defconfig
@@ -310,6 +310,8 @@  CONFIG_CRYPTO_TEA=m
 CONFIG_CRYPTO_TWOFISH=m
 CONFIG_CRYPTO_LZO=m
 CONFIG_CRYPTO_DEV_NX=y
+CONFIG_CRYPTO_DEV_VMX=y
+CONFIG_CRYPTO_DEV_VMX_ENCRYPT=m
 CONFIG_VIRTUALIZATION=y
 CONFIG_KVM_BOOK3S_64=m
 CONFIG_KVM_BOOK3S_64_HV=m
diff --git a/arch/powerpc/configs/pseries_defconfig b/arch/powerpc/configs/pseries_defconfig
index 99dec9d..1d7eea2 100644
--- a/arch/powerpc/configs/pseries_defconfig
+++ b/arch/powerpc/configs/pseries_defconfig
@@ -313,6 +313,8 @@  CONFIG_CRYPTO_LZO=m
 # CONFIG_CRYPTO_ANSI_CPRNG is not set
 CONFIG_CRYPTO_DEV_NX=y
 CONFIG_CRYPTO_DEV_NX_ENCRYPT=m
+CONFIG_CRYPTO_DEV_VMX=y
+CONFIG_CRYPTO_DEV_VMX_ENCRYPT=m
 CONFIG_VIRTUALIZATION=y
 CONFIG_KVM_BOOK3S_64=m
 CONFIG_KVM_BOOK3S_64_HV=m