From owner-svn-src-stable@freebsd.org Mon Apr 16 00:54:20 2018 Return-Path: Delivered-To: svn-src-stable@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 30CCCF9D250; Mon, 16 Apr 2018 00:54:20 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "smtp.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id D1E6C820E9; Mon, 16 Apr 2018 00:54:19 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from mail-lf0-f46.google.com (mail-lf0-f46.google.com [209.85.215.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) (Authenticated sender: kevans) by smtp.freebsd.org (Postfix) with ESMTPSA id 884F920D80; Mon, 16 Apr 2018 00:54:19 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: by mail-lf0-f46.google.com with SMTP id n14-v6so2020494lfe.5; Sun, 15 Apr 2018 17:54:19 -0700 (PDT) X-Gm-Message-State: ALQs6tB0uKQcVMGh52/ZWrjQ2pYDoxHI1q40XK3fqBZOpZ2gOeOsUYJY n2B01Ahjn/WqRbv/PeCoikdZebAC8ui+eWN4SuU= X-Google-Smtp-Source: AIpwx4/jMmVQdCP5N7fr4OLKu5M3/oq+mgS1KWCZozVjgTSjz1n2tFMdewbW2kPX+JZtvo7/sKi1VMKPk7ZSgiYuRzo= X-Received: by 10.46.155.202 with SMTP id w10mr1728200ljj.76.1523840057883; Sun, 15 Apr 2018 17:54:17 -0700 (PDT) MIME-Version: 1.0 Received: by 10.46.129.90 with HTTP; Sun, 15 Apr 2018 17:53:57 -0700 (PDT) In-Reply-To: <201804160028.w3G0S58v048517@pdx.rh.CN85.dnsmgr.net> References: <1523837543.69449.8.camel@freebsd.org> <201804160028.w3G0S58v048517@pdx.rh.CN85.dnsmgr.net> From: Kyle Evans Date: Sun, 15 Apr 2018 19:53:57 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: svn commit: r331722 - in stable/11: bin/cat bin/chflags bin/chmod bin/cp bin/date bin/dd bin/df bin/domainname bin/echo bin/ed bin/hostname bin/kill bin/ln bin/ls bin/mkdir bin/mv bin/pax bin/ps bi... To: "Rodney W. Grimes" Cc: Ian Lepore , Glen Barber , Eitan Adler , src-committers , svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org Content-Type: text/plain; charset="UTF-8" X-BeenThere: svn-src-stable@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for all the -stable branches of the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Apr 2018 00:54:20 -0000 On Sun, Apr 15, 2018 at 7:28 PM, Rodney W. Grimes wrote: >> On Wed, 2018-03-28 at 20:17 -0700, Rodney W. Grimes wrote: >> > -- Start of PGP signed section. >> > > >> > > On Thu, Mar 29, 2018 at 02:51:04AM +0000, Eitan Adler wrote: >> > > > >> > > > Author: eadler >> > > > Date: Thu Mar 29 02:50:57 2018 >> > > > New Revision: 331722 >> > > > URL: https://svnweb.freebsd.org/changeset/base/331722 >> > > > >> > > > Log: >> > > > ? Revert r330897: >> > > > ?? >> > > > ? This was intended to be a non-functional change. It wasn't. The commit >> > > > ? message was thus wrong. In addition it broke arm, and merged crypto >> > > > ? related code. >> > > > ?? >> > > > ? Revert with prejudice. >> > > > ?? >> > > > ? This revert skips files touched in r316370 since that commit was since >> > > > ? MFCed. >> > Um, not sure that was the right way to revert :-(. >> > >> > > >> > > > >> > > > ? This revert also skips files that require $FreeBSD$ property >> > > > ? changes. >> > And more doubts that this is right to skip anything. >> > >> > > >> > > > >> > > > ? Thank you to those who helped me get out of this mess including but not >> > > > ? limited to gonzo, kevans, rgrimes. >> > > > ?? >> > > > ? Requested by: gjb (re) >> > > > >> > > I did *not* request this, especially not on re@'s behalf.??I provided >> > > input on what I felt was the correct way to proceed in this situation, >> > > where a number of commits that were not related to the intent were >> > > included.??There is a difference. >> > I tried to get this slowed down until we could get to a concenses, >> > I think our mess is as big now as it was before. >> > >> >> Or bigger. I've just discovered this commit reverted stuff that wasn't >> part of the bad original MFC commit. Things I MFC'd to 11 before this >> revert are now missing changes and the symptoms don't show up until a >> module fails to load due to missing symbols. >> >> I guess every change on the stable-11 branch between r330897 and >> r331722 needs to be audited to see if MFC'd or changed code has been >> unintentionally reverted. >> >> Here's the example I've just run across... >> >> r330897 merged this: >> >> https://svnweb.freebsd.org/base/stable/11/sys/dev/spibus/ofw_spibus.c?r1=302408&r2=330897 >> >> Then I did some MFCing, then r331722 "reverted" this: >> >> https://svnweb.freebsd.org/base/stable/11/sys/dev/spibus/ofw_spibus.c?r1=331506&r2=331722 > > *sigh* Ok, I can at least do an audit of the r331722 > that was suppose to be a revert and see what else I find. > > I'll start on this tomarrow and get the first pass audit out fairly early > in the morning my time (PST) > > Do we revert the attempted revert, then properly revert the original? I support this effort, as long as we either still don't revert anything touching the files that gonzo went through the trouble of getting back into shape or we also revert gonzo's follow-up MFCs and re-MFC those bits in particular for him. eadler's original commit had included an impartial MFC of some of gonzo's commits (somehow?), hence the follow-up that completed the MFC.