[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
From: kernel test robot
Date: Wed Nov 10 2021 - 07:06:46 EST
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@xxxxxxxxx>
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@xxxxxxxxx>
:::::: CC: Faycal Benmlih <faycal.benmlih@xxxxxxxxx>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
Attachment:
.config.gz
Description: application/gzip