Re: [PATCH v4 13/22] KVM: arm64: ITS: KVM_DEV_ARM_VGIC_GRP_ITS_TABLES group

Auger Eric
2017-03-27T18:27:18Z

Hi,

On 27/03/2017 17:04, kbuild test robot wrote:
> Hi Eric,
>
> [auto build test ERROR on kvmarm/next]
> [also build test ERROR on v4.11-rc4 next-20170327]
> [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/Eric-Auger/vITS-save-restore/20170327-195443
> base: https://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm.git next
> config: arm64-defconfig (attached as .config)
> compiler: aarch64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
> reproduce:
> wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # save the attached .config to linux build tree
> make.cross ARCH=arm64
>
> All errors (new ones prefixed by >>):
>
> arch/arm64/kvm/../../../virt/kvm/arm/vgic/vgic-its.c: In function 'vgic_its_has_attr':
>>> arch/arm64/kvm/../../../virt/kvm/arm/vgic/vgic-its.c:1766:33: error: 'its' undeclared (first use in this function)
> return vgic_its_table_restore(its);

I think this is due to the fact the series is applied on KVM: arm/arm64:
Emulate the EL1 phys timer registers (kvmarm-for-4.11).

Please apply it to (kvmarm/queue, kvmarm/next, kvm_next) KVM: arm/arm64:
vgic: Improve sync_hwstate performance (255905e)
I also checked it compiles on 4.11-rc4.

Please let me know if I got something wrong.

Thanks

Eric
> ^~~
> arch/arm64/kvm/../../../virt/kvm/arm/vgic/vgic-its.c:1766:33: note: each undeclared identifier is reported only once for each function it appears in
>
> vim +/its +1766 arch/arm64/kvm/../../../virt/kvm/arm/vgic/vgic-its.c
>
> 1750 case KVM_DEV_ARM_VGIC_CTRL_INIT:
> 1751 return 0;
> 1752 }
> 1753 break;
> 1754 case KVM_DEV_ARM_VGIC_GRP_ITS_REGS: {
> 1755 u64 __user *uaddr = (u64 __user *)(long)attr->addr;
> 1756 u64 reg;
> 1757
> 1758 if (get_user(reg, uaddr))
> 1759 return -EFAULT;
> 1760
> 1761 return vgic_its_attr_regs_access(dev, attr, &reg, true);
> 1762 }
> 1763 case KVM_DEV_ARM_VGIC_GRP_ITS_TABLES:
> 1764 if (attr->attr)
> 1765 return -EINVAL;
>> 1766 return vgic_its_table_restore(its);
> 1767 }
> 1768 return -ENXIO;
> 1769 }
> 1770
> 1771 static int vgic_its_set_attr(struct kvm_device *dev,
> 1772 struct kvm_device_attr *attr)
> 1773 {
> 1774 struct vgic_its *its = dev->private;
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>