Juergen Gross, le mar. 21 déc. 2021 07:16:49 +0100, a ecrit:
> On 21.12.21 00:22, Samuel Thibault wrote:
> > Juergen Gross, le lun. 20 déc. 2021 17:07:10 +0100, a ecrit:
> > > +unsigned long e820_get_max_contig_pages(unsigned long pfn, unsigned long
> > > pages)
> > > +{
> > > + int i;
> > > + unsigned long end;
> > > +
> > > + for ( i = 0; i < e820_entries && e820_map[i].addr < (pfn <<
> > > PAGE_SHIFT);
> >
> > Shouldn't that be addr+size? Otherwise if pfn is in the middle of an
> > e820 entry, we will miss picking up that.
>
> No, we want to check all map entries starting below or at the given pfn.
> The test should be "e820_map[i].addr <= (pfn << PAGE_SHIFT)", of course.
Ah, yes, due to the "<" I mistook it for a loop that skips the entries
before the targetted pfn :)
With <=,
Reviewed-by: Samuel Thibault <samuel.thiba...@ens-lyon.org>
Samuel
