[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 3/3] xen: mem_access: conditionally compile vm_event.c & monitor.c
- To: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>
- From: Sergiy Kibrik <sergiy_kibrik@xxxxxxxx>
- Date: Wed, 12 Mar 2025 11:35:14 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none
- 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=7c+qSwfT/v0DQQKww8Qsx0ETY5tHj8E991ksUvFL25w=; b=rMxTUXGPzFEo//9pCUpGSgaGq4zf7iKafFJhSLFvatuu35SGoCMW9INhkT1Y/iiRxJYZYV4B9c1QDK6SiM2xPf+vC93o7ZKhvfLJxigFsh2YZvfPSwRLufV3dnkgNAFpYK0QvL7jY9ABOWzdeM2zsu/0f9eMp5yIf607+DSVxkXzgvCq3S4NlPe4HlwAFK2N73AnfG+v840HPBxTqpzF2Xs+tDh3LxTjvfMVDfvXibfiXfC/pl+oWCcRA8hW28lWGQ6nMD9EUiimjUNavht4ANSqrlA8kF5Sd9KpH0o6eUTPMCdGrzhu88NowWtP8YlAfPTIwoAxLM8wG2lmHwYVzQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ONxkCfV4NBNVJDrMFPrbELFTpaAislVRWfi5jF+yTA6nb/0v2bpegp0gJADLKGHBQLJLetxjf/myY06pjqWQ1zPs7gvV41BfVp6dNF5maCxJVmA8FL4859/U4QcHgTHZzEEd/a34zpZogeMiOh5EQyRI4N9gKiyDD1uutbNNofIrzWA/166D+JWUxQ1yoBLLYbon9FAh9Gm/VBdiv8Iucd05txD5DpTKManudfikLUIVAt5aGImIkPAsOrK4EavJ5BT9+wqVRA/C5TSfNyYpjDs/OPHqTriFPSiDVYyJctFV/zfWIHJximQPiJFy87pElnk0HIwgIyoFVaEsNOImlA==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com;
- Cc: Stefano Stabellini <stefano.stabellini@xxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Alexandru Isaila <aisaila@xxxxxxxxxxxxxxx>, Petre Pircalabu <ppircalabu@xxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Wed, 12 Mar 2025 09:35:39 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
11.03.25 15:39, Tamas K Lengyel:
On Tue, Mar 11, 2025 at 7:59 AM Jan Beulich<jbeulich@xxxxxxxx> wrote:
On 11.03.2025 11:27, Sergiy Kibrik wrote:
From: Stefano Stabellini<stefano.stabellini@xxxxxxx>
Extend coverage of CONFIG_VM_EVENT option and make the build of VM events
and monitoring support optional. Also make MEM_PAGING option depend on VM_EVENT
to document that mem_paging is relying on vm_event.
This is to reduce code size on Arm when this option isn't enabled.
CC: Jan Beulich<jbeulich@xxxxxxxx>
CC: Tamas K Lengyel<tamas@xxxxxxxxxxxxx>
Reviewed-by: Ayan Kumar Halder<ayan.kumar.halder@xxxxxxx>
Signed-off-by: Stefano Stabellini<stefano.stabellini@xxxxxxx>
Signed-off-by: Sergiy Kibrik<Sergiy_Kibrik@xxxxxxxx>
Please can tags be kept in chronological order? It's impossible to review
a patch that wasn't first signed-off on by the author(s).
---
changes in v3:
- add dependency MEM_PAGING -> VM_EVENT
This seems to be largely unnecessary since on x86 selecting HVM
already selects it but I guess it also doesn't hurt to explicitly mark
it like this either.
It's done just to highlight this dependency, more like a documentation
hint (as Jan suggested in previous discussion).
-Sergiy
|