Re: [PATCH v5 15/21] ASoC: qdsp6: audioreach: add q6apm support

On Wed, Sep 08, 2021 at 12:28:05PM +0100, Srinivas Kandagatla wrote:

On 07/09/2021 16:04, Pierre-Louis Bossart wrote:

quoted

quoted

quoted

quoted

+    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.

So why is it a spinlock then?