| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sat, 20 Jan 2024 16:16:45 +0800
From: kernel test robot <lkp@...el.com>
To: Jedrzej Jagielski <jedrzej.jagielski@...el.com>,
intel-wired-lan@...ts.osuosl.org
Cc: llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev,
anthony.l.nguyen@...el.com, netdev@...r.kernel.org,
Jedrzej Jagielski <jedrzej.jagielski@...el.com>
Subject: Re: [PATCH iwl-next v3 3/3] ixgbe: Cleanup after type convertion
Hi Jedrzej,
kernel test robot noticed the following build errors:
[auto build test ERROR on tnguy-next-queue/dev-queue]
url: https://github.com/intel-lab-lkp/linux/commits/Jedrzej-Jagielski/ixgbe-Fix-smatch-warnings-after-type-convertion/20240119-015659
base: https://git.kernel.org/pub/scm/linux/kernel/git/tnguy/next-queue.git dev-queue
patch link: https://lore.kernel.org/r/20240118134332.470907-3-jedrzej.jagielski%40intel.com
patch subject: [PATCH iwl-next v3 3/3] ixgbe: Cleanup after type convertion
config: x86_64-rhel-8.3-rust (https://download.01.org/0day-ci/archive/20240120/202401201602.Mg8UEw6K-lkp@intel.com/config)
compiler: ClangBuiltLinux clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240120/202401201602.Mg8UEw6K-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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202401201602.Mg8UEw6K-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c:774:27: error: use of undeclared identifier 'autoc2'
774 | u32 pma_pmd_10g_serial = autoc2 & IXGBE_AUTOC2_10G_SERIAL_PMA_PMD_MASK;
| ^
1 error generated.
vim +/autoc2 +774 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c
cd7e1f0b056c07 drivers/net/ixgbe/ixgbe_82599.c Don Skidmore 2009-10-08 761
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 762 /**
8620a103b5e38d drivers/net/ixgbe/ixgbe_82599.c Mallikarjuna R Chilakala 2009-09-01 763 * ixgbe_setup_mac_link_82599 - Set MAC link speed
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 764 * @hw: pointer to hardware structure
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 765 * @speed: new link speed
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 766 * @autoneg_wait_to_complete: true when waiting for completion is needed
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 767 *
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 768 * Set the link speed in the AUTOC register and restarts link.
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 769 **/
7e23e4751a3586 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jedrzej Jagielski 2024-01-18 770 static int ixgbe_setup_mac_link_82599(struct ixgbe_hw *hw,
fd0326f2cf9e5d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Josh Hay 2012-12-15 771 ixgbe_link_speed speed,
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 772 bool autoneg_wait_to_complete)
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 773 {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 @774 u32 pma_pmd_10g_serial = autoc2 & IXGBE_AUTOC2_10G_SERIAL_PMA_PMD_MASK;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 775 ixgbe_link_speed link_capabilities = IXGBE_LINK_SPEED_UNKNOWN;
36f3bb88d9a5de drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jedrzej Jagielski 2024-01-18 776 u32 autoc2 = IXGBE_READ_REG(hw, IXGBE_AUTOC2);
36f3bb88d9a5de drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jedrzej Jagielski 2024-01-18 777 u32 pma_pmd_1g, link_mode, links_reg, i;
36f3bb88d9a5de drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jedrzej Jagielski 2024-01-18 778 bool autoneg = false;
36f3bb88d9a5de drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jedrzej Jagielski 2024-01-18 779 int status;
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 780
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 781 /* holds the value of AUTOC register at this current point in time */
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 782 u32 current_autoc = IXGBE_READ_REG(hw, IXGBE_AUTOC);
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 783 /* holds the cached value of AUTOC register */
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 784 u32 orig_autoc = 0;
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 785 /* temporary variable used for comparison purposes */
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 786 u32 autoc = current_autoc;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 787
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 788 /* Check to see if speed passed in is supported. */
9cdcf098800d95 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Don Skidmore 2012-02-17 789 status = hw->mac.ops.get_link_capabilities(hw, &link_capabilities,
9cdcf098800d95 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Don Skidmore 2012-02-17 790 &autoneg);
e90dd264566405 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Mark Rustad 2014-07-22 791 if (status)
e90dd264566405 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Mark Rustad 2014-07-22 792 return status;
0b0c2b31bdf8d6 drivers/net/ixgbe/ixgbe_82599.c Emil Tantilov 2011-02-26 793
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 794 speed &= link_capabilities;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 795
e90dd264566405 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Mark Rustad 2014-07-22 796 if (speed == IXGBE_LINK_SPEED_UNKNOWN)
5795f533f30a80 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jedrzej Jagielski 2023-12-18 797 return -EINVAL;
50ac58ba1d707d drivers/net/ixgbe/ixgbe_82599.c Peter P Waskiewicz Jr 2009-06-04 798
1eb99d5ac44e2a drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-04-09 799 /* Use stored value (EEPROM defaults) of AUTOC to find KR/KX4 support*/
1eb99d5ac44e2a drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-04-09 800 if (hw->mac.orig_link_settings_stored)
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 801 orig_autoc = hw->mac.orig_autoc;
1eb99d5ac44e2a drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-04-09 802 else
1eb99d5ac44e2a drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-04-09 803 orig_autoc = autoc;
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 804
5e82f2f07645ef drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Emil Tantilov 2013-04-12 805 link_mode = autoc & IXGBE_AUTOC_LMS_MASK;
5e82f2f07645ef drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Emil Tantilov 2013-04-12 806 pma_pmd_1g = autoc & IXGBE_AUTOC_1G_PMA_PMD_MASK;
1eb99d5ac44e2a drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-04-09 807
50ac58ba1d707d drivers/net/ixgbe/ixgbe_82599.c Peter P Waskiewicz Jr 2009-06-04 808 if (link_mode == IXGBE_AUTOC_LMS_KX4_KX_KR ||
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 809 link_mode == IXGBE_AUTOC_LMS_KX4_KX_KR_1G_AN ||
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 810 link_mode == IXGBE_AUTOC_LMS_KX4_KX_KR_SGMII) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 811 /* Set KX4/KX/KR support according to speed requested */
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 812 autoc &= ~(IXGBE_AUTOC_KX4_KX_SUPP_MASK | IXGBE_AUTOC_KR_SUPP);
55461ddbcb0b36 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Emil Tantilov 2012-08-10 813 if (speed & IXGBE_LINK_SPEED_10GB_FULL) {
1eb99d5ac44e2a drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-04-09 814 if (orig_autoc & IXGBE_AUTOC_KX4_SUPP)
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 815 autoc |= IXGBE_AUTOC_KX4_SUPP;
cd7e1f0b056c07 drivers/net/ixgbe/ixgbe_82599.c Don Skidmore 2009-10-08 816 if ((orig_autoc & IXGBE_AUTOC_KR_SUPP) &&
cd7e1f0b056c07 drivers/net/ixgbe/ixgbe_82599.c Don Skidmore 2009-10-08 817 (hw->phy.smart_speed_active == false))
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 818 autoc |= IXGBE_AUTOC_KR_SUPP;
55461ddbcb0b36 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Emil Tantilov 2012-08-10 819 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 820 if (speed & IXGBE_LINK_SPEED_1GB_FULL)
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 821 autoc |= IXGBE_AUTOC_KX_SUPP;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 822 } else if ((pma_pmd_1g == IXGBE_AUTOC_1G_SFI) &&
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 823 (link_mode == IXGBE_AUTOC_LMS_1G_LINK_NO_AN ||
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 824 link_mode == IXGBE_AUTOC_LMS_1G_AN)) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 825 /* Switch from 1G SFI to 10G SFI if requested */
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 826 if ((speed == IXGBE_LINK_SPEED_10GB_FULL) &&
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 827 (pma_pmd_10g_serial == IXGBE_AUTOC2_10G_SFI)) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 828 autoc &= ~IXGBE_AUTOC_LMS_MASK;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 829 autoc |= IXGBE_AUTOC_LMS_10G_SERIAL;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 830 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 831 } else if ((pma_pmd_10g_serial == IXGBE_AUTOC2_10G_SFI) &&
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 832 (link_mode == IXGBE_AUTOC_LMS_10G_SERIAL)) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 833 /* Switch from 10G SFI to 1G SFI if requested */
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 834 if ((speed == IXGBE_LINK_SPEED_1GB_FULL) &&
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 835 (pma_pmd_1g == IXGBE_AUTOC_1G_SFI)) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 836 autoc &= ~IXGBE_AUTOC_LMS_MASK;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 837 if (autoneg)
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 838 autoc |= IXGBE_AUTOC_LMS_1G_AN;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 839 else
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 840 autoc |= IXGBE_AUTOC_LMS_1G_LINK_NO_AN;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 841 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 842 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 843
ee98b577e7711d drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 844 if (autoc != current_autoc) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 845 /* Restart link */
429d6a3be9b656 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Don Skidmore 2014-02-27 846 status = hw->mac.ops.prot_autoc_write(hw, autoc, false);
f8cf7a00d82b6c drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Don Skidmore 2014-03-19 847 if (status)
e90dd264566405 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Mark Rustad 2014-07-22 848 return status;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 849
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 850 /* Only poll for autoneg to complete if specified to do so */
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 851 if (autoneg_wait_to_complete) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 852 if (link_mode == IXGBE_AUTOC_LMS_KX4_KX_KR ||
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 853 link_mode == IXGBE_AUTOC_LMS_KX4_KX_KR_1G_AN ||
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 854 link_mode == IXGBE_AUTOC_LMS_KX4_KX_KR_SGMII) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 855 links_reg = 0; /*Just in case Autoneg time=0*/
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 856 for (i = 0; i < IXGBE_AUTO_NEG_TIME; i++) {
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 857 links_reg =
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 858 IXGBE_READ_REG(hw, IXGBE_LINKS);
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 859 if (links_reg & IXGBE_LINKS_KX_AN_COMP)
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 860 break;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 861 msleep(100);
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 862 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 863 if (!(links_reg & IXGBE_LINKS_KX_AN_COMP)) {
5795f533f30a80 drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jedrzej Jagielski 2023-12-18 864 status = -EIO;
305f8cec7be51e drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c Jacob Keller 2014-02-22 865 hw_dbg(hw, "Autoneg did not complete.\n");
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 866 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 867 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 868 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 869
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 870 /* Add delay to filter out noises during initial link setup */
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 871 msleep(50);
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 872 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 873
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 874 return status;
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 875 }
11afc1b1fd802c drivers/net/ixgbe/ixgbe_82599.c PJ Waskiewicz 2009-02-27 876
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists