From: Yuval Shaia <yuval.shaia-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
To: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Yonatan Cohen <yonatanc-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Subject: Re: [PATCH rdma-rc 1/4] IB/rxe: Fix kernel panic in udp_setup_tunnel
Date: Mon, 5 Sep 2016 16:24:02 +0300 [thread overview]
Message-ID: <20160905132402.GA14607@yuval-lap.uk.oracle.com> (raw)
In-Reply-To: <1473077359-11546-2-git-send-email-leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
On Mon, Sep 05, 2016 at 03:09:16PM +0300, Leon Romanovsky wrote:
> From: Yonatan Cohen <yonatanc-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
>
> Disable creation of a UDP socket for ipv6
> when CONFIG_IPV6 is not enabeld.
> Since udp_sock_create6() returns 0 when CONFIG_IPV6 is not set
>
> [ 46.888632] IP: [<c220705a>] setup_udp_tunnel_sock+0x6/0x4f
> [ 46.891355] *pdpt = 0000000000000000 *pde = f000ff53f000ff53
> [ 46.893918] Oops: 0002 [#1] PREEMPT
> [ 46.896014] CPU: 0 PID: 1 Comm: swapper Not tainted 4.7.0-rc4-00001-g8700e3e #1
> [ 46.900280] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Debian-1.8.2-1 04/01/2014
> [ 46.904905] task: cf06c040 ti: cf05e000 task.ti: cf05e000
> [ 46.907854] EIP: 0060:[<c220705a>] EFLAGS: 00210246 CPU: 0
> [ 46.911137] EIP is at setup_udp_tunnel_sock+0x6/0x4f
> [ 46.914070] EAX: 00000044 EBX: 00000001 ECX: cf05fef0 EDX: ca8142e0
> [ 46.917236] ESI: c2c4505b EDI: cf05fef0 EBP: cf05fed0 ESP: cf05fed0
> [ 46.919836] DS: 007b ES: 007b FS: 0000 GS: 00e0 SS: 0068
> [ 46.922046] CR0: 80050033 CR2: 000001fc CR3: 02cec000 CR4: 000006b0
> [ 46.924550] Stack:
> [ 46.926014] cf05ff10 c1fd4657 ca8142e0 0000000a 00000000 00000000 0000b712 00000008
> [ 46.931274] 00000000 6bb5bd01 c1fd48de 00000000 00000000 cf05ff1c 00000000 00000000
> [ 46.936122] cf05ff1c c1fd4bdf 00000000 cf05ff28 c2c4507b ffffffff cf05ff88 c2bf1c74
> [ 46.942350] Call Trace:
> [ 46.944403] [<c1fd4657>] rxe_setup_udp_tunnel+0x8f/0x99
> [ 46.947689] [<c1fd48de>] ? net_to_rxe+0x4e/0x4e
> [ 46.950567] [<c1fd4bdf>] rxe_net_init+0xe/0xa4
> [ 46.953147] [<c2c4507b>] rxe_module_init+0x20/0x4c
> [ 46.955448] [<c2bf1c74>] do_one_initcall+0x89/0x113
> [ 46.957797] [<c2bf15eb>] ? set_debug_rodata+0xf/0xf
> [ 46.959966] [<c2bf1dbc>] ? kernel_init_freeable+0xbe/0x15b
> [ 46.962262] [<c2bf1ddc>] kernel_init_freeable+0xde/0x15b
> [ 46.964418] [<c232eb54>] kernel_init+0x8/0xd0
> [ 46.966618] [<c2333122>] ret_from_kernel_thread+0xe/0x24
> [ 46.969592] [<c232eb4c>] ? rest_init+0x6f/0x6f
>
> Fixes: 8700e3e7c485 ("Soft RoCE driver")
> Signed-off-by: Yonatan Cohen <yonatanc-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> Signed-off-by: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> ---
> drivers/infiniband/sw/rxe/rxe.c | 14 ++++++++++--
> drivers/infiniband/sw/rxe/rxe_net.c | 45 ++++++++++++++++++++++++-------------
> drivers/infiniband/sw/rxe/rxe_net.h | 3 ++-
> 3 files changed, 43 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/infiniband/sw/rxe/rxe.c b/drivers/infiniband/sw/rxe/rxe.c
> index 55f0e8f..08e09d9 100644
> --- a/drivers/infiniband/sw/rxe/rxe.c
> +++ b/drivers/infiniband/sw/rxe/rxe.c
> @@ -362,12 +362,22 @@ static int __init rxe_module_init(void)
> return err;
> }
>
> - err = rxe_net_init();
> + err = rxe_net_ipv4_init();
> if (err) {
> - pr_err("rxe: unable to init\n");
> + pr_err("rxe: unable to init ipv4 tunnel\n");
> rxe_cache_exit();
> return err;
> }
> +
> +#if IS_ENABLED(CONFIG_IPV6)
> + err = rxe_net_ipv6_init();
> + if (err) {
> + pr_err("rxe: unable to init ipv6 tunnel\n");
> + rxe_cache_exit();
> + return err;
> + }
> +#endif
> +
> pr_info("rxe: loaded\n");
>
> return 0;
> diff --git a/drivers/infiniband/sw/rxe/rxe_net.c b/drivers/infiniband/sw/rxe/rxe_net.c
> index 0b8d2ea..e0866b4 100644
> --- a/drivers/infiniband/sw/rxe/rxe_net.c
> +++ b/drivers/infiniband/sw/rxe/rxe_net.c
> @@ -249,8 +249,8 @@ static struct socket *rxe_setup_udp_tunnel(struct net *net, __be16 port,
> memset(&udp_cfg, 0, sizeof(udp_cfg));
>
> if (ipv6) {
> - udp_cfg.family = AF_INET6;
> - udp_cfg.ipv6_v6only = 1;
> + udp_cfg.family = AF_INET6;
> + udp_cfg.ipv6_v6only = 1;
> } else {
> udp_cfg.family = AF_INET;
> }
> @@ -662,33 +662,22 @@ static struct notifier_block rxe_net_notifier = {
> .notifier_call = rxe_notify,
> };
>
> -int rxe_net_init(void)
> +int rxe_net_ipv4_init(void)
> {
> int err;
>
> spin_lock_init(&dev_list_lock);
>
> - recv_sockets.sk6 = rxe_setup_udp_tunnel(&init_net,
> - htons(ROCE_V2_UDP_DPORT), true);
> - if (IS_ERR(recv_sockets.sk6)) {
> - recv_sockets.sk6 = NULL;
> - pr_err("rxe: Failed to create IPv6 UDP tunnel\n");
> - return -1;
> - }
> -
> recv_sockets.sk4 = rxe_setup_udp_tunnel(&init_net,
> - htons(ROCE_V2_UDP_DPORT), false);
> + htons(ROCE_V2_UDP_DPORT), false);
> if (IS_ERR(recv_sockets.sk4)) {
> - rxe_release_udp_tunnel(recv_sockets.sk6);
> recv_sockets.sk4 = NULL;
> - recv_sockets.sk6 = NULL;
> pr_err("rxe: Failed to create IPv4 UDP tunnel\n");
> return -1;
> }
>
> err = register_netdevice_notifier(&rxe_net_notifier);
> if (err) {
> - rxe_release_udp_tunnel(recv_sockets.sk6);
> rxe_release_udp_tunnel(recv_sockets.sk4);
> pr_err("rxe: Failed to rigister netdev notifier\n");
> }
> @@ -696,11 +685,35 @@ int rxe_net_init(void)
> return err;
> }
>
> +int rxe_net_ipv6_init(void)
> +{
> + int err;
> +
> + spin_lock_init(&dev_list_lock);
> +
> + recv_sockets.sk6 = rxe_setup_udp_tunnel(&init_net,
> + htons(ROCE_V2_UDP_DPORT), true);
> + if (IS_ERR(recv_sockets.sk6)) {
> + recv_sockets.sk6 = NULL;
> + pr_err("rxe: Failed to create IPv6 UDP tunnel\n");
> + return -1;
> + }
> +
> + err = register_netdevice_notifier(&rxe_net_notifier);
> + if (err) {
> + rxe_release_udp_tunnel(recv_sockets.sk6);
> + pr_err("rxe: Failed to rigister netdev notifier\n");
> + }
> +
> + return err;
> +}
> +
Can this be also in IS_ENABLED(CONFIG_IPV6) scope?
> void rxe_net_exit(void)
> {
> +#if IS_ENABLED(CONFIG_IPV6)
> if (recv_sockets.sk6)
> rxe_release_udp_tunnel(recv_sockets.sk6);
> -
> +#endif
> if (recv_sockets.sk4)
> rxe_release_udp_tunnel(recv_sockets.sk4);
>
> diff --git a/drivers/infiniband/sw/rxe/rxe_net.h b/drivers/infiniband/sw/rxe/rxe_net.h
> index 7b06f76..b8e3284 100644
> --- a/drivers/infiniband/sw/rxe/rxe_net.h
> +++ b/drivers/infiniband/sw/rxe/rxe_net.h
> @@ -47,7 +47,8 @@ extern struct rxe_recv_sockets recv_sockets;
>
> struct rxe_dev *rxe_net_add(struct net_device *ndev);
>
> -int rxe_net_init(void);
> +int rxe_net_ipv4_init(void);
> +int rxe_net_ipv6_init(void);
Can this be also in IS_ENABLED(CONFIG_IPV6) scope?
> void rxe_net_exit(void);
>
> #endif /* RXE_NET_H */
> --
> 2.7.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-09-05 13:24 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-05 12:09 [PATCH rdma-rc 0/4] RXE fixes for 4.8 Leon Romanovsky
[not found] ` <1473077359-11546-1-git-send-email-leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2016-09-05 12:09 ` [PATCH rdma-rc 1/4] IB/rxe: Fix kernel panic in udp_setup_tunnel Leon Romanovsky
[not found] ` <1473077359-11546-2-git-send-email-leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2016-09-05 13:24 ` Yuval Shaia [this message]
[not found] ` <20160905132402.GA14607-Hxa29pjIrETlQW142y8m19+IiqhCXseY@public.gmane.org>
2016-09-05 14:03 ` Leon Romanovsky
2016-09-05 13:32 ` Doug Ledford
[not found] ` <e0399c1c-567c-4996-a454-4ebefbe6e665-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-09-05 14:04 ` Leon Romanovsky
[not found] ` <CAJ3xEMiBpGRy-JGuJ=P4eUF5zyz+2bm43F-jqPHtwm=3Uz9YYA@mail.gmail.com>
[not found] ` <CAJ3xEMiBpGRy-JGuJ=P4eUF5zyz+2bm43F-jqPHtwm=3Uz9YYA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-09-05 19:21 ` Or Gerlitz
[not found] ` <CAJ3xEMidqQRMqP6u+SJSJVbq0weREAtAJN=eXsA_zzS0KnQ_7Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-09-06 5:05 ` Leon Romanovsky
[not found] ` <20160906050518.GJ21847-2ukJVAZIZ/Y@public.gmane.org>
2016-09-06 6:06 ` Or Gerlitz
[not found] ` <CAJ3xEMiuuCTGiWbefAFLRYART4h8YypsvGqxT69cTpkbrDdx1g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-09-06 8:59 ` Leon Romanovsky
[not found] ` <20160906085943.GL21847-2ukJVAZIZ/Y@public.gmane.org>
2016-09-06 9:21 ` Or Gerlitz
2016-09-05 12:09 ` [PATCH rdma-rc 2/4] IB/rxe: Fix duplicate atomic request handling Leon Romanovsky
2016-09-05 12:09 ` [PATCH rdma-rc 3/4] IB/rxe: Fix race condition between requester and completer Leon Romanovsky
2016-09-05 12:09 ` [PATCH rdma-rc 4/4] IB/rxe: Fix kmem_cache leak Leon Romanovsky
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=20160905132402.GA14607@yuval-lap.uk.oracle.com \
--to=yuval.shaia-qhclzuegtsvqt0dzr+alfa@public.gmane.org \
--cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=yonatanc-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).