[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNE
From: |
David Hildenbrand |
Subject: |
Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD |
Date: |
Mon, 4 Dec 2017 19:37:26 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 04.12.2017 18:58, Cornelia Huck wrote:
> On Mon, 4 Dec 2017 18:56:00 +0100
> David Hildenbrand <address@hidden> wrote:
>
>> On 04.12.2017 18:53, Cornelia Huck wrote:
>>> On Mon, 4 Dec 2017 18:34:36 +0100
>>> David Hildenbrand <address@hidden> wrote:
>>>
>>>> On 04.12.2017 18:22, Cornelia Huck wrote:
>>>>> On Mon, 4 Dec 2017 13:55:05 +0100
>>>>> David Hildenbrand <address@hidden> wrote:
>>>>>
>>>>>> We somehow missed that, new kernels require it.
>>>>>
>>>>> Why _new_ kernels? I think we have unconditionally issued stcrw since
>>>>> back in 2.2?
>>>>
>>>> Okay, the problem is then rather related to my setup. No ccw devices ->
>>>> no stcrw.
>>>>
>>>> How could we ever add ccw devices to a TCG guest then?
>>>
>>> Hotplug never worked before your patches. Coldplug does not create
>>> machine checks and thus does not trigger the guest to do STCRW. (Only
>>> STSCH, in keeping with "all channel I/O acronyms look the same".)
>>>
>>
>> But booting Fedora 26/27 without any involved hotplugs (therefore
>> machine checks) triggers a STCRW.
>>
>> That's how I originally found it. (I started playing with machine checks
>> after I had fedora 26/27 running)
>>
>
> Confused. Do you have a backtrace? (Guest/host)
>
Unfortunately not. Strange, I just tried to reproduce but can't trigger
it. Maybe aside effect from the other BUGs I have been fixing :)
So I'll just rephrase this to "CRW machine check handling requires STCRW."
--
Thanks,
David / dhildenb
- Re: [qemu-s390x] [PATCH v1 for-2.12 1/5] s390x/kvm: factor out build_channel_report_mcic() into cpu.h, (continued)
[qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, David Hildenbrand, 2017/12/04
- Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, Cornelia Huck, 2017/12/04
- Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, David Hildenbrand, 2017/12/04
- Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, Cornelia Huck, 2017/12/04
- Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, David Hildenbrand, 2017/12/04
- Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, Cornelia Huck, 2017/12/04
- Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD,
David Hildenbrand <=
- Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, Cornelia Huck, 2017/12/05
Re: [qemu-s390x] [PATCH v1 for-2.12 5/5] s390x/tcg: wire up STORE CHANNEL REPORT WORD, Thomas Huth, 2017/12/06
Re: [qemu-s390x] [Qemu-devel] [PATCH v1 for-2.12 0/5] s390x/tcg: CCW hotplug support, no-reply, 2017/12/04
Re: [qemu-s390x] [PATCH v1 for-2.12 0/5] s390x/tcg: CCW hotplug support, Cornelia Huck, 2017/12/04
Re: [qemu-s390x] [Qemu-devel] [PATCH v1 for-2.12 0/5] s390x/tcg: CCW hotplug support, no-reply, 2017/12/07