On 10.09.2020 22:21, Oleksandr Tyshchenko wrote: > --- a/xen/common/ioreq.c > +++ b/xen/common/ioreq.c > @@ -189,7 +189,7 @@ bool handle_hvm_io_completion(struct vcpu *v) > break; > > case HVMIO_mmio_completion: > - return handle_mmio(); > + return ioreq_handle_complete_mmio();
Again the question: Any particular reason to have "handle" in here? With the abstraction simply named ioreq_complete_mmio() feel free to add Acked-by: Jan Beulich <jbeul...@suse.com> Jan