From: Christian Borntraeger <borntraeger@de.ibm.com>
To: Alexander Graf <agraf@suse.de>
Cc: Christian Paro <cparo@us.ibm.com>,
qemu-devel@nongnu.org, Dominik Dingel <dingel@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type
Date: Fri, 26 Apr 2013 18:01:44 +0200 [thread overview]
Message-ID: <517AA4E8.2000103@de.ibm.com> (raw)
In-Reply-To: <3D4AF3D5-CFA4-4352-8203-2739B17185A5@suse.de>
On 26/04/13 17:48, Alexander Graf wrote:
>> So I suggest to go with this patch and implement later on what we
>> agree upon?
>
> I don't see how having "first device we find" is any better than a
> rushed interface we need to agree on right before 1.5 hard freeze.
Exactly, find first device isnt better ;-)
See, the current code chooses the first subchannel that matches. This
boils down to "a random disk" as soon as you have more than one.
With this patch the user can at least specify the devno of the disk.It
even works out of the box with libvirt.
Let's just release 1.5 with the very simple one and then go for something
awesome in the next version.
Exactly - and having a list is more in the awesome area. Beiing able to
specify the first disk and pass that in a register to the bios is of
course not a full-features interface, but it works and can be changed.
Just to double check: do you agree that we can change the interface between
ipl device and bios at any time?
Christian
next prev parent reply other threads:[~2013-04-26 16:01 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-26 12:12 [Qemu-devel] [PATCH 00/10] S390: Enhance s390 BIOS to enable bootdevice selection Dominik Dingel
2013-04-26 12:12 ` [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices Dominik Dingel
2013-04-26 15:19 ` Alexander Graf
2013-04-26 16:36 ` Anthony Liguori
2013-04-26 18:01 ` Dominik Dingel
2013-04-26 18:55 ` Anthony Liguori
2013-04-26 19:13 ` Gleb Natapov
2013-04-26 21:34 ` Dominik Dingel
2013-04-27 5:44 ` Gleb Natapov
2013-04-26 19:38 ` Christian Borntraeger
2013-04-26 12:12 ` [Qemu-devel] [PATCH 02/10] Common: Add quick access to first boot device Dominik Dingel
2013-04-26 16:37 ` Anthony Liguori
2013-04-26 12:12 ` [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type Dominik Dingel
2013-04-26 15:22 ` Alexander Graf
2013-04-26 15:36 ` Dominik Dingel
2013-04-26 15:42 ` Christian Borntraeger
2013-04-26 15:48 ` Alexander Graf
2013-04-26 16:01 ` Christian Borntraeger [this message]
2013-04-26 16:05 ` Alexander Graf
2013-04-26 16:10 ` Dominik Dingel
2013-04-26 16:13 ` Alexander Graf
2013-04-26 16:11 ` Christian Borntraeger
2013-04-26 16:13 ` Alexander Graf
2013-04-26 16:45 ` Anthony Liguori
2013-04-26 16:04 ` Dominik Dingel
2013-04-26 16:07 ` Alexander Graf
2013-04-26 16:50 ` Alexander Graf
2013-04-26 12:12 ` [Qemu-devel] [PATCH 04/10] S390: check if BIOS is available and create links Dominik Dingel
2013-04-26 15:23 ` Alexander Graf
2013-04-26 15:48 ` Dominik Dingel
2013-04-26 15:57 ` Alexander Graf
2013-04-26 16:20 ` Dominik Dingel
2013-04-26 16:22 ` Alexander Graf
2013-04-26 16:38 ` Anthony Liguori
2013-04-26 18:03 ` Dominik Dingel
2013-04-26 18:04 ` Alexander Graf
2013-04-26 12:12 ` [Qemu-devel] [PATCH 05/10] s390-ccw.img: Detect devices with stsch Dominik Dingel
2013-04-26 12:12 ` [Qemu-devel] [PATCH 06/10] s390-ccw.img: Enhance drain_irqs() Dominik Dingel
2013-04-26 12:12 ` [Qemu-devel] [PATCH 07/10] s390-ccw.img: Rudimentary error checking Dominik Dingel
2013-04-26 12:12 ` [Qemu-devel] [PATCH 08/10] s390-ccw.img: Get queue config from host Dominik Dingel
2013-04-26 12:12 ` [Qemu-devel] [PATCH 09/10] S390: Pass per-device loadparm values for CCW blk and net devs Dominik Dingel
2013-04-26 16:52 ` Alexander Graf
2013-04-26 18:08 ` Dominik Dingel
2013-04-26 18:14 ` Alexander Graf
2013-04-26 19:54 ` Christian Borntraeger
2013-04-26 12:12 ` [Qemu-devel] [PATCH 10/10] S390: Enabling device and program selection Dominik Dingel
2013-04-26 15:29 ` Alexander Graf
2013-04-26 16:56 ` Alexander Graf
2013-04-26 17:55 ` Dominik Dingel
2013-04-26 17:56 ` Alexander Graf
2013-04-26 16:19 ` [Qemu-devel] [PATCH 00/10] S390: Enhance s390 BIOS to enable bootdevice selection Alexander Graf
2013-04-26 16:22 ` Dominik Dingel
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=517AA4E8.2000103@de.ibm.com \
--to=borntraeger@de.ibm.com \
--cc=agraf@suse.de \
--cc=cparo@us.ibm.com \
--cc=dingel@linux.vnet.ibm.com \
--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).