[PATCH v4 2/3] x86:monitor: control monitor.c build with CONFIG_VM_EVENT option

Sergiy Kibrik posted 3 patches 9 months, 1 week ago
[PATCH v4 2/3] x86:monitor: control monitor.c build with CONFIG_VM_EVENT option
Posted by Sergiy Kibrik 9 months, 1 week ago
Replace more general CONFIG_HVM option with CONFIG_VM_EVENT which is more
relevant and specific to monitoring. This is only to clarify at build level
to which subsystem this file belongs.

No functional change here, as VM_EVENT depends on HVM.

Acked-by: Jan Beulich <jbeulich@suse.com>
Signed-off-by: Sergiy Kibrik <Sergiy_Kibrik@epam.com>
---
 xen/arch/x86/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/Makefile b/xen/arch/x86/Makefile
index c763f80b0b..f59c9665fd 100644
--- a/xen/arch/x86/Makefile
+++ b/xen/arch/x86/Makefile
@@ -49,7 +49,7 @@ obj-$(CONFIG_PV) += ioport_emulate.o
 obj-y += irq.o
 obj-$(CONFIG_KEXEC) += machine_kexec.o
 obj-y += mm.o x86_64/mm.o
-obj-$(CONFIG_HVM) += monitor.o
+obj-$(CONFIG_VM_EVENT) += monitor.o
 obj-y += mpparse.o
 obj-y += nmi.o
 obj-y += numa.o
-- 
2.25.1
Re: [PATCH v4 2/3] x86:monitor: control monitor.c build with CONFIG_VM_EVENT option
Posted by Jan Beulich 9 months, 1 week ago
On 14.03.2025 06:23, Sergiy Kibrik wrote:
> Replace more general CONFIG_HVM option with CONFIG_VM_EVENT which is more
> relevant and specific to monitoring. This is only to clarify at build level
> to which subsystem this file belongs.
> 
> No functional change here, as VM_EVENT depends on HVM.
> 
> Acked-by: Jan Beulich <jbeulich@suse.com>
> Signed-off-by: Sergiy Kibrik <Sergiy_Kibrik@epam.com>

I see you've adjusted tag order in patch 3. Why not also here and in patch 1?

Jan
Re: [PATCH v4 2/3] x86:monitor: control monitor.c build with CONFIG_VM_EVENT option
Posted by Sergiy Kibrik 9 months, 1 week ago
14.03.25 13:11, Jan Beulich:
> On 14.03.2025 06:23, Sergiy Kibrik wrote:
>> Replace more general CONFIG_HVM option with CONFIG_VM_EVENT which is more
>> relevant and specific to monitoring. This is only to clarify at build level
>> to which subsystem this file belongs.
>>
>> No functional change here, as VM_EVENT depends on HVM.
>>
>> Acked-by: Jan Beulich <jbeulich@suse.com>
>> Signed-off-by: Sergiy Kibrik <Sergiy_Kibrik@epam.com>
> 
> I see you've adjusted tag order in patch 3. Why not also here and in patch 1?
> 

oops.. I've missed those, sorry..

   -Sergiy