From: Alex Longwall <1859021@bugs.launchpad.net>
To: qemu-devel@nongnu.org
Subject: [Bug 1859021] [NEW] qemu-system-aarch64 (tcg): cval + voff overflow not handled, causes qemu to hang
Date: Thu, 09 Jan 2020 13:24:58 -0000 [thread overview]
Message-ID: <157857629827.5165.2496570379985305724.malonedeb@gac.canonical.com> (raw)
Public bug reported:
The Armv8 architecture reference manual states that for any timer set
(e.g. CNTP* and CNTV*), the condition for such timer to generate an
interrupt (if enabled & unmasked) is:
CVAL <= CNT(P/V)CT
Although this is arguably sloppy coding, I have seen code that is
therefore assuming it can set CVAL to a very high value (e.g.
UINT64_MAX) and leave the interrupt enabled in CTL, and never get the
interrupt.
On latest master commit as the time of writing, there is an integer
overflow in target/arm/helper.c gt_recalc_timer affecting the virtual
timer when the interrupt is enabled in CTL:
/* Next transition is when we hit cval */
nexttick = gt->cval + offset;
When this overflow happens, I notice that qemu is no longer responsive and that I have to SIGKILL the process:
- qemu takes nearly all the cpu time of the cores it is running on (e.g. 50% cpu usage if running on half the cores) and is completely unresponsive
- no guest interrupt (reported via -d int) is generated
Here the minimal code example to reproduce the issue:
mov x0, #1
msr cntvoff_el2, x0
mov x0, #-1
msr cntv_cval_el0, x0
mov x0, #1
msr cntv_ctl_el0, x0 // interrupt generation enabled, not masked; qemu will start to hang here
Options used:
-nographic -machine virt,virtualization=on,gic-version=2,accel=tcg -cpu cortex-a57
-smp 4 -m 1024 -kernel whatever.elf -d unimp,guest_errors,int -semihosting-config enable,target=native
-serial mon:stdio
Version used: 4.2
** Affects: qemu
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1859021
Title:
qemu-system-aarch64 (tcg): cval + voff overflow not handled, causes
qemu to hang
Status in QEMU:
New
Bug description:
The Armv8 architecture reference manual states that for any timer set
(e.g. CNTP* and CNTV*), the condition for such timer to generate an
interrupt (if enabled & unmasked) is:
CVAL <= CNT(P/V)CT
Although this is arguably sloppy coding, I have seen code that is
therefore assuming it can set CVAL to a very high value (e.g.
UINT64_MAX) and leave the interrupt enabled in CTL, and never get the
interrupt.
On latest master commit as the time of writing, there is an integer
overflow in target/arm/helper.c gt_recalc_timer affecting the virtual
timer when the interrupt is enabled in CTL:
/* Next transition is when we hit cval */
nexttick = gt->cval + offset;
When this overflow happens, I notice that qemu is no longer responsive and that I have to SIGKILL the process:
- qemu takes nearly all the cpu time of the cores it is running on (e.g. 50% cpu usage if running on half the cores) and is completely unresponsive
- no guest interrupt (reported via -d int) is generated
Here the minimal code example to reproduce the issue:
mov x0, #1
msr cntvoff_el2, x0
mov x0, #-1
msr cntv_cval_el0, x0
mov x0, #1
msr cntv_ctl_el0, x0 // interrupt generation enabled, not masked; qemu will start to hang here
Options used:
-nographic -machine virt,virtualization=on,gic-version=2,accel=tcg -cpu cortex-a57
-smp 4 -m 1024 -kernel whatever.elf -d unimp,guest_errors,int -semihosting-config enable,target=native
-serial mon:stdio
Version used: 4.2
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1859021/+subscriptions
next reply other threads:[~2020-01-09 13:36 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-09 13:24 Alex Longwall [this message]
2020-01-09 14:44 ` [Bug 1859021] Re: qemu-system-aarch64 (tcg): cval + voff overflow not handled, causes qemu to hang Alex Bennée
2020-01-09 16:25 ` [RFC PATCH] tests/tcg: add a vtimer test for aarch64 Alex Bennée
2020-01-09 16:25 ` [Bug 1859021] Re: qemu-system-aarch64 (tcg): cval + voff overflow not handled, causes qemu to hang Alex Bennée
2020-07-28 14:44 ` Alex Bennée
2021-05-01 5:30 ` Thomas Huth
2020-01-10 16:16 [PATCH v1 0/2] fix for bug 1859021 Alex Bennée
2020-01-10 16:16 ` [PATCH v1 1/2] target/arm: detect 64 bit overflow caused by high cval + voff Alex Bennée
2020-01-10 16:16 ` [Bug 1859021] " Alex Bennée
2020-01-16 18:45 ` Peter Maydell
2020-01-16 18:45 ` [Bug 1859021] " Peter Maydell
2020-01-17 11:50 ` Peter Maydell
2020-01-17 11:50 ` [Bug 1859021] " Peter Maydell
2020-01-10 16:16 ` [PATCH v1 2/2] tests/tcg: add a vtimer test for aarch64 Alex Bennée
2020-01-17 14:07 ` Peter Maydell
2020-02-06 17:00 ` Alex Bennée
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=157857629827.5165.2496570379985305724.malonedeb@gac.canonical.com \
--to=1859021@bugs.launchpad.net \
--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).