Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2024-02-29 Thread Jan Beulich
On 29.02.2024 18:04, Jürgen Groß wrote: > On 29.02.24 17:54, Jan Beulich wrote: >> On 29.02.2024 17:45, Juergen Gross wrote: >>> On 29.02.24 17:31, Jan Beulich wrote: On 29.02.2024 17:29, Jürgen Groß wrote: > On 29.02.24 16:46, Jan Beulich wrote: >> On 12.12.2023 10:47, Juergen Gross

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2024-02-29 Thread Jürgen Groß
On 29.02.24 17:54, Jan Beulich wrote: On 29.02.2024 17:45, Juergen Gross wrote: On 29.02.24 17:31, Jan Beulich wrote: On 29.02.2024 17:29, Jürgen Groß wrote: On 29.02.24 16:46, Jan Beulich wrote: On 12.12.2023 10:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2024-02-29 Thread Jan Beulich
On 29.02.2024 17:45, Juergen Gross wrote: > On 29.02.24 17:31, Jan Beulich wrote: >> On 29.02.2024 17:29, Jürgen Groß wrote: >>> On 29.02.24 16:46, Jan Beulich wrote: On 12.12.2023 10:47, Juergen Gross wrote: > Allow 16 bits per cpu number, which is the limit imposed by >

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2024-02-29 Thread Juergen Gross
On 29.02.24 17:31, Jan Beulich wrote: On 29.02.2024 17:29, Jürgen Groß wrote: On 29.02.24 16:46, Jan Beulich wrote: On 12.12.2023 10:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2024-02-29 Thread Jan Beulich
On 29.02.2024 17:29, Jürgen Groß wrote: > On 29.02.24 16:46, Jan Beulich wrote: >> On 12.12.2023 10:47, Juergen Gross wrote: >>> Allow 16 bits per cpu number, which is the limit imposed by >>> spinlock_tickets_t. >>> >>> This will allow up to 65535 cpus, while increasing only the size of >>>

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2024-02-29 Thread Jürgen Groß
On 29.02.24 16:46, Jan Beulich wrote: On 12.12.2023 10:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size of recursive spinlocks in debug builds from 8 to 12 bytes. I think

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2024-02-29 Thread Jan Beulich
On 12.12.2023 10:47, Juergen Gross wrote: > Allow 16 bits per cpu number, which is the limit imposed by > spinlock_tickets_t. > > This will allow up to 65535 cpus, while increasing only the size of > recursive spinlocks in debug builds from 8 to 12 bytes. I think we want to be more conservative

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Julien Grall
On 12/12/2023 13:08, Juergen Gross wrote: On 12.12.23 13:39, Julien Grall wrote: Hi, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size of recursive

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Juergen Gross
On 12.12.23 13:39, Julien Grall wrote: Hi, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size of recursive spinlocks in debug builds from 8 to 12 bytes.

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Julien Grall
Hi, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size of recursive spinlocks in debug builds from 8 to 12 bytes. Signed-off-by: Juergen Gross ---

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Julien Grall
Hi, On 12/12/2023 12:11, Juergen Gross wrote: On 12.12.23 12:40, Julien Grall wrote: Hi Juergen, On 12/12/2023 11:09, Juergen Gross wrote: On 12.12.23 11:10, Julien Grall wrote: Hi Juergen, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Juergen Gross
On 12.12.23 12:40, Julien Grall wrote: Hi Juergen, On 12/12/2023 11:09, Juergen Gross wrote: On 12.12.23 11:10, Julien Grall wrote: Hi Juergen, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Julien Grall
Hi Juergen, On 12/12/2023 11:09, Juergen Gross wrote: On 12.12.23 11:10, Julien Grall wrote: Hi Juergen, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Juergen Gross
On 12.12.23 11:10, Julien Grall wrote: Hi Juergen, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size of recursive spinlocks in debug builds from 8 to 12

Re: [PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Julien Grall
Hi Juergen, On 12/12/2023 09:47, Juergen Gross wrote: Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size of recursive spinlocks in debug builds from 8 to 12 bytes. Looking at arch/Kconfig, it looks

[PATCH v4 12/12] xen/spinlock: support higher number of cpus

2023-12-12 Thread Juergen Gross
Allow 16 bits per cpu number, which is the limit imposed by spinlock_tickets_t. This will allow up to 65535 cpus, while increasing only the size of recursive spinlocks in debug builds from 8 to 12 bytes. Signed-off-by: Juergen Gross --- xen/common/spinlock.c | 1 +