Re: [PATCH v3 2/2] x86/kvm: Provide optimized version of vcpu_is_preempted() for x86-64

Hi Waiman,

[auto build test ERROR on kvm/linux-next]
[also build test ERROR on v4.10-rc8 next-20170215]
[cannot apply to tip/x86/core]
[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/Waiman-Long/x86-kvm-Reduce-vcpu_is_preempted-overhead/20170216-033517
base:   https://git.kernel.org/pub/scm/virt/kvm/kvm.git linux-next
config: i386-randconfig-h1-02160052 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

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

   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/uapi/linux/capability.h:16,
                    from include/linux/capability.h:15,
                    from include/linux/sched.h:15,
                    from include/linux/context_tracking.h:4,
                    from arch/x86/kernel/kvm.c:23:
   arch/x86/kernel/kvm.c: In function 'kvm_spinlock_init':

quoted

arch/x86/kernel/kvm.c:631:6: error: 'PREEMPTED_OFFSET' undeclared (first use in this function)

      != PREEMPTED_OFFSET) || (sizeof(steal_time.preempted) != 1));
         ^
   include/linux/compiler.h:498:19: note: in definition of macro '__compiletime_assert'
      bool __cond = !(condition);    \
                      ^~~~~~~~~
   include/linux/compiler.h:518:2: note: in expansion of macro '_compiletime_assert'
     _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
     ^~~~~~~~~~~~~~~~~~~
   include/linux/bug.h:54:37: note: in expansion of macro 'compiletime_assert'
    #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
                                        ^~~~~~~~~~~~~~~~~~
   include/linux/bug.h:78:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
     BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
     ^~~~~~~~~~~~~~~~

quoted

arch/x86/kernel/kvm.c:630:2: note: in expansion of macro 'BUILD_BUG_ON'

     BUILD_BUG_ON((offsetof(struct kvm_steal_time, preempted)
     ^~~~~~~~~~~~
   arch/x86/kernel/kvm.c:631:6: note: each undeclared identifier is reported only once for each function it appears in
      != PREEMPTED_OFFSET) || (sizeof(steal_time.preempted) != 1));
         ^
   include/linux/compiler.h:498:19: note: in definition of macro '__compiletime_assert'
      bool __cond = !(condition);    \
                      ^~~~~~~~~
   include/linux/compiler.h:518:2: note: in expansion of macro '_compiletime_assert'
     _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
     ^~~~~~~~~~~~~~~~~~~
   include/linux/bug.h:54:37: note: in expansion of macro 'compiletime_assert'
    #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
                                        ^~~~~~~~~~~~~~~~~~
   include/linux/bug.h:78:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
     BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
     ^~~~~~~~~~~~~~~~

quoted

arch/x86/kernel/kvm.c:630:2: note: in expansion of macro 'BUILD_BUG_ON'

     BUILD_BUG_ON((offsetof(struct kvm_steal_time, preempted)
     ^~~~~~~~~~~~

vim +/PREEMPTED_OFFSET +631 arch/x86/kernel/kvm.c

   624	
   625	/*
   626	 * Setup pv_lock_ops to exploit KVM_FEATURE_PV_UNHALT if present.
   627	 */
   628	void __init kvm_spinlock_init(void)
   629	{
 > 630		BUILD_BUG_ON((offsetof(struct kvm_steal_time, preempted)
 > 631			!= PREEMPTED_OFFSET) || (sizeof(steal_time.preempted) != 1));
   632	
   633		if (!kvm_para_available())
   634			return;

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