Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754406Ab2FMTPE (ORCPT ); Wed, 13 Jun 2012 15:15:04 -0400 Received: from sd-mail-sa-01.sanoma.fi ([158.127.18.161]:41008 "EHLO sd-mail-sa-01.sanoma.fi" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752321Ab2FMTPC (ORCPT ); Wed, 13 Jun 2012 15:15:02 -0400 Message-ID: <20120613221458.21085iwxklwljhwc@www.81.fi> Date: Wed, 13 Jun 2012 22:14:58 +0300 From: Jussi Kivilinna To: Johannes Goetzfried Cc: Randy Dunlap , Stephen Rothwell , linux-next@vger.kernel.org, LKML , linux-crypto@vger.kernel.org, David Miller , Herbert Xu Subject: Re: linux-next: Tree for Jun 13 (crypto) References: <20120613155440.14672516ffaf5b01f91bb0a2@canb.auug.org.au> <4FD8BB95.4000307@xenotime.net> <20120613200317.20733xjrnmv47y04@www.81.fi> <20120613190540.GY17705@kronos.redsun> In-Reply-To: <20120613190540.GY17705@kronos.redsun> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; DelSp="Yes"; format="flowed" Content-Disposition: inline Content-Transfer-Encoding: 7bit User-Agent: Internet Messaging Program (IMP) H3 (4.3.7) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2099 Lines: 49 Quoting Johannes Goetzfried : > On Wed, Jun 13, 2012 at 08:03:17PM +0300, Jussi Kivilinna wrote: >> >arch/x86/crypto/serpent-avx-x86_64.o: In function `serpent_xts_decrypt': >> >(.text+0x856c): multiple definition of `serpent_xts_decrypt' >> >arch/x86/crypto/serpent-sse2-x86_64.o:(.text+0xae8): first defined here >> >> Reverting commit e2f4d85d5b2049a90859aa10de914ed31a4f17a4, "crypto: >> serpent - add common glue code for avx and sse2 implementation", >> should help for most. >> >> >arch/x86/crypto/serpent-avx-x86_64.o: In function `serpent_dec_blk_8way': >> >(.text+0x3d70): multiple definition of `serpent_dec_blk_8way' >> >arch/x86/crypto/serpent-avx-x86_64.o: In function `__serpent_enc_blk_8way': >> >(.text+0x0): multiple definition of `__serpent_enc_blk_8way' >> >> Attached patch should solve these (patch depends on above revert). > > I'm really sorry about this, it shouldn't have happened. The most easiest > solution to this problem would be to make the choice between > CRYPTO_SERPENT_SSE2_X86_64 and CRYPTO_SERPENT_AVX_X86_64 an > exclusive-or choice > in crypto/Kconfig. I don't think there is an use-case, where somebody would > want to link both implementations into the same kernel image or even > load both > implementations as module. This would avoid duplicating the glue-code again. I'd think that most common use-case is distro-kernel, where all and everything is enabled by default. > > Another possibility would be to add a serpent-common-x86_64 module, but this > wouldn't be a trivial change and imo a little bit too much just to avoid > duplicating this code. I'm working on patch-set that takes common glue code from serpent-sse2, serpent-avx, twofish-3way, twofish-avx and camellia-x86_64, to shared ablk_helper and glue_helper modules. -Jussi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/