All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Brook <paul@codesourcery.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "QEMU Developers" <qemu-devel@nongnu.org>,
	"Anthony Liguori" <anthony@codemonkey.ws>,
	"Andreas Färber" <afaerber@suse.de>
Subject: Re: [Qemu-devel] ARM QOM conversion / class hierarchy
Date: Tue, 20 Mar 2012 17:14:44 +0000	[thread overview]
Message-ID: <201203201714.44924.paul@codesourcery.com> (raw)
In-Reply-To: <CAFEAcA8f2nKDzbjED6+LbkS2doM3eroRyKTGK1PtKJt8KvHTyQ@mail.gmail.com>

> >> Yes, I think I'd agree there. So should we just have an init function
> >> that provides the implementation-specific cp15 registers based on the
> >> value provided in the QOM property for the main ID register?
> > 
> > Something like that, yes.  I'm not convinced the main ID register is the
> > right property to use, but for actual implementation specific bits
> > (rather than bits where an implementation picks one of a few common
> > options) I guess we don't have any alternative but enumerating the
> > implementations we support.
> 
> Mmm, the disgusting thing the TI925T has where it can programmatically
> change the value of its main ID register does somewhat argue against using
> it for this.

I was thinking more for when we have multiple revisions of a chip that are 
(for these purposes) the same. Currently we only have this for pxa, but in 
principle we probably want it for others.

As I mentioned on IRC, this isn't particularly interesting for Linux, which 
will happily run on pretty much anything.  However there are other systems 
that care[1] whether the core reports itself as e.g. arm1136-r0p1 v.s. 
arm1136-r0p2.

Paul

[1] The reason for this is usually not relevant to qemu. e.g. They've got a 
hardcoded table of known CPUID values, and arbitrarily refuse to run on 
anything else.  Or you're checking that a workaround for a particular silicon 
errata doesn't make anything else explode.

  reply	other threads:[~2012-03-20 17:15 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-20 12:12 [Qemu-devel] ARM QOM conversion / class hierarchy Peter Maydell
2012-03-20 14:08 ` Paul Brook
2012-03-20 14:59   ` Peter Maydell
2012-03-20 16:06     ` Paul Brook
2012-03-20 16:20       ` Peter Maydell
2012-03-20 17:14         ` Paul Brook [this message]
2012-03-20 17:19           ` Peter Maydell
2012-03-20 16:56   ` Anthony Liguori
2012-03-20 17:14     ` Paul Brook
2012-03-20 17:20       ` Andreas Färber
2012-03-20 16:31 ` Michael Roth
2012-03-20 16:32   ` Peter Maydell
2012-03-20 19:04     ` Michael Roth
2012-03-20 17:01   ` Paul Brook

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=201203201714.44924.paul@codesourcery.com \
    --to=paul@codesourcery.com \
    --cc=afaerber@suse.de \
    --cc=anthony@codemonkey.ws \
    --cc=peter.maydell@linaro.org \
    --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.