diff options
| author | Guixin Liu <kanie@linux.alibaba.com> | 2026-10-08 11:46:13 +0800 |
|---|---|---|
| committer | Dave Jiang <dave.jiang@intel.com> | 2026-10-09 14:09:24 -0700 |
| commit | ec2e30a682945616d194d7999cf96606b7ee5a88 (patch) | |
| tree | 6d50025ded8033b61422935e3da8ccfd09e799ad /tools/perf/scripts/python | |
| parent | d2654fa0d841fa36ffa76577f83a2810d643ae19 (diff) | |
| download | linux-next-ec2e30a682945616d194d7999cf96606b7ee5a88.tar.gz linux-next-ec2e30a682945616d194d7999cf96606b7ee5a88.zip | |
cxl/pci: Program the Port GPF timeouts before caching the DVSEC
cxl_gpf_port_setup() caches the Port GPF DVSEC offset in the dport
before programming the phase timeouts, and ignores the return value
of both update_gpf_port_dvsec() calls: the function always reports
success.
A failing config write is therefore not only silent, it is also
permanent: the timeouts stay at the hardware defaults instead of the
maximum flush window the kernel is asking for, and the cached offset
keeps any later endpoint attach from retrying the update.
Propagate the errors and cache the offset only after both phases are
programmed, so the next endpoint attach retries the setup.
Fixes: 6af941db6a60 ("cxl/pci: Update Port GPF timeout only when the first EP attaching")
Cc: stable@vger.kernel.org
Assisted-by: LLM
Signed-off-by: Guixin Liu <kanie@linux.alibaba.com>
Reviewed-by: Jonathan Cameron <jonathan.cameron@oss.qualcomm.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
Reviewed-by: Alison Schofield <alison.schofield@intel.com>
Link: https://patch.msgid.link/20261008034619.28059-3-kanie@linux.alibaba.com
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions
