The state of in-flight I/O and how its completion will be handled are
logically separate and conflating the two makes the code unnecessarily
confusing.
Signed-off-by: Paul Durrant
Cc: Keir Fraser
Cc: Jan Beulich
Cc: Andrew Cooper
---
xen/arch/x86/hvm/hvm.c| 50 ++
>>> On 24.06.15 at 13:24, wrote:
> @@ -428,26 +429,12 @@ static void hvm_io_assist(ioreq_t *p)
> vio->io_state = HVMIO_completed;
> vio->io_data = p->data;
> break;
> -case HVMIO_handle_mmio_awaiting_completion:
> -vio->io_state = HVMIO_completed;
> -
> -Original Message-
> From: Jan Beulich [mailto:jbeul...@suse.com]
> Sent: 25 June 2015 10:41
> To: Paul Durrant
> Cc: Andrew Cooper; xen-de...@lists.xenproject.org; Keir (Xen.org)
> Subject: Re: [PATCH v4 12/17] x86/hvm: split I/O completion handling from
> state model
>
> >>> On 24.06.1