[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] x86: Perform mem_sharing teardown before paging teardown
- To: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Tue, 21 Feb 2023 14:54:25 +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=EVTJvYTKDnQXBucUhW3cMkKX3h61OEW67kbmqIYv/Js=; b=HSJKO2arEXReO2nkt6dSloHS/IzAZ7Fia+RAJCOmTUT30AEoXhr0ZVkhfO0bPWyGvzMyNpbWm0pN9xsvEANWaW8EVBOfS2UQvUUZbKpZ2h9awmesMWK5GKK0H6iQTk21O/xJ7E3nmrTFtLcruVJkNjhGszIXWHJWWuXAVGcChPySpl2keYvcwrTDABAk2aWoMqhrYhNZ1V2Lx1ovHKHPUdtv8e3yC3YsXGlgaO105So2fHkbZSCGO3xRwrnFO9i52PhL1W6q+rYUM3H2sCkm6dyW+EoRQ3xzxeBCfnumb7CjI9ytrnEm6oxQQJVlzNPFamFu3Eg8SIsmZLe/0lQFiQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KQSCL0fT9wusFuybQlxVp7I0Rl0K8lKj0EyfabK9I8rYt238E83tcK0fX3IdSodDC6ei0faKWUGtg3X3VZl+j+kIMv0skxuulwd5higzHCFu7QvlVqG5Klhb5Cxt7hQfaNrkAmULOKIEk3PwuIHbJTpEpTYNskqD0vmt/QNN1XpHuLyTlJvBJCcvRqC/LUOan5M1dnwzfH37C/M6IGa5HjFQc0J68pyTQuWNyftCltN+0NF2W1FTg/0Amdh7gFkfL42F/6IgW4riHR4rV10FoCZ+pFKrJtwuUG7UK6hnr/5kpS4zcLqwcWVJWwGqaqqqYwInVu7kM7twvdKa3sLfvg==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Tue, 21 Feb 2023 13:54:38 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 15.02.2023 18:07, Tamas K Lengyel wrote:
> An assert failure has been observed in p2m_teardown when performing vm
> forking and then destroying the forked VM (p2m-basic.c:173). The assert
> checks whether the domain's shared pages counter is 0. According to the
> patch that originally added the assert (7bedbbb5c31) the p2m_teardown
> should only happen after mem_sharing already relinquished all shared pages.
>
> In this patch we flip the order in which relinquish ops are called to avoid
> tripping the assert.
>
> Signed-off-by: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
> Fixes: e7aa55c0aab3 ("x86/p2m: free the paging memory pool preemptively")
Especially considering the backporting requirement I'm disappointed to
find that you haven't extended the description to explain why this
heavier code churn is to be preferred over an adjustment to the offending
assertion. As said in reply to v1 - I'm willing to accept arguments
towards this being better, but I need to know those arguments.
Jan
|