From owner-freebsd-fs@freebsd.org Sun Sep 17 04:16:44 2017 Return-Path: Delivered-To: freebsd-fs@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 334E1E21BC5 for ; Sun, 17 Sep 2017 04:16:44 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 22457711B3 for ; Sun, 17 Sep 2017 04:16:44 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id v8H4Ghb2084036 for ; Sun, 17 Sep 2017 04:16:44 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-fs@FreeBSD.org Subject: [Bug 222288] g_bio leak after zfs ABD commit Date: Sun, 17 Sep 2017 04:16:43 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Base System X-Bugzilla-Component: kern X-Bugzilla-Version: 11.1-STABLE X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Many People X-Bugzilla-Who: fk@fabiankeil.de X-Bugzilla-Status: Open X-Bugzilla-Resolution: X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: avg@FreeBSD.org X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: cc Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-fs@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Filesystems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 17 Sep 2017 04:16:44 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D222288 Fabian Keil changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fk@fabiankeil.de --- Comment #3 from Fabian Keil --- Thanks a lot for the report Dan. I noticed that something was leaking but didn't have time to track it down yet. Thanks to the report I didn't have to. To work around the issue in ElectroBSD I've reverted r321610/a0dddc24c9050 after reverting the follow-up commits that would cause revert conflicts. Your patch looks good to me, Andriy. I've imported it and will test it in the next couple of days. Thanks. It occurred to me that this issue could be easily detected automatically if there was a way to specify a time limit between uma_zalloc() and uma_zfree() calls for a given zone (or item from the zone). Obviously this only works if an upper limit makes sense (and items are expected to be freed), but in case of g_bio I believe that this is the case and there are a bunch of other zones where enforcing allocation time limits should work. I wouldn't be surprised if there were a bunch of other zone item leaks that haven't been detected yet because they don't occur frequently enough to have a big impact. --=20 You are receiving this mail because: You are on the CC list for the bug.=