Re: [Xen-devel] [PATCH v4 11/17] x86/hvm: only call hvm_io_assist() from hvm_wait_for_io()

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: xen-devel-bounces@xxxxxxxxxxxxx [mailto:xen-devel-
> bounces@xxxxxxxxxxxxx] On Behalf Of Jan Beulich
> Sent: 24 June 2015 16:37
> To: Paul Durrant
> Cc: Andrew Cooper; Keir (Xen.org); xen-devel@xxxxxxxxxxxxxxxxxxxx
> Subject: Re: [Xen-devel] [PATCH v4 11/17] x86/hvm: only call hvm_io_assist()
> from hvm_wait_for_io()
> 
> >>> On 24.06.15 at 13:24, <paul.durrant@xxxxxxxxxx> wrote:
> > By removing the calls in hvmemul_do_io() (which is replaced by a single
> > assignment) and hvm_complete_assist_request() (which is replaced by a
> > call to process_portio_intercept() with a suitable set of ops) then
> 
> With this saying hvm_process_io_intercept() ...
> 

Ah, good spot :-)

> > hvm_io_assist() can be moved into hvm.c and made static (and hence be a
> > candidate for inlining).
> >
> > This patch also fixes the I/O state test at the end of hvm_io_assist()
> > to check the correct value. Since the ioreq server patch series was
> > integrated the current ioreq state is no longer an indicator of in-flight
> > I/O state, since an I/O sheduled by re-emulation may be targetted at a
> > different ioreq server.
> >
> > Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
> 
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
> 

Thanks.

  Paul

> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel