summaryrefslogtreecommitdiff
path: root/drivers/media/platform
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab+samsung@kernel.org>2018-08-07 14:29:12 +0300
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>2018-08-08 17:57:14 +0300
commit3354b54f9f7037a1122d3b6009aa9d39829d6843 (patch)
treee10ad409f67d05d07100718a2f6856d6d8ed888e /drivers/media/platform
parent845b978a871bff3707eee611b32e4be0b9a94dd2 (diff)
downloadlinux-3354b54f9f7037a1122d3b6009aa9d39829d6843.tar.xz
media: vivid: shut up warnings due to a non-trivial logic
The vivid driver uses a complex logic to save one kalloc/kfree allocation. That non-trivial way of allocating data causes smatch to warn: drivers/media/platform/vivid/vivid-core.c:869 vivid_create_instance() warn: potentially one past the end of array 'dev->query_dv_timings_qmenu[dev->query_dv_timings_size]' drivers/media/platform/vivid/vivid-core.c:869 vivid_create_instance() warn: potentially one past the end of array 'dev->query_dv_timings_qmenu[dev->query_dv_timings_size]' I also needed to read the code several times in order to understand what it was desired there. It turns that the logic was right, although confusing to read. As it is doing allocations on a non-standard way, let's add some documentation while shutting up the false positive. Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Diffstat (limited to 'drivers/media/platform')
-rw-r--r--drivers/media/platform/vivid/vivid-core.c21
1 files changed, 19 insertions, 2 deletions
diff --git a/drivers/media/platform/vivid/vivid-core.c b/drivers/media/platform/vivid/vivid-core.c
index 31db363602e5..4c235ea27987 100644
--- a/drivers/media/platform/vivid/vivid-core.c
+++ b/drivers/media/platform/vivid/vivid-core.c
@@ -647,6 +647,7 @@ static int vivid_create_instance(struct platform_device *pdev, int inst)
unsigned node_type = node_types[inst];
unsigned int allocator = allocators[inst];
v4l2_std_id tvnorms_cap = 0, tvnorms_out = 0;
+ char *strings_base;
int ret;
int i;
@@ -859,17 +860,33 @@ static int vivid_create_instance(struct platform_device *pdev, int inst)
/* create a string array containing the names of all the preset timings */
while (v4l2_dv_timings_presets[dev->query_dv_timings_size].bt.width)
dev->query_dv_timings_size++;
+
+ /*
+ * In order to save one allocation and an extra free, let's optimize
+ * the allocation here: we'll use the first elements of the
+ * dev->query_dv_timings_qmenu to store the timing strings pointer,
+ * adding an extra space there to a store a string up to 32 bytes.
+ * So, instead of allocating an array with size of:
+ * dev->query_dv_timings_size * (sizeof(void *)
+ * it will allocate:
+ * dev->query_dv_timings_size * (sizeof(void *) +
+ * dev->query_dv_timings_size * 32
+ */
dev->query_dv_timings_qmenu = kmalloc_array(dev->query_dv_timings_size,
(sizeof(void *) + 32),
GFP_KERNEL);
if (dev->query_dv_timings_qmenu == NULL)
goto free_dev;
+
+ /* Sets strings_base to be after the space to store the pointers */
+ strings_base = ((char *)&dev->query_dv_timings_qmenu)
+ + dev->query_dv_timings_size * sizeof(void *);
+
for (i = 0; i < dev->query_dv_timings_size; i++) {
const struct v4l2_bt_timings *bt = &v4l2_dv_timings_presets[i].bt;
- char *p = (char *)&dev->query_dv_timings_qmenu[dev->query_dv_timings_size];
+ char *p = strings_base + i * 32;
u32 htot, vtot;
- p += i * 32;
dev->query_dv_timings_qmenu[i] = p;
htot = V4L2_DV_BT_FRAME_WIDTH(bt);