Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: Corey Minyard <tcminyard@gmail.com>
Date:   Wed, 27 Oct 2021 13:20:27 -0500
From:   Corey Minyard <minyard@acm.org>
To:     Sasha Levin <sashal@kernel.org>
Cc:     Anton Lundin <glance@acc.umu.se>,
        openipmi-developer@lists.sourceforge.net,
        LKML <linux-kernel@vger.kernel.org>
Subject: Re: [Openipmi-developer] Issue with panic handling and ipmi
Message-ID: <20211027182027.GG2744412@minyard.net>
Reply-To: minyard@acm.org
References: <20210916163945.GY545073@minyard.net>
 <20210917101419.GE108031@montezuma.acc.umu.se>
 <20210917120758.GA545073@minyard.net>
 <20210917125525.GF108031@montezuma.acc.umu.se>
 <20210917131916.GB545073@minyard.net>
 <20210917132648.GG108031@montezuma.acc.umu.se>
 <20210920113802.GC545073@minyard.net>
 <20210920141231.GH108031@montezuma.acc.umu.se>
 <20210920144146.GD545073@minyard.net>
 <YXmTbYhFvDJ0m5KX@sashalap>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YXmTbYhFvDJ0m5KX@sashalap>
Precedence: bulk

On Wed, Oct 27, 2021 at 01:59:09PM -0400, Sasha Levin wrote:
> On Mon, Sep 20, 2021 at 09:41:46AM -0500, Corey Minyard wrote:
> > On Mon, Sep 20, 2021 at 04:12:31PM +0200, Anton Lundin wrote:
> > > On 20 September, 2021 - Corey Minyard wrote:
> > > 
> > > > Well, that was dumb.  Fix follows...
> > > >
> > > > Thanks for working on this.  On your approval, I'll send this to Linus.
> > > 
> > > Winner winner chicken dinner!
> > > 
> > > This fixes the issue, and now panic timer works, and we get crashdumps
> > > to pstore.
> > > 
> > > Great job, I approve!
> > > 
> > > 
> > > Thanks for your help getting this fixed.
> > 
> > Thanks for reporting this.  I'll get the patch in.
> 
> Hey Corey,
> 
> Just checking in to see if this patch was lost; I haven't seen it in
> Linus's tree just yet.

I generally wait until the merge window for changes.  It's too late in
the process for a patch now unless it's really critical.

rc7 is out now, the merge window should be opening soon.

-corey