On 09/12/11 18:53, Sasha Levin wrote: > On Fri, 2011-12-09 at 17:56 +1100, Matt Evans wrote: >> @@ -30,4 +31,18 @@ struct kvm_cpu { >> struct kvm_coalesced_mmio_ring *ring; >> }; >> >> +/* >> + * As these are such simple wrappers, let's have them in the header so they'll >> + * be cheaper to call: >> + */ >> +static inline bool kvm_cpu__emulate_io(struct kvm *kvm, u16 port, void *data, int direction, int size, u32 count) >> +{ >> + return kvm__emulate_io(kvm, port, data, direction, size, count); >> +} >> + >> +static inline bool kvm_cpu__emulate_mmio(struct kvm *kvm, u64 phys_addr, u8 *data, u32 len, u8 is_write) >> +{ >> + return kvm_cpu__emulate_mmio(kvm, phys_addr, data, len, is_write); > > This is probably wrong. kvm_cpu__emulate_mmio just calls itself over and > over. Urgh, not just probably -- C&P strikes again. Consider it fixed. Thanks! Matt -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html