Drivers: hv: vmbus: Use READ_ONCE() to read variables that are volatile
authorK. Y. Srinivasan <kys@microsoft.com>
Sun, 3 Apr 2016 00:59:47 +0000 (17:59 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 30 Apr 2016 21:00:19 +0000 (14:00 -0700)
Use the READ_ONCE macro to access variabes that can change asynchronously.
This is the recommended mechanism for dealing with "unsafe" compiler
optimizations.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/hv/ring_buffer.c

index 544362c6a9caca236ea27d0c5f9597bc89b384c8..6ea1b552cc8a1dd44d997691a05e7e85115f4927 100644 (file)
@@ -69,7 +69,7 @@ u32 hv_end_read(struct hv_ring_buffer_info *rbi)
 static bool hv_need_to_signal(u32 old_write, struct hv_ring_buffer_info *rbi)
 {
        mb();
-       if (rbi->ring_buffer->interrupt_mask)
+       if (READ_ONCE(rbi->ring_buffer->interrupt_mask))
                return false;
 
        /* check interrupt_mask before read_index */
@@ -78,7 +78,7 @@ static bool hv_need_to_signal(u32 old_write, struct hv_ring_buffer_info *rbi)
         * This is the only case we need to signal when the
         * ring transitions from being empty to non-empty.
         */
-       if (old_write == rbi->ring_buffer->read_index)
+       if (old_write == READ_ONCE(rbi->ring_buffer->read_index))
                return true;
 
        return false;
@@ -117,7 +117,7 @@ static bool hv_need_to_signal_on_read(struct hv_ring_buffer_info *rbi)
         */
        mb();
 
-       pending_sz = rbi->ring_buffer->pending_send_sz;
+       pending_sz = READ_ONCE(rbi->ring_buffer->pending_send_sz);
        /* If the other end is not blocked on write don't bother. */
        if (pending_sz == 0)
                return false;