From: Pavel Fedin <p.fedin@samsung.com>
To: 'Peter Maydell' <peter.maydell@linaro.org>
Cc: 'Shlomo Pongratz' <shlomo.pongratz@huawei.com>,
'Peter Crosthwaite' <crosthwaitepeter@gmail.com>,
'QEMU Developers' <qemu-devel@nongnu.org>,
'Shlomo Pongratz' <shlomopongratz@gmail.com>
Subject: Re: [Qemu-devel] [PATCH v2] target-arm: Extract some external ARM CPU API
Date: Tue, 27 Oct 2015 17:18:06 +0300 [thread overview]
Message-ID: <002f01d110c2$4cd51d70$e67f5850$@samsung.com> (raw)
In-Reply-To: <CAFEAcA8cnXpFFLras9-uvsenmgFreoG09Ygy_T5qbT==0JyhNQ@mail.gmail.com>
Hello!
> But Peter C has a better grasp of the current status of
> multiarch than I do, so if he prefers using obj-y then
> I'm happy to take his recommendation.
I am neutral, and i will accept any solution. OTOH, he agreed with my proposal too, and even promised to pick it up into his patchsets, but suggested to change include pathname to include/hw/arm/cpu.h. And i have the last concern about it because hw/arm/ is a place where board-specific includes are placed.
Additionally, we already have three more specific ARM CPU files in include/hw/cpu/, so they just look nice together.
And you know... Theoretically... As far as i can understand, multi-arch is a way to enable emulation of heterogeneous systems, which combine different CPUs. Am i right about it? In this case, what if in future we have some heterogeneous HW, where e.g. ARM and x86 are interoperating using system registers? In this case, x86-targeted qemu code would probably have to talk to ARM-targeted one. And having this API separated from the inner CPU guts would only help.
Compared to this architectural improvement, obj-y looks like very simple way to ignore, but not to solve the problem. :)
So, Peter C, what is your final decision?
Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia
next prev parent reply other threads:[~2015-10-27 14:18 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-26 7:46 [Qemu-devel] [PATCH v2] target-arm: Extract some external ARM CPU API Pavel Fedin
2015-10-26 16:39 ` Peter Crosthwaite
2015-10-27 12:02 ` Pavel Fedin
2015-10-27 13:41 ` Peter Maydell
2015-10-27 14:18 ` Pavel Fedin [this message]
2015-10-27 20:17 ` Peter Crosthwaite
2015-10-28 6:46 ` Pavel Fedin
2015-10-28 11:12 ` Pavel Fedin
2015-10-28 11:34 ` Shlomo Pongratz
2015-10-27 12:03 ` Pavel Fedin
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='002f01d110c2$4cd51d70$e67f5850$@samsung.com' \
--to=p.fedin@samsung.com \
--cc=crosthwaitepeter@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=shlomo.pongratz@huawei.com \
--cc=shlomopongratz@gmail.com \
/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.