[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 08/11] s390x: allow only 1 CPU with TCG
From: |
David Hildenbrand |
Subject: |
Re: [Qemu-devel] [PATCH v1 08/11] s390x: allow only 1 CPU with TCG |
Date: |
Thu, 31 Aug 2017 15:24:18 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 30.08.2017 21:06, Thomas Huth wrote:
> On 30.08.2017 19:05, David Hildenbrand wrote:
>> Specifying more than 1 CPU (e.g. -smp 5) leads to SIGP errors (the
>> guest tries to bring these CPUs up but fails), because we don't support
>> multiple CPUs on s390x under TCG.
>>
>> Let's bail out if more than 1 are specified, so we don't raise people's
>> hope.
>
> Aurelien recently posted a patch to add that basic SIGP support:
>
> https://patchwork.kernel.org/patch/9717489/
>
> I think it would make more sense to get that included instead.
>
> Thomas
>
Even then, it doesn't work reliably:
- "*very rough* SMP support"
- "this patch is nothing more than a way to determine what needs to
be implemented"
- "It should be rewritten from scratch before reaching in an acceptable
state."
Such broken feature should not be exposed to the user. Once we have
properly fixed that we can enable and announce it "s390x now supports
more than 1 VCPU under TCG". At this point, this is just wrong, even
with this patch included.
So I'd suggest including this now and reverting it once we have actual
support. "so we don't raise people's hope."
--
Thanks,
David
- Re: [Qemu-devel] [PATCH v1 06/11] target/s390x: cleanup cpu number/address handling, (continued)
[Qemu-devel] [PATCH v1 05/11] s390x: rename s390-virtio.h to s390-virtio-hcall.h, David Hildenbrand, 2017/08/30
[Qemu-devel] [PATCH v1 07/11] target/s390x: rename next_cpu_id to next_cpu_addr, David Hildenbrand, 2017/08/30
[Qemu-devel] [PATCH v1 08/11] s390x: allow only 1 CPU with TCG, David Hildenbrand, 2017/08/30
[Qemu-devel] [PATCH v1 09/11] target/s390x: tcg_s390_program_interrupt() will never return, David Hildenbrand, 2017/08/30
[Qemu-devel] [PATCH v1 10/11] target/s390x: use trigger_pgm_exception() in s390_cpu_handle_mmu_fault(), David Hildenbrand, 2017/08/30