From: "Graf (AWS), Alexander" <graf@amazon.de>
To: "drjones@redhat.com" <drjones@redhat.com>
Cc: "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"kvmarm@lists.cs.columbia.edu" <kvmarm@lists.cs.columbia.edu>,
"pbonzini@redhat.com" <pbonzini@redhat.com>
Subject: Re: [PATCH kvm-unit-tests] arm/arm64: PL031: Fix check_rtc_irq
Date: Thu, 28 Nov 2019 18:30:27 +0000 [thread overview]
Message-ID: <94CC1391-FCAF-4889-A234-911C66D6D334@amazon.de> (raw)
In-Reply-To: <20191128155515.19013-1-drjones@redhat.com>
> Am 28.11.2019 um 17:56 schrieb "drjones@redhat.com" <drjones@redhat.com>:
>
> Since QEMU commit 83ad95957c7e ("pl031: Expose RTCICR as proper WC
> register") the PL031 test gets into an infinite loop. Now we must
> write bit zero of RTCICR to clear the IRQ status. Before, writing
> anything to RTCICR would work. As '1' is a member of 'anything'
> writing it should work for old QEMU as well.
>
> Cc: Alexander Graf <graf@amazon.com>
> Signed-off-by: Andrew Jones <drjones@redhat.com>
Reviewed-by: Alexander Graf <graf@amazon.com>
Sorry for introducing a test case on code that I then modify, without updating the test case as well :)
Alex
> ---
> arm/pl031.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arm/pl031.c b/arm/pl031.c
> index 1f63ef13994f..3b75fd653e96 100644
> --- a/arm/pl031.c
> +++ b/arm/pl031.c
> @@ -143,8 +143,8 @@ static void irq_handler(struct pt_regs *regs)
> report(readl(&pl031->ris) == 1, " RTC RIS == 1");
> report(readl(&pl031->mis) == 1, " RTC MIS == 1");
>
> - /* Writing any value should clear IRQ status */
> - writel(0x80000000ULL, &pl031->icr);
> + /* Writing one to bit zero should clear IRQ status */
> + writel(1, &pl031->icr);
>
> report(readl(&pl031->ris) == 0, " RTC RIS == 0");
> report(readl(&pl031->mis) == 0, " RTC MIS == 0");
> --
> 2.21.0
>
Amazon Development Center Germany GmbH
Krausenstr. 38
10117 Berlin
Geschaeftsfuehrung: Christian Schlaeger, Ralf Herbrich
Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B
Sitz: Berlin
Ust-ID: DE 289 237 879
prev parent reply other threads:[~2019-11-28 18:30 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-28 15:55 [PATCH kvm-unit-tests] arm/arm64: PL031: Fix check_rtc_irq Andrew Jones
2019-11-28 15:57 ` Andrew Jones
2019-11-28 18:30 ` Graf (AWS), Alexander [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=94CC1391-FCAF-4889-A234-911C66D6D334@amazon.de \
--to=graf@amazon.de \
--cc=drjones@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=kvmarm@lists.cs.columbia.edu \
--cc=pbonzini@redhat.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 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).