Re: [PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-10-15 Thread Dave Young
On 10/15/18 at 08:44am, Bjorn Helgaas wrote: > On Mon, Oct 15, 2018 at 12:51:38PM +0800, Dave Young wrote: > > On 09/30/18 at 05:27pm, Dave Young wrote: > > > On 09/30/18 at 05:21pm, Dave Young wrote: > > > > On 09/27/18 at 09:21am, Bjorn Helgaas wrote: > > > > > From: Bjorn Helgaas > > > > > >

Re: [PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-10-15 Thread Bjorn Helgaas
On Mon, Oct 15, 2018 at 12:51:38PM +0800, Dave Young wrote: > On 09/30/18 at 05:27pm, Dave Young wrote: > > On 09/30/18 at 05:21pm, Dave Young wrote: > > > On 09/27/18 at 09:21am, Bjorn Helgaas wrote: > > > > From: Bjorn Helgaas > > > > > > > > The only use of KEXEC_BACKUP_SRC_END is as an

Re: [PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-10-15 Thread Borislav Petkov
On Mon, Oct 15, 2018 at 12:51:38PM +0800, Dave Young wrote: > > Since the fix of checking the end is in another patch, probably merge > > these two patches so that they are in one patch to avoid break bisect. > > Not sure if above comment was missed, Boris, would you mind to fold the > patch 1

Re: [PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-10-14 Thread Dave Young
On 09/30/18 at 05:27pm, Dave Young wrote: > On 09/30/18 at 05:21pm, Dave Young wrote: > > Hi Bjorn, > > > > On 09/27/18 at 09:21am, Bjorn Helgaas wrote: > > > From: Bjorn Helgaas > > > > > > The only use of KEXEC_BACKUP_SRC_END is as an argument to > > > walk_system_ram_res(): > > > > > >

Re: [PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-09-30 Thread Dave Young
On 09/30/18 at 05:21pm, Dave Young wrote: > Hi Bjorn, > > On 09/27/18 at 09:21am, Bjorn Helgaas wrote: > > From: Bjorn Helgaas > > > > The only use of KEXEC_BACKUP_SRC_END is as an argument to > > walk_system_ram_res(): > > > > int crash_load_segments(struct kimage *image) > > { > >

Re: [PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-09-30 Thread Dave Young
Hi Bjorn, On 09/27/18 at 09:21am, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > The only use of KEXEC_BACKUP_SRC_END is as an argument to > walk_system_ram_res(): > > int crash_load_segments(struct kimage *image) > { > ... > walk_system_ram_res(KEXEC_BACKUP_SRC_START,

Re: [PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-09-28 Thread Borislav Petkov
On Thu, Sep 27, 2018 at 09:21:55AM -0500, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > The only use of KEXEC_BACKUP_SRC_END is as an argument to > walk_system_ram_res(): > > int crash_load_segments(struct kimage *image) > { > ... > walk_system_ram_res(KEXEC_BACKUP_SRC_START,

[PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-09-27 Thread Bjorn Helgaas
From: Bjorn Helgaas The only use of KEXEC_BACKUP_SRC_END is as an argument to walk_system_ram_res(): int crash_load_segments(struct kimage *image) { ... walk_system_ram_res(KEXEC_BACKUP_SRC_START, KEXEC_BACKUP_SRC_END, image, determine_backup_region);

[PATCH 1/3] x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error

2018-09-24 Thread Bjorn Helgaas
From: Bjorn Helgaas The only use of KEXEC_BACKUP_SRC_END is as an argument to walk_system_ram_res(): int crash_load_segments(struct kimage *image) { ... walk_system_ram_res(KEXEC_BACKUP_SRC_START, KEXEC_BACKUP_SRC_END, image, determine_backup_region);