2021-10-06 18:07:59

by Gustavo A. R. Silva

[permalink] [raw]
Subject: [PATCH][next] net: mana: Use kcalloc() instead of kzalloc()

Use 2-factor multiplication argument form kcalloc() instead
of kzalloc().

Link: https://github.com/KSPP/linux/issues/162
Signed-off-by: Gustavo A. R. Silva <[email protected]>
---
drivers/net/ethernet/microsoft/mana/hw_channel.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/microsoft/mana/hw_channel.c b/drivers/net/ethernet/microsoft/mana/hw_channel.c
index d5c485a6d284..7c7a5fb91f79 100644
--- a/drivers/net/ethernet/microsoft/mana/hw_channel.c
+++ b/drivers/net/ethernet/microsoft/mana/hw_channel.c
@@ -363,7 +363,7 @@ static int mana_hwc_create_cq(struct hw_channel_context *hwc, u16 q_depth,
}
hwc_cq->gdma_cq = cq;

- comp_buf = kcalloc(q_depth, sizeof(struct gdma_comp), GFP_KERNEL);
+ comp_buf = kcalloc(q_depth, sizeof(*comp_buf), GFP_KERNEL);
if (!comp_buf) {
err = -ENOMEM;
goto out;
@@ -580,7 +580,7 @@ static int mana_hwc_test_channel(struct hw_channel_context *hwc, u16 q_depth,
return err;
}

- ctx = kzalloc(q_depth * sizeof(struct hwc_caller_ctx), GFP_KERNEL);
+ ctx = kcalloc(q_depth, sizeof(*ctx), GFP_KERNEL);
if (!ctx)
return -ENOMEM;

--
2.27.0


2021-10-06 18:26:24

by Dexuan Cui

[permalink] [raw]
Subject: RE: [PATCH][next] net: mana: Use kcalloc() instead of kzalloc()

> From: Gustavo A. R. Silva <[email protected]>
> Sent: Wednesday, October 6, 2021 11:09 AM
> ...
> Subject: [PATCH][next] net: mana: Use kcalloc() instead of kzalloc()

The [next] should be [net-next], butI guess David can fix this for you :-)

Reviewed-by: Dexuan Cui <[email protected]>