Commit e0fd9b1d authored by Luis R. Rodriguez's avatar Luis R. Rodriguez Committed by Greg Kroah-Hartman

firmware: use const for remaining firmware names

We currently use flexible arrays with a char at the
end for the remaining internal firmware name uses.
There are two limitations with the way we use this.
Since we're using a flexible array for a string on the
struct if we wanted to use two strings it means we'd
have a disjoint means of handling the strings, one
using the flexible array, and another a char * pointer.
We're also currently not using 'const' for the string.

We wish to later extend some firmware data structures
with other string/char pointers, but we also want to be
very pedantic about const usage. Since we're going to
change things to use 'const' we might as well also address
unified way to use multiple strings on the structs.

Replace the flexible array practice for strings with
kstrdup_const() and kfree_const(), this will avoid
allocations when the vmlinux .rodata is used, and just
allocate a new proper string for us when needed. This
also means we can simplify the struct allocations by
removing the string length from the allocation size
computation, which would otherwise get even more
complicated when supporting multiple strings.

Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Rusty Russell <rusty@rustcorp.com.au>
Cc: David Howells <dhowells@redhat.com>
Cc: Ming Lei <ming.lei@canonical.com>
Cc: Seth Forshee <seth.forshee@canonical.com>
Cc: Kyle McMartin <kyle@kernel.org>
Signed-off-by: default avatarLuis R. Rodriguez <mcgrof@suse.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f9692b26
...@@ -150,17 +150,17 @@ struct firmware_buf { ...@@ -150,17 +150,17 @@ struct firmware_buf {
int page_array_size; int page_array_size;
struct list_head pending_list; struct list_head pending_list;
#endif #endif
char fw_id[]; const char *fw_id;
}; };
struct fw_cache_entry { struct fw_cache_entry {
struct list_head list; struct list_head list;
char name[]; const char *name;
}; };
struct fw_name_devm { struct fw_name_devm {
unsigned long magic; unsigned long magic;
char name[]; const char *name;
}; };
#define to_fwbuf(d) container_of(d, struct firmware_buf, ref) #define to_fwbuf(d) container_of(d, struct firmware_buf, ref)
...@@ -181,13 +181,17 @@ static struct firmware_buf *__allocate_fw_buf(const char *fw_name, ...@@ -181,13 +181,17 @@ static struct firmware_buf *__allocate_fw_buf(const char *fw_name,
{ {
struct firmware_buf *buf; struct firmware_buf *buf;
buf = kzalloc(sizeof(*buf) + strlen(fw_name) + 1, GFP_ATOMIC); buf = kzalloc(sizeof(*buf), GFP_ATOMIC);
if (!buf) if (!buf)
return buf; return NULL;
buf->fw_id = kstrdup_const(fw_name, GFP_ATOMIC);
if (!buf->fw_id) {
kfree(buf);
return NULL;
}
kref_init(&buf->ref); kref_init(&buf->ref);
strcpy(buf->fw_id, fw_name);
buf->fwc = fwc; buf->fwc = fwc;
init_completion(&buf->completion); init_completion(&buf->completion);
#ifdef CONFIG_FW_LOADER_USER_HELPER #ifdef CONFIG_FW_LOADER_USER_HELPER
...@@ -257,6 +261,7 @@ static void __fw_free_buf(struct kref *ref) ...@@ -257,6 +261,7 @@ static void __fw_free_buf(struct kref *ref)
} else } else
#endif #endif
vfree(buf->data); vfree(buf->data);
kfree_const(buf->fw_id);
kfree(buf); kfree(buf);
} }
...@@ -401,6 +406,7 @@ static void fw_name_devm_release(struct device *dev, void *res) ...@@ -401,6 +406,7 @@ static void fw_name_devm_release(struct device *dev, void *res)
if (fwn->magic == (unsigned long)&fw_cache) if (fwn->magic == (unsigned long)&fw_cache)
pr_debug("%s: fw_name-%s devm-%p released\n", pr_debug("%s: fw_name-%s devm-%p released\n",
__func__, fwn->name, res); __func__, fwn->name, res);
kfree_const(fwn->name);
} }
static int fw_devm_match(struct device *dev, void *res, static int fw_devm_match(struct device *dev, void *res,
...@@ -431,13 +437,17 @@ static int fw_add_devm_name(struct device *dev, const char *name) ...@@ -431,13 +437,17 @@ static int fw_add_devm_name(struct device *dev, const char *name)
if (fwn) if (fwn)
return 1; return 1;
fwn = devres_alloc(fw_name_devm_release, sizeof(struct fw_name_devm) + fwn = devres_alloc(fw_name_devm_release, sizeof(struct fw_name_devm),
strlen(name) + 1, GFP_KERNEL); GFP_KERNEL);
if (!fwn) if (!fwn)
return -ENOMEM; return -ENOMEM;
fwn->name = kstrdup_const(name, GFP_KERNEL);
if (!fwn->name) {
kfree(fwn);
return -ENOMEM;
}
fwn->magic = (unsigned long)&fw_cache; fwn->magic = (unsigned long)&fw_cache;
strcpy(fwn->name, name);
devres_add(dev, fwn); devres_add(dev, fwn);
return 0; return 0;
...@@ -1397,11 +1407,16 @@ static struct fw_cache_entry *alloc_fw_cache_entry(const char *name) ...@@ -1397,11 +1407,16 @@ static struct fw_cache_entry *alloc_fw_cache_entry(const char *name)
{ {
struct fw_cache_entry *fce; struct fw_cache_entry *fce;
fce = kzalloc(sizeof(*fce) + strlen(name) + 1, GFP_ATOMIC); fce = kzalloc(sizeof(*fce), GFP_ATOMIC);
if (!fce) if (!fce)
goto exit; goto exit;
strcpy(fce->name, name); fce->name = kstrdup_const(name, GFP_ATOMIC);
if (!fce->name) {
kfree(fce);
fce = NULL;
goto exit;
}
exit: exit:
return fce; return fce;
} }
...@@ -1441,6 +1456,7 @@ static int fw_cache_piggyback_on_request(const char *name) ...@@ -1441,6 +1456,7 @@ static int fw_cache_piggyback_on_request(const char *name)
static void free_fw_cache_entry(struct fw_cache_entry *fce) static void free_fw_cache_entry(struct fw_cache_entry *fce)
{ {
kfree_const(fce->name);
kfree(fce); kfree(fce);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment