All of lore.kernel.org
 help / color / mirror / Atom feed
* arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm'
@ 2020-09-11 11:43 ` kernel test robot
  0 siblings, 0 replies; 8+ messages in thread
From: kernel test robot @ 2020-09-11 11:43 UTC (permalink / raw)
  To: Junaid Shahid
  Cc: Paolo Bonzini, Thomas Gleixner, kbuild-all, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 2423 bytes --]

Hi Junaid,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
date:   10 months ago
config: powerpc-defconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
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
        git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
     633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
         |            ^~~~~~~~~~~~~~~~~~~~~
>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
     642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
         |             ^~~~~~~~~~~~~~~~~~~~~~~
   cc1: all warnings being treated as errors

# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1aa9b9572b10529c2e64e2b8f44025d86e124308
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus xlnx/master
git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c

   628	
   629	/*
   630	 * Called after the VM is otherwise initialized, but just before adding it to
   631	 * the vm_list.
   632	 */
 > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
   634	{
   635		return 0;
   636	}
   637	
   638	/*
   639	 * Called just after removing the VM from the vm_list, but before doing any
   640	 * other destruction.
   641	 */
 > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
   643	{
   644	}
   645	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 25606 bytes --]

[-- Attachment #3: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm'
@ 2020-09-11 11:43 ` kernel test robot
  0 siblings, 0 replies; 8+ messages in thread
From: kernel test robot @ 2020-09-11 11:43 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 2483 bytes --]

Hi Junaid,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
date:   10 months ago
config: powerpc-defconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
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
        git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
     633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
         |            ^~~~~~~~~~~~~~~~~~~~~
>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
     642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
         |             ^~~~~~~~~~~~~~~~~~~~~~~
   cc1: all warnings being treated as errors

# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1aa9b9572b10529c2e64e2b8f44025d86e124308
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus xlnx/master
git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c

   628	
   629	/*
   630	 * Called after the VM is otherwise initialized, but just before adding it to
   631	 * the vm_list.
   632	 */
 > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
   634	{
   635		return 0;
   636	}
   637	
   638	/*
   639	 * Called just after removing the VM from the vm_list, but before doing any
   640	 * other destruction.
   641	 */
 > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
   643	{
   644	}
   645	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 25606 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm'
  2020-09-11 11:43 ` kernel test robot
  (?)
@ 2020-09-12  6:52   ` Paolo Bonzini
  -1 siblings, 0 replies; 8+ messages in thread
From: Paolo Bonzini @ 2020-09-12  6:52 UTC (permalink / raw)
  To: kernel test robot, Junaid Shahid; +Cc: kbuild-all, kvm-ppc, linux-arm-kernel

On 11/09/20 13:43, kernel test robot wrote:
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>      633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>          |            ^~~~~~~~~~~~~~~~~~~~~
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>      642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>          |             ^~~~~~~~~~~~~~~~~~~~~~~
>    cc1: all warnings being treated as errors

This makes little sense, the prototypes are in include/linux/kvm_host.h
Also this code is not architecture-dependent...

kvm-ppc guys, can you reproduce it?

Paolo

> 
> 
> Hi Junaid,
> 
> FYI, the error/warning still remains.
> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
> commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
> date:   10 months ago
> config: powerpc-defconfig (attached as .config)
> compiler: powerpc64-linux-gcc (GCC) 9.3.0
> 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
>         git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>      633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>          |            ^~~~~~~~~~~~~~~~~~~~~
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>      642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>          |             ^~~~~~~~~~~~~~~~~~~~~~~
>    cc1: all warnings being treated as errors
> 
> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1aa9b9572b10529c2e64e2b8f44025d86e124308
> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> git fetch --no-tags linus xlnx/master
> git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
> vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c
> 
>    628	
>    629	/*
>    630	 * Called after the VM is otherwise initialized, but just before adding it to
>    631	 * the vm_list.
>    632	 */
>  > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>    634	{
>    635		return 0;
>    636	}
>    637	
>    638	/*
>    639	 * Called just after removing the VM from the vm_list, but before doing any
>    640	 * other destruction.
>    641	 */
>  > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>    643	{
>    644	}
>    645	
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
> 


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm'
@ 2020-09-12  6:52   ` Paolo Bonzini
  0 siblings, 0 replies; 8+ messages in thread
From: Paolo Bonzini @ 2020-09-12  6:52 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 3396 bytes --]

On 11/09/20 13:43, kernel test robot wrote:
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>      633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>          |            ^~~~~~~~~~~~~~~~~~~~~
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>      642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>          |             ^~~~~~~~~~~~~~~~~~~~~~~
>    cc1: all warnings being treated as errors

This makes little sense, the prototypes are in include/linux/kvm_host.h
Also this code is not architecture-dependent...

kvm-ppc guys, can you reproduce it?

Paolo

> 
> 
> Hi Junaid,
> 
> FYI, the error/warning still remains.
> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
> commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
> date:   10 months ago
> config: powerpc-defconfig (attached as .config)
> compiler: powerpc64-linux-gcc (GCC) 9.3.0
> 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
>         git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>      633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>          |            ^~~~~~~~~~~~~~~~~~~~~
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>      642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>          |             ^~~~~~~~~~~~~~~~~~~~~~~
>    cc1: all warnings being treated as errors
> 
> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1aa9b9572b10529c2e64e2b8f44025d86e124308
> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> git fetch --no-tags linus xlnx/master
> git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
> vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c
> 
>    628	
>    629	/*
>    630	 * Called after the VM is otherwise initialized, but just before adding it to
>    631	 * the vm_list.
>    632	 */
>  > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>    634	{
>    635		return 0;
>    636	}
>    637	
>    638	/*
>    639	 * Called just after removing the VM from the vm_list, but before doing any
>    640	 * other destruction.
>    641	 */
>  > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>    643	{
>    644	}
>    645	
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch
@ 2020-09-12  6:52   ` Paolo Bonzini
  0 siblings, 0 replies; 8+ messages in thread
From: Paolo Bonzini @ 2020-09-12  6:52 UTC (permalink / raw)
  To: kernel test robot, Junaid Shahid; +Cc: kbuild-all, kvm-ppc, linux-arm-kernel

On 11/09/20 13:43, kernel test robot wrote:
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>      633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>          |            ^~~~~~~~~~~~~~~~~~~~~
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>      642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>          |             ^~~~~~~~~~~~~~~~~~~~~~~
>    cc1: all warnings being treated as errors

This makes little sense, the prototypes are in include/linux/kvm_host.h
Also this code is not architecture-dependent...

kvm-ppc guys, can you reproduce it?

Paolo

> 
> 
> Hi Junaid,
> 
> FYI, the error/warning still remains.
> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
> commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
> date:   10 months ago
> config: powerpc-defconfig (attached as .config)
> compiler: powerpc64-linux-gcc (GCC) 9.3.0
> 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
>         git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>      633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>          |            ^~~~~~~~~~~~~~~~~~~~~
>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>      642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>          |             ^~~~~~~~~~~~~~~~~~~~~~~
>    cc1: all warnings being treated as errors
> 
> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id\x1aa9b9572b10529c2e64e2b8f44025d86e124308
> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> git fetch --no-tags linus xlnx/master
> git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
> vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c
> 
>    628	
>    629	/*
>    630	 * Called after the VM is otherwise initialized, but just before adding it to
>    631	 * the vm_list.
>    632	 */
>  > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>    634	{
>    635		return 0;
>    636	}
>    637	
>    638	/*
>    639	 * Called just after removing the VM from the vm_list, but before doing any
>    640	 * other destruction.
>    641	 */
>  > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>    643	{
>    644	}
>    645	
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [kbuild-all] Re: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm'
  2020-09-12  6:52   ` arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' Paolo Bonzini
  (?)
@ 2020-09-18  5:46     ` Rong Chen
  -1 siblings, 0 replies; 8+ messages in thread
From: Rong Chen @ 2020-09-18  5:46 UTC (permalink / raw)
  To: Paolo Bonzini, kernel test robot, Junaid Shahid
  Cc: kbuild-all, kvm-ppc, linux-arm-kernel



On 9/12/20 2:52 PM, Paolo Bonzini wrote:
> On 11/09/20 13:43, kernel test robot wrote:
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>>       633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>           |            ^~~~~~~~~~~~~~~~~~~~~
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>>       642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>           |             ^~~~~~~~~~~~~~~~~~~~~~~
>>     cc1: all warnings being treated as errors
> This makes little sense, the prototypes are in include/linux/kvm_host.h
> Also this code is not architecture-dependent...
>
> kvm-ppc guys, can you reproduce it?

Hi Paolo,

The error can be reproduced with W=1:

$ make W=1 
CROSS_COMPILE=/home/nfs/0day/gcc-9.3.0-nolibc/powerpc64-linux/bin/powerpc64-linux- 
ARCH=powerpc M=arch/powerpc/kvm | grep kvm_main
   CC [M]  arch/powerpc/kvm/../../../virt/kvm/kvm_main.o
arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: warning: no 
previous prototype for 'kvm_arch_post_init_vm' [-Wmissing-prototypes]
   633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
       |            ^~~~~~~~~~~~~~~~~~~~~
arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: warning: no 
previous prototype for 'kvm_arch_pre_destroy_vm' [-Wmissing-prototypes]
   642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
       |             ^~~~~~~~~~~~~~~~~~~~~~~

Best Regards,
Rong Chen

>
> Paolo
>
>>
>> Hi Junaid,
>>
>> FYI, the error/warning still remains.
>>
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>> head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
>> commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
>> date:   10 months ago
>> config: powerpc-defconfig (attached as .config)
>> compiler: powerpc64-linux-gcc (GCC) 9.3.0
>> 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
>>          git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>>          # save the attached .config to linux build tree
>>          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc
>>
>> If you fix the issue, kindly add following tag as appropriate
>> Reported-by: kernel test robot <lkp@intel.com>
>>
>> All errors (new ones prefixed by >>):
>>
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>>       633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>           |            ^~~~~~~~~~~~~~~~~~~~~
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>>       642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>           |             ^~~~~~~~~~~~~~~~~~~~~~~
>>     cc1: all warnings being treated as errors
>>
>> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1aa9b9572b10529c2e64e2b8f44025d86e124308
>> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>> git fetch --no-tags linus xlnx/master
>> git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>> vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c
>>
>>     628	
>>     629	/*
>>     630	 * Called after the VM is otherwise initialized, but just before adding it to
>>     631	 * the vm_list.
>>     632	 */
>>   > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>     634	{
>>     635		return 0;
>>     636	}
>>     637	
>>     638	/*
>>     639	 * Called just after removing the VM from the vm_list, but before doing any
>>     640	 * other destruction.
>>     641	 */
>>   > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>     643	{
>>     644	}
>>     645	
>>
>> ---
>> 0-DAY CI Kernel Test Service, Intel Corporation
>> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
>>
> _______________________________________________
> kbuild-all mailing list -- kbuild-all@lists.01.org
> To unsubscribe send an email to kbuild-all-leave@lists.01.org


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm'
@ 2020-09-18  5:46     ` Rong Chen
  0 siblings, 0 replies; 8+ messages in thread
From: Rong Chen @ 2020-09-18  5:46 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 4532 bytes --]



On 9/12/20 2:52 PM, Paolo Bonzini wrote:
> On 11/09/20 13:43, kernel test robot wrote:
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>>       633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>           |            ^~~~~~~~~~~~~~~~~~~~~
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>>       642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>           |             ^~~~~~~~~~~~~~~~~~~~~~~
>>     cc1: all warnings being treated as errors
> This makes little sense, the prototypes are in include/linux/kvm_host.h
> Also this code is not architecture-dependent...
>
> kvm-ppc guys, can you reproduce it?

Hi Paolo,

The error can be reproduced with W=1:

$ make W=1 
CROSS_COMPILE=/home/nfs/0day/gcc-9.3.0-nolibc/powerpc64-linux/bin/powerpc64-linux- 
ARCH=powerpc M=arch/powerpc/kvm | grep kvm_main
   CC [M]  arch/powerpc/kvm/../../../virt/kvm/kvm_main.o
arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: warning: no 
previous prototype for 'kvm_arch_post_init_vm' [-Wmissing-prototypes]
   633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
       |            ^~~~~~~~~~~~~~~~~~~~~
arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: warning: no 
previous prototype for 'kvm_arch_pre_destroy_vm' [-Wmissing-prototypes]
   642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
       |             ^~~~~~~~~~~~~~~~~~~~~~~

Best Regards,
Rong Chen

>
> Paolo
>
>>
>> Hi Junaid,
>>
>> FYI, the error/warning still remains.
>>
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>> head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
>> commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
>> date:   10 months ago
>> config: powerpc-defconfig (attached as .config)
>> compiler: powerpc64-linux-gcc (GCC) 9.3.0
>> 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
>>          git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>>          # save the attached .config to linux build tree
>>          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc
>>
>> If you fix the issue, kindly add following tag as appropriate
>> Reported-by: kernel test robot <lkp@intel.com>
>>
>> All errors (new ones prefixed by >>):
>>
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>>       633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>           |            ^~~~~~~~~~~~~~~~~~~~~
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>>       642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>           |             ^~~~~~~~~~~~~~~~~~~~~~~
>>     cc1: all warnings being treated as errors
>>
>> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1aa9b9572b10529c2e64e2b8f44025d86e124308
>> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>> git fetch --no-tags linus xlnx/master
>> git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>> vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c
>>
>>     628	
>>     629	/*
>>     630	 * Called after the VM is otherwise initialized, but just before adding it to
>>     631	 * the vm_list.
>>     632	 */
>>   > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>     634	{
>>     635		return 0;
>>     636	}
>>     637	
>>     638	/*
>>     639	 * Called just after removing the VM from the vm_list, but before doing any
>>     640	 * other destruction.
>>     641	 */
>>   > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>     643	{
>>     644	}
>>     645	
>>
>> ---
>> 0-DAY CI Kernel Test Service, Intel Corporation
>> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
>>
> _______________________________________________
> kbuild-all mailing list -- kbuild-all(a)lists.01.org
> To unsubscribe send an email to kbuild-all-leave(a)lists.01.org

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [kbuild-all] Re: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous
@ 2020-09-18  5:46     ` Rong Chen
  0 siblings, 0 replies; 8+ messages in thread
From: Rong Chen @ 2020-09-18  5:46 UTC (permalink / raw)
  To: Paolo Bonzini, kernel test robot, Junaid Shahid
  Cc: kbuild-all, kvm-ppc, linux-arm-kernel



On 9/12/20 2:52 PM, Paolo Bonzini wrote:
> On 11/09/20 13:43, kernel test robot wrote:
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>>       633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>           |            ^~~~~~~~~~~~~~~~~~~~~
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>>       642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>           |             ^~~~~~~~~~~~~~~~~~~~~~~
>>     cc1: all warnings being treated as errors
> This makes little sense, the prototypes are in include/linux/kvm_host.h
> Also this code is not architecture-dependent...
>
> kvm-ppc guys, can you reproduce it?

Hi Paolo,

The error can be reproduced with W=1:

$ make W=1 
CROSS_COMPILE=/home/nfs/0day/gcc-9.3.0-nolibc/powerpc64-linux/bin/powerpc64-linux- 
ARCH=powerpc M=arch/powerpc/kvm | grep kvm_main
   CC [M]  arch/powerpc/kvm/../../../virt/kvm/kvm_main.o
arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: warning: no 
previous prototype for 'kvm_arch_post_init_vm' [-Wmissing-prototypes]
   633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
       |            ^~~~~~~~~~~~~~~~~~~~~
arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: warning: no 
previous prototype for 'kvm_arch_pre_destroy_vm' [-Wmissing-prototypes]
   642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
       |             ^~~~~~~~~~~~~~~~~~~~~~~

Best Regards,
Rong Chen

>
> Paolo
>
>>
>> Hi Junaid,
>>
>> FYI, the error/warning still remains.
>>
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>> head:   e5bd0d2afe7061562c483d301e4aabb51b13fcfa
>> commit: 1aa9b9572b10529c2e64e2b8f44025d86e124308 kvm: x86: mmu: Recovery of shattered NX large pages
>> date:   10 months ago
>> config: powerpc-defconfig (attached as .config)
>> compiler: powerpc64-linux-gcc (GCC) 9.3.0
>> 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
>>          git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>>          # save the attached .config to linux build tree
>>          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc
>>
>> If you fix the issue, kindly add following tag as appropriate
>> Reported-by: kernel test robot <lkp@intel.com>
>>
>> All errors (new ones prefixed by >>):
>>
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' [-Werror=missing-prototypes]
>>       633 | int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>           |            ^~~~~~~~~~~~~~~~~~~~~
>>>> arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:642:13: error: no previous prototype for 'kvm_arch_pre_destroy_vm' [-Werror=missing-prototypes]
>>       642 | void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>           |             ^~~~~~~~~~~~~~~~~~~~~~~
>>     cc1: all warnings being treated as errors
>>
>> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id\x1aa9b9572b10529c2e64e2b8f44025d86e124308
>> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>> git fetch --no-tags linus xlnx/master
>> git checkout 1aa9b9572b10529c2e64e2b8f44025d86e124308
>> vim +/kvm_arch_post_init_vm +633 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c
>>
>>     628	
>>     629	/*
>>     630	 * Called after the VM is otherwise initialized, but just before adding it to
>>     631	 * the vm_list.
>>     632	 */
>>   > 633	int __weak kvm_arch_post_init_vm(struct kvm *kvm)
>>     634	{
>>     635		return 0;
>>     636	}
>>     637	
>>     638	/*
>>     639	 * Called just after removing the VM from the vm_list, but before doing any
>>     640	 * other destruction.
>>     641	 */
>>   > 642	void __weak kvm_arch_pre_destroy_vm(struct kvm *kvm)
>>     643	{
>>     644	}
>>     645	
>>
>> ---
>> 0-DAY CI Kernel Test Service, Intel Corporation
>> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
>>
> _______________________________________________
> kbuild-all mailing list -- kbuild-all@lists.01.org
> To unsubscribe send an email to kbuild-all-leave@lists.01.org

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-09-18  5:48 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-11 11:43 arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' kernel test robot
2020-09-11 11:43 ` kernel test robot
2020-09-12  6:52 ` Paolo Bonzini
2020-09-12  6:52   ` arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch Paolo Bonzini
2020-09-12  6:52   ` arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' Paolo Bonzini
2020-09-18  5:46   ` [kbuild-all] " Rong Chen
2020-09-18  5:46     ` [kbuild-all] Re: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous Rong Chen
2020-09-18  5:46     ` arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:633:12: error: no previous prototype for 'kvm_arch_post_init_vm' Rong Chen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.