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


[Xen-ia64-devel] Re: [Xen-devel] [RFC][PATCH] "Controller" pcibackend an

To: Alex Williamson <alex.williamson@xxxxxx>, xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-ia64-devel] Re: [Xen-devel] [RFC][PATCH] "Controller" pcibackend and frontend extensions
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Sun, 10 Jun 2007 20:13:26 +0100
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Sun, 10 Jun 2007 12:08:57 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1181499864.6221.437.camel@bling>
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acerk2vFqkBdoheGEdyF3QAWy6hiGQ==
Thread-topic: [Xen-devel] [RFC][PATCH] "Controller" pcibackend and frontend extensions
User-agent: Microsoft-Entourage/
On 10/6/07 19:24, "Alex Williamson" <alex.williamson@xxxxxx> wrote:

>    I would like to see this backend become the default backend for ia64,
> but it probably needs testing on other systems before we can make the
> switch.  I would appreciate testing and/or review feedback.  To make use
> of extended I/O port spaces, you'll need the patch I sent to
> xen-ia64-devel last week to register the I/O port spaces with Xen.

It would be nice to have just one backend type, with relevant bits of its
code made arch-specific. I'm sure x86 doesn't need vpci, passthru *and*

This is probably fine for now; it doesn't preclude merging or dropping other
backends down the line. Only thing I spotted is that you make an enormous
inline function in pcifront.h. That function will have to be moved elsewhere
into a .c file if it's getting that big.

 -- Keir

Xen-ia64-devel mailing list

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