All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.