linux-kernel - Re: [PATCH v6 02/27] ACPI/CDAT: Add CDAT/DSMAS shared and read only flag values

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] [day] [month] [year] [list]

Message-ID: <CAJZ5v0gRM_cgBPivkey7Z8P0kOFfhoBZC49RSwe9s_Cuax3Cag@mail.gmail.com>
Date: Tue, 5 Nov 2024 21:04:09 +0100
From: "Rafael J. Wysocki" <rafael@...nel.org>
To: Ira Weiny <ira.weiny@...el.com>
Cc: Dave Jiang <dave.jiang@...el.com>, Fan Ni <fan.ni@...sung.com>, 
	Jonathan Cameron <Jonathan.Cameron@...wei.com>, Navneet Singh <navneet.singh@...el.com>, 
	Jonathan Corbet <corbet@....net>, Andrew Morton <akpm@...ux-foundation.org>, 
	Dan Williams <dan.j.williams@...el.com>, Davidlohr Bueso <dave@...olabs.net>, 
	Alison Schofield <alison.schofield@...el.com>, Vishal Verma <vishal.l.verma@...el.com>, 
	linux-cxl@...r.kernel.org, linux-doc@...r.kernel.org, nvdimm@...ts.linux.dev, 
	linux-kernel@...r.kernel.org, Robert Moore <robert.moore@...el.com>, 
	Len Brown <lenb@...nel.org>, "Rafael J. Wysocki" <rafael.j.wysocki@...el.com>, linux-acpi@...r.kernel.org, 
	acpica-devel@...ts.linux.dev
Subject: Re: [PATCH v6 02/27] ACPI/CDAT: Add CDAT/DSMAS shared and read only
 flag values

On Tue, Nov 5, 2024 at 7:38 PM Ira Weiny <ira.weiny@...el.com> wrote:
>
> The Coherent Device Attribute Table (CDAT) Device Scoped Memory Affinity
> Structure (DSMAS) version 1.04 [1] defines flags to indicate if a DPA range
> is read only and/or shared.
>
> Add read only and shareable flag definitions.
>
> This change was merged in ACPI via PR 976.[2]

s/ACPI/ACPICA/

>
> Link: https://uefi.org/sites/default/files/resources/Coherent%20Device%20Attribute%20Table_1.04%20published_0.pdf [1]
> Link: https://github.com/acpica/acpica/pull/976 [2]
> Cc: Robert Moore <robert.moore@...el.com>
> Cc: Len Brown <lenb@...nel.org>
> Cc: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
> Cc: linux-acpi@...r.kernel.org
> Cc: acpica-devel@...ts.linux.dev
> Signed-off-by: Ira Weiny <ira.weiny@...el.com>

With the above change:

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>

> ---
>  include/acpi/actbl1.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/include/acpi/actbl1.h b/include/acpi/actbl1.h
> index 199afc2cd122ca8b383b1c9286f8c8cc33842fae..387fc821703a80b324637743f0d5afe03b8d7943 100644
> --- a/include/acpi/actbl1.h
> +++ b/include/acpi/actbl1.h
> @@ -403,6 +403,8 @@ struct acpi_cdat_dsmas {
>  /* Flags for subtable above */
>
>  #define ACPI_CDAT_DSMAS_NON_VOLATILE        (1 << 2)
> +#define ACPI_CDAT_DSMAS_SHAREABLE           (1 << 3)
> +#define ACPI_CDAT_DSMAS_READ_ONLY           (1 << 6)
>
>  /* Subtable 1: Device scoped Latency and Bandwidth Information Structure (DSLBIS) */
>
>
> --
> 2.47.0
>
>