qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [qemu-s390x] [PULL-for-s390x 06/14] s390-ccw: parse and


From: Collin L. Walling
Subject: Re: [Qemu-devel] [qemu-s390x] [PULL-for-s390x 06/14] s390-ccw: parse and set boot menu options
Date: Tue, 27 Feb 2018 10:04:17 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0

On 02/27/2018 04:22 AM, Thomas Huth wrote:
On 27.02.2018 10:12, Cornelia Huck wrote:
On Mon, 26 Feb 2018 14:44:45 -0500
"Collin L. Walling" <address@hidden> wrote:

On 02/26/2018 02:29 PM, Collin L. Walling wrote:
On 02/26/2018 01:48 PM, Cornelia Huck wrote:
On Mon, 26 Feb 2018 11:42:29 +0100
Thomas Huth <address@hidden> wrote:
[...]
+    switch (ipl->iplb.pbt) {
+    case S390_IPL_TYPE_CCW:
+        break;
+    default:
+        error_report("boot menu is not supported for this device
type.");
If I specify both a bootindex for a device and a -kernel parameter, I
get this error message. Looks a tad odd, but not sure how to avoid it.
Hmm... perhaps an additional check if no IPLB, then skip trying to set
any boot menu data?
[...]
How shall we proceed? I'd be willing to pull this now and then apply
fixups on top, or we can have a respin. Thomas?
Since this is about a change in hw/s390x/ and not about
pc-bios/s390-ccw/ (i.e. no need to rebuild the firmware binaries for
this), I think I'd rather prefer a fix-up patch on top instead of a re-spin.

  Thomas

I'll get right on it and post to the qemu lists.

--
- Collin L Walling




reply via email to

[Prev in Thread] Current Thread [Next in Thread]