From: Peter Maydell <peter.maydell@linaro.org>
To: Axel Heider <axelheider@gmx.de>
Cc: qemu-devel@nongnu.org, qemu-arm@nongnu.org
Subject: Re: [PATCH qemu.git 0/1] hw/arm/virt: add 2x sp804 timer
Date: Fri, 9 Dec 2022 10:58:51 +0000 [thread overview]
Message-ID: <CAFEAcA9h36KgFh9HHzF-YGmL3LLT3L9S74qhfv85d_8GdOOUUw@mail.gmail.com> (raw)
In-Reply-To: <2cbdc0b8-ff8e-f6e3-379a-82d25ad43514@gmx.de>
On Thu, 8 Dec 2022 at 17:25, Axel Heider <axelheider@gmx.de> wrote:
> >> For the seL4 specific case, this is currently not possible in
> >> the standard configuration. It's only exposed for a special
> >> debug and benchmarking configuration.
> >>
> > It's not clear to me what you mean here -- the generic
> > timer in the CPU exists in all configurations, so there
> > should be no obstacle to seL4 using it.
>
> Access is not exposed to userland in the standard configuration
> and the standard kernel API has no no timeouts besides zero and
> infinite. It's a design thing in the end. Nothing that could not
> be hacked around or be changed in the design in the long run. But
> my goal is not to hack around, but have a "proper" machine
> simulation instead. Which basically falls down to having a generic
> machine in mainline that has a few more customization options.
So, my take on this is that I'm open to adding things to
the virt board where we don't provide a feature that's
useful to guest code. The second UART falls in this category:
it lets you do things that you otherwise could not do (like
have one UART for firmware and one for a kernel). On the
other hand, this case with the sp804 sounds more like QEMU
is already providing functional timer facilities and the
problem is on the guest software side. To me the "non-hacky"
solution sounds like it is "sel4 should provide a better timer
related API to userland". I don't really want to work around
guest OS deficiencies in QEMU.
thanks
-- PMM
prev parent reply other threads:[~2022-12-09 10:59 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-30 18:56 [PATCH qemu.git 0/1] hw/arm/virt: add 2x sp804 timer ~axelheider
2022-11-30 17:14 ` [PATCH qemu.git 1/1] " ~axelheider
2022-12-01 10:28 ` [PATCH qemu.git 0/1] " Peter Maydell
2022-12-08 16:59 ` Axel Heider
2022-12-08 17:03 ` Peter Maydell
2022-12-08 17:25 ` Axel Heider
2022-12-09 10:58 ` Peter Maydell [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=CAFEAcA9h36KgFh9HHzF-YGmL3LLT3L9S74qhfv85d_8GdOOUUw@mail.gmail.com \
--to=peter.maydell@linaro.org \
--cc=axelheider@gmx.de \
--cc=qemu-arm@nongnu.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 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).