[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type |
Date: |
Fri, 26 Apr 2013 11:45:10 -0500 |
User-agent: |
Notmuch/0.15.2+77~g661dcf8 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Alexander Graf <address@hidden> writes:
> On 26.04.2013, at 18:11, Christian Borntraeger wrote:
>
>> On 26/04/13 18:05, Alexander Graf wrote:
>>>
>>> On 26.04.2013, at 18:01, Christian Borntraeger wrote:
>>>
>>>> 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.
>>>
>>> My main concern is backwards compatibility. If we introduce a command line
>>> interface now, we have to support it forever. I'd rather only support one
>>> interface, rather than 2 out of which one is only legacy for 1.5
>>> compatibility.
>>
>> The cool thing is, that we dont introduce a command line interface in this
>> patch.
>> We just use the existing bootindex.
>
> Get an ack from Anthony on the bootindex thing and I'm fine with
> pulling that in.
So I don't see a problem with the no-fallback behavior (nor with it
changing down the road), but the bootindex change would cause a
regression on x86.
Regards,
Anthony Liguori
>
>
> Alex
- [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, (continued)
- [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Dominik Dingel, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Alexander Graf, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Christian Borntraeger, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Alexander Graf, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Christian Borntraeger, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Alexander Graf, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Dominik Dingel, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Alexander Graf, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Christian Borntraeger, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Alexander Graf, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type,
Anthony Liguori <=
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Dominik Dingel, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Alexander Graf, 2013/04/26
- Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Dominik Dingel, 2013/04/26
Re: [Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type, Alexander Graf, 2013/04/26
[Qemu-devel] [PATCH 09/10] S390: Pass per-device loadparm values for CCW blk and net devs., Dominik Dingel, 2013/04/26
[Qemu-devel] [PATCH 08/10] s390-ccw.img: Get queue config from host., Dominik Dingel, 2013/04/26