Re: [PATCH 3.12 55/64] x86: vvar, fix excessive gcc-6 DECLARE_VVAR warnings

From: Andy Lutomirski
Date: Fri Feb 12 2016 - 11:42:28 EST


On Thu, Feb 11, 2016 at 5:59 AM, Jiri Slaby <jslaby@xxxxxxx> wrote:
> 3.12-stable review patch. If anyone has any objections, please let me know.
>
> ===============
>
> On 3.12, with gcc-6, I see a lot of:
> arch/x86/include/asm/vvar.h:33:28: warning: âvvaraddr_jiffiesâ defined but not used [-Wunused-const-variable]
> static type const * const vvaraddr_ ## name = \
> ^
> arch/x86/include/asm/vvar.h:46:1: note: in expansion of macro âDECLARE_VVARâ
> DECLARE_VVAR(0, volatile unsigned long, jiffies)
> ^~~~~~~~~~~~
>
> In upstream, this is fixed by ef721987ae (x86, vdso: Introduce VVAR
> marco for vdso32) and f40c330091 (x86, vdso: Move the vvar and hpet
> mappings next to the 64-bit vDSO). But this is not applicable to
> stable.
>
> So mark the vvar declaration as __maybe_unused and be done with it.
> This will generate it to the code only if it is used. I.e. the same as
> with gcc < 6.

LGTM.