Re: [PATCH v5] xen: simplify bitmap_to_xenctl_bitmap for little endian

2025-04-05 Thread Andrew Cooper
On 01/04/2025 11:09 am, Jan Beulich wrote: > --- a/xen/common/bitmap.c > +++ b/xen/common/bitmap.c > @@ -359,12 +343,11 @@ static void bitmap_byte_to_long(unsigned > > #elif defined(__LITTLE_ENDIAN) > > -static void bitmap_long_to_byte(uint8_t *bp, const unsigned long *lp, > -

Re: [PATCH v5] xen: simplify bitmap_to_xenctl_bitmap for little endian

2025-04-05 Thread Jan Beulich
On 02.04.2025 11:21, Andrew Cooper wrote: > On 01/04/2025 11:09 am, Jan Beulich wrote: >> --- a/xen/common/bitmap.c >> +++ b/xen/common/bitmap.c >> @@ -359,12 +343,11 @@ static void bitmap_byte_to_long(unsigned >> >> #elif defined(__LITTLE_ENDIAN) >> >> -static void bitmap_long_to_byte(uint8_t

Re: [PATCH v5] xen: simplify bitmap_to_xenctl_bitmap for little endian

2025-04-02 Thread Andrew Cooper
On 02/04/2025 10:30 am, Jan Beulich wrote: > On 02.04.2025 11:21, Andrew Cooper wrote: >> On 01/04/2025 11:09 am, Jan Beulich wrote: >>> --- a/xen/common/bitmap.c >>> +++ b/xen/common/bitmap.c >>> @@ -359,12 +343,11 @@ static void bitmap_byte_to_long(unsigned >>> >>> #elif defined(__LITTLE_ENDIA

Re: [PATCH v5] xen: simplify bitmap_to_xenctl_bitmap for little endian

2025-04-02 Thread Jan Beulich
On 02.04.2025 11:36, Andrew Cooper wrote: > On 02/04/2025 10:30 am, Jan Beulich wrote: >> On 02.04.2025 11:21, Andrew Cooper wrote: >>> On 01/04/2025 11:09 am, Jan Beulich wrote: --- a/xen/common/bitmap.c +++ b/xen/common/bitmap.c @@ -359,12 +343,11 @@ static void bitmap_byte_to_long

Re: [PATCH v5] xen: simplify bitmap_to_xenctl_bitmap for little endian

2025-04-02 Thread Jan Beulich
On 02.04.2025 02:14, Stefano Stabellini wrote: > On Tue, 1 Apr 2025, Jan Beulich wrote: >> From: Stefano Stabellini >> >> The little endian implementation of bitmap_to_xenctl_bitmap leads to >> unnecessary xmallocs and xfrees. Given that Xen only supports little >> endian architectures, it is wort

Re: [PATCH v5] xen: simplify bitmap_to_xenctl_bitmap for little endian

2025-04-01 Thread Stefano Stabellini
On Tue, 1 Apr 2025, Jan Beulich wrote: > From: Stefano Stabellini > > The little endian implementation of bitmap_to_xenctl_bitmap leads to > unnecessary xmallocs and xfrees. Given that Xen only supports little > endian architectures, it is worth optimizing. > > This patch removes the need for th

[PATCH v5] xen: simplify bitmap_to_xenctl_bitmap for little endian

2025-04-01 Thread Jan Beulich
From: Stefano Stabellini The little endian implementation of bitmap_to_xenctl_bitmap leads to unnecessary xmallocs and xfrees. Given that Xen only supports little endian architectures, it is worth optimizing. This patch removes the need for the xmalloc on little endian architectures. Remove cla