* Re: [Devel] [PATCH] ACPICA: acpidump -- provide initial implementation for Linux
@ 2013-06-01 4:22 Al Stone
0 siblings, 0 replies; 6+ messages in thread
From: Al Stone @ 2013-06-01 4:22 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 19960 bytes --]
On 05/31/2013 06:46 PM, Moore, Robert wrote:
> Al,
>
> We've just completed and checked in a full implementation for Linux.
>
> You might want to inspect the code and see if there is anything that can be or should be merged. The code is at (or near) the head of our git tree.
>
> Thanks,
> Bob
Ah, interesting; I did a pull this afternoon and had
not seen any such changes. I'll take a look and see
what can be merged.
>> -----Original Message-----
>> From: devel-bounces(a)acpica.org [mailto:devel-bounces(a)acpica.org] On Behalf
>> Of Al Stone
>> Sent: Friday, May 31, 2013 5:21 PM
>> To: devel(a)acpica.org
>> Subject: [Devel] [PATCH] ACPICA: acpidump -- provide initial
>> implementation for Linux
>>
>>
>> source/os_specific/service_layer/oslinuxtbl.c: provide implementations for
>> AcpiOsGetTableByName and AcpiOsGetTableByIndex.
>>
>> With these two routines implemented, one can now run a Linux version of
>> acpidump and dump tables by name, or by index. An implementation for
>> dumping tables by address has not been provided.
>>
>> Signed-off-by: Al Stone <ahs3(a)redhat.com>
>> ---
>> source/os_specific/service_layers/oslinuxtbl.c | 415
>> ++++++++++++++++++-------
>> 1 file changed, 304 insertions(+), 111 deletions(-)
>>
>> diff --git a/source/os_specific/service_layers/oslinuxtbl.c
>> b/source/os_specific/service_layers/oslinuxtbl.c
>> index f539693..4005e0d 100644
>> --- a/source/os_specific/service_layers/oslinuxtbl.c
>> +++ b/source/os_specific/service_layers/oslinuxtbl.c
>> @@ -4,123 +4,295 @@
>> *
>>
>> **************************************************************************
>> ***/
>>
>> -
>> /*************************************************************************
>> *****
>> - *
>> - * 1. Copyright Notice
>> - *
>> - * Some or all of this work - Copyright (c) 1999 - 2013, Intel Corp.
>> +/*
>> + * Copyright (C) 2000 - 2013, Intel Corp.
>> * All rights reserved.
>> *
>> - * 2. License
>> - *
>> - * 2.1. This is your license from Intel Corp. under its intellectual
>> property
>> - * rights. You may have additional license terms from the party that
>> provided
>> - * you this software, covering your right to use that party's
>> intellectual
>> - * property rights.
>> - *
>> - * 2.2. Intel grants, free of charge, to any person ("Licensee")
>> obtaining a
>> - * copy of the source code appearing in this file ("Covered Code") an
>> - * irrevocable, perpetual, worldwide license under Intel's copyrights in
>> the
>> - * base code distributed originally by Intel ("Original Intel Code") to
>> copy,
>> - * make derivatives, distribute, use and display any portion of the
>> Covered
>> - * Code in any form, with the right to sublicense such rights; and
>> - *
>> - * 2.3. Intel grants Licensee a non-exclusive and non-transferable patent
>> - * license (with the right to sublicense), under only those claims of
>> Intel
>> - * patents that are infringed by the Original Intel Code, to make, use,
>> sell,
>> - * offer to sell, and import the Covered Code and derivative works
>> thereof
>> - * solely to the minimum extent necessary to exercise the above copyright
>> - * license, and in no event shall the patent license extend to any
>> additions
>> - * to or modifications of the Original Intel Code. No other license or
>> right
>> - * is granted directly or by implication, estoppel or otherwise;
>> - *
>> - * The above copyright and patent license is granted only if the
>> following
>> - * conditions are met:
>> - *
>> - * 3. Conditions
>> - *
>> - * 3.1. Redistribution of Source with Rights to Further Distribute
>> Source.
>> - * Redistribution of source code of any substantial portion of the
>> Covered
>> - * Code or modification with rights to further distribute source must
>> include
>> - * the above Copyright Notice, the above License, this list of
>> Conditions,
>> - * and the following Disclaimer and Export Compliance provision. In
>> addition,
>> - * Licensee must cause all Covered Code to which Licensee contributes to
>> - * contain a file documenting the changes Licensee made to create that
>> Covered
>> - * Code and the date of any change. Licensee must include in that file
>> the
>> - * documentation of any changes made by any predecessor Licensee.
>> Licensee
>> - * must include a prominent statement that the modification is derived,
>> - * directly or indirectly, from Original Intel Code.
>> - *
>> - * 3.2. Redistribution of Source with no Rights to Further Distribute
>> Source.
>> - * Redistribution of source code of any substantial portion of the
>> Covered
>> - * Code or modification without rights to further distribute source must
>> - * include the following Disclaimer and Export Compliance provision in
>> the
>> - * documentation and/or other materials provided with distribution. In
>> - * addition, Licensee may not authorize further sublicense of source of
>> any
>> - * portion of the Covered Code, and must include terms to the effect that
>> the
>> - * license from Licensee to its licensee is limited to the intellectual
>> - * property embodied in the software Licensee provides to its licensee,
>> and
>> - * not to intellectual property embodied in modifications its licensee
>> may
>> - * make.
>> - *
>> - * 3.3. Redistribution of Executable. Redistribution in executable form
>> of any
>> - * substantial portion of the Covered Code or modification must reproduce
>> the
>> - * above Copyright Notice, and the following Disclaimer and Export
>> Compliance
>> - * provision in the documentation and/or other materials provided with
>> the
>> - * distribution.
>> - *
>> - * 3.4. Intel retains all right, title, and interest in and to the
>> Original
>> - * Intel Code.
>> - *
>> - * 3.5. Neither the name Intel nor any other trademark owned or
>> controlled by
>> - * Intel shall be used in advertising or otherwise to promote the sale,
>> use or
>> - * other dealings in products derived from or relating to the Covered
>> Code
>> - * without prior written authorization from Intel.
>> - *
>> - * 4. Disclaimer and Export Compliance
>> - *
>> - * 4.1. INTEL MAKES NO WARRANTY OF ANY KIND REGARDING ANY SOFTWARE
>> PROVIDED
>> - * HERE. ANY SOFTWARE ORIGINATING FROM INTEL OR DERIVED FROM INTEL
>> SOFTWARE
>> - * IS PROVIDED "AS IS," AND INTEL WILL NOT PROVIDE ANY SUPPORT,
>> ASSISTANCE,
>> - * INSTALLATION, TRAINING OR OTHER SERVICES. INTEL WILL NOT PROVIDE ANY
>> - * UPDATES, ENHANCEMENTS OR EXTENSIONS. INTEL SPECIFICALLY DISCLAIMS ANY
>> - * IMPLIED WARRANTIES OF MERCHANTABILITY, NONINFRINGEMENT AND FITNESS FOR
>> A
>> - * PARTICULAR PURPOSE.
>> - *
>> - * 4.2. IN NO EVENT SHALL INTEL HAVE ANY LIABILITY TO LICENSEE, ITS
>> LICENSEES
>> - * OR ANY OTHER THIRD PARTY, FOR ANY LOST PROFITS, LOST DATA, LOSS OF USE
>> OR
>> - * COSTS OF PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES, OR FOR ANY
>> INDIRECT,
>> - * SPECIAL OR CONSEQUENTIAL DAMAGES ARISING OUT OF THIS AGREEMENT, UNDER
>> ANY
>> - * CAUSE OF ACTION OR THEORY OF LIABILITY, AND IRRESPECTIVE OF WHETHER
>> INTEL
>> - * HAS ADVANCE NOTICE OF THE POSSIBILITY OF SUCH DAMAGES. THESE
>> LIMITATIONS
>> - * SHALL APPLY NOTWITHSTANDING THE FAILURE OF THE ESSENTIAL PURPOSE OF
>> ANY
>> - * LIMITED REMEDY.
>> - *
>> - * 4.3. Licensee shall not export, either directly or indirectly, any of
>> this
>> - * software or system incorporating such software without first obtaining
>> any
>> - * required license or other approval from the U. S. Department of
>> Commerce or
>> - * any other agency or department of the United States Government. In the
>> - * event Licensee exports any such software from the United States or
>> - * re-exports any such software from a foreign destination, Licensee
>> shall
>> - * ensure that the distribution and export/re-export of the software is
>> in
>> - * compliance with all laws, regulations, orders, or other restrictions
>> of the
>> - * U.S. Export Administration Regulations. Licensee agrees that neither
>> it nor
>> - * any of its subsidiaries will export/re-export any technical data,
>> process,
>> - * software, or service, directly or indirectly, to any country for which
>> the
>> - * United States government or any agency thereof requires an export
>> license,
>> - * other governmental approval, or letter of assurance, without first
>> obtaining
>> - * such license, approval or letter.
>> - *
>> -
>> **************************************************************************
>> ***/
>> + * Linux implementation (vs original module skeleton):
>> + * Copyright (c) 2013, Al Stone <ahs3(a)redhat.com>
>> + *
>> + * Redistribution and use in source and binary forms, with or without
>> + * modification, are permitted provided that the following conditions
>> + * are met:
>> + * 1. Redistributions of source code must retain the above copyright
>> + * notice, this list of conditions, and the following disclaimer,
>> + * without modification.
>> + * 2. Redistributions in binary form must reproduce at minimum a
>> disclaimer
>> + * substantially similar to the "NO WARRANTY" disclaimer below
>> + * ("Disclaimer") and any redistribution must be conditioned upon
>> + * including a substantially similar Disclaimer requirement for
>> further
>> + * binary redistribution.
>> + * 3. Neither the names of the above-listed copyright holders nor the
>> names
>> + * of any contributors may be used to endorse or promote products
>> derived
>> + * from this software without specific prior written permission.
>> + *
>> + * Alternatively, this software may be distributed under the terms of
>> + the
>> + * GNU General Public License ("GPL") version 2 as published by the
>> + Free
>> + * Software Foundation.
>> + *
>> + * NO WARRANTY
>> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
>> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
>> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTIBILITY AND FITNESS
>> + FOR
>> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
>> + * HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY, OR
>> CONSEQUENTIAL
>> + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
>> + GOODS
>> + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>> + INTERRUPTION)
>> + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
>> + * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>> + ARISING
>> + * IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
>> + * POSSIBILITY OF SUCH DAMAGES.
>> + */
>>
>> #include "acpi.h"
>>
>> +#include <fcntl.h>
>> #include <stdio.h>
>> #include <stdlib.h>
>> +#include <sys/types.h>
>> +#include <sys/stat.h>
>> +#include <sys/mman.h>
>> +#include <unistd.h>
>>
>> #define _COMPONENT ACPI_OS_SERVICES
>> ACPI_MODULE_NAME ("oslinuxtbl")
>>
>> +/*
>> + * Linux specific code
>> + *
>> + * Read in all the tables we can find and put them in a nice table at
>> + * startup so we don't have to keep doing searches for each table.
>> + *
>> + */
>> +
>> +#define ACPI_OS_MAX_TABLE_INDEX 128
>> +#define MAX_SIG 5
>> +#define MAX_PATH 256
>> +#define MAX_BUF (128*1024) /* no table bigger than 128 KiB? */
>> +#define TABLE_HEADER 256
>> +
>> +static int tables_found = 0;
>> +
>> +typedef struct table_info {
>> + char *filename;
>> + char signature[MAX_SIG];
>> + int instance; /* only for some tables */
>> + int dynamic;
>> + ACPI_PHYSICAL_ADDRESS address;
>> + int size;
>> + unsigned char *data;
>> +} TABLE_INFO;
>> +
>> +static TABLE_INFO acpi_tables[ACPI_OS_MAX_TABLE_INDEX];
>> +static int next_index = 0;
>> +
>> +/**********************************************************************
>> +********
>> + *
>> + * FUNCTION: read_table
>> + *
>> + * PARAMETERS: fullpath - absolute path to /sys ACPI table
>> + *
>> + * RETURN: If the table can be read, returns 0; returns non-zero
>> + * otherwise.
>> + *
>> + * DESCRIPTION: Read in the contents of an ACPI table as stored in
>> + * /sys/firmware/acpi. Fill in the contents of an entry
>> + * in acpi_tables with the size and data of the table.
>> + *
>> +
>> **************************************************************************
>> ***/
>> +
>> +static int
>> +read_table(
>> + char *fullpath)
>> +{
>> + static unsigned char buf[MAX_BUF];
>> + FILE *fd;
>> + int result;
>> + int cnt;
>> + unsigned char *up;
>> +
>> + result = 1;
>> +
>> + /* read the table once to figure out it's size, then put it in a
>> buffer */
>> + memset(buf, 0, MAX_BUF);
>> + fd = fopen(fullpath, "r");
>> + if (fd)
>> + {
>> + cnt = fread(buf, MAX_BUF, 1, fd);
>> + if (feof(fd))
>> + {
>> + cnt = ftell(fd);
>> + up = (unsigned char *)malloc(cnt);
>> + if (up)
>> + {
>> + memcpy(up, buf, cnt);
>> + acpi_tables[next_index].size = cnt;
>> + acpi_tables[next_index].data = up;
>> + memcpy(acpi_tables[next_index].signature, up, 4);
>> + result = 0;
>> + }
>> + }
>> + fclose(fd);
>> + }
>> + else
>> + {
>> + fprintf(stderr, "Cannot open %s\n", fullpath);
>> + }
>> + return (result);
>> +}
>> +
>> +/**********************************************************************
>> +********
>> + *
>> + * FUNCTION: get_table_info
>> + *
>> + * PARAMETERS: dirpath - absolute path to a /sys/firmware/acpi
>> + * directory
>> + *
>> + * RETURN: Status: AE_OK if all the ACPI tables in the directory
>> + * given by dirpath can be read in. If not, AE_LIMIT will
>> + * be returned if there is insufficient space for the table,
>> + * and AE_ERROR otherwise.
>> + *
>> + * DESCRIPTION: Given a directory path in /sys, read in each of the files
>> + * in that directory as an ACPI table. Store any information
>> + * gathered about the table in an acpi_tables entry.
>> + *
>> +
>> **************************************************************************
>> ***/
>> +
>> +static ACPI_STATUS
>> +get_table_info(
>> + char *dirpath)
>> +{
>> + ACPI_STATUS result;
>> + void *dir;
>> + char *fpath;
>> + char fullpath[MAX_PATH];
>> + int tmp;
>> + char *p;
>> + struct stat buf;
>> +
>> + result = AE_ERROR;
>> + dir = AcpiOsOpenDirectory(dirpath, "*", REQUEST_FILE_ONLY);
>> + if (dir)
>> + {
>> + fpath = AcpiOsGetNextFilename(dir);
>> + while (fpath)
>> + {
>> + memset(fullpath, 0, MAX_PATH);
>> + strcpy(fullpath, dirpath);
>> + strcat(fullpath, "/");
>> + strcat(fullpath, fpath);
>> + if (next_index >= ACPI_OS_MAX_TABLE_INDEX)
>> + {
>> + result = AE_LIMIT;
>> + break;
>> + }
>> + else
>> + {
>> + tmp = strlen(fullpath) + 1;
>> + p = (char *)malloc(tmp);
>> + if (p)
>> + {
>> + memset(p, 0, tmp);
>> + acpi_tables[next_index].filename = p;
>> + }
>> + else
>> + {
>> + result = AE_LIMIT;
>> + break;
>> + }
>> + strcpy(acpi_tables[next_index].filename, fullpath);
>> +
>> + acpi_tables[next_index].dynamic = 0;
>> +
>> + if (strlen(fpath) > 4)
>> + {
>> + tmp = strtol(&(fpath[4]), NULL, 10);
>> + acpi_tables[next_index].instance = tmp;
>> + }
>> +
>> + if (stat(fullpath, &buf) == 0)
>> + {
>> + if (read_table(fullpath) == 0)
>> + {
>> + result = AE_OK;
>> + }
>> + else
>> + {
>> + result = AE_LIMIT;
>> + break;
>> + }
>> + }
>> +
>> + next_index++;
>> + }
>> + fpath = AcpiOsGetNextFilename(dir);
>> + }
>> +
>> + AcpiOsCloseDirectory(dir);
>> + result = AE_OK;
>> + }
>> + else
>> + {
>> + result = AE_ERROR;
>> + }
>> +
>> + return (result);
>> +}
>> +
>> +/**********************************************************************
>> +********
>> + *
>> + * FUNCTION: init_table_info
>> + *
>> + * PARAMETERS: none
>> + *
>> + * RETURN: Status: AE_OK if initialization was properly done. If
>> + * it was not, an error status indicating what occurred.
>> + *
>> + * DESCRIPTION: This routine is called to set up a file global structure
>> + * (acpi_tables) that will then be used by the other
>> + * AcpiOsGetTableBy* routines to find and print out info
>> + * about one of the current systems ACPI tables. The
>> + * routine is also set up so that it should only do the
>> + * heavy initialization work once on first being called.
>> + *
>> +
>> **************************************************************************
>> ***/
>> +
>> +static ACPI_STATUS
>> +init_table_info(
>> + void)
>> +{
>> + ACPI_STATUS result;
>> +
>> + if (tables_found)
>> + return (AE_OK);
>> +
>> + /* must be superuser to read the tables */
>> + if (getuid() != 0)
>> + {
>> + fprintf(stderr, "Must be run as superuser\n");
>> + return (AE_SUPPORT);
>> + }
>> +
>> + /* basic initialization */
>> + memset(acpi_tables, 0, ACPI_OS_MAX_TABLE_INDEX *
>> + sizeof(TABLE_INFO));
>> +
>> + /*
>> + * Look for all the files in /sys/firmware/acpi/tables and in
>> + * /sys/firmware/acpi/tables/dynamic; each one is a table.
>> + */
>> + result = get_table_info("/sys/firmware/acpi/tables");
>> + if (result == AE_OK)
>> + {
>> + result = get_table_info("/sys/firmware/acpi/tables/dynamic");
>> + }
>> +
>> + if (result == AE_OK)
>> + {
>> + tables_found = 1;
>> + }
>> + return (result);
>> +}
>>
>>
>> /*************************************************************************
>> *****
>> *
>> @@ -170,9 +342,16 @@ AcpiOsGetTableByIndex (
>> ACPI_TABLE_HEADER **Table,
>> ACPI_PHYSICAL_ADDRESS *Address)
>> {
>> -
>> - fprintf (stderr, "Linux version not implemented yet\n");
>> - return (AE_SUPPORT);
>> + if (init_table_info() == AE_OK)
>> + {
>> + if (Index < next_index)
>> + {
>> + *Table = (ACPI_TABLE_HEADER *)acpi_tables[Index].data;
>> + *Address = 0;
>> + return (AE_OK);
>> + }
>> + }
>> + return (AE_LIMIT);
>> }
>>
>>
>> @@ -203,7 +382,21 @@ AcpiOsGetTableByName (
>> ACPI_TABLE_HEADER **Table,
>> ACPI_PHYSICAL_ADDRESS *Address)
>> {
>> + int ii;
>>
>> - fprintf (stderr, "Linux version not implemented yet\n");
>> - return (AE_SUPPORT);
>> + if (init_table_info() == AE_OK)
>> + {
>> + for (ii = 0; ii < next_index; ii++)
>> + {
>> + if ((strncmp(Signature, acpi_tables[ii].signature, 4) == 0) &&
>> + (acpi_tables[ii].instance == Instance))
>> + {
>> + *Table = (ACPI_TABLE_HEADER *)acpi_tables[ii].data;
>> + *Address = 0;
>> + return (AE_OK);
>> + }
>> + }
>> + return (AE_LIMIT);
>> + }
>> + return (AE_NOT_FOUND);
>> }
>> --
>> 1.8.1.4
>>
>> --
>> ciao,
>> al
>> -----------------------------------
>> Al Stone
>> Software Engineer
>> Red Hat, Inc.
>> ahs3(a)redhat.com
>> -----------------------------------
>> _______________________________________________
>> Devel mailing list
>> Devel(a)acpica.org
>> https://lists.acpica.org/mailman/listinfo/devel
--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3(a)redhat.com
-----------------------------------
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Devel] [PATCH] ACPICA: acpidump -- provide initial implementation for Linux
@ 2013-06-05 22:12 Al Stone
0 siblings, 0 replies; 6+ messages in thread
From: Al Stone @ 2013-06-05 22:12 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 1030 bytes --]
On 06/04/2013 06:56 PM, Moore, Robert wrote:
>
>
>> -----Original Message-----
>> From: devel-bounces(a)acpica.org [mailto:devel-bounces(a)acpica.org] On Behalf
>> Of Al Stone
>> Sent: Sunday, June 02, 2013 1:04 PM
>> To: devel(a)acpica.org
>> Subject: Re: [Devel] [PATCH] ACPICA: acpidump -- provide initial
>> implementation for Linux
>>
[snip...]
>>
>> 2) I've seen at least a few systems with more than one UEFI
>> table instance. Whether allowed or not, it looks like
>> the acpidump code will only allow this for SSDTs.
>>
>
> If you find such a system, we would like to see the acpidump.
> Thanks,
> Bob
Sure. It's my Lenovo T420s laptop -- my daily driver. I see
three UEFI tables in /sys/firmware/acpi/tables: UEFI1, UEFI2
and UEFI3. It seemed a bit odd, but I have not investigated
it any further. The tables are attached.
--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3(a)redhat.com
-----------------------------------
[-- Attachment #2: uefi.tar.gz --]
[-- Type: application/gzip, Size: 971 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Devel] [PATCH] ACPICA: acpidump -- provide initial implementation for Linux
@ 2013-06-05 0:56 Moore, Robert
0 siblings, 0 replies; 6+ messages in thread
From: Moore, Robert @ 2013-06-05 0:56 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 2427 bytes --]
> -----Original Message-----
> From: devel-bounces(a)acpica.org [mailto:devel-bounces(a)acpica.org] On Behalf
> Of Al Stone
> Sent: Sunday, June 02, 2013 1:04 PM
> To: devel(a)acpica.org
> Subject: Re: [Devel] [PATCH] ACPICA: acpidump -- provide initial
> implementation for Linux
>
> On 05/31/2013 10:22 PM, Al Stone wrote:
> > On 05/31/2013 06:46 PM, Moore, Robert wrote:
> >> Al,
> >>
> >> We've just completed and checked in a full implementation for Linux.
> >>
> >> You might want to inspect the code and see if there is anything that
> >> can be or should be merged. The code is at (or near) the head of our
> >> git tree.
> >>
> >> Thanks,
> >> Bob
> >
> > Ah, interesting; I did a pull this afternoon and had not seen any such
> > changes. I'll take a look and see what can be merged.
>
> After looking at the two patches, there's not much that they share in
> common; I don't think there's anything that would make sense to merge --
> the approaches are very different.
>
> I do have a couple of concerns with the implementation,
> though:
>
> 1) While mmap() of /dev/mem makes sense, I suspect there
> are cases where one can attempt to mmap() the wrong
> address and lock up the system. I have not seen a good
> way to avoid this just yet without a priori being able
> to establish the proper physical addresses for the tables
> but that may require a kernel patch to provide the addresses
> via /sys. At a minimum, we should warn folks that this
> could happen. This was the only reason I did not include
> an implementation of AcpiOsGetTableByAddress().
>
> 2) I've seen at least a few systems with more than one UEFI
> table instance. Whether allowed or not, it looks like
> the acpidump code will only allow this for SSDTs.
>
If you find such a system, we would like to see the acpidump.
Thanks,
Bob
> Thanks for checking the code in. Now that I know an acpidump will be
> there, I can fix up the distro packages to handle this new version of the
> command properly, too.
>
> --
> ciao,
> al
> -----------------------------------
> Al Stone
> Software Engineer
> Red Hat, Inc.
> ahs3(a)redhat.com
> -----------------------------------
> _______________________________________________
> Devel mailing list
> Devel(a)acpica.org
> https://lists.acpica.org/mailman/listinfo/devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Devel] [PATCH] ACPICA: acpidump -- provide initial implementation for Linux
@ 2013-06-02 20:04 Al Stone
0 siblings, 0 replies; 6+ messages in thread
From: Al Stone @ 2013-06-02 20:04 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 1801 bytes --]
On 05/31/2013 10:22 PM, Al Stone wrote:
> On 05/31/2013 06:46 PM, Moore, Robert wrote:
>> Al,
>>
>> We've just completed and checked in a full implementation for Linux.
>>
>> You might want to inspect the code and see if there is anything that
>> can be or should be merged. The code is at (or near) the head of our
>> git tree.
>>
>> Thanks,
>> Bob
>
> Ah, interesting; I did a pull this afternoon and had
> not seen any such changes. I'll take a look and see
> what can be merged.
After looking at the two patches, there's not much that
they share in common; I don't think there's anything that
would make sense to merge -- the approaches are very
different.
I do have a couple of concerns with the implementation,
though:
1) While mmap() of /dev/mem makes sense, I suspect there
are cases where one can attempt to mmap() the wrong
address and lock up the system. I have not seen a good
way to avoid this just yet without a priori being able
to establish the proper physical addresses for the tables
but that may require a kernel patch to provide the addresses
via /sys. At a minimum, we should warn folks that this
could happen. This was the only reason I did not include
an implementation of AcpiOsGetTableByAddress().
2) I've seen at least a few systems with more than one UEFI
table instance. Whether allowed or not, it looks like
the acpidump code will only allow this for SSDTs.
Thanks for checking the code in. Now that I know an acpidump
will be there, I can fix up the distro packages to handle this
new version of the command properly, too.
--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3(a)redhat.com
-----------------------------------
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Devel] [PATCH] ACPICA: acpidump -- provide initial implementation for Linux
@ 2013-06-01 0:46 Moore, Robert
0 siblings, 0 replies; 6+ messages in thread
From: Moore, Robert @ 2013-06-01 0:46 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 19036 bytes --]
Al,
We've just completed and checked in a full implementation for Linux.
You might want to inspect the code and see if there is anything that can be or should be merged. The code is at (or near) the head of our git tree.
Thanks,
Bob
> -----Original Message-----
> From: devel-bounces(a)acpica.org [mailto:devel-bounces(a)acpica.org] On Behalf
> Of Al Stone
> Sent: Friday, May 31, 2013 5:21 PM
> To: devel(a)acpica.org
> Subject: [Devel] [PATCH] ACPICA: acpidump -- provide initial
> implementation for Linux
>
>
> source/os_specific/service_layer/oslinuxtbl.c: provide implementations for
> AcpiOsGetTableByName and AcpiOsGetTableByIndex.
>
> With these two routines implemented, one can now run a Linux version of
> acpidump and dump tables by name, or by index. An implementation for
> dumping tables by address has not been provided.
>
> Signed-off-by: Al Stone <ahs3(a)redhat.com>
> ---
> source/os_specific/service_layers/oslinuxtbl.c | 415
> ++++++++++++++++++-------
> 1 file changed, 304 insertions(+), 111 deletions(-)
>
> diff --git a/source/os_specific/service_layers/oslinuxtbl.c
> b/source/os_specific/service_layers/oslinuxtbl.c
> index f539693..4005e0d 100644
> --- a/source/os_specific/service_layers/oslinuxtbl.c
> +++ b/source/os_specific/service_layers/oslinuxtbl.c
> @@ -4,123 +4,295 @@
> *
>
> **************************************************************************
> ***/
>
> -
> /*************************************************************************
> *****
> - *
> - * 1. Copyright Notice
> - *
> - * Some or all of this work - Copyright (c) 1999 - 2013, Intel Corp.
> +/*
> + * Copyright (C) 2000 - 2013, Intel Corp.
> * All rights reserved.
> *
> - * 2. License
> - *
> - * 2.1. This is your license from Intel Corp. under its intellectual
> property
> - * rights. You may have additional license terms from the party that
> provided
> - * you this software, covering your right to use that party's
> intellectual
> - * property rights.
> - *
> - * 2.2. Intel grants, free of charge, to any person ("Licensee")
> obtaining a
> - * copy of the source code appearing in this file ("Covered Code") an
> - * irrevocable, perpetual, worldwide license under Intel's copyrights in
> the
> - * base code distributed originally by Intel ("Original Intel Code") to
> copy,
> - * make derivatives, distribute, use and display any portion of the
> Covered
> - * Code in any form, with the right to sublicense such rights; and
> - *
> - * 2.3. Intel grants Licensee a non-exclusive and non-transferable patent
> - * license (with the right to sublicense), under only those claims of
> Intel
> - * patents that are infringed by the Original Intel Code, to make, use,
> sell,
> - * offer to sell, and import the Covered Code and derivative works
> thereof
> - * solely to the minimum extent necessary to exercise the above copyright
> - * license, and in no event shall the patent license extend to any
> additions
> - * to or modifications of the Original Intel Code. No other license or
> right
> - * is granted directly or by implication, estoppel or otherwise;
> - *
> - * The above copyright and patent license is granted only if the
> following
> - * conditions are met:
> - *
> - * 3. Conditions
> - *
> - * 3.1. Redistribution of Source with Rights to Further Distribute
> Source.
> - * Redistribution of source code of any substantial portion of the
> Covered
> - * Code or modification with rights to further distribute source must
> include
> - * the above Copyright Notice, the above License, this list of
> Conditions,
> - * and the following Disclaimer and Export Compliance provision. In
> addition,
> - * Licensee must cause all Covered Code to which Licensee contributes to
> - * contain a file documenting the changes Licensee made to create that
> Covered
> - * Code and the date of any change. Licensee must include in that file
> the
> - * documentation of any changes made by any predecessor Licensee.
> Licensee
> - * must include a prominent statement that the modification is derived,
> - * directly or indirectly, from Original Intel Code.
> - *
> - * 3.2. Redistribution of Source with no Rights to Further Distribute
> Source.
> - * Redistribution of source code of any substantial portion of the
> Covered
> - * Code or modification without rights to further distribute source must
> - * include the following Disclaimer and Export Compliance provision in
> the
> - * documentation and/or other materials provided with distribution. In
> - * addition, Licensee may not authorize further sublicense of source of
> any
> - * portion of the Covered Code, and must include terms to the effect that
> the
> - * license from Licensee to its licensee is limited to the intellectual
> - * property embodied in the software Licensee provides to its licensee,
> and
> - * not to intellectual property embodied in modifications its licensee
> may
> - * make.
> - *
> - * 3.3. Redistribution of Executable. Redistribution in executable form
> of any
> - * substantial portion of the Covered Code or modification must reproduce
> the
> - * above Copyright Notice, and the following Disclaimer and Export
> Compliance
> - * provision in the documentation and/or other materials provided with
> the
> - * distribution.
> - *
> - * 3.4. Intel retains all right, title, and interest in and to the
> Original
> - * Intel Code.
> - *
> - * 3.5. Neither the name Intel nor any other trademark owned or
> controlled by
> - * Intel shall be used in advertising or otherwise to promote the sale,
> use or
> - * other dealings in products derived from or relating to the Covered
> Code
> - * without prior written authorization from Intel.
> - *
> - * 4. Disclaimer and Export Compliance
> - *
> - * 4.1. INTEL MAKES NO WARRANTY OF ANY KIND REGARDING ANY SOFTWARE
> PROVIDED
> - * HERE. ANY SOFTWARE ORIGINATING FROM INTEL OR DERIVED FROM INTEL
> SOFTWARE
> - * IS PROVIDED "AS IS," AND INTEL WILL NOT PROVIDE ANY SUPPORT,
> ASSISTANCE,
> - * INSTALLATION, TRAINING OR OTHER SERVICES. INTEL WILL NOT PROVIDE ANY
> - * UPDATES, ENHANCEMENTS OR EXTENSIONS. INTEL SPECIFICALLY DISCLAIMS ANY
> - * IMPLIED WARRANTIES OF MERCHANTABILITY, NONINFRINGEMENT AND FITNESS FOR
> A
> - * PARTICULAR PURPOSE.
> - *
> - * 4.2. IN NO EVENT SHALL INTEL HAVE ANY LIABILITY TO LICENSEE, ITS
> LICENSEES
> - * OR ANY OTHER THIRD PARTY, FOR ANY LOST PROFITS, LOST DATA, LOSS OF USE
> OR
> - * COSTS OF PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES, OR FOR ANY
> INDIRECT,
> - * SPECIAL OR CONSEQUENTIAL DAMAGES ARISING OUT OF THIS AGREEMENT, UNDER
> ANY
> - * CAUSE OF ACTION OR THEORY OF LIABILITY, AND IRRESPECTIVE OF WHETHER
> INTEL
> - * HAS ADVANCE NOTICE OF THE POSSIBILITY OF SUCH DAMAGES. THESE
> LIMITATIONS
> - * SHALL APPLY NOTWITHSTANDING THE FAILURE OF THE ESSENTIAL PURPOSE OF
> ANY
> - * LIMITED REMEDY.
> - *
> - * 4.3. Licensee shall not export, either directly or indirectly, any of
> this
> - * software or system incorporating such software without first obtaining
> any
> - * required license or other approval from the U. S. Department of
> Commerce or
> - * any other agency or department of the United States Government. In the
> - * event Licensee exports any such software from the United States or
> - * re-exports any such software from a foreign destination, Licensee
> shall
> - * ensure that the distribution and export/re-export of the software is
> in
> - * compliance with all laws, regulations, orders, or other restrictions
> of the
> - * U.S. Export Administration Regulations. Licensee agrees that neither
> it nor
> - * any of its subsidiaries will export/re-export any technical data,
> process,
> - * software, or service, directly or indirectly, to any country for which
> the
> - * United States government or any agency thereof requires an export
> license,
> - * other governmental approval, or letter of assurance, without first
> obtaining
> - * such license, approval or letter.
> - *
> -
> **************************************************************************
> ***/
> + * Linux implementation (vs original module skeleton):
> + * Copyright (c) 2013, Al Stone <ahs3(a)redhat.com>
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions
> + * are met:
> + * 1. Redistributions of source code must retain the above copyright
> + * notice, this list of conditions, and the following disclaimer,
> + * without modification.
> + * 2. Redistributions in binary form must reproduce at minimum a
> disclaimer
> + * substantially similar to the "NO WARRANTY" disclaimer below
> + * ("Disclaimer") and any redistribution must be conditioned upon
> + * including a substantially similar Disclaimer requirement for
> further
> + * binary redistribution.
> + * 3. Neither the names of the above-listed copyright holders nor the
> names
> + * of any contributors may be used to endorse or promote products
> derived
> + * from this software without specific prior written permission.
> + *
> + * Alternatively, this software may be distributed under the terms of
> + the
> + * GNU General Public License ("GPL") version 2 as published by the
> + Free
> + * Software Foundation.
> + *
> + * NO WARRANTY
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTIBILITY AND FITNESS
> + FOR
> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> + * HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY, OR
> CONSEQUENTIAL
> + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
> + GOODS
> + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
> + INTERRUPTION)
> + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
> + * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> + ARISING
> + * IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
> + * POSSIBILITY OF SUCH DAMAGES.
> + */
>
> #include "acpi.h"
>
> +#include <fcntl.h>
> #include <stdio.h>
> #include <stdlib.h>
> +#include <sys/types.h>
> +#include <sys/stat.h>
> +#include <sys/mman.h>
> +#include <unistd.h>
>
> #define _COMPONENT ACPI_OS_SERVICES
> ACPI_MODULE_NAME ("oslinuxtbl")
>
> +/*
> + * Linux specific code
> + *
> + * Read in all the tables we can find and put them in a nice table at
> + * startup so we don't have to keep doing searches for each table.
> + *
> + */
> +
> +#define ACPI_OS_MAX_TABLE_INDEX 128
> +#define MAX_SIG 5
> +#define MAX_PATH 256
> +#define MAX_BUF (128*1024) /* no table bigger than 128 KiB? */
> +#define TABLE_HEADER 256
> +
> +static int tables_found = 0;
> +
> +typedef struct table_info {
> + char *filename;
> + char signature[MAX_SIG];
> + int instance; /* only for some tables */
> + int dynamic;
> + ACPI_PHYSICAL_ADDRESS address;
> + int size;
> + unsigned char *data;
> +} TABLE_INFO;
> +
> +static TABLE_INFO acpi_tables[ACPI_OS_MAX_TABLE_INDEX];
> +static int next_index = 0;
> +
> +/**********************************************************************
> +********
> + *
> + * FUNCTION: read_table
> + *
> + * PARAMETERS: fullpath - absolute path to /sys ACPI table
> + *
> + * RETURN: If the table can be read, returns 0; returns non-zero
> + * otherwise.
> + *
> + * DESCRIPTION: Read in the contents of an ACPI table as stored in
> + * /sys/firmware/acpi. Fill in the contents of an entry
> + * in acpi_tables with the size and data of the table.
> + *
> +
> **************************************************************************
> ***/
> +
> +static int
> +read_table(
> + char *fullpath)
> +{
> + static unsigned char buf[MAX_BUF];
> + FILE *fd;
> + int result;
> + int cnt;
> + unsigned char *up;
> +
> + result = 1;
> +
> + /* read the table once to figure out it's size, then put it in a
> buffer */
> + memset(buf, 0, MAX_BUF);
> + fd = fopen(fullpath, "r");
> + if (fd)
> + {
> + cnt = fread(buf, MAX_BUF, 1, fd);
> + if (feof(fd))
> + {
> + cnt = ftell(fd);
> + up = (unsigned char *)malloc(cnt);
> + if (up)
> + {
> + memcpy(up, buf, cnt);
> + acpi_tables[next_index].size = cnt;
> + acpi_tables[next_index].data = up;
> + memcpy(acpi_tables[next_index].signature, up, 4);
> + result = 0;
> + }
> + }
> + fclose(fd);
> + }
> + else
> + {
> + fprintf(stderr, "Cannot open %s\n", fullpath);
> + }
> + return (result);
> +}
> +
> +/**********************************************************************
> +********
> + *
> + * FUNCTION: get_table_info
> + *
> + * PARAMETERS: dirpath - absolute path to a /sys/firmware/acpi
> + * directory
> + *
> + * RETURN: Status: AE_OK if all the ACPI tables in the directory
> + * given by dirpath can be read in. If not, AE_LIMIT will
> + * be returned if there is insufficient space for the table,
> + * and AE_ERROR otherwise.
> + *
> + * DESCRIPTION: Given a directory path in /sys, read in each of the files
> + * in that directory as an ACPI table. Store any information
> + * gathered about the table in an acpi_tables entry.
> + *
> +
> **************************************************************************
> ***/
> +
> +static ACPI_STATUS
> +get_table_info(
> + char *dirpath)
> +{
> + ACPI_STATUS result;
> + void *dir;
> + char *fpath;
> + char fullpath[MAX_PATH];
> + int tmp;
> + char *p;
> + struct stat buf;
> +
> + result = AE_ERROR;
> + dir = AcpiOsOpenDirectory(dirpath, "*", REQUEST_FILE_ONLY);
> + if (dir)
> + {
> + fpath = AcpiOsGetNextFilename(dir);
> + while (fpath)
> + {
> + memset(fullpath, 0, MAX_PATH);
> + strcpy(fullpath, dirpath);
> + strcat(fullpath, "/");
> + strcat(fullpath, fpath);
> + if (next_index >= ACPI_OS_MAX_TABLE_INDEX)
> + {
> + result = AE_LIMIT;
> + break;
> + }
> + else
> + {
> + tmp = strlen(fullpath) + 1;
> + p = (char *)malloc(tmp);
> + if (p)
> + {
> + memset(p, 0, tmp);
> + acpi_tables[next_index].filename = p;
> + }
> + else
> + {
> + result = AE_LIMIT;
> + break;
> + }
> + strcpy(acpi_tables[next_index].filename, fullpath);
> +
> + acpi_tables[next_index].dynamic = 0;
> +
> + if (strlen(fpath) > 4)
> + {
> + tmp = strtol(&(fpath[4]), NULL, 10);
> + acpi_tables[next_index].instance = tmp;
> + }
> +
> + if (stat(fullpath, &buf) == 0)
> + {
> + if (read_table(fullpath) == 0)
> + {
> + result = AE_OK;
> + }
> + else
> + {
> + result = AE_LIMIT;
> + break;
> + }
> + }
> +
> + next_index++;
> + }
> + fpath = AcpiOsGetNextFilename(dir);
> + }
> +
> + AcpiOsCloseDirectory(dir);
> + result = AE_OK;
> + }
> + else
> + {
> + result = AE_ERROR;
> + }
> +
> + return (result);
> +}
> +
> +/**********************************************************************
> +********
> + *
> + * FUNCTION: init_table_info
> + *
> + * PARAMETERS: none
> + *
> + * RETURN: Status: AE_OK if initialization was properly done. If
> + * it was not, an error status indicating what occurred.
> + *
> + * DESCRIPTION: This routine is called to set up a file global structure
> + * (acpi_tables) that will then be used by the other
> + * AcpiOsGetTableBy* routines to find and print out info
> + * about one of the current systems ACPI tables. The
> + * routine is also set up so that it should only do the
> + * heavy initialization work once on first being called.
> + *
> +
> **************************************************************************
> ***/
> +
> +static ACPI_STATUS
> +init_table_info(
> + void)
> +{
> + ACPI_STATUS result;
> +
> + if (tables_found)
> + return (AE_OK);
> +
> + /* must be superuser to read the tables */
> + if (getuid() != 0)
> + {
> + fprintf(stderr, "Must be run as superuser\n");
> + return (AE_SUPPORT);
> + }
> +
> + /* basic initialization */
> + memset(acpi_tables, 0, ACPI_OS_MAX_TABLE_INDEX *
> + sizeof(TABLE_INFO));
> +
> + /*
> + * Look for all the files in /sys/firmware/acpi/tables and in
> + * /sys/firmware/acpi/tables/dynamic; each one is a table.
> + */
> + result = get_table_info("/sys/firmware/acpi/tables");
> + if (result == AE_OK)
> + {
> + result = get_table_info("/sys/firmware/acpi/tables/dynamic");
> + }
> +
> + if (result == AE_OK)
> + {
> + tables_found = 1;
> + }
> + return (result);
> +}
>
>
> /*************************************************************************
> *****
> *
> @@ -170,9 +342,16 @@ AcpiOsGetTableByIndex (
> ACPI_TABLE_HEADER **Table,
> ACPI_PHYSICAL_ADDRESS *Address)
> {
> -
> - fprintf (stderr, "Linux version not implemented yet\n");
> - return (AE_SUPPORT);
> + if (init_table_info() == AE_OK)
> + {
> + if (Index < next_index)
> + {
> + *Table = (ACPI_TABLE_HEADER *)acpi_tables[Index].data;
> + *Address = 0;
> + return (AE_OK);
> + }
> + }
> + return (AE_LIMIT);
> }
>
>
> @@ -203,7 +382,21 @@ AcpiOsGetTableByName (
> ACPI_TABLE_HEADER **Table,
> ACPI_PHYSICAL_ADDRESS *Address)
> {
> + int ii;
>
> - fprintf (stderr, "Linux version not implemented yet\n");
> - return (AE_SUPPORT);
> + if (init_table_info() == AE_OK)
> + {
> + for (ii = 0; ii < next_index; ii++)
> + {
> + if ((strncmp(Signature, acpi_tables[ii].signature, 4) == 0) &&
> + (acpi_tables[ii].instance == Instance))
> + {
> + *Table = (ACPI_TABLE_HEADER *)acpi_tables[ii].data;
> + *Address = 0;
> + return (AE_OK);
> + }
> + }
> + return (AE_LIMIT);
> + }
> + return (AE_NOT_FOUND);
> }
> --
> 1.8.1.4
>
> --
> ciao,
> al
> -----------------------------------
> Al Stone
> Software Engineer
> Red Hat, Inc.
> ahs3(a)redhat.com
> -----------------------------------
> _______________________________________________
> Devel mailing list
> Devel(a)acpica.org
> https://lists.acpica.org/mailman/listinfo/devel
^ permalink raw reply [flat|nested] 6+ messages in thread* [Devel] [PATCH] ACPICA: acpidump -- provide initial implementation for Linux
@ 2013-06-01 0:20 Al Stone
0 siblings, 0 replies; 6+ messages in thread
From: Al Stone @ 2013-06-01 0:20 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 17195 bytes --]
source/os_specific/service_layer/oslinuxtbl.c: provide implementations
for AcpiOsGetTableByName and AcpiOsGetTableByIndex.
With these two routines implemented, one can now run a Linux version
of acpidump and dump tables by name, or by index. An implementation
for dumping tables by address has not been provided.
Signed-off-by: Al Stone <ahs3(a)redhat.com>
---
source/os_specific/service_layers/oslinuxtbl.c | 415
++++++++++++++++++-------
1 file changed, 304 insertions(+), 111 deletions(-)
diff --git a/source/os_specific/service_layers/oslinuxtbl.c
b/source/os_specific/service_layers/oslinuxtbl.c
index f539693..4005e0d 100644
--- a/source/os_specific/service_layers/oslinuxtbl.c
+++ b/source/os_specific/service_layers/oslinuxtbl.c
@@ -4,123 +4,295 @@
*
*****************************************************************************/
-/******************************************************************************
- *
- * 1. Copyright Notice
- *
- * Some or all of this work - Copyright (c) 1999 - 2013, Intel Corp.
+/*
+ * Copyright (C) 2000 - 2013, Intel Corp.
* All rights reserved.
*
- * 2. License
- *
- * 2.1. This is your license from Intel Corp. under its intellectual
property
- * rights. You may have additional license terms from the party that
provided
- * you this software, covering your right to use that party's intellectual
- * property rights.
- *
- * 2.2. Intel grants, free of charge, to any person ("Licensee")
obtaining a
- * copy of the source code appearing in this file ("Covered Code") an
- * irrevocable, perpetual, worldwide license under Intel's copyrights
in the
- * base code distributed originally by Intel ("Original Intel Code") to
copy,
- * make derivatives, distribute, use and display any portion of the Covered
- * Code in any form, with the right to sublicense such rights; and
- *
- * 2.3. Intel grants Licensee a non-exclusive and non-transferable patent
- * license (with the right to sublicense), under only those claims of Intel
- * patents that are infringed by the Original Intel Code, to make, use,
sell,
- * offer to sell, and import the Covered Code and derivative works thereof
- * solely to the minimum extent necessary to exercise the above copyright
- * license, and in no event shall the patent license extend to any
additions
- * to or modifications of the Original Intel Code. No other license or
right
- * is granted directly or by implication, estoppel or otherwise;
- *
- * The above copyright and patent license is granted only if the following
- * conditions are met:
- *
- * 3. Conditions
- *
- * 3.1. Redistribution of Source with Rights to Further Distribute Source.
- * Redistribution of source code of any substantial portion of the Covered
- * Code or modification with rights to further distribute source must
include
- * the above Copyright Notice, the above License, this list of Conditions,
- * and the following Disclaimer and Export Compliance provision. In
addition,
- * Licensee must cause all Covered Code to which Licensee contributes to
- * contain a file documenting the changes Licensee made to create that
Covered
- * Code and the date of any change. Licensee must include in that file the
- * documentation of any changes made by any predecessor Licensee. Licensee
- * must include a prominent statement that the modification is derived,
- * directly or indirectly, from Original Intel Code.
- *
- * 3.2. Redistribution of Source with no Rights to Further Distribute
Source.
- * Redistribution of source code of any substantial portion of the Covered
- * Code or modification without rights to further distribute source must
- * include the following Disclaimer and Export Compliance provision in the
- * documentation and/or other materials provided with distribution. In
- * addition, Licensee may not authorize further sublicense of source of any
- * portion of the Covered Code, and must include terms to the effect
that the
- * license from Licensee to its licensee is limited to the intellectual
- * property embodied in the software Licensee provides to its licensee, and
- * not to intellectual property embodied in modifications its licensee may
- * make.
- *
- * 3.3. Redistribution of Executable. Redistribution in executable form
of any
- * substantial portion of the Covered Code or modification must
reproduce the
- * above Copyright Notice, and the following Disclaimer and Export
Compliance
- * provision in the documentation and/or other materials provided with the
- * distribution.
- *
- * 3.4. Intel retains all right, title, and interest in and to the Original
- * Intel Code.
- *
- * 3.5. Neither the name Intel nor any other trademark owned or
controlled by
- * Intel shall be used in advertising or otherwise to promote the sale,
use or
- * other dealings in products derived from or relating to the Covered Code
- * without prior written authorization from Intel.
- *
- * 4. Disclaimer and Export Compliance
- *
- * 4.1. INTEL MAKES NO WARRANTY OF ANY KIND REGARDING ANY SOFTWARE PROVIDED
- * HERE. ANY SOFTWARE ORIGINATING FROM INTEL OR DERIVED FROM INTEL SOFTWARE
- * IS PROVIDED "AS IS," AND INTEL WILL NOT PROVIDE ANY SUPPORT, ASSISTANCE,
- * INSTALLATION, TRAINING OR OTHER SERVICES. INTEL WILL NOT PROVIDE ANY
- * UPDATES, ENHANCEMENTS OR EXTENSIONS. INTEL SPECIFICALLY DISCLAIMS ANY
- * IMPLIED WARRANTIES OF MERCHANTABILITY, NONINFRINGEMENT AND FITNESS FOR A
- * PARTICULAR PURPOSE.
- *
- * 4.2. IN NO EVENT SHALL INTEL HAVE ANY LIABILITY TO LICENSEE, ITS
LICENSEES
- * OR ANY OTHER THIRD PARTY, FOR ANY LOST PROFITS, LOST DATA, LOSS OF
USE OR
- * COSTS OF PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES, OR FOR ANY
INDIRECT,
- * SPECIAL OR CONSEQUENTIAL DAMAGES ARISING OUT OF THIS AGREEMENT,
UNDER ANY
- * CAUSE OF ACTION OR THEORY OF LIABILITY, AND IRRESPECTIVE OF WHETHER
INTEL
- * HAS ADVANCE NOTICE OF THE POSSIBILITY OF SUCH DAMAGES. THESE LIMITATIONS
- * SHALL APPLY NOTWITHSTANDING THE FAILURE OF THE ESSENTIAL PURPOSE OF ANY
- * LIMITED REMEDY.
- *
- * 4.3. Licensee shall not export, either directly or indirectly, any
of this
- * software or system incorporating such software without first
obtaining any
- * required license or other approval from the U. S. Department of
Commerce or
- * any other agency or department of the United States Government. In the
- * event Licensee exports any such software from the United States or
- * re-exports any such software from a foreign destination, Licensee shall
- * ensure that the distribution and export/re-export of the software is in
- * compliance with all laws, regulations, orders, or other restrictions
of the
- * U.S. Export Administration Regulations. Licensee agrees that neither
it nor
- * any of its subsidiaries will export/re-export any technical data,
process,
- * software, or service, directly or indirectly, to any country for
which the
- * United States government or any agency thereof requires an export
license,
- * other governmental approval, or letter of assurance, without first
obtaining
- * such license, approval or letter.
- *
-
*****************************************************************************/
+ * Linux implementation (vs original module skeleton):
+ * Copyright (c) 2013, Al Stone <ahs3(a)redhat.com>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions, and the following disclaimer,
+ * without modification.
+ * 2. Redistributions in binary form must reproduce at minimum a disclaimer
+ * substantially similar to the "NO WARRANTY" disclaimer below
+ * ("Disclaimer") and any redistribution must be conditioned upon
+ * including a substantially similar Disclaimer requirement for further
+ * binary redistribution.
+ * 3. Neither the names of the above-listed copyright holders nor the names
+ * of any contributors may be used to endorse or promote products
derived
+ * from this software without specific prior written permission.
+ *
+ * Alternatively, this software may be distributed under the terms of the
+ * GNU General Public License ("GPL") version 2 as published by the Free
+ * Software Foundation.
+ *
+ * NO WARRANTY
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY, OR
CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+ * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING
+ * IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGES.
+ */
#include "acpi.h"
+#include <fcntl.h>
#include <stdio.h>
#include <stdlib.h>
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <sys/mman.h>
+#include <unistd.h>
#define _COMPONENT ACPI_OS_SERVICES
ACPI_MODULE_NAME ("oslinuxtbl")
+/*
+ * Linux specific code
+ *
+ * Read in all the tables we can find and put them in a nice table at
+ * startup so we don't have to keep doing searches for each table.
+ *
+ */
+
+#define ACPI_OS_MAX_TABLE_INDEX 128
+#define MAX_SIG 5
+#define MAX_PATH 256
+#define MAX_BUF (128*1024) /* no table bigger than 128 KiB? */
+#define TABLE_HEADER 256
+
+static int tables_found = 0;
+
+typedef struct table_info {
+ char *filename;
+ char signature[MAX_SIG];
+ int instance; /* only for some tables */
+ int dynamic;
+ ACPI_PHYSICAL_ADDRESS address;
+ int size;
+ unsigned char *data;
+} TABLE_INFO;
+
+static TABLE_INFO acpi_tables[ACPI_OS_MAX_TABLE_INDEX];
+static int next_index = 0;
+
+/******************************************************************************
+ *
+ * FUNCTION: read_table
+ *
+ * PARAMETERS: fullpath - absolute path to /sys ACPI table
+ *
+ * RETURN: If the table can be read, returns 0; returns non-zero
+ * otherwise.
+ *
+ * DESCRIPTION: Read in the contents of an ACPI table as stored in
+ * /sys/firmware/acpi. Fill in the contents of an entry
+ * in acpi_tables with the size and data of the table.
+ *
+
*****************************************************************************/
+
+static int
+read_table(
+ char *fullpath)
+{
+ static unsigned char buf[MAX_BUF];
+ FILE *fd;
+ int result;
+ int cnt;
+ unsigned char *up;
+
+ result = 1;
+
+ /* read the table once to figure out it's size, then put it in a
buffer */
+ memset(buf, 0, MAX_BUF);
+ fd = fopen(fullpath, "r");
+ if (fd)
+ {
+ cnt = fread(buf, MAX_BUF, 1, fd);
+ if (feof(fd))
+ {
+ cnt = ftell(fd);
+ up = (unsigned char *)malloc(cnt);
+ if (up)
+ {
+ memcpy(up, buf, cnt);
+ acpi_tables[next_index].size = cnt;
+ acpi_tables[next_index].data = up;
+ memcpy(acpi_tables[next_index].signature, up, 4);
+ result = 0;
+ }
+ }
+ fclose(fd);
+ }
+ else
+ {
+ fprintf(stderr, "Cannot open %s\n", fullpath);
+ }
+ return (result);
+}
+
+/******************************************************************************
+ *
+ * FUNCTION: get_table_info
+ *
+ * PARAMETERS: dirpath - absolute path to a /sys/firmware/acpi
+ * directory
+ *
+ * RETURN: Status: AE_OK if all the ACPI tables in the directory
+ * given by dirpath can be read in. If not, AE_LIMIT will
+ * be returned if there is insufficient space for the table,
+ * and AE_ERROR otherwise.
+ *
+ * DESCRIPTION: Given a directory path in /sys, read in each of the files
+ * in that directory as an ACPI table. Store any information
+ * gathered about the table in an acpi_tables entry.
+ *
+
*****************************************************************************/
+
+static ACPI_STATUS
+get_table_info(
+ char *dirpath)
+{
+ ACPI_STATUS result;
+ void *dir;
+ char *fpath;
+ char fullpath[MAX_PATH];
+ int tmp;
+ char *p;
+ struct stat buf;
+
+ result = AE_ERROR;
+ dir = AcpiOsOpenDirectory(dirpath, "*", REQUEST_FILE_ONLY);
+ if (dir)
+ {
+ fpath = AcpiOsGetNextFilename(dir);
+ while (fpath)
+ {
+ memset(fullpath, 0, MAX_PATH);
+ strcpy(fullpath, dirpath);
+ strcat(fullpath, "/");
+ strcat(fullpath, fpath);
+ if (next_index >= ACPI_OS_MAX_TABLE_INDEX)
+ {
+ result = AE_LIMIT;
+ break;
+ }
+ else
+ {
+ tmp = strlen(fullpath) + 1;
+ p = (char *)malloc(tmp);
+ if (p)
+ {
+ memset(p, 0, tmp);
+ acpi_tables[next_index].filename = p;
+ }
+ else
+ {
+ result = AE_LIMIT;
+ break;
+ }
+ strcpy(acpi_tables[next_index].filename, fullpath);
+
+ acpi_tables[next_index].dynamic = 0;
+
+ if (strlen(fpath) > 4)
+ {
+ tmp = strtol(&(fpath[4]), NULL, 10);
+ acpi_tables[next_index].instance = tmp;
+ }
+
+ if (stat(fullpath, &buf) == 0)
+ {
+ if (read_table(fullpath) == 0)
+ {
+ result = AE_OK;
+ }
+ else
+ {
+ result = AE_LIMIT;
+ break;
+ }
+ }
+
+ next_index++;
+ }
+ fpath = AcpiOsGetNextFilename(dir);
+ }
+
+ AcpiOsCloseDirectory(dir);
+ result = AE_OK;
+ }
+ else
+ {
+ result = AE_ERROR;
+ }
+
+ return (result);
+}
+
+/******************************************************************************
+ *
+ * FUNCTION: init_table_info
+ *
+ * PARAMETERS: none
+ *
+ * RETURN: Status: AE_OK if initialization was properly done. If
+ * it was not, an error status indicating what occurred.
+ *
+ * DESCRIPTION: This routine is called to set up a file global structure
+ * (acpi_tables) that will then be used by the other
+ * AcpiOsGetTableBy* routines to find and print out info
+ * about one of the current systems ACPI tables. The
+ * routine is also set up so that it should only do the
+ * heavy initialization work once on first being called.
+ *
+
*****************************************************************************/
+
+static ACPI_STATUS
+init_table_info(
+ void)
+{
+ ACPI_STATUS result;
+
+ if (tables_found)
+ return (AE_OK);
+
+ /* must be superuser to read the tables */
+ if (getuid() != 0)
+ {
+ fprintf(stderr, "Must be run as superuser\n");
+ return (AE_SUPPORT);
+ }
+
+ /* basic initialization */
+ memset(acpi_tables, 0, ACPI_OS_MAX_TABLE_INDEX * sizeof(TABLE_INFO));
+
+ /*
+ * Look for all the files in /sys/firmware/acpi/tables and in
+ * /sys/firmware/acpi/tables/dynamic; each one is a table.
+ */
+ result = get_table_info("/sys/firmware/acpi/tables");
+ if (result == AE_OK)
+ {
+ result = get_table_info("/sys/firmware/acpi/tables/dynamic");
+ }
+
+ if (result == AE_OK)
+ {
+ tables_found = 1;
+ }
+ return (result);
+}
/******************************************************************************
*
@@ -170,9 +342,16 @@ AcpiOsGetTableByIndex (
ACPI_TABLE_HEADER **Table,
ACPI_PHYSICAL_ADDRESS *Address)
{
-
- fprintf (stderr, "Linux version not implemented yet\n");
- return (AE_SUPPORT);
+ if (init_table_info() == AE_OK)
+ {
+ if (Index < next_index)
+ {
+ *Table = (ACPI_TABLE_HEADER *)acpi_tables[Index].data;
+ *Address = 0;
+ return (AE_OK);
+ }
+ }
+ return (AE_LIMIT);
}
@@ -203,7 +382,21 @@ AcpiOsGetTableByName (
ACPI_TABLE_HEADER **Table,
ACPI_PHYSICAL_ADDRESS *Address)
{
+ int ii;
- fprintf (stderr, "Linux version not implemented yet\n");
- return (AE_SUPPORT);
+ if (init_table_info() == AE_OK)
+ {
+ for (ii = 0; ii < next_index; ii++)
+ {
+ if ((strncmp(Signature, acpi_tables[ii].signature, 4) == 0) &&
+ (acpi_tables[ii].instance == Instance))
+ {
+ *Table = (ACPI_TABLE_HEADER *)acpi_tables[ii].data;
+ *Address = 0;
+ return (AE_OK);
+ }
+ }
+ return (AE_LIMIT);
+ }
+ return (AE_NOT_FOUND);
}
--
1.8.1.4
--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3(a)redhat.com
-----------------------------------
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-06-05 22:12 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-01 4:22 [Devel] [PATCH] ACPICA: acpidump -- provide initial implementation for Linux Al Stone
-- strict thread matches above, loose matches on Subject: below --
2013-06-05 22:12 Al Stone
2013-06-05 0:56 Moore, Robert
2013-06-02 20:04 Al Stone
2013-06-01 0:46 Moore, Robert
2013-06-01 0:20 Al Stone
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.