[patch] nouveau: free the memory in nouveau_gart_manager_del()

2011-06-25 Thread Ben Skeggs
On Sat, 2011-06-25 at 08:53 +0300, Dan Carpenter wrote: > In the original code the kfree() doesn't do anything because > "mem->mm_node" has been set to NULL. This was introduced in > d2f9c56a "drm/nouveau: create temp vmas for both src and dst of > bo moves" Already fixed in nouveau git, I'll

[patch] nouveau: free the memory in nouveau_gart_manager_del()

2011-06-25 Thread Dan Carpenter
In the original code the kfree() doesn't do anything because "mem->mm_node" has been set to NULL. This was introduced in d2f9c56a "drm/nouveau: create temp vmas for both src and dst of bo moves" Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/nouveau/nouveau_mem.c b/drivers/gpu/d

Re: [patch] nouveau: free the memory in nouveau_gart_manager_del()

2011-06-25 Thread Ben Skeggs
On Sat, 2011-06-25 at 08:53 +0300, Dan Carpenter wrote: > In the original code the kfree() doesn't do anything because > "mem->mm_node" has been set to NULL. This was introduced in > d2f9c56a "drm/nouveau: create temp vmas for both src and dst of > bo moves" Already fixed in nouveau git, I'll

[patch] nouveau: free the memory in nouveau_gart_manager_del()

2011-06-24 Thread Dan Carpenter
In the original code the kfree() doesn't do anything because "mem->mm_node" has been set to NULL. This was introduced in d2f9c56a "drm/nouveau: create temp vmas for both src and dst of bo moves" Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/nouveau/nouveau_mem.c b/drivers/gpu/d