This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH] HVM Debug Console Support

To: Owen Smith <owen.smith@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] HVM Debug Console Support
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Mon, 19 Jul 2010 15:04:13 +0100
Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Mon, 19 Jul 2010 07:04:52 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <DECC4DEE86A8074C9ABE8DFD330CB6E37A5586F215@xxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcsnST26EIRdjxVpRV6wfq6VCmdyVgAAgZwV
Thread-topic: [Xen-devel] [PATCH] HVM Debug Console Support
User-agent: Microsoft-Entourage/
On 19/07/2010 14:51, "Owen Smith" <owen.smith@xxxxxxxxxx> wrote:

> Creates and assigns a shared memory page and event channel for the console
> ring buffer.
> The Windows PV drivers can use this console ring buffer and event channel to
> pass on debug output.
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>

I ack this conceptually at least. I leave it to the tools maintainers to
comment on the bulk of the implementation -- I notice some hard versus soft
indentation issues for example. Ian: you can check the whole lot of this
patch into your tree when you're happy with it, even though it touches a
hypervisor header file.

 -- Keir

Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>