Messages in this thread | | | Date | Wed, 10 Nov 2021 20:06:21 +0800 | From | kernel test robot <> | Subject | [mchinth:sep_socwatch_linux_5_15 57/59] drivers/platform/x86/sepdk/sep/sys_info.c:320:6: error: variable 'model' set but not used |
| |
tree: https://github.com/mchinth/linux sep_socwatch_linux_5_15 head: 3af50f40725e78c505cbf840fb0be424c2f33e5e commit: 6622fb2e6554a0edd0ebbbd582565bbe84ca9dfe [57/59] Merge branch 'master' into sep_socwatch_linux_5_15 config: i386-allyesconfig (attached as .config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=1 build): # https://github.com/mchinth/linux/commit/6622fb2e6554a0edd0ebbbd582565bbe84ca9dfe git remote add mchinth https://github.com/mchinth/linux git fetch --no-tags mchinth sep_socwatch_linux_5_15 git checkout 6622fb2e6554a0edd0ebbbd582565bbe84ca9dfe # save the attached .config to linux build tree make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com>
Note: the mchinth/sep_socwatch_linux_5_15 HEAD 3af50f40725e78c505cbf840fb0be424c2f33e5e builds fine. It only hurts bisectability.
All errors (new ones prefixed by >>):
drivers/platform/x86/sepdk/sep/sys_info.c: In function 'sys_info_Fill_CPUID': >> drivers/platform/x86/sepdk/sep/sys_info.c:320:6: error: variable 'model' set but not used [-Werror=unused-but-set-variable] 320 | U32 model = 0; | ^~~~~ cc1: all warnings being treated as errors -- drivers/platform/x86/sepdk/sep/utility.c: In function 'utility_Log_Write': >> drivers/platform/x86/sepdk/sep/utility.c:702:3: error: function 'utility_Log_Write' might be a candidate for 'gnu_printf' format attribute [-Werror=suggest-attribute=format] 702 | DRV_VSNPRINTF(DRV_LOG_ENTRY_message(entry), | ^~~~~~~~~~~~~ drivers/platform/x86/sepdk/sep/utility.c: In function 'UTILITY_Log': >> drivers/platform/x86/sepdk/sep/utility.c:845:4: error: function 'UTILITY_Log' might be a candidate for 'gnu_printf' format attribute [-Werror=suggest-attribute=format] 845 | nb_written_characters += DRV_VSNPRINTF( | ^~~~~~~~~~~~~~~~~~~~~ >> drivers/platform/x86/sepdk/sep/utility.c:859:22: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context] 858 | if ((category_verbosity & LOG_CHANNEL_PRINTK) * | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 859 | !in_interrupt * !in_notification) { | ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~ cc1: all warnings being treated as errors -- drivers/platform/x86/sepdk/sep/pebs.c: In function 'PEBS_Flush_Buffer': >> drivers/platform/x86/sepdk/sep/pebs.c:977:6: error: variable 'cur_grp' set but not used [-Werror=unused-but-set-variable] 977 | U32 cur_grp; | ^~~~~~~ cc1: all warnings being treated as errors -- In file included from drivers/platform/x86/sepdk/sep/unc_mmio.c:38: drivers/platform/x86/sepdk/sep/unc_mmio.c: In function 'unc_mmio_Write_PMU': >> drivers/platform/x86/sepdk/inc/utility.h:56:18: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] 56 | ((base) ? readl((void __iomem *)(base) + (offset)) : 0) | ^ drivers/platform/x86/sepdk/sep/unc_mmio.c:201:16: note: in expansion of macro 'SYS_MMIO_Read32' 201 | tmp_value = SYS_MMIO_Read32(virtual_addr, offset_delta); | ^~~~~~~~~~~~~~~ drivers/platform/x86/sepdk/sep/unc_mmio.c: In function 'unc_mmio_Enable_PMU': >> drivers/platform/x86/sepdk/inc/utility.h:56:18: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] 56 | ((base) ? readl((void __iomem *)(base) + (offset)) : 0) | ^ drivers/platform/x86/sepdk/sep/unc_mmio.c:365:18: note: in expansion of macro 'SYS_MMIO_Read32' 365 | buffer[j] = SYS_MMIO_Read32( | ^~~~~~~~~~~~~~~ drivers/platform/x86/sepdk/sep/unc_mmio.c: In function 'unc_mmio_Trigger_Read': >> drivers/platform/x86/sepdk/inc/utility.h:56:18: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] 56 | ((base) ? readl((void __iomem *)(base) + (offset)) : 0) | ^ drivers/platform/x86/sepdk/sep/unc_mmio.c:573:12: note: in expansion of macro 'SYS_MMIO_Read32' 573 | value = SYS_MMIO_Read32(virtual_addr, offset_delta); | ^~~~~~~~~~~~~~~ drivers/platform/x86/sepdk/sep/unc_mmio.c: In function 'unc_mmio_Read_PMU_Data': >> drivers/platform/x86/sepdk/inc/utility.h:56:18: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] 56 | ((base) ? readl((void __iomem *)(base) + (offset)) : 0) | ^ drivers/platform/x86/sepdk/sep/unc_mmio.c:736:17: note: in expansion of macro 'SYS_MMIO_Read32' 736 | tmp_value = SYS_MMIO_Read32(virtual_addr, | ^~~~~~~~~~~~~~~ cc1: all warnings being treated as errors -- drivers/platform/x86/sepdk/sep/unc_common.c: In function 'UNC_COMMON_Get_Platform_Topology': >> drivers/platform/x86/sepdk/sep/unc_common.c:280:6: error: variable 'device_id' set but not used [-Werror=unused-but-set-variable] 280 | U32 device_id; | ^~~~~~~~~ cc1: all warnings being treated as errors -- drivers/platform/x86/sepdk/sep/unc_pci.c: In function 'unc_pci_Write_PMU': >> drivers/platform/x86/sepdk/sep/unc_pci.c:57:6: error: variable 'device_id' set but not used [-Werror=unused-but-set-variable] 57 | U32 device_id; | ^~~~~~~~~ cc1: all warnings being treated as errors
vim +/model +320 drivers/platform/x86/sepdk/sep/sys_info.c
d605aabab41663 Jon Moeller 2018-11-13 289 d605aabab41663 Jon Moeller 2018-11-13 290 /* ------------------------------------------------------------------------- */ d605aabab41663 Jon Moeller 2018-11-13 291 /*! d605aabab41663 Jon Moeller 2018-11-13 292 * @fn extern void sys_info_Fill_CPUID(...) d605aabab41663 Jon Moeller 2018-11-13 293 * d605aabab41663 Jon Moeller 2018-11-13 294 * @param num_cpuids, d605aabab41663 Jon Moeller 2018-11-13 295 * @param basic_funcs, d605aabab41663 Jon Moeller 2018-11-13 296 * @param extended_funcs, d605aabab41663 Jon Moeller 2018-11-13 297 * @param cpu, d605aabab41663 Jon Moeller 2018-11-13 298 * @param *current_cpuid d605aabab41663 Jon Moeller 2018-11-13 299 * @param *gen_per_cpu, d605aabab41663 Jon Moeller 2018-11-13 300 * @param *local_gpc d605aabab41663 Jon Moeller 2018-11-13 301 * d605aabab41663 Jon Moeller 2018-11-13 302 * @return None d605aabab41663 Jon Moeller 2018-11-13 303 * d605aabab41663 Jon Moeller 2018-11-13 304 * @brief This routine is called to build per cpu information. d605aabab41663 Jon Moeller 2018-11-13 305 * @brief Fills in the cpuid for the processor in the right location in the buffer d605aabab41663 Jon Moeller 2018-11-13 306 * d605aabab41663 Jon Moeller 2018-11-13 307 */ d605aabab41663 Jon Moeller 2018-11-13 308 static void sys_info_Fill_CPUID(U32 num_cpuids, U32 basic_funcs, d605aabab41663 Jon Moeller 2018-11-13 309 U32 extended_funcs, U32 cpu, d605aabab41663 Jon Moeller 2018-11-13 310 VTSA_CPUID *current_cpuid, d605aabab41663 Jon Moeller 2018-11-13 311 VTSA_GEN_PER_CPU *gen_per_cpu, d605aabab41663 Jon Moeller 2018-11-13 312 VTSA_GEN_PER_CPU *local_gpc) d605aabab41663 Jon Moeller 2018-11-13 313 { d605aabab41663 Jon Moeller 2018-11-13 314 U32 i, index, j; d605aabab41663 Jon Moeller 2018-11-13 315 U64 cpuid_function; d605aabab41663 Jon Moeller 2018-11-13 316 U64 rax, rbx, rcx, rdx; d605aabab41663 Jon Moeller 2018-11-13 317 VTSA_CPUID *cpuid_el; d605aabab41663 Jon Moeller 2018-11-13 318 U32 shift_nbits_core = 0; d605aabab41663 Jon Moeller 2018-11-13 319 U32 shift_nbits_pkg = 0; d605aabab41663 Jon Moeller 2018-11-13 @320 U32 model = 0; d605aabab41663 Jon Moeller 2018-11-13 321 DRV_BOOL ht_supported = FALSE; d605aabab41663 Jon Moeller 2018-11-13 322 U32 apic_id = 0; d605aabab41663 Jon Moeller 2018-11-13 323 U32 num_logical_per_physical = 0; d605aabab41663 Jon Moeller 2018-11-13 324 U32 cores_per_die = 1; d605aabab41663 Jon Moeller 2018-11-13 325 U32 thread_id = 0; d605aabab41663 Jon Moeller 2018-11-13 326 U32 core_id = 0; d605aabab41663 Jon Moeller 2018-11-13 327 U32 package_id = 0; d605aabab41663 Jon Moeller 2018-11-13 328 U32 module_id = 0; d605aabab41663 Jon Moeller 2018-11-13 329 U32 cores_sharing_cache = 0; d605aabab41663 Jon Moeller 2018-11-13 330 U32 cache_mask_width = 0; d605aabab41663 Jon Moeller 2018-11-13 331 U32 num_cores = 0; d605aabab41663 Jon Moeller 2018-11-13 332 d605aabab41663 Jon Moeller 2018-11-13 333 SEP_DRV_LOG_TRACE_IN("CPU: %x.", cpu); d605aabab41663 Jon Moeller 2018-11-13 334 d605aabab41663 Jon Moeller 2018-11-13 335 apic_id = CPU_STATE_apic_id(&pcb[cpu]); d605aabab41663 Jon Moeller 2018-11-13 336 SEP_DRV_LOG_TRACE("Cpu %x: apic_id = %d.", cpu, apic_id); d605aabab41663 Jon Moeller 2018-11-13 337 d605aabab41663 Jon Moeller 2018-11-13 338 for (i = 0, index = 0; index < num_cpuids; i++) { d605aabab41663 Jon Moeller 2018-11-13 339 cpuid_function = d605aabab41663 Jon Moeller 2018-11-13 340 (i < basic_funcs) ? i : (0x80000000 + i - basic_funcs); d605aabab41663 Jon Moeller 2018-11-13 341 d605aabab41663 Jon Moeller 2018-11-13 342 if (cpuid_function == 0x4) { d605aabab41663 Jon Moeller 2018-11-13 343 for (j = 0, rax = (U64)-1; (rax & 0x1f) != 0; j++) { d605aabab41663 Jon Moeller 2018-11-13 344 rcx = j; d605aabab41663 Jon Moeller 2018-11-13 345 UTILITY_Read_Cpuid(cpuid_function, &rax, &rbx, d605aabab41663 Jon Moeller 2018-11-13 346 &rcx, &rdx); d605aabab41663 Jon Moeller 2018-11-13 347 cpuid_el = ¤t_cpuid[index]; d605aabab41663 Jon Moeller 2018-11-13 348 index++; d605aabab41663 Jon Moeller 2018-11-13 349
:::::: The code at line 320 was first introduced by commit :::::: d605aabab41663c9be9e8c549042933189adbf2f platform/x86: add sep and socwatch drivers without socperf.
:::::: TO: Jon Moeller <jon.moeller@intel.com> :::::: CC: Faycal Benmlih <faycal.benmlih@intel.com>
--- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [unhandled content-type:application/gzip] | |