Re: 2.6.20-rc6-mm[2-3] ACPI issues

[Bob Picco]

Alexey Starikovskiy wrote:	[Fri Feb 02 2007, 09:20:35AM EST]
> Bob Picco wrote:
> >BTW, this isn't specific to rx2600. Lee Schermerhorn reported same -mm3 
> >problem on rx8620. Stephane Eranian reported the -mm2 problem mentioned 
> >above on rx2620.
> >
> >The debug information you requested is below. 
> >
> >thanks,
> >
> >bob
> Bob, thanks for debug information.
> Could you please try following patch?
> 
> Thanks,
your welcome,

It boots rx2600 and NUMA simulator successfully. The NUMA simulator has my M$
SRAT 1.0 hack applied.

thanks,

bob
>    Alex.

> Copy space_id of GAS structure to newly created GAS.
> 
> From: Alexey Starikovskiy <alexey.y.starikovskiy@xxxxxxxxxxxxxxx>
> 
> 
> ---
> 
>  drivers/acpi/tables/tbfadt.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/acpi/tables/tbfadt.c b/drivers/acpi/tables/tbfadt.c
> index 30350dd..807c711 100644
> --- a/drivers/acpi/tables/tbfadt.c
> +++ b/drivers/acpi/tables/tbfadt.c
> @@ -333,6 +333,8 @@ static void acpi_tb_convert_fadt(void)
>  				     pm1_register_length,
>  				     (acpi_gbl_FADT.xpm1a_event_block.address +
>  				      pm1_register_length));
> +	/* Don't forget to copy space_id of the GAS */
> +	acpi_gbl_xpm1a_enable.space_id = acpi_gbl_FADT.xpm1a_event_block.space_id;
>  
>  	/* The PM1B register block is optional, ignore if not present */
>  
> @@ -341,6 +343,9 @@ static void acpi_tb_convert_fadt(void)
>  					     pm1_register_length,
>  					     (acpi_gbl_FADT.xpm1b_event_block.
>  					      address + pm1_register_length));
> +		/* Don't forget to copy space_id of the GAS */
> +		acpi_gbl_xpm1b_enable.space_id = acpi_gbl_FADT.xpm1a_event_block.space_id;
> +
>  	}
>  }
>  

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