Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Wed, Sep 28, 2016 at 10:29:23AM -0300, Mauricio Faria de Oliveira wrote:
> On 09/28/2016 05:45 AM, Andrew Donnellan wrote:
> > 2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl
> > case. Mauricio - any objections from the pseries/dlpar side?
> 
> No objections.

Thanks, have queued them up now.

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Wed, Sep 28, 2016 at 10:29:23AM -0300, Mauricio Faria de Oliveira wrote:
> On 09/28/2016 05:45 AM, Andrew Donnellan wrote:
> > 2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl
> > case. Mauricio - any objections from the pseries/dlpar side?
> 
> No objections.

Thanks, have queued them up now.

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Mauricio Faria de Oliveira]

On 09/28/2016 05:45 AM, Andrew Donnellan wrote:

2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl
case. Mauricio - any objections from the pseries/dlpar side?


No objections.


--
Mauricio Faria de Oliveira
IBM Linux Technology Center

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Mauricio Faria de Oliveira]

On 09/28/2016 05:45 AM, Andrew Donnellan wrote:

2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl
case. Mauricio - any objections from the pseries/dlpar side?


No objections.


--
Mauricio Faria de Oliveira
IBM Linux Technology Center

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Andrew Donnellan]

On 24/09/16 01:30, Greg Kroah-Hartman wrote:

On Fri, Sep 23, 2016 at 05:11:48PM +0200, Greg Kroah-Hartman wrote:

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...


Should be removed, hopefully all will build properly...



It would've been helpful if Mauricio and I were Cc'd on this, but in any 
case I was away for most of last week :)


Looks like I suggested that 2dd9c11b9d4 be Cc: stable on the list but 
that didn't quite make it in to the final commit. Probably didn't help 
that our patches were then applied to powerpc#fixes in the wrong order...


2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl 
case. Mauricio - any objections from the pseries/dlpar side?



Andrew


--
Andrew Donnellan  OzLabs, ADL Canberra
andrew.donnel...@au1.ibm.com  IBM Australia Limited

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Andrew Donnellan]

On 24/09/16 01:30, Greg Kroah-Hartman wrote:

On Fri, Sep 23, 2016 at 05:11:48PM +0200, Greg Kroah-Hartman wrote:

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...


Should be removed, hopefully all will build properly...



It would've been helpful if Mauricio and I were Cc'd on this, but in any 
case I was away for most of last week :)


Looks like I suggested that 2dd9c11b9d4 be Cc: stable on the list but 
that didn't quite make it in to the final commit. Probably didn't help 
that our patches were then applied to powerpc#fixes in the wrong order...


2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl 
case. Mauricio - any objections from the pseries/dlpar side?



Andrew


--
Andrew Donnellan  OzLabs, ADL Canberra
andrew.donnel...@au1.ibm.com  IBM Australia Limited

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Shuah Khan]

On 09/22/2016 11:28 AM, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.4.22 release.
> There are 118 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.
> 
> Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> Anything received after that time might be too late.
> 
> The whole patch series can be found in one patch at:
>   kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.22-rc1.gz
> or in the git tree and branch at:
>   git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git 
> linux-4.4.y
> and the diffstat can be found below.
> 
> thanks,
> 
> greg k-h
> 

Compiled and booted on my test system. No dmesg regressions.

thanks,
-- Shuah


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America(Silicon Valley)
shuah...@samsung.com

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Shuah Khan]

On 09/22/2016 11:28 AM, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.4.22 release.
> There are 118 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.
> 
> Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> Anything received after that time might be too late.
> 
> The whole patch series can be found in one patch at:
>   kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.22-rc1.gz
> or in the git tree and branch at:
>   git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git 
> linux-4.4.y
> and the diffstat can be found below.
> 
> thanks,
> 
> greg k-h
> 

Compiled and booted on my test system. No dmesg regressions.

thanks,
-- Shuah


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America(Silicon Valley)
shuah...@samsung.com

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Fri, Sep 23, 2016 at 05:11:48PM +0200, Greg Kroah-Hartman wrote:
> 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...

Should be removed, hopefully all will build properly...

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Fri, Sep 23, 2016 at 05:11:48PM +0200, Greg Kroah-Hartman wrote:
> 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...

