On Wed, Sep 7, 2016 at 3:13 PM, Leon Romanovsky <leonro@xxxxxxxxxxxx> wrote: > On Wed, Sep 07, 2016 at 02:20:50PM +0300, Or Gerlitz wrote: >> On Wed, Sep 7, 2016 at 2:04 PM, Leon Romanovsky <leon@xxxxxxxxxx> 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 >> >> what an unnice way to settle a review discussion. I replied to all >> your arguments explaining >> why the change log has to go, and the next step is to ignore this >> discussion and do it again >> wrong w.o replying? so... again: > > Or, > > You didn't like the format of commit message, while I liked - that's > all, it is the end of discussion. I made **concrete** **arguments** replying to yours why to my opinion the oops stack listing doesn't help anyone and hence doesn't belong to the kernel logs and need not be there forever. This is not liking thing, this is technical talking thing. > We are not in debate club here and while commit messages are > important, the code and bug fixes are much more. so strip this stupid stack trace which helps no one and set the patch free. > Do you think that there is something in the code which needs to be > addressed before Doug's take this patchset and releases RXE with > less unaddressed issues? yes, see my comments to the rest of the patches of the series, some small fixes are needed. >> Moni, can you please put your maintainer say here. > You are sitting next to him and you can freely ask him if he reviewed it. Your assumption is (1) wrong and (2) irrelevant to this discussion > The answer will be YES. Moni reviewed first version and I reviewed > the second one. There's a reviewer discussion on a patch for a driver he (and not you) maintain, he has to put his maintainer say. Or. -- 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
