mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-04-13 09:59:31 +00:00
net/smc: delete pointless divide by one
Here "buf" is a void pointer so sizeof(*buf) is one. Doing a divide by one makes the code less readable. Delete it. Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Reviewed-by: Simon Horman <horms@kernel.org> Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com> Link: https://patch.msgid.link/ee1a790b-f874-4512-b3ae-9c45f99dc640@stanley.mountain Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
c5a9657018
commit
10bc9761d1
2 changed files with 2 additions and 2 deletions
|
@ -2155,7 +2155,7 @@ static int smcr_buf_map_link(struct smc_buf_desc *buf_desc, bool is_rmb,
|
|||
for_each_sg(buf_desc->sgt[lnk->link_idx].sgl, sg, nents, i) {
|
||||
size = min_t(int, PAGE_SIZE - offset, buf_size);
|
||||
sg_set_page(sg, vmalloc_to_page(buf), size, offset);
|
||||
buf += size / sizeof(*buf);
|
||||
buf += size;
|
||||
buf_size -= size;
|
||||
offset = 0;
|
||||
}
|
||||
|
|
|
@ -197,7 +197,7 @@ static int smc_rx_splice(struct pipe_inode_info *pipe, char *src, size_t len,
|
|||
partial[i].offset = offset;
|
||||
partial[i].len = size;
|
||||
partial[i].private = (unsigned long)priv[i];
|
||||
buf += size / sizeof(*buf);
|
||||
buf += size;
|
||||
left -= size;
|
||||
offset = 0;
|
||||
}
|
||||
|
|
Loading…
Add table
Reference in a new issue