Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) client-ip=23.128.96.35;
MIME-Version: 1.0
References: <202310270909.QBAOkiZP-lkp@intel.com>
In-Reply-To: <202310270909.QBAOkiZP-lkp@intel.com>
From:   Uros Bizjak <ubizjak@gmail.com>
Date:   Fri, 27 Oct 2023 09:46:58 +0200
Message-ID: <CAFULd4YbDKr-R5GOS44q+e=1qJNJXgBbvJ_gy1hA8G1VxzgVrA@mail.gmail.com>
Subject: Re: [tip:x86/percpu 21/22] arch/x86/include/asm/processor.h:522:24:
 sparse: sparse: dereference of noderef expression
To:     kernel test robot <lkp@intel.com>
Cc:     oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
        x86@kernel.org, Ingo Molnar <mingo@kernel.org>,
        Nadav Amit <namit@vmware.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Fri, Oct 27, 2023 at 4:01=E2=80=AFAM kernel test robot <lkp@intel.com> w=
rote:
>
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/p=
ercpu
> head:   0548eb067ed664b93043e033295ca71e3e706245
> commit: ed2f752e0e0a21d941ca0ee539ef3d4cd576bc5e [21/22] x86/percpu: Intr=
oduce const-qualified const_pcpu_hot to micro-optimize code generation
> config: x86_64-randconfig-122-20231027 (https://download.01.org/0day-ci/a=
rchive/20231027/202310270909.QBAOkiZP-lkp@intel.com/config)
> compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
> reproduce (this is a W=3D1 build): (https://download.01.org/0day-ci/archi=
ve/20231027/202310270909.QBAOkiZP-lkp@intel.com/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new vers=
ion of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202310270909.QBAOkiZP-lkp=
@intel.com/

Already fixed in [1].

[1] https://lore.kernel.org/lkml/20231024142830.3226-1-ubizjak@gmail.com/

Thanks,
Uros.

>
> sparse warnings: (new ones prefixed by >>)
>    kernel/entry/common.c: note: in included file (through include/linux/s=
ched.h, include/linux/context_tracking.h):
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    kernel/entry/common.c: note: in included file (through arch/x86/includ=
e/asm/cpufeature.h, arch/x86/include/asm/thread_info.h, include/linux/threa=
d_info.h, ...):
> >> arch/x86/include/asm/processor.h:522:24: sparse: sparse: dereference o=
f noderef expression
>    kernel/entry/common.c: note: in included file (through include/linux/s=
ched.h, include/linux/context_tracking.h):
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    kernel/entry/common.c: note: in included file (through arch/x86/includ=
e/asm/cpufeature.h, arch/x86/include/asm/thread_info.h, include/linux/threa=
d_info.h, ...):
> >> arch/x86/include/asm/processor.h:522:24: sparse: sparse: dereference o=
f noderef expression
>    kernel/entry/common.c: note: in included file (through include/linux/s=
ched.h, include/linux/context_tracking.h):
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    kernel/entry/common.c: note: in included file (through arch/x86/includ=
e/asm/cpufeature.h, arch/x86/include/asm/thread_info.h, include/linux/threa=
d_info.h, ...):
> >> arch/x86/include/asm/processor.h:522:24: sparse: sparse: dereference o=
f noderef expression
>    kernel/entry/common.c: note: in included file (through include/linux/s=
ched.h, include/linux/context_tracking.h):
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    kernel/entry/common.c: note: in included file (through arch/x86/includ=
e/asm/cpufeature.h, arch/x86/include/asm/thread_info.h, include/linux/threa=
d_info.h, ...):
> >> arch/x86/include/asm/processor.h:522:24: sparse: sparse: dereference o=
f noderef expression
>    kernel/entry/common.c: note: in included file (through include/linux/s=
ched.h, include/linux/context_tracking.h):
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    kernel/entry/common.c: note: in included file (through arch/x86/includ=
e/asm/cpufeature.h, arch/x86/include/asm/thread_info.h, include/linux/threa=
d_info.h, ...):
> >> arch/x86/include/asm/processor.h:522:24: sparse: sparse: dereference o=
f noderef expression
>    kernel/entry/common.c: note: in included file (through include/linux/s=
ched.h, include/linux/context_tracking.h):
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>    arch/x86/include/asm/current.h:46:24: sparse: sparse: dereference of n=
oderef expression
>
> vim +522 arch/x86/include/asm/processor.h
>
>    513
>    514  static __always_inline unsigned long current_top_of_stack(void)
>    515  {
>    516          /*
>    517           *  We can't read directly from tss.sp0: sp0 on x86_32 is=
 special in
>    518           *  and around vm86 mode and sp0 on x86_64 is special bec=
ause of the
>    519           *  entry trampoline.
>    520           */
>    521          if (IS_ENABLED(CONFIG_USE_X86_SEG_SUPPORT))
>  > 522                  return pcpu_hot.top_of_stack;
>    523
>    524          return this_cpu_read_stable(pcpu_hot.top_of_stack);
>    525  }
>    526
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki