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;
Date:   Thu, 16 May 2019 07:57:08 +0200
From:   Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To:     "Tobin C. Harding" <me@tobin.cc>
Cc:     linux-kernel@vger.kernel.org, stable@vger.kernel.org,
        "Tobin C. Harding" <tobin@kernel.org>,
        "David S. Miller" <davem@davemloft.net>
Subject: Re: [PATCH 3.18 78/86] bridge: Fix error path for
 kobject_init_and_add()
Message-ID: <20190516055708.GA12518@kroah.com>
References: <20190515090642.339346723@linuxfoundation.org>
 <20190515090655.622147146@linuxfoundation.org>
 <20190515204840.GE11749@eros.localdomain>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20190515204840.GE11749@eros.localdomain>
User-Agent: Mutt/1.11.4 (2019-03-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, May 16, 2019 at 06:48:40AM +1000, Tobin C. Harding wrote:
> On Wed, May 15, 2019 at 12:55:55PM +0200, Greg Kroah-Hartman wrote:
> > From: "Tobin C. Harding" <tobin@kernel.org>
> > 
> > [ Upstream commit bdfad5aec1392b93495b77b864d58d7f101dc1c1 ]
> 
> Greg you are not going to back port all of these kobject fixes are you?
> There is going to be a _lot_ of them.  I'm not super comfortable
> generating all this work for you.  And besides that, I keep making
> mistakes (reference to last nights find of double free in powerpc that
> you reviewed already), then we have to back port those too.
> 
> For the record I've been going through all uses of kobject and splitting
> them into categories
> 
>  1. Broken
>  2. Too complex to immediately tell
>  3. Done correctly
> 
> I'm not getting many in category #3, let's hope that some in #1 and #2 are
> my misunderstanding and that many in #2 should be in #3.  I'm having fun
> fixing them but I shudder at making life hard for other people.

I took this one as it was forwarded on to me by David Miller as a fix to
be queued up for networking issues.

If a maintainer wants to mark the patch for stable, I'll be glad to take
it, but I'm not going to be going and digging all of these out by hand
an backporting them :)

thanks,

greg k-h