On Mon, Sep 05, 2016 at 04:24:02PM +0300, Yuval Shaia wrote:
> On Mon, Sep 05, 2016 at 03:09:16PM +0300, Leon Romanovsky wrote:
> > From: Yonatan Cohen <yonatanc@xxxxxxxxxxxx>
> >
> > 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@xxxxxxxxxxxx>
> > Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxx>
> > ---
> > 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?
In current implementation it should.
I personally would implement these config checks in rxe_net_ipv6_init()
function and not outside.
>
> > 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@xxxxxxxxxxxxxxx
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
Attachment:
signature.asc
Description: PGP signature
