>>> On 24.06.15 at 13:24, <paul.durr...@citrix.com> wrote: > --- a/xen/arch/x86/hvm/emulate.c > +++ b/xen/arch/x86/hvm/emulate.c > @@ -138,20 +138,14 @@ static int hvmemul_do_io( > if ( data_is_addr || dir == IOREQ_WRITE ) > return X86EMUL_UNHANDLEABLE; > goto finish_access; > - case HVMIO_dispatched: > - /* May have to wait for previous cycle of a multi-write to complete. > */ > - if ( is_mmio && !data_is_addr && (dir == IOREQ_WRITE) && > - (addr == (vio->mmio_large_write_pa + > - vio->mmio_large_write_bytes)) ) > - return X86EMUL_RETRY; > - /* fallthrough */
The description is plausible, and the rest of the patch looks plausible too, but I can't seem to follow why the above can go away without any replacement (or what the replacement would be). Could you clarify that, please? Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
