* Andi Kleen <[email protected]> wrote:
> > * This grunge runs the startup process for
> > * the targeted processor.
> > */
> > + cpu_gdt_descr[cpu].address = __get_free_page(GFP_KERNEL|__GFP_ZERO);
>
> I can see why don't check it for NULL, but it's a ugly reason and
> would be better fixed. It at least needs a comment.
it's so early in the bootup that any failure here would probably be
fatal anyway. But yeah, a comment would be nice.
> -Andi (who would still prefer just going back to the array in head.S -
> would work as well and waste less memory)
that doesnt really solve the problem for e.g. Xen, which needs a
separate page for each GDT. (xenolinux is a separate arch right now, but
it will/should be merged back into its base architectures) So why not
solve all the problems at once?
Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
| |
 |