Re: [PATCH v10 1/4] qrwlock: A queue read/write lock implementation

From: Waiman Long
Date: Thu Jan 23 2014 - 15:46:23 EST


On 01/23/2014 12:52 PM, Linus Torvalds wrote:
On Thu, Jan 23, 2014 at 9:47 AM, Waiman Long<waiman.long@xxxxxx> wrote:
Thank for the info. I am less familiar with that kind of issues on other
architecture. I will add a smp_mb__after_atomic_dec()& send out a new
patch.
SInce it's the unlock path,. you need to use the "mb__*before*"
versions, since presumably you want to protect what is inside the lock
from leaking out, not some random thing after the lock from leaking
in..

Linus

My bad. I am a bit sloppy not using the right function in my email. I will certainly put the right mb functions depending on whether it is a lock or unlock function.

-Longman
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/