On Mon, Sep 19, 2011 at 3:04 PM, Lin Ming <ming.m.lin@xxxxxxxxx> wrote:
> On Mon, 2011-09-19 at 10:12 +0800, Jim Green wrote:
>> On 19 September 2011 00:23, Lin Ming <ming.m.lin@xxxxxxxxx> wrote:
>> > On Mon, Sep 19, 2011 at 1:18 AM, Jim Green
>> > <student.northwestern@xxxxxxxxx> wrote:
>> >> Hello:
>> >>
>> >> I am running 2.6.38.8-4 kernel on p8p67 deluxe motherboard(bios
>> >> version 1702, latest stable)
>> >>
>> >> 2357 Sep 18 13:05:07 localhost kernel: [ 1.045727] ACPI: EC: Look
>> >> up EC in DSDT
>> >> 2358 Sep 18 13:05:07 localhost kernel: [ 1.046535] ACPI: Executed 1
>> >> blocks of module-level executable AML code
>> >> 2359 Sep 18 13:05:07 localhost kernel: [ 1.049876] ACPI Error:
>> >> [RAMB] Namespace lookup failure, AE_NOT_FOUND (20110112/psargs-359)
>> >> 2360 Sep 18 13:05:07 localhost kernel: [ 1.049880] ACPI Exception:
>> >> AE_NOT_FOUND, Could not execute arguments for [RAMW] (Region)
>> >> (20110112/ nsinit-349)
>> >> 2361 Sep 18 13:05:07 localhost kernel: [ 1.050347] ACPI: SSDT
>> >> 00000000df5d4818 0079C (v01 AMI IST 00000001 MSFT 03000001)
>> >> 2362 Sep 18 13:05:07 localhost kernel: [ 1.050805] ACPI: Dynamic
>> >> OEM Table Load:
>> >> 2363 Sep 18 13:05:07 localhost kernel: [ 1.050807] ACPI: SSDT
>> >> (null) 0079C (v01 AMI IST 00000001 MSFT 03000001)
>> >> 2364 Sep 18 13:05:07 localhost kernel: [ 1.050883] ACPI: SSDT
>> >> 00000000df5dba18 0021C (v01 AMI CST 00000001 MSFT 03000001)
>> >> 2365 Sep 18 13:05:07 localhost kernel: [ 1.051168] ACPI: Dynamic
>> >> OEM Table Load:
>> >> 2366 Sep 18 13:05:07 localhost kernel: [ 1.051169] ACPI: SSDT
>> >> (null) 0021C (v01 AMI CST 00000001 MSFT 03000001)
>> >> 2367 Sep 18 13:05:07 localhost kernel: [ 1.052510] ACPI: Interpreter enabled
>> >> 2368 Sep 18 13:05:07 localhost kernel: [ 1.052512] ACPI: (supports
>> >> S0 S1 S3 S4 S5)
>> >> 2369 Sep 18 13:05:07 localhost kernel: [ 1.052530] ACPI: Using
>> >> IOAPIC for interrupt routing
>> >>
>> >> Could anyone help how to fix this error?.. is this bug fixed in some
>> >> later kernel versions?
>> >
>> > Could you attach the acpidump output?
>>
>> Hi!
>>
>> Pls see attached, THanks!
>
> Hi, please try below patch on 2.6.38.* kernel.
Hi, Jim
Did you get a chance to test below patch?
Thanks,
Lin Ming
>
> (Note, this patch can't be applied on top of latest 3.1-rc kernel,
> because dsopcode.c file was renamed).
>
> From 5eeaba8a77213dc9ce777a69fd178468253ef578 Mon Sep 17 00:00:00 2001
> From: Lin Ming <ming.m.lin@xxxxxxxxx>
> Date: Mon, 19 Sep 2011 14:36:42 +0800
> Subject: [PATCH] ACPICA: Save scope node to resolve region argument
>
> Scope (_SB)
> {
> Name (RAMB, 0xDF5A1018)
> OperationRegion (\RAMW, SystemMemory, RAMB, 0x00010000)
> }
>
> For above ASL code, we need to save scope node(\_SB) to lookup
> the argument node(\_SB.RAMB).
>
> Reported-by: Jim Green <student.northwestern@xxxxxxxxx>
> Signed-off-by: Lin Ming <ming.m.lin@xxxxxxxxx>
> ---
> drivers/acpi/acpica/acobject.h | 1 +
> drivers/acpi/acpica/dsopcode.c | 2 +-
> drivers/acpi/acpica/excreate.c | 6 ++++++
> 3 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/acpi/acpica/acobject.h b/drivers/acpi/acpica/acobject.h
> index 1055769..6d276c2 100644
> --- a/drivers/acpi/acpica/acobject.h
> +++ b/drivers/acpi/acpica/acobject.h
> @@ -358,6 +358,7 @@ typedef enum {
> */
> struct acpi_object_extra {
> ACPI_OBJECT_COMMON_HEADER struct acpi_namespace_node *method_REG; /* _REG method for this region (if any) */
> + struct acpi_namespace_node *scope_node;
> void *region_context; /* Region-specific data */
> u8 *aml_start;
> u32 aml_length;
> diff --git a/drivers/acpi/acpica/dsopcode.c b/drivers/acpi/acpica/dsopcode.c
> index bbecf29..7ba0335 100644
> --- a/drivers/acpi/acpica/dsopcode.c
> +++ b/drivers/acpi/acpica/dsopcode.c
> @@ -394,7 +394,7 @@ acpi_status acpi_ds_get_region_arguments(union acpi_operand_object *obj_desc)
>
> /* Execute the argument AML */
>
> - status = acpi_ds_execute_arguments(node, node->parent,
> + status = acpi_ds_execute_arguments(node, extra_desc->extra.scope_node,
> extra_desc->extra.aml_length,
> extra_desc->extra.aml_start);
> if (ACPI_FAILURE(status)) {
> diff --git a/drivers/acpi/acpica/excreate.c b/drivers/acpi/acpica/excreate.c
> index e7b372d..df6ef00 100644
> --- a/drivers/acpi/acpica/excreate.c
> +++ b/drivers/acpi/acpica/excreate.c
> @@ -329,6 +329,12 @@ acpi_ex_create_region(u8 * aml_start,
> region_obj2 = obj_desc->common.next_object;
> region_obj2->extra.aml_start = aml_start;
> region_obj2->extra.aml_length = aml_length;
> + if (walk_state->scope_info) {
> + region_obj2->extra.scope_node =
> + walk_state->scope_info->scope.node;
> + } else {
> + region_obj2->extra.scope_node = node;
> + }
>
> /* Init the region from the operands */
>
> --
> 1.7.2.5
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
