From patchwork Wed Jul 11 21:17:09 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Verma, Vishal L" X-Patchwork-Id: 10520609 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 B3184600CA for ; Wed, 11 Jul 2018 21:17:20 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A6862287D5 for ; Wed, 11 Jul 2018 21:17:20 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 98860298B2; Wed, 11 Jul 2018 21:17:20 +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=-2.9 required=2.0 tests=BAYES_00, MAILING_LIST_MULTI, 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 1C4E4287D5 for ; Wed, 11 Jul 2018 21:17:19 +0000 (UTC) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id AC09D2097E274; Wed, 11 Jul 2018 14:17:19 -0700 (PDT) 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.24; helo=mga09.intel.com; envelope-from=vishal.l.verma@intel.com; receiver=linux-nvdimm@lists.01.org Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) (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 E72B7209605D8 for ; Wed, 11 Jul 2018 14:17:17 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Jul 2018 14:17:17 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,339,1526367600"; d="scan'208";a="53685001" Received: from vverma7-desk1.lm.intel.com ([10.232.112.133]) by fmsmga007.fm.intel.com with ESMTP; 11 Jul 2018 14:17:16 -0700 From: Vishal Verma To: Subject: [ndctl PATCH] ndctl: add an API to check support for smart injection Date: Wed, 11 Jul 2018 15:17:09 -0600 Message-Id: <20180711211709.3007-1-vishal.l.verma@intel.com> X-Mailer: git-send-email 2.14.4 X-BeenThere: linux-nvdimm@lists.01.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: "Linux-nvdimm developer list." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" X-Virus-Scanned: ClamAV using ClamSMTP Add an API to check whether smart injection is supported, and add an intel-dsm implementation for it. Use this check in the ndctl inject-smart command. Reported-by: Leszek Lugin Cc: Dan Williams Signed-off-by: Vishal Verma --- ndctl/inject-smart.c | 16 ++++++++++++++++ ndctl/lib/intel.c | 25 +++++++++++++++++++++++++ ndctl/lib/libndctl.sym | 5 +++++ ndctl/lib/private.h | 1 + ndctl/lib/smart.c | 10 ++++++++++ ndctl/libndctl.h | 1 + 6 files changed, 58 insertions(+) diff --git a/ndctl/inject-smart.c b/ndctl/inject-smart.c index 60de9fe..7793706 100644 --- a/ndctl/inject-smart.c +++ b/ndctl/inject-smart.c @@ -352,6 +352,22 @@ static int dimm_inject_smart(struct ndctl_dimm *dimm) struct json_object *jdimm; int rc; + rc = ndctl_dimm_smart_inject_supported(dimm); + switch (rc) { + case -ENOTTY: + error("%s: smart injection not supported by ndctl.", + ndctl_dimm_get_devname(dimm)); + return rc; + case -EOPNOTSUPP: + error("%s: smart injection not supported by the kernel", + ndctl_dimm_get_devname(dimm)); + return rc; + case -EIO: + error("%s: smart injection not supported by either platform firmware or the kernel.", + ndctl_dimm_get_devname(dimm)); + return rc; + } + if (sctx.op_mask & (1 << OP_SET)) { rc = smart_set_thresh(dimm); if (rc) diff --git a/ndctl/lib/intel.c b/ndctl/lib/intel.c index 49a4360..5a8ecba 100644 --- a/ndctl/lib/intel.c +++ b/ndctl/lib/intel.c @@ -362,6 +362,30 @@ static int intel_cmd_smart_inject_unsafe_shutdown(struct ndctl_cmd *cmd, return 0; } +static int intel_dimm_smart_inject_supported(struct ndctl_dimm *dimm) +{ + struct ndctl_ctx *ctx = ndctl_dimm_get_ctx(dimm); + + if (!ndctl_dimm_is_cmd_supported(dimm, ND_CMD_CALL)) { + dbg(ctx, "unsupported cmd: %d\n", ND_CMD_CALL); + return -EOPNOTSUPP; + } + + if (test_dimm_dsm(dimm, ND_INTEL_SMART_INJECT_MTEMP) == + DIMM_DSM_UNSUPPORTED || + test_dimm_dsm(dimm, ND_INTEL_SMART_INJECT_SPARE) == + DIMM_DSM_UNSUPPORTED || + test_dimm_dsm(dimm, ND_INTEL_SMART_INJECT_FATAL) == + DIMM_DSM_UNSUPPORTED || + test_dimm_dsm(dimm, ND_INTEL_SMART_INJECT_SHUTDOWN) == + DIMM_DSM_UNSUPPORTED) { + dbg(ctx, "smart injection functions unsupported\n"); + return -EIO; + } + + return 0; +} + static const char *intel_cmd_desc(int fn) { static const char *descs[] = { @@ -714,6 +738,7 @@ struct ndctl_dimm_ops * const intel_dimm_ops = &(struct ndctl_dimm_ops) { .smart_inject_spares = intel_cmd_smart_inject_spares, .smart_inject_fatal = intel_cmd_smart_inject_fatal, .smart_inject_unsafe_shutdown = intel_cmd_smart_inject_unsafe_shutdown, + .smart_inject_supported = intel_dimm_smart_inject_supported, .new_fw_get_info = intel_dimm_cmd_new_fw_get_info, .fw_info_get_storage_size = intel_cmd_fw_info_get_storage_size, .fw_info_get_max_send_len = intel_cmd_fw_info_get_max_send_len, diff --git a/ndctl/lib/libndctl.sym b/ndctl/lib/libndctl.sym index e939993..8932ef6 100644 --- a/ndctl/lib/libndctl.sym +++ b/ndctl/lib/libndctl.sym @@ -367,3 +367,8 @@ global: ndctl_namespace_uninject_error2; ndctl_cmd_ars_stat_get_flag_overflow; } LIBNDCTL_15; + +LIBNDCTL_17 { +global: + ndctl_dimm_smart_inject_supported; +} LIBNDCTL_16; diff --git a/ndctl/lib/private.h b/ndctl/lib/private.h index b756b74..a94f894 100644 --- a/ndctl/lib/private.h +++ b/ndctl/lib/private.h @@ -311,6 +311,7 @@ struct ndctl_dimm_ops { int (*smart_inject_spares)(struct ndctl_cmd *, bool, unsigned int); int (*smart_inject_fatal)(struct ndctl_cmd *, bool); int (*smart_inject_unsafe_shutdown)(struct ndctl_cmd *, bool); + int (*smart_inject_supported)(struct ndctl_dimm *); struct ndctl_cmd *(*new_fw_get_info)(struct ndctl_dimm *); unsigned int (*fw_info_get_storage_size)(struct ndctl_cmd *); unsigned int (*fw_info_get_max_send_len)(struct ndctl_cmd *); diff --git a/ndctl/lib/smart.c b/ndctl/lib/smart.c index 0455252..7ba46d1 100644 --- a/ndctl/lib/smart.c +++ b/ndctl/lib/smart.c @@ -172,3 +172,13 @@ ndctl_dimm_cmd_new_ack_shutdown_count(struct ndctl_dimm *dimm) else return NULL; } + +NDCTL_EXPORT int ndctl_dimm_smart_inject_supported(struct ndctl_dimm *dimm) +{ + struct ndctl_dimm_ops *ops = dimm->ops; + + if (ops && ops->smart_inject_supported) + return ops->smart_inject_supported(dimm); + else + return -ENOTTY; +} diff --git a/ndctl/libndctl.h b/ndctl/libndctl.h index 9270bae..8a96c84 100644 --- a/ndctl/libndctl.h +++ b/ndctl/libndctl.h @@ -279,6 +279,7 @@ int ndctl_cmd_smart_inject_spares(struct ndctl_cmd *cmd, bool enable, unsigned int spares); int ndctl_cmd_smart_inject_fatal(struct ndctl_cmd *cmd, bool enable); int ndctl_cmd_smart_inject_unsafe_shutdown(struct ndctl_cmd *cmd, bool enable); +int ndctl_dimm_smart_inject_supported(struct ndctl_dimm *dimm); struct ndctl_cmd *ndctl_dimm_cmd_new_vendor_specific(struct ndctl_dimm *dimm, unsigned int opcode, size_t input_size, size_t output_size);