lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Fri, 22 May 2015 15:28:04 +0000
From:	"Moore, Robert" <robert.moore@...el.com>
To:	Linda Knippers <linda.knippers@...com>,
	"Zheng, Lv" <lv.zheng@...el.com>,
	"Wysocki, Rafael J" <rafael.j.wysocki@...el.com>,
	"Brown, Len" <len.brown@...el.com>
CC:	Lv Zheng <zetalog@...il.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>
Subject: RE: [PATCH 18/19] ACPICA: ACPI 6.0: Add support for NFIT table.

Yes, you are correct, the NFIT strings are wrong. For example, the first string for UUID_VOLATILE_MEMORY should be:


"7305944F-FDDA-44E3-B16C-3F22D252E5D0"

You might want to verify this.

We will fix the original patch ASAP.

Thanks,
Bob


> -----Original Message-----
> From: Linda Knippers [mailto:linda.knippers@...com]
> Sent: Friday, May 22, 2015 8:17 AM
> To: Zheng, Lv; Wysocki, Rafael J; Brown, Len
> Cc: Lv Zheng; linux-kernel@...r.kernel.org; linux-acpi@...r.kernel.org;
> Moore, Robert
> Subject: Re: [PATCH 18/19] ACPICA: ACPI 6.0: Add support for NFIT table.
> 
> On 5/20/2015 10:31 PM, Lv Zheng wrote:
> > From: Bob Moore <robert.moore@...el.com>
> >
> > ACPICA commit e4e17ca361373e9b81494bb4ca697a12cef3cba6
> >
> > NVDIMM Firmware Interface Table.
> >
> > Link: https://github.com/acpica/acpica/commit/e4e17ca3
> > Signed-off-by: Bob Moore <robert.moore@...el.com>
> > Signed-off-by: Lv Zheng <lv.zheng@...el.com>
> > ---
> >  drivers/acpi/acpica/uthex.c |    4 +-
> >  include/acpi/actbl1.h       |  154
> +++++++++++++++++++++++++++++++++++++++++++
> >  include/acpi/acuuid.h       |   64 ++++++++++++++++++
> >  3 files changed, 220 insertions(+), 2 deletions(-)  create mode
> > 100644 include/acpi/acuuid.h
> >
> > diff --git a/drivers/acpi/acpica/uthex.c b/drivers/acpi/acpica/uthex.c
> > index aa44827..fda8b3d 100644
> > --- a/drivers/acpi/acpica/uthex.c
> > +++ b/drivers/acpi/acpica/uthex.c
> > @@ -75,9 +75,9 @@ char acpi_ut_hex_to_ascii_char(u64 integer, u32
> > position)
> >
> >
> /*************************************************************************
> ******
> >   *
> > - * FUNCTION:    acpi_ut_hex_char_to_value
> > + * FUNCTION:    acpi_ut_ascii_char_to_hex
> >   *
> > - * PARAMETERS:  ascii_char            - Hex character in Ascii
> > + * PARAMETERS:  hex_char                - Hex character in Ascii
> >   *
> >   * RETURN:      The binary value of the ascii/hex character
> >   *
> > diff --git a/include/acpi/actbl1.h b/include/acpi/actbl1.h index
> > cadf21c..06b61f0 100644
> > --- a/include/acpi/actbl1.h
> > +++ b/include/acpi/actbl1.h
> > @@ -71,6 +71,7 @@
> >  #define ACPI_SIG_SBST           "SBST"	/* Smart Battery Specification
> Table */
> >  #define ACPI_SIG_SLIT           "SLIT"	/* System Locality Distance
> Information Table */
> >  #define ACPI_SIG_SRAT           "SRAT"	/* System Resource Affinity
> Table */
> > +#define ACPI_SIG_NFIT           "NFIT"	/* NVDIMM Firmware Interface
> Table */
> >
> >  /*
> >   * All tables must be byte-packed to match the ACPI specification,
> > since @@ -922,6 +923,159 @@ struct acpi_msct_proximity {
> >
> >
> /*************************************************************************
> ******
> >   *
> > + * NFIT - NVDIMM Interface Table (ACPI 6.0)
> > + *        Version 1
> > + *
> > + ********************************************************************
> > + **********/
> > +
> > +struct acpi_table_nfit {
> > +	struct acpi_table_header header;	/* Common ACPI table header */
> > +	u32 reserved;		/* Reserved, must be zero */
> > +};
> > +
> > +/* Subtable header for NFIT */
> > +
> > +struct acpi_nfit_header {
> > +	u16 type;
> > +	u16 length;
> > +};
> > +
> > +/* Values for subtable type in struct acpi_nfit_header */
> > +
> > +enum acpi_nfit_type {
> > +	ACPI_NFIT_TYPE_SYSTEM_ADDRESS = 0,
> > +	ACPI_NFIT_TYPE_MEMORY_MAP = 1,
> > +	ACPI_NFIT_TYPE_INTERLEAVE = 2,
> > +	ACPI_NFIT_TYPE_SMBIOS = 3,
> > +	ACPI_NFIT_TYPE_CONTROL_REGION = 4,
> > +	ACPI_NFIT_TYPE_DATA_REGION = 5,
> > +	ACPI_NFIT_TYPE_FLUSH_ADDRESS = 6,
> > +	ACPI_NFIT_TYPE_RESERVED = 7	/* 7 and greater are reserved */
> > +};
> > +
> > +/*
> > + * NFIT Subtables
> > + */
> > +
> > +/* 0: System Physical Address Range Structure */
> > +
> > +struct acpi_nfit_system_address {
> > +	struct acpi_nfit_header header;
> > +	u16 range_index;
> > +	u16 flags;
> > +	u32 reserved;		/* Reseved, must be zero */
> > +	u32 proximity_domain;
> > +	u8 range_guid[16];
> > +	u64 address;
> > +	u64 length;
> > +	u64 memory_mapping;
> > +};
> > +
> > +/* Flags */
> > +
> > +#define ACPI_NFIT_ADD_ONLINE_ONLY       (1)	/* 00: Add/Online
> Operation Only */
> > +#define ACPI_NFIT_PROXIMITY_VALID       (1<<1)	/* 01: Proximity
> Domain Valid */
> > +
> > +/* Range Type GUIDs appear in the include/acuuid.h file */
> > +
> > +/* 1: Memory Device to System Address Range Map Structure */
> > +
> > +struct acpi_nfit_memory_map {
> > +	struct acpi_nfit_header header;
> > +	u32 device_handle;
> > +	u16 physical_id;
> > +	u16 region_id;
> > +	u16 range_index;
> > +	u16 region_index;
> > +	u64 region_size;
> > +	u64 region_offset;
> > +	u64 address;
> > +	u16 interleave_index;
> > +	u16 interleave_ways;
> > +	u16 flags;
> > +	u16 reserved;		/* Reserved, must be zero */
> > +};
> > +
> > +/* Flags */
> > +
> > +#define ACPI_NFIT_MEM_SAVE_FAILED       (1)	/* 00: Last SAVE to Memory
> Device failed */
> > +#define ACPI_NFIT_MEM_RESTORE_FAILED    (1<<1)	/* 01: Last RESTORE
> from Memory Device failed */
> > +#define ACPI_NFIT_MEM_FLUSH_FAILED      (1<<2)	/* 02: Platform
> flush failed */
> > +#define ACPI_NFIT_MEM_ARMED             (1<<3)	/* 03: Memory Device
> observed to be not armed */
> > +#define ACPI_NFIT_MEM_HEALTH_OBSERVED   (1<<4)	/* 04: Memory Device
> observed SMART/health events */
> > +#define ACPI_NFIT_MEM_HEALTH_ENABLED    (1<<5)	/* 05: SMART/health
> events enabled */
> > +
> > +/* 2: Interleave Structure */
> > +
> > +struct acpi_nfit_interleave {
> > +	struct acpi_nfit_header header;
> > +	u16 interleave_index;
> > +	u16 reserved;		/* Reserved, must be zero */
> > +	u32 line_count;
> > +	u32 line_size;
> > +	u32 line_offset[1];	/* Variable length */
> > +};
> > +
> > +/* 3: SMBIOS Management Information Structure */
> > +
> > +struct acpi_nfit_smbios {
> > +	struct acpi_nfit_header header;
> > +	u32 reserved;		/* Reserved, must be zero */
> > +	u8 data[1];		/* Variable length */
> > +};
> > +
> > +/* 4: NVDIMM Control Region Structure */
> > +
> > +struct acpi_nfit_control_region {
> > +	struct acpi_nfit_header header;
> > +	u16 region_index;
> > +	u16 vendor_id;
> > +	u16 device_id;
> > +	u16 revision_id;
> > +	u16 subsystem_vendor_id;
> > +	u16 subsystem_device_id;
> > +	u16 subsystem_revision_id;
> > +	u8 reserved[6];		/* Reserved, must be zero */
> > +	u32 serial_number;
> > +	u16 code;
> > +	u16 windows;
> > +	u64 window_size;
> > +	u64 command_offset;
> > +	u64 command_size;
> > +	u64 status_offset;
> > +	u64 status_size;
> > +	u16 flags;
> > +	u8 reserved1[6];	/* Reserved, must be zero */
> > +};
> > +
> > +/* Flags */
> > +
> > +#define ACPI_NFIT_CONTROL_BUFFERED      (1)	/* Block Data Windows
> implementation is buffered */
> > +
> > +/* 5: NVDIMM Block Data Window Region Structure */
> > +
> > +struct acpi_nfit_data_region {
> > +	struct acpi_nfit_header header;
> > +	u16 region_index;
> > +	u16 windows;
> > +	u64 offset;
> > +	u64 size;
> > +	u64 capacity;
> > +	u64 start_address;
> > +};
> > +
> > +/* 6: Flush Hint Address Structure */
> > +
> > +struct acpi_nfit_flush_address {
> > +	struct acpi_nfit_header header;
> > +	u32 device_handle;
> > +	u16 hint_count;
> > +	u8 reserved[6];		/* Reserved, must be zero */
> > +	u64 hint_address[1];	/* Variable length */
> > +};
> > +
> > +/********************************************************************
> > +***********
> > + *
> >   * SBST - Smart Battery Specification Table
> >   *        Version 1
> >   *
> > diff --git a/include/acpi/acuuid.h b/include/acpi/acuuid.h new file
> > mode 100644 index 0000000..4955d5e
> > --- /dev/null
> > +++ b/include/acpi/acuuid.h
> > @@ -0,0 +1,64 @@
> > +/********************************************************************
> > +**********
> > + *
> > + * Name: acuuid.h - ACPI-related UUID/GUID definitions
> > + *
> > +
> > +*********************************************************************
> > +********/
> > +
> > +/*
> > + * Copyright (C) 2000 - 2015, Intel Corp.
> > + * All rights reserved.
> > + *
> > + * 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.
> > + */
> > +
> > +#ifndef __ACUUID_H__
> > +#define __ACUUID_H__
> > +
> > +/*
> > + * Note1: UUIDs and GUIDs are defined to be identical in ACPI.
> > + *
> > + * Note2: This file is standalone and should remain that way.
> > + */
> > +
> > +/* NFIT/NVDIMM */
> > +
> > +#define UUID_VOLATILE_MEMORY            "4F940573-DAFD-E344-B16C-
> 3F22D252E5D0"
> > +#define UUID_PERSISTENT_MEMORY          "79D3F066-F3B4-7440-AC43-
> 0D3318B78CDB"
> > +#define UUID_CONTROL_REGION             "F601F792-B413-5D40-910B-
> 299367E8234C"
> > +#define UUID_DATA_REGION                "3005AF91-865D-0E47-A6B0-
> 0A2DB9408249"
> > +#define UUID_VOLATILE_VIRTUAL_DISK      "5A53AB77-FC45-4B62-5560-
> F7B281D1F96E"
> > +#define UUID_VOLATILE_VIRTUAL_CD        "30BD5A3D-7541-CE87-6D64-
> D2ADE523C4BB"
> > +#define UUID_PERSISTENT_VIRTUAL_DISK    "C902EA5C-074D-69D3-269F-
> 4496FBE096F9"
> > +#define UUID_PERSISTENT_VIRTUAL_CD      "88810108-CD42-48BB-100F-
> 5387D53DED3D"
> 
> I think we've concluded in the review of a different patch set with these
> same definitions that these are incorrect.  Perhaps Bob can confirm?
> 
> -- ljk
> 
> 
> > +
> > +#endif				/* __AUUID_H__ */
> >

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists