11:06 a.m.
On Fri, Jun 19, 2020 at 5:16 PM Stephen Boyd <swboyd(a)google.com> wrote:
On Fri, Jun 19, 2020 at 9:47 AM kernel test robot <lkp(a)intel.com> wrote:
>
> tree: https://chromium.googlesource.com/chromiumos/third_party/kernel
chromeos-5.4
> head: e32b8d66db3100a239a87642f0a8025c396c0670
> commit: 7580417d2782bc89f27754eefaa2c9f0c8023a48 [9731/9999] UPSTREAM: ASoC: Fix
SND_SOC_ALL_CODECS imply misc fallout
> config: arm64-randconfig-r003-20200619 (attached as .config)
> compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project
487ca07fcc75d52755c9fe2ee05bcb3b6eeeec44)
> reproduce (this is a W=1 build):
> wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
> chmod +x ~/bin/make.cross
> # install arm64 cross compiling tool for clang build
> # apt-get install binutils-aarch64-linux-gnu
> git checkout 7580417d2782bc89f27754eefaa2c9f0c8023a48
> # save the attached .config to linux build tree
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp(a)intel.com>
>
> All errors (new ones prefixed by >>):
>
> WARNING: unmet direct dependencies detected for SND_SOC_WM5110
> Depends on SOUND && !UML && SND && SND_SOC &&
MFD_WM5110
> Selected by
> - SND_SOC_SAMSUNG_TM2_WM5110 && SOUND && !UML && SND
&& SND_SOC && SND_SOC_SAMSUNG && MFD_ARIZONA && I2C
&& SPI_MASTER && (GPIOLIB || COMPILE_TEST
> >> arch/arm64/kernel/vdso/vgettimeofday.c:9:5: error: no previous prototype
for function '__kernel_clock_gettime'
> int __kernel_clock_gettime(clockid_t clock,
> ^
> arch/arm64/kernel/vdso/vgettimeofday.c:9:1: note: declare 'static' if the
function is not intended to be used outside of this translation unit
> int __kernel_clock_gettime(clockid_t clock,
> ^
> static
> >> arch/arm64/kernel/vdso/vgettimeofday.c:15:5: error: no previous prototype
for function '__kernel_gettimeofday'
> int __kernel_gettimeofday(struct __kernel_old_timeval
> ^
> arch/arm64/kernel/vdso/vgettimeofday.c:15:1: note: declare 'static' if
the function is not intended to be used outside of this translation unit
> int __kernel_gettimeofday(struct __kernel_old_timeval
> ^
> static
> >> arch/arm64/kernel/vdso/vgettimeofday.c:21:5: error: no previous prototype
for function '__kernel_clock_getres'
> int __kernel_clock_getres(clockid_t clock_id,
> ^
> arch/arm64/kernel/vdso/vgettimeofday.c:21:1: note: declare 'static' if
the function is not intended to be used outside of this translation unit
> int __kernel_clock_getres(clockid_t clock_id,
> ^
> static
> 3 errors generated.
I've been seeing these errors upstream too. I don't know if there's
anything to fix besides making up some header file that confirms the
ABI that is being exposed to userspace and then including it here.
Yes, I noticed as well; that is why I didn't submit a CL. Lots of
other vdso functions seem to have the same problem. For example, I
don't see __cvdso_clock_gettime32() declared anywhere. Wonder why, and
why it doesn't generate similar warnings.
Guenter