* [PATCH v2 1/4] bitmap: introduce bitmap_weighted_xor()
2026-03-02 1:11 [PATCH v2 0/4] ice: use better bitmap API Yury Norov
@ 2026-03-02 1:11 ` Yury Norov
2026-03-02 7:12 ` [Intel-wired-lan] " Loktionov, Aleksandr
2026-03-02 1:11 ` [PATCH v2 2/4] ice: use bitmap_weighted_xor() in ice_find_free_recp_res_idx() Yury Norov
` (3 subsequent siblings)
4 siblings, 1 reply; 13+ messages in thread
From: Yury Norov @ 2026-03-02 1:11 UTC (permalink / raw)
To: Tony Nguyen, David S. Miller, Thomas Hellström, Andrew Lunn,
Andrew Morton, David Airlie, Eric Dumazet, Jakub Kicinski,
Matthew Brost, Paolo Abeni, Przemek Kitszel, Rodrigo Vivi,
Simona Vetter, Yury Norov, Rasmus Villemoes, dri-devel, intel-xe,
linux-kernel, netdev, intel-wired-lan
Cc: Yury Norov, Simon Horman, David Laight
The function helps to XOR bitmaps and calculate Hamming weight of
the result in one pass.
Signed-off-by: Yury Norov <ynorov@nvidia.com>
---
include/linux/bitmap.h | 15 +++++++++++++++
lib/bitmap.c | 7 +++++++
2 files changed, 22 insertions(+)
diff --git a/include/linux/bitmap.h b/include/linux/bitmap.h
index b0395e4ccf90..874f744870ef 100644
--- a/include/linux/bitmap.h
+++ b/include/linux/bitmap.h
@@ -46,6 +46,7 @@ struct device;
* bitmap_and(dst, src1, src2, nbits) *dst = *src1 & *src2
* bitmap_or(dst, src1, src2, nbits) *dst = *src1 | *src2
* bitmap_weighted_or(dst, src1, src2, nbits) *dst = *src1 | *src2. Returns Hamming Weight of dst
+ * bitmap_weighted_xor(dst, src1, src2, nbits) *dst = *src1 ^ *src2. Returns Hamming Weight of dst
* bitmap_xor(dst, src1, src2, nbits) *dst = *src1 ^ *src2
* bitmap_andnot(dst, src1, src2, nbits) *dst = *src1 & ~(*src2)
* bitmap_complement(dst, src, nbits) *dst = ~(*src)
@@ -168,6 +169,8 @@ void __bitmap_or(unsigned long *dst, const unsigned long *bitmap1,
const unsigned long *bitmap2, unsigned int nbits);
unsigned int __bitmap_weighted_or(unsigned long *dst, const unsigned long *bitmap1,
const unsigned long *bitmap2, unsigned int nbits);
+unsigned int __bitmap_weighted_xor(unsigned long *dst, const unsigned long *bitmap1,
+ const unsigned long *bitmap2, unsigned int nbits);
void __bitmap_xor(unsigned long *dst, const unsigned long *bitmap1,
const unsigned long *bitmap2, unsigned int nbits);
bool __bitmap_andnot(unsigned long *dst, const unsigned long *bitmap1,
@@ -352,6 +355,18 @@ unsigned int bitmap_weighted_or(unsigned long *dst, const unsigned long *src1,
}
}
+static __always_inline
+unsigned int bitmap_weighted_xor(unsigned long *dst, const unsigned long *src1,
+ const unsigned long *src2, unsigned int nbits)
+{
+ if (small_const_nbits(nbits)) {
+ *dst = *src1 ^ *src2;
+ return hweight_long(*dst & BITMAP_LAST_WORD_MASK(nbits));
+ } else {
+ return __bitmap_weighted_xor(dst, src1, src2, nbits);
+ }
+}
+
static __always_inline
void bitmap_xor(unsigned long *dst, const unsigned long *src1,
const unsigned long *src2, unsigned int nbits)
diff --git a/lib/bitmap.c b/lib/bitmap.c
index 9dc526507875..a2bcb5b1fe99 100644
--- a/lib/bitmap.c
+++ b/lib/bitmap.c
@@ -361,6 +361,13 @@ unsigned int __bitmap_weighted_or(unsigned long *dst, const unsigned long *bitma
return BITMAP_WEIGHT(({dst[idx] = bitmap1[idx] | bitmap2[idx]; dst[idx]; }), bits);
}
+unsigned int __bitmap_weighted_xor(unsigned long *dst, const unsigned long *bitmap1,
+ const unsigned long *bitmap2, unsigned int bits)
+{
+ return BITMAP_WEIGHT(({dst[idx] = bitmap1[idx] ^ bitmap2[idx]; dst[idx]; }), bits);
+}
+EXPORT_SYMBOL(__bitmap_weighted_xor);
+
void __bitmap_set(unsigned long *map, unsigned int start, int len)
{
unsigned long *p = map + BIT_WORD(start);
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* RE: [Intel-wired-lan] [PATCH v2 1/4] bitmap: introduce bitmap_weighted_xor()
2026-03-02 1:11 ` [PATCH v2 1/4] bitmap: introduce bitmap_weighted_xor() Yury Norov
@ 2026-03-02 7:12 ` Loktionov, Aleksandr
0 siblings, 0 replies; 13+ messages in thread
From: Loktionov, Aleksandr @ 2026-03-02 7:12 UTC (permalink / raw)
To: Yury Norov, Nguyen, Anthony L, David S. Miller,
Thomas Hellström, Andrew Lunn, Andrew Morton, David Airlie,
Eric Dumazet, Jakub Kicinski, Brost, Matthew, Paolo Abeni,
Kitszel, Przemyslaw, Vivi, Rodrigo, Simona Vetter, Yury Norov,
Rasmus Villemoes, dri-devel@lists.freedesktop.org,
intel-xe@lists.freedesktop.org, linux-kernel@vger.kernel.org,
netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org
Cc: Simon Horman, David Laight
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf
> Of Yury Norov via Intel-wired-lan
> Sent: Monday, March 2, 2026 2:12 AM
> To: Nguyen, Anthony L <anthony.l.nguyen@intel.com>; David S. Miller
> <davem@davemloft.net>; Thomas Hellström
> <thomas.hellstrom@linux.intel.com>; Andrew Lunn
> <andrew+netdev@lunn.ch>; Andrew Morton <akpm@linux-foundation.org>;
> David Airlie <airlied@gmail.com>; Eric Dumazet <edumazet@google.com>;
> Jakub Kicinski <kuba@kernel.org>; Brost, Matthew
> <matthew.brost@intel.com>; Paolo Abeni <pabeni@redhat.com>; Kitszel,
> Przemyslaw <przemyslaw.kitszel@intel.com>; Vivi, Rodrigo
> <rodrigo.vivi@intel.com>; Simona Vetter <simona@ffwll.ch>; Yury Norov
> <yury.norov@gmail.com>; Rasmus Villemoes <linux@rasmusvillemoes.dk>;
> dri-devel@lists.freedesktop.org; intel-xe@lists.freedesktop.org;
> linux-kernel@vger.kernel.org; netdev@vger.kernel.org; intel-wired-
> lan@lists.osuosl.org
> Cc: Yury Norov <ynorov@nvidia.com>; Simon Horman <horms@kernel.org>;
> David Laight <david.laight.linux@gmail.com>
> Subject: [Intel-wired-lan] [PATCH v2 1/4] bitmap: introduce
> bitmap_weighted_xor()
>
> The function helps to XOR bitmaps and calculate Hamming weight of the
> result in one pass.
>
> Signed-off-by: Yury Norov <ynorov@nvidia.com>
> ---
> include/linux/bitmap.h | 15 +++++++++++++++
> lib/bitmap.c | 7 +++++++
> 2 files changed, 22 insertions(+)
>
> diff --git a/include/linux/bitmap.h b/include/linux/bitmap.h index
> b0395e4ccf90..874f744870ef 100644
> --- a/include/linux/bitmap.h
> +++ b/include/linux/bitmap.h
> @@ -46,6 +46,7 @@ struct device;
> * bitmap_and(dst, src1, src2, nbits) *dst = *src1 & *src2
> * bitmap_or(dst, src1, src2, nbits) *dst = *src1 | *src2
> * bitmap_weighted_or(dst, src1, src2, nbits) *dst = *src1 | *src2.
> Returns Hamming Weight of dst
> + * bitmap_weighted_xor(dst, src1, src2, nbits) *dst = *src1 ^
> *src2. Returns Hamming Weight of dst
> * bitmap_xor(dst, src1, src2, nbits) *dst = *src1 ^ *src2
> * bitmap_andnot(dst, src1, src2, nbits) *dst = *src1 &
> ~(*src2)
> * bitmap_complement(dst, src, nbits) *dst = ~(*src)
> @@ -168,6 +169,8 @@ void __bitmap_or(unsigned long *dst, const
> unsigned long *bitmap1,
> const unsigned long *bitmap2, unsigned int nbits);
> unsigned int __bitmap_weighted_or(unsigned long *dst, const unsigned
> long *bitmap1,
> const unsigned long *bitmap2, unsigned
> int nbits);
> +unsigned int __bitmap_weighted_xor(unsigned long *dst, const unsigned
> long *bitmap1,
> + const unsigned long *bitmap2, unsigned
> int nbits);
> void __bitmap_xor(unsigned long *dst, const unsigned long *bitmap1,
> const unsigned long *bitmap2, unsigned int nbits);
> bool __bitmap_andnot(unsigned long *dst, const unsigned long *bitmap1,
> @@ -352,6 +355,18 @@ unsigned int bitmap_weighted_or(unsigned long
> *dst, const unsigned long *src1,
> }
> }
>
> +static __always_inline
> +unsigned int bitmap_weighted_xor(unsigned long *dst, const unsigned
> long *src1,
> + const unsigned long *src2, unsigned int
> nbits) {
> + if (small_const_nbits(nbits)) {
> + *dst = *src1 ^ *src2;
> + return hweight_long(*dst &
> BITMAP_LAST_WORD_MASK(nbits));
> + } else {
> + return __bitmap_weighted_xor(dst, src1, src2, nbits);
> + }
> +}
> +
> static __always_inline
> void bitmap_xor(unsigned long *dst, const unsigned long *src1,
> const unsigned long *src2, unsigned int nbits) diff --
> git a/lib/bitmap.c b/lib/bitmap.c index 9dc526507875..a2bcb5b1fe99
> 100644
> --- a/lib/bitmap.c
> +++ b/lib/bitmap.c
> @@ -361,6 +361,13 @@ unsigned int __bitmap_weighted_or(unsigned long
> *dst, const unsigned long *bitma
> return BITMAP_WEIGHT(({dst[idx] = bitmap1[idx] | bitmap2[idx];
> dst[idx]; }), bits); }
>
> +unsigned int __bitmap_weighted_xor(unsigned long *dst, const unsigned
> long *bitmap1,
> + const unsigned long *bitmap2, unsigned
> int bits) {
> + return BITMAP_WEIGHT(({dst[idx] = bitmap1[idx] ^ bitmap2[idx];
> +dst[idx]; }), bits); } EXPORT_SYMBOL(__bitmap_weighted_xor);
> +
> void __bitmap_set(unsigned long *map, unsigned int start, int len) {
> unsigned long *p = map + BIT_WORD(start);
> --
> 2.43.0
Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH v2 2/4] ice: use bitmap_weighted_xor() in ice_find_free_recp_res_idx()
2026-03-02 1:11 [PATCH v2 0/4] ice: use better bitmap API Yury Norov
2026-03-02 1:11 ` [PATCH v2 1/4] bitmap: introduce bitmap_weighted_xor() Yury Norov
@ 2026-03-02 1:11 ` Yury Norov
2026-03-02 7:13 ` [Intel-wired-lan] " Loktionov, Aleksandr
2026-03-02 1:11 ` [PATCH v2 3/4] ice: use bitmap_empty() in ice_vf_has_no_qs_ena Yury Norov
` (2 subsequent siblings)
4 siblings, 1 reply; 13+ messages in thread
From: Yury Norov @ 2026-03-02 1:11 UTC (permalink / raw)
To: Tony Nguyen, David S. Miller, Thomas Hellström, Andrew Lunn,
Andrew Morton, David Airlie, Eric Dumazet, Jakub Kicinski,
Matthew Brost, Paolo Abeni, Przemek Kitszel, Rodrigo Vivi,
Simona Vetter, Yury Norov, Rasmus Villemoes, dri-devel, intel-xe,
linux-kernel, netdev, intel-wired-lan
Cc: Yury Norov, Simon Horman, David Laight
Use the right helper and save one bitmaps traverse.
Signed-off-by: Yury Norov <ynorov@nvidia.com>
---
drivers/net/ethernet/intel/ice/ice_switch.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/intel/ice/ice_switch.c b/drivers/net/ethernet/intel/ice/ice_switch.c
index bb0f990fa2c6..6a5875bd9c6b 100644
--- a/drivers/net/ethernet/intel/ice/ice_switch.c
+++ b/drivers/net/ethernet/intel/ice/ice_switch.c
@@ -4984,10 +4984,8 @@ ice_find_free_recp_res_idx(struct ice_hw *hw, const unsigned long *profiles,
hw->switch_info->recp_list[bit].res_idxs,
ICE_MAX_FV_WORDS);
- bitmap_xor(free_idx, used_idx, possible_idx, ICE_MAX_FV_WORDS);
-
/* return number of free indexes */
- return (u16)bitmap_weight(free_idx, ICE_MAX_FV_WORDS);
+ return (u16)bitmap_weighted_xor(free_idx, used_idx, possible_idx, ICE_MAX_FV_WORDS);
}
/**
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* RE: [Intel-wired-lan] [PATCH v2 2/4] ice: use bitmap_weighted_xor() in ice_find_free_recp_res_idx()
2026-03-02 1:11 ` [PATCH v2 2/4] ice: use bitmap_weighted_xor() in ice_find_free_recp_res_idx() Yury Norov
@ 2026-03-02 7:13 ` Loktionov, Aleksandr
0 siblings, 0 replies; 13+ messages in thread
From: Loktionov, Aleksandr @ 2026-03-02 7:13 UTC (permalink / raw)
To: Yury Norov, Nguyen, Anthony L, David S. Miller,
Thomas Hellström, Andrew Lunn, Andrew Morton, David Airlie,
Eric Dumazet, Jakub Kicinski, Brost, Matthew, Paolo Abeni,
Kitszel, Przemyslaw, Vivi, Rodrigo, Simona Vetter, Yury Norov,
Rasmus Villemoes, dri-devel@lists.freedesktop.org,
intel-xe@lists.freedesktop.org, linux-kernel@vger.kernel.org,
netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org
Cc: Simon Horman, David Laight
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf
> Of Yury Norov via Intel-wired-lan
> Sent: Monday, March 2, 2026 2:12 AM
> To: Nguyen, Anthony L <anthony.l.nguyen@intel.com>; David S. Miller
> <davem@davemloft.net>; Thomas Hellström
> <thomas.hellstrom@linux.intel.com>; Andrew Lunn
> <andrew+netdev@lunn.ch>; Andrew Morton <akpm@linux-foundation.org>;
> David Airlie <airlied@gmail.com>; Eric Dumazet <edumazet@google.com>;
> Jakub Kicinski <kuba@kernel.org>; Brost, Matthew
> <matthew.brost@intel.com>; Paolo Abeni <pabeni@redhat.com>; Kitszel,
> Przemyslaw <przemyslaw.kitszel@intel.com>; Vivi, Rodrigo
> <rodrigo.vivi@intel.com>; Simona Vetter <simona@ffwll.ch>; Yury Norov
> <yury.norov@gmail.com>; Rasmus Villemoes <linux@rasmusvillemoes.dk>;
> dri-devel@lists.freedesktop.org; intel-xe@lists.freedesktop.org;
> linux-kernel@vger.kernel.org; netdev@vger.kernel.org; intel-wired-
> lan@lists.osuosl.org
> Cc: Yury Norov <ynorov@nvidia.com>; Simon Horman <horms@kernel.org>;
> David Laight <david.laight.linux@gmail.com>
> Subject: [Intel-wired-lan] [PATCH v2 2/4] ice: use
> bitmap_weighted_xor() in ice_find_free_recp_res_idx()
>
> Use the right helper and save one bitmaps traverse.
>
> Signed-off-by: Yury Norov <ynorov@nvidia.com>
> ---
> drivers/net/ethernet/intel/ice/ice_switch.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/ice/ice_switch.c
> b/drivers/net/ethernet/intel/ice/ice_switch.c
> index bb0f990fa2c6..6a5875bd9c6b 100644
> --- a/drivers/net/ethernet/intel/ice/ice_switch.c
> +++ b/drivers/net/ethernet/intel/ice/ice_switch.c
> @@ -4984,10 +4984,8 @@ ice_find_free_recp_res_idx(struct ice_hw *hw,
> const unsigned long *profiles,
> hw->switch_info->recp_list[bit].res_idxs,
> ICE_MAX_FV_WORDS);
>
> - bitmap_xor(free_idx, used_idx, possible_idx, ICE_MAX_FV_WORDS);
> -
> /* return number of free indexes */
> - return (u16)bitmap_weight(free_idx, ICE_MAX_FV_WORDS);
> + return (u16)bitmap_weighted_xor(free_idx, used_idx,
> possible_idx, ICE_MAX_FV_WORDS);
> }
>
> /**
> --
> 2.43.0
Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH v2 3/4] ice: use bitmap_empty() in ice_vf_has_no_qs_ena
2026-03-02 1:11 [PATCH v2 0/4] ice: use better bitmap API Yury Norov
2026-03-02 1:11 ` [PATCH v2 1/4] bitmap: introduce bitmap_weighted_xor() Yury Norov
2026-03-02 1:11 ` [PATCH v2 2/4] ice: use bitmap_weighted_xor() in ice_find_free_recp_res_idx() Yury Norov
@ 2026-03-02 1:11 ` Yury Norov
2026-03-02 7:14 ` [Intel-wired-lan] " Loktionov, Aleksandr
2026-03-02 1:11 ` [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or() Yury Norov
2026-03-02 23:01 ` [PATCH v2 0/4] ice: use better bitmap API Jacob Keller
4 siblings, 1 reply; 13+ messages in thread
From: Yury Norov @ 2026-03-02 1:11 UTC (permalink / raw)
To: Tony Nguyen, David S. Miller, Thomas Hellström, Andrew Lunn,
Andrew Morton, David Airlie, Eric Dumazet, Jakub Kicinski,
Matthew Brost, Paolo Abeni, Przemek Kitszel, Rodrigo Vivi,
Simona Vetter, Yury Norov, Rasmus Villemoes, dri-devel, intel-xe,
linux-kernel, netdev, intel-wired-lan
Cc: Yury Norov, Simon Horman, David Laight
bitmap_empty() is more verbose and efficient, as it stops traversing
{r,t}xq_ena as soon as the 1st set bit found.
Signed-off-by: Yury Norov <ynorov@nvidia.com>
---
drivers/net/ethernet/intel/ice/ice_vf_lib.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/intel/ice/ice_vf_lib.c b/drivers/net/ethernet/intel/ice/ice_vf_lib.c
index c8bc952f05cd..772f6b07340d 100644
--- a/drivers/net/ethernet/intel/ice/ice_vf_lib.c
+++ b/drivers/net/ethernet/intel/ice/ice_vf_lib.c
@@ -1210,8 +1210,8 @@ bool ice_is_vf_trusted(struct ice_vf *vf)
*/
bool ice_vf_has_no_qs_ena(struct ice_vf *vf)
{
- return (!bitmap_weight(vf->rxq_ena, ICE_MAX_RSS_QS_PER_VF) &&
- !bitmap_weight(vf->txq_ena, ICE_MAX_RSS_QS_PER_VF));
+ return bitmap_empty(vf->rxq_ena, ICE_MAX_RSS_QS_PER_VF) &&
+ bitmap_empty(vf->txq_ena, ICE_MAX_RSS_QS_PER_VF);
}
/**
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* RE: [Intel-wired-lan] [PATCH v2 3/4] ice: use bitmap_empty() in ice_vf_has_no_qs_ena
2026-03-02 1:11 ` [PATCH v2 3/4] ice: use bitmap_empty() in ice_vf_has_no_qs_ena Yury Norov
@ 2026-03-02 7:14 ` Loktionov, Aleksandr
0 siblings, 0 replies; 13+ messages in thread
From: Loktionov, Aleksandr @ 2026-03-02 7:14 UTC (permalink / raw)
To: Yury Norov, Nguyen, Anthony L, David S. Miller,
Thomas Hellström, Andrew Lunn, Andrew Morton, David Airlie,
Eric Dumazet, Jakub Kicinski, Brost, Matthew, Paolo Abeni,
Kitszel, Przemyslaw, Vivi, Rodrigo, Simona Vetter, Yury Norov,
Rasmus Villemoes, dri-devel@lists.freedesktop.org,
intel-xe@lists.freedesktop.org, linux-kernel@vger.kernel.org,
netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org
Cc: Simon Horman, David Laight
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf
> Of Yury Norov via Intel-wired-lan
> Sent: Monday, March 2, 2026 2:12 AM
> To: Nguyen, Anthony L <anthony.l.nguyen@intel.com>; David S. Miller
> <davem@davemloft.net>; Thomas Hellström
> <thomas.hellstrom@linux.intel.com>; Andrew Lunn
> <andrew+netdev@lunn.ch>; Andrew Morton <akpm@linux-foundation.org>;
> David Airlie <airlied@gmail.com>; Eric Dumazet <edumazet@google.com>;
> Jakub Kicinski <kuba@kernel.org>; Brost, Matthew
> <matthew.brost@intel.com>; Paolo Abeni <pabeni@redhat.com>; Kitszel,
> Przemyslaw <przemyslaw.kitszel@intel.com>; Vivi, Rodrigo
> <rodrigo.vivi@intel.com>; Simona Vetter <simona@ffwll.ch>; Yury Norov
> <yury.norov@gmail.com>; Rasmus Villemoes <linux@rasmusvillemoes.dk>;
> dri-devel@lists.freedesktop.org; intel-xe@lists.freedesktop.org;
> linux-kernel@vger.kernel.org; netdev@vger.kernel.org; intel-wired-
> lan@lists.osuosl.org
> Cc: Yury Norov <ynorov@nvidia.com>; Simon Horman <horms@kernel.org>;
> David Laight <david.laight.linux@gmail.com>
> Subject: [Intel-wired-lan] [PATCH v2 3/4] ice: use bitmap_empty() in
> ice_vf_has_no_qs_ena
>
> bitmap_empty() is more verbose and efficient, as it stops traversing
> {r,t}xq_ena as soon as the 1st set bit found.
>
> Signed-off-by: Yury Norov <ynorov@nvidia.com>
> ---
> drivers/net/ethernet/intel/ice/ice_vf_lib.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/ice/ice_vf_lib.c
> b/drivers/net/ethernet/intel/ice/ice_vf_lib.c
> index c8bc952f05cd..772f6b07340d 100644
> --- a/drivers/net/ethernet/intel/ice/ice_vf_lib.c
> +++ b/drivers/net/ethernet/intel/ice/ice_vf_lib.c
> @@ -1210,8 +1210,8 @@ bool ice_is_vf_trusted(struct ice_vf *vf)
> */
> bool ice_vf_has_no_qs_ena(struct ice_vf *vf) {
> - return (!bitmap_weight(vf->rxq_ena, ICE_MAX_RSS_QS_PER_VF) &&
> - !bitmap_weight(vf->txq_ena, ICE_MAX_RSS_QS_PER_VF));
> + return bitmap_empty(vf->rxq_ena, ICE_MAX_RSS_QS_PER_VF) &&
> + bitmap_empty(vf->txq_ena, ICE_MAX_RSS_QS_PER_VF);
> }
>
> /**
> --
> 2.43.0
Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or()
2026-03-02 1:11 [PATCH v2 0/4] ice: use better bitmap API Yury Norov
` (2 preceding siblings ...)
2026-03-02 1:11 ` [PATCH v2 3/4] ice: use bitmap_empty() in ice_vf_has_no_qs_ena Yury Norov
@ 2026-03-02 1:11 ` Yury Norov
2026-03-02 7:14 ` [Intel-wired-lan] " Loktionov, Aleksandr
` (2 more replies)
2026-03-02 23:01 ` [PATCH v2 0/4] ice: use better bitmap API Jacob Keller
4 siblings, 3 replies; 13+ messages in thread
From: Yury Norov @ 2026-03-02 1:11 UTC (permalink / raw)
To: Tony Nguyen, David S. Miller, Thomas Hellström, Andrew Lunn,
Andrew Morton, David Airlie, Eric Dumazet, Jakub Kicinski,
Matthew Brost, Paolo Abeni, Przemek Kitszel, Rodrigo Vivi,
Simona Vetter, Yury Norov, Rasmus Villemoes, dri-devel, intel-xe,
linux-kernel, netdev, intel-wired-lan
Cc: Yury Norov, Simon Horman, David Laight
The function calls bitmap_or() immediately followed by bitmap_weight().
Switch to using the dedicated bitmap_weighted_or() and save one bitmap
traverse.
Signed-off-by: Yury Norov <ynorov@nvidia.com>
---
drivers/gpu/drm/xe/xe_pagefault.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/xe/xe_pagefault.c b/drivers/gpu/drm/xe/xe_pagefault.c
index 6bee53d6ffc3..c4ce3cfe2164 100644
--- a/drivers/gpu/drm/xe/xe_pagefault.c
+++ b/drivers/gpu/drm/xe/xe_pagefault.c
@@ -285,10 +285,9 @@ static int xe_pagefault_queue_init(struct xe_device *xe,
xe_dss_mask_t all_dss;
int num_dss, num_eus;
- bitmap_or(all_dss, gt->fuse_topo.g_dss_mask,
+ num_dss = bitmap_weighted_or(all_dss, gt->fuse_topo.g_dss_mask,
gt->fuse_topo.c_dss_mask, XE_MAX_DSS_FUSE_BITS);
- num_dss = bitmap_weight(all_dss, XE_MAX_DSS_FUSE_BITS);
num_eus = bitmap_weight(gt->fuse_topo.eu_mask_per_dss,
XE_MAX_EU_FUSE_BITS) * num_dss;
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* RE: [Intel-wired-lan] [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or()
2026-03-02 1:11 ` [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or() Yury Norov
@ 2026-03-02 7:14 ` Loktionov, Aleksandr
2026-03-02 19:49 ` Matthew Brost
2026-03-04 11:43 ` Simon Horman
2 siblings, 0 replies; 13+ messages in thread
From: Loktionov, Aleksandr @ 2026-03-02 7:14 UTC (permalink / raw)
To: Yury Norov, Nguyen, Anthony L, David S. Miller,
Thomas Hellström, Andrew Lunn, Andrew Morton, David Airlie,
Eric Dumazet, Jakub Kicinski, Brost, Matthew, Paolo Abeni,
Kitszel, Przemyslaw, Vivi, Rodrigo, Simona Vetter, Yury Norov,
Rasmus Villemoes, dri-devel@lists.freedesktop.org,
intel-xe@lists.freedesktop.org, linux-kernel@vger.kernel.org,
netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org
Cc: Simon Horman, David Laight
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf
> Of Yury Norov via Intel-wired-lan
> Sent: Monday, March 2, 2026 2:12 AM
> To: Nguyen, Anthony L <anthony.l.nguyen@intel.com>; David S. Miller
> <davem@davemloft.net>; Thomas Hellström
> <thomas.hellstrom@linux.intel.com>; Andrew Lunn
> <andrew+netdev@lunn.ch>; Andrew Morton <akpm@linux-foundation.org>;
> David Airlie <airlied@gmail.com>; Eric Dumazet <edumazet@google.com>;
> Jakub Kicinski <kuba@kernel.org>; Brost, Matthew
> <matthew.brost@intel.com>; Paolo Abeni <pabeni@redhat.com>; Kitszel,
> Przemyslaw <przemyslaw.kitszel@intel.com>; Vivi, Rodrigo
> <rodrigo.vivi@intel.com>; Simona Vetter <simona@ffwll.ch>; Yury Norov
> <yury.norov@gmail.com>; Rasmus Villemoes <linux@rasmusvillemoes.dk>;
> dri-devel@lists.freedesktop.org; intel-xe@lists.freedesktop.org;
> linux-kernel@vger.kernel.org; netdev@vger.kernel.org; intel-wired-
> lan@lists.osuosl.org
> Cc: Yury Norov <ynorov@nvidia.com>; Simon Horman <horms@kernel.org>;
> David Laight <david.laight.linux@gmail.com>
> Subject: [Intel-wired-lan] [PATCH v2 4/4] drm/xe: switch
> xe_pagefault_queue_init() to using bitmap_weighted_or()
>
> The function calls bitmap_or() immediately followed by
> bitmap_weight().
> Switch to using the dedicated bitmap_weighted_or() and save one bitmap
> traverse.
>
> Signed-off-by: Yury Norov <ynorov@nvidia.com>
> ---
> drivers/gpu/drm/xe/xe_pagefault.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_pagefault.c
> b/drivers/gpu/drm/xe/xe_pagefault.c
> index 6bee53d6ffc3..c4ce3cfe2164 100644
> --- a/drivers/gpu/drm/xe/xe_pagefault.c
> +++ b/drivers/gpu/drm/xe/xe_pagefault.c
> @@ -285,10 +285,9 @@ static int xe_pagefault_queue_init(struct
> xe_device *xe,
> xe_dss_mask_t all_dss;
> int num_dss, num_eus;
>
> - bitmap_or(all_dss, gt->fuse_topo.g_dss_mask,
> + num_dss = bitmap_weighted_or(all_dss, gt-
> >fuse_topo.g_dss_mask,
> gt->fuse_topo.c_dss_mask,
> XE_MAX_DSS_FUSE_BITS);
>
> - num_dss = bitmap_weight(all_dss, XE_MAX_DSS_FUSE_BITS);
> num_eus = bitmap_weight(gt->fuse_topo.eu_mask_per_dss,
> XE_MAX_EU_FUSE_BITS) * num_dss;
>
> --
> 2.43.0
Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or()
2026-03-02 1:11 ` [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or() Yury Norov
2026-03-02 7:14 ` [Intel-wired-lan] " Loktionov, Aleksandr
@ 2026-03-02 19:49 ` Matthew Brost
2026-03-04 11:43 ` Simon Horman
2 siblings, 0 replies; 13+ messages in thread
From: Matthew Brost @ 2026-03-02 19:49 UTC (permalink / raw)
To: Yury Norov
Cc: Tony Nguyen, David S. Miller, Thomas Hellström, Andrew Lunn,
Andrew Morton, David Airlie, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Przemek Kitszel, Rodrigo Vivi, Simona Vetter,
Yury Norov, Rasmus Villemoes, dri-devel, intel-xe, linux-kernel,
netdev, intel-wired-lan, Simon Horman, David Laight
On Sun, Mar 01, 2026 at 08:11:58PM -0500, Yury Norov wrote:
> The function calls bitmap_or() immediately followed by bitmap_weight().
> Switch to using the dedicated bitmap_weighted_or() and save one bitmap
> traverse.
>
> Signed-off-by: Yury Norov <ynorov@nvidia.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
> ---
> drivers/gpu/drm/xe/xe_pagefault.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_pagefault.c b/drivers/gpu/drm/xe/xe_pagefault.c
> index 6bee53d6ffc3..c4ce3cfe2164 100644
> --- a/drivers/gpu/drm/xe/xe_pagefault.c
> +++ b/drivers/gpu/drm/xe/xe_pagefault.c
> @@ -285,10 +285,9 @@ static int xe_pagefault_queue_init(struct xe_device *xe,
> xe_dss_mask_t all_dss;
> int num_dss, num_eus;
>
> - bitmap_or(all_dss, gt->fuse_topo.g_dss_mask,
> + num_dss = bitmap_weighted_or(all_dss, gt->fuse_topo.g_dss_mask,
> gt->fuse_topo.c_dss_mask, XE_MAX_DSS_FUSE_BITS);
>
> - num_dss = bitmap_weight(all_dss, XE_MAX_DSS_FUSE_BITS);
> num_eus = bitmap_weight(gt->fuse_topo.eu_mask_per_dss,
> XE_MAX_EU_FUSE_BITS) * num_dss;
>
> --
> 2.43.0
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or()
2026-03-02 1:11 ` [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or() Yury Norov
2026-03-02 7:14 ` [Intel-wired-lan] " Loktionov, Aleksandr
2026-03-02 19:49 ` Matthew Brost
@ 2026-03-04 11:43 ` Simon Horman
2026-03-05 22:40 ` Tony Nguyen
2 siblings, 1 reply; 13+ messages in thread
From: Simon Horman @ 2026-03-04 11:43 UTC (permalink / raw)
To: Yury Norov
Cc: Tony Nguyen, David S. Miller, Thomas Hellström, Andrew Lunn,
Andrew Morton, David Airlie, Eric Dumazet, Jakub Kicinski,
Matthew Brost, Paolo Abeni, Przemek Kitszel, Rodrigo Vivi,
Simona Vetter, Yury Norov, Rasmus Villemoes, dri-devel, intel-xe,
linux-kernel, netdev, intel-wired-lan, David Laight
On Sun, Mar 01, 2026 at 08:11:58PM -0500, Yury Norov wrote:
> The function calls bitmap_or() immediately followed by bitmap_weight().
> Switch to using the dedicated bitmap_weighted_or() and save one bitmap
> traverse.
>
> Signed-off-by: Yury Norov <ynorov@nvidia.com>
It's not entirely clear to me why this patch is included in a patchset
for the ice driver.
And it's also not clear to me why, but allmodconfigs - fo4 at least x86_32
and x86_64 - fail with this patch applied to net-next [1].
ERROR: modpost: "__bitmap_weighted_or" [drivers/gpu/drm/xe/xe.ko] undefined!
[1] 2b12ffb66955 ("net: mana: Trigger VF reset/recovery on health check failure due to HWC timeout")
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or()
2026-03-04 11:43 ` Simon Horman
@ 2026-03-05 22:40 ` Tony Nguyen
0 siblings, 0 replies; 13+ messages in thread
From: Tony Nguyen @ 2026-03-05 22:40 UTC (permalink / raw)
To: Simon Horman, Yury Norov
Cc: David S. Miller, Thomas Hellström, Andrew Lunn,
Andrew Morton, David Airlie, Eric Dumazet, Jakub Kicinski,
Matthew Brost, Paolo Abeni, Przemek Kitszel, Rodrigo Vivi,
Simona Vetter, Yury Norov, Rasmus Villemoes, dri-devel, intel-xe,
linux-kernel, netdev, intel-wired-lan, David Laight
On 3/4/2026 3:43 AM, Simon Horman wrote:
> On Sun, Mar 01, 2026 at 08:11:58PM -0500, Yury Norov wrote:
>> The function calls bitmap_or() immediately followed by bitmap_weight().
>> Switch to using the dedicated bitmap_weighted_or() and save one bitmap
>> traverse.
>>
>> Signed-off-by: Yury Norov <ynorov@nvidia.com>
>
> It's not entirely clear to me why this patch is included in a patchset
> for the ice driver.
>
> And it's also not clear to me why, but allmodconfigs - fo4 at least x86_32
> and x86_64 - fail with this patch applied to net-next [1].
>
> ERROR: modpost: "__bitmap_weighted_or" [drivers/gpu/drm/xe/xe.ko] undefined!
>
> [1] 2b12ffb66955 ("net: mana: Trigger VF reset/recovery on health check failure due to HWC timeout")
I'm also seeing the same error and no apparent reason. Since this
doesn't seem dependent on the others, I'll take the other 3 and this can
get sent/resolved separately.
Thanks,
Tony
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2 0/4] ice: use better bitmap API
2026-03-02 1:11 [PATCH v2 0/4] ice: use better bitmap API Yury Norov
` (3 preceding siblings ...)
2026-03-02 1:11 ` [PATCH v2 4/4] drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or() Yury Norov
@ 2026-03-02 23:01 ` Jacob Keller
4 siblings, 0 replies; 13+ messages in thread
From: Jacob Keller @ 2026-03-02 23:01 UTC (permalink / raw)
To: Yury Norov, Tony Nguyen, David S. Miller, Thomas Hellström,
Andrew Lunn, Andrew Morton, David Airlie, Eric Dumazet,
Jakub Kicinski, Matthew Brost, Paolo Abeni, Przemek Kitszel,
Rodrigo Vivi, Simona Vetter, Yury Norov, Rasmus Villemoes,
dri-devel, intel-xe, linux-kernel, netdev, intel-wired-lan
Cc: Simon Horman, David Laight
On 3/1/2026 5:11 PM, Yury Norov wrote:
> Use better bitmap API where appropriate. Patches #3,4 may be applied
> separately.
>
> v1: https://lore.kernel.org/all/20251223162303.434659-1-yury.norov@gmail.com/
> v2:
> - new v2#4;
> - add documentation line (Simon);
>
> Yury Norov (4):
> bitmap: introduce bitmap_weighted_xor()
> ice: use bitmap_weighted_xor() in ice_find_free_recp_res_idx()
> ice: use bitmap_empty() in ice_vf_has_no_qs_ena
> drm/xe: switch xe_pagefault_queue_init() to using bitmap_weighted_or()
>
Good cleanups!
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
> drivers/gpu/drm/xe/xe_pagefault.c | 3 +--
> drivers/net/ethernet/intel/ice/ice_switch.c | 4 +---
> drivers/net/ethernet/intel/ice/ice_vf_lib.c | 4 ++--
> include/linux/bitmap.h | 15 +++++++++++++++
> lib/bitmap.c | 7 +++++++
> 5 files changed, 26 insertions(+), 7 deletions(-)
>
^ permalink raw reply [flat|nested] 13+ messages in thread