Re: [Xen-devel] [PATCH v3 2/3] libxl: add support for channels

[Dave Scott <Dave.Scott@xxxxxxxxxx>]

On 24 Jun 2014, at 11:41, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:

> Dave Scott writes ("Re: [PATCH v3 2/3] libxl: add support for channels"):
>> On 23 Jun 2014, at 15:52, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:
>>> Why should an additional channel not be handled by xenconsoled and
>>> logged ?
>> 
>> I believe the current situation (unchanged by these patches) is that
>> xenconsoled is only capable of handling console 0, [...]
> 
> Yes.  I guess what I'm saying is that this is IMO a deficiency.
> Obviously there's no need for you to rectify this right now but we
> should leave space to extend the API in case we decide to fix it
> later.

I agree — I’d like to see a more capable xenconsoled in future.

> 
>> My plan was to build these ‘channel’ things on top of the existing
>> additional console support, which happens to rely on qemu. It would
>> be nice to teach xenconsoled about these new channels/consoles but I
>> thought I’d try a simple implementation first. This should all be
>> hidden behind the interface so we can change the implementation
>> later.
> 
> Right.  One of the reasons I'm asking these questions is that because
> your document described all the layers at once, I wasn't sure which
> parts of the text were parts of the libxl-user-visible specification
> and which were implementation details.

OK, I think I’ve got some work to do on the documentation. I’ll try to take a 
step back clearly separate the interface-level stuff from the miscellaneous 
implementation-level details.

Thanks again,
Dave
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel