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

From: Dave Young
Date: Sun Sep 30 2018 - 05:27:53 EST


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 <bhelgaas@xxxxxxxxxx>
> >
> > 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);
> >
> > walk_system_ram_res() expects "start, end" arguments that are inclusive,
> > i.e., the range to be walked includes both the start and end addresses.
>
> Looking at the function comment of find_next_iomem_res, the res->end
> should be exclusive, am I missing something?

Oops, you fix it in 2nd patch, I apparently miss that.

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.

Thanks
Dave