Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Message-ID: <1525469881.4114.5.camel@HansenPartnership.com>
Subject: Re: [Ksummit-discuss] bug-introducing patches
From:   James Bottomley <James.Bottomley@HansenPartnership.com>
To:     "Theodore Y. Ts'o" <tytso@mit.edu>,
        Greg KH <gregkh@linuxfoundation.org>
Cc:     "ksummit-discuss@lists.linuxfoundation.org" 
        <ksummit-discuss@lists.linuxfoundation.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "w@1wt.eu" <w@1wt.eu>
Date:   Fri, 04 May 2018 14:38:01 -0700
In-Reply-To: <20180504211317.GK29205@thunk.org>
References: <CAMuHMdW_i-KQpqMz0zsz77tRcE-Sron5zAyh0SHAOJ5ap7x60w@mail.gmail.com>
         <20180503000620.GA29205@thunk.org> <20180503144612.GJ18390@sasha-vm>
         <20180503165446.GB30522@ZenIV.linux.org.uk>
         <20180503173422.GR18390@sasha-vm>
         <20180503182039.GC30522@ZenIV.linux.org.uk>
         <8669.1525427874@warthog.procyon.org.uk> <87fu377gbu.fsf@intel.com>
         <20180504130932.GI29205@thunk.org> <20180504174055.GF4649@kroah.com>
         <20180504211317.GK29205@thunk.org>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, 2018-05-04 at 17:13 -0400, Theodore Y. Ts'o wrote:
> If it's not necessary, fine.  But we should still delete what is
> currently documented in stable_kernel_rules and was introduced in
> 8e9b9362266d, because it doesn't describe current practice.

It definitely doesn't seem to describe current practice.  It looks like
it got applied because the commit description bears a somewhat strange
relation to the actual text that was added:  The commit talks about the
 original script that used to forward to stable (although it got me and
hpa confused) which seems to refer to a tiny deletion and the rest is
adding an Ingo one off proposal for dependencies.

For the record: Greg runs his own script now and I'm not involved.

Current process (at least from the SCSI centric view) is that if we
screw up and forward a commit with missing dependencies to stable via a
cc: tag, it won't apply and Greg tells us to fix it, which we do.  That
seems to be an adequately functional process for the odd times we run
into this.

James