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: [Qemu-devel] [PATCH 02/15] xen: Add xen_machine_fv

To: Anthony Liguori <anthony@xxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [Qemu-devel] [PATCH 02/15] xen: Add xen_machine_fv
From: Kevin Wolf <kwolf@xxxxxxxxxx>
Date: Mon, 16 Aug 2010 16:51:28 +0200
Cc: Anthony Perard <anthony.perard@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "qemu-devel@xxxxxxxxxx" <qemu-devel@xxxxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Delivery-date: Wed, 18 Aug 2010 08:00:23 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4C694D64.8040402@xxxxxxxxxxxxx>
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: <alpine.DEB.2.00.1008121244200.2545@kaball-desktop> <1281622202-3453-2-git-send-email-stefano.stabellini@xxxxxxxxxxxxx> <4C694054.8010600@xxxxxxxxxx> <alpine.DEB.2.00.1008161454150.2545@kaball-desktop> <4C6947A6.70908@xxxxxxxxxx> <4C694D64.8040402@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.11) Gecko/20100720 Fedora/3.0.6-1.fc12 Thunderbird/3.0.6
Am 16.08.2010 16:38, schrieb Anthony Liguori:
> On 08/16/2010 09:13 AM, Kevin Wolf wrote:
>> Am 16.08.2010 16:04, schrieb Stefano Stabellini:
>>    
>>> On Mon, 16 Aug 2010, Kevin Wolf wrote:
>>>      
>>>> Am 12.08.2010 16:09, schrieb stefano.stabellini@xxxxxxxxxxxxx:
>>>>        
>>>>> From: Anthony PERARD<anthony.perard@xxxxxxxxxx>
>>>>>
>>>>> Add the Xen FV (Fully Virtualized) machine to Qemu;
>>>>> this is groundwork to add Xen device model support in Qemu.
>>>>>
>>>>> Signed-off-by: Anthony PERARD<anthony.perard@xxxxxxxxxx>
>>>>> Signed-off-by: Stefano Stabellini<stefano.stabellini@xxxxxxxxxxxxx>
>>>>>          
>>>> Why does this need its own machine type? Shouldn't an HVM machine really
>>>> look like a PC? And indeed most of this code looks like a (slightly
>>>> outdated) copy of pc_piix.c with !pci_enabled code paths removed.
>>>>        
>>>
>>> The main reason is that we need some xen specific initializations, as you 
>>> can
>>> see from xen_init_fv.
>>>      
>> Right, there are some more Xen specific things added later. However, the
>> main part of it is duplicated from pc_piix.c. I'm sure that with some
>> refactoring we could call these functions instead of copying and
>> modifying them. The problem with the latter is that they will inevitably
>> diverge even for changes that make sense for both.
>>
>> I'm not even sure that the machine is the right place to do these Xen
>> specific initializations (expect for the Xen platform PCI device). As
>> far as I understand, the QEMUMachine is considered guest state whereas
>> most of these initializations concern host state.
>>    
> 
> To be honest, I think we'll need KVM, Xen, and QEMU specific machines.
> 
> The right default set of hardware for all three is different.

Right, I agree. This is why I put the exception for the platform device.
There are probably some more devices for which the same applies.

But these exceptions all about guest state. If qdev was finished, this
would be a matter of having a different configuration file, right?

This series, as I understand it, is adding much more to the Xen FV
machine. Things that are not about which devices a VM contains, but
about some implementation details of the host.

Kevin

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

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