[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v10 2/9] xen: do not free reserved memory into heap
- To: Penny Zheng <Penny.Zheng@xxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Tue, 16 Aug 2022 08:40:50 +0200
- 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=8HZIlDw6oVBNNzAsssQSx18YDkXRqbj57+Fd8gx3mM4=; b=dqDMbh5Y+RFebxXMsHqZHaLWV4f0M+pY4S2bvIINLRMCwbOA3Qn6MyhN4bSjFkrPDBQtE58PkbQfeeqmo8gFwRap628jgHi894VX3BLvmdGNpOV5NUVY28qjwTKmPiWm55mgplu28k6lJUWAa6kprFNga1j0QrqzCHPvb7PF97bPs3WjUARiMalwnNGXT6vvhY8saOwVwVsNIUhVlPwVHgJZNTNwlhuBvIf0v75n04QPfJ+DqIE1rXe/gNfL+vruMUKQ4VldRcOSLZRIAwLr+z65muKbkNQh/ZWg+Owqvec3j+Zd72ZeIvNTkDZluYe3euhigLv3U2O+tvLHadSbbg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Oewf3UwgysH6ZLp+uKfX5FTTAs8hD3JwQhMdNzTei6ExLIMk74WPC7M0u+ECRvqZwYFMydwmUUywvN9f50LgXMjR2eN/658xQZLNJe6ZRN+hBIWetTnO1LvKf4UiyZ2kkQGgKJquhAzcywkUPDgX0yEoujRnpjdOgUs4hEW04tFLXsWP8GZTHTbT1dnwTC/Oi98kkgF0Tc7wbl3Uz3y/009H5DBeOQQSNPdUbIrQwoHyl9UGftf0UbREcEVZaJ/5b+Egau5rKZ1uD8ITCPacVqJgdAwpMaQJpPiOdzRtbI+RIyvQ4ObOK9lWUGGyF1yiZVVFovN/hEyWPhI8/9zhVQ==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: wei.chen@xxxxxxx, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Julien Grall <jgrall@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Tue, 16 Aug 2022 06:40:59 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 16.08.2022 04:36, Penny Zheng wrote:
> +void free_domstatic_page(struct page_info *page)
> +{
> + struct domain *d = page_get_owner(page);
> + bool drop_dom_ref;
> +
> + if ( unlikely(!d) )
> + {
> + ASSERT_UNREACHABLE();
> + printk("The about-to-free static page %"PRI_mfn" must be owned by a
> domain\n",
> + mfn_x(page_to_mfn(page)));
> + return;
> + }
For the message to be useful as a hint if the assertion triggers, it
wants printing ahead of the assertion. I also think it wants to be a
XENLOG_G_* kind of log level, so it would be rate limited by default
in release builds. Just to be on the safe side. (I'm not in favor of
the log message in the first place, but I do know that Julien had
asked for one.)
Jan
|
