WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [PATCH 2 of 3] libxl: Add support for passing in the mac

To: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH 2 of 3] libxl: Add support for passing in the machine's E820 for PCI passthrough
From: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Date: Tue, 17 May 2011 17:05:12 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Delivery-date: Tue, 17 May 2011 09:08:17 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20110517160053.GC3657@xxxxxxxxxxxx>
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>
Organization: Citrix Systems, Inc.
References: <patchbomb.1304518670@xxxxxxxxxxxxxxxxxxxxxxx> <ba218fa1a48ed682651f.1304518672@xxxxxxxxxxxxxxxxxxxxxxx> <19922.36862.514514.475625@xxxxxxxxxxxxxxxxxxxxxxxx> <20110517160053.GC3657@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Tue, 2011-05-17 at 17:00 +0100, Konrad Rzeszutek Wilk wrote:
> On Tue, May 17, 2011 at 04:10:54PM +0100, Ian Jackson wrote:
> > Sorry for the delay replying.  But:
> > 
> > Konrad Rzeszutek Wilk writes ("[PATCH 2 of 3] libxl: Add support for 
> > passing in the machine's E820 for PCI passthrough"):
> > > diff -r b6af9b428bb1 -r ba218fa1a48e tools/libxl/xl_cmdimpl.c
> > ...
> > > @@ -994,6 +995,8 @@ skip_vfb:
> > >              if (!libxl_device_pci_parse_bdf(ctx, pcidev, buf))
> > >                  d_config->num_pcidevs++;
> > >          }
> > > +        if (d_config->num_pcidevs && !c_info->hvm)
> > > +          b_info->u.pv.machine_e820 = true;
> > >      }
> > 
> > This doesn't seem to provide a way to force this behaviour on, which
> > is (I assume) necessary if PCI devices are to be hotplugged later ?
> 
> Right. We never resolved that question. One idea was that if you had
> 'pci=[]' it would trigger this. But that just seemed wrong.

The e820_host option which you just posted suits this purpose (so long
as we decide to make it permanent), doesn't it?

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

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