On Wed, Sep 08, 2021 at 12:28:05PM +0100, Srinivas Kandagatla wrote: > On 07/09/2021 16:04, Pierre-Louis Bossart wrote: > > > > > + 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?
Attachment:
signature.asc
Description: PGP signature
