[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/2] x86/HVM: fix forwarding of internally cached requests (part 2)


  • To: Jan Beulich <JBeulich@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Paul Durrant <Paul.Durrant@xxxxxxxxxx>
  • Date: Thu, 28 Apr 2016 09:49:11 +0000
  • Accept-language: en-GB, en-US
  • Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
  • Delivery-date: Thu, 28 Apr 2016 09:49:17 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>
  • Thread-index: AQHRoTED4uA4byaaM0q6MnsoWmJ5RZ+fI/lQ
  • Thread-topic: [PATCH 2/2] x86/HVM: fix forwarding of internally cached requests (part 2)

> -----Original Message-----
> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
> Sent: 28 April 2016 10:33
> To: xen-devel
> Cc: Paul Durrant; Wei Liu
> Subject: [PATCH 2/2] x86/HVM: fix forwarding of internally cached requests
> (part 2)
> 
> Commit 96ae556569 ("x86/HVM: fix forwarding of internally cached
> requests") wasn't quite complete: hvmemul_do_io() also needs to
> propagate up the clipped count. (I really should have re-tested the
> forward port resulting in the earlier change, instead of relying on the
> testing done on the older version of Xen which the fix was first needed
> for.)
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

> 
> --- a/xen/arch/x86/hvm/emulate.c
> +++ b/xen/arch/x86/hvm/emulate.c
> @@ -137,7 +137,7 @@ static int hvmemul_do_io(
>          if ( (p.type != (is_mmio ? IOREQ_TYPE_COPY : IOREQ_TYPE_PIO)) ||
>               (p.addr != addr) ||
>               (p.size != size) ||
> -             (p.count != *reps) ||
> +             (p.count > *reps) ||
>               (p.dir != dir) ||
>               (p.df != df) ||
>               (p.data_is_ptr != data_is_addr) )
> @@ -145,6 +145,8 @@ static int hvmemul_do_io(
> 
>          if ( data_is_addr )
>              return X86EMUL_UNHANDLEABLE;
> +
> +        *reps = p.count;
>          goto finish_access;
>      default:
>          return X86EMUL_UNHANDLEABLE;
> @@ -162,6 +164,13 @@ static int hvmemul_do_io(
> 
>      rc = hvm_io_intercept(&p);
> 
> +    /*
> +     * p.count may have got reduced (see hvm_process_io_intercept()) -
> inform
> +     * our callers and mirror this into latched state.
> +     */
> +    ASSERT(p.count <= *reps);
> +    *reps = vio->io_req.count = p.count;
> +
>      switch ( rc )
>      {
>      case X86EMUL_OKAY:
> 
> 


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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.