diff options
| author | Jakub Kicinski <[email protected]> | 2024-07-11 18:32:52 -0700 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2024-07-11 18:32:52 -0700 |
| commit | 2a4183f864dad50f84edb6b67a2807a438f944fd (patch) | |
| tree | 91a6bc92f672ab709a03338ef89e1ca7a4d6b979 | |
| parent | 5e724cb688a207ae7a348d57f9ea77f475998883 (diff) | |
| parent | 0066623d40085bb5d0af8d645a3bba27fc5cfced (diff) | |
Merge branch 'netconsole-fix-potential-race-condition-and-improve-code-clarity'
Breno Leitao says:
====================
netconsole: improve code clarity
These changes aim to enhance the reliability of netconsole by
eliminating the potential race condition and improve maintainability
by making the code more straightforward to understand and modify.
====================
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
| -rw-r--r-- | drivers/net/netconsole.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c index ab8a0623b1a1..5ef680cf994a 100644 --- a/drivers/net/netconsole.c +++ b/drivers/net/netconsole.c @@ -344,7 +344,7 @@ static ssize_t enabled_store(struct config_item *item, goto out_unlock; err = -EINVAL; - if ((bool)enabled == nt->enabled) { + if (enabled == nt->enabled) { pr_info("network logging has already %s\n", nt->enabled ? "started" : "stopped"); goto out_unlock; @@ -369,6 +369,7 @@ static ssize_t enabled_store(struct config_item *item, if (err) goto out_unlock; + nt->enabled = true; pr_info("network logging started\n"); } else { /* false */ /* We need to disable the netconsole before cleaning it up @@ -381,8 +382,6 @@ static ssize_t enabled_store(struct config_item *item, netpoll_cleanup(&nt->np); } - nt->enabled = enabled; - mutex_unlock(&dynamic_netconsole_mutex); return strnlen(buf, count); out_unlock: |