* [PATCH v1] crypto: aesni - fix build on x86 (32bit)
@ 2013-12-30 13:52 Andy Shevchenko
2013-12-31 11:57 ` Herbert Xu
2014-01-06 17:41 ` Tim Chen
0 siblings, 2 replies; 15+ messages in thread
From: Andy Shevchenko @ 2013-12-30 13:52 UTC (permalink / raw)
To: linux-crypto, Herbert Xu, Tim Chen, Borislav Petkov, hpa; +Cc: Andy Shevchenko
It seems commit d764593a "crypto: aesni - AVX and AVX2 version of AESNI-GCM
encode and decode" breaks a build on x86_32 since it's designed only for
x86_64. This patch makes a compilation unit conditional to CONFIG_64BIT and
functions usage to CONFIG_X86_64.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
arch/x86/crypto/Makefile | 3 ++-
arch/x86/crypto/aesni-intel_glue.c | 2 ++
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/arch/x86/crypto/Makefile b/arch/x86/crypto/Makefile
index 84ee1e1..188b993 100644
--- a/arch/x86/crypto/Makefile
+++ b/arch/x86/crypto/Makefile
@@ -75,7 +75,8 @@ ifeq ($(avx2_supported),yes)
serpent-avx2-y := serpent-avx2-asm_64.o serpent_avx2_glue.o
endif
-aesni-intel-y := aesni-intel_asm.o aesni-intel_avx.o aesni-intel_glue.o fpu.o
+aesni-intel-y := aesni-intel_asm.o aesni-intel_glue.o fpu.o
+aesni-intel-$(CONFIG_64BIT) += aesni-intel_avx.o
ghash-clmulni-intel-y := ghash-clmulni-intel_asm.o ghash-clmulni-intel_glue.o
sha1-ssse3-y := sha1_ssse3_asm.o sha1_ssse3_glue.o
crc32c-intel-y := crc32c-intel_glue.o
diff --git a/arch/x86/crypto/aesni-intel_glue.c b/arch/x86/crypto/aesni-intel_glue.c
index 3ae311d..948ad0e 100644
--- a/arch/x86/crypto/aesni-intel_glue.c
+++ b/arch/x86/crypto/aesni-intel_glue.c
@@ -1473,6 +1473,7 @@ static int __init aesni_init(void)
if (!x86_match_cpu(aesni_cpu_id))
return -ENODEV;
+#ifdef CONFIG_X86_64
#ifdef CONFIG_AS_AVX2
if (boot_cpu_has(X86_FEATURE_AVX2)) {
pr_info("AVX2 version of gcm_enc/dec engaged.\n");
@@ -1492,6 +1493,7 @@ static int __init aesni_init(void)
aesni_gcm_enc_tfm = aesni_gcm_enc;
aesni_gcm_dec_tfm = aesni_gcm_dec;
}
+#endif
err = crypto_fpu_init();
if (err)
--
1.8.5.2
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2013-12-30 13:52 [PATCH v1] crypto: aesni - fix build on x86 (32bit) Andy Shevchenko
@ 2013-12-31 11:57 ` Herbert Xu
2014-01-06 17:41 ` Tim Chen
1 sibling, 0 replies; 15+ messages in thread
From: Herbert Xu @ 2013-12-31 11:57 UTC (permalink / raw)
To: Andy Shevchenko; +Cc: linux-crypto, Tim Chen, Borislav Petkov, hpa
On Mon, Dec 30, 2013 at 03:52:24PM +0200, Andy Shevchenko wrote:
> It seems commit d764593a "crypto: aesni - AVX and AVX2 version of AESNI-GCM
> encode and decode" breaks a build on x86_32 since it's designed only for
> x86_64. This patch makes a compilation unit conditional to CONFIG_64BIT and
> functions usage to CONFIG_X86_64.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Patch applied. Thanks a lot!
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2013-12-30 13:52 [PATCH v1] crypto: aesni - fix build on x86 (32bit) Andy Shevchenko
2013-12-31 11:57 ` Herbert Xu
@ 2014-01-06 17:41 ` Tim Chen
2014-01-06 17:45 ` H. Peter Anvin
1 sibling, 1 reply; 15+ messages in thread
From: Tim Chen @ 2014-01-06 17:41 UTC (permalink / raw)
To: Andy Shevchenko; +Cc: linux-crypto, Herbert Xu, Borislav Petkov, hpa
On Mon, 2013-12-30 at 15:52 +0200, Andy Shevchenko wrote:
> It seems commit d764593a "crypto: aesni - AVX and AVX2 version of AESNI-GCM
> encode and decode" breaks a build on x86_32 since it's designed only for
> x86_64. This patch makes a compilation unit conditional to CONFIG_64BIT and
> functions usage to CONFIG_X86_64.
Thanks for catching and fixing it.
Tim
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 17:41 ` Tim Chen
@ 2014-01-06 17:45 ` H. Peter Anvin
2014-01-06 17:57 ` Tim Chen
0 siblings, 1 reply; 15+ messages in thread
From: H. Peter Anvin @ 2014-01-06 17:45 UTC (permalink / raw)
To: Tim Chen, Andy Shevchenko; +Cc: linux-crypto, Herbert Xu, Borislav Petkov
Can the code be adjusted to compile for 32 bit x86 or is that pointless?
Tim Chen <tim.c.chen@linux.intel.com> wrote:
>
>
>On Mon, 2013-12-30 at 15:52 +0200, Andy Shevchenko wrote:
>> It seems commit d764593a "crypto: aesni - AVX and AVX2 version of
>AESNI-GCM
>> encode and decode" breaks a build on x86_32 since it's designed only
>for
>> x86_64. This patch makes a compilation unit conditional to
>CONFIG_64BIT and
>> functions usage to CONFIG_X86_64.
>
>Thanks for catching and fixing it.
>
>Tim
>>
>> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
--
Sent from my mobile phone. Please pardon brevity and lack of formatting.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 17:45 ` H. Peter Anvin
@ 2014-01-06 17:57 ` Tim Chen
2014-01-06 18:00 ` H. Peter Anvin
0 siblings, 1 reply; 15+ messages in thread
From: Tim Chen @ 2014-01-06 17:57 UTC (permalink / raw)
To: H. Peter Anvin; +Cc: Andy Shevchenko, linux-crypto, Herbert Xu, Borislav Petkov
On Mon, 2014-01-06 at 09:45 -0800, H. Peter Anvin wrote:
> Can the code be adjusted to compile for 32 bit x86 or is that pointless?
>
Code was optimized for wide registers. So it is only meant for x86_64.
Tim
> Tim Chen <tim.c.chen@linux.intel.com> wrote:
> >
> >
> >On Mon, 2013-12-30 at 15:52 +0200, Andy Shevchenko wrote:
> >> It seems commit d764593a "crypto: aesni - AVX and AVX2 version of
> >AESNI-GCM
> >> encode and decode" breaks a build on x86_32 since it's designed only
> >for
> >> x86_64. This patch makes a compilation unit conditional to
> >CONFIG_64BIT and
> >> functions usage to CONFIG_X86_64.
> >
> >Thanks for catching and fixing it.
> >
> >Tim
> >>
> >> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 17:57 ` Tim Chen
@ 2014-01-06 18:00 ` H. Peter Anvin
2014-01-06 18:10 ` Tim Chen
0 siblings, 1 reply; 15+ messages in thread
From: H. Peter Anvin @ 2014-01-06 18:00 UTC (permalink / raw)
To: Tim Chen; +Cc: Andy Shevchenko, linux-crypto, Herbert Xu, Borislav Petkov
On 01/06/2014 09:57 AM, Tim Chen wrote:
> On Mon, 2014-01-06 at 09:45 -0800, H. Peter Anvin wrote:
>> Can the code be adjusted to compile for 32 bit x86 or is that pointless?
>>
>
> Code was optimized for wide registers. So it is only meant for x86_64.
>
Aren't the "wide registers" the vector registers? Or are you also
relying on 64-bit integer registers (in which case we should just rename
the file to make it clear it is x86-64 only.)
-hpa
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 18:00 ` H. Peter Anvin
@ 2014-01-06 18:10 ` Tim Chen
2014-01-06 20:26 ` Borislav Petkov
0 siblings, 1 reply; 15+ messages in thread
From: Tim Chen @ 2014-01-06 18:10 UTC (permalink / raw)
To: H. Peter Anvin; +Cc: Andy Shevchenko, linux-crypto, Herbert Xu, Borislav Petkov
On Mon, 2014-01-06 at 10:00 -0800, H. Peter Anvin wrote:
> On 01/06/2014 09:57 AM, Tim Chen wrote:
> > On Mon, 2014-01-06 at 09:45 -0800, H. Peter Anvin wrote:
> >> Can the code be adjusted to compile for 32 bit x86 or is that pointless?
> >>
> >
> > Code was optimized for wide registers. So it is only meant for x86_64.
> >
>
> Aren't the "wide registers" the vector registers? Or are you also
> relying on 64-bit integer registers (in which case we should just rename
> the file to make it clear it is x86-64 only.)
>
> -hpa
>
>
Yes, the code is in the file named aesni_intel_avx.S. So it should
be clear that the code is meant for x86_64.
Tim
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 18:10 ` Tim Chen
@ 2014-01-06 20:26 ` Borislav Petkov
2014-01-06 21:21 ` H. Peter Anvin
0 siblings, 1 reply; 15+ messages in thread
From: Borislav Petkov @ 2014-01-06 20:26 UTC (permalink / raw)
To: Tim Chen; +Cc: H. Peter Anvin, Andy Shevchenko, linux-crypto, Herbert Xu
On Mon, Jan 06, 2014 at 10:10:55AM -0800, Tim Chen wrote:
> Yes, the code is in the file named aesni_intel_avx.S. So it should be
> clear that the code is meant for x86_64.
How do you deduce aesni_intel_avx.S is meant for x86_64 only from the
name?
Shouldn't it be called aesni_intel_avx-x86_64.S, as is the naming
convention in arch/x86/crypto/
?
--
Regards/Gruss,
Boris.
Sent from a fat crate under my desk. Formatting is fine.
--
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 20:26 ` Borislav Petkov
@ 2014-01-06 21:21 ` H. Peter Anvin
2014-01-06 23:39 ` Tim Chen
0 siblings, 1 reply; 15+ messages in thread
From: H. Peter Anvin @ 2014-01-06 21:21 UTC (permalink / raw)
To: Borislav Petkov, Tim Chen; +Cc: Andy Shevchenko, linux-crypto, Herbert Xu
On 01/06/2014 12:26 PM, Borislav Petkov wrote:
> On Mon, Jan 06, 2014 at 10:10:55AM -0800, Tim Chen wrote:
>> Yes, the code is in the file named aesni_intel_avx.S. So it should be
>> clear that the code is meant for x86_64.
>
> How do you deduce aesni_intel_avx.S is meant for x86_64 only from the
> name?
>
> Shouldn't it be called aesni_intel_avx-x86_64.S, as is the naming
> convention in arch/x86/crypto/
>
Quite.
-hpa
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 21:21 ` H. Peter Anvin
@ 2014-01-06 23:39 ` Tim Chen
2014-01-06 23:41 ` H. Peter Anvin
2014-01-09 9:03 ` Herbert Xu
0 siblings, 2 replies; 15+ messages in thread
From: Tim Chen @ 2014-01-06 23:39 UTC (permalink / raw)
To: H. Peter Anvin; +Cc: Borislav Petkov, Andy Shevchenko, linux-crypto, Herbert Xu
On Mon, 2014-01-06 at 13:21 -0800, H. Peter Anvin wrote:
> On 01/06/2014 12:26 PM, Borislav Petkov wrote:
> > On Mon, Jan 06, 2014 at 10:10:55AM -0800, Tim Chen wrote:
> >> Yes, the code is in the file named aesni_intel_avx.S. So it should be
> >> clear that the code is meant for x86_64.
> >
> > How do you deduce aesni_intel_avx.S is meant for x86_64 only from the
> > name?
> >
> > Shouldn't it be called aesni_intel_avx-x86_64.S, as is the naming
> > convention in arch/x86/crypto/
> >
>
> Quite.
>
> -hpa
>
>
Will renaming the file to aesni_intel_avx-x86_64.S make things clearer
now?
Tim
---cut---here---
>From 41656afcbd63ccb92357d4937a75629499f4fd4f Mon Sep 17 00:00:00 2001
From: Tim Chen <tim.c.chen@linux.intel.com>
Date: Mon, 6 Jan 2014 07:23:52 -0800
Subject: [PATCH] crypto: Rename aesni-intel_avx.S to indicate it only
supports x86_64
To: Herbert Xu <herbert@gondor.apana.org.au>, H. Peter Anvin <hpa@zytor.com>
Cc: Borislav Petkov <bp@alien8.de>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, linux-crypto@vger.kernel.org
We rename aesni-intel_avx.S to aesni-intel_avx-x86_64.S to indicate
that it is only used by x86_64 architecture.
---
arch/x86/crypto/Makefile | 2 +-
arch/x86/crypto/{aesni-intel_avx.S => aesni-intel_avx-x86_64.S} | 0
2 files changed, 1 insertion(+), 1 deletion(-)
rename arch/x86/crypto/{aesni-intel_avx.S => aesni-intel_avx-x86_64.S} (100%)
diff --git a/arch/x86/crypto/Makefile b/arch/x86/crypto/Makefile
index 188b993..6ba54d6 100644
--- a/arch/x86/crypto/Makefile
+++ b/arch/x86/crypto/Makefile
@@ -76,7 +76,7 @@ ifeq ($(avx2_supported),yes)
endif
aesni-intel-y := aesni-intel_asm.o aesni-intel_glue.o fpu.o
-aesni-intel-$(CONFIG_64BIT) += aesni-intel_avx.o
+aesni-intel-$(CONFIG_64BIT) += aesni-intel_avx-x86_64.o
ghash-clmulni-intel-y := ghash-clmulni-intel_asm.o ghash-clmulni-intel_glue.o
sha1-ssse3-y := sha1_ssse3_asm.o sha1_ssse3_glue.o
crc32c-intel-y := crc32c-intel_glue.o
diff --git a/arch/x86/crypto/aesni-intel_avx.S b/arch/x86/crypto/aesni-intel_avx-x86_64.S
similarity index 100%
rename from arch/x86/crypto/aesni-intel_avx.S
rename to arch/x86/crypto/aesni-intel_avx-x86_64.S
--
1.7.11.7
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 23:39 ` Tim Chen
@ 2014-01-06 23:41 ` H. Peter Anvin
2014-01-07 1:30 ` Herbert Xu
2014-01-09 9:03 ` Herbert Xu
1 sibling, 1 reply; 15+ messages in thread
From: H. Peter Anvin @ 2014-01-06 23:41 UTC (permalink / raw)
To: Tim Chen; +Cc: Borislav Petkov, Andy Shevchenko, linux-crypto, Herbert Xu
On 01/06/2014 03:39 PM, Tim Chen wrote:
>
> Will renaming the file to aesni_intel_avx-x86_64.S make things clearer
> now?
>
> Tim
Yes.
Acked-by: H. Peter Anvin <hpa@linux.intel.com>
Herbert, can you pick it up?
-hpa
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 23:41 ` H. Peter Anvin
@ 2014-01-07 1:30 ` Herbert Xu
0 siblings, 0 replies; 15+ messages in thread
From: Herbert Xu @ 2014-01-07 1:30 UTC (permalink / raw)
To: H. Peter Anvin; +Cc: Tim Chen, Borislav Petkov, Andy Shevchenko, linux-crypto
On Mon, Jan 06, 2014 at 03:41:51PM -0800, H. Peter Anvin wrote:
> On 01/06/2014 03:39 PM, Tim Chen wrote:
> >
> > Will renaming the file to aesni_intel_avx-x86_64.S make things clearer
> > now?
> >
> > Tim
>
> Yes.
>
> Acked-by: H. Peter Anvin <hpa@linux.intel.com>
>
> Herbert, can you pick it up?
Sure I'll apply this patch.
Thanks!
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-06 23:39 ` Tim Chen
2014-01-06 23:41 ` H. Peter Anvin
@ 2014-01-09 9:03 ` Herbert Xu
2014-01-09 16:57 ` Tim Chen
1 sibling, 1 reply; 15+ messages in thread
From: Herbert Xu @ 2014-01-09 9:03 UTC (permalink / raw)
To: Tim Chen; +Cc: H. Peter Anvin, Borislav Petkov, Andy Shevchenko, linux-crypto
On Mon, Jan 06, 2014 at 03:39:06PM -0800, Tim Chen wrote:
> On Mon, 2014-01-06 at 13:21 -0800, H. Peter Anvin wrote:
> > On 01/06/2014 12:26 PM, Borislav Petkov wrote:
> > > On Mon, Jan 06, 2014 at 10:10:55AM -0800, Tim Chen wrote:
> > >> Yes, the code is in the file named aesni_intel_avx.S. So it should be
> > >> clear that the code is meant for x86_64.
> > >
> > > How do you deduce aesni_intel_avx.S is meant for x86_64 only from the
> > > name?
> > >
> > > Shouldn't it be called aesni_intel_avx-x86_64.S, as is the naming
> > > convention in arch/x86/crypto/
> > >
> >
> > Quite.
> >
> > -hpa
> >
> >
>
> Will renaming the file to aesni_intel_avx-x86_64.S make things clearer
> now?
>
> Tim
>
> ---cut---here---
>
> >From 41656afcbd63ccb92357d4937a75629499f4fd4f Mon Sep 17 00:00:00 2001
> From: Tim Chen <tim.c.chen@linux.intel.com>
> Date: Mon, 6 Jan 2014 07:23:52 -0800
> Subject: [PATCH] crypto: Rename aesni-intel_avx.S to indicate it only
> supports x86_64
> To: Herbert Xu <herbert@gondor.apana.org.au>, H. Peter Anvin <hpa@zytor.com>
> Cc: Borislav Petkov <bp@alien8.de>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, linux-crypto@vger.kernel.org
>
> We rename aesni-intel_avx.S to aesni-intel_avx-x86_64.S to indicate
> that it is only used by x86_64 architecture.
Hmm, where is the signed-off-by line? Please resend. Thanks.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-09 9:03 ` Herbert Xu
@ 2014-01-09 16:57 ` Tim Chen
2014-01-15 3:43 ` Herbert Xu
0 siblings, 1 reply; 15+ messages in thread
From: Tim Chen @ 2014-01-09 16:57 UTC (permalink / raw)
To: Herbert Xu; +Cc: H. Peter Anvin, Borislav Petkov, Andy Shevchenko, linux-crypto
On Thu, 2014-01-09 at 17:03 +0800, Herbert Xu wrote:
> On Mon, Jan 06, 2014 at 03:39:06PM -0800, Tim Chen wrote:
> > On Mon, 2014-01-06 at 13:21 -0800, H. Peter Anvin wrote:
> > > On 01/06/2014 12:26 PM, Borislav Petkov wrote:
> > > > On Mon, Jan 06, 2014 at 10:10:55AM -0800, Tim Chen wrote:
> > > >> Yes, the code is in the file named aesni_intel_avx.S. So it should be
> > > >> clear that the code is meant for x86_64.
> > > >
> > > > How do you deduce aesni_intel_avx.S is meant for x86_64 only from the
> > > > name?
> > > >
> > > > Shouldn't it be called aesni_intel_avx-x86_64.S, as is the naming
> > > > convention in arch/x86/crypto/
> > > >
> > >
> > > Quite.
> > >
> > > -hpa
> > >
> > >
> >
> > Will renaming the file to aesni_intel_avx-x86_64.S make things clearer
> > now?
> >
> > Tim
> >
> > ---cut---here---
> >
> > >From 41656afcbd63ccb92357d4937a75629499f4fd4f Mon Sep 17 00:00:00 2001
> > From: Tim Chen <tim.c.chen@linux.intel.com>
> > Date: Mon, 6 Jan 2014 07:23:52 -0800
> > Subject: [PATCH] crypto: Rename aesni-intel_avx.S to indicate it only
> > supports x86_64
> > To: Herbert Xu <herbert@gondor.apana.org.au>, H. Peter Anvin <hpa@zytor.com>
> > Cc: Borislav Petkov <bp@alien8.de>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, linux-crypto@vger.kernel.org
> >
> > We rename aesni-intel_avx.S to aesni-intel_avx-x86_64.S to indicate
> > that it is only used by x86_64 architecture.
>
> Hmm, where is the signed-off-by line? Please resend. Thanks.
Oops, here it is again.
Tim
---cut---here---
>From 41656afcbd63ccb92357d4937a75629499f4fd4f Mon Sep 17 00:00:00 2001
From: Tim Chen <tim.c.chen@linux.intel.com>
Date: Mon, 6 Jan 2014 07:23:52 -0800
Subject: [PATCH] crypto: Rename aesni-intel_avx.S to indicate it only
supports x86_64
To: Herbert Xu <herbert@gondor.apana.org.au>, H. Peter Anvin <hpa@zytor.com>
Cc: Borislav Petkov <bp@alien8.de>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, linux-crypto@vger.kernel.org
We rename aesni-intel_avx.S to aesni-intel_avx-x86_64.S to indicate
that it is only used by x86_64 architecture.
---
arch/x86/crypto/Makefile | 2 +-
arch/x86/crypto/{aesni-intel_avx.S => aesni-intel_avx-x86_64.S} | 0
2 files changed, 1 insertion(+), 1 deletion(-)
rename arch/x86/crypto/{aesni-intel_avx.S => aesni-intel_avx-x86_64.S} (100%)
Signed-off-by: Tim Chen <tim.c.chen@linux.intel.com>
---
diff --git a/arch/x86/crypto/Makefile b/arch/x86/crypto/Makefile
index 188b993..6ba54d6 100644
--- a/arch/x86/crypto/Makefile
+++ b/arch/x86/crypto/Makefile
@@ -76,7 +76,7 @@ ifeq ($(avx2_supported),yes)
endif
aesni-intel-y := aesni-intel_asm.o aesni-intel_glue.o fpu.o
-aesni-intel-$(CONFIG_64BIT) += aesni-intel_avx.o
+aesni-intel-$(CONFIG_64BIT) += aesni-intel_avx-x86_64.o
ghash-clmulni-intel-y := ghash-clmulni-intel_asm.o ghash-clmulni-intel_glue.o
sha1-ssse3-y := sha1_ssse3_asm.o sha1_ssse3_glue.o
crc32c-intel-y := crc32c-intel_glue.o
diff --git a/arch/x86/crypto/aesni-intel_avx.S b/arch/x86/crypto/aesni-intel_avx-x86_64.S
similarity index 100%
rename from arch/x86/crypto/aesni-intel_avx.S
rename to arch/x86/crypto/aesni-intel_avx-x86_64.S
--
1.7.11.7
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v1] crypto: aesni - fix build on x86 (32bit)
2014-01-09 16:57 ` Tim Chen
@ 2014-01-15 3:43 ` Herbert Xu
0 siblings, 0 replies; 15+ messages in thread
From: Herbert Xu @ 2014-01-15 3:43 UTC (permalink / raw)
To: Tim Chen; +Cc: H. Peter Anvin, Borislav Petkov, Andy Shevchenko, linux-crypto
On Thu, Jan 09, 2014 at 08:57:42AM -0800, Tim Chen wrote:
>
> >From 41656afcbd63ccb92357d4937a75629499f4fd4f Mon Sep 17 00:00:00 2001
> From: Tim Chen <tim.c.chen@linux.intel.com>
> Date: Mon, 6 Jan 2014 07:23:52 -0800
> Subject: [PATCH] crypto: Rename aesni-intel_avx.S to indicate it only
> supports x86_64
> To: Herbert Xu <herbert@gondor.apana.org.au>, H. Peter Anvin <hpa@zytor.com>
> Cc: Borislav Petkov <bp@alien8.de>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, linux-crypto@vger.kernel.org
>
> We rename aesni-intel_avx.S to aesni-intel_avx-x86_64.S to indicate
> that it is only used by x86_64 architecture.
> ---
> arch/x86/crypto/Makefile | 2 +-
> arch/x86/crypto/{aesni-intel_avx.S => aesni-intel_avx-x86_64.S} | 0
> 2 files changed, 1 insertion(+), 1 deletion(-)
> rename arch/x86/crypto/{aesni-intel_avx.S => aesni-intel_avx-x86_64.S} (100%)
>
> Signed-off-by: Tim Chen <tim.c.chen@linux.intel.com>
Patch applied.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2014-01-15 3:44 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-30 13:52 [PATCH v1] crypto: aesni - fix build on x86 (32bit) Andy Shevchenko
2013-12-31 11:57 ` Herbert Xu
2014-01-06 17:41 ` Tim Chen
2014-01-06 17:45 ` H. Peter Anvin
2014-01-06 17:57 ` Tim Chen
2014-01-06 18:00 ` H. Peter Anvin
2014-01-06 18:10 ` Tim Chen
2014-01-06 20:26 ` Borislav Petkov
2014-01-06 21:21 ` H. Peter Anvin
2014-01-06 23:39 ` Tim Chen
2014-01-06 23:41 ` H. Peter Anvin
2014-01-07 1:30 ` Herbert Xu
2014-01-09 9:03 ` Herbert Xu
2014-01-09 16:57 ` Tim Chen
2014-01-15 3:43 ` Herbert Xu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.