iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

firmware: use static inlines for state machine checking

Browse Source 
This will allow us to do proper typechecking on users both of
values passed and return types expected.

While at it, change the parameter passed to be the struct fw_priv,
so we can move around the state machine variable as we see fit with
these helpers.

Signed-off-by: Luis R. Rodriguez <mcgrof@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Luis R. Rodriguez 2017-11-20 10:23:53 -08:00 committed by Greg Kroah-Hartman
parent c73bf7f487
commit f1b9cf39af
1 changed files with 69 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

109
drivers/base/firmware_class.c
View File

@ -195,14 +195,17 @@ static inline long firmware_loading_timeout(void)
return loading_timeout > 0 ? loading_timeout * HZ : MAX_JIFFY_OFFSET; return loading_timeout > 0 ? loading_timeout * HZ : MAX_JIFFY_OFFSET;
} }
static void fw_state_init(struct fw_state *fw_st) static void fw_state_init(struct fw_priv *fw_priv)
{ {
struct fw_state *fw_st = &fw_priv->fw_st;
init_completion(&fw_st->completion); init_completion(&fw_st->completion);
fw_st->status = FW_STATUS_UNKNOWN; fw_st->status = FW_STATUS_UNKNOWN;
} }
static int __fw_state_wait_common(struct fw_state *fw_st, long timeout) static int __fw_state_wait_common(struct fw_priv *fw_priv, long timeout)
{ {
struct fw_state *fw_st = &fw_priv->fw_st;
long ret; long ret;
ret = wait_for_completion_killable_timeout(&fw_st->completion, timeout); ret = wait_for_completion_killable_timeout(&fw_st->completion, timeout);
@ -214,40 +217,66 @@ static int __fw_state_wait_common(struct fw_state *fw_st, long timeout)
return ret < 0 ? ret : 0; return ret < 0 ? ret : 0;
} }
static void __fw_state_set(struct fw_state *fw_st, static void __fw_state_set(struct fw_priv *fw_priv,
enum fw_status status) enum fw_status status)
{ {
struct fw_state *fw_st = &fw_priv->fw_st;
WRITE_ONCE(fw_st->status, status); WRITE_ONCE(fw_st->status, status);
if (status == FW_STATUS_DONE || status == FW_STATUS_ABORTED) if (status == FW_STATUS_DONE || status == FW_STATUS_ABORTED)
complete_all(&fw_st->completion); complete_all(&fw_st->completion);
} }
#define fw_state_start(fw_st) \ static inline void fw_state_start(struct fw_priv *fw_priv)
__fw_state_set(fw_st, FW_STATUS_LOADING)
#define fw_state_done(fw_st) \
__fw_state_set(fw_st, FW_STATUS_DONE)
#define fw_state_aborted(fw_st) \
__fw_state_set(fw_st, FW_STATUS_ABORTED)
#define fw_state_wait(fw_st) \
__fw_state_wait_common(fw_st, MAX_SCHEDULE_TIMEOUT)
static int __fw_state_check(struct fw_state *fw_st, enum fw_status status)
{ {
__fw_state_set(fw_priv, FW_STATUS_LOADING);
}
static inline void fw_state_done(struct fw_priv *fw_priv)
{
__fw_state_set(fw_priv, FW_STATUS_DONE);
}
static inline void fw_state_aborted(struct fw_priv *fw_priv)
{
__fw_state_set(fw_priv, FW_STATUS_ABORTED);
}
static inline int fw_state_wait(struct fw_priv *fw_priv)
{
return __fw_state_wait_common(fw_priv, MAX_SCHEDULE_TIMEOUT);
}
static bool __fw_state_check(struct fw_priv *fw_priv,
enum fw_status status)
{
struct fw_state *fw_st = &fw_priv->fw_st;
return fw_st->status == status; return fw_st->status == status;
} }
#define fw_state_is_aborted(fw_st) \ static inline bool fw_state_is_aborted(struct fw_priv *fw_priv)
__fw_state_check(fw_st, FW_STATUS_ABORTED) {
return __fw_state_check(fw_priv, FW_STATUS_ABORTED);
}
#ifdef CONFIG_FW_LOADER_USER_HELPER #ifdef CONFIG_FW_LOADER_USER_HELPER
#define fw_state_is_done(fw_st) \ static inline bool fw_state_is_done(struct fw_priv *fw_priv)
__fw_state_check(fw_st, FW_STATUS_DONE) {
#define fw_state_is_loading(fw_st) \ return __fw_state_check(fw_priv, FW_STATUS_DONE);
__fw_state_check(fw_st, FW_STATUS_LOADING) }
#define fw_state_wait_timeout(fw_st, timeout) \
__fw_state_wait_common(fw_st, timeout) static inline bool fw_state_is_loading(struct fw_priv *fw_priv)
{
return __fw_state_check(fw_priv, FW_STATUS_LOADING);
}
static inline int fw_state_wait_timeout(struct fw_priv *fw_priv, long timeout)
{
return __fw_state_wait_common(fw_priv, timeout);
}
#endif /* CONFIG_FW_LOADER_USER_HELPER */ #endif /* CONFIG_FW_LOADER_USER_HELPER */
@ -273,7 +302,7 @@ static struct fw_priv *__allocate_fw_priv(const char *fw_name,
fw_priv->fwc = fwc; fw_priv->fwc = fwc;
fw_priv->data = dbuf; fw_priv->data = dbuf;
fw_priv->allocated_size = size; fw_priv->allocated_size = size;
fw_state_init(&fw_priv->fw_st); fw_state_init(fw_priv);
#ifdef CONFIG_FW_LOADER_USER_HELPER #ifdef CONFIG_FW_LOADER_USER_HELPER
INIT_LIST_HEAD(&fw_priv->pending_list); INIT_LIST_HEAD(&fw_priv->pending_list);
#endif #endif
@ -421,7 +450,7 @@ fw_get_filesystem_firmware(struct device *device, struct fw_priv *fw_priv)
} }
dev_dbg(device, "direct-loading %s\n", fw_priv->fw_name); dev_dbg(device, "direct-loading %s\n", fw_priv->fw_name);
fw_priv->size = size; fw_priv->size = size;
fw_state_done(&fw_priv->fw_st); fw_state_done(fw_priv);
break; break;
} }
__putname(path); __putname(path);
@ -522,7 +551,7 @@ static int assign_fw(struct firmware *fw, struct device *device,
struct fw_priv *fw_priv = fw->priv; struct fw_priv *fw_priv = fw->priv;
mutex_lock(&fw_lock); mutex_lock(&fw_lock);
if (!fw_priv->size || fw_state_is_aborted(&fw_priv->fw_st)) { if (!fw_priv->size || fw_state_is_aborted(fw_priv)) {
mutex_unlock(&fw_lock); mutex_unlock(&fw_lock);
return -ENOENT; return -ENOENT;
} }
@ -577,11 +606,11 @@ static void __fw_load_abort(struct fw_priv *fw_priv)
* There is a small window in which user can write to 'loading' * There is a small window in which user can write to 'loading'
* between loading done and disappearance of 'loading' * between loading done and disappearance of 'loading'
*/ */
if (fw_state_is_done(&fw_priv->fw_st)) if (fw_state_is_done(fw_priv))
return; return;
list_del_init(&fw_priv->pending_list); list_del_init(&fw_priv->pending_list);
fw_state_aborted(&fw_priv->fw_st); fw_state_aborted(fw_priv);
} }
static void fw_load_abort(struct fw_sysfs *fw_sysfs) static void fw_load_abort(struct fw_sysfs *fw_sysfs)
@ -699,7 +728,7 @@ static ssize_t firmware_loading_show(struct device *dev,
mutex_lock(&fw_lock); mutex_lock(&fw_lock);
if (fw_sysfs->fw_priv) if (fw_sysfs->fw_priv)
loading = fw_state_is_loading(&fw_sysfs->fw_priv->fw_st); loading = fw_state_is_loading(fw_sysfs->fw_priv);
mutex_unlock(&fw_lock); mutex_unlock(&fw_lock);
return sprintf(buf, "%d\n", loading); return sprintf(buf, "%d\n", loading);
@ -749,24 +778,24 @@ static ssize_t firmware_loading_store(struct device *dev,
mutex_lock(&fw_lock); mutex_lock(&fw_lock);
fw_priv = fw_sysfs->fw_priv; fw_priv = fw_sysfs->fw_priv;
if (fw_state_is_aborted(&fw_priv->fw_st)) if (fw_state_is_aborted(fw_priv))
goto out; goto out;
switch (loading) { switch (loading) {
case 1: case 1:
/* discarding any previous partial load */ /* discarding any previous partial load */
if (!fw_state_is_done(&fw_priv->fw_st)) { if (!fw_state_is_done(fw_priv)) {
for (i = 0; i < fw_priv->nr_pages; i++) for (i = 0; i < fw_priv->nr_pages; i++)
__free_page(fw_priv->pages[i]); __free_page(fw_priv->pages[i]);
vfree(fw_priv->pages); vfree(fw_priv->pages);
fw_priv->pages = NULL; fw_priv->pages = NULL;
fw_priv->page_array_size = 0; fw_priv->page_array_size = 0;
fw_priv->nr_pages = 0; fw_priv->nr_pages = 0;
fw_state_start(&fw_priv->fw_st); fw_state_start(fw_priv);
} }
break; break;
case 0: case 0:
if (fw_state_is_loading(&fw_priv->fw_st)) { if (fw_state_is_loading(fw_priv)) {
int rc; int rc;
/* /*
@ -790,10 +819,10 @@ static ssize_t firmware_loading_store(struct device *dev,
*/ */
list_del_init(&fw_priv->pending_list); list_del_init(&fw_priv->pending_list);
if (rc) { if (rc) {
fw_state_aborted(&fw_priv->fw_st); fw_state_aborted(fw_priv);
written = rc; written = rc;
} else { } else {
fw_state_done(&fw_priv->fw_st); fw_state_done(fw_priv);
} }
break; break;
} }
@ -855,7 +884,7 @@ static ssize_t firmware_data_read(struct file *filp, struct kobject *kobj,
mutex_lock(&fw_lock); mutex_lock(&fw_lock);
fw_priv = fw_sysfs->fw_priv; fw_priv = fw_sysfs->fw_priv;
if (!fw_priv || fw_state_is_done(&fw_priv->fw_st)) { if (!fw_priv || fw_state_is_done(fw_priv)) {
ret_count = -ENODEV; ret_count = -ENODEV;
goto out; goto out;
} }
@ -942,7 +971,7 @@ static ssize_t firmware_data_write(struct file *filp, struct kobject *kobj,
mutex_lock(&fw_lock); mutex_lock(&fw_lock);
fw_priv = fw_sysfs->fw_priv; fw_priv = fw_sysfs->fw_priv;
if (!fw_priv || fw_state_is_done(&fw_priv->fw_st)) { if (!fw_priv || fw_state_is_done(fw_priv)) {
retval = -ENODEV; retval = -ENODEV;
goto out; goto out;
} }
@ -1055,14 +1084,14 @@ static int _request_firmware_load(struct fw_sysfs *fw_sysfs,
timeout = MAX_JIFFY_OFFSET; timeout = MAX_JIFFY_OFFSET;
} }
retval = fw_state_wait_timeout(&fw_priv->fw_st, timeout); retval = fw_state_wait_timeout(fw_priv, timeout);
if (retval < 0) { if (retval < 0) {
mutex_lock(&fw_lock); mutex_lock(&fw_lock);
fw_load_abort(fw_sysfs); fw_load_abort(fw_sysfs);
mutex_unlock(&fw_lock); mutex_unlock(&fw_lock);
} }
if (fw_state_is_aborted(&fw_priv->fw_st)) { if (fw_state_is_aborted(fw_priv)) {
if (retval == -ERESTARTSYS) if (retval == -ERESTARTSYS)
retval = -EINTR; retval = -EINTR;
else else
@ -1173,7 +1202,7 @@ _request_firmware_prepare(struct firmware **firmware_p, const char *name,
firmware->priv = fw_priv; firmware->priv = fw_priv;
if (ret > 0) { if (ret > 0) {
ret = fw_state_wait(&fw_priv->fw_st); ret = fw_state_wait(fw_priv);
if (!ret) { if (!ret) {
fw_set_page_data(fw_priv, firmware); fw_set_page_data(fw_priv, firmware);
return 0; /* assigned */ return 0; /* assigned */
@ -1203,8 +1232,8 @@ static void fw_abort_batch_reqs(struct firmware *fw)
return; return;
fw_priv = fw->priv; fw_priv = fw->priv;
if (!fw_state_is_aborted(&fw_priv->fw_st)) if (!fw_state_is_aborted(fw_priv))
fw_state_aborted(&fw_priv->fw_st); fw_state_aborted(fw_priv);
} }
/* called from request_firmware() and request_firmware_work_func() */ /* called from request_firmware() and request_firmware_work_func() */