From: Glenn Washburn <development@efficientek.com>
To: grub-devel@gnu.org, Daniel Kiper <daniel.kiper@oracle.com>
Cc: Glenn Washburn <development@efficientek.com>
Subject: [PATCH v2] grub-shell: Boot PowerPC using PMU instead of CUDA for power management
Date: Thu, 7 Oct 2021 15:52:51 -0500 [thread overview]
Message-ID: <20211007205251.159703-1-development@efficientek.com> (raw)
A recent refactoring of CUDA command code has exposed a bug in OpenBIOS[1]
which was causing system powerdown and system reset to fail, thus causing
the Qemu instance to hang. This in turn caused the grub-shell command to
timeout causing it to return an error code when the test actually completed
successfully.
Since it could be a while before the patch fixing this issue in OpenBIOS
filters down to the average distro, switch to PMU to allow powerdowns and
reboots to work as expected.
[1] https://gitlab.com/qemu-project/qemu/-/issues/624
Signed-off-by: Glenn Washburn <development@efficientek.com>
---
This is a correction of the commit message from patch #8 of the patch series
titled "More test fixes/improvements". The changes remain the same.
Glenn
Range-diff against v1:
1: e13454839 ! 1: 754e7e77b grub-shell: Boot PowerPC using PMU instead of CUDA for power management
@@ Metadata
## Commit message ##
grub-shell: Boot PowerPC using PMU instead of CUDA for power management
- At some point it looks like the defualt machine for qemu-system-ppc started
- using CUDA as a backend for power management. This causes the machine to
- throw an exception and not actually power down the VM[1]. Switching to PMU
- allows power downs and reboots to work as expceted.
+ A recent refactoring of CUDA command code has exposed a bug in OpenBIOS[1]
+ which was causing system powerdown and system reset to fail, thus causing
+ the Qemu instance to hang. This in turn caused the grub-shell command to
+ timeout causing it to return an error code when the test actually completed
+ successfully.
+
+ Since it could be a while before the patch fixing this issue in OpenBIOS
+ filters down to the average distro, switch to PMU to allow powerdowns and
+ reboots to work as expected.
[1] https://gitlab.com/qemu-project/qemu/-/issues/624
tests/util/grub-shell.in | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/util/grub-shell.in b/tests/util/grub-shell.in
index 93e9f5148..5354d8678 100644
--- a/tests/util/grub-shell.in
+++ b/tests/util/grub-shell.in
@@ -84,6 +84,7 @@ case "${grub_modinfo_target_cpu}-${grub_modinfo_platform}" in
serial_null="-serial null"
netbootext=elf
trim=1
+ qemuopts="-M mac99,via=pmu $qemuopts"
;;
sparc64-ieee1275)
--
2.27.0
next reply other threads:[~2021-10-07 20:53 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-07 20:52 Glenn Washburn [this message]
2021-10-11 10:48 ` [PATCH v2] grub-shell: Boot PowerPC using PMU instead of CUDA for power management Daniel Axtens
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=20211007205251.159703-1-development@efficientek.com \
--to=development@efficientek.com \
--cc=daniel.kiper@oracle.com \
--cc=grub-devel@gnu.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.