When remaining resources are being cleaned up on driver close,
outstanding VM mappings may result in resources being leaked, due
to an object reference loop, as shown below, with each object (or
set of objects) referencing the object below it:
PVR GEM Object
GPU scheduler "finished" fence
GPU scheduler "scheduled" fence
PVR driver "done" fence
PVR Context
PVR VM Context
PVR VM Mappings
PVR GEM Object
The reference that the PVR VM Context has on the VM mappings is a
soft one, in the sense that the freeing of outstanding VM mappings
is done as part of VM context destruction; no reference counts are
involved, as is the case for all the other references in the loop.
To break the reference loop during cleanup, free the outstanding
VM mappings before destroying the PVR Context associated with the
VM context.
This is facilitated by tracking the live contexts associated with each
list, implemented in a separate patch to make the main patch cleaner
and easier to grok.
---
Changes in v1 -> v2:
- [PATCH 1/2] Add justification to the commit message so it stands on
its own (e.g. during bisect)
Changes in v2 -> v3:
- [ALL] Add Frank's Rb [1][2]
- [ALL] Add Cc: stable tag
[1]: https://lore.kernel.org/dri-devel/1a206b08485a1ebea0aeff5306c0f5b6eb7c315d.camel@xxxxxxxxxx/
[2]: https://lore.kernel.org/dri-devel/b75e658cc5ce9ac24df2c31c1a6fd798a5f3a87f.camel@xxxxxxxxxx/
---
Brendan King (2):
drm/imagination: Add a per-file PVR context list
drm/imagination: Break an object reference loop
drivers/gpu/drm/imagination/pvr_context.c | 33 +++++++++++++++++++++++
drivers/gpu/drm/imagination/pvr_context.h | 21 +++++++++++++++
drivers/gpu/drm/imagination/pvr_device.h | 10 +++++++
drivers/gpu/drm/imagination/pvr_drv.c | 3 +++
drivers/gpu/drm/imagination/pvr_vm.c | 22 ++++++++++++---
drivers/gpu/drm/imagination/pvr_vm.h | 1 +
6 files changed, 86 insertions(+), 4 deletions(-)
base-commit: 83f000784844cb9d4669ef1a3366479db3197b33
--
2.47.0
Attachment:
OpenPGP_signature.asc
Description: OpenPGP digital signature
