diff options
| author | Yanmin Zhang <yanmin.zhang@intel.com> | 2004-08-25 18:03:28 +0000 |
|---|---|---|
| committer | Tony Luck <tony.luck@intel.com> | 2004-08-25 18:03:28 +0000 |
| commit | 59c77efa069b9c8153f80dbf76b78784073a098a (patch) | |
| tree | ad07113568cfb6e2e1ff6d225d89d4d2ef01b0b2 /include/linux | |
| parent | 04362c4c457fe43c41bc8d4debba56273e6cdd8d (diff) | |
[IA64] contig.c: Function find_bootmap_location has 2 bugs.
Firstly, if it's done successfully, it should return -1 instead of 1
because its caller, efi_memmap_walk, will end when find_bootmap_location
returns a value smaller than 0.
Secondly, statement "free_start = PAGE_ALIGN(rsvd_region[i].end)" should
been moved forward. free_start needs to be initialized for every loop
iteration. Current implementation is buggy where initialization is
skipped if range_end <= range_start. Skipping initializing will leads
to overlapping bootmap with one of the rsvd_regions and subsequently
kernel hang at boot time.
Singed-off-by:(sic) Zhang Yanmin <yanmin.zhang@intel.com>
Signed-off-by: Yao Jun <junx.yao@intel.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions
