Hi Vitaly,

[auto build test WARNING on next-20170619]
[cannot apply to tip/x86/core linus/master linux/master v4.12-rc6 v4.12-rc5 v4.12-rc4 v4.12-rc7]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/kys-exchange-microsoft-com/x86-hyper-v-include-hyperv-only-when-CONFIG_HYPERV-is-set/20170626-133601
config: x86_64-randconfig-ws0-07010157 (attached as .config)
compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

All warnings (new ones prefixed by >>):

arch/x86//hyperv/mmu.c: In function 'hyperv_flush_tlb_others':
>> arch/x86//hyperv/mmu.c:120:2: warning: passing argument 2 of 'native_flush_tlb_others' from incompatible pointer type [enabled by default]
native_flush_tlb_others(cpus, mm, start, end);
^
In file included from arch/x86//hyperv/mmu.c:9:0:
arch/x86/include/asm/tlbflush.h:256:6: note: expected 'const struct flush_tlb_info *' but argument is of type 'struct mm_struct *'
void native_flush_tlb_others(const struct cpumask *cpumask,
^
arch/x86//hyperv/mmu.c:120:2: error: too many arguments to function 'native_flush_tlb_others'
native_flush_tlb_others(cpus, mm, start, end);
^
In file included from arch/x86//hyperv/mmu.c:9:0:
arch/x86/include/asm/tlbflush.h:256:6: note: declared here
void native_flush_tlb_others(const struct cpumask *cpumask,
^
arch/x86//hyperv/mmu.c: In function 'hyperv_setup_mmu_ops':
>> arch/x86//hyperv/mmu.c:127:31: warning: assignment from incompatible pointer type [enabled by default]
pv_mmu_ops.flush_tlb_others = hyperv_flush_tlb_others;
^

vim +/native_flush_tlb_others +120 arch/x86//hyperv/mmu.c

114
115 local_irq_restore(flags);
116
117 if (!(status & HV_HYPERCALL_RESULT_MASK))
118 return;
119 do_native:
> 120 native_flush_tlb_others(cpus, mm, start, end);
121 }
122
123 void hyperv_setup_mmu_ops(void)
124 {
125 if (ms_hyperv.hints & HV_X64_REMOTE_TLB_FLUSH_RECOMMENDED) {
126 pr_info("Hyper-V: Using hypercall for remote TLB flush\n");
> 127 pv_mmu_ops.flush_tlb_others = hyperv_flush_tlb_others;
128 }
129 }
130

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachments:

(No filename) (2.43 kB)
.config.gz (28.71 kB)
Download all attachments

2017-07-17 15:22:01

by Greg Kroah-Hartman

[permalink] [raw]

Subject: Re: [PATCH 03/10] x86/hyper-v: make hv_do_hypercall() inline

On Sun, Jun 25, 2017 at 10:06:42AM -0700, [email protected] wrote:
> From: Vitaly Kuznetsov <[email protected]>
>
> We have only three call sites for hv_do_hypercall() and we're going to
> change HVCALL_SIGNAL_EVENT to doing fast hypercall so we can inline this
> function for optimization.
>
> Hyper-V top level functional specification states that r9-r11 registers
> and flags may be clobbered by the hypervisor during hypercall and with
> inlining this is somewhat important, add the clobbers.
>
> Signed-off-by: Vitaly Kuznetsov <[email protected]>
> Reviewed-by: Andy Shevchenko <[email protected]>
> Signed-off-by: K. Y. Srinivasan <[email protected]>

This patch breaks the build, so can you please rebase and resend this,
and the rest of this series, if you wish to have it applied?

thanks,

greg k-h