On 6/7/21 12:40 PM, Christian König wrote:
That somehow got missing.
Signed-off-by: Christian König <christian.koenig@xxxxxxx>
Fixes: cb1c81467af3 ("drm/ttm: flip the switch for driver allocated resources v2")
Reported-by: Thomas Hellström <thomas.hellstrom@xxxxxxxxxxxxxxx> Reviewed-by: Thomas Hellström <thomas.hellstrom@xxxxxxxxxxxxxxx>
--- drivers/gpu/drm/ttm/ttm_range_manager.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/ttm/ttm_range_manager.c b/drivers/gpu/drm/ttm/ttm_range_manager.c index c32e1aee2481..03395386e8a7 100644 --- a/drivers/gpu/drm/ttm/ttm_range_manager.c +++ b/drivers/gpu/drm/ttm/ttm_range_manager.c @@ -88,12 +88,14 @@ static int ttm_range_man_alloc(struct ttm_resource_manager *man, place->fpfn, lpfn, mode); spin_unlock(&rman->lock);- if (unlikely(ret))+ if (unlikely(ret)) { kfree(node); - else - node->base.start = node->mm_nodes[0].start; + return ret; + }- return ret;+ node->base.start = node->mm_nodes[0].start; + *res = &node->base; + return 0; }static void ttm_range_man_free(struct ttm_resource_manager *man,
