From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([140.186.70.92]:47076)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <anthony@codemonkey.ws>) id 1Ry0nt-00019P-E5
	for qemu-devel@nongnu.org; Thu, 16 Feb 2012 07:51:22 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <anthony@codemonkey.ws>) id 1Ry0nn-0006G4-HU
	for qemu-devel@nongnu.org; Thu, 16 Feb 2012 07:51:17 -0500
Received: from mail-pz0-f45.google.com ([209.85.210.45]:63092)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <anthony@codemonkey.ws>) id 1Ry0nn-0006G0-9C
	for qemu-devel@nongnu.org; Thu, 16 Feb 2012 07:51:11 -0500
Received: by dadp14 with SMTP id p14so2472367dad.4
	for <qemu-devel@nongnu.org>; Thu, 16 Feb 2012 04:51:10 -0800 (PST)
Message-ID: <4F3CFBBB.5020600@codemonkey.ws>
Date: Thu, 16 Feb 2012 06:51:07 -0600
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
References: <1329347774-23262-1-git-send-email-imammedo@redhat.com>	<1329347774-23262-3-git-send-email-imammedo@redhat.com>
	<4F3CF010.1040607@siemens.com>
In-Reply-To: <4F3CF010.1040607@siemens.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 2/7] Convert pc cpu to qdev
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: Igor Mammedov <imammedo@redhat.com>, "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>, "gleb@redhat.com" <gleb@redhat.com>

On 02/16/2012 06:01 AM, Jan Kiszka wrote:
> On 2012-02-16 00:16, Igor Mammedov wrote:
>> +static ICCBusDeviceInfo cpu_device_info = {
>> +    .qdev.name = "cpu-pc",
>> +    .qdev.size = sizeof(CPUPC),
>> +    .qdev.reset = cpu_device_reset,
>> +    .init = cpu_device_init,
>> +    .qdev.props = (Property[]) {
>> +        DEFINE_PROP_STRING("model", CPUPC, model),
>
> And how do you pass in feature flags? Or the core layout? Basically both
> -cpu and -smp need to be expressible via multiple "-device cpu-x86,xxx"
> (not "pc") commands.

The approach that I'd recommend is:

1) convert CPU_COMMON_STATE to a structure named CPUCommonState, query/replace 
all references to members of CPU_COMMON_STATE appropriately.

2) convert as many users of CPUState to CPUCommonState as possible.

3) make CPUCommonState a base class, move it to the front of the structure, and 
attempt to measure the impact to TCG.

4) if (3) is significant, special case things in QOM

5) make target specific code use target specific CPUState typename

6) eliminate #define CPUState

7) make on-processor devices (like lapic) children of target specific CPUState

8) express target specific flags/features via QOM properties

9) make machine expose target specific CPUState links that can be set by the user.

Regards,

Anthony Liguori

> As a shortcut, "-smp N" would continue to replicate
> a "-device cpu-XX" N times. Otherwise, configuration becomes inconsistent.
>
> Jan
>