| 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
| ||
|
Message-ID: <Y9k7z4zUlsSz7/sE@nanopsycho>
Date: Tue, 31 Jan 2023 17:03:27 +0100
From: Jiri Pirko <jiri@...nulli.us>
To: alejandro.lucero-palau@....com
Cc: netdev@...r.kernel.org, linux-net-drivers@....com,
davem@...emloft.net, kuba@...nel.org, pabeni@...hat.com,
edumazet@...gle.com, habetsm.xilinx@...il.com,
ecree.xilinx@...il.com, linux-doc@...r.kernel.org, corbet@....net,
jiri@...dia.com
Subject: Re: [PATCH v4 net-next 2/8] sfc: add devlink info support for ef100
Tue, Jan 31, 2023 at 03:58:16PM CET, alejandro.lucero-palau@....com wrote:
>From: Alejandro Lucero <alejandro.lucero-palau@....com>
>
>Support for devlink info command.
>
>Signed-off-by: Alejandro Lucero <alejandro.lucero-palau@....com>
>---
> Documentation/networking/devlink/sfc.rst | 57 ++++
> MAINTAINERS | 1 +
> drivers/net/ethernet/sfc/efx_devlink.c | 404 +++++++++++++++++++++++
> drivers/net/ethernet/sfc/efx_devlink.h | 17 +
> drivers/net/ethernet/sfc/mcdi.c | 72 ++++
> drivers/net/ethernet/sfc/mcdi.h | 3 +
> 6 files changed, 554 insertions(+)
> create mode 100644 Documentation/networking/devlink/sfc.rst
>
>diff --git a/Documentation/networking/devlink/sfc.rst b/Documentation/networking/devlink/sfc.rst
>new file mode 100644
>index 000000000000..e2541a2f18ee
>--- /dev/null
>+++ b/Documentation/networking/devlink/sfc.rst
>@@ -0,0 +1,57 @@
>+.. SPDX-License-Identifier: GPL-2.0
>+
>+===================
>+sfc devlink support
>+===================
>+
>+This document describes the devlink features implemented by the ``sfc``
>+device driver for the ef100 device.
>+
>+Info versions
>+=============
>+
>+The ``sfc`` driver reports the following versions
>+
>+.. list-table:: devlink info versions implemented
>+ :widths: 5 5 90
>+
>+ * - Name
>+ - Type
>+ - Description
>+ * - ``fw.mgmt.suc``
>+ - running
>+ - For boards where the management function is split between multiple
>+ control units, this is the SUC control unit's firmware version.
>+ * - ``fw.mgmt.cmc``
>+ - running
>+ - For boards where the management function is split between multiple
>+ control units, this is the CMC control unit's firmware version.
>+ * - ``fpga.rev``
>+ - running
>+ - FPGA design revision.
>+ * - ``fpga.app``
>+ - running
>+ - Datapath programmable logic version.
>+ * - ``fw.app``
>+ - running
>+ - Datapath software/microcode/firmware version.
>+ * - ``coproc.boot``
>+ - running
>+ - SmartNIC application co-processor (APU) first stage boot loader version.
>+ * - ``coproc.uboot``
>+ - running
>+ - SmartNIC application co-processor (APU) co-operating system loader version.
>+ * - ``coproc.main``
>+ - running
>+ - SmartNIC application co-processor (APU) main operating system version.
>+ * - ``coproc.recovery``
>+ - running
>+ - SmartNIC application co-processor (APU) recovery operating system version.
>+ * - ``fw.exprom``
>+ - running
>+ - Expansion ROM version. For boards where the expansion ROM is split between
>+ multiple images (e.g. PXE and UEFI), this is the specifically the PXE boot
>+ ROM version.
>+ * - ``fw.uefi``
>+ - running
>+ - UEFI driver version (No UNDI support).
>diff --git a/MAINTAINERS b/MAINTAINERS
>index f82dd8d43c2b..57c9cb55cfd3 100644
>--- a/MAINTAINERS
>+++ b/MAINTAINERS
>@@ -18920,6 +18920,7 @@ M: Martin Habets <habetsm.xilinx@...il.com>
> L: netdev@...r.kernel.org
> S: Supported
> F: drivers/net/ethernet/sfc/
>+F: Documentation/networking/devlink/sfc.rst
>
> SFF/SFP/SFP+ MODULE SUPPORT
> M: Russell King <linux@...linux.org.uk>
>diff --git a/drivers/net/ethernet/sfc/efx_devlink.c b/drivers/net/ethernet/sfc/efx_devlink.c
>index fab06aaa4b8a..ff5adfe3905e 100644
>--- a/drivers/net/ethernet/sfc/efx_devlink.c
>+++ b/drivers/net/ethernet/sfc/efx_devlink.c
>@@ -21,7 +21,411 @@ struct efx_devlink {
> struct efx_nic *efx;
> };
>
>+static int efx_devlink_info_nvram_partition(struct efx_nic *efx,
>+ struct devlink_info_req *req,
>+ unsigned int partition_type,
>+ const char *version_name)
>+{
>+ char buf[EFX_MAX_VERSION_INFO_LEN];
>+ u16 version[4];
>+ int rc;
>+
>+ rc = efx_mcdi_nvram_metadata(efx, partition_type, NULL, version, NULL,
>+ 0);
>+ if (rc)
>+ return rc;
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u", version[0],
>+ version[1], version[2], version[3]);
>+ devlink_info_version_stored_put(req, version_name, buf);
>+
>+ return 0;
>+}
>+
>+static void efx_devlink_info_stored_versions(struct efx_nic *efx,
>+ struct devlink_info_req *req)
>+{
>+ efx_devlink_info_nvram_partition(efx, req, NVRAM_PARTITION_TYPE_BUNDLE,
>+ DEVLINK_INFO_VERSION_GENERIC_FW_BUNDLE_ID);
>+ efx_devlink_info_nvram_partition(efx, req,
>+ NVRAM_PARTITION_TYPE_MC_FIRMWARE,
>+ DEVLINK_INFO_VERSION_GENERIC_FW_MGMT);
>+ efx_devlink_info_nvram_partition(efx, req,
>+ NVRAM_PARTITION_TYPE_SUC_FIRMWARE,
>+ EFX_DEVLINK_INFO_VERSION_FW_MGMT_SUC);
>+ efx_devlink_info_nvram_partition(efx, req,
>+ NVRAM_PARTITION_TYPE_EXPANSION_ROM,
>+ EFX_DEVLINK_INFO_VERSION_FW_EXPROM);
>+ efx_devlink_info_nvram_partition(efx, req,
>+ NVRAM_PARTITION_TYPE_EXPANSION_UEFI,
>+ EFX_DEVLINK_INFO_VERSION_FW_UEFI);
>+}
>+
>+#define EFX_VER_FLAG(_f) \
>+ (MC_CMD_GET_VERSION_V5_OUT_ ## _f ## _PRESENT_LBN)
>+
>+static void efx_devlink_info_running_v2(struct efx_nic *efx,
>+ struct devlink_info_req *req,
>+ unsigned int flags, efx_dword_t *outbuf)
>+{
>+ char buf[EFX_MAX_VERSION_INFO_LEN];
>+ union {
>+ const __le32 *dwords;
>+ const __le16 *words;
>+ const char *str;
>+ } ver;
>+ struct rtc_time build_date;
>+ unsigned int build_id;
>+ size_t offset;
>+ u64 tstamp;
>+
>+ if (flags & BIT(EFX_VER_FLAG(BOARD_EXT_INFO))) {
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%s",
>+ MCDI_PTR(outbuf, GET_VERSION_V2_OUT_BOARD_NAME));
>+ devlink_info_version_fixed_put(req,
>+ DEVLINK_INFO_VERSION_GENERIC_BOARD_ID,
>+ buf);
>+
>+ /* Favour full board version if present (in V5 or later) */
>+ if (~flags & BIT(EFX_VER_FLAG(BOARD_VERSION))) {
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u",
>+ MCDI_DWORD(outbuf,
>+ GET_VERSION_V2_OUT_BOARD_REVISION));
>+ devlink_info_version_fixed_put(req,
>+ DEVLINK_INFO_VERSION_GENERIC_BOARD_REV,
>+ buf);
>+ }
>+
>+ ver.str = MCDI_PTR(outbuf, GET_VERSION_V2_OUT_BOARD_SERIAL);
>+ if (ver.str[0])
>+ devlink_info_board_serial_number_put(req, ver.str);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(FPGA_EXT_INFO))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V2_OUT_FPGA_VERSION);
>+ offset = snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u_%c%u",
>+ le32_to_cpu(ver.dwords[0]),
>+ 'A' + le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]));
>+
>+ ver.str = MCDI_PTR(outbuf, GET_VERSION_V2_OUT_FPGA_EXTRA);
>+ if (ver.str[0])
>+ snprintf(&buf[offset], EFX_MAX_VERSION_INFO_LEN - offset,
>+ " (%s)", ver.str);
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_FPGA_REV,
>+ buf);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(CMC_EXT_INFO))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V2_OUT_CMCFW_VERSION);
>+ offset = snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]),
>+ le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ tstamp = MCDI_QWORD(outbuf,
>+ GET_VERSION_V2_OUT_CMCFW_BUILD_DATE);
>+ if (tstamp) {
>+ rtc_time64_to_tm(tstamp, &build_date);
>+ snprintf(&buf[offset], EFX_MAX_VERSION_INFO_LEN - offset,
>+ " (%ptRd)", &build_date);
>+ }
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_FW_MGMT_CMC,
>+ buf);
>+ }
>+
>+ ver.words = (__le16 *)MCDI_PTR(outbuf, GET_VERSION_V2_OUT_VERSION);
>+ offset = snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le16_to_cpu(ver.words[0]), le16_to_cpu(ver.words[1]),
>+ le16_to_cpu(ver.words[2]), le16_to_cpu(ver.words[3]));
>+ if (flags & BIT(EFX_VER_FLAG(MCFW_EXT_INFO))) {
>+ build_id = MCDI_DWORD(outbuf, GET_VERSION_V2_OUT_MCFW_BUILD_ID);
>+ snprintf(&buf[offset], EFX_MAX_VERSION_INFO_LEN - offset,
>+ " (%x) %s", build_id,
>+ MCDI_PTR(outbuf, GET_VERSION_V2_OUT_MCFW_BUILD_NAME));
>+ }
>+ devlink_info_version_running_put(req,
>+ DEVLINK_INFO_VERSION_GENERIC_FW_MGMT,
>+ buf);
>+
>+ if (flags & BIT(EFX_VER_FLAG(SUCFW_EXT_INFO))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V2_OUT_SUCFW_VERSION);
>+ tstamp = MCDI_QWORD(outbuf,
>+ GET_VERSION_V2_OUT_SUCFW_BUILD_DATE);
>+ rtc_time64_to_tm(tstamp, &build_date);
>+ build_id = MCDI_DWORD(outbuf, GET_VERSION_V2_OUT_SUCFW_CHIP_ID);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN,
>+ "%u.%u.%u.%u type %x (%ptRd)",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]), le32_to_cpu(ver.dwords[3]),
>+ build_id, &build_date);
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_FW_MGMT_SUC,
>+ buf);
>+ }
>+}
>+
>+static void efx_devlink_info_running_v3(struct efx_nic *efx,
>+ struct devlink_info_req *req,
>+ unsigned int flags, efx_dword_t *outbuf)
>+{
>+ char buf[EFX_MAX_VERSION_INFO_LEN];
>+ union {
>+ const __le32 *dwords;
>+ const __le16 *words;
>+ const char *str;
>+ } ver;
>+
>+ if (flags & BIT(EFX_VER_FLAG(DATAPATH_HW_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V3_OUT_DATAPATH_HW_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]));
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_DATAPATH_HW,
>+ buf);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(DATAPATH_FW_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V3_OUT_DATAPATH_FW_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]));
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_DATAPATH_FW,
>+ buf);
>+ }
>+}
>+
>+static void efx_devlink_info_running_v4(struct efx_nic *efx,
>+ struct devlink_info_req *req,
>+ unsigned int flags, efx_dword_t *outbuf)
>+{
>+ char buf[EFX_MAX_VERSION_INFO_LEN];
>+ union {
>+ const __le32 *dwords;
>+ const __le16 *words;
>+ const char *str;
>+ } ver;
>+
>+ if (flags & BIT(EFX_VER_FLAG(SOC_BOOT_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V4_OUT_SOC_BOOT_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_SOC_BOOT,
>+ buf);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(SOC_UBOOT_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V4_OUT_SOC_UBOOT_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_SOC_UBOOT,
>+ buf);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(SOC_MAIN_ROOTFS_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V4_OUT_SOC_MAIN_ROOTFS_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_SOC_MAIN,
>+ buf);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(SOC_RECOVERY_BUILDROOT_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V4_OUT_SOC_RECOVERY_BUILDROOT_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_SOC_RECOVERY,
>+ buf);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(SUCFW_VERSION)) &&
>+ ~flags & BIT(EFX_VER_FLAG(SUCFW_EXT_INFO))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V4_OUT_SUCFW_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ devlink_info_version_running_put(req,
>+ EFX_DEVLINK_INFO_VERSION_FW_MGMT_SUC,
>+ buf);
>+ }
>+}
>+
>+static void efx_devlink_info_running_v5(struct efx_nic *efx,
>+ struct devlink_info_req *req,
>+ unsigned int flags, efx_dword_t *outbuf)
>+{
>+ char buf[EFX_MAX_VERSION_INFO_LEN];
>+ union {
>+ const __le32 *dwords;
>+ const __le16 *words;
>+ const char *str;
>+ } ver;
>+
>+ if (flags & BIT(EFX_VER_FLAG(BOARD_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V5_OUT_BOARD_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ devlink_info_version_running_put(req,
>+ DEVLINK_INFO_VERSION_GENERIC_BOARD_REV,
>+ buf);
>+ }
>+
>+ if (flags & BIT(EFX_VER_FLAG(BUNDLE_VERSION))) {
>+ ver.dwords = (__le32 *)MCDI_PTR(outbuf,
>+ GET_VERSION_V5_OUT_BUNDLE_VERSION);
>+
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le32_to_cpu(ver.dwords[0]), le32_to_cpu(ver.dwords[1]),
>+ le32_to_cpu(ver.dwords[2]),
>+ le32_to_cpu(ver.dwords[3]));
>+
>+ devlink_info_version_running_put(req,
>+ DEVLINK_INFO_VERSION_GENERIC_FW_BUNDLE_ID,
>+ buf);
>+ }
>+}
>+
>+static void efx_devlink_info_running_versions(struct efx_nic *efx,
>+ struct devlink_info_req *req)
>+{
>+ MCDI_DECLARE_BUF(outbuf, MC_CMD_GET_VERSION_V5_OUT_LEN);
>+ MCDI_DECLARE_BUF(inbuf, MC_CMD_GET_VERSION_EXT_IN_LEN);
>+ char buf[EFX_MAX_VERSION_INFO_LEN];
>+ union {
>+ const __le32 *dwords;
>+ const __le16 *words;
>+ const char *str;
>+ } ver;
>+ size_t outlength;
>+ unsigned int flags;
>+ int rc;
>+
>+ rc = efx_mcdi_rpc(efx, MC_CMD_GET_VERSION, inbuf, sizeof(inbuf),
>+ outbuf, sizeof(outbuf), &outlength);
>+ if (rc || outlength < MC_CMD_GET_VERSION_OUT_LEN)
>+ return;
>+
>+ /* Handle previous output */
>+ if (outlength < MC_CMD_GET_VERSION_V2_OUT_LEN) {
>+ ver.words = (__le16 *)MCDI_PTR(outbuf,
>+ GET_VERSION_EXT_OUT_VERSION);
>+ snprintf(buf, EFX_MAX_VERSION_INFO_LEN, "%u.%u.%u.%u",
>+ le16_to_cpu(ver.words[0]),
>+ le16_to_cpu(ver.words[1]),
>+ le16_to_cpu(ver.words[2]),
>+ le16_to_cpu(ver.words[3]));
>+
>+ devlink_info_version_running_put(req,
>+ DEVLINK_INFO_VERSION_GENERIC_FW_MGMT,
>+ buf);
>+ return;
>+ }
>+
>+ /* Handle V2 additions */
>+ flags = MCDI_DWORD(outbuf, GET_VERSION_V2_OUT_FLAGS);
>+ efx_devlink_info_running_v2(efx, req, flags, outbuf);
>+
>+ if (outlength < MC_CMD_GET_VERSION_V3_OUT_LEN)
>+ return;
>+
>+ /* Handle V3 additions */
>+ efx_devlink_info_running_v3(efx, req, flags, outbuf);
>+
>+ if (outlength < MC_CMD_GET_VERSION_V4_OUT_LEN)
>+ return;
>+
>+ /* Handle V4 additions */
>+ efx_devlink_info_running_v4(efx, req, flags, outbuf);
>+
>+ if (outlength < MC_CMD_GET_VERSION_V5_OUT_LEN)
>+ return;
>+
>+ /* Handle V5 additions */
>+ efx_devlink_info_running_v5(efx, req, flags, outbuf);
>+}
>+
>+#define EFX_MAX_SERIALNUM_LEN (ETH_ALEN * 2 + 1)
>+
>+static void efx_devlink_info_board_cfg(struct efx_nic *efx,
>+ struct devlink_info_req *req)
>+{
>+ char sn[EFX_MAX_SERIALNUM_LEN];
>+ u8 mac_address[ETH_ALEN];
>+ int rc;
>+
>+ rc = efx_mcdi_get_board_cfg(efx, (u8 *)mac_address, NULL, NULL);
>+ if (!rc) {
>+ snprintf(sn, EFX_MAX_SERIALNUM_LEN, "%pm", mac_address);
>+ devlink_info_serial_number_put(req, sn);
>+ }
>+}
>+
>+static int efx_devlink_info_get(struct devlink *devlink,
>+ struct devlink_info_req *req,
>+ struct netlink_ext_ack *extack)
>+{
>+ struct efx_devlink *devlink_private = devlink_priv(devlink);
>+ struct efx_nic *efx = devlink_private->efx;
>+
>+ efx_devlink_info_board_cfg(efx, req);
>+ efx_devlink_info_stored_versions(efx, req);
>+ efx_devlink_info_running_versions(efx, req);
I wonder, why don't you propagate error to the user? He may be
interested that you had troubles of getting some info, instead of silent
failure.
>+ return 0;
>+}
>+
> static const struct devlink_ops sfc_devlink_ops = {
>+ .info_get = efx_devlink_info_get,
> };
>
> void efx_fini_devlink_start(struct efx_nic *efx)
>diff --git a/drivers/net/ethernet/sfc/efx_devlink.h b/drivers/net/ethernet/sfc/efx_devlink.h
>index 55d0d8aeca1e..8bcd077d8d8d 100644
>--- a/drivers/net/ethernet/sfc/efx_devlink.h
>+++ b/drivers/net/ethernet/sfc/efx_devlink.h
>@@ -14,6 +14,23 @@
> #include "net_driver.h"
> #include <net/devlink.h>
>
>+/* Custom devlink-info version object names for details that do not map to the
>+ * generic standardized names.
>+ */
>+#define EFX_DEVLINK_INFO_VERSION_FW_MGMT_SUC "fw.mgmt.suc"
>+#define EFX_DEVLINK_INFO_VERSION_FW_MGMT_CMC "fw.mgmt.cmc"
>+#define EFX_DEVLINK_INFO_VERSION_FPGA_REV "fpga.rev"
>+#define EFX_DEVLINK_INFO_VERSION_DATAPATH_HW "fpga.app"
>+#define EFX_DEVLINK_INFO_VERSION_DATAPATH_FW DEVLINK_INFO_VERSION_GENERIC_FW_APP
>+#define EFX_DEVLINK_INFO_VERSION_SOC_BOOT "coproc.boot"
>+#define EFX_DEVLINK_INFO_VERSION_SOC_UBOOT "coproc.uboot"
>+#define EFX_DEVLINK_INFO_VERSION_SOC_MAIN "coproc.main"
>+#define EFX_DEVLINK_INFO_VERSION_SOC_RECOVERY "coproc.recovery"
>+#define EFX_DEVLINK_INFO_VERSION_FW_EXPROM "fw.exprom"
>+#define EFX_DEVLINK_INFO_VERSION_FW_UEFI "fw.uefi"
>+
>+#define EFX_MAX_VERSION_INFO_LEN 64
>+
> int efx_probe_devlink(struct efx_nic *efx);
> void efx_probe_devlink_done(struct efx_nic *efx);
> void efx_fini_devlink_start(struct efx_nic *efx);
>diff --git a/drivers/net/ethernet/sfc/mcdi.c b/drivers/net/ethernet/sfc/mcdi.c
>index af338208eae9..a7f2c31071e8 100644
>--- a/drivers/net/ethernet/sfc/mcdi.c
>+++ b/drivers/net/ethernet/sfc/mcdi.c
>@@ -2175,6 +2175,78 @@ int efx_mcdi_get_privilege_mask(struct efx_nic *efx, u32 *mask)
> return 0;
> }
>
>+int efx_mcdi_nvram_metadata(struct efx_nic *efx, unsigned int type,
>+ u32 *subtype, u16 version[4], char *desc,
>+ size_t descsize)
>+{
>+ MCDI_DECLARE_BUF(inbuf, MC_CMD_NVRAM_METADATA_IN_LEN);
>+ efx_dword_t *outbuf;
>+ size_t outlen;
>+ u32 flags;
>+ int rc;
>+
>+ outbuf = kzalloc(MC_CMD_NVRAM_METADATA_OUT_LENMAX_MCDI2, GFP_KERNEL);
>+ if (!outbuf)
>+ return -ENOMEM;
>+
>+ MCDI_SET_DWORD(inbuf, NVRAM_METADATA_IN_TYPE, type);
>+
>+ rc = efx_mcdi_rpc_quiet(efx, MC_CMD_NVRAM_METADATA, inbuf,
>+ sizeof(inbuf), outbuf,
>+ MC_CMD_NVRAM_METADATA_OUT_LENMAX_MCDI2,
>+ &outlen);
>+ if (rc)
>+ goto out_free;
>+ if (outlen < MC_CMD_NVRAM_METADATA_OUT_LENMIN) {
>+ rc = -EIO;
>+ goto out_free;
>+ }
>+
>+ flags = MCDI_DWORD(outbuf, NVRAM_METADATA_OUT_FLAGS);
>+
>+ if (desc && descsize > 0) {
>+ if (flags & BIT(MC_CMD_NVRAM_METADATA_OUT_DESCRIPTION_VALID_LBN)) {
>+ if (descsize <=
>+ MC_CMD_NVRAM_METADATA_OUT_DESCRIPTION_NUM(outlen)) {
>+ rc = -E2BIG;
>+ goto out_free;
>+ }
>+
>+ strncpy(desc,
>+ MCDI_PTR(outbuf, NVRAM_METADATA_OUT_DESCRIPTION),
>+ MC_CMD_NVRAM_METADATA_OUT_DESCRIPTION_NUM(outlen));
>+ desc[MC_CMD_NVRAM_METADATA_OUT_DESCRIPTION_NUM(outlen)] = '\0';
>+ } else {
>+ desc[0] = '\0';
>+ }
>+ }
>+
>+ if (subtype) {
>+ if (flags & BIT(MC_CMD_NVRAM_METADATA_OUT_SUBTYPE_VALID_LBN))
>+ *subtype = MCDI_DWORD(outbuf, NVRAM_METADATA_OUT_SUBTYPE);
>+ else
>+ *subtype = 0;
>+ }
>+
>+ if (version) {
>+ if (flags & BIT(MC_CMD_NVRAM_METADATA_OUT_VERSION_VALID_LBN)) {
>+ version[0] = MCDI_WORD(outbuf, NVRAM_METADATA_OUT_VERSION_W);
>+ version[1] = MCDI_WORD(outbuf, NVRAM_METADATA_OUT_VERSION_X);
>+ version[2] = MCDI_WORD(outbuf, NVRAM_METADATA_OUT_VERSION_Y);
>+ version[3] = MCDI_WORD(outbuf, NVRAM_METADATA_OUT_VERSION_Z);
>+ } else {
>+ version[0] = 0;
>+ version[1] = 0;
>+ version[2] = 0;
>+ version[3] = 0;
>+ }
>+ }
>+
>+out_free:
>+ kfree(outbuf);
>+ return rc;
>+}
>+
> #ifdef CONFIG_SFC_MTD
>
> #define EFX_MCDI_NVRAM_LEN_MAX 128
>diff --git a/drivers/net/ethernet/sfc/mcdi.h b/drivers/net/ethernet/sfc/mcdi.h
>index 7e35fec9da35..5cb202684858 100644
>--- a/drivers/net/ethernet/sfc/mcdi.h
>+++ b/drivers/net/ethernet/sfc/mcdi.h
>@@ -378,6 +378,9 @@ int efx_mcdi_nvram_info(struct efx_nic *efx, unsigned int type,
> size_t *size_out, size_t *erase_size_out,
> bool *protected_out);
> int efx_new_mcdi_nvram_test_all(struct efx_nic *efx);
>+int efx_mcdi_nvram_metadata(struct efx_nic *efx, unsigned int type,
>+ u32 *subtype, u16 version[4], char *desc,
>+ size_t descsize);
> int efx_mcdi_nvram_test_all(struct efx_nic *efx);
> int efx_mcdi_handle_assertion(struct efx_nic *efx);
> int efx_mcdi_set_id_led(struct efx_nic *efx, enum efx_led_mode mode);
>--
>2.17.1
>
Powered by blists - more mailing lists