All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Graf <graf@amazon.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: kvm-devel <kvm@vger.kernel.org>,
	Marc Zyngier <marc.zyngier@arm.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	<kvmarm@lists.cs.columbia.edu>
Subject: Re: [PATCH kvm-unit-tests] arm: Add PL031 test
Date: Thu, 11 Jul 2019 11:11:25 +0200	[thread overview]
Message-ID: <a29ea772-0565-98cb-61d8-3042b2df39b1@amazon.com> (raw)
In-Reply-To: <CAFEAcA81mQ780H5EY8uV6AvbXzeZA60eCHoE_n9yzeZgw+ru4w@mail.gmail.com>



On 11.07.19 10:51, Peter Maydell wrote:
> On Wed, 10 Jul 2019 at 14:35, Alexander Graf <graf@amazon.com> wrote:
>>
>> This patch adds a unit test for the PL031 RTC that is used in the virt machine.
>> It just pokes basic functionality. I've mostly written it to familiarize myself
>> with the device, but I suppose having the test around does not hurt, as it also
>> exercises the GIC SPI interrupt path.
> 
> 
> Have you tested this against a real hardware pl031? I appreciate
> that the scaffolding to let you do that is probably pretty
> painful, but it would be interesting to test, because I'm
> not really all that confident in the accuracy of QEMU's
> pl031 model. (Notably there are some places where it absolutely
> does not work like the real h/w; in some ways it's a bit
> like "a pl031 that some imaginary firmware has initialized
> and enabled"...)

Do you have any pointers to devices I might own that have one?

Alex

WARNING: multiple messages have this Message-ID (diff)
From: Alexander Graf <graf@amazon.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Marc Zyngier <marc.zyngier@arm.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	kvmarm@lists.cs.columbia.edu, kvm-devel <kvm@vger.kernel.org>
Subject: Re: [PATCH kvm-unit-tests] arm: Add PL031 test
Date: Thu, 11 Jul 2019 11:11:25 +0200	[thread overview]
Message-ID: <a29ea772-0565-98cb-61d8-3042b2df39b1@amazon.com> (raw)
In-Reply-To: <CAFEAcA81mQ780H5EY8uV6AvbXzeZA60eCHoE_n9yzeZgw+ru4w@mail.gmail.com>



On 11.07.19 10:51, Peter Maydell wrote:
> On Wed, 10 Jul 2019 at 14:35, Alexander Graf <graf@amazon.com> wrote:
>>
>> This patch adds a unit test for the PL031 RTC that is used in the virt machine.
>> It just pokes basic functionality. I've mostly written it to familiarize myself
>> with the device, but I suppose having the test around does not hurt, as it also
>> exercises the GIC SPI interrupt path.
> 
> 
> Have you tested this against a real hardware pl031? I appreciate
> that the scaffolding to let you do that is probably pretty
> painful, but it would be interesting to test, because I'm
> not really all that confident in the accuracy of QEMU's
> pl031 model. (Notably there are some places where it absolutely
> does not work like the real h/w; in some ways it's a bit
> like "a pl031 that some imaginary firmware has initialized
> and enabled"...)

Do you have any pointers to devices I might own that have one?

Alex
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

  reply	other threads:[~2019-07-11  9:11 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-10 13:27 [PATCH kvm-unit-tests] arm: Add PL031 test Alexander Graf
2019-07-10 13:27 ` Alexander Graf
2019-07-10 14:25 ` Marc Zyngier
2019-07-10 14:25   ` Marc Zyngier
2019-07-12  8:29   ` Alexander Graf
2019-07-12  8:29     ` Alexander Graf
2019-07-12  8:51     ` Marc Zyngier
2019-07-12  8:51       ` Marc Zyngier
2019-07-10 14:37 ` Alexandru Elisei
2019-07-10 14:37   ` Alexandru Elisei
2019-07-10 17:02 ` Andre Przywara
2019-07-10 17:02   ` Andre Przywara
2019-07-10 17:06   ` Paolo Bonzini
2019-07-10 17:06     ` Paolo Bonzini
2019-07-11  5:49     ` Alexander Graf
2019-07-11  5:49       ` Alexander Graf
2019-07-11  7:52       ` Paolo Bonzini
2019-07-11  7:52         ` Paolo Bonzini
2019-07-11  9:42         ` Andre Przywara
2019-07-11  9:42           ` Andre Przywara
2019-07-11  9:52           ` Marc Zyngier
2019-07-11  9:52             ` Marc Zyngier
2019-07-11  9:59           ` Alexander Graf
2019-07-11  9:59             ` Alexander Graf
2019-07-11  8:51 ` Peter Maydell
2019-07-11  8:51   ` Peter Maydell
2019-07-11  9:11   ` Alexander Graf [this message]
2019-07-11  9:11     ` Alexander Graf
2019-07-11  9:13     ` Peter Maydell
2019-07-11  9:13       ` Peter Maydell

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=a29ea772-0565-98cb-61d8-3042b2df39b1@amazon.com \
    --to=graf@amazon.com \
    --cc=kvm@vger.kernel.org \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=marc.zyngier@arm.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.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.