On 6/7/21 12:40 PM, Christian König wrote:
That somehow got missing.
Signed-off-by: Christian König christian.koenig@amd.com Fixes: cb1c81467af3 ("drm/ttm: flip the switch for driver allocated resources v2")
Reported-by: Thomas Hellström thomas.hellstrom@linux.intel.com
Reviewed-by: Thomas Hellström thomas.hellstrom@linux.intel.com
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,