Thread (232 messages) 232 messages, 19 authors, 2018-02-08

Re: [PATCH 4.14 023/159] mm/sparsemem: Allocate mem_section at runtime for CONFIG_SPARSEMEM_EXTREME=y

From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: 2018-01-08 08:33:03
Also in: linux-mm, lkml

On Mon, Jan 08, 2018 at 09:15:33AM +0100, Mike Galbraith wrote:
On Mon, 2018-01-08 at 08:53 +0100, Greg Kroah-Hartman wrote:
quoted
On Sun, Jan 07, 2018 at 02:23:09PM +0100, Michal Hocko wrote:
quoted
On Sun 07-01-18 13:44:02, Mike Galbraith wrote:
quoted
On Sun, 2018-01-07 at 11:18 +0100, Michal Hocko wrote:
quoted
On Sun 07-01-18 10:11:15, Greg KH wrote:
quoted
On Sun, Jan 07, 2018 at 06:14:22AM +0100, Mike Galbraith wrote:
quoted
On Fri, 2017-12-22 at 09:45 +0100, Greg Kroah-Hartman wrote:
quoted
4.14-stable review patch.  If anyone has any objections, please let me know.
FYI, this broke kdump, or rather the makedumpfile part thereof.
�Forward looking wreckage is par for the kdump course, but...
Is it also broken in Linus's tree with this patch?  Or is there an
add-on patch that I should apply to 4.14 to resolve this issue there?
This one http://lkml.kernel.org/r/1513932498-20350-1-git-send-email-bhe@redhat.com
I guess.
That won't unbreak kdump, else master wouldn't be broken. �I don't care
deeply, or know if anyone else does, I'm just reporting it because I
met it and chased it down.
OK, I didn't notice that d8cfbbfa0f7 ("mm/sparse.c: wrong allocation
for mem_section") made it in after rc6. I am still wondering why
83e3c48729 ("mm/sparsemem: Allocate mem_section at runtime for
CONFIG_SPARSEMEM_EXTREME=y") made it into the stable tree in the first
place.
It was part of the prep for the KTPI code from what I can tell.  If you
think it should be reverted, just let me know and I'll be glad to do so.
No preference here. �I have to patch master regardless if I want kdump
to work while I patiently wait for userspace to get fixed up (either
that or use time I don't have to go fix it up myself).
I'll stay "bug compatible" for the time being.  If you do fix this up,
can you add a cc: stable tag in your patch so I can pick it up when it
gets merged?

thanks,

greg k-h

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help