Re: [PATCH tip:x86/asm] x86: Introduce GDT_ENTRY_INIT()

["H. Peter Anvin" <hpa@zytor.com>]

On 08/08/2009 02:49 AM, Akinobu Mita wrote:
> diff --git a/drivers/char/vr41xx_giu.c b/drivers/char/vr41xx_giu.c
> deleted file mode 100644
> index e69de29..0000000
> diff --git a/drivers/pnp/pnpbios/bioscalls.c b/drivers/pnp/pnpbios/bioscalls.c
> index 45ad3e9..bd035e3 100644
> --- a/drivers/pnp/pnpbios/bioscalls.c
> +++ b/drivers/pnp/pnpbios/bioscalls.c
> @@ -60,7 +60,7 @@ do { \
>  	set_desc_limit(&gdt[(selname) >> 3], (size) - 1); \
>  } while(0)
>  
> -static struct desc_struct bad_bios_desc;
> +static struct desc_struct bad_bios_desc = GDT_ENTRY_INIT(0x4092, 0, 0);
>  
>  /*
>   * At some point we want to use this stack frame pointer to unwind
> @@ -476,9 +476,6 @@ void pnpbios_calls_init(union pnp_bios_install_struct *header)
>  	pnp_bios_callpoint.offset = header->fields.pm16offset;
>  	pnp_bios_callpoint.segment = PNP_CS16;
>  
> -	bad_bios_desc.a = 0;
> -	bad_bios_desc.b = 0x00409200;
> -
>  	set_desc_base(&bad_bios_desc, (unsigned long)__va(0x40UL << 4));
>  	set_desc_limit(&bad_bios_desc, 4095 - (0x40 << 4));
>  	for_each_possible_cpu(i) {

Please also fix this hunk to do the full initialization statically
instead of doing some fields statically and some dynamically.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.