mbox series

[v4,0/3] make build of vm_event/mem_access/monitor optional

Message ID cover.1741772330.git.Sergiy_Kibrik@epam.com (mailing list archive)
Headers show
Series make build of vm_event/mem_access/monitor optional | expand

Message

Sergiy Kibrik March 14, 2025, 5:19 a.m. UTC 
This series aims to provide a possibility to build Xen without mem_access and
related features. It is already largely covered by corresponding
CONFIG_MEM_ACCESS option, yet there're still some parts remaining.
Hopefully this would help to reduce dead code a bit.

As coverage of MEM_ACCESS config option begins to extend beyond actual
mem_access code it has been suggested to rename it into VM_EVENT, as a more
general option controlling mem_access, vm_event and monitor code.

v3 series:
https://lore.kernel.org/xen-devel/cover.1741687645.git.Sergiy_Kibrik@epam.com/
  -Sergiy


Sergiy Kibrik (2):
  xen: kconfig: rename MEM_ACCESS -> VM_EVENT
  x86:monitor: control monitor.c build with CONFIG_VM_EVENT option

Stefano Stabellini (1):
  xen: mem_access: conditionally compile vm_event.c & monitor.c

 automation/gitlab-ci/build.yaml         |  2 +-
 xen/arch/arm/Makefile                   |  6 +++---
 xen/arch/arm/configs/tiny64_defconfig   |  2 +-
 xen/arch/arm/include/asm/mem_access.h   |  4 ++--
 xen/arch/arm/vsmc.c                     |  3 ++-
 xen/arch/ppc/configs/ppc64_defconfig    |  2 +-
 xen/arch/riscv/configs/tiny64_defconfig |  2 +-
 xen/arch/x86/Kconfig                    |  2 +-
 xen/arch/x86/Makefile                   |  2 +-
 xen/arch/x86/mm/Makefile                |  2 +-
 xen/common/Kconfig                      |  2 +-
 xen/common/Makefile                     |  6 +++---
 xen/common/domctl.c                     |  2 +-
 xen/include/xen/mem_access.h            |  6 +++---
 xen/include/xen/monitor.h               |  9 +++++++++
 xen/include/xen/vm_event.h              | 14 +++++++++++---
 xen/include/xsm/dummy.h                 |  2 +-
 xen/include/xsm/xsm.h                   |  4 ++--
 xen/xsm/dummy.c                         |  2 +-
 xen/xsm/flask/hooks.c                   |  4 ++--
 20 files changed, 48 insertions(+), 30 deletions(-)

Comments

Andrew Cooper March 14, 2025, 5:38 p.m. UTC | #1 
On 14/03/2025 5:19 am, Sergiy Kibrik wrote:
> This series aims to provide a possibility to build Xen without mem_access and
> related features. It is already largely covered by corresponding
> CONFIG_MEM_ACCESS option, yet there're still some parts remaining.
> Hopefully this would help to reduce dead code a bit.
>
> As coverage of MEM_ACCESS config option begins to extend beyond actual
> mem_access code it has been suggested to rename it into VM_EVENT, as a more
> general option controlling mem_access, vm_event and monitor code.
>
> v3 series:
> https://lore.kernel.org/xen-devel/cover.1741687645.git.Sergiy_Kibrik@epam.com/
>   -Sergiy
>
>
> Sergiy Kibrik (2):
>   xen: kconfig: rename MEM_ACCESS -> VM_EVENT
>   x86:monitor: control monitor.c build with CONFIG_VM_EVENT option
>
> Stefano Stabellini (1):
>   xen: mem_access: conditionally compile vm_event.c & monitor.c

I've taken this as it's been going on for long enough.

I've folded a few fixes.  Please be aware of these in the future.

In EXTRA_RANDCONFIG in patch 1, the CONFIG items are sorted, so the
rename caused a resorting to be needed.

In patch 3, extra comments on the #else and #endif, before more content
gets added.

~Andrew