Should be removed, hopefully all will build properly...

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

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

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

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

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Guenter Roeck]

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.

Guenter

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Guenter Roeck]

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.

Guenter

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Fri, Sep 23, 2016 at 10:43:27AM +0200, Holger Hoffstätte wrote:
> On 09/23/16 10:14, Greg Kroah-Hartman wrote:
> > On Thu, Sep 22, 2016 at 09:56:28PM +0200, Holger Hoffstätte wrote:
> >> did you forget to add the -net patches in stable-queue.git/tree/net-4.4
> >> or are they on hold for some reason? I think they were supposed to go
> >> into .22.
> > 
> > Really?  I just put them there as I was pondering applying them, waiting
> > for the network maintainer to send me some more patches, or not.  They
> 
> Ahh..alright then.
> 
> > Why do you think they should go into this release?
> 
> Never mind - they are not really urgent. I just had most of them merged
> locally already and was just wondering.

If you have them merged, and tested, mind sending them to me with your
tested-by mark on them?  That way I know they work and can focus on the
ones that will require backporting effort.

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Fri, Sep 23, 2016 at 10:43:27AM +0200, Holger Hoffstätte wrote:
> On 09/23/16 10:14, Greg Kroah-Hartman wrote:
> > On Thu, Sep 22, 2016 at 09:56:28PM +0200, Holger Hoffstätte wrote:
> >> did you forget to add the -net patches in stable-queue.git/tree/net-4.4
> >> or are they on hold for some reason? I think they were supposed to go
> >> into .22.
> > 
> > Really?  I just put them there as I was pondering applying them, waiting
> > for the network maintainer to send me some more patches, or not.  They
> 
> Ahh..alright then.
> 
> > Why do you think they should go into this release?
> 
> Never mind - they are not really urgent. I just had most of them merged
> locally already and was just wondering.

If you have them merged, and tested, mind sending them to me with your
tested-by mark on them?  That way I know they work and can focus on the
ones that will require backporting effort.

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Holger Hoffstätte]

On 09/23/16 10:14, Greg Kroah-Hartman wrote:
> On Thu, Sep 22, 2016 at 09:56:28PM +0200, Holger Hoffstätte wrote:
>> did you forget to add the -net patches in stable-queue.git/tree/net-4.4
>> or are they on hold for some reason? I think they were supposed to go
>> into .22.
> 
> Really?  I just put them there as I was pondering applying them, waiting
> for the network maintainer to send me some more patches, or not.  They

Ahh..alright then.

> Why do you think they should go into this release?

Never mind - they are not really urgent. I just had most of them merged
locally already and was just wondering.

thanks.

Holger

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Holger Hoffstätte]

On 09/23/16 10:14, Greg Kroah-Hartman wrote:
> On Thu, Sep 22, 2016 at 09:56:28PM +0200, Holger Hoffstätte wrote:
>> did you forget to add the -net patches in stable-queue.git/tree/net-4.4
>> or are they on hold for some reason? I think they were supposed to go
>> into .22.
> 
> Really?  I just put them there as I was pondering applying them, waiting
> for the network maintainer to send me some more patches, or not.  They

Ahh..alright then.

> Why do you think they should go into this release?

Never mind - they are not really urgent. I just had most of them merged
locally already and was just wondering.

thanks.

Holger

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 09:56:28PM +0200, Holger Hoffstätte wrote:
> On 09/22/16 19:28, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.22 release.
> > There are 118 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> 
> Greg,
> 
> did you forget to add the -net patches in stable-queue.git/tree/net-4.4
> or are they on hold for some reason? I think they were supposed to go
> into .22.

Really?  I just put them there as I was pondering applying them, waiting
for the network maintainer to send me some more patches, or not.  They
aren't all correct, and are missing some patches, I just haven't gotten
the chance to do the work yet.

Why do you think they should go into this release?

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 09:56:28PM +0200, Holger Hoffstätte wrote:
> On 09/22/16 19:28, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.22 release.
> > There are 118 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> 
> Greg,
> 
> did you forget to add the -net patches in stable-queue.git/tree/net-4.4
> or are they on hold for some reason? I think they were supposed to go
> into .22.

Really?  I just put them there as I was pondering applying them, waiting
for the network maintainer to send me some more patches, or not.  They
aren't all correct, and are missing some patches, I just haven't gotten
the chance to do the work yet.

Why do you think they should go into this release?

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 08:01:35PM -0700, Guenter Roeck wrote:
> On 09/22/2016 10:28 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.22 release.
> > There are 118 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> > 
> > Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> > Anything received after that time might be too late.
> > 
> 
> openrisc:
> 
> upstream patch 8e4b72054f55 ("openrisc: fix the fix of copy_from_user()")
>   is needed in both v4.4 and v4.7 to fix the runtime problem.

Now applied.

So, I think we should have fixed all of the build issues, but I do still
have that question for you about the cxl driver that I don't understand.

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 08:01:35PM -0700, Guenter Roeck wrote:
> On 09/22/2016 10:28 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.22 release.
> > There are 118 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> > 
> > Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> > Anything received after that time might be too late.
> > 
> 
> openrisc:
> 
> upstream patch 8e4b72054f55 ("openrisc: fix the fix of copy_from_user()")
>   is needed in both v4.4 and v4.7 to fix the runtime problem.

Now applied.

So, I think we should have fixed all of the build issues, but I do still
have that question for you about the cxl driver that I don't understand.

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 08:01:35PM -0700, Guenter Roeck wrote:
> On 09/22/2016 10:28 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.22 release.
> > There are 118 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> > 
> > Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> > Anything received after that time might be too late.
> > 
> 
> avr32:
> 
> upstream patch 65c0044ca8d7 ("avr32: fix 'undefined reference to 
> `___copy_from_user'")
>   is needed in both v4.4 and v4.7 to fix the build problem.

Now applied, thanks.

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 08:01:35PM -0700, Guenter Roeck wrote:
> On 09/22/2016 10:28 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.22 release.
> > There are 118 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> > 
> > Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> > Anything received after that time might be too late.
> > 
> 
> avr32:
> 
> upstream patch 65c0044ca8d7 ("avr32: fix 'undefined reference to 
> `___copy_from_user'")
>   is needed in both v4.4 and v4.7 to fix the build problem.

Now applied, thanks.

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

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?

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

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?

thanks,

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 08:01:35PM -0700, Guenter Roeck wrote:
> ppc:
> 
> 5b675d932899 ("ppc32: fix copy_from_user()")
>   is corrupt in both v4.4 and v4.7.

Ugh, my fault, now fixed up.

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

On Thu, Sep 22, 2016 at 08:01:35PM -0700, Guenter Roeck wrote:
> ppc:
> 
> 5b675d932899 ("ppc32: fix copy_from_user()")
>   is corrupt in both v4.4 and v4.7.

Ugh, my fault, now fixed up.

greg k-h

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Guenter Roeck]

On 09/22/2016 10:28 AM, Greg Kroah-Hartman wrote:

This is the start of the stable review cycle for the 4.4.22 release.
There are 118 patches in this series, all will be posted as a response
to this one.  If anyone has any issues with these being applied, please
let me know.

Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
Anything received after that time might be too late.



avr32:

upstream patch 65c0044ca8d7 ("avr32: fix 'undefined reference to 
`___copy_from_user'")
is needed in both v4.4 and v4.7 to fix the build problem.

ppc:

5b675d932899 ("ppc32: fix copy_from_user()")
is corrupt in both v4.4 and v4.7.

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 ???).

openrisc:

upstream patch 8e4b72054f55 ("openrisc: fix the fix of copy_from_user()")
is needed in both v4.4 and v4.7 to fix the runtime problem.

Guenter

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Guenter Roeck]

On 09/22/2016 10:28 AM, Greg Kroah-Hartman wrote:

This is the start of the stable review cycle for the 4.4.22 release.
There are 118 patches in this series, all will be posted as a response
to this one.  If anyone has any issues with these being applied, please
let me know.

Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
Anything received after that time might be too late.



avr32:

upstream patch 65c0044ca8d7 ("avr32: fix 'undefined reference to 
`___copy_from_user'")
is needed in both v4.4 and v4.7 to fix the build problem.

ppc:

5b675d932899 ("ppc32: fix copy_from_user()")
is corrupt in both v4.4 and v4.7.

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 ???).

