On 02.04.2025 02:14, Stefano Stabellini wrote: > On Tue, 1 Apr 2025, Jan Beulich wrote: >> From: Stefano Stabellini <stefano.stabell...@amd.com> >> >> 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 clamp_last_byte as it is only called once and only needs to >> modify one byte. Inline it. >> >> Signed-off-by: Stefano Stabellini <stefano.stabell...@amd.com> >> Signed-off-by: Jan Beulich <jbeul...@suse.com> >> --- >> v5: Fix IS_ENABLED() use. Keep more code common. Move comment. >> Convert LE bitmap_long_to_byte() to just a declaration. > > Thanks Jan, I looked at your version carefully and it looks correct to > me. I could give my reviewed-by but it looks weird given that I am also > the first author.
./MAINTAINERS has a section specifically on this situation, starting with "In the case where two people collaborate on a patch, ..." > Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Thanks. Jan
