;Ian Jackson'
> ; 'Stefano Stabellini'
> ; 'Wei Liu' ; 'Roger Pau Monné'
> ; 'Jun
> Nakajima' ; 'Kevin Tian' ; 'Tim
> Deegan' ;
> 'Julien Grall'
> Subject: Re: [PATCH V1 02/16] xen/ioreq: Make x86&#x
On 30.09.2020 19:47, Julien Grall wrote:
> Regarding the fix itself, I am not sure what sort of synchronization we
> can do. Are you suggesting to wait for the I/O to complete? If so, how
> do we handle the case the IOREQ server died?
In simple cases retrying the entire request may be an option.
n'
> ; 'Jan Beulich' ; 'Stefano
> Stabellini'
> ; 'Wei Liu' ; 'Roger Pau Monné'
> ; 'Jun
> Nakajima' ; 'Kevin Tian' ; 'Tim
> Deegan' ;
> 'Julien Grall'
> Subject: Re: [PATCH
Jackson
; Jan Beulich
; Stefano Stabellini ; Wei
Liu ; Roger Pau
Monné ; Paul Durrant ; Jun
Nakajima ;
Kevin Tian ; Tim Deegan ; Julien
Grall
Subject: Re: [PATCH V1 02/16] xen/ioreq: Make x86's IOREQ feature common
On 10/09/2020 21:21, Oleksandr Tyshchenko wrote:
+static
; Wei Liu
; Roger Pau
Monné ; Paul Durrant ; Jun Nakajima
;
Kevin Tian ; Tim Deegan ; Julien Grall
Subject: Re: [PATCH V1 02/16] xen/ioreq: Make x86's IOREQ feature common
On 10/09/2020 21:21, Oleksandr Tyshchenko wrote:
+static bool hvm_wait_for_io(struct hvm_ioreq_vcpu *sv, ioreq
Liu
> ; Roger Pau
> Monné ; Paul Durrant ; Jun Nakajima
> ;
> Kevin Tian ; Tim Deegan ; Julien Grall
>
> Subject: Re: [PATCH V1 02/16] xen/ioreq: Make x86's IOREQ feature common
>
>
>
> On 10/09/2020 21:21, Oleksandr Tyshchenko wrote:
> >
On 10/09/2020 21:21, Oleksandr Tyshchenko wrote:
+static bool hvm_wait_for_io(struct hvm_ioreq_vcpu *sv, ioreq_t *p)
+{
+unsigned int prev_state = STATE_IOREQ_NONE;
+unsigned int state = p->state;
+uint64_t data = ~0;
+
+smp_rmb();
+
+/*
+ * The only reason we should se
On 24.09.2020 17:38, Oleksandr wrote:
> On 24.09.20 13:58, Jan Beulich wrote:
>> On 23.09.2020 14:28, Oleksandr wrote:
>>> On 22.09.20 18:52, Jan Beulich wrote:
On 22.09.2020 17:05, Oleksandr wrote:
> @@ -247,7 +247,7 @@ static gfn_t hvm_alloc_legacy_ioreq_gfn(struct
> hvm_ioreq_server
On 24.09.20 13:58, Jan Beulich wrote:
Hi Jan
On 23.09.2020 14:28, Oleksandr wrote:
On 22.09.20 18:52, Jan Beulich wrote:
On 22.09.2020 17:05, Oleksandr wrote:
3. *arch.hvm.hvm_io*: We could also use the following:
#define ioreq_get_io_completion(v) ((v)->arch.hvm.hvm_io.io_completio
On 23.09.2020 14:28, Oleksandr wrote:
> On 22.09.20 18:52, Jan Beulich wrote:
>> On 22.09.2020 17:05, Oleksandr wrote:
>>> 3. *arch.hvm.hvm_io*: We could also use the following:
>>>
>>> #define ioreq_get_io_completion(v) ((v)->arch.hvm.hvm_io.io_completion)
>>> #define ioreq_get_io_req(v)
On 22.09.20 18:52, Jan Beulich wrote:
Hi Jan
On 22.09.2020 17:05, Oleksandr wrote:
2. *arch.hvm.params*: Two functions that use it
(hvm_alloc_legacy_ioreq_gfn/hvm_free_legacy_ioreq_gfn) either go into
arch code completely or
specific macro is used in common code:
#define ioreq_ge
On 22.09.2020 17:05, Oleksandr wrote:
> 2. *arch.hvm.params*: Two functions that use it
> (hvm_alloc_legacy_ioreq_gfn/hvm_free_legacy_ioreq_gfn) either go into
> arch code completely or
> specific macro is used in common code:
>
> #define ioreq_get_params(d, i) ((d)->arch.hvm.params[i])
On 22.09.20 13:54, Jan Beulich wrote:
Hi Jan
On 22.09.2020 11:58, Oleksandr wrote:
On 22.09.20 09:33, Jan Beulich wrote:
On 21.09.2020 21:02, Oleksandr wrote:
On 14.09.20 17:17, Jan Beulich wrote:
On 10.09.2020 22:21, Oleksandr Tyshchenko wrote:
+#define GET_IOREQ_SERVER(d, id) \
+(d
On 22.09.2020 11:58, Oleksandr wrote:
> On 22.09.20 09:33, Jan Beulich wrote:
>> On 21.09.2020 21:02, Oleksandr wrote:
>>> On 14.09.20 17:17, Jan Beulich wrote:
On 10.09.2020 22:21, Oleksandr Tyshchenko wrote:
> +#define GET_IOREQ_SERVER(d, id) \
> +(d)->arch.hvm.ioreq_server.serve
On 22.09.20 09:33, Jan Beulich wrote:
Hi Jan
On 21.09.2020 21:02, Oleksandr wrote:
On 14.09.20 17:17, Jan Beulich wrote:
On 10.09.2020 22:21, Oleksandr Tyshchenko wrote:
--- /dev/null
+++ b/xen/include/xen/ioreq.h
@@ -0,0 +1,82 @@
+/*
+ * ioreq.h: Hardware virtual machine assist interface
On 21.09.2020 21:02, Oleksandr wrote:
> On 14.09.20 17:17, Jan Beulich wrote:
>> On 10.09.2020 22:21, Oleksandr Tyshchenko wrote:
>>> --- /dev/null
>>> +++ b/xen/include/xen/ioreq.h
>>> @@ -0,0 +1,82 @@
>>> +/*
>>> + * ioreq.h: Hardware virtual machine assist interface definitions.
>>> + *
>>> + *
On 14.09.20 17:17, Jan Beulich wrote:
Hi Jan
On 10.09.2020 22:21, Oleksandr Tyshchenko wrote:
---
MAINTAINERS |8 +-
xen/arch/x86/Kconfig|1 +
xen/arch/x86/hvm/dm.c |2 +-
xen/arch/x86/hvm/emulate.c |2 +-
xen/arch/x86/hvm/
On 10.09.2020 22:21, Oleksandr Tyshchenko wrote:
> ---
> MAINTAINERS |8 +-
> xen/arch/x86/Kconfig|1 +
> xen/arch/x86/hvm/dm.c |2 +-
> xen/arch/x86/hvm/emulate.c |2 +-
> xen/arch/x86/hvm/hvm.c |2 +-
> xen/arch/x86/hvm/
18 matches
Mail list logo
