cxl/region: Fix memdev reuse check
commitf04facfb99
upstream. Due to a typo, the check of whether or not a memdev has already been used as a target for the region (above code piece) will always be skipped. Given a memdev with more than one HDM decoder, an interleaved region can be created that maps multiple HPAs to the same DPA. According to CXL spec 3.0 8.1.3.8.4, "Aliasing (mapping more than one Host Physical Address (HPA) to a single Device Physical Address) is forbidden." Fix this by using existing iterator for memdev reuse check. Cc: <stable@vger.kernel.org> Fixes:384e624bb2
("cxl/region: Attach endpoint decoders") Signed-off-by: Fan Ni <fan.ni@samsung.com> Link: https://lore.kernel.org/r/20221107212153.745993-1-fan.ni@samsung.com Signed-off-by: Dan Williams <dan.j.williams@intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
fab4655990
commit
189c499376
|
@ -1226,7 +1226,7 @@ static int cxl_region_attach(struct cxl_region *cxlr,
|
|||
struct cxl_endpoint_decoder *cxled_target;
|
||||
struct cxl_memdev *cxlmd_target;
|
||||
|
||||
cxled_target = p->targets[pos];
|
||||
cxled_target = p->targets[i];
|
||||
if (!cxled_target)
|
||||
continue;
|
||||
|
||||
|
|
Loading…
Reference in New Issue