• Paul Gortmaker's avatar
    x86, crypto: Restore MODULE_LICENSE() to glue_helper.c so it loads · a47177d3
    Paul Gortmaker authored
    In commit:
    
      eb008eb6 ("x86: Audit and remove any remaining unnecessary uses of module.h")
    
    ... we looked for instances of module.h that were not supporting anything
    more than exported symbols.
    
    To facilitate the exchange of module.h to the much smaller export.h
    we occasionally remove tags like MODULE_AUTHOR() etc. which in the case
    of built in files, are no-ops and hence that is fine, assuming the
    info is already in the comments at the top of the file..
    
    However the error here is that I overlooked that this file was used
    not as a driver, but as a library of functions, and hence has no
    explicit modular linkage functions or similar, making it _appear_
    non-modular.  We can see that in retrospect with:
    
      arch/x86/crypto/Makefile:obj-$(CONFIG_CRYPTO_GLUE_HELPER_X86) += glue_helper.o
    
      crypto/Kconfig:config CRYPTO_GLUE_HELPER_X86
      crypto/Kconfig: tristate
    
    Since we removed what was an active MODULE_LICENSE(), the module failed
    to load and then automated testing showed the missing glue helpers as:
    
      glue_helper: Unknown symbol blkcipher_walk_done (err 0)
      glue_helper: Unknown symbol blkcipher_walk_virt (err 0)
      glue_helper: Unknown symbol kernel_fpu_end (err 0)
      glue_helper: Unknown symbol kernel_fpu_begin (err 0)
      glue_helper: Unknown symbol blkcipher_walk_virt_block (err 0)
    
    So we do a partial revert of that change to just this one file, and
    watch for similar MODULE_LICENSE() only cases in future audits.
    Reported-by: default avatarkernel test robot <xiaolong.ye@intel.com>
    Signed-off-by: default avatarPaul Gortmaker <paul.gortmaker@windriver.com>
    Cc: David S. Miller <davem@davemloft.net>
    Cc: Herbert Xu <herbert@gondor.apana.org.au>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Stephen Rothwell <sfr@canb.auug.org.au>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: linux-crypto@vger.kernel.org
    Cc: lkp@01.org
    Fixes: eb008eb6 ("x86: Audit and remove any remaining unnecessary uses of module.h")
    Link: http://lkml.kernel.org/r/20160719144243.GK21225@windriver.comSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
    a47177d3
glue_helper.c 10.1 KB