Re: [win-pv-devel] [PATCH 3/5] Zero ring stats before ASSERTing ring struct is zeroed

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: win-pv-devel [mailto:win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx] On
> Behalf Of owen.smith@xxxxxxxxxx
> Sent: 18 October 2017 13:43
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Owen Smith <owen.smith@xxxxxxxxxx>
> Subject: [win-pv-devel] [PATCH 3/5] Zero ring stats before ASSERTing ring
> struct is zeroed
> 
> From: Owen Smith <owen.smith@xxxxxxxxxx>
> 
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>

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

> ---
>  src/xenvbd/ring.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/src/xenvbd/ring.c b/src/xenvbd/ring.c
> index b572b07..0b0df72 100644
> --- a/src/xenvbd/ring.c
> +++ b/src/xenvbd/ring.c
> @@ -1754,6 +1754,16 @@ RingDestroy(
>      RtlZeroMemory(&Ring->Lock, sizeof(KSPIN_LOCK));
>      Ring->Frontend = NULL;
> 
> +    Ring->BlkOpRead = 0;
> +    Ring->BlkOpWrite = 0;
> +    Ring->BlkOpIndirectRead = 0;
> +    Ring->BlkOpIndirectWrite = 0;
> +    Ring->BlkOpBarrier = 0;
> +    Ring->BlkOpDiscard = 0;
> +    Ring->BlkOpFlush = 0;
> +    Ring->SegsGranted = 0;
> +    Ring->SegsBounced = 0;
> +
>      ASSERT(IsZeroMemory(Ring, sizeof(XENVBD_RING)));
>      __RingFree(Ring);
>  }
> --
> 2.8.3
> 
> 
> _______________________________________________
> win-pv-devel mailing list
> win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> https://lists.xenproject.org/cgi-bin/mailman/listinfo/win-pv-devel
_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/win-pv-devel