Re: [RFC PATCH V2 0/8] xen: Clean-up of mem_event subsystem

[Andrew Cooper <andrew.cooper3@citrix.com>]

On 18/01/15 15:17, Tamas K Lengyel wrote:
> This patch series aims to clean up the mem_event subsystem within Xen. The
> original use-case for this system was to allow external helper applications
> running in privileged domains to control various memory operations performed
> by Xen. Amongs these were paging, sharing and access control. The subsystem
> has since been extended to also deliver non-memory related events, namely
> various HVM debugging events (INT3, MTF, MOV-TO-CR, MOV-TO-MSR). The structures
> and naming of related functions however has not caught up to these new
> use-cases, thus leaving many ambigouities in the code.
>
> In this series we convert the mem_event structures to a union of sub-structures
> which clearly define the scope of information that is transmitted via the event
> delivery mechanism. Afterwards, we clean up the naming of the structures and
> related functions to more clearly be in line with their actual operations.
>
> This RFC PATCH series is also available at:
> https://github.com/tklengyel/xen/tree/mem_event_cleanup2

One note to start with.  There are a lot of file renames in this
series.  Using gits -M option for patches should make the series rather
smaller.

I also wonder whether it might be easier to organise the series along
the lines of introducing vm_event, move the existing interfaces across,
then deleting mem_event.  Lets see how easy it is to review in its
current form first though.

~Andrew