|
 |
26ccd9 |
From 2dbe3b45879ad614968a75392cb1ef9907d8648d Mon Sep 17 00:00:00 2001
|
|
|
26ccd9 |
From: Dan Williams <dan.j.williams@intel.com>
|
|
|
26ccd9 |
Date: Thu, 28 Apr 2022 15:10:37 -0700
|
|
|
26ccd9 |
Subject: [PATCH 161/217] cxl/memdev: Fix bus_invalidate() crash
|
|
|
26ccd9 |
|
|
|
26ccd9 |
bus_invalidate() attempts to limit the invalidation of memdevs to a single
|
|
|
26ccd9 |
bus scope. However, the ordering of bus_invalidate() leads to a use after
|
|
|
26ccd9 |
free. Unconditionally invalidate memdevs (disconnect memdevs from their
|
|
|
26ccd9 |
endpoints) and resotre on next lookup. Otherwise the following command
|
|
|
26ccd9 |
results in the following backtrace with cxl_test:
|
|
|
26ccd9 |
|
|
|
26ccd9 |
cxl disable-memdev 5,1 --force
|
|
|
26ccd9 |
|
|
|
26ccd9 |
#2 0x00007ffff7fb97d4 in snprintf (__fmt=0x7ffff7fbc3ed "%s/driver", __n=98,
|
|
|
26ccd9 |
__s=0x574d545619f7bae2 <error: Cannot access memory at address 0x574d545619f7bae2>)
|
|
|
26ccd9 |
at /usr/include/bits/stdio2.h:71
|
|
|
26ccd9 |
#3 cxl_port_is_enabled (port=port@entry=0x422eb0) at ../cxl/lib/libcxl.c:1379
|
|
|
26ccd9 |
#4 0x00007ffff7fb99a9 in cxl_port_get_bus (port=0x422eb0) at ../cxl/lib/libcxl.c:1339
|
|
|
26ccd9 |
#5 0x00007ffff7fba3d0 in bus_invalidate (bus=bus@entry=0x421740) at ../cxl/lib/libcxl.c:549
|
|
|
26ccd9 |
#6 0x00007ffff7fba4e7 in cxl_memdev_disable_invalidate (memdev=0x416fd0) at ../cxl/lib/libcxl.c:596
|
|
|
26ccd9 |
#7 0x000000000040624e in memdev_action (argc=<optimized out>, argv=<optimized out>, ctx=0x4152a0,
|
|
|
26ccd9 |
action=action@entry=0x406b70 <action_disable>, options=options@entry=0x40fca0 <disable_options>,
|
|
|
26ccd9 |
usage=usage@entry=0x40f4b0 "cxl disable-memdev <mem0> [<mem1>..<memN>] [<options>]")
|
|
|
26ccd9 |
at ../cxl/memdev.c:506
|
|
|
26ccd9 |
#8 0x0000000000406d57 in cmd_disable_memdev (argc=<optimized out>, argv=<optimized out>,
|
|
|
26ccd9 |
|
|
|
26ccd9 |
Link: https://lore.kernel.org/r/165118383756.1676208.5717187278816036969.stgit@dwillia2-desk3.amr.corp.intel.com
|
|
|
26ccd9 |
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
|
|
|
26ccd9 |
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
|
|
|
26ccd9 |
---
|
|
|
26ccd9 |
cxl/lib/libcxl.c | 3 +--
|
|
|
26ccd9 |
1 file changed, 1 insertion(+), 2 deletions(-)
|
|
|
26ccd9 |
|
|
|
26ccd9 |
diff --git a/cxl/lib/libcxl.c b/cxl/lib/libcxl.c
|
|
|
26ccd9 |
index 0e8dd20..374b0f1 100644
|
|
|
26ccd9 |
--- a/cxl/lib/libcxl.c
|
|
|
26ccd9 |
+++ b/cxl/lib/libcxl.c
|
|
|
26ccd9 |
@@ -546,8 +546,7 @@ static void bus_invalidate(struct cxl_bus *bus)
|
|
|
26ccd9 |
* indeterminate, delete them all and start over.
|
|
|
26ccd9 |
*/
|
|
|
26ccd9 |
cxl_memdev_foreach(ctx, memdev)
|
|
|
26ccd9 |
- if (cxl_memdev_get_bus(memdev) == bus)
|
|
|
26ccd9 |
- memdev->endpoint = NULL;
|
|
|
26ccd9 |
+ memdev->endpoint = NULL;
|
|
|
26ccd9 |
|
|
|
26ccd9 |
bus_port = cxl_bus_get_port(bus);
|
|
|
26ccd9 |
list_for_each_safe(&bus_port->child_ports, port, _p, list)
|
|
|
26ccd9 |
--
|
|
|
26ccd9 |
2.27.0
|
|
|
26ccd9 |
|