Re: [PATCH v2 1/2] dmaengine: avalon: Intel Avalon-MM DMA Interface for PCIe

From: Alexander Gordeev
Date: Tue Oct 15 2019 - 08:27:27 EST


On Tue, Oct 15, 2019 at 02:41:08PM +0300, Dan Carpenter wrote:
> > > > > > > > + spin_lock(lock);
> >
> > [*]
>
> [ snip ]
>
> > I struggle to realize how the spinlock I use (see [*] above) does not
> > protect the reader.
>
> Argh.... I'm really sorry. I completely didn't see the spinlock. :P

Np ;) May be in the next version it will be more visible.
I done it the way you asked ( even though I do not like it :D ):

spin_lock(&chan->vchan.lock);

> I am embarrassed. Wow...
>
> regards,
> dan carpenter
>