From: Jason Andryuk <jason.andryuk@amd.com>
To: "Penny, Zheng" <penny.zheng@amd.com>, Jan Beulich <jbeulich@suse.com>
Cc: "Andrew Cooper" <andrew.cooper3@citrix.com>,
"Roger Pau Monné" <roger.pau@citrix.com>,
"Tamas K Lengyel" <tamas@tklengyel.com>,
"Alexandru Isaila" <aisaila@bitdefender.com>,
"Petre Pircalabu" <ppircalabu@bitdefender.com>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: Re: [PATCH v1 5/7] xen/x86: move declaration from mem_access.h to altp2m.h
Date: Tue, 18 Nov 2025 18:40:53 -0500 [thread overview]
Message-ID: <7cebd161-5eb2-4755-a78b-28cea5ebd9ff@amd.com> (raw)
In-Reply-To: <DM4PR12MB845171D6D974E63DDF6D4D6DE1D6A@DM4PR12MB8451.namprd12.prod.outlook.com>
On 2025-11-18 06:09, Penny, Zheng wrote:
> [Public]
>
>> -----Original Message-----
>> From: Jan Beulich <jbeulich@suse.com>
>> Sent: Thursday, November 13, 2025 5:35 PM
>> To: Penny, Zheng <penny.zheng@amd.com>
>> Cc: Andrew Cooper <andrew.cooper3@citrix.com>; Roger Pau Monné
>> <roger.pau@citrix.com>; Tamas K Lengyel <tamas@tklengyel.com>; Alexandru
>> Isaila <aisaila@bitdefender.com>; Petre Pircalabu <ppircalabu@bitdefender.com>;
>> xen-devel@lists.xenproject.org
>> Subject: Re: [PATCH v1 5/7] xen/x86: move declaration from mem_access.h to
>> altp2m.h
>>
>> On 13.11.2025 04:16, Penny Zheng wrote:
>>> Memory access and ALTP2M are two seperate features, and each could be
>>> controlled via VM_EVENT or ALTP2M. In order to avoid implicit
>>> declaration when ALTP2M=y and VM_EVENT=n on compiling hvm.o/altp2m.o,
>>> we move declaration of the following functions from <asm/mem_access.h> to
>> <asm/altp2m.h>:
>>> - p2m_set_suppress_ve
>>> - p2m_set_suppress_ve_multi
>>> - p2m_get_suppress_ve
>>> Potential error on altp2m.c also breaks Misra Rule 8.4.
>>>
>>> Signed-off-by: Penny Zheng <Penny.Zheng@amd.com>
>>
>> Reviewed-by: Jan Beulich <jbeulich@suse.com>
>>
>
> Thx
Reviewed-by: Jason Andryuk <jason.andryuk@amd.com>
next prev parent reply other threads:[~2025-11-19 0:17 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-13 3:16 [PATCH v1 0/7] consolidate vm event subsystem Penny Zheng
2025-11-13 3:16 ` [PATCH v1 3/7] xen/monitor: wrap monitor_op under CONFIG_VM_EVENT Penny Zheng
2025-11-13 9:18 ` Jan Beulich
2025-11-18 23:32 ` Jason Andryuk
2025-11-13 3:16 ` [PATCH v1 4/7] xen/p2m: move xenmem_access_to_p2m_access() to common p2m.c Penny Zheng
2025-11-13 9:29 ` Jan Beulich
2025-11-13 3:16 ` [PATCH v1 5/7] xen/x86: move declaration from mem_access.h to altp2m.h Penny Zheng
2025-11-13 9:35 ` Jan Beulich
2025-11-18 11:09 ` Penny, Zheng
2025-11-18 23:40 ` Jason Andryuk [this message]
2025-11-13 3:16 ` [PATCH v1 6/7] xen/mem_access: wrap memory access when VM_EVENT=n Penny Zheng
2025-11-13 9:48 ` Jan Beulich
2025-11-13 3:16 ` [PATCH v1 7/7] xen/vm_event: consolidate CONFIG_VM_EVENT Penny Zheng
2025-11-13 10:16 ` Jan Beulich
2025-11-18 23:49 ` Jason Andryuk
[not found] ` <20251113031630.1465599-2-Penny.Zheng@amd.com>
2025-11-13 8:56 ` [PATCH v1 1/7] xen/svm: limit the scope of "rc" Jan Beulich
2025-11-18 7:11 ` Penny, Zheng
2025-11-18 23:20 ` Jason Andryuk
[not found] ` <20251113031630.1465599-3-Penny.Zheng@amd.com>
2025-11-13 9:13 ` [PATCH v1 2/7] xen/vm_event: introduce vm_event_is_enabled() Jan Beulich
2025-11-18 10:11 ` Penny, Zheng
2025-11-18 23:30 ` Jason Andryuk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7cebd161-5eb2-4755-a78b-28cea5ebd9ff@amd.com \
--to=jason.andryuk@amd.com \
--cc=aisaila@bitdefender.com \
--cc=andrew.cooper3@citrix.com \
--cc=jbeulich@suse.com \
--cc=penny.zheng@amd.com \
--cc=ppircalabu@bitdefender.com \
--cc=roger.pau@citrix.com \
--cc=tamas@tklengyel.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.