Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1432640
| From | Herbert Xu <herbert@gondor.apana.org.au> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] crypto: ux500: do not build with -O0 |
| Date | 2016-06-28 10:40 +0200 |
| Message-ID | <rOWIO-4s2-27@gated-at.bofh.it> (permalink) |
| References | <rOARY-6GU-23@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Mon, Jun 27, 2016 at 11:17:40AM +0200, Arnd Bergmann wrote:
> The ARM allmodconfig build currently warngs because of the
> ux500 crypto driver not working well with the jump label
> implementation that we started using for dynamic debug, which
> breaks building with 'gcc -O0':
>
> In file included from /git/arm-soc/include/linux/jump_label.h:105:0,
> from /git/arm-soc/include/linux/dynamic_debug.h:5,
> from /git/arm-soc/include/linux/printk.h:289,
> from /git/arm-soc/include/linux/kernel.h:13,
> from /git/arm-soc/include/linux/clk.h:16,
> from /git/arm-soc/drivers/crypto/ux500/hash/hash_core.c:16:
> /git/arm-soc/arch/arm/include/asm/jump_label.h: In function 'hash_set_dma_transfer':
> /git/arm-soc/arch/arm/include/asm/jump_label.h:13:7: error: asm operand 0 probably doesn't match constraints [-Werror]
> asm_volatile_goto("1:\n\t"
>
> Turning off compiler optimizations has never really been supported
> here, and it's only used when debugging the driver. I have not found
> a good reason for doing this here, other than a misguided attempt
> to produce more readable assembly output. Also, the driver is only
> used in obsolete hardware that almost certainly nobody will spend
> time debugging any more.
>
> This just removes the -O0 flag from the compiler options.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Patch applied. 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
Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread
[PATCH] crypto: ux500: do not build with -O0 Arnd Bergmann <arnd@arndb.de> - 2016-06-27 11:20 +0200 Re: [PATCH] crypto: ux500: do not build with -O0 Herbert Xu <herbert@gondor.apana.org.au> - 2016-06-28 10:40 +0200
csiph-web