[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all a
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices |
Date: |
Fri, 26 Apr 2013 21:38:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130221 Thunderbird/17.0.3 |
On 26/04/13 20:01, Dominik Dingel wrote:
> On Fri, 26 Apr 2013 11:36:11 -0500
> Anthony Liguori <address@hidden> wrote:
>
>> Dominik Dingel <address@hidden> writes:
>>
>>> Currently only devices with a positive boot index will be pushed in the
>>> fw_boot_order queue, so if no boot index at all will be specified,
>>> the queue ends up empty.
>>>
>>> Instead we push exactly as docs/bootindex.txt says the devices with
>>> the lowest possible boot priority at the tail of the queue,
>>> because we give them the highest available boot index.
>>>
>>> Signed-off-by: Dominik Dingel <address@hidden>
>>
>> Wouldn't this break the ability to say: "don't every try to boot from
>> this device?"
>>
>> As an example, some people want to force PXE boot to not be tried on
>> certain networks.
>>
>> Regards,
>>
>> Anthony Liguori
>
> That is correct, hmm. The thing is, if we don't submit a bootindex, we will
> assign -1 in virtio-blk and virtio-net. This would forbid that the device
> would be booted from.
> Where docs/bootindex.txt says: if a device got no bootindex, it gets the
> lowest possibly priority...
Hmm, reading all this, I changed my mind :-)
I think we should just leave out patch 1 for now. Anthony brought up a
valid point, we want to be able to specify for a device "never boot from it".
>From an s390 point of view, it would even make sense to say
"If the user does not provide a boot index, then the system wont boot.
We leave the system in stopped state." Predictability is more important
than clever guessing in this environment.
Christian
- Re: [Qemu-devel] [PATCH 10/10] S390: Enabling device and program selection, (continued)
[Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Dominik Dingel, 2013/04/26
- Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Alexander Graf, 2013/04/26
- Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Anthony Liguori, 2013/04/26
- Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Dominik Dingel, 2013/04/26
- Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Anthony Liguori, 2013/04/26
- Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Gleb Natapov, 2013/04/26
- Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Dominik Dingel, 2013/04/26
- Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Gleb Natapov, 2013/04/27
Re: [Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices,
Christian Borntraeger <=
[Qemu-devel] [PATCH 02/10] Common: Add quick access to first boot device, Dominik Dingel, 2013/04/26
Re: [Qemu-devel] [PATCH 00/10] S390: Enhance s390 BIOS to enable bootdevice selection, Alexander Graf, 2013/04/26