openrisc:

upstream patch 8e4b72054f55 ("openrisc: fix the fix of copy_from_user()")
is needed in both v4.4 and v4.7 to fix the runtime problem.

Guenter

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Guenter Roeck]

On Thu, Sep 22, 2016 at 07:28:20PM +0200, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.4.22 release.
> There are 118 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.
> 
> Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> Anything received after that time might be too late.
> 
Build results:
total: 149 pass: 136 fail: 13
Failed builds:
avr32:defconfig
avr32:allnoconfig
avr32:merisc_defconfig
avr32:atngw100mkii_evklcd101_defconfig
powerpc:defconfig
powerpc:allmodconfig
powerpc:allnoconfig
powerpc:ppc6xx_defconfig
powerpc:mpc83xx_defconfig
powerpc:tqm8xx_defconfig
powerpc:85xx/sbc8548_defconfig
powerpc:83xx/mpc834x_mds_defconfig
powerpc:86xx/sbc8641d_defconfig

Qemu test results:
total: 101 pass: 90 fail: 11
Failed tests:
openrisc:or1ksim_defconfig
powerpc:mac99:nosmp:ppc_book3s_defconfig
powerpc:g3beige:nosmp:ppc_book3s_defconfig
powerpc:mac99:smp:ppc_book3s_defconfig
powerpc:virtex-ml507:44x/virtex5_defconfig
powerpc:mpc8548cds:85xx/mpc85xx_cds_defconfig
powerpc:mpc8548cds:smpdev:85xx/mpc85xx_cds_defconfig
powerpc:bamboo:44x/bamboo_defconfig
powerpc:mac99:ppc64_book3s_defconfig:nosmp
powerpc:mac99:ppc64_book3s_defconfig:smp4
powerpc:pseries:pseries_defconfig

Build errors:

avr32:

arch/avr32/kernel/built-in.o: In function `arch_ptrace':
(.text+0x810): undefined reference to `___copy_from_user'
arch/avr32/kernel/built-in.o:(___ksymtab+___copy_from_user+0x0): undefined
reference to `___copy_from_user'
kernel/built-in.o: In function `devm_request_resource':
(.text+0x52c8): undefined reference to `___copy_from_user'
kernel/built-in.o: In function `proc_do_large_bitmap':
(.text+0x588c): undefined reference to `___copy_from_user'
kernel/built-in.o: In function `proc_dostring':
(.text+0x5b20): undefined reference to `___copy_from_user'
kernel/built-in.o:sysctl.c:(.text+0x6088): more undefined references to
`___copy_from_user' follow

Hmm .. I've seen that before. Looks like a missing commit from upstream.
I'll check later tonight.

---

powerpc:

drivers/misc/cxl/vphb.c:263:9: error: 'pcibios_free_controller_deferred'
undeclared

arch/powerpc/include/asm/uaccess.h: In function 'copy_from_user':
arch/powerpc/include/asm/uaccess.h:328:1: error: wrong type argument to unary 
plus
 + memset(to, 0, n);

I'll have to look into those.

---

runtime:

qemu openrisc crashes with a NULL pointer dereference. There is no backtrace;
I'll have to bisect.

qemu ppc all fail to build with "drivers/misc/cxl/vphb.c:263:9: error:
'pcibios_free_controller_deferred' undeclared".


Details are available at http://kerneltests.org/builders.

Guenter

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Guenter Roeck]

On Thu, Sep 22, 2016 at 07:28:20PM +0200, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.4.22 release.
> There are 118 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.
> 
> Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
> Anything received after that time might be too late.
> 
Build results:
total: 149 pass: 136 fail: 13
Failed builds:
avr32:defconfig
avr32:allnoconfig
avr32:merisc_defconfig
avr32:atngw100mkii_evklcd101_defconfig
powerpc:defconfig
powerpc:allmodconfig
powerpc:allnoconfig
powerpc:ppc6xx_defconfig
powerpc:mpc83xx_defconfig
powerpc:tqm8xx_defconfig
powerpc:85xx/sbc8548_defconfig
powerpc:83xx/mpc834x_mds_defconfig
powerpc:86xx/sbc8641d_defconfig

