Skip to content

Commit

Permalink
atm: he: Use the bitmap API to allocate bitmaps
Browse files Browse the repository at this point in the history
Use bitmap_zalloc()/bitmap_free() instead of hand-writing them.

It is less verbose and it improves the semantic.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/7f795bd6d5b2a00f581175b7069b229c2e5a4192.1657379127.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
  • Loading branch information
Christophe JAILLET authored and Jakub Kicinski committed Jul 12, 2022
1 parent 2b8bf3d commit 9e433ac
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions drivers/atm/he.c
Original file line number Diff line number Diff line change
Expand Up @@ -780,14 +780,11 @@ static int he_init_group(struct he_dev *he_dev, int group)
G0_RBPS_BS + (group * 32));

/* bitmap table */
he_dev->rbpl_table = kmalloc_array(BITS_TO_LONGS(RBPL_TABLE_SIZE),
sizeof(*he_dev->rbpl_table),
GFP_KERNEL);
he_dev->rbpl_table = bitmap_zalloc(RBPL_TABLE_SIZE, GFP_KERNEL);
if (!he_dev->rbpl_table) {
hprintk("unable to allocate rbpl bitmap table\n");
return -ENOMEM;
}
bitmap_zero(he_dev->rbpl_table, RBPL_TABLE_SIZE);

/* rbpl_virt 64-bit pointers */
he_dev->rbpl_virt = kmalloc_array(RBPL_TABLE_SIZE,
Expand Down Expand Up @@ -902,7 +899,7 @@ static int he_init_group(struct he_dev *he_dev, int group)
out_free_rbpl_virt:
kfree(he_dev->rbpl_virt);
out_free_rbpl_table:
kfree(he_dev->rbpl_table);
bitmap_free(he_dev->rbpl_table);

return -ENOMEM;
}
Expand Down Expand Up @@ -1578,7 +1575,7 @@ he_stop(struct he_dev *he_dev)
}

kfree(he_dev->rbpl_virt);
kfree(he_dev->rbpl_table);
bitmap_free(he_dev->rbpl_table);
dma_pool_destroy(he_dev->rbpl_pool);

if (he_dev->rbrq_base)
Expand Down

0 comments on commit 9e433ac

Please sign in to comment.