[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [win-pv-devel] [PATCH 1/3] Update rsp_event during Poll


  • To: Owen Smith <owen.smith@xxxxxxxxxx>, "win-pv-devel@xxxxxxxxxxxxxxxxxxxx" <win-pv-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Paul Durrant <Paul.Durrant@xxxxxxxxxx>
  • Date: Thu, 5 Sep 2019 08:06:26 +0000
  • Accept-language: en-GB, en-US
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=Paul.Durrant@xxxxxxxxxx; spf=Pass smtp.mailfrom=Paul.Durrant@xxxxxxxxxx; spf=None smtp.helo=postmaster@xxxxxxxxxxxxxxx
  • Cc: Owen Smith <owen.smith@xxxxxxxxxx>
  • Delivery-date: Thu, 05 Sep 2019 08:06:34 +0000
  • Ironport-sdr: 7jSTsDr/z2vHbWt3Zs9qK4G8R4DL3RZUbDp/2Y2UdomKCr6LvO6qESw0hjClFCPyoedsR18uG5 LdfhVdNjvgzdiY+S/WhDuYHgEJ6OX7gpeFNcrAIYMqPidtt2WQsNjFR0EaIdQacsa77UIvUPHQ 7fkybx3KxTOjaiu+kiKILvntZzLE6o4Bb/9CbHTZiUovQsRcdXXIkA49wd7NFAdGk9o2qhK7bk kU75uVg8UUQE+OXDyoWaoQ4T+kt1CEa/3YvN1+o7cQi8yXQIVOSN6hTcq0gaCivbiD3YYis+9m rVA=
  • List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>
  • Thread-index: AQHVYmYH0EW8y6tJlEiz7/2VtpOne6ccvHzw
  • Thread-topic: [win-pv-devel] [PATCH 1/3] Update rsp_event during Poll

> -----Original Message-----
> From: win-pv-devel <win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx> On Behalf Of 
> Owen Smith
> Sent: 03 September 2019 15:43
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Owen Smith <owen.smith@xxxxxxxxxx>
> Subject: [win-pv-devel] [PATCH 1/3] Update rsp_event during Poll
> 
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>

For posterity I think we need a justification for this change in the commit 
message. I think I understand things well enough to add one myself so...

Acked-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

> ---
>  src/xenvbd/ring.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/xenvbd/ring.c b/src/xenvbd/ring.c
> index 0dfba7f..74426f1 100644
> --- a/src/xenvbd/ring.c
> +++ b/src/xenvbd/ring.c
> @@ -1294,6 +1294,7 @@ BlkifRingPoll(
>          KeMemoryBarrier();
> 
>          BlkifRing->Front.rsp_cons = rsp_cons;
> +        BlkifRing->Shared->rsp_event = rsp_cons + 1;
>      }
> 
>  done:
> @@ -1327,8 +1328,6 @@ __BlkifRingPushRequests(
>  #pragma warning (push)
>  #pragma warning (disable:4244)
> 
> -    BlkifRing->Shared->rsp_event = BlkifRing->Front.req_prod_pvt;
> -
>      // Make the requests visible to the backend
>      RING_PUSH_REQUESTS_AND_CHECK_NOTIFY(&BlkifRing->Front, Notify);
> 
> --
> 2.16.2.windows.1
> 
> 
> _______________________________________________
> win-pv-devel mailing list
> win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> https://lists.xenproject.org/mailman/listinfo/win-pv-devel
_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/win-pv-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.