> -----Original Message----- > From: Jason Wang [mailto:jasow...@redhat.com] > Sent: Thursday, June 20, 2013 12:59 AM > To: KY Srinivasan; de...@linuxdriverproject.org; linux-kernel@vger.kernel.org; > gre...@linuxfoundation.org > Cc: Haiyang Zhang; Jason Wang > Subject: [PATCH] drivers: hv: check interrupt mask before read_index > > This patches add a read barriers to force the driver to check the interrupt > mask > before read_index. Otherwise we may lost a kick to host. > > Cc: K. Y. Srinivasan <k...@microsoft.com> > Cc: Haiyang Zhang <haiya...@microsoft.com> > Signed-off-by: Jason Wang <jasow...@redhat.com> Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> > --- > drivers/hv/ring_buffer.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/hv/ring_buffer.c b/drivers/hv/ring_buffer.c > index 791f45d..26c93cf 100644 > --- a/drivers/hv/ring_buffer.c > +++ b/drivers/hv/ring_buffer.c > @@ -75,6 +75,8 @@ static bool hv_need_to_signal(u32 old_write, struct > hv_ring_buffer_info *rbi) > if (rbi->ring_buffer->interrupt_mask) > return false; > > + /* check interrupt_mask before read_index */ > + rmb(); > /* > * This is the only case we need to signal when the > * ring transitions from being empty to non-empty. > -- > 1.7.1 > >
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/