2016-01-13 22:24:41

by Arnd Bergmann

[permalink] [raw]
Subject: lib: sw842: select crc32

The sw842 library code was merged in linux-4.1 and causes a very rare randconfig
failure when CONFIG_CRC32 is not set:

lib/built-in.o: In function `sw842_compress':
oid_registry.c:(.text+0x12ddc): undefined reference to `crc32_be'
lib/built-in.o: In function `sw842_decompress':
oid_registry.c:(.text+0x137e4): undefined reference to `crc32_be'

This adds an explict 'select CRC32' statement, similar to what the other users
of the crc32 code have. In practice, CRC32 is always enabled anyway because
over 100 other symbols select it.

Signed-off-by: Arnd Bergmann <[email protected]>
Fixes: 2da572c959dd ("lib: add software 842 compression/decompression")

diff --git a/lib/Kconfig b/lib/Kconfig
index 1ce5af9cc418..ee38a3fd2814 100644
--- a/lib/Kconfig
+++ b/lib/Kconfig
@@ -217,9 +217,11 @@ config RANDOM32_SELFTEST
# compression support is select'ed if needed
#
config 842_COMPRESS
+ select CRC32
tristate

config 842_DECOMPRESS
+ select CRC32
tristate

config ZLIB_INFLATE


2016-01-14 03:36:57

by Dan Streetman

[permalink] [raw]
Subject: Re: lib: sw842: select crc32

On Wed, Jan 13, 2016 at 5:24 PM, Arnd Bergmann <[email protected]> wrote:
> The sw842 library code was merged in linux-4.1 and causes a very rare randconfig
> failure when CONFIG_CRC32 is not set:
>
> lib/built-in.o: In function `sw842_compress':
> oid_registry.c:(.text+0x12ddc): undefined reference to `crc32_be'
> lib/built-in.o: In function `sw842_decompress':
> oid_registry.c:(.text+0x137e4): undefined reference to `crc32_be'
>
> This adds an explict 'select CRC32' statement, similar to what the other users
> of the crc32 code have. In practice, CRC32 is always enabled anyway because
> over 100 other symbols select it.
>
> Signed-off-by: Arnd Bergmann <[email protected]>
> Fixes: 2da572c959dd ("lib: add software 842 compression/decompression")

Acked-by: Dan Streetman <[email protected]>

>
> diff --git a/lib/Kconfig b/lib/Kconfig
> index 1ce5af9cc418..ee38a3fd2814 100644
> --- a/lib/Kconfig
> +++ b/lib/Kconfig
> @@ -217,9 +217,11 @@ config RANDOM32_SELFTEST
> # compression support is select'ed if needed
> #
> config 842_COMPRESS
> + select CRC32
> tristate
>
> config 842_DECOMPRESS
> + select CRC32
> tristate
>
> config ZLIB_INFLATE
>

2016-01-18 15:25:08

by Herbert Xu

[permalink] [raw]
Subject: Re: lib: sw842: select crc32

On Wed, Jan 13, 2016 at 11:24:02PM +0100, Arnd Bergmann wrote:
> The sw842 library code was merged in linux-4.1 and causes a very rare randconfig
> failure when CONFIG_CRC32 is not set:
>
> lib/built-in.o: In function `sw842_compress':
> oid_registry.c:(.text+0x12ddc): undefined reference to `crc32_be'
> lib/built-in.o: In function `sw842_decompress':
> oid_registry.c:(.text+0x137e4): undefined reference to `crc32_be'
>
> This adds an explict 'select CRC32' statement, similar to what the other users
> of the crc32 code have. In practice, CRC32 is always enabled anyway because
> over 100 other symbols select it.
>
> Signed-off-by: Arnd Bergmann <[email protected]>
> Fixes: 2da572c959dd ("lib: add software 842 compression/decompression")

Patch applied. Thanks!
--
Email: Herbert Xu <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt