Re: [Xen-devel] [PATCH RESENTv2] libxl: add all pci devices to xenstore at once (during VM create)

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Marek Marczykowski writes ("[Xen-devel] [PATCH RESENTv2] libxl: add all pci 
devices to xenstore at once (during VM create)"):
> diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c
> --- a/tools/libxl/libxl_create.c
> +++ b/tools/libxl/libxl_create.c
> @@ -521,6 +521,11 @@ static int do_domain_create(libxl__gc *g
>  
>      for (i = 0; i < d_config->num_pcidevs; i++)
>          libxl__device_pci_add(gc, domid, &d_config->pcidevs[i], 1);
> +    ret = libxl__create_pci_backend(gc, domid, d_config->pcidevs, 
> d_config->num_pcidevs);
> +    if (ret < 0) {
> +        LIBXL__LOG(ctx, LIBXL__LOG_ERROR, "libxl_create_pci_backend failed: 
> %d", ret);
> +        goto error_out;
> +    }

There is an indenting error here.

Also your lines are still >75 characters.  Please wrap them
appropriately, including splitting strings if necessary.

Thanks,
Ian.

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