Re: [PATCH] x86: fix up bootparam.h for userspace inclusion

From: Thomas Gleixner
Date: Sat May 03 2008 - 17:04:45 EST


On Fri, 2 May 2008, Rusty Russell wrote:

> commit 8b664aa66e824a0ddf4ec56d41fa0cf7bb374de6 (x86, boot: add linked
> list of struct setup_data) put a new struct in bootparam.h, but didn't
> use the userspace-safe types.
>
> CC: Huang, Ying <ying.huang@xxxxxxxxx>
> Signed-off-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx>
>
> diff -r 3b015cd747e0 include/asm-x86/bootparam.h
> --- a/include/asm-x86/bootparam.h Fri May 02 09:32:30 2008 +1000
> +++ b/include/asm-x86/bootparam.h Fri May 02 18:35:11 2008 +1000
> @@ -14,10 +14,10 @@
>
> /* extensible setup data list node */
> struct setup_data {
> - u64 next;
> - u32 type;
> - u32 len;
> - u8 data[0];
> + __u64 next;
> + __u32 type;
> + __u32 len;
> + __u8 data[0];
> };
>
> struct setup_header {

Applied, thanks

tglx
--
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/