Qemu test results:
total: 101 pass: 90 fail: 11
Failed tests:
openrisc:or1ksim_defconfig
powerpc:mac99:nosmp:ppc_book3s_defconfig
powerpc:g3beige:nosmp:ppc_book3s_defconfig
powerpc:mac99:smp:ppc_book3s_defconfig
powerpc:virtex-ml507:44x/virtex5_defconfig
powerpc:mpc8548cds:85xx/mpc85xx_cds_defconfig
powerpc:mpc8548cds:smpdev:85xx/mpc85xx_cds_defconfig
powerpc:bamboo:44x/bamboo_defconfig
powerpc:mac99:ppc64_book3s_defconfig:nosmp
powerpc:mac99:ppc64_book3s_defconfig:smp4
powerpc:pseries:pseries_defconfig

Build errors:

avr32:

arch/avr32/kernel/built-in.o: In function `arch_ptrace':
(.text+0x810): undefined reference to `___copy_from_user'
arch/avr32/kernel/built-in.o:(___ksymtab+___copy_from_user+0x0): undefined
reference to `___copy_from_user'
kernel/built-in.o: In function `devm_request_resource':
(.text+0x52c8): undefined reference to `___copy_from_user'
kernel/built-in.o: In function `proc_do_large_bitmap':
(.text+0x588c): undefined reference to `___copy_from_user'
kernel/built-in.o: In function `proc_dostring':
(.text+0x5b20): undefined reference to `___copy_from_user'
kernel/built-in.o:sysctl.c:(.text+0x6088): more undefined references to
`___copy_from_user' follow

Hmm .. I've seen that before. Looks like a missing commit from upstream.
I'll check later tonight.

---

powerpc:

drivers/misc/cxl/vphb.c:263:9: error: 'pcibios_free_controller_deferred'
undeclared

arch/powerpc/include/asm/uaccess.h: In function 'copy_from_user':
arch/powerpc/include/asm/uaccess.h:328:1: error: wrong type argument to unary 
plus
 + memset(to, 0, n);

I'll have to look into those.

---

runtime:

qemu openrisc crashes with a NULL pointer dereference. There is no backtrace;
I'll have to bisect.

qemu ppc all fail to build with "drivers/misc/cxl/vphb.c:263:9: error:
'pcibios_free_controller_deferred' undeclared".


Details are available at http://kerneltests.org/builders.

Guenter

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Holger Hoffstätte]

On 09/22/16 19:28, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.4.22 release.
> There are 118 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.

Greg,

did you forget to add the -net patches in stable-queue.git/tree/net-4.4
or are they on hold for some reason? I think they were supposed to go
into .22.

Holger

Re: [PATCH 4.4 000/118] 4.4.22-stable review

[Holger Hoffstätte]

On 09/22/16 19:28, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.4.22 release.
> There are 118 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.

Greg,

did you forget to add the -net patches in stable-queue.git/tree/net-4.4
or are they on hold for some reason? I think they were supposed to go
into .22.

Holger

[PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

This is the start of the stable review cycle for the 4.4.22 release.
There are 118 patches in this series, all will be posted as a response
to this one.  If anyone has any issues with these being applied, please
let me know.

Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
Anything received after that time might be too late.

The whole patch series can be found in one patch at:
kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.22-rc1.gz
or in the git tree and branch at:
  git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git 
linux-4.4.y
and the diffstat can be found below.

thanks,

greg k-h

-
Pseudo-Shortlog of commits:

Greg Kroah-Hartman 
Linux 4.4.22-rc1

Al Viro 
ia64: copy_from_user() should zero the destination on access_ok() failure

Thomas Gleixner 
genirq/msi: Fix broken debug output

Al Viro 
ppc32: fix copy_from_user()

Al Viro 
sparc32: fix copy_from_user()

Al Viro 
mn10300: copy_from_user() should zero on access_ok() failure...

Al Viro 
nios2: copy_from_user() should zero the tail of destination

Al Viro 
openrisc: fix copy_from_user()

Al Viro 
parisc: fix copy_from_user()

Al Viro 
metag: copy_from_user() should zero the destination on access_ok() failure

Al Viro 
alpha: fix copy_from_user()

Al Viro 
asm-generic: make copy_from_user() zero the destination properly

Al Viro 
mips: copy_from_user() must zero the destination on access_ok() failure

Al Viro 
hexagon: fix strncpy_from_user() error return

Al Viro 
sh: fix copy_from_user()

Al Viro 
score: fix copy_from_user() and friends

Al Viro 
blackfin: fix copy_from_user()

Al Viro 
cris: buggered copy_from_user/copy_to_user/clear_user

Al Viro 
frv: fix clear_user()

Al Viro 
asm-generic: make get_user() clear the destination on errors

Vineet Gupta 
ARC: uaccess: get_user to zero out dest in cause of fault

Al Viro 
s390: get_user() should zero on failure

Al Viro 
score: fix __get_user/get_user

Al Viro 
nios2: fix __get_user()

Al Viro 
sh64: failing __get_user() should zero

Al Viro 
m32r: fix __get_user()

Al Viro 
mn10300: failing __get_user() and get_user() should zero

Al Viro 
fix minor infoleak in get_user_ex()

Al Viro 
microblaze: fix copy_from_user()

Al Viro 
avr32: fix copy_from_user()

Al Viro 
microblaze: fix __get_user()

Al Viro 
fix iov_iter_fault_in_readable()

Boris Brezillon 
irqchip/atmel-aic: Fix potential deadlock in ->xlate()

Boris Brezillon 
genirq: Provide irq_gc_{lock_irqsave,unlock_irqrestore}() helpers

Kristian H. Kristensen 
drm: Only use compat ioctl for addfb2 on X86/IA64

Jan Leupold 
drm: atmel-hlcdc: Fix vertical scaling

Arnd Bergmann 
net: simplify napi_synchronize() to avoid warnings

Arnd Bergmann 
kconfig: tinyconfig: provide whole choice blocks to avoid warnings

Arnd Bergmann 
soc: qcom/spm: shut up uninitialized variable warning

Arnd Bergmann 
pinctrl: at91-pio4: use %pr format string for resource

Arnd Bergmann 
mmc: dw_mmc: use resource_size_t to store physical address

Mika Kuoppala 
drm/i915: Avoid pointer arithmetic in calculating plane surface offset

Mike Danese 
mpssd: fix buffer overflow warning

Linus Torvalds 
gma500: remove annoying deprecation warning

Wei Yongjun 
ipv6: addrconf: fix dev refcont leak when DAD failed

Balbir Singh 
sched/core: Fix a race between try_to_wake_up() and a woken up task

Johannes Berg 
Revert "wext: Fix 32 bit iwpriv compatibility issue with 64 bit Kernel"

Felix Fietkau 
ath9k: fix using sta->drv_priv before initializing it

Guoqing Jiang 
md-cluster: make md-cluster also can work when compiled into kernel

Mathias Nyman 

[PATCH 4.4 000/118] 4.4.22-stable review

[Greg Kroah-Hartman]

This is the start of the stable review cycle for the 4.4.22 release.
There are 118 patches in this series, all will be posted as a response
to this one.  If anyone has any issues with these being applied, please
let me know.

Responses should be made by Sat Sep 24 17:29:17 UTC 2016.
Anything received after that time might be too late.

The whole patch series can be found in one patch at:
kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.22-rc1.gz
or in the git tree and branch at:
  git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git 
linux-4.4.y
and the diffstat can be found below.

thanks,

greg k-h

-
Pseudo-Shortlog of commits:

Greg Kroah-Hartman 
Linux 4.4.22-rc1

Al Viro 
ia64: copy_from_user() should zero the destination on access_ok() failure

Thomas Gleixner 
genirq/msi: Fix broken debug output

Al Viro 
ppc32: fix copy_from_user()

Al Viro 
sparc32: fix copy_from_user()

Al Viro 
mn10300: copy_from_user() should zero on access_ok() failure...

Al Viro 
nios2: copy_from_user() should zero the tail of destination

Al Viro 
openrisc: fix copy_from_user()

Al Viro 
parisc: fix copy_from_user()

Al Viro 
metag: copy_from_user() should zero the destination on access_ok() failure

Al Viro 
alpha: fix copy_from_user()

Al Viro 
asm-generic: make copy_from_user() zero the destination properly

Al Viro 
mips: copy_from_user() must zero the destination on access_ok() failure

Al Viro 
hexagon: fix strncpy_from_user() error return

Al Viro 
sh: fix copy_from_user()

Al Viro 
score: fix copy_from_user() and friends

Al Viro 
blackfin: fix copy_from_user()

Al Viro 
cris: buggered copy_from_user/copy_to_user/clear_user

Al Viro 
frv: fix clear_user()

Al Viro 
asm-generic: make get_user() clear the destination on errors

Vineet Gupta 
ARC: uaccess: get_user to zero out dest in cause of fault

Al Viro 
s390: get_user() should zero on failure

Al Viro 
score: fix __get_user/get_user

Al Viro 
nios2: fix __get_user()

Al Viro 
sh64: failing __get_user() should zero

Al Viro 
m32r: fix __get_user()

Al Viro 
mn10300: failing __get_user() and get_user() should zero

Al Viro 
fix minor infoleak in get_user_ex()

Al Viro 
microblaze: fix copy_from_user()

Al Viro 
avr32: fix copy_from_user()

Al Viro 
microblaze: fix __get_user()

Al Viro 
fix iov_iter_fault_in_readable()

Boris Brezillon 
irqchip/atmel-aic: Fix potential deadlock in ->xlate()

Boris Brezillon 
genirq: Provide irq_gc_{lock_irqsave,unlock_irqrestore}() helpers

Kristian H. Kristensen 
drm: Only use compat ioctl for addfb2 on X86/IA64

Jan Leupold 
drm: atmel-hlcdc: Fix vertical scaling

Arnd Bergmann 
net: simplify napi_synchronize() to avoid warnings

Arnd Bergmann 
kconfig: tinyconfig: provide whole choice blocks to avoid warnings

Arnd Bergmann 
soc: qcom/spm: shut up uninitialized variable warning

Arnd Bergmann 
pinctrl: at91-pio4: use %pr format string for resource

Arnd Bergmann 
mmc: dw_mmc: use resource_size_t to store physical address

Mika Kuoppala 
drm/i915: Avoid pointer arithmetic in calculating plane surface offset

Mike Danese 
mpssd: fix buffer overflow warning

Linus Torvalds 
gma500: remove annoying deprecation warning

Wei Yongjun 
ipv6: addrconf: fix dev refcont leak when DAD failed

Balbir Singh 
sched/core: Fix a race between try_to_wake_up() and a woken up task

Johannes Berg 
Revert "wext: Fix 32 bit iwpriv compatibility issue with 64 bit Kernel"

Felix Fietkau 
ath9k: fix using sta->drv_priv before initializing it

Guoqing Jiang 
md-cluster: make md-cluster also can work when compiled into kernel

Mathias Nyman 
xhci: fix null pointer dereference in stop command timeout function

Miklos Szeredi 
fuse: direct-io: don't dirty ITER_BVEC pages

Chris Mason 
Btrfs: remove root_log_ctx from ctx list before btrfs_sync_log returns

Ard Biesheuvel 
crypto: cryptd - initialize child shash_desc on import

Will Deacon 
arm64: spinlocks: implement smp_mb__before_spinlock() as smp_mb()

Icenowy Zheng 
pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on A23/A33

James Hartley 
pinctrl: pistachio: fix mfio pll_lock pinmux

Mikulas Patocka 
dm crypt: fix error with too large bios

Mikulas Patocka 
dm log writes: move IO accounting earlier to fix error path

Vladimir Zapolskiy 
dm log writes: fix check of kthread_run() return value

Pawel Moll 
bus: arm-ccn: Fix XP watchpoint settings bitmask

Pawel Moll 
bus: arm-ccn: Do not attempt to configure XPs for cycle counter

Pawel Moll 
bus: arm-ccn: Fix PMU handling of MN

Lee Jones 
ARM: dts: STiH407-family: Provide interconnect clock for consumption in ST 
SDHCI

Johan Hovold 
ARM: dts: overo: fix gpmc nand on boards with ethernet

Johan Hovold 
ARM: dts: overo: fix gpmc nand cs0 range

Fabio