Re: [Xen-devel] [PATCH 0/2] libxl: add PV display device driver interface

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Fri, Mar 24, 2017 at 12:35:22PM +0200, Oleksandr Grytsov wrote:
> To summarize:
> 
> There are two ways to rework the patches:
> 
> 1. Keep interface between xl an libxl as is and put duplicated code into
> libxl_device_type specific functions.
> 
> 2. Change interface to call libxl_device_type specific functions directly
> from xl:
> 
> libxl_device_vtpm *vtpms;
> 
> vtpms = (libxl_device_vtpm*)libxl_device_list_get(LIBXL_DEVTYPE_VTPM, ...);
> 
> ...
> 
> libxl__device_list_free(LIBXL_DEVTYPE_VTPM, vtpms, nr);
> 
> But I need feedback from maintainers.
> 
> Adding maintainers to CC.

I'm leaning towards #1 because we get type-safety that way.

Ian?

Wei.

> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel