On 2023-11-24 09:06, Jan Beulich wrote:
On 23.11.2023 08:37, Nicola Vetrini wrote:
The definitions of ffs{l}? violate Rule 10.1, by using the well-known
pattern (x & -x); its usage is wrapped by the ISOLATE_LSB macro.

No functional change.

Signed-off-by: Nicola Vetrini <nicola.vetr...@bugseng.com>
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>

This patch failed my pre-push build test; apparently xen/macros.h needs
including explicitly.

Jan

Yes, indeed. I must have dropped that change when experimenting with single evaluation and then never reinstated it.
--
Nicola Vetrini, BSc
Software Engineer, BUGSENG srl (https://bugseng.com)

Reply via email to