WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Wait the state Connect of the frontend before co

To: anthony.perard@xxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] Wait the state Connect of the frontend before connect the backend.
From: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Date: Thu, 20 May 2010 14:24:52 -0700
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Thu, 20 May 2010 14:25:50 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1274116684-6349-1-git-send-email-anthony.perard@xxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1274116684-6349-1-git-send-email-anthony.perard@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100430 Fedora/3.0.4-2.fc12 Lightning/1.0b2pre Thunderbird/3.0.4
On 05/17/2010 10:18 AM, anthony.perard@xxxxxxxxxx wrote:
> From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
>
> The frontend of the framebuffer set a value (request-abs-pointer) and go
> to the state Connected.  The backend must read this value only when the
> frontend has the state Connected.
>   

This works well.  It should go into git.

(Anthony: you'll need to append a "Signed-off-by:" line to be strictly
complete.)

Tested-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@xxxxxxxxxx>

    J

> ---
>  hw/xen_backend.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/hw/xen_backend.c b/hw/xen_backend.c
> index 76d07ec..31ed7b0 100644
> --- a/hw/xen_backend.c
> +++ b/hw/xen_backend.c
> @@ -411,8 +411,7 @@ static int xen_be_try_connect(struct XenDevice *xendev)
>  {
>      int rc = 0;
>  
> -    if (xendev->fe_state != XenbusStateInitialised  &&
> -     xendev->fe_state != XenbusStateConnected) {
> +    if (xendev->fe_state != XenbusStateConnected) {
>       if (xendev->ops->flags & DEVOPS_FLAG_IGNORE_STATE) {
>           xen_be_printf(xendev, 2, "frontend not ready, ignoring\n");
>       } else {
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
>
>   


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel