From mboxrd@z Thu Jan 1 00:00:00 1970 From: Razvan Cojocaru Subject: Requesting a freeze exception for vm_event memory introspection helpers Date: Mon, 13 Jul 2015 10:35:44 +0300 Message-ID: <55A36A50.1010404@bitdefender.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "xen-devel@lists.xen.org" Cc: Ian Jackson , kevin.tian@intel.com, keir@xen.org, ian.campbell@citrix.com, stefano.stabellini@eu.citrix.com, george.dunlap@eu.citrix.com, andrew.cooper3@citrix.com, eddie.dong@intel.com, Jan Beulich , Aravind.Gopalakrishnan@amd.com, Jun Nakajima , tlengyel@novetta.com, wei.liu2@citrix.com, boris.ostrovsky@oracle.com, suravee.suthikulpanit@amd.com List-Id: xen-devel@lists.xenproject.org Hello, I'd like to ask for a freeze exception for the "vm_event memory introspection helpers" series. [PATCH 1/3] xen/mem_access: Support for memory-content hiding [PATCH 2/3] xen/vm_event: Support for guest-requested events [PATCH 3/3] xen/vm_event: Deny register writes if refused by vm_event reply All patches have been acked by at least one person (though patch 1 is still under some discussion). 1. Benefits of the series making it in this release: * Probably the most important benefit is that 4.6 development has been very open to refactoring vm_events, and patch 3/3 makes vm_events behave in a consistent manner (all register-write vm_events are pre-write events). * There are 3rd parties interested in these features (Tamas, for example, has already expressed interest in uses of patch 1/1). 2. Risks of including the series: * Since two of the three patches have already received acks from 3+ people, I would assume that the risks for those are minimal. As for the first patch, unless a vm_event consumer uses the new VM_EVENT_FLAG_SET_EMUL_READ_DATA vm_event response flag in conjunction with VM_EVENT_FLAG_EMULATE, the impact should be close to 0 (only a few "if ( unlikely(set_context_enabled) )" extra statements). A new series will follow as soon as possible, addressing Jan's comments on the first patch and cleaning up patch 3 a little more. Thank you in advance for considering this, Razvan