diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-08-28 20:40:15 +0200 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2019-09-04 06:47:14 +0200 |
commit | 25af5afe77a8b4278103c4c4fe23fe8d6f095490 (patch) | |
tree | 8182d495cc698ca3b434bb01bb626298e367fa1b /Documentation | |
parent | dmaengine: iop-adma: remove set but not used variable 'slots_per_op' (diff) | |
download | linux-25af5afe77a8b4278103c4c4fe23fe8d6f095490.tar.xz linux-25af5afe77a8b4278103c4c4fe23fe8d6f095490.zip |
dmanegine: ioat/dca: Use struct_size() helper
One of the more common cases of allocation size calculations is finding
the size of a structure that has a zero-sized array at the end, along
with memory for some number of elements for that array. For example:
struct ioat_dca_priv {
...
struct ioat_dca_slot req_slots[0];
};
Make use of the struct_size() helper instead of an open-coded version
in order to avoid any potential type mistakes.
So, replace the following form:
sizeof(*ioatdca) + (sizeof(struct ioat_dca_slot) * slots)
with:
struct_size(ioatdca, req_slots, slots)
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Acked-by: Dave Jiang <dave.jiang@intel.com>
Link: https://lore.kernel.org/r/20190828184015.GA4273@embeddedor
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions