Re: [win-pv-devel] [PATCH 1/2] Only break FrontendBackend thread loop when alerted

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: win-pv-devel [mailto:win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx] On
> Behalf Of Ben Chalmers
> Sent: 12 July 2016 09:54
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Ben Chalmers
> Subject: [win-pv-devel] [PATCH 1/2] Only break FrontendBackend thread
> loop when alerted
> 
> Returns a missed '!' character without which the FrontendBackend
> thread exits immediately - rather than when we Alert the thread
> that we wish it to exit
> 
> Signed-off-by : Ben.Chalmers@xxxxxxxxxx

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

> ---
>  src/xenvbd/frontend.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/xenvbd/frontend.c b/src/xenvbd/frontend.c
> index 2dd9fc7..1aec99e 100644
> --- a/src/xenvbd/frontend.c
> +++ b/src/xenvbd/frontend.c
> @@ -1515,7 +1515,7 @@ FrontendBackend(
>      for (;;) {
>          KIRQL       Irql;
> 
> -        if (ThreadWait(Thread))
> +        if (!ThreadWait(Thread))
>              break;
> 
>          KeAcquireSpinLock(&Frontend->StateLock, &Irql);
> --
> 2.7.0.windows.1
> 
> 
> _______________________________________________
> 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