From patchwork Thu Mar 29 22:37:19 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dave Jiang X-Patchwork-Id: 10316631 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 68B2860212 for ; Thu, 29 Mar 2018 22:37:22 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 519FA2A29E for ; Thu, 29 Mar 2018 22:37:22 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 4645A2A53F; Thu, 29 Mar 2018 22:37:22 +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 80C5C2A29E for ; Thu, 29 Mar 2018 22:37:21 +0000 (UTC) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 3B457225E9677; Thu, 29 Mar 2018 15:30:41 -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.20; helo=mga02.intel.com; envelope-from=dave.jiang@intel.com; receiver=linux-nvdimm@lists.01.org Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (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 E1FEB224DD140 for ; Thu, 29 Mar 2018 15:30:39 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 29 Mar 2018 15:37:20 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,378,1517904000"; d="scan'208";a="212472625" Received: from djiang5-desk3.ch.intel.com ([143.182.136.93]) by orsmga005.jf.intel.com with ESMTP; 29 Mar 2018 15:37:19 -0700 Subject: [PATCH 3/4] acpi/nfit: removing ARS timeout and change scrubbing to delayed work From: Dave Jiang To: dan.j.williams@intel.com Date: Thu, 29 Mar 2018 15:37:19 -0700 Message-ID: <152236303903.35558.14431510784850314944.stgit@djiang5-desk3.ch.intel.com> In-Reply-To: <152236282506.35558.2067249639136170490.stgit@djiang5-desk3.ch.intel.com> References: <152236282506.35558.2067249639136170490.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.26 Precedence: list List-Id: "Linux-nvdimm developer list." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-acpi@vger.kernel.org, tony.luck@intel.com, rjw@rjwysocki.net, lenb@kernel.org, linux-nvdimm@lists.01.org Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" X-Virus-Scanned: ClamAV using ClamSMTP With the introduction of BERT parsing, we have added the poison regions to badblocks and no longer need to wait until scrubbing ARS to complete for the bad areas before we surface the regions. Changing acpi_nfit_scrub() to delayed work. Instead of keep polling with a timeout we will just schedule to try again at a later time. The timeout will be doubled every time we hit busy. Signed-off-by: Dave Jiang --- drivers/acpi/nfit/core.c | 233 ++++++++++++++++++---------------------------- drivers/acpi/nfit/nfit.h | 13 ++- 2 files changed, 101 insertions(+), 145 deletions(-) diff --git a/drivers/acpi/nfit/core.c b/drivers/acpi/nfit/core.c index 3e3b95298a21..668d040bf108 100644 --- a/drivers/acpi/nfit/core.c +++ b/drivers/acpi/nfit/core.c @@ -35,10 +35,6 @@ static bool force_enable_dimms; module_param(force_enable_dimms, bool, S_IRUGO|S_IWUSR); MODULE_PARM_DESC(force_enable_dimms, "Ignore _STA (ACPI DIMM device) status"); -static unsigned int scrub_timeout = NFIT_ARS_TIMEOUT; -module_param(scrub_timeout, uint, S_IRUGO|S_IWUSR); -MODULE_PARM_DESC(scrub_timeout, "Initial scrub timeout in seconds"); - /* after three payloads of overflow, it's dead jim */ static unsigned int scrub_overflow_abort = 3; module_param(scrub_overflow_abort, uint, S_IRUGO|S_IWUSR); @@ -1251,7 +1247,8 @@ static ssize_t scrub_show(struct device *dev, struct acpi_nfit_desc *acpi_desc = to_acpi_desc(nd_desc); rc = sprintf(buf, "%d%s", acpi_desc->scrub_count, - (work_busy(&acpi_desc->work)) ? "+\n" : "\n"); + (work_busy(&acpi_desc->dwork.work)) ? + "+\n" : "\n"); } device_unlock(dev); return rc; @@ -2819,86 +2816,6 @@ static int acpi_nfit_query_poison(struct acpi_nfit_desc *acpi_desc, return 0; } -static void acpi_nfit_async_scrub(struct acpi_nfit_desc *acpi_desc, - struct nfit_spa *nfit_spa) -{ - struct acpi_nfit_system_address *spa = nfit_spa->spa; - unsigned int overflow_retry = scrub_overflow_abort; - u64 init_ars_start = 0, init_ars_len = 0; - struct device *dev = acpi_desc->dev; - unsigned int tmo = scrub_timeout; - int rc; - - if (!nfit_spa->ars_required || !nfit_spa->nd_region) - return; - - rc = ars_start(acpi_desc, nfit_spa); - /* - * If we timed out the initial scan we'll still be busy here, - * and will wait another timeout before giving up permanently. - */ - if (rc < 0 && rc != -EBUSY) - return; - - do { - u64 ars_start, ars_len; - - if (acpi_desc->cancel) - break; - rc = acpi_nfit_query_poison(acpi_desc, nfit_spa); - if (rc == -ENOTTY) - break; - if (rc == -EBUSY && !tmo) { - dev_warn(dev, "range %d ars timeout, aborting\n", - spa->range_index); - break; - } - - if (rc == -EBUSY) { - /* - * Note, entries may be appended to the list - * while the lock is dropped, but the workqueue - * being active prevents entries being deleted / - * freed. - */ - mutex_unlock(&acpi_desc->init_mutex); - ssleep(1); - tmo--; - mutex_lock(&acpi_desc->init_mutex); - continue; - } - - /* we got some results, but there are more pending... */ - if (rc == -ENOSPC && overflow_retry--) { - if (!init_ars_len) { - init_ars_len = acpi_desc->ars_status->length; - init_ars_start = acpi_desc->ars_status->address; - } - rc = ars_continue(acpi_desc); - } - - if (rc < 0) { - dev_warn(dev, "range %d ars continuation failed\n", - spa->range_index); - break; - } - - if (init_ars_len) { - ars_start = init_ars_start; - ars_len = init_ars_len; - } else { - ars_start = acpi_desc->ars_status->address; - ars_len = acpi_desc->ars_status->length; - } - dev_dbg(dev, "spa range: %d ars from %#llx + %#llx complete\n", - spa->range_index, ars_start, ars_len); - /* notify the region about new poison entries */ - nvdimm_region_notify(nfit_spa->nd_region, - NVDIMM_REVALIDATE_POISON); - break; - } while (1); -} - static void acpi_nfit_scrub(struct work_struct *work) { struct device *dev; @@ -2907,37 +2824,60 @@ static void acpi_nfit_scrub(struct work_struct *work) u64 init_scrub_address = 0; bool init_ars_done = false; struct acpi_nfit_desc *acpi_desc; - unsigned int tmo = scrub_timeout; unsigned int overflow_retry = scrub_overflow_abort; + int ars_needed = 0; - acpi_desc = container_of(work, typeof(*acpi_desc), work); + acpi_desc = container_of(work, typeof(*acpi_desc), dwork.work); dev = acpi_desc->dev; /* - * We scrub in 2 phases. The first phase waits for any platform - * firmware initiated scrubs to complete and then we go search for the - * affected spa regions to mark them scanned. In the second phase we - * initiate a directed scrub for every range that was not scrubbed in - * phase 1. If we're called for a 'rescan', we harmlessly pass through - * the first phase, but really only care about running phase 2, where - * regions can be notified of new poison. + * We can register all regions right away at init since BERT will + * prevent us from hitting the problem areas. */ + list_for_each_entry(nfit_spa, &acpi_desc->spas, list) { + if (!nfit_spa->nd_region) + acpi_nfit_register_region(acpi_desc, nfit_spa); + } - /* process platform firmware initiated scrubs */ retry: mutex_lock(&acpi_desc->init_mutex); list_for_each_entry(nfit_spa, &acpi_desc->spas, list) { struct nd_cmd_ars_status *ars_status; - struct acpi_nfit_system_address *spa; - u64 ars_start, ars_len; + struct acpi_nfit_system_address *spa = nfit_spa->spa; + u64 astart, ars_len; int rc; if (acpi_desc->cancel) - break; + goto out; - if (nfit_spa->nd_region) + if (nfit_spa->ars_state == NFIT_ARS_STATE_COMPLETE) + continue; + + if (nfit_spa->ars_state == NFIT_ARS_STATE_UNSUPPORTED) continue; + if (nfit_spa->ars_state == NFIT_ARS_STATE_REQUESTED) { + dev_dbg(dev, "range %d starting ARS\n", + spa->range_index); + rc = ars_start(acpi_desc, nfit_spa); + if (rc == -EBUSY) { + queue_delayed_work(nfit_wq, &acpi_desc->dwork, + acpi_desc->scrub_timeout * HZ); + /* + * Increase timeout for next time around. + * We'll max it at 30mins. + */ + acpi_desc->scrub_timeout = + min_t(unsigned int, + acpi_desc->scrub_timeout * 2, + 1800); + goto out; + } + if (rc < 0) + goto out; + nfit_spa->ars_state = NFIT_ARS_STATE_IN_PROGRESS; + } + if (init_ars_done) { /* * No need to re-query, we're now just @@ -2951,22 +2891,26 @@ static void acpi_nfit_scrub(struct work_struct *work) if (rc == -ENOTTY) { /* no ars capability, just register spa and move on */ acpi_nfit_register_region(acpi_desc, nfit_spa); + nfit_spa->ars_state = NFIT_ARS_STATE_UNSUPPORTED; continue; } - if (rc == -EBUSY && !tmo) { - /* fallthrough to directed scrub in phase 2 */ - dev_warn(dev, "timeout awaiting ars results, continuing...\n"); - break; - } else if (rc == -EBUSY) { - mutex_unlock(&acpi_desc->init_mutex); - ssleep(1); - tmo--; - goto retry; + if (rc == -EBUSY) { + nfit_spa->ars_state = NFIT_ARS_STATE_IN_PROGRESS; + queue_delayed_work(nfit_wq, &acpi_desc->dwork, + acpi_desc->scrub_timeout * HZ); + /* + * Increase timeout for next time around. We'll max + * it at 30mins + */ + acpi_desc->scrub_timeout = min_t(unsigned int, + acpi_desc->scrub_timeout * 2, 1800); + goto out; } /* we got some results, but there are more pending... */ if (rc == -ENOSPC && overflow_retry--) { + nfit_spa->ars_state = NFIT_ARS_STATE_IN_PROGRESS; ars_status = acpi_desc->ars_status; /* * Record the original scrub range, so that we @@ -2985,57 +2929,55 @@ static void acpi_nfit_scrub(struct work_struct *work) } if (rc < 0) { - /* - * Initial scrub failed, we'll give it one more - * try below... - */ - break; + nfit_spa->ars_state = NFIT_ARS_STATE_IDLE; + dev_warn(dev, "range %d ars continuation failed\n", + spa->range_index); + goto out; } /* We got some final results, record completed ranges */ ars_status = acpi_desc->ars_status; if (init_scrub_length) { - ars_start = init_scrub_address; - ars_len = ars_start + init_scrub_length; + astart = init_scrub_address; + ars_len = astart + init_scrub_length; } else { - ars_start = ars_status->address; + astart = ars_status->address; ars_len = ars_status->length; } - spa = nfit_spa->spa; if (!init_ars_done) { init_ars_done = true; - dev_dbg(dev, "init scrub %#llx + %#llx complete\n", - ars_start, ars_len); + dev_dbg(dev, "Scrub %#llx + %#llx complete\n", + astart, ars_len); } - if (ars_start <= spa->address && ars_start + ars_len - >= spa->address + spa->length) - acpi_nfit_register_region(acpi_desc, nfit_spa); + nfit_spa->ars_state = NFIT_ARS_STATE_COMPLETE; + acpi_desc->scrub_timeout = 1; + if (nfit_spa->nd_region) + nvdimm_region_notify(nfit_spa->nd_region, + NVDIMM_REVALIDATE_POISON); } - /* - * For all the ranges not covered by an initial scrub we still - * want to see if there are errors, but it's ok to discover them - * asynchronously. - */ list_for_each_entry(nfit_spa, &acpi_desc->spas, list) { - /* - * Flag all the ranges that still need scrubbing, but - * register them now to make data available. - */ - if (!nfit_spa->nd_region) { - nfit_spa->ars_required = 1; - acpi_nfit_register_region(acpi_desc, nfit_spa); + if (nfit_spa->ars_state == NFIT_ARS_STATE_IDLE) { + dev_dbg(dev, "range %d set for ARS\n", + nfit_spa->spa->range_index); + nfit_spa->ars_state = NFIT_ARS_STATE_REQUESTED; + ars_needed++; } } - acpi_desc->init_complete = 1; - list_for_each_entry(nfit_spa, &acpi_desc->spas, list) - acpi_nfit_async_scrub(acpi_desc, nfit_spa); + if (ars_needed) { + queue_delayed_work(nfit_wq, &acpi_desc->dwork, + acpi_desc->scrub_timeout * HZ); + goto out; + } + + acpi_desc->init_complete = 1; acpi_desc->scrub_count++; acpi_desc->ars_start_flags = 0; if (acpi_desc->scrub_count_state) sysfs_notify_dirent(acpi_desc->scrub_count_state); +out: mutex_unlock(&acpi_desc->init_mutex); } @@ -3054,7 +2996,7 @@ static int acpi_nfit_register_regions(struct acpi_nfit_desc *acpi_desc) acpi_desc->ars_start_flags = 0; if (!acpi_desc->cancel) - queue_work(nfit_wq, &acpi_desc->work); + queue_delayed_work(nfit_wq, &acpi_desc->dwork, 0); return 0; } @@ -3251,7 +3193,7 @@ static int acpi_nfit_clear_to_send(struct nvdimm_bus_descriptor *nd_desc, * just needs guarantees that any ars it initiates are not * interrupted by any intervening start reqeusts from userspace. */ - if (work_busy(&acpi_desc->work)) + if (work_busy(&acpi_desc->dwork.work)) return -EBUSY; return 0; @@ -3262,7 +3204,7 @@ int acpi_nfit_ars_rescan(struct acpi_nfit_desc *acpi_desc, u8 flags) struct device *dev = acpi_desc->dev; struct nfit_spa *nfit_spa; - if (work_busy(&acpi_desc->work)) + if (work_busy(&acpi_desc->dwork.work)) return -EBUSY; mutex_lock(&acpi_desc->init_mutex); @@ -3277,10 +3219,13 @@ int acpi_nfit_ars_rescan(struct acpi_nfit_desc *acpi_desc, u8 flags) if (nfit_spa_type(spa) != NFIT_SPA_PM) continue; - nfit_spa->ars_required = 1; + if (nfit_spa->ars_state == NFIT_ARS_STATE_UNSUPPORTED) + continue; + + nfit_spa->ars_state = NFIT_ARS_STATE_REQUESTED; } acpi_desc->ars_start_flags = flags; - queue_work(nfit_wq, &acpi_desc->work); + queue_delayed_work(nfit_wq, &acpi_desc->dwork, 0); dev_dbg(dev, "%s: ars_scan triggered\n", __func__); mutex_unlock(&acpi_desc->init_mutex); @@ -3311,7 +3256,8 @@ void acpi_nfit_desc_init(struct acpi_nfit_desc *acpi_desc, struct device *dev) INIT_LIST_HEAD(&acpi_desc->dimms); INIT_LIST_HEAD(&acpi_desc->list); mutex_init(&acpi_desc->init_mutex); - INIT_WORK(&acpi_desc->work, acpi_nfit_scrub); + acpi_desc->scrub_timeout = 1; + INIT_DELAYED_WORK(&acpi_desc->dwork, acpi_nfit_scrub); } EXPORT_SYMBOL_GPL(acpi_nfit_desc_init); @@ -3335,6 +3281,7 @@ void acpi_nfit_shutdown(void *data) mutex_lock(&acpi_desc->init_mutex); acpi_desc->cancel = 1; + cancel_delayed_work_sync(&acpi_desc->dwork); mutex_unlock(&acpi_desc->init_mutex); /* diff --git a/drivers/acpi/nfit/nfit.h b/drivers/acpi/nfit/nfit.h index 50d36e166d70..e1dcbbdc5adb 100644 --- a/drivers/acpi/nfit/nfit.h +++ b/drivers/acpi/nfit/nfit.h @@ -117,10 +117,18 @@ enum nfit_dimm_notifiers { NFIT_NOTIFY_DIMM_HEALTH = 0x81, }; +enum nfit_ars_state { + NFIT_ARS_STATE_IDLE = 0, + NFIT_ARS_STATE_REQUESTED, + NFIT_ARS_STATE_IN_PROGRESS, + NFIT_ARS_STATE_COMPLETE, + NFIT_ARS_STATE_UNSUPPORTED, +}; + struct nfit_spa { struct list_head list; struct nd_region *nd_region; - unsigned int ars_required:1; + enum nfit_ars_state ars_state; u32 clear_err_unit; u32 max_ars; struct acpi_nfit_system_address spa[0]; @@ -192,7 +200,7 @@ struct acpi_nfit_desc { u8 ars_start_flags; struct nd_cmd_ars_status *ars_status; size_t ars_status_size; - struct work_struct work; + struct delayed_work dwork; struct list_head list; struct kernfs_node *scrub_count_state; unsigned int scrub_count; @@ -203,6 +211,7 @@ struct acpi_nfit_desc { unsigned long bus_cmd_force_en; unsigned long bus_nfit_cmd_force_en; unsigned int platform_cap; + unsigned int scrub_timeout; int (*blk_do_io)(struct nd_blk_region *ndbr, resource_size_t dpa, void *iobuf, u64 len, int rw); };