[PATCH] crypto: rng: Remove unused function __crypto_rng_cast()

2017-05-22 Thread Matthias Kaehlcke
This fixes the following warning when building with clang: crypto/rng.c:35:34: error: unused function '__crypto_rng_cast' [-Werror,-Wunused-function] Signed-off-by: Matthias Kaehlcke --- crypto/rng.c | 5 - 1 file changed, 5 deletions(-) diff --git a/crypto/rng.c b/crypto/r

Re: [PATCH] crypto: arm64/sha - avoid non-standard inline asm tricks

2017-05-17 Thread Matthias Kaehlcke
El Wed, Apr 26, 2017 at 05:11:32PM +0100 Ard Biesheuvel ha dit: > Replace the inline asm which exports struct offsets as ELF symbols > with proper const variables exposing the same values. This works > around an issue with Clang which does not interpret the "i" (or "I") > constraints in the same w

Re: [PATCH] crypto: arm64/sha - avoid non-standard inline asm tricks

2017-04-26 Thread Matthias Kaehlcke
t;) > constraints in the same way as GCC. > > Signed-off-by: Ard Biesheuvel Tested-by: Matthias Kaehlcke Thanks! Matthias

Re: [PATCH v2] crypto: arm64/sha: Add constant operand modifier to ASM_EXPORT

2017-04-25 Thread Matthias Kaehlcke
El Tue, Apr 25, 2017 at 07:06:30PM +0100 Ard Biesheuvel ha dit: > On 25 April 2017 at 18:39, Matthias Kaehlcke wrote: > > Hi, > > > > El Tue, Apr 18, 2017 at 04:35:02PM +0100 Ard Biesheuvel ha dit: > > > >> On 18 April 2017 at 15:47, Paul Gortmaker > &g

Re: [PATCH v2] crypto: arm64/sha: Add constant operand modifier to ASM_EXPORT

2017-04-25 Thread Matthias Kaehlcke
Hi, El Tue, Apr 18, 2017 at 04:35:02PM +0100 Ard Biesheuvel ha dit: > On 18 April 2017 at 15:47, Paul Gortmaker > wrote: > > On Wed, Apr 5, 2017 at 2:34 PM, Matthias Kaehlcke wrote: > >> The operand is an integer constant, make the constness explicit by > >>

Re: [PATCH v2] crypto: arm64/sha: Add constant operand modifier to ASM_EXPORT

2017-04-18 Thread Matthias Kaehlcke
El Tue, Apr 18, 2017 at 04:35:02PM +0100 Ard Biesheuvel ha dit: > On 18 April 2017 at 15:47, Paul Gortmaker > wrote: > > On Wed, Apr 5, 2017 at 2:34 PM, Matthias Kaehlcke wrote: > >> The operand is an integer constant, make the constness explicit by > >> adding th

Re: [PATCH 3/7] x86, LLVM: suppress clang warnings about unaligned accesses

2017-04-13 Thread Matthias Kaehlcke
El Mon, Apr 03, 2017 at 04:01:58PM -0700 Matthias Kaehlcke ha dit: > El Fri, Mar 17, 2017 at 04:50:19PM -0700 h...@zytor.com ha dit: > > > On March 16, 2017 5:15:16 PM PDT, Michael Davidson wrote: > > >Suppress clang warnings about potential unaliged accesses > > &g

Re: [PATCH 2/7] Makefile, x86, LLVM: disable unsupported optimization flags

2017-04-05 Thread Matthias Kaehlcke
Hi Masahiro, El Thu, Apr 06, 2017 at 03:08:26AM +0900 Masahiro Yamada ha dit: > 2017-03-17 9:15 GMT+09:00 Michael Davidson : > > Unfortunately, while clang generates a warning about these flags > > being unsupported it still exits with a status of 0 so we have > > to explicitly disable them inste

[PATCH v2] crypto: arm64/sha: Add constant operand modifier to ASM_EXPORT

2017-04-05 Thread Matthias Kaehlcke
ADD instruction", AArch64) to 'i' ("An immediate integer operand"). Based-on-patch-from: Greg Hackmann Signed-off-by: Greg Hackmann Signed-off-by: Matthias Kaehlcke --- Changes in v2: - Changed operand constraint from I to i - Updated commit message - Changed 'From'

Re: [PATCH] crypto: arm64/sha: use %c constraint code in ASM_EXPORT

2017-04-05 Thread Matthias Kaehlcke
Hoi Ard! El Wed, Apr 05, 2017 at 06:08:37PM +0100 Ard Biesheuvel ha dit: > On 5 April 2017 at 17:56, Matthias Kaehlcke wrote: > > From: Greg Hackmann > > > > The current definition of ASM_EXPORT doesn't work properly with clang, > > according to https://bugs.ll

[PATCH] crypto: arm64/sha: use %c constraint code in ASM_EXPORT

2017-04-05 Thread Matthias Kaehlcke
clang. From: Greg Hackmann Commit-message-by: Matthias Kaehlcke Signed-off-by: Greg Hackmann Signed-off-by: Matthias Kaehlcke --- arch/arm64/crypto/sha1-ce-glue.c | 2 +- arch/arm64/crypto/sha2-ce-glue.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/crypto/sh

Re: [PATCH 7/7] crypto, x86, LLVM: aesni - fix token pasting

2017-04-03 Thread Matthias Kaehlcke
El Thu, Mar 16, 2017 at 05:15:20PM -0700 Michael Davidson ha dit: > aes_ctrby8_avx-x86_64.S uses the C preprocessor for token pasting > of character sequences that are not valid preprocessor tokens. > While this is allowed when preprocessing assembler files it exposes > an incompatibilty between t

Re: [PATCH 3/7] x86, LLVM: suppress clang warnings about unaligned accesses

2017-04-03 Thread Matthias Kaehlcke
El Fri, Mar 17, 2017 at 04:50:19PM -0700 h...@zytor.com ha dit: > On March 16, 2017 5:15:16 PM PDT, Michael Davidson wrote: > >Suppress clang warnings about potential unaliged accesses > >to members in packed structs. This gets rid of almost 10,000 > >warnings about accesses to the ring 0 stack p

Re: [PATCH 1/7] Makefile, LLVM: add -no-integrated-as to KBUILD_[AC]FLAGS

2017-04-03 Thread Matthias Kaehlcke
El Thu, Mar 16, 2017 at 05:15:14PM -0700 Michael Davidson ha dit: > Add -no-integrated-as to KBUILD_AFLAGS and KBUILD_CFLAGS > for clang. > > Signed-off-by: Michael Davidson > --- > Makefile | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Makefile b/Makefile > index b841fb36beb2..b2