Re: [PATCH 2/2] x86:kvm:hyperv: guest->host event signaling via eventfd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Roman,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on kvm/linux-next]
[also build test ERROR on v4.15-rc2]
[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/Roman-Kagan/x86-kvm-hyperv-guest-host-event-signaling-via-eventfd/20171207-065837
base:   https://git.kernel.org/pub/scm/virt/kvm/kvm.git linux-next
config: i386-randconfig-x011-201749 (attached as .config)
compiler: gcc-7 (Debian 7.2.0-12) 7.2.1 20171025
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/init.h:5:0,
                    from arch/x86/include/asm/mem_encrypt.h:18,
                    from include/linux/mem_encrypt.h:20,
                    from arch/x86/include/asm/processor-flags.h:6,
                    from arch/x86/include/asm/processor.h:5,
                    from arch/x86//kvm/x86.h:5,
                    from arch/x86//kvm/hyperv.c:24:
   arch/x86//kvm/hyperv.c: In function 'hvcall_sigevent_param':
>> arch/x86//kvm/hyperv.c:1236:26: error: dereferencing pointer to incomplete type 'struct hv_input_signal_event'
     if ((gpa & (__alignof__(*msg) - 1)) ||
                             ^~
   include/linux/compiler.h:58:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> arch/x86//kvm/hyperv.c:1236:2: note: in expansion of macro 'if'
     if ((gpa & (__alignof__(*msg) - 1)) ||
     ^~

vim +1236 arch/x86//kvm/hyperv.c

  1229	
  1230	static u16 hvcall_sigevent_param(struct kvm_vcpu *vcpu, gpa_t gpa, u32 *conn_id)
  1231	{
  1232		struct page *page;
  1233		void *pg;
  1234		struct hv_input_signal_event *msg;
  1235	
> 1236		if ((gpa & (__alignof__(*msg) - 1)) ||
  1237		    offset_in_page(gpa) + sizeof(*msg) > PAGE_SIZE)
  1238			return HV_STATUS_INVALID_ALIGNMENT;
  1239	
  1240		page = kvm_vcpu_gfn_to_page(vcpu, gpa >> PAGE_SHIFT);
  1241		if (is_error_page(page))
  1242			return HV_STATUS_INSUFFICIENT_MEMORY;
  1243	
  1244		pg = kmap_atomic(page);
  1245		msg = pg + offset_in_page(gpa);
  1246		*conn_id = msg->connectionid.u.id + msg->flag_number;
  1247		kunmap_atomic(pg);
  1248		return HV_STATUS_SUCCESS;
  1249	}
  1250	

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

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux