[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] x86/HVM: don't mark evtchn upcall vector as pending when vLAPIC is disabled
- To: Roger Pau Monné <roger.pau@xxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Mon, 21 Nov 2022 13:06:38 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=PfWqe8txZ8pTIPnXFfeSBoTDiYmJpnxz5DrlDj77Cnw=; b=WBG/7GBqKkPWICmg21vzuubj89rkqJ3eIP5SA3sV2Y8mH+5FqGyYd6nhWDxEbcx/srYSsdg/k0DfgESkUJT6eofR6kfOKMGum7KIgc3AYChYW8PXUUJhCyRAP5g1VcDLxAcDtMJUFcm3nQVxV04bWuoaVlVhm8fDwz+XHKMcU7Y0ucCzSlw33UiVOyAvckc5dhxk79y6bEpqI8IZ1b4kQch/D8/Q00WXP5aeIDzjRQUklwrUWjDXb4Fo0CNToGVZewYSmZVP4dNW5PfTcEcYTHm1Yw34T585KuO4mt+RvyENq/N345K3s8QQrB/TgnTrgfFs88eDOlUqAGxgyEw3dA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KYlUCWHbOHLKXVcGDhYsLpn2aHWqmBfHlxEMSVxBn24JH8VkBrA479gklJF+jmVP6pXVlDleJvyYfOUOMEJBWVkT2qbM5GE2s02tgaCXggrujQMYK3ZfVg7AJ3fUHBgZlcYUyPnhBNE60cc1Pp42kh0FAhaCAaqB+Z0tFkKuX6IHOXoRCJymSmh7AO7NgYKmuWDpme7wOzdNo8g2iOWE4CXGphnB5wIygVEGH/XYphl/cTtVQtvO+/nF1MKWvd+E7oQK6gJADND/UXISJZ+bSh0ljcjzZB2C3n2HGnn9mMRguAtAWbNEL9TuVMOfuXkF5JrhUag9l5ibwzdYYZZ0Kw==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Paul Durrant <paul@xxxxxxx>
- Delivery-date: Mon, 21 Nov 2022 12:06:48 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 21.11.2022 09:33, Jan Beulich wrote:
> On 18.11.2022 15:26, Roger Pau Monné wrote:
>> Maybe best to add an ASSERT in vlapic_set_irq() to be sure the lapic is
>> enabled, as other callers already check this before trying to inject?
>
> Perhaps, yes (once we've fixed paths where the check is presently
> missing).
Actually - no, such an ASSERT() would then be racy against the vLAPIC
being disabled elsewhere at this very moment. It would at best be valid
when done on the vCPU in question. The SDM also provides for this: "The
reception of any interrupt or transmission of any IPIs that are in
progress when the local APIC is disabled are completed before the local
APIC enters the software-disabled state." We don't follow this to the
letter, but you get the point.
Jan
|