mirror of https://github.com/torvalds/linux.git
tracing: fprobe: Remove unused local variable
The 'ret' local variable in fprobe_remove_node_in_module() was used
for checking the error state in the loop, but commit dfe0d675df82
("tracing: fprobe: use rhltable for fprobe_ip_table") removed the loop.
So we don't need it anymore.
Link: https://lore.kernel.org/all/175867358989.600222.6175459620045800878.stgit@devnote2/
Fixes: e5a4cc28a052 ("tracing: fprobe: use rhltable for fprobe_ip_table")
Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Acked-by: Menglong Dong <menglong8.dong@gmail.com>
This commit is contained in:
parent
cbe1e1241a
commit
90e69d291d
|
|
@ -458,8 +458,6 @@ static int fprobe_addr_list_add(struct fprobe_addr_list *alist, unsigned long ad
|
||||||
static void fprobe_remove_node_in_module(struct module *mod, struct fprobe_hlist_node *node,
|
static void fprobe_remove_node_in_module(struct module *mod, struct fprobe_hlist_node *node,
|
||||||
struct fprobe_addr_list *alist)
|
struct fprobe_addr_list *alist)
|
||||||
{
|
{
|
||||||
int ret = 0;
|
|
||||||
|
|
||||||
if (!within_module(node->addr, mod))
|
if (!within_module(node->addr, mod))
|
||||||
return;
|
return;
|
||||||
if (delete_fprobe_node(node))
|
if (delete_fprobe_node(node))
|
||||||
|
|
@ -468,8 +466,7 @@ static void fprobe_remove_node_in_module(struct module *mod, struct fprobe_hlist
|
||||||
* If failed to update alist, just continue to update hlist.
|
* If failed to update alist, just continue to update hlist.
|
||||||
* Therefore, at list user handler will not hit anymore.
|
* Therefore, at list user handler will not hit anymore.
|
||||||
*/
|
*/
|
||||||
if (!ret)
|
fprobe_addr_list_add(alist, node->addr);
|
||||||
ret = fprobe_addr_list_add(alist, node->addr);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Handle module unloading to manage fprobe_ip_table. */
|
/* Handle module unloading to manage fprobe_ip_table. */
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue