diff mbox

lib: sw842: select crc32

Message ID 4168537.6I1FVkp3DA@wuerfel
State New
Headers show

Commit Message

Arnd Bergmann Jan. 13, 2016, 10:24 p.m. UTC
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 <arnd@arndb.de>

Fixes: 2da572c959dd ("lib: add software 842 compression/decompression")


--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Herbert Xu Jan. 18, 2016, 3:24 p.m. UTC | #1
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 <arnd@arndb.de>

> Fixes: 2da572c959dd ("lib: add software 842 compression/decompression")


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
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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