qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 0/2] chardev/char-mux: tweak mux bitset operations


From: Roman Penyaev
Subject: Re: [PATCH v2 0/2] chardev/char-mux: tweak mux bitset operations
Date: Mon, 25 Nov 2024 14:46:21 +0100

Hi Paolo,

Do I need to resend the fixed version, or if it is queued to what
exact repo or branch? I'm a bit confused. Thanks.

--
Roman

On Wed, Nov 20, 2024 at 3:44 PM Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> Queued, with just one change:
>
> > Also add a static compile check that MAX_MUX never bigger than
> > `sizeof(d->mux_bitset)`.
>
> This needs to be multiplied by CHAR_BIT.
>
> Paolo
>
> On 11/20/24 08:53, Roman Penyaev wrote:
> > Patchset tweaks bitset operations by changing a constant to unsigned
> > long, introduces a static compile check and simplifies bitset operations.
> >
> > v1..v2:
> >
> > Rebase on latest master, incorporate review tags.
> >
> > Roman Penyaev (2):
> >    chardev/char-mux: shift unsigned long to avoid 32-bit overflow
> >    chardev/char-mux: make boolean bit check instead of find_next_bit()
> >
> >   chardev/char-mux.c | 15 +++++++--------
> >   1 file changed, 7 insertions(+), 8 deletions(-)
> >
> >
> > Signed-off-by: Roman Penyaev <r.peniaev@gmail.com>
> > Reviewed-by: "Marc-André Lureau" <marcandre.lureau@redhat.com>
> > Cc: Peter Maydell <peter.maydell@linaro.org>
> > Reviewed-by: Clément Mathieu--Drif <clement.mathieu--drif@eviden.com>
> > Cc: qemu-devel@nongnu.org
> >
> >
>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]