From: Anthony Liguori <anthony@codemonkey.ws>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Support for out-of-code cpu definitions
Date: Wed, 11 Feb 2009 09:21:45 -0600 [thread overview]
Message-ID: <4992ED09.9030400@codemonkey.ws> (raw)
In-Reply-To: <f43fc5580902110711p7157f2e2x407c350875bd6860@mail.gmail.com>
Blue Swirl wrote:
> On 2/11/09, Amit Shah <amit.shah@redhat.com> wrote:
>
>> This patch series adds support for out-of-code cpu definitions. This helps
>> us move the definitions out of target-i386/helper.c. The result is we can
>> store the definitions in plain-text; add, modify the definitions without
>> having to recompile and most importantly, some management servers can
>> define their own CPU types based on the network topology to create
>> migration-safe cpu types.
>>
>> The first patch in the series was sent earlier, just converts the usage of
>> bios_dir to get_datafile_dir(), as the path that's referred to stores more
>> than just the bios (keymaps, and now, the cpudefs).
>>
>> The cpudef file currently is very similar to the code that was in helper.c.
>>
>> Comments welcome on the format.
>>
>
> Instead of this change, IMHO we should wait for either the
> configuration file format to be fixed, or use FDT for all HW
> definitions. This format is not generic enough, definitely not
> suitable for Sparc CPU features. But if the config or FDT does not
> appear, we could use this as a starting point.
>
I agree. If you switch cpudef to FDT, that could provide an
intermediate step between what we have today and a full machine description.
Regards,
Anthony Liguori
prev parent reply other threads:[~2009-02-11 15:22 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-11 13:15 [Qemu-devel] Support for out-of-code cpu definitions Amit Shah
2009-02-11 13:15 ` [Qemu-devel] [PATCH 1/4] Introduce get_datafile_dir() to replace bios_dir Amit Shah
2009-02-11 13:15 ` [Qemu-devel] [PATCH 2/4] Add independent cpu definitions for the coreduo and core2duo cpu types Amit Shah
2009-02-11 13:15 ` [Qemu-devel] [PATCH 3/4] Update Makefile to copy the cpudef files to install location Amit Shah
2009-02-11 13:15 ` [Qemu-devel] [PATCH 4/4] Support for cpudef files placed in the share/ directory Amit Shah
2009-02-11 15:11 ` [Qemu-devel] Support for out-of-code cpu definitions Blue Swirl
2009-02-11 15:21 ` Anthony Liguori [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4992ED09.9030400@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.