cxl/hdm: Fix skip allocations vs multiple pmem allocations
authorDan Williams <dan.j.williams@intel.com>
Fri, 5 Aug 2022 22:12:27 +0000 (15:12 -0700)
committerDan Williams <dan.j.williams@intel.com>
Fri, 5 Aug 2022 23:11:38 +0000 (16:11 -0700)
commit1cd8a2537eb07751d405ab7e2223f20338a90506
tree50ec48c10cb3a7155a2c02b96733de53b7353f2b
parent4d8e4ea5bb396897111e8a740201bfd3c5926170
cxl/hdm: Fix skip allocations vs multiple pmem allocations

Vishal notes that when attempting to define a second pmem region on a
device the DPA allocation fails with a message of the form:

    decoder11.1: failed to reserve skipped space

Recall that the skip setting is used when there is a pmem allocation in
the presence of free ram DPA space. The first pmem allocation skips over
the free ram and subsequent pmem allocations do not require a skip. The
bug is that a skip is still attempted and the DPA reservation code
flags the double skip allocation conflict.

Fixes: cf880423b6a0 ("cxl/hdm: Add support for allocating DPA to an endpoint decoder")
Reported-by: Vishal Verma <vishal.l.verma@intel.com>
Tested-by: Vishal Verma <vishal.l.verma@intel.com>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
Link: https://lore.kernel.org/r/165973754730.1558392.15466392461645857658.stgit@dwillia2-xfh.jf.intel.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/cxl/core/hdm.c