]> git.baikalelectronics.ru Git - kernel.git/commitdiff
hinic: Use the bitmap API when applicable
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sun, 26 Jun 2022 16:27:45 +0000 (18:27 +0200)
committerJakub Kicinski <kuba@kernel.org>
Tue, 28 Jun 2022 05:14:09 +0000 (22:14 -0700)
'vlan_bitmap' is a bitmap and is used as such. So allocate it with
devm_bitmap_zalloc() and its explicit bit size (i.e. VLAN_N_VID).

This avoids the need of the VLAN_BITMAP_SIZE macro which:
   - needlessly has a 'nic_dev' parameter
   - should be "long" (and not byte) aligned, so that the bitmap semantic
     is respected

This is in fact not an issue because VLAN_N_VID is 4096 at the time
being, but devm_bitmap_zalloc() is less verbose and easier to understand.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/6ff7b7d21414240794a77dc2456914412718a145.1656260842.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/huawei/hinic/hinic_main.c

index 05329292d940f8b2d83206750d1496239b1b604a..56a89793f47d4209b9e0dc3a122801d476e61381 100644 (file)
@@ -62,8 +62,6 @@ MODULE_PARM_DESC(rx_weight, "Number Rx packets for NAPI budget (default=64)");
 
 #define HINIC_LRO_RX_TIMER_DEFAULT     16
 
-#define VLAN_BITMAP_SIZE(nic_dev)       (ALIGN(VLAN_N_VID, 8) / 8)
-
 #define work_to_rx_mode_work(work)      \
                container_of(work, struct hinic_rx_mode_work, work)
 
@@ -1242,9 +1240,8 @@ static int nic_dev_init(struct pci_dev *pdev)
        u64_stats_init(&tx_stats->syncp);
        u64_stats_init(&rx_stats->syncp);
 
-       nic_dev->vlan_bitmap = devm_kzalloc(&pdev->dev,
-                                           VLAN_BITMAP_SIZE(nic_dev),
-                                           GFP_KERNEL);
+       nic_dev->vlan_bitmap = devm_bitmap_zalloc(&pdev->dev, VLAN_N_VID,
+                                                 GFP_KERNEL);
        if (!nic_dev->vlan_bitmap) {
                err = -ENOMEM;
                goto err_vlan_bitmap;