[dm-devel] [PATCH] Remove the spinlock protecting the pages allocation
Mikulas Patocka
mpatocka at redhat.com
Thu May 26 12:50:55 UTC 2011
On Mon, 23 May 2011, Joe Thornber wrote:
> On Thu, 2011-05-19 at 12:58 -0400, Mikulas Patocka wrote:
> > The spinlock is only taken on initialization or from single-threaded
> > workqueue.
> > Therefore, the spinlock is useless.
>
> The spinlock has memory barrier semantics as well. How are you
> guaranteeing the changes from the initialisation thread are visible in
> the workqueue thread?
There is another spinlock, kc->job_lock, that is taken each time someone
pushes or pops some work for the worker thread.
Once we take kc->job_lock, we guarantee that any written memory is
visible to the other CPUs.
Mikulas
> - Joe
>
> --
> dm-devel mailing list
> dm-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel
>
More information about the dm-devel
mailing list