From: Marc Kleine-Budde <mkl@pengutronix.de> To: linux-can@vger.kernel.org Cc: kernel@pengutronix.de, Zhang Changzhong <zhangchangzhong@huawei.com>, Oliver Hartkopp <socketcan@hartkopp.net>, Marc Kleine-Budde <mkl@pengutronix.de> Subject: [net-rfc 01/16] can: proc: can_remove_proc(): silence remove_proc_entry warning Date: Mon, 19 Oct 2020 21:05:09 +0200 Message-ID: <20201019190524.1285319-2-mkl@pengutronix.de> (raw) In-Reply-To: <20201019190524.1285319-1-mkl@pengutronix.de> From: Zhang Changzhong <zhangchangzhong@huawei.com> If can_init_proc() fail to create /proc/net/can directory, can_remove_proc() will trigger a warning: WARNING: CPU: 6 PID: 7133 at fs/proc/generic.c:672 remove_proc_entry+0x17b0 Kernel panic - not syncing: panic_on_warn set ... Fix to return early from can_remove_proc() if can proc_dir does not exists. Signed-off-by: Zhang Changzhong <zhangchangzhong@huawei.com> Link: https://lore.kernel.org/r/1594709090-3203-1-git-send-email-zhangchangzhong@huawei.com Fixes: 8e8cda6d737d ("can: initial support for network namespaces") Acked-by: Oliver Hartkopp <socketcan@hartkopp.net> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de> --- net/can/proc.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/net/can/proc.c b/net/can/proc.c index 550928b8b8a2..5ea8695f507e 100644 --- a/net/can/proc.c +++ b/net/can/proc.c @@ -462,6 +462,9 @@ void can_init_proc(struct net *net) */ void can_remove_proc(struct net *net) { + if (!net->can.proc_dir) + return; + if (net->can.pde_stats) remove_proc_entry(CAN_PROC_STATS, net->can.proc_dir); @@ -486,6 +489,5 @@ void can_remove_proc(struct net *net) if (net->can.pde_rcvlist_sff) remove_proc_entry(CAN_PROC_RCVLIST_SFF, net->can.proc_dir); - if (net->can.proc_dir) - remove_proc_entry("can", net->proc_net); + remove_proc_entry("can", net->proc_net); } -- 2.28.0
next prev parent reply index Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-10-19 19:05 [RFC]: can 2020-10-19 Marc Kleine-Budde 2020-10-19 19:05 ` Marc Kleine-Budde [this message] 2020-10-19 19:05 ` [net-rfc 02/16] can: rx-offload: don't call kfree_skb() from IRQ context Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 03/16] can: dev: can_get_echo_skb(): prevent call to kfree_skb() in hard " Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 04/16] can: dev: can_get_len(): add a helper function to get the correct length of Classical frames Marc Kleine-Budde 2020-10-19 20:35 ` Oliver Hartkopp 2020-10-20 6:35 ` Marc Kleine-Budde 2020-10-20 11:30 ` Vincent Mailhol 2020-10-20 11:48 ` Marc Kleine-Budde 2020-10-20 12:38 ` Oliver Hartkopp 2020-10-20 15:02 ` Marc Kleine-Budde 2020-10-20 16:07 ` Vincent Mailhol 2020-10-20 17:04 ` Oliver Hartkopp 2020-10-20 18:50 ` Marc Kleine-Budde 2020-10-21 0:52 ` Vincent Mailhol 2020-10-21 6:23 ` Vincent MAILHOL 2020-10-21 7:11 ` Joakim Zhang 2020-10-21 7:21 ` Marc Kleine-Budde 2020-10-21 7:48 ` Joakim Zhang 2020-10-21 9:21 ` Oliver Hartkopp 2020-10-21 9:48 ` Oliver Hartkopp 2020-10-21 11:55 ` Vincent MAILHOL 2020-10-21 17:52 ` Oliver Hartkopp 2020-10-22 3:30 ` Vincent MAILHOL 2020-10-22 7:15 ` Oliver Hartkopp 2020-10-22 12:23 ` Vincent MAILHOL 2020-10-22 13:28 ` Oliver Hartkopp 2020-10-22 15:46 ` Vincent MAILHOL 2020-10-22 17:06 ` Oliver Hartkopp 2020-10-23 10:36 ` Vincent MAILHOL 2020-10-23 16:47 ` Oliver Hartkopp 2020-10-24 5:25 ` Vincent MAILHOL 2020-10-24 11:31 ` Oliver Hartkopp 2020-10-19 19:05 ` [net-rfc 05/16] can: dev: __can_get_echo_skb(): fix the returned length of CAN frame Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 06/16] can: can_create_echo_skb(): fix echo skb generation: always use skb_clone() Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 07/16] can: j1939: j1939_sk_bind(): return failure if netdev is down Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 08/16] can: isotp: Explain PDU in CAN_ISOTP help text Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 09/16] can: isotp: enable RX timeout handling in listen-only mode Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 10/16] can: ti_hecc: add missed clk_disable_unprepare() in error path Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 11/16] can: xilinx_can: handle failure cases of pm_runtime_get_sync Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 12/16] can: peak_usb: fix timestamp wrapping Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 13/16] can: peak_canfd: fix echo management when loopback is on Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 14/16] can: mcp251xfd: mcp251xfd_regmap_crc_read(): increase severity of CRC read error messages Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 15/16] can: mcp251xfd: mcp251xfd_regmap_nocrc_read(): fix semicolon.cocci warnings Marc Kleine-Budde 2020-10-19 19:05 ` [net-rfc 16/16] can: mcp251xfd: remove unneeded break Marc Kleine-Budde
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20201019190524.1285319-2-mkl@pengutronix.de \ --to=mkl@pengutronix.de \ --cc=kernel@pengutronix.de \ --cc=linux-can@vger.kernel.org \ --cc=socketcan@hartkopp.net \ --cc=zhangchangzhong@huawei.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Linux-Can Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/linux-can/0 linux-can/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 linux-can linux-can/ https://lore.kernel.org/linux-can \ linux-can@vger.kernel.org public-inbox-index linux-can Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kernel.vger.linux-can AGPL code for this site: git clone https://public-inbox.org/public-inbox.git