From owner-svn-src-head@freebsd.org Fri Aug 12 15:23:51 2016 Return-Path: Delivered-To: svn-src-head@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 B3CACBB782D; Fri, 12 Aug 2016 15:23:51 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 915571796; Fri, 12 Aug 2016 15:23:51 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [IPv6:::1]) by freefall.freebsd.org (Postfix) with ESMTP id 8569917D6; Fri, 12 Aug 2016 15:23:51 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id 3DEFA7D49; Fri, 12 Aug 2016 15:23:51 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id Ud53tiU6bk1E; Fri, 12 Aug 2016 15:23:47 +0000 (UTC) Subject: Re: svn commit: r303019 - head/sys/geom DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com CF6167D43 To: Warner Losh , "Kenneth D. Merry" References: <201607190536.u6J5aLl7015268@repo.freebsd.org> <3277576.dFZ2EGo2Fh@overcee.wemm.org> <6198652.UmU69kS6Zt@overcee.wemm.org> <181fcc35-3a5d-043f-7dc4-7a01a53eebae@FreeBSD.org> <7372bf93-69a2-f5f5-1d07-204fd31f252c@FreeBSD.org> <20160812151117.GA52309@mithlond.kdm.org> <20160812151745.GA52527@mithlond.kdm.org> Cc: "Andrey V. Elsukov" , Peter Wemm , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" , src-committers From: Bryan Drewery Organization: FreeBSD Message-ID: Date: Fri, 12 Aug 2016 08:23:43 -0700 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="cUcpUOGMI3scLeW38BQE1FqfCJfLSM1TH" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 12 Aug 2016 15:23:51 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --cUcpUOGMI3scLeW38BQE1FqfCJfLSM1TH Content-Type: multipart/mixed; boundary="srVN6Bwftn190dQEXwoKeebQISJUiN0dC" From: Bryan Drewery To: Warner Losh , "Kenneth D. Merry" Cc: "Andrey V. Elsukov" , Peter Wemm , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" , src-committers Message-ID: Subject: Re: svn commit: r303019 - head/sys/geom References: <201607190536.u6J5aLl7015268@repo.freebsd.org> <3277576.dFZ2EGo2Fh@overcee.wemm.org> <6198652.UmU69kS6Zt@overcee.wemm.org> <181fcc35-3a5d-043f-7dc4-7a01a53eebae@FreeBSD.org> <7372bf93-69a2-f5f5-1d07-204fd31f252c@FreeBSD.org> <20160812151117.GA52309@mithlond.kdm.org> <20160812151745.GA52527@mithlond.kdm.org> In-Reply-To: --srVN6Bwftn190dQEXwoKeebQISJUiN0dC Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 8/12/16 8:21 AM, Warner Losh wrote: > On Fri, Aug 12, 2016 at 9:17 AM, Kenneth D. Merry wro= te: >> On Fri, Aug 12, 2016 at 09:13:58 -0600, Warner Losh wrote: >>> On Fri, Aug 12, 2016 at 9:11 AM, Kenneth D. Merry w= rote: >>>> On Fri, Aug 12, 2016 at 13:38:21 +0300, Andrey V. Elsukov wrote: >>>>> On 12.08.16 03:26, Bryan Drewery wrote: >>>>>> On r303467 I ran into this: >>>>>> >>>>>> panic @ time 1470916206.652, thread 0xfffff8000412f000: >>>>>> g_resize_provider_event but withered >>>>>> cpuid =3D 0 >>>>>> Panic occurred in module kernel loaded at 0xffffffff80200000: >>>>>> >>>>>> Stack: -------------------------------------------------- >>>>>> kernel:kassert_panic+0x166 >>>>>> kernel:g_resize_provider_event+0x181 >>>>>> kernel:g_run_events+0x186^M^M >>>>>> kernel:fork_exit+0x83^M^M >>>>>> -------------------------------------------------- >>>>>> >>>>>> No further information available unfortunately. >>>>> >>>>> This one is related to r302087 :) >>>> >>>> It looks like there is a race. I think we need to replace the KASSE= RT >>>> in g_resize_provider_event() with a return in case the provider is >>>> withered. >>>> >>>> I won't be able to work on or test this until sometime next week. S= o if >>>> you guys want to go ahead and make the change, please do. >>> >>> But why are we calling g_resize_provider on a withered object? That's= >>> the part I don't understand in this thread. >> >> It isn't withered when the event is queued, but it is withered by the = time >> the event is executed. >> >> There is a check in g_resize_provider() to make sure it isn't withered= =2E If >> not, the event is queued. But once g_resize_provider_event() runs, it= is >> withered and we run into the KASSERT. >> >> There isn't adequate locking and ordering in there to prevent the race= >> from happening, so the assert should be replaced with an "if (withered= ) >> return" statement. >=20 > I'll grant that we may wither with outstanding events, but why is it > withering? That seems odd. Either we're bogusly posting this event > just before it will wither, or something else is bogusly withering it. > Just removing the assert isn't going to fix the underlying issue. >=20 > Back to Bryan: just to be clear, this is with the latest version of > the code, and not the intermediate version that was fixed after > numerous problems surfaced, right? >=20 No, I was missing r303637. Hard to say if it is related... Andrey says it's not. I haven't dived into it yet and it's so far only happened once (out of a few tests). We do have various customizations but I'm inclined to think it's the stock code having problems. --=20 Regards, Bryan Drewery --srVN6Bwftn190dQEXwoKeebQISJUiN0dC-- --cUcpUOGMI3scLeW38BQE1FqfCJfLSM1TH Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQEcBAEBCgAGBQJXreoAAAoJEDXXcbtuRpfPivgIAJPcWlrUrEp21ocPeeIpFiWL OWx/And4RE/SOtH9+355Bf30mQnSJUQKxZM7Wo/PNfXhVmkaqbLaiJcu+3C4FTJL VUXpuP1CEzuzpveWbhYy2b4aHa8ape47qBnVjjnNe+cFtPXKeOge6Ymscu9NbHJf 0xSUgMx/3xexOHpVAnUrEHzhNsP3f8yKN6HJN5GF0Sq7SkpG7n35b3u7oEuSDn4z KYzZ8x2qjo8p6VMkMds+fHaS5Dn73EYPXYpRfEtRJdo3+ZcJFTyZQp8EvjQf+mMY eoMoBc5a1vcx0hcsxMB7XX6+2RWUuewLgJDdjRObr2RMy/yHA0XEEGnlgewfYDE= =DYMe -----END PGP SIGNATURE----- --cUcpUOGMI3scLeW38BQE1FqfCJfLSM1TH--