[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160923151148.GA14168@kroah.com>
Date: Fri, 23 Sep 2016 17:11:48 +0200
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Guenter Roeck <linux@...ck-us.net>
Cc: linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
akpm@...ux-foundation.org, shuah.kh@...sung.com,
patches@...nelci.org, ben.hutchings@...ethink.co.uk,
stable@...r.kernel.org
Subject: Re: [PATCH 4.4 000/118] 4.4.22-stable review
On Fri, Sep 23, 2016 at 06:25:33AM -0700, Guenter Roeck wrote:
> On 09/23/2016 01:08 AM, Greg Kroah-Hartman wrote:
> > On Thu, Sep 22, 2016 at 08:01:35PM -0700, Guenter Roeck wrote:
> > > e5767a13d7cb ("cxl: use pcibios_free_controller_deferred() when removing vPHBs")
> > > does not apply to v4.4 nor to v4.7 (and isn't marked stable ???).
> >
> > I don't understand what you mean here. It did apply, and it was marked
> > for stable inclusion. It is commit
> > 6f38a8b9a45833495dc878c335c5431cd98a16ed in Linus's tree. Does it break
> > the build?
> >
>
> drivers/misc/cxl/vphb.c: In function 'cxl_pci_vphb_add':
> drivers/misc/cxl/vphb.c:263:9: error: 'pcibios_free_controller_deferred' undeclared
>
> Yes, it does break the build, by calling a function which does not exist in 4.4 or 4.7.
>
> Sorry, the Cc: stable was dropped in the 4.4/4.7 versions, so I didn't see it.
> I meant to say "it doesn't build".
>
> You will either have to drop e5767a13d7cb. or you'll also need commit 2dd9c11b9d4
> ("powerpc/pseries: use pci_host_bridge.release_fn() to kfree(phb)") from upstream.
> I didn't check though if that applies (or works) with the older kernels.
I'm going to drop it, thanks for pointing this out, I should have caught
it earlier too...
greg k-h
Powered by blists - more mailing lists