* arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true
@ 2023-11-16 1:52 kernel test robot
0 siblings, 0 replies; 3+ messages in thread
From: kernel test robot @ 2023-11-16 1:52 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp
::::::
:::::: Manual check reason: "low confidence static check warning: arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true"
::::::
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Juergen Gross <jgross@suse.com>
CC: "Borislav Petkov (AMD)" <bp@alien8.de>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c42d9eeef8e5ba9292eda36fd8e3c11f35ee065c
commit: 03409069520974361ffb510c725305239b78b39f x86/mtrr: Replace vendor tests in MTRR code
date: 6 months ago
:::::: branch date: 21 hours ago
:::::: commit date: 6 months ago
config: x86_64-randconfig-x001-20230717 (https://download.01.org/0day-ci/archive/20231116/202311160928.Pyg4eZ5v-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231116/202311160928.Pyg4eZ5v-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202311160928.Pyg4eZ5v-lkp@intel.com/
sparse warnings: (new ones prefixed by >>)
>> arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true
vim +650 arch/x86/kernel/cpu/mtrr/mtrr.c
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 600
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 601 /**
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 602 * mtrr_bp_init - initialize MTRRs on the boot CPU
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 603 *
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 604 * This needs to be called early; before any of the other CPUs are
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 605 * initialized (i.e. before smp_init()).
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 606 */
9ef231a436fddb arch/i386/kernel/cpu/mtrr/main.c Sam Ravnborg 2007-07-21 607 void __init mtrr_bp_init(void)
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 608 {
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 609 bool generic_mtrrs = cpu_feature_enabled(X86_FEATURE_MTRR);
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 610 const char *why = "(not available)";
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 611 unsigned long config, dummy;
dbd51be026eaf8 arch/x86/kernel/cpu/mtrr/main.c Jaswinder Singh Rajput 2009-07-04 612
d053b481a5f16d arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 613 phys_hi_rsvd = GENMASK(31, boot_cpu_data.x86_phys_bits - 32);
95ffa2438d0e9c arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-04-29 614
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 615 if (!generic_mtrrs && mtrr_state.enabled) {
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 616 /*
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 617 * Software overwrite of MTRR state, only for generic case.
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 618 * Note that X86_FEATURE_MTRR has been reset in this case.
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 619 */
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 620 init_table();
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 621 pr_info("MTRRs set to read-only\n");
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 622
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 623 return;
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 624 }
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 625
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 626 if (generic_mtrrs) {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 627 mtrr_if = &generic_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 628 } else {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 629 switch (boot_cpu_data.x86_vendor) {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 630 case X86_VENDOR_AMD:
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 631 /* Pre-Athlon (K6) AMD CPU MTRRs */
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 632 if (cpu_feature_enabled(X86_FEATURE_K6_MTRR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 633 mtrr_if = &amd_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 634 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 635 case X86_VENDOR_CENTAUR:
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 636 if (cpu_feature_enabled(X86_FEATURE_CENTAUR_MCR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 637 mtrr_if = ¢aur_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 638 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 639 case X86_VENDOR_CYRIX:
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 640 if (cpu_feature_enabled(X86_FEATURE_CYRIX_ARR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 641 mtrr_if = &cyrix_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 642 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 643 default:
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 644 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 645 }
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 646 }
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 647
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 648 if (mtrr_enabled()) {
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 649 /* Get the number of variable MTRR ranges. */
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 @650 if (mtrr_if == &generic_mtrr_ops)
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 651 rdmsr(MSR_MTRRcap, config, dummy);
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 652 else
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 653 config = mtrr_if->var_regs;
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 654 num_var_ranges = config & MTRR_CAP_VCNT;
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 655
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 656 init_table();
45fa71f19a2d73 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 657 if (mtrr_if == &generic_mtrr_ops) {
f9626104a5b681 arch/x86/kernel/cpu/mtrr/main.c Luis R. Rodriguez 2015-05-26 658 /* BIOS may override */
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 659 if (get_mtrr_state()) {
adfe7512e1d0b2 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 660 memory_caching_control |= CACHE_MTRR;
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 661 changed_by_mtrr_cleanup = mtrr_cleanup();
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 662 } else {
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 663 mtrr_if = NULL;
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 664 why = "by BIOS";
12031a624af781 arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-05-02 665 }
95ffa2438d0e9c arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-04-29 666 }
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 667 }
f9626104a5b681 arch/x86/kernel/cpu/mtrr/main.c Luis R. Rodriguez 2015-05-26 668
adfe7512e1d0b2 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 669 if (!mtrr_enabled())
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 670 pr_info("MTRRs disabled %s\n", why);
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 671 }
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 672
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 3+ messages in thread* arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true
@ 2023-11-15 19:23 kernel test robot
0 siblings, 0 replies; 3+ messages in thread
From: kernel test robot @ 2023-11-15 19:23 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp
::::::
:::::: Manual check reason: "low confidence static check warning: arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true"
::::::
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Juergen Gross <jgross@suse.com>
CC: "Borislav Petkov (AMD)" <bp@alien8.de>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c42d9eeef8e5ba9292eda36fd8e3c11f35ee065c
commit: 03409069520974361ffb510c725305239b78b39f x86/mtrr: Replace vendor tests in MTRR code
date: 6 months ago
:::::: branch date: 15 hours ago
:::::: commit date: 6 months ago
config: x86_64-randconfig-121-20231114 (https://download.01.org/0day-ci/archive/20231116/202311160308.FGdlo5nZ-lkp@intel.com/config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231116/202311160308.FGdlo5nZ-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202311160308.FGdlo5nZ-lkp@intel.com/
sparse warnings: (new ones prefixed by >>)
>> arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true
vim +650 arch/x86/kernel/cpu/mtrr/mtrr.c
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 600
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 601 /**
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 602 * mtrr_bp_init - initialize MTRRs on the boot CPU
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 603 *
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 604 * This needs to be called early; before any of the other CPUs are
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 605 * initialized (i.e. before smp_init()).
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 606 */
9ef231a436fddb arch/i386/kernel/cpu/mtrr/main.c Sam Ravnborg 2007-07-21 607 void __init mtrr_bp_init(void)
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 608 {
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 609 bool generic_mtrrs = cpu_feature_enabled(X86_FEATURE_MTRR);
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 610 const char *why = "(not available)";
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 611 unsigned long config, dummy;
dbd51be026eaf8 arch/x86/kernel/cpu/mtrr/main.c Jaswinder Singh Rajput 2009-07-04 612
d053b481a5f16d arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 613 phys_hi_rsvd = GENMASK(31, boot_cpu_data.x86_phys_bits - 32);
95ffa2438d0e9c arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-04-29 614
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 615 if (!generic_mtrrs && mtrr_state.enabled) {
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 616 /*
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 617 * Software overwrite of MTRR state, only for generic case.
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 618 * Note that X86_FEATURE_MTRR has been reset in this case.
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 619 */
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 620 init_table();
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 621 pr_info("MTRRs set to read-only\n");
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 622
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 623 return;
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 624 }
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 625
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 626 if (generic_mtrrs) {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 627 mtrr_if = &generic_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 628 } else {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 629 switch (boot_cpu_data.x86_vendor) {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 630 case X86_VENDOR_AMD:
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 631 /* Pre-Athlon (K6) AMD CPU MTRRs */
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 632 if (cpu_feature_enabled(X86_FEATURE_K6_MTRR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 633 mtrr_if = &amd_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 634 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 635 case X86_VENDOR_CENTAUR:
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 636 if (cpu_feature_enabled(X86_FEATURE_CENTAUR_MCR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 637 mtrr_if = ¢aur_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 638 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 639 case X86_VENDOR_CYRIX:
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 640 if (cpu_feature_enabled(X86_FEATURE_CYRIX_ARR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 641 mtrr_if = &cyrix_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 642 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 643 default:
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 644 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 645 }
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 646 }
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 647
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 648 if (mtrr_enabled()) {
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 649 /* Get the number of variable MTRR ranges. */
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 @650 if (mtrr_if == &generic_mtrr_ops)
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 651 rdmsr(MSR_MTRRcap, config, dummy);
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 652 else
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 653 config = mtrr_if->var_regs;
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 654 num_var_ranges = config & MTRR_CAP_VCNT;
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 655
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 656 init_table();
45fa71f19a2d73 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 657 if (mtrr_if == &generic_mtrr_ops) {
f9626104a5b681 arch/x86/kernel/cpu/mtrr/main.c Luis R. Rodriguez 2015-05-26 658 /* BIOS may override */
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 659 if (get_mtrr_state()) {
adfe7512e1d0b2 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 660 memory_caching_control |= CACHE_MTRR;
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 661 changed_by_mtrr_cleanup = mtrr_cleanup();
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 662 } else {
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 663 mtrr_if = NULL;
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 664 why = "by BIOS";
12031a624af781 arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-05-02 665 }
95ffa2438d0e9c arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-04-29 666 }
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 667 }
f9626104a5b681 arch/x86/kernel/cpu/mtrr/main.c Luis R. Rodriguez 2015-05-26 668
adfe7512e1d0b2 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 669 if (!mtrr_enabled())
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 670 pr_info("MTRRs disabled %s\n", why);
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 671 }
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 672
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 3+ messages in thread* arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true
@ 2023-11-14 23:56 kernel test robot
0 siblings, 0 replies; 3+ messages in thread
From: kernel test robot @ 2023-11-14 23:56 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp
::::::
:::::: Manual check reason: "low confidence static check warning: arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true"
::::::
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Juergen Gross <jgross@suse.com>
CC: "Borislav Petkov (AMD)" <bp@alien8.de>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 9bacdd8996c77c42ca004440be610692275ff9d0
commit: 03409069520974361ffb510c725305239b78b39f x86/mtrr: Replace vendor tests in MTRR code
date: 6 months ago
:::::: branch date: 31 hours ago
:::::: commit date: 6 months ago
config: x86_64-randconfig-121-20231114 (https://download.01.org/0day-ci/archive/20231115/202311150717.3wStmMmM-lkp@intel.com/config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231115/202311150717.3wStmMmM-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202311150717.3wStmMmM-lkp@intel.com/
sparse warnings: (new ones prefixed by >>)
>> arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true
vim +650 arch/x86/kernel/cpu/mtrr/mtrr.c
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 600
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 601 /**
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 602 * mtrr_bp_init - initialize MTRRs on the boot CPU
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 603 *
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 604 * This needs to be called early; before any of the other CPUs are
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 605 * initialized (i.e. before smp_init()).
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 606 */
9ef231a436fddb arch/i386/kernel/cpu/mtrr/main.c Sam Ravnborg 2007-07-21 607 void __init mtrr_bp_init(void)
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 608 {
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 609 bool generic_mtrrs = cpu_feature_enabled(X86_FEATURE_MTRR);
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 610 const char *why = "(not available)";
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 611 unsigned long config, dummy;
dbd51be026eaf8 arch/x86/kernel/cpu/mtrr/main.c Jaswinder Singh Rajput 2009-07-04 612
d053b481a5f16d arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 613 phys_hi_rsvd = GENMASK(31, boot_cpu_data.x86_phys_bits - 32);
95ffa2438d0e9c arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-04-29 614
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 615 if (!generic_mtrrs && mtrr_state.enabled) {
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 616 /*
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 617 * Software overwrite of MTRR state, only for generic case.
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 618 * Note that X86_FEATURE_MTRR has been reset in this case.
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 619 */
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 620 init_table();
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 621 pr_info("MTRRs set to read-only\n");
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 622
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 623 return;
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 624 }
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 625
29055dc7428746 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 626 if (generic_mtrrs) {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 627 mtrr_if = &generic_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 628 } else {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 629 switch (boot_cpu_data.x86_vendor) {
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 630 case X86_VENDOR_AMD:
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 631 /* Pre-Athlon (K6) AMD CPU MTRRs */
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 632 if (cpu_feature_enabled(X86_FEATURE_K6_MTRR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 633 mtrr_if = &amd_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 634 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 635 case X86_VENDOR_CENTAUR:
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 636 if (cpu_feature_enabled(X86_FEATURE_CENTAUR_MCR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 637 mtrr_if = ¢aur_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 638 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 639 case X86_VENDOR_CYRIX:
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 640 if (cpu_feature_enabled(X86_FEATURE_CYRIX_ARR))
f8bd9f25c98151 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 641 mtrr_if = &cyrix_mtrr_ops;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 642 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 643 default:
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 644 break;
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 645 }
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 646 }
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 647
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 648 if (mtrr_enabled()) {
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 649 /* Get the number of variable MTRR ranges. */
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 @650 if (mtrr_if == &generic_mtrr_ops)
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 651 rdmsr(MSR_MTRRcap, config, dummy);
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 652 else
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 653 config = mtrr_if->var_regs;
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 654 num_var_ranges = config & MTRR_CAP_VCNT;
03409069520974 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 655
^1da177e4c3f41 arch/i386/kernel/cpu/mtrr/main.c Linus Torvalds 2005-04-16 656 init_table();
45fa71f19a2d73 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 657 if (mtrr_if == &generic_mtrr_ops) {
f9626104a5b681 arch/x86/kernel/cpu/mtrr/main.c Luis R. Rodriguez 2015-05-26 658 /* BIOS may override */
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 659 if (get_mtrr_state()) {
adfe7512e1d0b2 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 660 memory_caching_control |= CACHE_MTRR;
f6b980646b93a8 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2023-05-02 661 changed_by_mtrr_cleanup = mtrr_cleanup();
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 662 } else {
2c15679e8687d5 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 663 mtrr_if = NULL;
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 664 why = "by BIOS";
12031a624af781 arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-05-02 665 }
95ffa2438d0e9c arch/x86/kernel/cpu/mtrr/main.c Yinghai Lu 2008-04-29 666 }
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 667 }
f9626104a5b681 arch/x86/kernel/cpu/mtrr/main.c Luis R. Rodriguez 2015-05-26 668
adfe7512e1d0b2 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-11-02 669 if (!mtrr_enabled())
7882b69eb6cdf6 arch/x86/kernel/cpu/mtrr/mtrr.c Juergen Gross 2022-12-05 670 pr_info("MTRRs disabled %s\n", why);
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 671 }
3b520b238e018e arch/i386/kernel/cpu/mtrr/main.c Shaohua Li 2005-07-07 672
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-11-16 1:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-16 1:52 arch/x86/kernel/cpu/mtrr/mtrr.c:650:33: sparse: sparse: self-comparison always evaluates to true kernel test robot
-- strict thread matches above, loose matches on Subject: below --
2023-11-15 19:23 kernel test robot
2023-11-14 23:56 kernel test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.