From mboxrd@z Thu Jan  1 00:00:00 1970
From: Fantu <fantonifabio@tiscali.it>
Subject: Re: Libxl regression on pv domU start
Date: Tue, 6 Mar 2012 06:31:44 -0800 (PST)
Message-ID: <1331044304980-5540987.post@n5.nabble.com>
References: <1331042415153-5540887.post@n5.nabble.com>
	<1331043530.22559.87.camel@cthulhu.hellion.org.uk>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1331043530.22559.87.camel@cthulhu.hellion.org.uk>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: xen-devel@lists.xensource.com
List-Id: xen-devel@lists.xenproject.org


Ian Campbell-10 wrote
> 
> On Tue, 2012-03-06 at 09:00 -0500, Fantu wrote:
>> Dom0 is Wheezy 64 bit with xen-unstable from source.
>> With changeset 24858:a88ba599add1 work, after update to
>> 24967:33659563f589
>> not.
>> 
>> xl create /etc/xen/lucid.cfg
>> Parsing config file /etc/xen/lucid.cfg
>> xl: libxl.c:214: libxl_defbool_val: Assertion
>> `!libxl_defbool_is_default(db)' failed.
>> Annullato
> 
> The following fixes this for me.
> 
> 8<----------------------------
> 
> # HG changeset patch
> # User Ian Campbell &lt;ian.campbell@&gt;
> # Date 1331043402 0
> # Node ID 7cedb78a4a1d7b410a787da6401044485a82e1b3
> # Parent  e8efe0a654517d9695c391b55b07612ebe79b409
> libxl: always set a default value for VFB.opengl
> 
> Fixes an assertion reported by Fantu.
> 
> Signed-off-by: Ian Campbell &lt;ian.campbell@&gt;
> 
> diff -r e8efe0a65451 -r 7cedb78a4a1d tools/libxl/libxl.c
> --- a/tools/libxl/libxl.c	Fri Mar 02 21:06:14 2012 +0000
> +++ b/tools/libxl/libxl.c	Tue Mar 06 14:16:42 2012 +0000
> @@ -2255,9 +2255,7 @@ int libxl__device_vfb_setdefault(libxl__
>      }
>  
>      libxl_defbool_setdefault(&vfb->sdl.enable, false);
> -    if (libxl_defbool_val(vfb->sdl.enable)) {
> -        libxl_defbool_setdefault(&vfb->sdl.opengl, false);
> -    }
> +    libxl_defbool_setdefault(&vfb->sdl.opengl, false);
>  
>      return 0;
>  }
> 
> 

Patch applied, problem solved.

--
View this message in context: http://xen.1045712.n5.nabble.com/Libxl-regression-on-pv-domU-start-tp5540887p5540987.html
Sent from the Xen - Dev mailing list archive at Nabble.com.