[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] xen/dt: Don't advertise Xen static-heap region to hwdom


  • To: Kuan-Wei Chiu <visitorckw@xxxxxxxxx>
  • From: "Orzel, Michal" <michal.orzel@xxxxxxx>
  • Date: Fri, 30 Oct 2026 09:49:13 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=gmail.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=vXGmtj+iz10g+AM3qEIZdYwY3Unl0j7ECxtlYWcVhHs=; b=jHWvVbrXqO46yaEsEZ1T/DQNDrQeDB8aN2xDaJEUGi3bADc5zFuREzLKCRJh4Y96HhL+0NO2m4jzbZh2kpm8H5Bu/5K+lL2zrVzJqA39P0lYMoDcATV98HInzE6u+V9aM2f2AjItnTHC+o7PXiGBMc8OYaNhd4vN/NpamyvpMjFySZ9UzD2RYnO04nBkrNgFvpFfpDc2yVqBtl2w74TnZCL9TnxBlkaw08i0Em09peVnI+lHmh7PW+3iN7OSArVZK+reupTn2aKk6KeBnL8q++sVEJXIXHSXeRHxyWpTUy9amg9W2bS1zVhgTwBS7YIgtsMpTx+3fvjDFh74VmeGgA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=hv4wF5aWkyvjYE6YwSZLM9vLzKPPxbR8kXL8wtQMdVwf7NJieGDzSSSJLFNaDxYvkqv70V+2towlzoDbc5C0gNnYkOFW/UzOJ+mEH4vBFshMqCGZGP0qbznkEyIQUp12qv4wFMCs++A4eMIqDBtdXajjXc7mNfiRjyR0v2b9foWbHtzfbmk/4nH4gy/Pj0fwIQ2PofXWGKctPdmu9MBL0ETXHtZIL1tFl1lDP5+6rt9ZprpI5vZtCE+C8EQP5urcCbwJSiXaEg/B/g99EqLifKcoKAsap31E/CJsVJ8IR96DvQjIz2LsYta8XdkzaYoRpLYq7hlLeYLCpFzfFy6DNw==
  • Authentication-results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=amd.com header.i="@amd.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck"
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>
  • Delivery-date: Fri, 30 Oct 2026 08:49:46 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>


On 30-Oct-26 04:42, Kuan-Wei Chiu wrote:
> On Thu, Oct 29, 2026 at 11:43:05AM +0100, Michal Orzel wrote:
>> bootinfo.reserved_mem is used to store different types of memory that
>> Xen considers as reserved: domU static mem, normal /reserved-memory,
>> FDT memreserve and Xen static heap. make_memory_node(), invoked during
>> FDT generation for a hwdom for reserved memory, only skips the domU
>> static memory and FDT memreserve. This results in advertising Xen static
>> heap (if used) to a hwdom as /memory. This memory is not mapped in P2M
>> and guest attempt to access it fails. Interestingly, if the
>> /reserved-memory node is placed after /chosen, there is no problem because
>> of the check at the beginning of make_memory_node() that finds the first
>> memory range using the correct type.
>>
>> Fixes: 4596329291f5 ("docs, xen/arm: Introduce static heap memory")
>> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
> 
> Reviewed-by: Kuan-Wei Chiu <visitorckw@xxxxxxxxx>
> 
> I guess git will treat this as conflicting with my dt patch when
> applying. Let me know if this gets merged first and you would like me
> to rebase and resend.
It will likely get merged first but don't worry. I can solve the conflict when I
start reviewing your patch.

~Michal




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.