From nate@rootlabs.com  Thu Nov 20 20:45:59 2003
Return-Path: <nate@rootlabs.com>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id A830B16A4CE
	for <freebsd-gnats-submit@freebsd.org>; Thu, 20 Nov 2003 20:45:59 -0800 (PST)
Received: from rootlabs.com (root.org [67.118.192.226])
	by mx1.FreeBSD.org (Postfix) with SMTP id 155DA43FA3
	for <freebsd-gnats-submit@freebsd.org>; Thu, 20 Nov 2003 20:45:59 -0800 (PST)
	(envelope-from nate@rootlabs.com)
Received: (qmail 73518 invoked by uid 1000); 21 Nov 2003 04:46:01 -0000
Message-Id: <20031120204523.B73432@root.org>
Date: Thu, 20 Nov 2003 20:46:01 -0800 (PST)
From: Nate Lawson <nate@root.org>
To: sebastian ssmoller <sebastian.ssmoller@gmx.net>
Cc: freebsd-gnats-submit@freebsd.org
In-Reply-To: <1069356348.919.56.camel@tyrael.linnet>
Subject: Re: PR i386/57909
References: <18025.1068470822@www24.gmx.net>  <20031119153032.D69420@root.org>
 <1069356348.919.56.camel@tyrael.linnet>

>Number:         59541
>Category:       i386
>Synopsis:       Re: PR i386/57909
>Confidential:   yes
>Severity:       serious
>Priority:       medium
>Responsible:    gnats-admin
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Nov 20 20:50:22 PST 2003
>Closed-Date:    Fri Nov 21 18:09:29 PST 2003
>Last-Modified:  Fri Nov 21 18:09:29 PST 2003
>Originator:     
>Release:        
>Organization:
>Environment:
>Description:
 Thanks for the information.  Can you try this patch?
 
 --- /sys/contrib/dev/acpica/exfldio.c	Wed Oct 22 15:08:20 2003
 +++ /sys/contrib/dev/acpica/exfldio.c	Thu Oct  2 11:48:07 2003
 @@ -441,10 +437,10 @@
      /*
       * The four types of fields are:
       *
 -     * BufferFields - Read/write from/to a Buffer
 -     * RegionFields - Read/write from/to a Operation Region.
 -     * BankFields   - Write to a Bank Register, then read/write from/to an OpRegion
 -     * IndexFields  - Write to an Index Register, then read/write from/to a Data Register
 +     * BufferField - Read/write from/to a Buffer
 +     * RegionField - Read/write from/to a Operation Region.
 +     * BankField   - Write to a Bank Register, then read/write from/to an OpRegion
 +     * IndexField  - Write to an Index Register, then read/write from/to a Data Register
       */
      switch (ACPI_GET_OBJECT_TYPE (ObjDesc))
      {
 @@ -542,27 +538,37 @@
 
          /* Write the index value to the IndexRegister (itself a RegionField) */
 
 +        FieldDatumByteOffset += ObjDesc->IndexField.Value;
 +
 +        ACPI_DEBUG_PRINT ((ACPI_DB_BFIELD,
 +                "Write to Index Register: Value %8.8X\n",
 +                FieldDatumByteOffset));
 +
          Status = AcpiExInsertIntoField (ObjDesc->IndexField.IndexObj,
 -                                &ObjDesc->IndexField.Value,
 -                                sizeof (ObjDesc->IndexField.Value));
 +                                &FieldDatumByteOffset,
 +                                sizeof (FieldDatumByteOffset));
          if (ACPI_FAILURE (Status))
          {
              return_ACPI_STATUS (Status);
          }
 
 +        ACPI_DEBUG_PRINT ((ACPI_DB_BFIELD,
 +                "I/O to Data Register: ValuePtr %p\n",
 +                Value));
 +
          if (ReadWrite == ACPI_READ)
          {
              /* Read the datum from the DataRegister */
 
              Status = AcpiExExtractFromField (ObjDesc->IndexField.DataObj,
 -                            Value, ObjDesc->CommonField.AccessByteWidth);
 +                            Value, sizeof (ACPI_INTEGER));
          }
          else
          {
 -            /* Write the datum to the Data register */
 +            /* Write the datum to the DataRegister */
 
              Status = AcpiExInsertIntoField (ObjDesc->IndexField.DataObj,
 -                            Value, ObjDesc->CommonField.AccessByteWidth);
 +                            Value, sizeof (ACPI_INTEGER));
          }
          break;
 
>How-To-Repeat:
>Fix:
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: linimon 
State-Changed-When: Fri Nov 21 18:08:27 PST 2003 
State-Changed-Why:  
Close misfiled followup to 57909, which has already been committed. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=59541 
>Unformatted:
