Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751510AbcLEOho (ORCPT ); Mon, 5 Dec 2016 09:37:44 -0500 Received: from mx1.redhat.com ([209.132.183.28]:36256 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750951AbcLEOhl (ORCPT ); Mon, 5 Dec 2016 09:37:41 -0500 Subject: Re: [PATCH 4/4] KVM: x86: allow hotplug of VCPU with APIC ID over 0xff To: =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= , linux-kernel@vger.kernel.org, kvm@vger.kernel.org References: <20161202194401.10038-1-rkrcmar@redhat.com> <20161202194401.10038-5-rkrcmar@redhat.com> Cc: Paolo Bonzini , Igor Mammedov From: David Hildenbrand Organization: Red Hat GmbH Message-ID: Date: Mon, 5 Dec 2016 15:37:37 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <20161202194401.10038-5-rkrcmar@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.26]); Mon, 05 Dec 2016 14:37:40 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 925 Lines: 24 Am 02.12.2016 um 20:44 schrieb Radim Krčmář: > LAPIC after reset is in xAPIC mode, which poses a problem for hotplug of > VCPUs with high APIC ID, because reset VCPU is waiting for INIT/SIPI, > but there is no way to uniquely address it using xAPIC. > > From many possible options, we chose the one that also works on real > hardware: accepting interrupts addressed to LAPIC's x2APIC ID even in > xAPIC mode. > > KVM intentionally differs from real hardware, because real hardware > (Knights Landing) does just "x2apic_id & 0xff" to decide whether to > accept the interrupt in xAPIC mode and it can deliver one interrupt to > more than one physical destination, e.g. 0x123 to 0x123 and 0x23. > > Add a capability to let userspace know that we do something now. Should we allow user space to turn it on/off for compatibility handling? Or do we just not care? (or how will this capability be used later on?) -- David