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
