Re: [PATCH 2/2] kvm tools: Add MMIO address mapper

[Ingo Molnar <mingo@elte.hu>]

* Sasha Levin <levinsasha928@gmail.com> wrote:

> When we have a MMIO exit, we need to find which device
> has registered to use the accessed MMIO space.
> 
> The mapper maps ranges of guest physical addresses to
> callback functions.
> 
> Implementation is based on an interval red-black tree.
> 
> Signed-off-by: Sasha Levin <levinsasha928@gmail.com>
> ---
>  tools/kvm/include/kvm/kvm.h |    2 +
>  tools/kvm/mmio.c            |   79 +++++++++++++++++++++++++++++++++++++++++-
>  2 files changed, 79 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/kvm/include/kvm/kvm.h b/tools/kvm/include/kvm/kvm.h
> index b310d50..d9943bf 100644
> --- a/tools/kvm/include/kvm/kvm.h
> +++ b/tools/kvm/include/kvm/kvm.h
> @@ -44,6 +44,8 @@ void kvm__stop_timer(struct kvm *kvm);
>  void kvm__irq_line(struct kvm *kvm, int irq, int level);
>  bool kvm__emulate_io(struct kvm *kvm, u16 port, void *data, int direction, int size, u32 count);
>  bool kvm__emulate_mmio(struct kvm *kvm, u64 phys_addr, u8 *data, u32 len, u8 is_write);
> +bool kvm__register_mmio(u64 phys_addr, u64 phys_addr_len, void (*kvm_mmio_callback_fn)(u64 addr, u8 *data, u32 len, u8 is_write));
> +bool kvm__deregister_mmio(u64 phys_addr);
>  
>  /*
>   * Debugging
> diff --git a/tools/kvm/mmio.c b/tools/kvm/mmio.c
> index 848267d..fab6489 100644
> --- a/tools/kvm/mmio.c
> +++ b/tools/kvm/mmio.c
> @@ -1,7 +1,48 @@
>  #include "kvm/kvm.h"
> +#include "kvm/interval-rbtree.h"
>  
>  #include <stdio.h>
> +#include <stdlib.h>
> +
>  #include <linux/types.h>
> +#include <linux/rbtree.h>
> +
> +#define MMIO_NODE(n) container_of(n, struct mmio_mapping, node)

Please make this lowercase, mmio_node() or mmio_entry().

Other than that:

Acked-by: Ingo Molnar <mingo@elte.hu>

Thanks,

	Ingo