amdkfd: Fix memory leak of mqds on dqm fini
authorOded Gabbay <oded.gabbay@amd.com>
Tue, 25 Nov 2014 13:16:38 +0000 (15:16 +0200)
committerOded Gabbay <oded.gabbay@amd.com>
Tue, 25 Nov 2014 13:16:38 +0000 (15:16 +0200)
The mqds array members are not freed when dqm is uninitialized.

Reviewed-by: Ben Goz <Ben.Goz@amd.com>
Signed-off-by: Oded Gabbay <oded.gabbay@amd.com>
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c

index 904eb38..924e90c 100644 (file)
@@ -575,11 +575,15 @@ static int initialize_nocpsch(struct device_queue_manager *dqm)
 
 static void uninitialize_nocpsch(struct device_queue_manager *dqm)
 {
+       int i;
+
        BUG_ON(!dqm);
 
        BUG_ON(dqm->queue_count > 0 || dqm->processes_count > 0);
 
        kfree(dqm->allocated_queues);
+       for (i = 0 ; i < KFD_MQD_TYPE_MAX ; i++)
+               kfree(dqm->mqds[i]);
        mutex_destroy(&dqm->lock);
        kfd2kgd->free_mem(dqm->dev->kgd,
                        (struct kgd_mem *) dqm->pipeline_mem);