Re: [PATCH v7u1 23/31] x86, boot: update comments about entries for64bit image

From: Yinghai Lu
Date: Mon Jan 14 2013 - 13:37:25 EST


On Mon, Jan 14, 2013 at 10:35 AM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote:
> On Mon, Jan 14, 2013 at 3:20 AM, Borislav Petkov <bp@xxxxxxxxx> wrote:
>>> the boot loader should
>>> +also fill the additional fields of the struct boot_params as that
>>
>> remove "that"
>>
>>> +described in zero-page.txt.
>>
>> Btw, you could document the sentinel mechanism here or in zero-page.txt,
>> for example.
>
>
> that is:
> ---
> The memory for struct boot_params
> could be allocated anywhere (even above 4G) and initialized to all zero.
> ---
>
> bootloader memset zero_page at first means.
> and all bootloader did that except kexec. and it is fixed patches for
> kexec-tools.

also I fix some errors that you mentioned, but not all of them
because I copied 32bit portion to 64bit and modified it.

So you may submit patch to fix 32bit and 64bit at the same time if you like.

Thanks

Yinghai
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/