From nobody Mon Apr 10 04:41:12 2023 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4PvxBP362Fz44lZs; Mon, 10 Apr 2023 04:41:17 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from omta001.cacentral1.a.cloudfilter.net (omta001.cacentral1.a.cloudfilter.net [3.97.99.32]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "Client", Issuer "CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4PvxBN3v6Sz47Zq; Mon, 10 Apr 2023 04:41:16 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Authentication-Results: mx1.freebsd.org; none Received: from shw-obgw-4004a.ext.cloudfilter.net ([10.228.9.227]) by cmsmtp with ESMTP id lUbNpxsYuuZMSljL9pDcS5; Mon, 10 Apr 2023 04:41:15 +0000 Received: from spqr.komquats.com ([70.66.148.124]) by cmsmtp with ESMTPA id ljL7pCPLR3fOSljL8pWllI; Mon, 10 Apr 2023 04:41:14 +0000 X-Authority-Analysis: v=2.4 cv=J8G5USrS c=1 sm=1 tr=0 ts=6433936a a=Cwc3rblV8FOMdVN/wOAqyQ==:117 a=Cwc3rblV8FOMdVN/wOAqyQ==:17 a=xqWC_Br6kY4A:10 a=IkcTkHD0fZMA:10 a=dKHAf1wccvYA:10 a=6I5d2MoRAAAA:8 a=NEAV23lmAAAA:8 a=YxBL1-UpAAAA:8 a=EkcXrb_YAAAA:8 a=kenwOhc8DBI5jldCQOAA:9 a=QEXdDO2ut3YA:10 a=IjZwj45LgO3ly-622nXo:22 a=Ia-lj3WSrqcvXOmTRaiG:22 a=LK5xJRSDVpKd5WXXoEvA:22 Received: from slippy.cwsent.com (slippy [10.1.1.91]) by spqr.komquats.com (Postfix) with ESMTP id E065F4964; Sun, 9 Apr 2023 21:41:12 -0700 (PDT) Received: from localhost (localhost [IPv6:::1]) by slippy.cwsent.com (Postfix) with ESMTP id A6F661AF; Sun, 9 Apr 2023 21:41:12 -0700 (PDT) Date: Sun, 9 Apr 2023 21:41:12 -0700 From: Cy Schubert To: Charlie Li Cc: Rick Macklem , Martin Matuska , src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 2a58b312b62f - main - zfs: merge openzfs/zfs@431083f75 Message-ID: <20230409205826.7802259d@cschubert.com> In-Reply-To: <707e4671-d746-aa23-e340-6eb8f50f78c6@freebsd.org> References: <202304031513.333FD6qw014903@gitrepo.freebsd.org> <20230403231444.CF48911F@slippy.cwsent.com> <20230403232549.73E331A2@slippy.cwsent.com> <20230403235851.84C0467@slippy.cwsent.com> <20230404052811.DA2172C1@slippy.cwsent.com> <7c75b934-cb0a-b32e-bc19-b1e15e8cf3aa@freebsd.org> <20230409154042.0685a273@cschubert.com> <707e4671-d746-aa23-e340-6eb8f50f78c6@freebsd.org> Organization: KOMQUATS X-Mailer: Claws Mail 3.19.0 (GTK+ 2.24.33; amd64-portbld-freebsd14.0) List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-main@freebsd.org X-BeenThere: dev-commits-src-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-CMAE-Envelope: MS4xfP8kN/cfLkLnclXeIJBY4MibOoJ90Rl77sNndKyFS/7VylEpSY/MJqe3+vlwj2AOhr3VHsQlpUj7Y+pLB7qCFl/aLFOUHw7uxnAH0oQ7GBCVsnd1uC7t 9XTMqyiuD3nsQHBwbjAcUyv+i3xDjzUjEL0W1HXiZ2iJhqmF/wu6nKDwGUjurWX+zulOkBpTj0qN1gTrCehDUJEsLn1qcEwW8yMT+UukLfVN9FFbkoZVLQja UFW+E35bQjsg96RI2l3S/xOeNqoWEBXY5NjzNGg9chfQO1vWDRAQMgiYUl0IsCLuVWXdQYmeKDOFGhYLB3N86PU8m4G4/phrO8YH80Jk4n2UbKf4zCdysRji xC9HUjXRcMQhhSynoSdlVvrjgwUSuUw6/13OveK7Yj58+8QMwZ8= X-Rspamd-Queue-Id: 4PvxBN3v6Sz47Zq X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; TAGGED_RCPT(0.00)[]; ASN(0.00)[asn:16509, ipnet:3.96.0.0/15, country:US] X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-ThisMailContainsUnwantedMimeParts: N On Sun, 9 Apr 2023 23:25:44 -0400 Charlie Li wrote: > Charlie Li wrote: > > Cy Schubert wrote: =20 > >>> The file corruption was prior to enabling block_cloning but after=20 > >>> this import. > >>> =20 > >> > >> This regression was fixed=C2=A0 by mjg's commit. I'm not sure which (I= 'm=20 > >> presently AFK). > >> =20 > > Both the block_cloning panic and file corruption are still happening as= =20 > > of 351e4592f64b, which is after any such commit here, unless referring= =20 > > to https://github.com/openzfs/zfs/pull/14723 . > > =20 > ...which was committed here prior to 351e4592f64b, so the issues still=20 > persist. >=20 Hmm, interesting. I'm experiencing no such panics nor corruption since the commit. Reading a previous email of yours from today block_cloning is not enabled. Is it possible that before the regression was fixed, while it was wreaking havoc in your zpool, that your zpool became irreversibly corrupted resulting in panics, even with the fixed code? One way, probably the best way, to test would be to revert back to the commit prior to the import. If you still experience panics and corruption, your zpool is damaged. At the moment we don't know if the code is still broken or if it has been fixed but residual damage is still causing creeping rot and panics. I don't know if zpool scrub can fix this -- reading one comment on FreeBSD-current, zpool scrub fails to complete. I'm not convinced, yet, that the problem code has not been fixed. We don't know if the panics are a result of corruption as a result of the regression. Would it be best if we reverted the seven commits to main? I don't know. I could argue it either way. My problems, on four machines, have been fixed by the subsequent commits. We don't know if there are other regressions or if the current problems are due to corruption caused writes prior to patches addressing the regression. Maybe reverting the seven commits and taking a watch for further fallout approach, whether the panics and problems persist post revert. If the problems persist post revert we know for sure the regression has caused some permanent corruption. This is a radical option. IMO, I'm torn whether a revert would be the best approach or not. It has its merits but significant limitations too. --=20 Cheers, Cy Schubert FreeBSD UNIX: Web: https://FreeBSD.org NTP: Web: https://nwtime.org e^(i*pi)+1=3D0