cxl/port: Fix endpoint refcount leak

An endpoint can be unregistered via two paths. Either its parent port is
unregistered, or the memdev that registered the endpoint is removed. The
memdev remove path is responsible for synchronizing against the parent
->remove() event and if the memdev remove path wins, manually trigger
unregister_port() via devm_release_action(). Until that race is resolved
the memdev remove path holds a reference on the endpoint.

If the parent port for the endpoint can not be found that is an
indication that the endpoint has already been registered. Be sure to
drop the reference in all exit paths from delete_endpoint().

Fixes: 8dd2bc0f8e ("cxl/mem: Add the cxl_mem driver")
Link: https://lore.kernel.org/r/164454148209.3429624.12905500880311609053.stgit@dwillia2-desk3.amr.corp.intel.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
This commit is contained in:
Dan Williams 2022-02-10 17:04:42 -08:00
parent e6e17cc6ed
commit 41ae9105f5
1 changed files with 2 additions and 1 deletions

View File

@ -850,7 +850,7 @@ static void delete_endpoint(void *data)
parent_port = cxl_mem_find_port(cxlmd);
if (!parent_port)
return;
goto out;
parent = &parent_port->dev;
cxl_device_lock(parent);
@ -860,6 +860,7 @@ static void delete_endpoint(void *data)
}
cxl_device_unlock(parent);
put_device(parent);
out:
put_device(&endpoint->dev);
}