[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 5/6] xen/mem_access: wrap memory access when VM_EVENT=n
- To: Jason Andryuk <jason.andryuk@xxxxxxx>
- From: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
- Date: Fri, 16 Jan 2026 18:19:00 -0500
- Arc-authentication-results: i=1; mx.zohomail.com; dkim=pass header.i=tklengyel.com; spf=pass smtp.mailfrom=tamas@xxxxxxxxxxxxx; dmarc=pass header.from=<tamas@xxxxxxxxxxxxx>
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1768605649; h=Content-Type:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=TKtiMH8oBdrfkRfa3Gcqh/WpBKYSshT5TbDes+EMBp0=; b=DxHsd30+UCmB5fQFmKOD0cyOe2kqXDkvsd3aNDuUzSVLm9KzrHQrLPfUpf0HnoV4I04Ra5C8d9Peu/edIvZuLKvhj6nidC3Y0gs3BYy+8xKKEdvNXRlh7X0w5/3TwWMHuJVzIDMX9j9uq0QRNKjgtP/IVumHjU+jWT35vmap0nk=
- Arc-seal: i=1; a=rsa-sha256; t=1768605649; cv=none; d=zohomail.com; s=zohoarc; b=Un8is00hNzVntpxcDK4/ddu+5be/GO5xL2T3YfGmhwK0J1hAHmUZn4exi5V516ocbBK66Q0K5ZJy34+sSGqBozD0vXm/IJUdL4/tzTnqtGsfAXmM/2kd0hzvToZU85uyCVouUWBs5+mC5RFVelRx4GiYttaZdd/c/YhE6k5Spog=
- Cc: Penny Zheng <Penny.Zheng@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, ray.huang@xxxxxxx, Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Alexandru Isaila <aisaila@xxxxxxxxxxxxxxx>, Petre Pircalabu <ppircalabu@xxxxxxxxxxxxxxx>
- Delivery-date: Fri, 16 Jan 2026 23:20:55 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 2026-01-15 04:28, Penny Zheng wrote:
> Feature memory access is based on vm event subsystem, and it could be disabled
> in the future. So a few switch-blocks in do_altp2m_op() need
> vm_event_is_enabled() condition check to pass compilation when ALTP2M=y and
> VM_EVENT=n(, hence MEM_ACCESS=n), like HVMOP_altp2m_set_mem_access, etc.
> Function p2m_mem_access_check() still needs stub when VM_EVENT=n to
> pass compilation.
> Although local variable "req_ptr" still remains NULL throughout its lifetime,
> with the change of NULL assignment, we will face runtime undefined error only
> when CONFIG_USBAN is on. So we strengthen the condition check via adding
> vm_event_is_enabled() for the special case.
>
> Signed-off-by: Penny Zheng <Penny.Zheng@xxxxxxx>
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Jason Andryuk <jason.andryuk@xxxxxxx>
|
