Re: [PATCH V3 00/11] mmc: Add Command Queue support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Adrian,

On 2017/6/15 19:06, Adrian Hunter wrote:
Hi

Here is V3 of the hardware command queue patches without the software
command queue patches.


I can now boot my board with v4.13 finally this morning[1] and
apply this patchset and add my private patch for supporting CQE for
sdhci-of-arasan. Great to see it works and I don't see any
regression until now. I need more test but I would appreciate it
if Ulf can pick all these up into linux-next that folks can help
to test and not need to manually apply them. And it will be easy
for me to submit patch for sdhci-of-arasan to support CQE later.

I haven't have time to review all these, but I will do it next week.
And I will also try to run more iozone/fio to see how much it gains
when enabling CQE, comparing to non-CQE support.

But at least currently feels free to add:

Tested-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxx>

Thanks.

[1]:

[    2.859981] Synopsys Designware Multimedia Card Interface Driver
[    2.924875] sdhci-pltfm: SDHCI platform and OF driver helper
[    2.994768] mmc0: CQHCI version 5.10
[ 3.053787] mmc0: SDHCI controller on fe330000.sdhci [fe330000.sdhci] using ADMA
[    3.073534] ledtrig-cpu: registered to indicate activity on CPUs
[    3.079239] usbcore: registered new interface driver usbhid
[    3.080183] usbhid: USB HID core driver
[    3.094221] NET: Registered protocol family 17
[    3.095069] 9pnet: Installing 9P2000 support
[    3.095710] Key type dns_resolver registered
[    3.098371] registered taskstats version 1
[    3.155130] hctosys: unable to open rtc device (rtc0)
[    3.166186] mmc0: Command Queue Engine enabled
[    3.168514] mmc0: new HS400 Enhanced strobe MMC card at address 0001
[    3.188861] mmcblk0: mmc0:0001 AJNB4R 14.6 GiB
[    3.200381] mmcblk0boot0: mmc0:0001 AJNB4R partition 1 4.00 MiB
[    3.200702] ALSA device list:
[    3.200747]   No soundcards found.
[    3.204146] ttyS2 - failed to request DMA
[    3.212363] mmcblk0boot1: mmc0:0001 AJNB4R partition 2 4.00 MiB
[    3.221031] mmcblk0rpmb: mmc0:0001 AJNB4R partition 3 4.00 MiB
[    3.277636]  mmcblk0: p1 p2 p3 p4 p5 p6 p7
[    3.522777] EXT4-fs (mmcblk0p7): recovery complete
[ 3.523907] EXT4-fs (mmcblk0p7): mounted filesystem with ordered data mode. Opts: (null)
[    3.524699] VFS: Mounted root (ext4 filesystem) on device 179:7.


Is there any chance of this being queued for v4.13?


Changes since V2:
	Dropped patches that have been applied.
	Re-based
	Added "mmc: sdhci-pci: Add CQHCI support for Intel GLK"

Changes since V1:

	"Share mmc request array between partitions" is dependent
	on changes in "Introduce queue semantics", so added that
	and block fixes:

	Added "Fix is_waiting_last_req set incorrectly"
	Added "Fix cmd error reset failure path"
	Added "Use local var for mqrq_cur"
	Added "Introduce queue semantics"

Changes since RFC:

	Re-based on next.
	Added comment about command queue priority.
	Added some acks and reviews.


Adrian Hunter (10):
       mmc: core: Add mmc_retune_hold_now()
       mmc: core: Add members to mmc_request and mmc_data for CQE's
       mmc: host: Add CQE interface
       mmc: core: Turn off CQE before sending commands
       mmc: core: Add support for handling CQE requests
       mmc: mmc: Enable Command Queuing
       mmc: mmc: Enable CQE's
       mmc: block: Prepare CQE data
       mmc: block: Add CQE support
       mmc: sdhci-pci: Add CQHCI support for Intel GLK

Venkat Gopalakrishnan (1):
       mmc: cqhci: support for command queue enabled host

  drivers/mmc/core/block.c          |  240 +++++++-
  drivers/mmc/core/block.h          |    7 +
  drivers/mmc/core/bus.c            |    7 +
  drivers/mmc/core/core.c           |  156 ++++-
  drivers/mmc/core/host.c           |    6 +
  drivers/mmc/core/host.h           |    1 +
  drivers/mmc/core/mmc.c            |   30 +-
  drivers/mmc/core/queue.c          |  273 ++++++++-
  drivers/mmc/core/queue.h          |   42 +-
  drivers/mmc/host/Kconfig          |   14 +
  drivers/mmc/host/Makefile         |    1 +
  drivers/mmc/host/cqhci.c          | 1146 +++++++++++++++++++++++++++++++++++++
  drivers/mmc/host/cqhci.h          |  240 ++++++++
  drivers/mmc/host/sdhci-pci-core.c |  153 ++++-
  include/linux/mmc/core.h          |   18 +-
  include/linux/mmc/host.h          |   24 +
  include/trace/events/mmc.h        |   36 +-
  17 files changed, 2354 insertions(+), 40 deletions(-)
  create mode 100644 drivers/mmc/host/cqhci.c
  create mode 100644 drivers/mmc/host/cqhci.h


Regards
Adrian
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux