block: drop double zeroing
authorJulia Lawall <Julia.Lawall@inria.fr>
Sun, 20 Sep 2020 11:26:18 +0000 (13:26 +0200)
committerJens Axboe <axboe@kernel.dk>
Wed, 23 Sep 2020 15:18:13 +0000 (09:18 -0600)
sg_init_table zeroes its first argument, so the allocation of that argument
doesn't have to.

the semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
expression x;
@@

x =
- kzalloc
+ kmalloc
 (...)
...
sg_init_table(x,...)
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/bsg-lib.c

index d185396..330fede 100644 (file)
@@ -207,7 +207,7 @@ static int bsg_map_buffer(struct bsg_buffer *buf, struct request *req)
 
        BUG_ON(!req->nr_phys_segments);
 
-       buf->sg_list = kzalloc(sz, GFP_KERNEL);
+       buf->sg_list = kmalloc(sz, GFP_KERNEL);
        if (!buf->sg_list)
                return -ENOMEM;
        sg_init_table(buf->sg_list, req->nr_phys_segments);