samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

msm: adsprpc: Free the memory allocated for status notification

Browse Source 
Currently, memory allocated for status notification is only
freed by the notif thread. If notif thread exits, notif entries
will not be freed. Free the notif entries while closing
the fastrpc file.

Change-Id: I8e715a4c449a595ce492379bfc50eaf456bbccf6
Signed-off-by: Abhishek Singh <quic_abhishes@quicinc.com>
This commit is contained in:
Abhishek Singh
2024-02-16 11:17:03 +05:30
parent d95b6bb0db
commit 941b3f835f
1 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
dsp/adsprpc.c
View File

@@ -2369,6 +2369,8 @@ static void fastrpc_context_list_dtor(struct fastrpc_file *fl)
struct fastrpc_ctx_lst *clst = &fl->clst; struct fastrpc_ctx_lst *clst = &fl->clst;
struct smq_invoke_ctx *ictx = NULL, *ctxfree; struct smq_invoke_ctx *ictx = NULL, *ctxfree;
struct hlist_node *n; struct hlist_node *n;
unsigned long irq_flags = 0;
struct smq_notif_rsp *inotif = NULL, *n1 = NULL;
do { do {
ctxfree = NULL; ctxfree = NULL;
@@ -2396,6 +2398,14 @@ static void fastrpc_context_list_dtor(struct fastrpc_file *fl)
if (ctxfree) if (ctxfree)
context_free(ctxfree); context_free(ctxfree);
} while (ctxfree); } while (ctxfree);
spin_lock_irqsave(&fl->proc_state_notif.nqlock, irq_flags);
list_for_each_entry_safe(inotif, n1, &clst->notif_queue, notifn) {
list_del_init(&inotif->notifn);
atomic_sub(1, &fl->proc_state_notif.notif_queue_count);
kfree(inotif);
}
spin_unlock_irqrestore(&fl->proc_state_notif.nqlock, irq_flags);
} }
static int fastrpc_file_free(struct fastrpc_file *fl); static int fastrpc_file_free(struct fastrpc_file *fl);