| 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
| ||
|
Message-ID:
<DS0PR07MB10492CF1D21B9FCF3F94D0A4EA22BA@DS0PR07MB10492.namprd07.prod.outlook.com>
Date: Tue, 12 Aug 2025 04:00:44 +0000
From: Manikandan Karunakaran Pillai <mpillai@...ence.com>
To: kernel test robot <lkp@...el.com>,
"hans.zhang@...tech.com"
<hans.zhang@...tech.com>,
"bhelgaas@...gle.com" <bhelgaas@...gle.com>,
"lpieralisi@...nel.org" <lpieralisi@...nel.org>,
"kw@...ux.com"
<kw@...ux.com>, "mani@...nel.org" <mani@...nel.org>,
"robh@...nel.org"
<robh@...nel.org>,
"kwilczynski@...nel.org" <kwilczynski@...nel.org>,
"krzk+dt@...nel.org" <krzk+dt@...nel.org>,
"conor+dt@...nel.org"
<conor+dt@...nel.org>
CC: "llvm@...ts.linux.dev" <llvm@...ts.linux.dev>,
"oe-kbuild-all@...ts.linux.dev" <oe-kbuild-all@...ts.linux.dev>,
"fugang.duan@...tech.com" <fugang.duan@...tech.com>,
"guoyin.chen@...tech.com" <guoyin.chen@...tech.com>,
"peter.chen@...tech.com"
<peter.chen@...tech.com>,
"cix-kernel-upstream@...tech.com"
<cix-kernel-upstream@...tech.com>,
"linux-pci@...r.kernel.org"
<linux-pci@...r.kernel.org>,
"devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>,
Hans Zhang <hans.zhang@...tech.com>
Subject: RE: [PATCH v6 06/12] PCI: cadence: Add support for High Performance
Arch(HPA) controller
>-----Original Message-----
>From: kernel test robot <lkp@...el.com>
>Sent: Saturday, August 9, 2025 1:27 AM
>To: hans.zhang@...tech.com; bhelgaas@...gle.com; lpieralisi@...nel.org;
>kw@...ux.com; mani@...nel.org; robh@...nel.org; kwilczynski@...nel.org;
>krzk+dt@...nel.org; conor+dt@...nel.org
>Cc: llvm@...ts.linux.dev; oe-kbuild-all@...ts.linux.dev; Manikandan
>Karunakaran Pillai <mpillai@...ence.com>; fugang.duan@...tech.com;
>guoyin.chen@...tech.com; peter.chen@...tech.com; cix-kernel-
>upstream@...tech.com; linux-pci@...r.kernel.org;
>devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; Hans Zhang
><hans.zhang@...tech.com>
>Subject: Re: [PATCH v6 06/12] PCI: cadence: Add support for High Performance
>Arch(HPA) controller
>
>EXTERNAL MAIL
>
>
>Hi,
>
>kernel test robot noticed the following build warnings:
>
>[auto build test WARNING on
>37816488247ddddbc3de113c78c83572274b1e2e]
>
>url: https://urldefense.com/v3/__https://github.com/intel-lab-
>lkp/linux/commits/hans-zhang-cixtech-com/PCI-cadence-Split-PCIe-controller-
>header-file/20250808-
>154018__;!!EHscmS1ygiU1lA!ASDwczVdgevm90G9xJCBQnIw0Q52SASLPGwt1M
>z_xvH1T9cvMw2V5abHUz_wykYs72cK6Uit$
>base: 37816488247ddddbc3de113c78c83572274b1e2e
>patch link:
>https://urldefense.com/v3/__https://lore.kernel.org/r/20250808072929.40906
>94-7-
>hans.zhang*40cixtech.com__;JQ!!EHscmS1ygiU1lA!ASDwczVdgevm90G9xJCBQ
>nIw0Q52SASLPGwt1Mz_xvH1T9cvMw2V5abHUz_wykYs7_AhVc73$
>patch subject: [PATCH v6 06/12] PCI: cadence: Add support for High
>Performance Arch(HPA) controller
>config: arm64-randconfig-002-20250809
>(https://urldefense.com/v3/__https://download.01.org/0day-
>ci/archive/20250809/202508090343.TWUqM8E7-
>lkp@...el.com/config__;!!EHscmS1ygiU1lA!ASDwczVdgevm90G9xJCBQnIw0Q5
>2SASLPGwt1Mz_xvH1T9cvMw2V5abHUz_wykYs74HFn7yG$ )
>compiler: clang version 22.0.0git
>(https://urldefense.com/v3/__https://github.com/llvm/llvm-
>project__;!!EHscmS1ygiU1lA!ASDwczVdgevm90G9xJCBQnIw0Q52SASLPGwt1M
>z_xvH1T9cvMw2V5abHUz_wykYs745bte2U$
>3769ce013be2879bf0b329c14a16f5cb766f26ce)
>reproduce (this is a W=1 build):
>(https://urldefense.com/v3/__https://download.01.org/0day-
>ci/archive/20250809/202508090343.TWUqM8E7-
>lkp@...el.com/reproduce__;!!EHscmS1ygiU1lA!ASDwczVdgevm90G9xJCBQnIw
>0Q52SASLPGwt1Mz_xvH1T9cvMw2V5abHUz_wykYs78iupW3p$ )
>
>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://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-
>all/202508090343.TWUqM8E7-
>lkp@...el.com/__;!!EHscmS1ygiU1lA!ASDwczVdgevm90G9xJCBQnIw0Q52SASL
>PGwt1Mz_xvH1T9cvMw2V5abHUz_wykYs7_x3Ix-3$
>
>All warnings (new ones prefixed by >>):
>
>>> drivers/pci/controller/cadence/pcie-cadence-host-hpa.c:66:3: warning:
>variable 'desc0' is uninitialized when used here [-Wuninitialized]
> 66 | desc0 |=
>CDNS_PCIE_HPA_AT_OB_REGION_DESC0_TYPE_CONF_TYPE0;
> | ^~~~~
> drivers/pci/controller/cadence/pcie-cadence-host-hpa.c:28:18: note: initialize
>the variable 'desc0' to silence this warning
> 28 | u32 addr0, desc0, desc1, ctrl0;
> | ^
> | = 0
> 1 warning generated.
>
>
>vim +/desc0 +66 drivers/pci/controller/cadence/pcie-cadence-host-hpa.c
>
> 20
> 21 void __iomem *cdns_pci_hpa_map_bus(struct pci_bus *bus, unsigned
>int devfn,
> 22 int where)
> 23 {
> 24 struct pci_host_bridge *bridge = pci_find_host_bridge(bus);
> 25 struct cdns_pcie_rc *rc = pci_host_bridge_priv(bridge);
> 26 struct cdns_pcie *pcie = &rc->pcie;
> 27 unsigned int busn = bus->number;
> 28 u32 addr0, desc0, desc1, ctrl0;
> 29 u32 regval;
> 30
> 31 if (pci_is_root_bus(bus)) {
> 32 /*
> 33 * Only the root port (devfn == 0) is connected to this
>bus.
> 34 * All other PCI devices are behind some bridge hence
>on another
> 35 * bus.
> 36 */
> 37 if (devfn)
> 38 return NULL;
> 39
> 40 return pcie->reg_base + (where & 0xfff);
> 41 }
> 42
> 43 /* Clear AXI link-down status */
> 44 regval = cdns_pcie_hpa_readl(pcie, REG_BANK_AXI_SLAVE,
>CDNS_PCIE_HPA_AT_LINKDOWN);
> 45 cdns_pcie_hpa_writel(pcie, REG_BANK_AXI_SLAVE,
>CDNS_PCIE_HPA_AT_LINKDOWN,
> 46 (regval & ~GENMASK(0, 0)));
> 47
> 48 desc1 = 0;
> 49 ctrl0 = 0;
> 50
> 51 /* Update Output registers for AXI region 0. */
> 52 addr0 =
>CDNS_PCIE_HPA_AT_OB_REGION_PCI_ADDR0_NBITS(12) |
> 53
> CDNS_PCIE_HPA_AT_OB_REGION_PCI_ADDR0_DEVFN(devfn) |
> 54
> CDNS_PCIE_HPA_AT_OB_REGION_PCI_ADDR0_BUS(busn);
> 55 cdns_pcie_hpa_writel(pcie, REG_BANK_AXI_SLAVE,
> 56
>CDNS_PCIE_HPA_AT_OB_REGION_PCI_ADDR0(0), addr0);
> 57
> 58 desc1 = cdns_pcie_hpa_readl(pcie, REG_BANK_AXI_SLAVE,
> 59
>CDNS_PCIE_HPA_AT_OB_REGION_DESC1(0));
> 60 desc1 &=
>~CDNS_PCIE_HPA_AT_OB_REGION_DESC1_DEVFN_MASK;
> 61 desc1 |= CDNS_PCIE_HPA_AT_OB_REGION_DESC1_DEVFN(0);
> 62 ctrl0 = CDNS_PCIE_HPA_AT_OB_REGION_CTRL0_SUPPLY_BUS |
> 63
> CDNS_PCIE_HPA_AT_OB_REGION_CTRL0_SUPPLY_DEV_FN;
> 64
> 65 if (busn == bridge->busnr + 1)
> > 66 desc0 |=
>CDNS_PCIE_HPA_AT_OB_REGION_DESC0_TYPE_CONF_TYPE0;
> 67 else
> 68 desc0 |=
>CDNS_PCIE_HPA_AT_OB_REGION_DESC0_TYPE_CONF_TYPE1;
> 69
> 70 cdns_pcie_hpa_writel(pcie, REG_BANK_AXI_SLAVE,
> 71 CDNS_PCIE_HPA_AT_OB_REGION_DESC0(0),
>desc0);
> 72 cdns_pcie_hpa_writel(pcie, REG_BANK_AXI_SLAVE,
> 73 CDNS_PCIE_HPA_AT_OB_REGION_DESC1(0),
>desc1);
> 74 cdns_pcie_hpa_writel(pcie, REG_BANK_AXI_SLAVE,
> 75 CDNS_PCIE_HPA_AT_OB_REGION_CTRL0(0),
>ctrl0);
> 76
> 77 return rc->cfg_base + (where & 0xfff);
> 78 }
> 79
>
Missed these changes while restructuring the patch again. Will fix these in the next version.
>--
>0-DAY CI Kernel Test Service
>https://urldefense.com/v3/__https://github.com/intel/lkp-
>tests/wiki__;!!EHscmS1ygiU1lA!ASDwczVdgevm90G9xJCBQnIw0Q52SASLPGwt1
>Mz_xvH1T9cvMw2V5abHUz_wykYs7-WOjZLt$
Powered by blists - more mailing lists