Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-10 Thread Oleksandr
' ; 'Andrew Cooper' ; 'George Dunlap' ; 'Ian Jackson' ; 'Wei Liu' ; 'Julien Grall' ; xen-devel@lists.xenproject.org Subject: Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server() Hi Paul. On 30.11.2020 11:31, Oleksandr Tysh

RE: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-10 Thread Paul Durrant
x27;Andrew Cooper' ; > 'George Dunlap' > ; 'Ian Jackson' ; 'Wei Liu' > ; 'Julien Grall' > ; xen-devel@lists.xenproject.org > Subject: Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server() > > > Hi Paul. > >

Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-09 Thread Oleksandr
'Oleksandr Tyshchenko' ; 'Stefano Stabellini' ; 'Julien Grall' ; 'Volodymyr Babchuk' ; 'Andrew Cooper' ; 'George Dunlap' ; 'Ian Jackson' ; 'Wei Liu' ; 'Julien Grall' ; xen-devel@lists.xenproject.org Subj

Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-09 Thread Oleksandr
Hi Paul. On 30.11.2020 11:31, Oleksandr Tyshchenko wrote: --- a/xen/include/xen/ioreq.h +++ b/xen/include/xen/ioreq.h @@ -55,6 +55,20 @@ struct ioreq_server { uint8_tbufioreq_handling; }; +/* + * This should only be used when d == current->domain and it's not

Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-09 Thread Julien Grall
abellini' ; 'Julien Grall' ; 'Volodymyr Babchuk' ; 'Andrew Cooper' ; 'George Dunlap' ; 'Ian Jackson' ; 'Wei Liu' ; 'Julien Grall' ; xen-devel@lists.xenproject.org Subject: Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_ser

RE: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-09 Thread Paul Durrant
x27;Andrew Cooper' ; > 'George Dunlap' > ; 'Ian Jackson' ; 'Wei Liu' > ; 'Julien Grall' > ; xen-devel@lists.xenproject.org > Subject: Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server() > > > On 08.12.20 2

Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-08 Thread Oleksandr
; Julien Grall ; xen-devel@lists.xenproject.org Subject: Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server() Hi Paul. On 08.12.20 17:33, Oleksandr wrote: On 08.12.20 17:11, Jan Beulich wrote: Hi Jan On 30.11.2020 11:31, Oleksandr Tyshchenko wrote: --- a/xen/include/xen

RE: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-08 Thread Paul Durrant
u > ; Julien Grall > ; xen-devel@lists.xenproject.org > Subject: Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server() > > > Hi Paul. > > > On 08.12.20 17:33, Oleksandr wrote: > > > > On 08.12.20 17:11, Jan Beulich wrote: > > > &

Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-08 Thread Oleksandr
Hi Paul. On 08.12.20 17:33, Oleksandr wrote: On 08.12.20 17:11, Jan Beulich wrote: Hi Jan On 30.11.2020 11:31, Oleksandr Tyshchenko wrote: --- a/xen/include/xen/ioreq.h +++ b/xen/include/xen/ioreq.h @@ -55,6 +55,20 @@ struct ioreq_server {   uint8_t    bufioreq_handling;

Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-08 Thread Oleksandr
On 08.12.20 17:11, Jan Beulich wrote: Hi Jan On 30.11.2020 11:31, Oleksandr Tyshchenko wrote: --- a/xen/include/xen/ioreq.h +++ b/xen/include/xen/ioreq.h @@ -55,6 +55,20 @@ struct ioreq_server { uint8_tbufioreq_handling; }; +/* + * This should only be used when d

Re: [PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-12-08 Thread Jan Beulich
On 30.11.2020 11:31, Oleksandr Tyshchenko wrote: > --- a/xen/include/xen/ioreq.h > +++ b/xen/include/xen/ioreq.h > @@ -55,6 +55,20 @@ struct ioreq_server { > uint8_tbufioreq_handling; > }; > > +/* > + * This should only be used when d == current->domain and it's not paused,

[PATCH V3 17/23] xen/ioreq: Introduce domain_has_ioreq_server()

2020-11-30 Thread Oleksandr Tyshchenko
From: Oleksandr Tyshchenko This patch introduces a helper the main purpose of which is to check if a domain is using IOREQ server(s). On Arm the current benefit is to avoid calling vcpu_ioreq_handle_completion() (which implies iterating over all possible IOREQ servers anyway) on every return in