cve-2024-53153
Vulnerability from cvelistv5
Published
2024-12-24 11:28
Modified
2025-01-20 06:19
Severity ?
Summary
In the Linux kernel, the following vulnerability has been resolved: PCI: qcom-ep: Move controller cleanups to qcom_pcie_perst_deassert() Currently, the endpoint cleanup function dw_pcie_ep_cleanup() and EPF deinit notify function pci_epc_deinit_notify() are called during the execution of qcom_pcie_perst_assert() i.e., when the host has asserted PERST#. But quickly after this step, refclk will also be disabled by the host. All of the Qcom endpoint SoCs supported as of now depend on the refclk from the host for keeping the controller operational. Due to this limitation, any access to the hardware registers in the absence of refclk will result in a whole endpoint crash. Unfortunately, most of the controller cleanups require accessing the hardware registers (like eDMA cleanup performed in dw_pcie_ep_cleanup(), powering down MHI EPF etc...). So these cleanup functions are currently causing the crash in the endpoint SoC once host asserts PERST#. One way to address this issue is by generating the refclk in the endpoint itself and not depending on the host. But that is not always possible as some of the endpoint designs do require the endpoint to consume refclk from the host (as I was told by the Qcom engineers). Thus, fix this crash by moving the controller cleanups to the start of the qcom_pcie_perst_deassert() function. qcom_pcie_perst_deassert() is called whenever the host has deasserted PERST# and it is guaranteed that the refclk would be active at this point. So at the start of this function (after enabling resources), the controller cleanup can be performed. Once finished, rest of the code execution for PERST# deassert can continue as usual.
References
416baaa9-dc9f-4396-8d5f-8c081fb06d67https://git.kernel.org/stable/c/516969d5765e2302d33b4f251496eedb757d55ea
416baaa9-dc9f-4396-8d5f-8c081fb06d67https://git.kernel.org/stable/c/7d7cf89b119af433354f865fc01017b9f8aa411a
416baaa9-dc9f-4396-8d5f-8c081fb06d67https://git.kernel.org/stable/c/e03b5f1615c84f4139cb53ef8659f4cdb8d6a563
Impacted products
Vendor Product Version
Linux Linux Version: 570d7715eed8a29ac5bd96c7694f060a991e5a31
Version: 570d7715eed8a29ac5bd96c7694f060a991e5a31
Version: 570d7715eed8a29ac5bd96c7694f060a991e5a31
 Create a notification for this product.
   Linux Linux Version: 6.10
 Create a notification for this product.
Show details on NVD website

To clipboard 

