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

[Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>]

Mon, 1 Jun 2009 14:01:58 +0800, "Zhai, Edwin" wrote:

>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.

Hi Edwin,

As far as I know, info['platform']['pci'] is referred in the following 
places.
 - _constructDomain@xxxxxxxxxxxxxxxxx
 - image_sxpr@xxxxxxxxxxxxx

Are there any others than the above?  If there are others, I will 
confirm them.

Best regards,
 Kan

>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


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