On Mon, Mar 13, 2017 at 12:14:58PM -0300, Marcelo Henrique Cerri wrote: > Reorder the operations in decompress_kernel() to ensure initrd is moved > to a safe location before the bss section is zeroed. > > During decompression bss can overlap with the initrd and this can > corrupt the initrd contents depending on the size of the compressed > kernel (which affects where the initrd is placed by the bootloader) and > the size of the bss section of the decompressor. > > Also use the correct initrd size when checking for overlaps with > parmblock. > > Fixes: 06c0dd72aea3 ([S390] fix boot failures with compressed kernels) > Cc: stable@xxxxxxxxxxxxxxx > Reviewed-by: Joy Latten <joy.latten@xxxxxxxxxxxxx> > Reviewed-by: Vineetha HariPai <vineetha.hari.pai@xxxxxxxxxxxxx> > Signed-off-by: Marcelo Henrique Cerri <marcelo.cerri@xxxxxxxxxxxxx> > --- > arch/s390/boot/compressed/misc.c | 35 +++++++++++++++++++---------------- > 1 file changed, 19 insertions(+), 16 deletions(-) Indeed. I just checked that we currently have 32 bytes within the bss section of the compressed kernel image, where clearing it would corrupt the initrd if would be placed directly behind the image. I will look into the patch tomorrow. Thanks, Heiko -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
