From patchwork Wed Jan 31 22:06:01 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dave Jiang X-Patchwork-Id: 10194899 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id B4C2C603EE for ; Wed, 31 Jan 2018 22:06:05 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9AA7728759 for ; Wed, 31 Jan 2018 22:06:05 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 8D78C287F4; Wed, 31 Jan 2018 22:06:05 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 Received: from ml01.01.org (ml01.01.org [198.145.21.10]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 78D4B28759 for ; Wed, 31 Jan 2018 22:06:04 +0000 (UTC) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id BD1C721E25681; Wed, 31 Jan 2018 14:00:27 -0800 (PST) X-Original-To: linux-nvdimm@lists.01.org Delivered-To: linux-nvdimm@lists.01.org Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.31; helo=mga06.intel.com; envelope-from=dave.jiang@intel.com; receiver=linux-nvdimm@lists.01.org Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 6390521945DF3 for ; Wed, 31 Jan 2018 14:00:26 -0800 (PST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 31 Jan 2018 14:06:02 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,441,1511856000"; d="scan'208";a="26318154" Received: from djiang5-desk3.ch.intel.com ([143.182.136.93]) by fmsmga004.fm.intel.com with ESMTP; 31 Jan 2018 14:06:02 -0800 Subject: [PATCH v5 3/4] ndctl: add firmware update command option for ndctl From: Dave Jiang To: vishal.l.verma@intel.com, dan.j.williams@intel.com Date: Wed, 31 Jan 2018 15:06:01 -0700 Message-ID: <151743636176.50357.18416590433415496346.stgit@djiang5-desk3.ch.intel.com> In-Reply-To: <151743627634.50357.2692734095508615676.stgit@djiang5-desk3.ch.intel.com> References: <151743627634.50357.2692734095508615676.stgit@djiang5-desk3.ch.intel.com> User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 X-BeenThere: linux-nvdimm@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Linux-nvdimm developer list." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-nvdimm@lists.01.org Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" X-Virus-Scanned: ClamAV using ClamSMTP Adding option "update-firmware" to ndctl for update firmware support from Intel DSM v1.6. ndctl update-firmware takes an option of -f for a firmware binary and a -d for the DIMM name: ndctl update-firmware -d nmem0 -f new_firmware.bin Signed-off-by: Dave Jiang --- Documentation/ndctl/Makefile.am | 1 Documentation/ndctl/ndctl-update-firmware.txt | 18 + builtin.h | 1 ndctl/Makefile.am | 3 ndctl/ndctl.c | 1 ndctl/update.c | 534 +++++++++++++++++++++++++ 6 files changed, 557 insertions(+), 1 deletion(-) create mode 100644 Documentation/ndctl/ndctl-update-firmware.txt create mode 100644 ndctl/update.c diff --git a/Documentation/ndctl/Makefile.am b/Documentation/ndctl/Makefile.am index 615baf0..27b2076 100644 --- a/Documentation/ndctl/Makefile.am +++ b/Documentation/ndctl/Makefile.am @@ -31,6 +31,7 @@ man1_MANS = \ ndctl-destroy-namespace.1 \ ndctl-check-namespace.1 \ ndctl-inject-error.1 \ + ndctl-update-firmware.1 \ ndctl-list.1 CLEANFILES = $(man1_MANS) diff --git a/Documentation/ndctl/ndctl-update-firmware.txt b/Documentation/ndctl/ndctl-update-firmware.txt new file mode 100644 index 0000000..d742302 --- /dev/null +++ b/Documentation/ndctl/ndctl-update-firmware.txt @@ -0,0 +1,18 @@ +ndctl-update-firmware(1) +======================== + +NAME +---- +ndctl-update-firmware - provides updating of NVDIMM firmware + +SYNOPSIS +-------- +[verse] +'ndctl update-firmware' -f -d + +COPYRIGHT +--------- +Copyright (c) 2016 - 2017, Intel Corporation. License GPLv2: GNU GPL +version 2 . This is free software: +you are free to change and redistribute it. There is NO WARRANTY, to +the extent permitted by law. diff --git a/builtin.h b/builtin.h index 5e1b7ef..1f423dc 100644 --- a/builtin.h +++ b/builtin.h @@ -43,4 +43,5 @@ int cmd_test(int argc, const char **argv, void *ctx); #ifdef ENABLE_DESTRUCTIVE int cmd_bat(int argc, const char **argv, void *ctx); #endif +int cmd_update_firmware(int argc, const char **argv, void *ctx); #endif /* _NDCTL_BUILTIN_H_ */ diff --git a/ndctl/Makefile.am b/ndctl/Makefile.am index 6677607..5cd8678 100644 --- a/ndctl/Makefile.am +++ b/ndctl/Makefile.am @@ -13,7 +13,8 @@ ndctl_SOURCES = ndctl.c \ test.c \ ../util/json.c \ util/json-smart.c \ - inject-error.c + inject-error.c \ + update.c if ENABLE_DESTRUCTIVE ndctl_SOURCES += ../test/blk_namespaces.c \ diff --git a/ndctl/ndctl.c b/ndctl/ndctl.c index 0f748e1..a0e5153 100644 --- a/ndctl/ndctl.c +++ b/ndctl/ndctl.c @@ -84,6 +84,7 @@ static struct cmd_struct commands[] = { { "init-labels", cmd_init_labels }, { "check-labels", cmd_check_labels }, { "inject-error", cmd_inject_error }, + { "update-firmware", cmd_update_firmware }, { "list", cmd_list }, { "help", cmd_help }, #ifdef ENABLE_TEST diff --git a/ndctl/update.c b/ndctl/update.c new file mode 100644 index 0000000..e006115 --- /dev/null +++ b/ndctl/update.c @@ -0,0 +1,534 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +/* Copyright(c) 2018 Intel Corporation. All rights reserved. */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#ifdef HAVE_NDCTL_H +#include +#else +#include +#endif + +#include +#include "private.h" +#include +#include + +#define ND_CMD_STATUS_SUCCESS 0 +#define ND_CMD_STATUS_NOTSUPP 1 +#define ND_CMD_STATUS_NOTEXIST 2 +#define ND_CMD_STATUS_INVALPARM 3 +#define ND_CMD_STATUS_HWERR 4 +#define ND_CMD_STATUS_RETRY 5 +#define ND_CMD_STATUS_UNKNOWN 6 +#define ND_CMD_STATUS_EXTEND 7 +#define ND_CMD_STATUS_NORES 8 +#define ND_CMD_STATUS_NOTREADY 9 + +#define ND_CMD_STATUS_START_BUSY 0x10000 +#define ND_CMD_STATUS_SEND_CTXINVAL 0x10000 +#define ND_CMD_STATUS_FIN_CTXINVAL 0x10000 +#define ND_CMD_STATUS_FIN_DONE 0x20000 +#define ND_CMD_STATUS_FIN_BAD 0x30000 +#define ND_CMD_STATUS_FIN_ABORTED 0x40000 +#define ND_CMD_STATUS_FQ_CTXINVAL 0x10000 +#define ND_CMD_STATUS_FQ_BUSY 0x20000 +#define ND_CMD_STATUS_FQ_BAD 0x30000 +#define ND_CMD_STATUS_FQ_ORDER 0x40000 + +struct fw_info { + uint32_t store_size; + uint32_t update_size; + uint32_t query_interval; + uint32_t max_query; + uint64_t run_version; + uint32_t context; +}; + +struct update_context { + int fw_fd; + size_t fw_size; + const char *fw_path; + const char *dimm_id; + struct ndctl_dimm *dimm; + struct fw_info dimm_fw; + struct ndctl_cmd *start; +}; + +/* + * updating firmware consists of performing the following steps: + * 1. Call GET_FIMRWARE_INFO DSM. The return results provide: + * A. Size of the firmware storage area + * B. Max size per send command + * C. Polling interval for check finish status + * D. Max time for finish update poll + * E. Update capabilities + * F. Running FIS version + * G. Running FW revision + * H. Updated FW revision. Only valid after firmware update done. + * 2. Call START_FW_UPDATE. The return results provide: + * A. Ready to start status + * B. Valid FW update context + * 3. Call SEND_FW_UPDATE_DATA with valid payload + * Repeat until done. + * 4. Call FINISH_FW_UPDATE + * 5. Poll with QUERY_FINISH_UPDATE success or failure + */ + +static int verify_fw_size(struct update_context *uctx) +{ + struct fw_info *fw = &uctx->dimm_fw; + + if (uctx->fw_size > fw->store_size) { + error("Firmware file size greater than DIMM store\n"); + return -ENOSPC; + } + + return 0; +} + +static int submit_get_firmware_info(struct update_context *uctx) +{ + struct ndctl_cmd *cmd; + int rc; + enum ND_FW_STATUS status; + struct fw_info *fw = &uctx->dimm_fw; + + cmd = ndctl_dimm_cmd_new_fw_get_info(uctx->dimm); + if (!cmd) + return -ENXIO; + + rc = ndctl_cmd_submit(cmd); + if (rc < 0) + return rc; + + status = ndctl_cmd_fw_xlat_firmware_status(cmd); + if (status != FW_SUCCESS) { + error("GET FIRMWARE INFO failed: %#x\n", status); + return -ENXIO; + } + + fw->store_size = ndctl_cmd_fw_info_get_storage_size(cmd); + if (fw->store_size == UINT_MAX) + return -ENXIO; + + fw->update_size = ndctl_cmd_fw_info_get_max_send_len(cmd); + if (fw->update_size == UINT_MAX) + return -ENXIO; + + fw->query_interval = ndctl_cmd_fw_info_get_query_interval(cmd); + if (fw->query_interval == UINT_MAX) + return -ENXIO; + + fw->max_query = ndctl_cmd_fw_info_get_max_query_time(cmd); + if (fw->max_query == UINT_MAX) + return -ENXIO; + + fw->run_version = ndctl_cmd_fw_info_get_run_version(cmd); + if (fw->run_version == ULLONG_MAX) + return -ENXIO; + + rc = verify_fw_size(uctx); + ndctl_cmd_unref(cmd); + return rc; +} + +static int submit_start_firmware_upload(struct update_context *uctx) +{ + struct ndctl_cmd *cmd; + int rc; + enum ND_FW_STATUS status; + struct fw_info *fw = &uctx->dimm_fw; + + cmd = ndctl_dimm_cmd_new_fw_start_update(uctx->dimm); + if (!cmd) + return -ENXIO; + + rc = ndctl_cmd_submit(cmd); + if (rc < 0) + return rc; + + status = ndctl_cmd_fw_xlat_firmware_status(cmd); + if (status != FW_SUCCESS) { + error("START FIRMWARE UPDATE failed: %#x\n", status); + if (status == FW_EBUSY) + error("Another firmware upload in progress or finished.\n"); + return -ENXIO; + } + + fw->context = ndctl_cmd_fw_start_get_context(cmd); + if (fw->context == UINT_MAX) + return -ENXIO; + + uctx->start = cmd; + + return 0; +} + +static int get_fw_data_from_file(int fd, void *buf, uint32_t len, + uint32_t offset) +{ + ssize_t rc, total = len; + + while (len) { + rc = pread(fd, buf, len, offset); + if (rc < 0) + return -errno; + len -= rc; + } + + return total; +} + +static int send_firmware(struct update_context *uctx) +{ + struct ndctl_cmd *cmd = NULL; + ssize_t read; + int rc; + enum ND_FW_STATUS status; + struct fw_info *fw = &uctx->dimm_fw; + uint32_t copied = 0, len, remain; + void *buf; + + buf = malloc(fw->update_size); + if (!buf) + return -ENOMEM; + + remain = uctx->fw_size; + + while (remain) { + len = min(fw->update_size, remain); + read = get_fw_data_from_file(uctx->fw_fd, buf, len, copied); + if (read < 0) { + rc = read; + goto cleanup; + } + + cmd = ndctl_dimm_cmd_new_fw_send(uctx->start, copied, read, + buf); + if (!cmd) { + rc = -ENXIO; + goto cleanup; + } + + rc = ndctl_cmd_submit(cmd); + if (rc < 0) + goto cleanup; + + status = ndctl_cmd_fw_xlat_firmware_status(cmd); + if (status != FW_SUCCESS) { + error("SEND FIRMWARE failed: %#x\n", status); + rc = -ENXIO; + goto cleanup; + } + + copied += read; + remain -= read; + + ndctl_cmd_unref(cmd); + cmd = NULL; + } + +cleanup: + if (cmd) + ndctl_cmd_unref(cmd); + free(buf); + return rc; +} + +static int submit_finish_firmware(struct update_context *uctx) +{ + struct ndctl_cmd *cmd; + int rc; + enum ND_FW_STATUS status; + + cmd = ndctl_dimm_cmd_new_fw_finish(uctx->start); + if (!cmd) + return -ENXIO; + + rc = ndctl_cmd_submit(cmd); + if (rc < 0) + goto out; + + status = ndctl_cmd_fw_xlat_firmware_status(cmd); + if (status != FW_SUCCESS) { + error("FINISH FIRMWARE UPDATE failed: %#x\n", status); + rc = -ENXIO; + goto out; + } + +out: + ndctl_cmd_unref(cmd); + return rc; +} + +static int submit_abort_firmware(struct update_context *uctx) +{ + struct ndctl_cmd *cmd; + int rc; + enum ND_FW_STATUS status; + + cmd = ndctl_dimm_cmd_new_fw_abort(uctx->start); + if (!cmd) + return -ENXIO; + + rc = ndctl_cmd_submit(cmd); + if (rc < 0) + goto out; + + status = ndctl_cmd_fw_xlat_firmware_status(cmd); + if (!(status & ND_CMD_STATUS_FIN_ABORTED)) { + error("FW update abort failed: %#x\n", status); + rc = -ENXIO; + goto out; + } + +out: + ndctl_cmd_unref(cmd); + return rc; +} + +static int query_fw_finish_status(struct update_context *uctx) +{ + struct ndctl_cmd *cmd; + int rc; + enum ND_FW_STATUS status; + struct fw_info *fw = &uctx->dimm_fw; + bool done = false; + struct timespec now, before, after; + uint64_t ver; + + cmd = ndctl_dimm_cmd_new_fw_finish_query(uctx->start); + if (!cmd) + return -ENXIO; + + rc = clock_gettime(CLOCK_MONOTONIC, &before); + if (rc < 0) + goto out; + + now.tv_nsec = fw->query_interval / 1000; + now.tv_sec = 0; + + do { + rc = ndctl_cmd_submit(cmd); + if (rc < 0) + break; + + status = ndctl_cmd_fw_xlat_firmware_status(cmd); + switch (status) { + case FW_SUCCESS: + ver = ndctl_cmd_fw_fquery_get_fw_rev(cmd); + if (ver == 0) { + printf("No firmware updated\n"); + rc = -ENXIO; + goto out; + } + + printf("Image %s updated successfully to DIMM %s\n", + uctx->fw_path, uctx->dimm_id); + printf("Firmware version %#lx.\n", ver); + printf("Reboot to activate.\n"); + done = true; + rc = 0; + break; + case FW_EBUSY: + /* Still on going, continue */ + rc = clock_gettime(CLOCK_MONOTONIC, &after); + if (rc < 0) { + rc = -errno; + goto out; + } + + /* + * If we expire max query time, + * we timed out + */ + if (after.tv_sec - before.tv_sec > + fw->max_query / 1000000) { + rc = -ETIMEDOUT; + goto out; + } + + /* + * Sleep the interval dictated by firmware + * before query again. + */ + rc = nanosleep(&now, NULL); + if (rc < 0) { + rc = -errno; + goto out; + } + break; + case FW_EBADFW: + printf("Image failed to verify by DIMM\n"); + case FW_EINVAL_CTX: + case FW_ESEQUENCE: + done = true; + rc = -ENXIO; + goto out; + case FW_ENORES: + printf("Firmware update sequence timed out\n"); + rc = -ETIMEDOUT; + done = true; + goto out; + default: + rc = -EINVAL; + done = true; + goto out; + } + } while (!done); + +out: + ndctl_cmd_unref(cmd); + return rc; +} + +static int update_firmware(struct update_context *uctx) +{ + int rc; + + rc = submit_get_firmware_info(uctx); + if (rc < 0) + return rc; + + rc = submit_start_firmware_upload(uctx); + if (rc < 0) + return rc; + + printf("Uploading %s to DIMM %s\n", uctx->fw_path, uctx->dimm_id); + + rc = send_firmware(uctx); + if (rc < 0) { + error("Firmware send failed. Aborting...\n"); + rc = submit_abort_firmware(uctx); + if (rc < 0) + error("Aborting update sequence failed\n"); + return rc; + } + + rc = submit_finish_firmware(uctx); + if (rc < 0) { + error("Unable to end update sequence\n"); + rc = submit_abort_firmware(uctx); + if (rc < 0) + error("Aborting update sequence failed\n"); + return rc; + } + + rc = query_fw_finish_status(uctx); + if (rc < 0) + return rc; + + return 0; +} + +static int get_ndctl_dimm(struct update_context *uctx, void *ctx) +{ + struct ndctl_dimm *dimm; + struct ndctl_bus *bus; + + ndctl_bus_foreach(ctx, bus) + ndctl_dimm_foreach(bus, dimm) { + if (!util_dimm_filter(dimm, uctx->dimm_id)) + continue; + uctx->dimm = dimm; + return 0; + } + + return -ENODEV; +} + +static int verify_fw_file(struct update_context *uctx) +{ + struct stat st; + + if (stat(uctx->fw_path, &st) < 0) + return -errno; + if (!S_ISREG(st.st_mode)) + return -EINVAL; + + uctx->fw_size = st.st_size; + if (uctx->fw_size == 0) + return -EINVAL; + + uctx->fw_fd = open(uctx->fw_path, O_RDONLY); + if (uctx->fw_fd < 0) + return -errno; + + return 0; +} + +int cmd_update_firmware(int argc, const char **argv, void *ctx) +{ + struct update_context uctx = { 0 }; + const struct option options[] = { + OPT_STRING('f', "firmware", &uctx.fw_path, + "file-name", "name of firmware"), + OPT_STRING('d', "dimm", &uctx.dimm_id, "dimm-id", + "dimm to be updated"), + OPT_END(), + }; + const char * const u[] = { + "ndctl update_firmware []", + NULL + }; + int i, rc; + + argc = parse_options(argc, argv, options, u, 0); + for (i = 0; i < argc; i++) + error("unknown parameter \"%s\"\n", argv[i]); + if (argc) + usage_with_options(u, options); + + if (!uctx.fw_path) { + error("No firmware file provided\n"); + usage_with_options(u, options); + return -EINVAL; + } + + if (!uctx.dimm_id) { + error("No DIMM ID provided\n"); + usage_with_options(u, options); + return -EINVAL; + } + + rc = verify_fw_file(&uctx); + if (rc < 0) + return rc; + + rc = get_ndctl_dimm(&uctx, ctx); + if (rc < 0) + return rc; + + rc = update_firmware(&uctx); + if (rc < 0) + return rc; + + if (uctx.start) + ndctl_cmd_unref(uctx.start); + + return 0; +}