From: "Andreas Färber" <afaerber@suse.de>
To: peter@chubb.wattle.id.au
Cc: Peter Maydell <peter.maydell@linaro.org>,
Peter Chubb <peter.chubb@nicta.com.au>,
qemu-devel@nongnu.org, Jan Kiszka <jan.kiszka@siemens.com>
Subject: Re: [Qemu-devel] [Patch] ARM: Add an L2 cache controller to KZM
Date: Tue, 06 Aug 2013 10:27:11 +0200 [thread overview]
Message-ID: <5200B35F.2090808@suse.de> (raw)
In-Reply-To: <84zjsvk7rq.wl%peter@chubb.wattle.id.au>
Am 06.08.2013 02:00, schrieb peter@chubb.wattle.id.au:
>>>>>> "Andreas" == Andreas Färber <afaerber@suse.de> writes:
>
> Andreas> Am 05.08.2013 11:18, schrieb Peter Maydell:
>>> On 5 August 2013 02:21, Peter Chubb <peter.chubb@nicta.com.au>
>>> wrote:
>>>> Reads to unassigned memory now return non-zero (since patch
>>>> 9b8c69243585). This breaks guests runnong on i.MX31 that use the
>>>> cache controller --- they poll forever waiting for the L2CC cache
>>>> invalidate regsiter to be zero.
>>>
> Andreas> Peter Ch., if you know the exact differences, why don't you
> Andreas> just derive an imx-l2cc type (or so) derived from ARM's type,
> Andreas> overriding the values mentioned above? Sounds trivial to me.
>
> Because I don't know how -- can you point me at some documentation?
There's no official how-to, but QOM is documented in include/qom/object.h.
May I simply point you to an example:
http://git.qemu.org/?p=qemu.git;a=commit;h=692a76d1c4a32573bf3cc19110c7fa6cc8c93f60
pl061 has Luminary and ARM IDs, with ARM in the base type and Luminary
overriding values. Another idea is to use an abstract base type and
several derived types if the differences are bigger.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
next prev parent reply other threads:[~2013-08-06 8:27 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-05 1:21 [Qemu-devel] [Patch] ARM: Add an L2 cache controller to KZM Peter Chubb
2013-08-05 9:18 ` Peter Maydell
2013-08-05 9:24 ` Andreas Färber
2013-08-06 0:00 ` peter
2013-08-06 8:27 ` Andreas Färber [this message]
2013-08-07 2:21 ` Peter Chubb
2013-08-06 0:53 ` Peter Chubb
2013-08-05 23:31 ` Peter Chubb
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=5200B35F.2090808@suse.de \
--to=afaerber@suse.de \
--cc=jan.kiszka@siemens.com \
--cc=peter.chubb@nicta.com.au \
--cc=peter.maydell@linaro.org \
--cc=peter@chubb.wattle.id.au \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).