From owner-acpi-jp@jp.freebsd.org  Fri Nov  9 22:07:10 2001
Received: (from daemon@localhost)
	by castle.jp.freebsd.org (8.9.3+3.2W/8.7.3) id WAA49929;
	Fri, 9 Nov 2001 22:07:10 +0900 (JST)
	(envelope-from owner-acpi-jp@jp.FreeBSD.org)
Received: from tasogare.imasy.or.jp (root@tasogare.imasy.or.jp [202.227.24.5])
	by castle.jp.freebsd.org (8.9.3+3.2W/8.7.3) with ESMTP id WAA49922
	for <acpi-jp@jp.freebsd.org>; Fri, 9 Nov 2001 22:07:10 +0900 (JST)
	(envelope-from iwasaki@jp.FreeBSD.org)
Received: from localhost (iwasaki.imasy.or.jp [202.227.24.92])
	(authenticated as iwa with CRAM-MD5)
	by tasogare.imasy.or.jp (8.11.6+3.4W/8.11.6/tasogare) with ESMTP/inet id fA9D78W43942;
	Fri, 9 Nov 2001 22:07:08 +0900 (JST)
	(envelope-from iwasaki@jp.FreeBSD.org)
Date: Fri, 09 Nov 2001 22:07:03 +0900 (JST)
Message-Id: <20011109.220703.74703220.iwasaki@jp.FreeBSD.org>
To: acpi-jp@jp.freebsd.org
Cc: robert.moore@intel.com
From: Mitsuru IWASAKI <iwasaki@jp.freebsd.org>
In-Reply-To: <20011109.124736.74756033.iwasaki@jp.FreeBSD.org>
References: <B9ECACBD6885D5119ADC00508B68C1EA2FDFFC@orsmsx107.jf.intel.com>
	<20011108162233.1144.T-KOUCHI@mvf.biglobe.ne.jp>
	<20011109.124736.74756033.iwasaki@jp.FreeBSD.org>
X-Mailer: Mew version 2.0 on Emacs 20.7 / Mule 4.0 (HANANOEN)
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Reply-To: acpi-jp@jp.freebsd.org
Precedence: list
X-Distribute: distribute version 2.1 (Alpha) patchlevel 24e+010328
X-Sequence: acpi-jp 1464
Subject: [acpi-jp 1464] Re: acpica-unix-20011102 is out
Errors-To: owner-acpi-jp@jp.freebsd.org
Sender: owner-acpi-jp@jp.freebsd.org
X-Originator: iwasaki@jp.freebsd.org

> And,
> > 1. Store operation to reference of local variable is broken again.
> > 
> > Method(GBFE,3)
> > {
> >         CreateByteField(Arg0, Arg1, TIDX)
> >         Store(TIDX, Arg2)
> > }
> > Method(BUG1)
> > {
> >         Store(Buffer(0xd) {0x0, 0x18, 0x10, 0x0}, Local2)
> >         Store(1, Local1)
> >         GBFE(Local2, Local1, RefOf(Local3))
> >         if (Local3) {
> >                 Store("OK", DEBUG)
> >         }
> > }
> 
> this problem, I've not get some time to track down, but I got debug traces
> of Store operation from 20011018 and 20011102 version.

I've made the attached patches based on the comparison between debug
traces of both versions, confirmed the problem is solved with the test
case and running 20011102 kernel, so far so good :-)
Bob, I'm not sure whether this is a correct fix, could you verify it?

Thanks

--- dsmthdat.c-	Thu Nov  8 03:59:20 2001
+++ dsmthdat.c	Fri Nov  9 14:55:29 2001
@@ -713,13 +713,13 @@
 
             /* Detach an existing object from the Node */
 
-            AcpiNsDetachObject (Node);
+            AcpiNsDetachObject ((ACPI_NAMESPACE_NODE *)Node->Object);
 
             /*
              * Store this object into the Node
              * (perform the indirect store)
              */
-            Status = AcpiNsAttachObject (Node, ObjDesc, ObjDesc->Common.Type);
+            Status = AcpiNsAttachObject ((ACPI_NAMESPACE_NODE *)Node->Object, ObjDesc, ObjDesc->Common.Type);
             return_ACPI_STATUS (Status);
         }
 

