From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: [PATCH rdma-rc V1 1/4] IB/rxe: Fix kernel panic in udp_setup_tunnel Date: Wed, 7 Sep 2016 15:13:03 +0300 Message-ID: <20160907121303.GD18398@leon.nu> References: <1473246247-16536-1-git-send-email-leon@kernel.org> <1473246247-16536-2-git-send-email-leon@kernel.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="9dgjiU4MmWPVapMU" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Or Gerlitz Cc: Moni Shoua , Doug Ledford , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Yonatan Cohen List-Id: linux-rdma@vger.kernel.org --9dgjiU4MmWPVapMU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Sep 07, 2016 at 02:20:50PM +0300, Or Gerlitz wrote: > On Wed, Sep 7, 2016 at 2:04 PM, Leon Romanovsky wrote: > > From: Yonatan Cohen > > > > 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: [] 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:[] 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] [] rxe_setup_udp_tunnel+0x8f/0x99 > > [ 46.947689] [] ? net_to_rxe+0x4e/0x4e > > [ 46.950567] [] rxe_net_init+0xe/0xa4 > > [ 46.953147] [] rxe_module_init+0x20/0x4c > > [ 46.955448] [] do_one_initcall+0x89/0x113 > > [ 46.957797] [] ? set_debug_rodata+0xf/0xf > > [ 46.959966] [] ? kernel_init_freeable+0xbe/0x15b > > [ 46.962262] [] kernel_init_freeable+0xde/0x15b > > [ 46.964418] [] kernel_init+0x8/0xd0 > > [ 46.966618] [] ret_from_kernel_thread+0xe/0x24 > > [ 46.969592] [] ? 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. We are not in debate club here and while commit messages are important, the code and bug fixes are much more. If you think that kernel oops doesn't belong to commit message, please fix SubmittingPatches [1] guide and don't forget to announce it and enforce it in various subsystems. 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? > > $ ./scripts/get_maintainer.pl drivers/infiniband/sw/rxe > Moni Shoua (supporter:SOFT-ROCE DRIVER (rxe)) > Doug Ledford (supporter:INFINIBAND SUBSYSTEM) > Sean Hefty (supporter:INFINIBAND SUBSYSTEM) > Hal Rosenstock (supporter:INFINIBAND SUBSYSTEM) > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org (open list:SOFT-ROCE DRIVER (rxe)) > linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org (open list) > > 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. The answer will be YES. Moni reviewed first version and I reviewed the second one. [1] http://lxr.free-electrons.com/source/Documentation/SubmittingPatches#L691 > > > > Or. > -- > 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 --9dgjiU4MmWPVapMU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJX0ARPAAoJEORje4g2clins5gP/im8YvvTlsrfsf0SS9+B3MbG HGyA0/Xho5w8jb9lkfjn5IJUH6sppetDb05mRXCaUWthrqceWEYjfBPfHvc9YsZU mMJWzW0SPw5Yj4xWrN9AZXoKWsJrZ5YYFezZp2PqPQGiDKJRSbSowJbvkvluuzXJ r+5Pte9tZocsLfsoETrNen9neq8f+5nncd58bHGldFgpIz0h5zS1bORSRjuWrSFA pQ7NMP8nVK44kr03dbjxX4nQNB3euL87XP4MnPvqsN4ubl28V3W5H9thkHSQsJl9 Obm+UF8GA+29fwIeAzBhrSHhFmniA0JeelfNMcpxLQPLo7OZRaar74h5RqLQRYm7 mDtCIwwSJ4bufxs3t9nQHs+4a7Xoukk6MaWfF2VAczTXwXYK6U3CxN/3koCSYoF4 ht7U4cVl5EM+MUKsHyeTfdMXA54/kQH+A38hout7pUVQkDsPsozR1zbTBpAMM9p3 83D5R6p3VDWrQ+Wsxzc7fMTSZtaOoE535v2i6sDsrNT8S6/J2PFwRPw7I5c1Qxdv O0kzBEoflobokL4jnEqwSkwP6qYp+HMedGHoMNumBQd85wyfvVimXA1ip0eKWKTa sgxB4/qFTpq9OHPfQ8H9sC+WKkNrFS0ofFhbQbTRQgpWpi/0T2qOZRDASQIsDVNf Y7QgfSbNIQm6Owjyy77S =B3MW -----END PGP SIGNATURE----- --9dgjiU4MmWPVapMU-- -- 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