Re: [PATCH V3] xl: create VFB for PV guest when VNC is specified

[Ian Campbell <Ian.Campbell@citrix.com>]

On Tue, 2013-12-17 at 16:15 +0000, Ian Jackson wrote:
> Wei Liu writes ("Re: [Xen-devel] [PATCH V3] xl: create VFB for PV guest when VNC is specified"):
> > On Tue, Dec 17, 2013 at 03:02:07PM +0000, Ian Jackson wrote:
> > > Finally: IMO the macro should probably initialise the new element
> > > itself.  So it will be more than just "ARRAY_EXTEND" because it will
> > > have some knowledge of the kind of array it is, and might need to be
> > > renamed.  Ian C, what do you think ?
> > 
> > Huh? Elements are initialized by different funcitons. You mean I need to
> > add extra argument to the macro to pass in the initilization function?
> 
> Either that, or pass in just the core of the token eg "vkb" and use
> token pasting to automatically create the right function name.
> 
> But please wait to see what Ian C says as he may disagree with me.

I'm happy either way. A more helpful macro is good but it is perhaps
getting a bit complex, so I wouldn't blame Wei for wanting to keep it
simpler by doing the init afterwards.

There's also an argument that this should be functionality provided in
helpers generated by the IDL, but that certainly isn't 4.4 material...

Ian.