From owner-freebsd-net@freebsd.org Wed Sep 23 07:12:59 2015 Return-Path: Delivered-To: freebsd-net@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id D53C1A0686C for ; Wed, 23 Sep 2015 07:12:59 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id EA54A1618; Wed, 23 Sep 2015 07:12:58 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from porto.starpoint.kiev.ua (porto-e.starpoint.kiev.ua [212.40.38.100]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id KAA18113; Wed, 23 Sep 2015 10:12:50 +0300 (EEST) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1ZeeEU-000BJV-8v; Wed, 23 Sep 2015 10:12:50 +0300 Subject: Re: page fault in tcp_do_segment (r287759 suspected) To: freebsd-net , "George V. Neville-Neil" References: <56011276.4060206@FreeBSD.org> From: Andriy Gapon Message-ID: <560250B9.3080002@FreeBSD.org> Date: Wed, 23 Sep 2015 10:11:53 +0300 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <56011276.4060206@FreeBSD.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Sep 2015 07:12:59 -0000 On 22/09/2015 11:33, Andriy Gapon wrote: > That code actually looks like the following DTrace probe a few lines below: > > TCP_PROBE3(debug__input, tp, th, mtod(m, const char *)); > > So, it seems like 'm' could be NULL here. > I see two places in tcp_do_segment() where m gets assigned with NULL followed by > goto drop. If I had to guess then my guess would be that one of those code > paths was taken. > Since those NULL assignments were there for more than a year, then I would guess > that the addition of the probe is to blame: > https://svnweb.freebsd.org/base?view=revision&revision=287759 Should I file a bug report about this? Does anyone has suggestion for a simple fix? -- Andriy Gapon