8 Sep
2021
8 Sep
'21
1:28 p.m.
On 07/09/2021 16:04, Pierre-Louis Bossart wrote:
+ graph->graph = audioreach_alloc_graph_pkt(apm, &info->sg_list, graph_id); + if (IS_ERR(graph->graph)) { + kfree(graph); + return ERR_PTR(-ENOMEM); + }
+ spin_lock(&apm->lock); + idr_alloc(&apm->graph_idr, graph, graph_id, + graph_id + 1, GFP_ATOMIC);
does this need to be ATOMIC?
We are inside spinlock.
but this is not used in an interrupt handler or anything that isn't in a process context, is it?
No, it is in process context only.
--srini