Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20221208232536.591347-1-stuart.w.hayes@gmail.com>
In-Reply-To: <20221208232536.591347-1-stuart.w.hayes@gmail.com>
From:   "Rafael J. Wysocki" <rafael@kernel.org>
Date:   Fri, 30 Dec 2022 18:45:32 +0100
Message-ID: <CAJZ5v0jq_z3H5Uw7vM1998pgtUyHE0M19aOg+mf1O=UgApf7cg@mail.gmail.com>
Subject: Re: [PATCH] acpi: add support for the NBFT
To:     Stuart Hayes <stuart.w.hayes@gmail.com>
Cc:     "Rafael J . Wysocki" <rafael@kernel.org>,
        Len Brown <lenb@kernel.org>, linux-kernel@vger.kernel.org,
        linux-acpi@vger.kernel.org, John Meneghini <jmeneghi@redhat.com>,
        Charles Rose <charles_rose@dell.com>,
        Doug Farley <Douglas_Farley@dell.com>,
        Lenny Szubowicz <lszubowi@redhat.com>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Fri, Dec 9, 2022 at 12:25 AM Stuart Hayes <stuart.w.hayes@gmail.com> wrote:
>
> Add support for the NVMe Boot Firmware Table (NBFT) to facilitate
> booting from NVM Express namespaces which are accessed via
> NVMe over Fabrics (NVMe-oF).
>
> Signed-off-by: Stuart Hayes <stuart.w.hayes@gmail.com>

Tentatively applied as 6.3 material, but it was unclear what the S-o-b
tags below mean, so I've dropped them.

If you are not the original author of the patch, you should add a From
line pointing to the original author to it and the corresponding S-o-b
tag along with your S-o-b.

If you have developed the patch in collaboration with someone, there
should be a Co-developed-by tag pointing to the other author along
with the corresponding S-o-b tag.

S-o-b alone is meaningful only if you are sending a patch from someone
else, for example as a code maintainer.

> Signed-off-by: Doug Farley <Douglas_Farley@dell.com>
> Signed-off-by: Lenny Szubowicz <lszubowi@redhat.com>
> ---
>  drivers/acpi/tables.c | 3 ++-
>  include/acpi/actbl1.h | 2 ++
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
> index 47ec11d4c68e..f390c5883b56 100644
> --- a/drivers/acpi/tables.c
> +++ b/drivers/acpi/tables.c
> @@ -545,7 +545,8 @@ static const char table_sigs[][ACPI_NAMESEG_SIZE] __initconst = {
>         ACPI_SIG_WDDT, ACPI_SIG_WDRT, ACPI_SIG_DSDT, ACPI_SIG_FADT,
>         ACPI_SIG_PSDT, ACPI_SIG_RSDT, ACPI_SIG_XSDT, ACPI_SIG_SSDT,
>         ACPI_SIG_IORT, ACPI_SIG_NFIT, ACPI_SIG_HMAT, ACPI_SIG_PPTT,
> -       ACPI_SIG_NHLT, ACPI_SIG_AEST, ACPI_SIG_CEDT, ACPI_SIG_AGDI };
> +       ACPI_SIG_NHLT, ACPI_SIG_AEST, ACPI_SIG_CEDT, ACPI_SIG_AGDI,
> +       ACPI_SIG_NBFT };
>
>  #define ACPI_HEADER_SIZE sizeof(struct acpi_table_header)
>
> diff --git a/include/acpi/actbl1.h b/include/acpi/actbl1.h
> index 15c78678c5d3..6ec43410288a 100644
> --- a/include/acpi/actbl1.h
> +++ b/include/acpi/actbl1.h
> @@ -49,6 +49,8 @@
>  #define ACPI_SIG_S3PT           "S3PT" /* S3 Performance (sub)Table */
>  #define ACPI_SIG_PCCS           "PCC"  /* PCC Shared Memory Region */
>
> +#define ACPI_SIG_NBFT          "NBFT"  /* NVMe Boot Firmware Table */
> +
>  /* Reserved table signatures */
>
>  #define ACPI_SIG_MATR           "MATR" /* Memory Address Translation Table */
> --
> 2.31.1
>