On Thu, 24 Sep 2020 at 22:25, Christian König <christian.koenig@xxxxxxx> wrote: > > Am 24.09.20 um 07:18 schrieb Dave Airlie: > > From: Dave Airlie <airlied@xxxxxxxxxx> > > > > Signed-off-by: Dave Airlie <airlied@xxxxxxxxxx> > > 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. _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel