Bug 13234 - [2.7 M3 RC1] qemumips & qemumips64: failed to shutdown
Summary: [2.7 M3 RC1] qemumips & qemumips64: failed to shutdown
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: virtual machines (show other bugs)
Version: 2.7
Hardware: x86 Multiple
: High normal
Target Milestone: 2.7 M4
Assignee: Kevin Hao
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2019-03-25 06:59 UTC by yeoh ee peng
Modified: 2019-04-11 14:55 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Don't know


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description yeoh ee peng 2019-03-25 06:59:35 UTC
Both qemumips and qemumips64 both failed to shutdown for 2.7 M3 rc1, where these qemus were able to shutdown prior to 2.7 M3 rc1. 
Currently, both qemumips and qemumips64 hang/stuck while shutting down, need to manually click on the "x" button to close the qemu.

Steps to reproduce:
1. Git clone poky
2. Git checkout poky: 753e2a0ede4449917c75353b57f13bbafe70fac8
3. Configure MACHINE to be qemumips or qemumips64
4. Bitbake core-image-minimal
5. Runqemu to boot qemumips or qemumips64
6. Perform shutdown in command line (eg. shutdown -h now)

Expected:
Qemu instance to close automatically
Comment 1 Richard Purdie 2019-04-02 19:46:49 UTC
As a data point this works with the 4.18 kernel but fails with the 4.19 kernel for qemumips so something changed between those two kernel versions.
Comment 2 Richard Purdie 2019-04-02 20:02:58 UTC
qemumips and qemumips64 need new defconfig entries:

CONFIG_POWER_RESET=y
CONFIG_POWER_RESET_PIIX4_POWEROFF=y

See https://github.com/torvalds/linux/commit/dd129c6374e90e665ec022019bcc2f392f25cf2c
Comment 3 Richard Purdie 2019-04-02 20:03:13 UTC
(I can confirm that 5.0 works with this added)
Comment 4 Bruce Ashfield 2019-04-03 02:53:42 UTC
Makes sense to me. If Kevin hasn't gotten to this by next week, I can also take care of it.
Comment 5 Kevin Hao 2019-04-03 11:39:47 UTC
Thanks Richard, patch sent out.