diff options
| author |   Yury Norov <yury.norov@gmail.com> | 2022-01-23 10:38:40 -0800 |
|---|---|---|
| committer | Yury Norov <yury.norov@gmail.com> | 2022-05-02 06:30:39 -0700 |
| commit | 2b330a6a6c21a9b9f4ce71e5c0be494ab577423c (patch) | |
| tree | c541bdc9f154949aa0f99639657e0969bccff623 /drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c | |
| parent | KVM: x86: replace bitmap_weight with bitmap_empty where appropriate (diff) | |
| download | linux-dev-2b330a6a6c21a9b9f4ce71e5c0be494ab577423c.tar.xz linux-dev-2b330a6a6c21a9b9f4ce71e5c0be494ab577423c.zip | |
octeontx2-pf: replace bitmap_weight with bitmap_empty where appropriate
In some places, octeontx2 code calls bitmap_weight() to check if any bit of
a given bitmap is set. It's better to use bitmap_empty() in that case
because bitmap_empty() stops traversing the bitmap as soon as it finds
first set bit, while bitmap_weight() counts all bits unconditionally.
Signed-off-by: Yury Norov <yury.norov@gmail.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c b/drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c index 441aafc26a08..184e22795685 100644 --- a/drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c +++ b/drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c @@ -1120,7 +1120,7 @@ static int otx2_cgx_config_loopback(struct otx2_nic *pf, bool enable) struct msg_req *msg; int err; - if (enable && bitmap_weight(&pf->flow_cfg->dmacflt_bmap, + if (enable && !bitmap_empty(&pf->flow_cfg->dmacflt_bmap, pf->flow_cfg->dmacflt_max_flows)) netdev_warn(pf->netdev, "CGX/RPM internal loopback might not work as DMAC filters are active\n"); |