Re: [PATCH] perf/x86/amd/power: Add AMD accumulated power reporting

[kbuild test robot <lkp@intel.com>]

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

Hi Borislav,

[auto build test WARNING on tip/x86/core]
[also build test WARNING on v4.5-rc1 next-20160128]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Borislav-Petkov/perf-x86-amd-power-Add-AMD-accumulated-power-reporting/20160128-170527
config: x86_64-randconfig-i0-01270829 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All 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/linux/list.h:4,
                    from include/linux/module.h:9,
                    from arch/x86/kernel/cpu/perf_event_amd_power.c:13:
   arch/x86/kernel/cpu/perf_event_amd_power.c: In function 'amd_power_pmu_init':
   arch/x86/kernel/cpu/perf_event_amd_power.c:432:20: error: 'X86_FEATURE_ACC_POWER' undeclared (first use in this function)
     if (!boot_cpu_has(X86_FEATURE_ACC_POWER))
                       ^
   include/linux/compiler.h:147:28: note: in definition of macro '__trace_if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
                               ^
>> arch/x86/kernel/cpu/perf_event_amd_power.c:432:2: note: in expansion of macro 'if'
     if (!boot_cpu_has(X86_FEATURE_ACC_POWER))
     ^
   arch/x86/include/asm/cpufeature.h:338:27: note: in expansion of macro 'cpu_has'
    #define boot_cpu_has(bit) cpu_has(&boot_cpu_data, bit)
                              ^
   arch/x86/kernel/cpu/perf_event_amd_power.c:432:7: note: in expansion of macro 'boot_cpu_has'
     if (!boot_cpu_has(X86_FEATURE_ACC_POWER))
          ^
   arch/x86/kernel/cpu/perf_event_amd_power.c:432:20: note: each undeclared identifier is reported only once for each function it appears in
     if (!boot_cpu_has(X86_FEATURE_ACC_POWER))
                       ^
   include/linux/compiler.h:147:28: note: in definition of macro '__trace_if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
                               ^
>> arch/x86/kernel/cpu/perf_event_amd_power.c:432:2: note: in expansion of macro 'if'
     if (!boot_cpu_has(X86_FEATURE_ACC_POWER))
     ^
   arch/x86/include/asm/cpufeature.h:338:27: note: in expansion of macro 'cpu_has'
    #define boot_cpu_has(bit) cpu_has(&boot_cpu_data, bit)
                              ^
   arch/x86/kernel/cpu/perf_event_amd_power.c:432:7: note: in expansion of macro 'boot_cpu_has'
     if (!boot_cpu_has(X86_FEATURE_ACC_POWER))
          ^
   arch/x86/kernel/cpu/perf_event_amd_power.c:435:17: error: implicit declaration of function 'amd_get_cores_per_cu' [-Werror=implicit-function-declaration]
     cores_per_cu = amd_get_cores_per_cu();
                    ^
   cc1: some warnings being treated as errors

vim +/if +432 arch/x86/kernel/cpu/perf_event_amd_power.c

   416		return NOTIFY_OK;
   417	}
   418	
   419	static const struct x86_cpu_id cpu_match[] = {
   420		{ .vendor = X86_VENDOR_AMD, .family = 0x15 },
   421		{},
   422	};
   423	
   424	static int __init amd_power_pmu_init(void)
   425	{
   426		int i, ret;
   427		u64 tmp;
   428	
   429		if (!x86_match_cpu(cpu_match))
   430			return 0;
   431	
 > 432		if (!boot_cpu_has(X86_FEATURE_ACC_POWER))
   433			return -ENODEV;
   434	
   435		cores_per_cu = amd_get_cores_per_cu();
   436		cu_num = boot_cpu_data.x86_max_cores / cores_per_cu;
   437	
   438		if (WARN_ON_ONCE(cu_num > MAX_CUS))
   439			return -EINVAL;
   440	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 20607 bytes --]