[Xen-devel] Re: [PATCH] libxenlight: implement libxl_set_memory_target

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Wed, 9 Dec 2009, Andres Lagar-Cavilla wrote:
> Stefano Stabellini wrote:
> > On Tue, 8 Dec 2009, Andres Lagar-Cavilla wrote:
> >   
> >> Hi,
> >> couple of comments:
> >> - PV domains without videoram won't be able to use this
> >>     
> >
> > PV domains just have videoram = 0.
> >   
> But you abort libxl_set_memory_target if the videoram node is not found. 
> Which won't be for PVs with no videoram...
> +    videoram_s = libxl_xs_read(ctx, XBT_NULL, libxl_sprintf(ctx, 
> "%s/memory/videoram", dompath));
> +    if (!videoram_s)
> +        return -1;
> 

The videoram node will be found, and the value will be 0.


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