From patchwork Wed Sep 11 13:14:39 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "\(Exiting\) Baolin Wang" X-Patchwork-Id: 173611 Delivered-To: patch@linaro.org Received: by 2002:a05:6e02:ce:0:0:0:0 with SMTP id r14csp730627ilq; Wed, 11 Sep 2019 06:15:06 -0700 (PDT) X-Google-Smtp-Source: APXvYqwNF/rD4SAQV9Xdt99GjRlqqozWfuTxmKyxTI+uO84MDrzcK5ZuBd153pnY2J4HyMYrK06P X-Received: by 2002:a17:906:4d86:: with SMTP id s6mr29766922eju.154.1568207705992; Wed, 11 Sep 2019 06:15:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1568207705; cv=none; d=google.com; s=arc-20160816; b=uPk50oMRjqfL2e2eueMobesLmrd6WXm5cFuLsJNnODJXYlAFHK8Pris5ooDbpuLcVO XjF61szGgHP900bc/f+sT/9tpYaK/8oLdI9DaA2AApyq7KloNdo3JAM099W4FhaHGVAD 6D8z9WeN35xKo6EuDOz6wHZsCTIxra1s6Ku7nsAsHL6k9dnaEn+cBOspsK/iT6gYsa00 iDo7bSX1C62pk9UwWMoen9CWcQ+qNCRUy1mmQPYONEuiUeaB/bgn2HWPFzw44Mp/DsL2 GvvWEFq6OwOnz5VyITqq49lXvdR5rRQj1sYWl4D1e3sdxYqFEV/A66kPlaADFDZBmYZZ 77fg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :dkim-signature; bh=vtC3RazXzeFZetJddN0/LRTlwPvo9DJjWZdotj+TRx4=; b=Yyc9AxTcko8ynGYLRwwI+4bNS7512r+Gpc1e0/P6jd7fmTXgrhmKnwukVWHa+2uXz8 q0n+WvljXVSL6usncUZHCzgf2NCZPdomtjoDbEJTHAZDvXDsKFQeLeGAGTm3Ehvc+xey E5Gnr9TL6RD/XqTNrNUNuzC1p9a47Lm5zPKdNrbwo32k1p52YSvDhmn2dk1zEswjMqXS hrU1BoK90Pm7yEl+75pTO9uYqkmMTzyfT3T+9XvOXQJYQbUwycDqlQMNZKXf5EIGuWkb EbHkRsSVnC0fJAEnQn2DuiBzx+nigCME0oIU/lPk4wf1iT7DQwIyxYUSVHnzMHYulQuk U1xg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=M2nPb2Ez; spf=pass (google.com: best guess record for domain of linux-mmc-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-mmc-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id fk4si10862797ejb.161.2019.09.11.06.15.05; Wed, 11 Sep 2019 06:15:05 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-mmc-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=M2nPb2Ez; spf=pass (google.com: best guess record for domain of linux-mmc-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-mmc-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727851AbfIKNPF (ORCPT + 5 others); Wed, 11 Sep 2019 09:15:05 -0400 Received: from mail-pf1-f194.google.com ([209.85.210.194]:44964 "EHLO mail-pf1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727302AbfIKNPF (ORCPT ); Wed, 11 Sep 2019 09:15:05 -0400 Received: by mail-pf1-f194.google.com with SMTP id q21so13635820pfn.11 for ; Wed, 11 Sep 2019 06:15:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=vtC3RazXzeFZetJddN0/LRTlwPvo9DJjWZdotj+TRx4=; b=M2nPb2Ez0HjNcUp+BHaiCNfkWF6hykMlJ14HyWMHQCsYNT68/6MYRHhGYTVlh8nrca JsIDHwsnt7MeGfREMA4zA9G8PssI9JQ0AutJMaHOelzPBUK1IUjcjsWCHugFhfAYBQ02 GhDIyM4jYcEzDvHtytUgVYrXm4HNj9h1SvQG9eQ4koaLQEyOCetDcRFiTEobq6rl31qP BKHMaD0YTE91VWGr6pkJukv9dqOTXZn6buFwBQZOtZFoRcXd1OJzmu+AJ02s45q79fka SjRmWMpc0lo54y2ihJLxKT/7nwJsesJIZEVCk8Bzlr7QCJO/S7CjnfeKv+ejg5OT5w0n uzIQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=vtC3RazXzeFZetJddN0/LRTlwPvo9DJjWZdotj+TRx4=; b=E5auMSl75GARgrkhen5YIgSbLdIt7TBbKF5PDfKF3TdE7gsN/TyEfWDOL9HUfBFh+G tOIarX45VKTTHDz3UtEosyuTVeQ1CVUJ70OPCCvcQaqUjkBciCuoc0+W8YpvR2AW6ztc K/cBnEyqt0kFXV1ZxW0weFVwg0hAEFwzl78YkJS0omCBH00ejfP/Vpoi/FovjiCcDCMS s174KxZaqCbI4Otq2Xwv4VbJZzOHEoDkuTDhAnim73dWCRLqwL2SlHGSrBf2EQN/NDMb TK0ngFu8xgKwZAMSWkLws4kqkF6TDgFJXDWX1F7vnoZTb0jFBDLA7H9Noc4l1SaTrknG x2sA== X-Gm-Message-State: APjAAAX5xTYzayolF7VXD9XfchQNWgkSEdqIkfB+t8EHxMCSptMaSV4T B10XIGHWFzLy0Sb/+6hYCieNFQ== X-Received: by 2002:a17:90a:8d0c:: with SMTP id c12mr5471391pjo.119.1568207704284; Wed, 11 Sep 2019 06:15:04 -0700 (PDT) Received: from baolinwangubtpc.spreadtrum.com ([117.18.48.82]) by smtp.gmail.com with ESMTPSA id e21sm6420120pgr.43.2019.09.11.06.14.59 (version=TLS1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 11 Sep 2019 06:15:03 -0700 (PDT) From: Baolin Wang To: adrian.hunter@intel.com, ulf.hansson@linaro.org, asutoshd@codeaurora.org Cc: orsonzhai@gmail.com, zhang.lyra@gmail.com, arnd@arndb.de, linus.walleij@linaro.org, vincent.guittot@linaro.org, baolin.wang@linaro.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 0/4] Add MMC software queue support Date: Wed, 11 Sep 2019 21:14:39 +0800 Message-Id: X-Mailer: git-send-email 1.7.9.5 Sender: linux-mmc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org Hi All, Now the MMC read/write stack will always wait for previous request is completed by mmc_blk_rw_wait(), before sending a new request to hardware, or queue a work to complete request, that will bring context switching overhead, especially for high I/O per second rates, to affect the IO performance. Thus this patch set will introduce the MMC software command queue support based on command queue engine's interfaces, and set the queue depth as 2, that means we do not need wait for previous request is completed and can queue 2 requests in flight. It is enough to let the irq handler always trigger the next request without a context switch and then ask the blk_mq layer for the next one to get queued, as well as avoiding a long latency. Moreover we can expand the MMC software queue interface to support MMC packed request or packed command instead of adding new interfaces, according to previosus discussion. Below are some comparison data with fio tool. The fio command I used is like below with changing the '--rw' parameter and enabling the direct IO flag to measure the actual hardware transfer speed in 4K block size. ./fio --filename=/dev/mmcblk0p30 --direct=1 --iodepth=20 --rw=read --bs=4K --size=512M --group_reporting --numjobs=20 --name=test_read My eMMC card working at HS400 Enhanced strobe mode: [ 2.229856] mmc0: new HS400 Enhanced strobe MMC card at address 0001 [ 2.237566] mmcblk0: mmc0:0001 HBG4a2 29.1 GiB [ 2.242621] mmcblk0boot0: mmc0:0001 HBG4a2 partition 1 4.00 MiB [ 2.249110] mmcblk0boot1: mmc0:0001 HBG4a2 partition 2 4.00 MiB [ 2.255307] mmcblk0rpmb: mmc0:0001 HBG4a2 partition 3 4.00 MiB, chardev (248:0) 1. Without MMC software queue I tested 3 times for each case and output a average speed. 1) Sequential read: Speed: 28.9MiB/s, 26.4MiB/s, 30.9MiB/s Average speed: 28.7MiB/s 2) Random read: Speed: 18.2MiB/s, 8.9MiB/s, 15.8MiB/s Average speed: 14.3MiB/s 3) Sequential write: Speed: 21.1MiB/s, 27.9MiB/s, 25MiB/s Average speed: 24.7MiB/s 4) Random write: Speed: 21.5MiB/s, 18.1MiB/s, 18.1MiB/s Average speed: 19.2MiB/s 2. With MMC software queue I tested 3 times for each case and output a average speed. 1) Sequential read: Speed: 44.1MiB/s, 42.3MiB/s, 44.4MiB/s Average speed: 43.6MiB/s 2) Random read: Speed: 30.6MiB/s, 30.9MiB/s, 30.5MiB/s Average speed: 30.6MiB/s 3) Sequential write: Speed: 44.1MiB/s, 45.9MiB/s, 44.2MiB/s Average speed: 44.7MiB/s 4) Random write: Speed: 45.1MiB/s, 43.3MiB/s, 42.4MiB/s Average speed: 43.6MiB/s Form above data, we can see the MMC software queue can help to improve the performance obviously. Any comments are welcome. Thanks a lot. Changes from v1: - Add request_done ops for sdhci_ops. - Replace virtual command queue with software queue for functions and variables. - Rename the software queue file and add sqhci.h header file. Baolin Wang (4): mmc: host: cqhci: Move the struct cqhci_slot into header file mmc: Add MMC software queue support mmc: host: sdhci: Add request_done ops for struct sdhci_ops mmc: host: sdhci-sprd: Add software queue support drivers/mmc/core/block.c | 61 ++++++++ drivers/mmc/core/mmc.c | 13 +- drivers/mmc/core/queue.c | 25 ++- drivers/mmc/host/Kconfig | 9 ++ drivers/mmc/host/Makefile | 1 + drivers/mmc/host/cqhci.c | 10 -- drivers/mmc/host/cqhci.h | 17 +- drivers/mmc/host/sdhci-sprd.c | 26 ++++ drivers/mmc/host/sdhci.c | 12 +- drivers/mmc/host/sdhci.h | 2 + drivers/mmc/host/sqhci.c | 345 +++++++++++++++++++++++++++++++++++++++++ drivers/mmc/host/sqhci.h | 35 +++++ include/linux/mmc/host.h | 3 + 13 files changed, 536 insertions(+), 23 deletions(-) create mode 100644 drivers/mmc/host/sqhci.c create mode 100644 drivers/mmc/host/sqhci.h -- 1.7.9.5