Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-22 Thread Jan Beulich
>>> On 22.09.17 at 10:44, wrote: > On 22/09/17 10:35, Jan Beulich wrote: > On 22.09.17 at 10:27, wrote: >>> On 22/09/17 09:53, Jan Beulich wrote: >>> On 22.09.17 at 08:19, wrote: > On 21/09/17 13:48, Jan Beulich wrote: > On 21.09.17 at 13:39, wrote: >>> On 21/09/17 13:31

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-22 Thread Juergen Gross
On 22/09/17 10:35, Jan Beulich wrote: On 22.09.17 at 10:27, wrote: >> On 22/09/17 09:53, Jan Beulich wrote: >> On 22.09.17 at 08:19, wrote: On 21/09/17 13:48, Jan Beulich wrote: On 21.09.17 at 13:39, wrote: >> On 21/09/17 13:31, Jan Beulich wrote: >> On 21.09.1

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-22 Thread Jan Beulich
>>> On 22.09.17 at 10:27, wrote: > On 22/09/17 09:53, Jan Beulich wrote: > On 22.09.17 at 08:19, wrote: >>> On 21/09/17 13:48, Jan Beulich wrote: >>> On 21.09.17 at 13:39, wrote: > On 21/09/17 13:31, Jan Beulich wrote: > On 21.09.17 at 09:53, wrote: >>> On 21/09/17 08:15

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-22 Thread Juergen Gross
On 22/09/17 09:53, Jan Beulich wrote: On 22.09.17 at 08:19, wrote: >> On 21/09/17 13:48, Jan Beulich wrote: >> On 21.09.17 at 13:39, wrote: On 21/09/17 13:31, Jan Beulich wrote: On 21.09.17 at 09:53, wrote: >> On 21/09/17 08:15, Jan Beulich wrote: >> On 21.09.1

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-22 Thread Jan Beulich
>>> On 22.09.17 at 08:19, wrote: > On 21/09/17 13:48, Jan Beulich wrote: > On 21.09.17 at 13:39, wrote: >>> On 21/09/17 13:31, Jan Beulich wrote: >>> On 21.09.17 at 09:53, wrote: > On 21/09/17 08:15, Jan Beulich wrote: > On 21.09.17 at 06:35, wrote: >>> On 20/09/17 17:35

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-21 Thread Juergen Gross
On 21/09/17 13:48, Jan Beulich wrote: On 21.09.17 at 13:39, wrote: >> On 21/09/17 13:31, Jan Beulich wrote: >> On 21.09.17 at 09:53, wrote: On 21/09/17 08:15, Jan Beulich wrote: On 21.09.17 at 06:35, wrote: >> On 20/09/17 17:35, Jan Beulich wrote: >> On 20.09.1

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-21 Thread Juergen Gross
On 21/09/17 13:48, Jan Beulich wrote: On 21.09.17 at 13:39, wrote: >> On 21/09/17 13:31, Jan Beulich wrote: >> On 21.09.17 at 09:53, wrote: On 21/09/17 08:15, Jan Beulich wrote: On 21.09.17 at 06:35, wrote: >> On 20/09/17 17:35, Jan Beulich wrote: >> On 20.09.1

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-21 Thread Jan Beulich
>>> On 21.09.17 at 13:39, wrote: > On 21/09/17 13:31, Jan Beulich wrote: > On 21.09.17 at 09:53, wrote: >>> On 21/09/17 08:15, Jan Beulich wrote: >>> On 21.09.17 at 06:35, wrote: > On 20/09/17 17:35, Jan Beulich wrote: > On 20.09.17 at 14:44, wrote: >>> On 20/09/17 13:48

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-21 Thread Juergen Gross
On 21/09/17 13:31, Jan Beulich wrote: On 21.09.17 at 09:53, wrote: >> On 21/09/17 08:15, Jan Beulich wrote: >> On 21.09.17 at 06:35, wrote: On 20/09/17 17:35, Jan Beulich wrote: On 20.09.17 at 14:44, wrote: >> On 20/09/17 13:48, Jan Beulich wrote: >> On 20.09.1

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-21 Thread Jan Beulich
>>> On 21.09.17 at 09:53, wrote: > On 21/09/17 08:15, Jan Beulich wrote: > On 21.09.17 at 06:35, wrote: >>> On 20/09/17 17:35, Jan Beulich wrote: >>> On 20.09.17 at 14:44, wrote: > On 20/09/17 13:48, Jan Beulich wrote: > On 20.09.17 at 13:10, wrote: >>> I thought about a

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-21 Thread Juergen Gross
On 21/09/17 08:15, Jan Beulich wrote: On 21.09.17 at 06:35, wrote: >> On 20/09/17 17:35, Jan Beulich wrote: >> On 20.09.17 at 14:44, wrote: On 20/09/17 13:48, Jan Beulich wrote: On 20.09.17 at 13:10, wrote: >> I thought about a cap and TBH I'm not sure which would be s

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Jan Beulich
>>> On 21.09.17 at 06:35, wrote: > On 20/09/17 17:35, Jan Beulich wrote: > On 20.09.17 at 14:44, wrote: >>> On 20/09/17 13:48, Jan Beulich wrote: >>> On 20.09.17 at 13:10, wrote: > I thought about a cap and TBH I'm not sure which would be sane to > apply. The global limits seem w

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Juergen Gross
On 20/09/17 17:35, Jan Beulich wrote: On 20.09.17 at 14:44, wrote: >> On 20/09/17 13:48, Jan Beulich wrote: >> On 20.09.17 at 13:10, wrote: On 20/09/17 12:34, Jan Beulich wrote: On 20.09.17 at 08:34, wrote: >> --- a/xen/common/compat/grant_table.c >> +++ b/xen/comm

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Jan Beulich
>>> On 20.09.17 at 14:44, wrote: > On 20/09/17 13:48, Jan Beulich wrote: > On 20.09.17 at 13:10, wrote: >>> On 20/09/17 12:34, Jan Beulich wrote: >>> On 20.09.17 at 08:34, wrote: > --- a/xen/common/compat/grant_table.c > +++ b/xen/common/compat/grant_table.c > @@ -1777,13 +17

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Julien Grall
Hi Jan, On 20/09/17 11:34, Jan Beulich wrote: On 20.09.17 at 08:34, wrote: --- a/xen/include/asm-arm/grant_table.h +++ b/xen/include/asm-arm/grant_table.h @@ -26,8 +26,8 @@ static inline int replace_grant_supported(void) return 1; } -#define gnttab_init_arch(gt)

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Juergen Gross
On 20/09/17 13:48, Jan Beulich wrote: On 20.09.17 at 13:10, wrote: >> On 20/09/17 12:34, Jan Beulich wrote: >> On 20.09.17 at 08:34, wrote: --- a/xen/common/compat/grant_table.c +++ b/xen/common/compat/grant_table.c @@ -1777,13 +1784,15 @@ active_alloc_failed:

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Jan Beulich
>>> On 20.09.17 at 13:10, wrote: > On 20/09/17 12:34, Jan Beulich wrote: > On 20.09.17 at 08:34, wrote: >>> --- a/xen/common/compat/grant_table.c >>> +++ b/xen/common/compat/grant_table.c >>> @@ -157,21 +157,14 @@ int compat_grant_table_op(unsigned int cmd, >>> unsigned int m

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Juergen Gross
On 20/09/17 12:34, Jan Beulich wrote: On 20.09.17 at 08:34, wrote: >> --- a/xen/common/compat/grant_table.c >> +++ b/xen/common/compat/grant_table.c >> @@ -157,21 +157,14 @@ int compat_grant_table_op(unsigned int cmd, >> unsigned int max_frame_list_size_in_page = >>

Re: [Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-20 Thread Jan Beulich
>>> On 20.09.17 at 08:34, wrote: > --- a/xen/common/compat/grant_table.c > +++ b/xen/common/compat/grant_table.c > @@ -157,21 +157,14 @@ int compat_grant_table_op(unsigned int cmd, > unsigned int max_frame_list_size_in_page = > (COMPAT_ARG_XLAT_SIZE - sizeof(*

[Xen-devel] [PATCH v8 13/15] xen: make grant resource limits per domain

2017-09-19 Thread Juergen Gross
Instead of using the same global resource limits of grant tables (max. number of grant frames, max. number of maptrack frames) for all domains make these limits per domain. Set those per-domain limits in grant_table_set_limits(). Signed-off-by: Juergen Gross Reviewed-by: Paul Durrant --- V6: - s