Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-26 Thread Oleksandr
On 26.09.20 16:21, Julien Grall wrote: Hi Oleksandr, Hi Julien. On 24/09/2020 19:22, Oleksandr wrote: On 24.09.20 20:25, Julien Grall wrote: On 23/09/2020 21:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-26 Thread Julien Grall
Hi Oleksandr, On 24/09/2020 19:22, Oleksandr wrote: On 24.09.20 20:25, Julien Grall wrote: On 23/09/2020 21:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko Could you please clarify how this patch could

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-26 Thread Oleksandr
On 26.09.20 16:12, Julien Grall wrote: Hi, Hi Julien. On 24/09/2020 19:02, Oleksandr wrote: On 24.09.20 19:02, Oleksandr wrote: On 24.09.20 14:08, Jan Beulich wrote: On 23.09.2020 22:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: On 10/09/2020 21:22, Oleksandr

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-26 Thread Julien Grall
Hi, On 24/09/2020 19:02, Oleksandr wrote: On 24.09.20 19:02, Oleksandr wrote: On 24.09.20 14:08, Jan Beulich wrote: On 23.09.2020 22:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko @@ -91,6 +108,28 @@

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-25 Thread Oleksandr
On 25.09.20 09:51, Jan Beulich wrote: Hi Jan On 24.09.2020 20:02, Oleksandr wrote: On 24.09.20 19:02, Oleksandr wrote: Julien, I noticed that three fields mmio* are not used within current series on Arm. Do we expect them to be used later on? I would rather not add fields which are not

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-25 Thread Jan Beulich
On 24.09.2020 20:02, Oleksandr wrote: > On 24.09.20 19:02, Oleksandr wrote: > Julien, I noticed that three fields mmio* are not used within current > series on Arm. Do we expect them to be used later on? > I would rather not add fields which are not used. We could add them when > needed. > >

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-24 Thread Oleksandr
On 24.09.20 20:25, Julien Grall wrote: Hi Julien. On 23/09/2020 21:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: Hi, Hi Julien On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko I believe I am the originally author of this code...

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-24 Thread Oleksandr
On 24.09.20 19:02, Oleksandr wrote: Hi On 24.09.20 14:08, Jan Beulich wrote: Hi Jan On 23.09.2020 22:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko @@ -91,6 +108,28 @@ struct arch_domain   

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-24 Thread Julien Grall
On 23/09/2020 21:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: Hi, Hi Julien On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko I believe I am the originally author of this code... Sorry, will fix This patch adds basic IOREQ/DM support

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-24 Thread Julien Grall
On 24/09/2020 12:08, Jan Beulich wrote: On 23.09.2020 22:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko @@ -91,6 +108,28 @@ struct arch_domain   #endif   }  __cacheline_aligned;   +enum

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-24 Thread Oleksandr
On 24.09.20 14:08, Jan Beulich wrote: Hi Jan On 23.09.2020 22:16, Oleksandr wrote: On 23.09.20 21:03, Julien Grall wrote: On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko @@ -91,6 +108,28 @@ struct arch_domain   #endif   }  __cacheline_aligned;   +enum

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-24 Thread Jan Beulich
On 23.09.2020 22:16, Oleksandr wrote: > On 23.09.20 21:03, Julien Grall wrote: >> On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: >>> From: Oleksandr Tyshchenko >>> @@ -91,6 +108,28 @@ struct arch_domain >>>   #endif >>>   }  __cacheline_aligned; >>>   +enum hvm_io_completion { >>> +   

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-23 Thread Oleksandr
On 23.09.20 21:03, Julien Grall wrote: Hi, Hi Julien On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko I believe I am the originally author of this code... Sorry, will fix This patch adds basic IOREQ/DM support on Arm. The subsequent patches will

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-23 Thread Julien Grall
Hi, On 10/09/2020 21:22, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko I believe I am the originally author of this code... This patch adds basic IOREQ/DM support on Arm. The subsequent patches will improve functionality, add remaining bits as well as address several TODOs. Find

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-22 Thread Oleksandr
On 16.09.20 10:51, Jan Beulich wrote: Hi Jan On 10.09.2020 22:22, Oleksandr Tyshchenko wrote: @@ -2277,8 +2299,10 @@ void leave_hypervisor_to_guest(void) { local_irq_disable(); -check_for_vcpu_work(); -check_for_pcpu_work(); +do +{ +check_for_pcpu_work();

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-16 Thread Jan Beulich
On 10.09.2020 22:22, Oleksandr Tyshchenko wrote: > @@ -2277,8 +2299,10 @@ void leave_hypervisor_to_guest(void) > { > local_irq_disable(); > > -check_for_vcpu_work(); > -check_for_pcpu_work(); > +do > +{ > +check_for_pcpu_work(); > +} while (

Re: [PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-11 Thread Oleksandr
Hello all. /* diff --git a/xen/include/asm-arm/hvm/ioreq.h b/xen/include/asm-arm/hvm/ioreq.h new file mode 100644 index 000..1c34df0 --- /dev/null +++ b/xen/include/asm-arm/hvm/ioreq.h @@ -0,0 +1,108 @@ +/* + * hvm.h: Hardware virtual machine assist interface definitions. + * + *

[PATCH V1 09/16] arm/ioreq: Introduce arch specific bits for IOREQ/DM features

2020-09-10 Thread Oleksandr Tyshchenko
From: Oleksandr Tyshchenko This patch adds basic IOREQ/DM support on Arm. The subsequent patches will improve functionality, add remaining bits as well as address several TODOs. Please note, the "PIO handling" TODO is expected to left unaddressed for the current series. It is not an big issue