]>
Commit | Line | Data |
---|---|---|
05ebb0af GKH |
1 | From 82fcee526ba8ca2c5d378bdf51b21b7eb058fe3a Mon Sep 17 00:00:00 2001 |
2 | From: Deepak Rawat <drawat@vmware.com> | |
3 | Date: Mon, 26 Jun 2017 14:39:08 +0200 | |
4 | Subject: drm/vmwgfx: Free hash table allocated by cmdbuf managed res mgr | |
5 | ||
6 | From: Deepak Rawat <drawat@vmware.com> | |
7 | ||
8 | commit 82fcee526ba8ca2c5d378bdf51b21b7eb058fe3a upstream. | |
9 | ||
10 | The hash table created during vmw_cmdbuf_res_man_create was | |
11 | never freed. This causes memory leak in context creation. | |
12 | Added the corresponding drm_ht_remove in vmw_cmdbuf_res_man_destroy. | |
13 | ||
14 | Tested for memory leak by running piglit overnight and kernel | |
15 | memory is not inflated which earlier was. | |
16 | ||
17 | Signed-off-by: Deepak Rawat <drawat@vmware.com> | |
18 | Reviewed-by: Sinclair Yeh <syeh@vmware.com> | |
19 | Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com> | |
20 | Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
21 | ||
22 | --- | |
23 | drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf_res.c | 1 + | |
24 | 1 file changed, 1 insertion(+) | |
25 | ||
26 | --- a/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf_res.c | |
27 | +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf_res.c | |
28 | @@ -317,6 +317,7 @@ void vmw_cmdbuf_res_man_destroy(struct v | |
29 | list_for_each_entry_safe(entry, next, &man->list, head) | |
30 | vmw_cmdbuf_res_free(man, entry); | |
31 | ||
32 | + drm_ht_remove(&man->resources); | |
33 | kfree(man); | |
34 | } | |
35 |