diff options
| author | Vamsidhar reddy Gaddam <gvamsi@google.com> | 2024-02-19 17:24:56 +0000 |
|---|---|---|
| committer | Vamsidhar reddy Gaddam <gvamsi@google.com> | 2024-02-19 17:52:17 +0000 |
| commit | dcc4755a0e3ed4c9884da9c105a27fe85a586ffe (patch) | |
| tree | 0b59e7859374932750f8609009d79e13846c2963 | |
| parent | 6b217b43f0424690a7febf9b2b60aa34ea973eda (diff) | |
| download | gpu-dcc4755a0e3ed4c9884da9c105a27fe85a586ffe.tar.gz | |
Free kprcs in the kobject release callback
Freeing kprcs using kfree() triggered a bunch of warning in debug_api
kernel build flavor.
Bug: 325892033
Test: Make sure that debug_api build does not trigger a warning.
Change-Id: Iddea1d12a9190fa328b488ee01065c5e2abc9557
Signed-off-by: Vamsidhar reddy Gaddam <gvamsi@google.com>
| -rw-r--r-- | mali_kbase/context/mali_kbase_context.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/mali_kbase/context/mali_kbase_context.c b/mali_kbase/context/mali_kbase_context.c index 42f09f3..9b8cc5d 100644 --- a/mali_kbase/context/mali_kbase_context.c +++ b/mali_kbase/context/mali_kbase_context.c @@ -46,7 +46,8 @@ static void kbase_kprcs_release(struct kobject *kobj) { - // Nothing to release + struct kbase_process *kprcs = to_kprcs(kobj); + kfree(kprcs); } static ssize_t total_gpu_mem_show(struct kobject *kobj, struct kobj_attribute *attr, char *buf) @@ -315,7 +316,8 @@ static void kbase_remove_kctx_from_process(struct kbase_context *kctx) WARN_ON(!RB_EMPTY_ROOT(&kprcs->dma_buf_root)); kobject_del(&kprcs->kobj); kobject_put(&kprcs->kobj); - kfree(kprcs); + // kfree(kprcs); -> done in kobject release callback, left here + // for easy tracking of differences from upstream. } } |