[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1
From: |
Halil Pasic |
Subject: |
Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1 |
Date: |
Mon, 28 Aug 2017 16:19:21 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 |
On 08/28/2017 04:15 PM, Farhan Ali wrote:
>
>
> On 08/28/2017 10:05 AM, Cornelia Huck wrote:
>>>>> It's the alignment of the CCW which causes the problem.
>>>>>
>>>>> The exact error message when starting the guest was:
>>>>>
>>>>> ! No virtio device found !
>>>>>
>>>>> Since it worked for SCSI and CDL, and failed for LDL disks on that
>>>>> particular system, we are not really sure what caused the failure.
>>>>> Debugging it further showed the CCW for LDL disks were not aligned at
>>>>> double word boundary.
>> This is really, really odd, as the low-level ccw code is the same for
>> any disk type...
>>
> Exactly!
>
>>>>> Trying the test on a different system with LDL disks worked fine, with
>>>>> the aligned(8) fix.
>>>> Do you happen to have an old s390-ccw.img laying around in the test
>>>> folder? QEMU might pick up
>>>> this one (e.g. when calling it without libvirt from the command line).
>>>>
>>> I explicitly mention the bios to use with '-bios' option and pick up the
>>> latest bios. Without the aligned fix I see the error and with the fix it
>>> works fine.
>> Wait, so the fix fixes it? Or am I confused now?
>>
>
> It fixes in my system and one other system we tried on. But fails on a system
> where this issue was first noticed.
This is very confusing. So you have tried -bios on the system
where the issue was first noticed and the issue still persists
despite of the fixed bios is specified?
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, (continued)
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Cornelia Huck, 2017/08/25
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/25
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Cornelia Huck, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Cornelia Huck, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Christian Borntraeger, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Cornelia Huck, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1,
Halil Pasic <=
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Cornelia Huck, 2017/08/29
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/29
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Cornelia Huck, 2017/08/29
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Eric Farman, 2017/08/29
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Cornelia Huck, 2017/08/30
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Philippe Mathieu-Daudé, 2017/08/28
- Re: [Qemu-devel] [PATCH] s390-ccw: Fix alignment for CCW1, Farhan Ali, 2017/08/28