diff mbox

crypto: powerpc - CRYPT_CRC32C_VPMSUM should depend on ALTIVEC

Message ID 1470048886-8899-1-git-send-email-mpe@ellerman.id.au (mailing list archive)
State Superseded
Headers show

Commit Message

Michael Ellerman Aug. 1, 2016, 10:54 a.m. UTC
The optimised crc32c implementation depends on VMX (aka. Altivec)
instructions, so the kernel must be built with Altivec support in order
for the crc32c code to build.

Fixes: 6dd7a82cc54e ("crypto: powerpc - Add POWER8 optimised crc32c")
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
---
 crypto/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Ellerman Aug. 5, 2016, 3:28 a.m. UTC | #1
Anton Blanchard <anton@samba.org> writes:

> Hi Michael,
>
>> The optimised crc32c implementation depends on VMX (aka. Altivec)
>> instructions, so the kernel must be built with Altivec support in
>> order for the crc32c code to build.
>
> Thanks for that, looks good.
>
> Acked-by: Anton Blanchard <anton@samba.org>

Thanks. Herbert, expecting you'll pick this up.

cheers
Herbert Xu Aug. 8, 2016, 3:24 p.m. UTC | #2
On Fri, Aug 05, 2016 at 01:28:03PM +1000, Michael Ellerman wrote:
> Anton Blanchard <anton@samba.org> writes:
> 
> > Hi Michael,
> >
> >> The optimised crc32c implementation depends on VMX (aka. Altivec)
> >> instructions, so the kernel must be built with Altivec support in
> >> order for the crc32c code to build.
> >
> > Thanks for that, looks good.
> >
> > Acked-by: Anton Blanchard <anton@samba.org>
> 
> Thanks. Herbert, expecting you'll pick this up.

In that case Anton can you please post it to linux-crypto?

Thanks,
Michael Ellerman Aug. 8, 2016, 10:46 p.m. UTC | #3
Herbert Xu <herbert@gondor.apana.org.au> writes:

> On Fri, Aug 05, 2016 at 01:28:03PM +1000, Michael Ellerman wrote:
>> Anton Blanchard <anton@samba.org> writes:
>> 
>> > Hi Michael,
>> >
>> >> The optimised crc32c implementation depends on VMX (aka. Altivec)
>> >> instructions, so the kernel must be built with Altivec support in
>> >> order for the crc32c code to build.
>> >
>> > Thanks for that, looks good.
>> >
>> > Acked-by: Anton Blanchard <anton@samba.org>
>> 
>> Thanks. Herbert, expecting you'll pick this up.
>
> In that case Anton can you please post it to linux-crypto?

Sorry that's my fault, will repost to linux-crypto.

cheers
diff mbox

Patch

diff --git a/crypto/Kconfig b/crypto/Kconfig
index a9377bef25e3..84d71482bf08 100644
--- a/crypto/Kconfig
+++ b/crypto/Kconfig
@@ -439,7 +439,7 @@  config CRYPTO_CRC32C_INTEL
 
 config CRYPT_CRC32C_VPMSUM
 	tristate "CRC32c CRC algorithm (powerpc64)"
-	depends on PPC64
+	depends on PPC64 && ALTIVEC
 	select CRYPTO_HASH
 	select CRC32
 	help