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

Re: [Xen-devel] [PATCH] xend: Update info['platform']['pci']

To: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xend: Update info['platform']['pci']
From: "Zhai, Edwin" <edwin.zhai@xxxxxxxxx>
Date: Mon, 1 Jun 2009 14:01:58 +0800
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "Zhai, Edwin" <edwin.zhai@xxxxxxxxx>, "horms@xxxxxxxxxxxx" <horms@xxxxxxxxxxxx>
Delivery-date: Sun, 31 May 2009 23:04:10 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <78C9DF622C5003kanno.masaki@xxxxxxxxxxxxxx>
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>
References: <78C9DF622C5003kanno.masaki@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.16 (2007-06-09)
Masaki,
How about removing the 'pci' config from the info['platform']? We already have 
the info['devices'] to kept the pci/vif/disk config, so duplicated info in 
['platform']['pci'] would cause synchronization troubles.


There are only several places using info['platform']['pci'], so take place of 
them with _getDeviceInfo_pci('0') should be okay.

Thanks,


On Thu, May 28, 2009 at 03:01:50PM +0800, Masaki Kanno wrote:
> Hi,
> 
> This patch updates info['platform']['pci'] for PCI devices assignment 
> to domains.
> 
> When a domain is started, xend confirms by using xc.test_assign_device 
> whether PCI devices can be assigned to the domain. 
> For the confirmation, info['platform']['pci'] must be an appropriate 
> value.  However, info['platform']['pci'] may be not appropriate. 
> Because info['platform']['pci'] isn't almost always updated even if 
> the PCI device configuration of the domain was changed by using 
> xm pci-attach/detach. 
> This patch updates info['platform']['pci'] to the appropriate value 
> when domains are started.
> 
> Cc: Simon Horman <horms@xxxxxxxxxxxx>
> Signed-off-by: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>
> 
> Best regards,
>  Kan
> 


Content-Description: ATT00001.txt
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


-- 
best rgds,
edwin

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