[PATCH] UBUNTU: SAUCE: x86/kernel/setup.c: re-add include of module.h

[Seth Forshee]

On Thu, Jun 01, 2017 at 09:45:01AM +0200, Rasmus Villemoes wrote:
> This reverts the part of
> 1a4573084ebb1298a90694259f853c1327a62079 (x86/kernel: Audit and remove
> any unnecessary uses of module.h) that touches x86/kernel/setup.c. While
> nothing in mainline's version of the file uses anything from module.h,
> Ubuntu's kernel has a call to enforce_signed_modules().
> 
> This goes unnoticed with the defalt Ubuntu config, because the file
> includes linux/kexec.h, which in turn includes linux/module.h. However,
> that #include is guarded by CONFIG_KEXEC_CORE, so if one disables
> CONFIG_KEXEC and CONFIG_KEXEC_FILE, the result is
> 
> arch/x86/kernel/setup.c:1149:3: error: implicit declaration of function
> ‘enforce_signed_modules’ [-Werror=implicit-function-declaration]
> 
> Signed-off-by: Rasmus Villemoes <linux at rasmusvillemoes.dk>

Thanks for the patch. I think this looks fine, but you're missing a
few things:

 - You need to specify which releases you are targeting with the patch. 

 - All patches for stable releases require a corresponding bug, and the
   commit message for the patch must include a BugLink line indicating
   which bug the patch fixes, i.e.
   
     BugLink: http://bugs.launchpad.net/bugs/XXXXXX

 - The bug description should follow the SRU bug template (see
   https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template).

Thanks,
Seth