{
  "containers": {
    "cna": {
      "affected": [
        {
          "defaultStatus": "unaffected",
          "product": "Linux",
          "programFiles": [
            "drivers/pci/controller/dwc/pcie-qcom-ep.c"
          ],
          "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
          "vendor": "Linux",
          "versions": [
            {
              "lessThan": "e03b5f1615c84f4139cb53ef8659f4cdb8d6a563",
              "status": "affected",
              "version": "570d7715eed8a29ac5bd96c7694f060a991e5a31",
              "versionType": "git"
            },
            {
              "lessThan": "516969d5765e2302d33b4f251496eedb757d55ea",
              "status": "affected",
              "version": "570d7715eed8a29ac5bd96c7694f060a991e5a31",
              "versionType": "git"
            },
            {
              "lessThan": "7d7cf89b119af433354f865fc01017b9f8aa411a",
              "status": "affected",
              "version": "570d7715eed8a29ac5bd96c7694f060a991e5a31",
              "versionType": "git"
            }
          ]
        },
        {
          "defaultStatus": "affected",
          "product": "Linux",
          "programFiles": [
            "drivers/pci/controller/dwc/pcie-qcom-ep.c"
          ],
          "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
          "vendor": "Linux",
          "versions": [
            {
              "status": "affected",
              "version": "6.10"
            },
            {
              "lessThan": "6.10",
              "status": "unaffected",
              "version": "0",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "6.11.*",
              "status": "unaffected",
              "version": "6.11.11",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "6.12.*",
              "status": "unaffected",
              "version": "6.12.2",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "*",
              "status": "unaffected",
              "version": "6.13",
              "versionType": "original_commit_for_fix"
            }
          ]
        }
      ],
      "descriptions": [
        {
          "lang": "en",
          "value": "In the Linux kernel, the following vulnerability has been resolved:\n\nPCI: qcom-ep: Move controller cleanups to qcom_pcie_perst_deassert()\n\nCurrently, the endpoint cleanup function dw_pcie_ep_cleanup() and EPF\ndeinit notify function pci_epc_deinit_notify() are called during the\nexecution of qcom_pcie_perst_assert() i.e., when the host has asserted\nPERST#. But quickly after this step, refclk will also be disabled by the\nhost.\n\nAll of the Qcom endpoint SoCs supported as of now depend on the refclk from\nthe host for keeping the controller operational. Due to this limitation,\nany access to the hardware registers in the absence of refclk will result\nin a whole endpoint crash. Unfortunately, most of the controller cleanups\nrequire accessing the hardware registers (like eDMA cleanup performed in\ndw_pcie_ep_cleanup(), powering down MHI EPF etc...). So these cleanup\nfunctions are currently causing the crash in the endpoint SoC once host\nasserts PERST#.\n\nOne way to address this issue is by generating the refclk in the endpoint\nitself and not depending on the host. But that is not always possible as\nsome of the endpoint designs do require the endpoint to consume refclk from\nthe host (as I was told by the Qcom engineers).\n\nThus, fix this crash by moving the controller cleanups to the start of\nthe qcom_pcie_perst_deassert() function. qcom_pcie_perst_deassert() is\ncalled whenever the host has deasserted PERST# and it is guaranteed that\nthe refclk would be active at this point. So at the start of this function\n(after enabling resources), the controller cleanup can be performed. Once\nfinished, rest of the code execution for PERST# deassert can continue as\nusual."
        }
      ],
      "providerMetadata": {
        "dateUpdated": "2025-01-20T06:19:56.924Z",
        "orgId": "416baaa9-dc9f-4396-8d5f-8c081fb06d67",
        "shortName": "Linux"
      },
      "references": [
        {
          "url": "https://git.kernel.org/stable/c/e03b5f1615c84f4139cb53ef8659f4cdb8d6a563"
        },
        {
          "url": "https://git.kernel.org/stable/c/516969d5765e2302d33b4f251496eedb757d55ea"
        },
        {
          "url": "https://git.kernel.org/stable/c/7d7cf89b119af433354f865fc01017b9f8aa411a"
        }
      ],
      "title": "PCI: qcom-ep: Move controller cleanups to qcom_pcie_perst_deassert()",
      "x_generator": {
        "engine": "bippy-5f407fcff5a0"
      }
    }
  },
  "cveMetadata": {
    "assignerOrgId": "416baaa9-dc9f-4396-8d5f-8c081fb06d67",
    "assignerShortName": "Linux",
    "cveId": "CVE-2024-53153",
    "datePublished": "2024-12-24T11:28:52.518Z",
    "dateReserved": "2024-11-19T17:17:25.000Z",
    "dateUpdated": "2025-01-20T06:19:56.924Z",
    "state": "PUBLISHED"
  },
  "dataType": "CVE_RECORD",
  "dataVersion": "5.1",
  "vulnerability-lookup:meta": {
    "nvd": "{\"cve\":{\"id\":\"CVE-2024-53153\",\"sourceIdentifier\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\",\"published\":\"2024-12-24T12:15:23.467\",\"lastModified\":\"2024-12-24T12:15:23.467\",\"vulnStatus\":\"Received\",\"cveTags\":[],\"descriptions\":[{\"lang\":\"en\",\"value\":\"In the Linux kernel, the following vulnerability has been resolved:\\n\\nPCI: qcom-ep: Move controller cleanups to qcom_pcie_perst_deassert()\\n\\nCurrently, the endpoint cleanup function dw_pcie_ep_cleanup() and EPF\\ndeinit notify function pci_epc_deinit_notify() are called during the\\nexecution of qcom_pcie_perst_assert() i.e., when the host has asserted\\nPERST#. But quickly after this step, refclk will also be disabled by the\\nhost.\\n\\nAll of the Qcom endpoint SoCs supported as of now depend on the refclk from\\nthe host for keeping the controller operational. Due to this limitation,\\nany access to the hardware registers in the absence of refclk will result\\nin a whole endpoint crash. Unfortunately, most of the controller cleanups\\nrequire accessing the hardware registers (like eDMA cleanup performed in\\ndw_pcie_ep_cleanup(), powering down MHI EPF etc...). So these cleanup\\nfunctions are currently causing the crash in the endpoint SoC once host\\nasserts PERST#.\\n\\nOne way to address this issue is by generating the refclk in the endpoint\\nitself and not depending on the host. But that is not always possible as\\nsome of the endpoint designs do require the endpoint to consume refclk from\\nthe host (as I was told by the Qcom engineers).\\n\\nThus, fix this crash by moving the controller cleanups to the start of\\nthe qcom_pcie_perst_deassert() function. qcom_pcie_perst_deassert() is\\ncalled whenever the host has deasserted PERST# and it is guaranteed that\\nthe refclk would be active at this point. So at the start of this function\\n(after enabling resources), the controller cleanup can be performed. Once\\nfinished, rest of the code execution for PERST# deassert can continue as\\nusual.\"}],\"metrics\":{},\"references\":[{\"url\":\"https://git.kernel.org/stable/c/516969d5765e2302d33b4f251496eedb757d55ea\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/7d7cf89b119af433354f865fc01017b9f8aa411a\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/e03b5f1615c84f4139cb53ef8659f4cdb8d6a563\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"}]}}"
  }
}


Log in or create an account to share your comment.




Tags
Taxonomy of the tags.




Sightings

Author Source Type Date

Nomenclature

  • Seen: The vulnerability was mentioned, discussed, or seen somewhere by the user.
  • Confirmed: The vulnerability is confirmed from an analyst perspective.
  • Exploited: This vulnerability was exploited and seen by the user reporting the sighting.
  • Patched: This vulnerability was successfully patched by the user reporting the sighting.
  • Not exploited: This vulnerability was not exploited or seen by the user reporting the sighting.
  • Not confirmed: The user expresses doubt about the veracity of the vulnerability.
  • Not patched: This vulnerability was not successfully patched by the user reporting the sighting.