On Thu, 24 Sep 2020 at 22:25, Christian König christian.koenig@amd.com wrote:
Am 24.09.20 um 07:18 schrieb Dave Airlie:
From: Dave Airlie airlied@redhat.com
Signed-off-by: Dave Airlie airlied@redhat.com
NAK, completely unnecessary.
We should rather do the remaining accounting in the already existing release_notify() callback.
That makes much more sense and if I'm not completely mistaken could actually fix a bug in amdgpu.
release_notify is called from one path, but there is a path in eviction where if it gets pass 0 placements it tears down the memory, and allocates a tt.
I'm considering whether it should be acceptable to call evict with no placements (though maybe that just means evict to system).
Dave.