linux-kernel - Re: [PATCH v10 04/10] PCI: cadence: Add support for High Perf Architecture (HPA) controller

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <b6f8d7cf-bbd0-42eb-8d73-5715a614ae98@cixtech.com>
Date: Fri, 31 Oct 2025 17:20:08 +0800
From: Hans Zhang <hans.zhang@...tech.com>
To: Manivannan Sadhasivam <mani@...nel.org>
Cc: bhelgaas@...gle.com, helgaas@...nel.org, lpieralisi@...nel.org,
 kw@...ux.com, robh@...nel.org, kwilczynski@...nel.org, krzk+dt@...nel.org,
 conor+dt@...nel.org, 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
Subject: Re: [PATCH v10 04/10] PCI: cadence: Add support for High Perf
 Architecture (HPA) controller



On 10/31/2025 5:11 PM, Manivannan Sadhasivam wrote:
>> +
>> +static int cdns_pcie_hpa_host_wait_for_link(struct cdns_pcie *pcie)
>> +{
>> +     struct device *dev = pcie->dev;
>> +     struct cdns_pcie_rc *rc;
>> +     int retries, ret;
>> +
>> +     rc = container_of(pcie, struct cdns_pcie_rc, pcie);
>> +
>> +     /* Check if the link is up or not */
>> +     for (retries = 0; retries < LINK_WAIT_MAX_RETRIES; retries++) {
>> +             if (cdns_pcie_hpa_link_up(pcie)) {
>> +                     dev_info(dev, "Link up\n");
>> +                     return 0;
>> +             }
>> +             usleep_range(LINK_WAIT_USLEEP_MIN, LINK_WAIT_USLEEP_MAX);
>> +     }
>> +     if (rc->quirk_retrain_flag)
>> +             ret = cdns_pcie_retrain(pcie);
>> +     return ret;
> If 'quirk_retrain_flag' was not set, you are 'ret' will be uninitialized.


Hi Mani,

Thank you very much for your reply.

When the robot gave a warning, I replied as follows:

https://lore.kernel.org/linux-pci/293858b1-db91-4525-b8b3-c98c7837ec73@cixtech.com/

Will change.


Please ask Manikandan to explain any other questions to Mani.


Best regards,
Hans