* [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ [not found] <CGME20240531213424epcas2p16d7360e12d310c9f299d449e66af07b3@epcas2p1.samsung.com> @ 2024-05-31 21:22 ` Minwoo Im 2024-05-31 21:22 ` [PATCH v2 1/2] ufs: pci: Add support MCQ for QEMU-based UFS Minwoo Im ` (5 more replies) 0 siblings, 6 replies; 7+ messages in thread From: Minwoo Im @ 2024-05-31 21:22 UTC (permalink / raw) To: James E . J . Bottomley, Martin K . Petersen, Bart Van Assche Cc: Alim Akhtar, Avri Altman, Minwoo Im, gost.dev, linux-scsi, linux-kernel, Jeuk Kim This patchset introduces add support for MCQ introduced in UFSHCI 4.0. The first patch adds a simple helper to get the address of MCQ queue config registers. The second one enables MCQ feature by adding mandatory vops callback functions required at MCQ initialization phase. The last one is to prevent a case where number of MCQ is given 1 since driver allocates poll_queues first rather than I/O queues to handle device commands. Instead of causing exception handlers due to no I/O queue, failfast during the initialization time. --- v2: - https://lore.kernel.org/linux-scsi/20240531103821.1583934-1-minwoo.im@samsung.com/T/#t - Not separate the newly introduced function from the actuall caller in the other patch by squash the second patch to the first one (Bart). - Rename ufs_redhat_* in ufshcd-pci.c to ufs_qemu_* to represent that it's for QEMU UFS PCI device (Bart). Minwoo Im (2): ufs: pci: Add support MCQ for QEMU-based UFS ufs: mcq: Prevent no I/O queue case for MCQ drivers/ufs/core/ufs-mcq.c | 23 +++++++++++++++++ drivers/ufs/host/ufshcd-pci.c | 48 ++++++++++++++++++++++++++++++++++- include/ufs/ufshcd.h | 1 + 3 files changed, 71 insertions(+), 1 deletion(-) -- 2.34.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 1/2] ufs: pci: Add support MCQ for QEMU-based UFS 2024-05-31 21:22 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im @ 2024-05-31 21:22 ` Minwoo Im 2024-05-31 21:22 ` [PATCH v2 2/2] ufs: mcq: Prevent no I/O queue case for MCQ Minwoo Im ` (4 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: Minwoo Im @ 2024-05-31 21:22 UTC (permalink / raw) To: James E . J . Bottomley, Martin K . Petersen, Bart Van Assche Cc: Alim Akhtar, Avri Altman, Minwoo Im, gost.dev, linux-scsi, linux-kernel, Jeuk Kim Recently, ufs-mcq feature has been introduced to QEMU hw/ufs device [1]. This patch adds MCQ support for upstream QEMU UFS PCI controller. This patch provides mandatory vops callbacks to make UFS controller work properly on MCQ mode. Operation and Runtime Config register stride is fixed to 48bytes which is implemented by qemu. [1] https://lore.kernel.org/qemu-devel/cover.1716876237.git.jeuk20.kim@samsung.com/ Signed-off-by: Minwoo Im <minwoo.im@samsung.com> --- drivers/ufs/core/ufs-mcq.c | 14 ++++++++++ drivers/ufs/host/ufshcd-pci.c | 48 ++++++++++++++++++++++++++++++++++- include/ufs/ufshcd.h | 1 + 3 files changed, 62 insertions(+), 1 deletion(-) diff --git a/drivers/ufs/core/ufs-mcq.c b/drivers/ufs/core/ufs-mcq.c index 52210c4c20dc..46faa54aea94 100644 --- a/drivers/ufs/core/ufs-mcq.c +++ b/drivers/ufs/core/ufs-mcq.c @@ -18,6 +18,7 @@ #include <linux/iopoll.h> #define MAX_QUEUE_SUP GENMASK(7, 0) +#define QCFGPTR GENMASK(23, 16) #define UFS_MCQ_MIN_RW_QUEUES 2 #define UFS_MCQ_MIN_READ_QUEUES 0 #define UFS_MCQ_MIN_POLL_QUEUES 0 @@ -116,6 +117,19 @@ struct ufs_hw_queue *ufshcd_mcq_req_to_hwq(struct ufs_hba *hba, return &hba->uhq[hwq]; } +/** + * ufshcd_mcq_queue_cfg_addr - get an start address of the MCQ Queue Config + * Registers. + * @hba: per adapter instance + * + * Return: Start address of MCQ Queue Config Registers in HCI + */ +unsigned int ufshcd_mcq_queue_cfg_addr(struct ufs_hba *hba) +{ + return FIELD_GET(QCFGPTR, hba->mcq_capabilities) * 0x200; +} +EXPORT_SYMBOL_GPL(ufshcd_mcq_queue_cfg_addr); + /** * ufshcd_mcq_decide_queue_depth - decide the queue depth * @hba: per adapter instance diff --git a/drivers/ufs/host/ufshcd-pci.c b/drivers/ufs/host/ufshcd-pci.c index 0aca666d2199..ba8af9c0e77f 100644 --- a/drivers/ufs/host/ufshcd-pci.c +++ b/drivers/ufs/host/ufshcd-pci.c @@ -20,6 +20,8 @@ #include <linux/acpi.h> #include <linux/gpio/consumer.h> +#define MAX_SUPP_MAC 64 + struct ufs_host { void (*late_init)(struct ufs_hba *hba); }; @@ -446,6 +448,49 @@ static int ufs_intel_mtl_init(struct ufs_hba *hba) return ufs_intel_common_init(hba); } +static int ufs_qemu_get_hba_mac(struct ufs_hba *hba) +{ + return MAX_SUPP_MAC; +} + +static int ufs_qemu_mcq_config_resource(struct ufs_hba *hba) +{ + hba->mcq_base = hba->mmio_base + ufshcd_mcq_queue_cfg_addr(hba); + + return 0; +} + +static int ufs_qemu_op_runtime_config(struct ufs_hba *hba) +{ + struct ufshcd_mcq_opr_info_t *opr; + int i; + + u32 sqdao = ufsmcq_readl(hba, ufshcd_mcq_cfg_offset(REG_SQDAO, 0)); + u32 sqisao = ufsmcq_readl(hba, ufshcd_mcq_cfg_offset(REG_SQISAO, 0)); + u32 cqdao = ufsmcq_readl(hba, ufshcd_mcq_cfg_offset(REG_CQDAO, 0)); + u32 cqisao = ufsmcq_readl(hba, ufshcd_mcq_cfg_offset(REG_CQISAO, 0)); + + hba->mcq_opr[OPR_SQD].offset = sqdao; + hba->mcq_opr[OPR_SQIS].offset = sqisao; + hba->mcq_opr[OPR_CQD].offset = cqdao; + hba->mcq_opr[OPR_CQIS].offset = cqisao; + + for (i = 0; i < OPR_MAX; i++) { + opr = &hba->mcq_opr[i]; + opr->stride = 48; + opr->base = hba->mmio_base + opr->offset; + } + + return 0; +} + +static struct ufs_hba_variant_ops ufs_qemu_hba_vops = { + .name = "qemu-pci", + .get_hba_mac = ufs_qemu_get_hba_mac, + .mcq_config_resource = ufs_qemu_mcq_config_resource, + .op_runtime_config = ufs_qemu_op_runtime_config, +}; + static struct ufs_hba_variant_ops ufs_intel_cnl_hba_vops = { .name = "intel-pci", .init = ufs_intel_common_init, @@ -591,7 +636,8 @@ static const struct dev_pm_ops ufshcd_pci_pm_ops = { }; static const struct pci_device_id ufshcd_pci_tbl[] = { - { PCI_VENDOR_ID_REDHAT, 0x0013, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, + { PCI_VENDOR_ID_REDHAT, 0x0013, PCI_ANY_ID, PCI_ANY_ID, 0, 0, + (kernel_ulong_t)&ufs_qemu_hba_vops }, { PCI_VENDOR_ID_SAMSUNG, 0xC00C, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, { PCI_VDEVICE(INTEL, 0x9DFA), (kernel_ulong_t)&ufs_intel_cnl_hba_vops }, { PCI_VDEVICE(INTEL, 0x4B41), (kernel_ulong_t)&ufs_intel_ehl_hba_vops }, diff --git a/include/ufs/ufshcd.h b/include/ufs/ufshcd.h index df68fb1d4f3f..9e0581115b34 100644 --- a/include/ufs/ufshcd.h +++ b/include/ufs/ufshcd.h @@ -1278,6 +1278,7 @@ void ufshcd_update_evt_hist(struct ufs_hba *hba, u32 id, u32 val); void ufshcd_hba_stop(struct ufs_hba *hba); void ufshcd_schedule_eh_work(struct ufs_hba *hba); void ufshcd_mcq_config_mac(struct ufs_hba *hba, u32 max_active_cmds); +unsigned int ufshcd_mcq_queue_cfg_addr(struct ufs_hba *hba); u32 ufshcd_mcq_read_cqis(struct ufs_hba *hba, int i); void ufshcd_mcq_write_cqis(struct ufs_hba *hba, u32 val, int i); unsigned long ufshcd_mcq_poll_cqe_lock(struct ufs_hba *hba, -- 2.34.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2 2/2] ufs: mcq: Prevent no I/O queue case for MCQ 2024-05-31 21:22 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im 2024-05-31 21:22 ` [PATCH v2 1/2] ufs: pci: Add support MCQ for QEMU-based UFS Minwoo Im @ 2024-05-31 21:22 ` Minwoo Im 2024-06-04 21:25 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im ` (3 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: Minwoo Im @ 2024-05-31 21:22 UTC (permalink / raw) To: James E . J . Bottomley, Martin K . Petersen, Bart Van Assche Cc: Alim Akhtar, Avri Altman, Minwoo Im, gost.dev, linux-scsi, linux-kernel, Jeuk Kim If hba_maxq equals poll_queues, which means there are no I/O queues (HCTX_TYPE_DEFAULT, HCTX_TYPE_READ), the very first hw queue will be allocated as HCTX_TYPE_POLL and it will be used as the dev_cmd_queue. In this case, device commands such as QUERY cannot be properly handled. This patch prevents the initialization of MCQ when the number of I/O queues is not set and only the number of POLL queues is set. Signed-off-by: Minwoo Im <minwoo.im@samsung.com> Reviewed-by: Bart Van Assche <bvanassche@acm.org> --- drivers/ufs/core/ufs-mcq.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/ufs/core/ufs-mcq.c b/drivers/ufs/core/ufs-mcq.c index 46faa54aea94..4bcae410c268 100644 --- a/drivers/ufs/core/ufs-mcq.c +++ b/drivers/ufs/core/ufs-mcq.c @@ -179,6 +179,15 @@ static int ufshcd_mcq_config_nr_queues(struct ufs_hba *hba) return -EOPNOTSUPP; } + /* + * Device should support at least one I/O queue to handle device + * commands via hba->dev_cmd_queue. + */ + if (hba_maxq == poll_queues) { + dev_err(hba->dev, "At least one non-poll queue required\n"); + return -EOPNOTSUPP; + } + rem = hba_maxq; if (rw_queues) { -- 2.34.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ 2024-05-31 21:22 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im 2024-05-31 21:22 ` [PATCH v2 1/2] ufs: pci: Add support MCQ for QEMU-based UFS Minwoo Im 2024-05-31 21:22 ` [PATCH v2 2/2] ufs: mcq: Prevent no I/O queue case for MCQ Minwoo Im @ 2024-06-04 21:25 ` Minwoo Im 2024-06-04 22:17 ` Bart Van Assche ` (2 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: Minwoo Im @ 2024-06-04 21:25 UTC (permalink / raw) To: James E . J . Bottomley, Martin K . Petersen, Bart Van Assche Cc: Alim Akhtar, Avri Altman, gost.dev, linux-scsi, linux-kernel, Jeuk Kim, Minwoo Im [-- Attachment #1: Type: text/plain, Size: 1506 bytes --] On 24-06-01 06:22:42, Minwoo Im wrote: > This patchset introduces add support for MCQ introduced in UFSHCI 4.0. The > first patch adds a simple helper to get the address of MCQ queue config > registers. The second one enables MCQ feature by adding mandatory vops > callback functions required at MCQ initialization phase. The last one is to > prevent a case where number of MCQ is given 1 since driver allocates poll_queues > first rather than I/O queues to handle device commands. Instead of causing > exception handlers due to no I/O queue, failfast during the initialization time. > > --- > v2: > - https://lore.kernel.org/linux-scsi/20240531103821.1583934-1-minwoo.im@samsung.com/T/#t Now the MCQ feature of hw/ufs has been pulled to QEMU [1]. You can test this patchset with [1]. [1] https://github.com/qemu/qemu/commit/5c079578d2e46df626d13eeb629c7d761a5c4e44 > - Not separate the newly introduced function from the actuall caller in the > other patch by squash the second patch to the first one (Bart). > - Rename ufs_redhat_* in ufshcd-pci.c to ufs_qemu_* to represent that it's > for QEMU UFS PCI device (Bart). > > Minwoo Im (2): > ufs: pci: Add support MCQ for QEMU-based UFS > ufs: mcq: Prevent no I/O queue case for MCQ > > drivers/ufs/core/ufs-mcq.c | 23 +++++++++++++++++ > drivers/ufs/host/ufshcd-pci.c | 48 ++++++++++++++++++++++++++++++++++- > include/ufs/ufshcd.h | 1 + > 3 files changed, 71 insertions(+), 1 deletion(-) > > -- > 2.34.1 > [-- Attachment #2: Type: text/plain, Size: 0 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ 2024-05-31 21:22 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im ` (2 preceding siblings ...) 2024-06-04 21:25 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im @ 2024-06-04 22:17 ` Bart Van Assche 2024-06-05 2:17 ` Martin K. Petersen 2024-06-12 1:59 ` Martin K. Petersen 5 siblings, 0 replies; 7+ messages in thread From: Bart Van Assche @ 2024-06-04 22:17 UTC (permalink / raw) To: Minwoo Im, James E . J . Bottomley, Martin K . Petersen Cc: Alim Akhtar, Avri Altman, gost.dev, linux-scsi, linux-kernel, Jeuk Kim On 5/31/24 15:22, Minwoo Im wrote: > This patchset introduces add support for MCQ introduced in UFSHCI 4.0. The > first patch adds a simple helper to get the address of MCQ queue config > registers. The second one enables MCQ feature by adding mandatory vops > callback functions required at MCQ initialization phase. The last one is to > prevent a case where number of MCQ is given 1 since driver allocates poll_queues > first rather than I/O queues to handle device commands. Instead of causing > exception handlers due to no I/O queue, failfast during the initialization time. For both patches, feel free to add: Reviewed-by: Bart Van Assche <bvanassche@acm.org> ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ 2024-05-31 21:22 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im ` (3 preceding siblings ...) 2024-06-04 22:17 ` Bart Van Assche @ 2024-06-05 2:17 ` Martin K. Petersen 2024-06-12 1:59 ` Martin K. Petersen 5 siblings, 0 replies; 7+ messages in thread From: Martin K. Petersen @ 2024-06-05 2:17 UTC (permalink / raw) To: Minwoo Im Cc: James E . J . Bottomley, Martin K . Petersen, Bart Van Assche, Alim Akhtar, Avri Altman, gost.dev, linux-scsi, linux-kernel, Jeuk Kim Minwoo, > This patchset introduces add support for MCQ introduced in UFSHCI 4.0. > The first patch adds a simple helper to get the address of MCQ queue > config registers. The second one enables MCQ feature by adding > mandatory vops callback functions required at MCQ initialization > phase. The last one is to prevent a case where number of MCQ is given > 1 since driver allocates poll_queues first rather than I/O queues to > handle device commands. Instead of causing exception handlers due to > no I/O queue, failfast during the initialization time. Applied to 6.11/scsi-staging, thanks! Martin ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ 2024-05-31 21:22 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im ` (4 preceding siblings ...) 2024-06-05 2:17 ` Martin K. Petersen @ 2024-06-12 1:59 ` Martin K. Petersen 5 siblings, 0 replies; 7+ messages in thread From: Martin K. Petersen @ 2024-06-12 1:59 UTC (permalink / raw) To: James E . J . Bottomley, Bart Van Assche, Minwoo Im Cc: Martin K . Petersen, Alim Akhtar, Avri Altman, gost.dev, linux-scsi, linux-kernel, Jeuk Kim On Sat, 01 Jun 2024 06:22:42 +0900, Minwoo Im wrote: > This patchset introduces add support for MCQ introduced in UFSHCI 4.0. The > first patch adds a simple helper to get the address of MCQ queue config > registers. The second one enables MCQ feature by adding mandatory vops > callback functions required at MCQ initialization phase. The last one is to > prevent a case where number of MCQ is given 1 since driver allocates poll_queues > first rather than I/O queues to handle device commands. Instead of causing > exception handlers due to no I/O queue, failfast during the initialization time. > > [...] Applied to 6.11/scsi-queue, thanks! [1/2] ufs: pci: Add support MCQ for QEMU-based UFS https://git.kernel.org/mkp/scsi/c/175d1825ca4d [2/2] ufs: mcq: Prevent no I/O queue case for MCQ https://git.kernel.org/mkp/scsi/c/a420a8ed0a92 -- Martin K. Petersen Oracle Linux Engineering ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-06-12 1:59 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20240531213424epcas2p16d7360e12d310c9f299d449e66af07b3@epcas2p1.samsung.com>
2024-05-31 21:22 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im
2024-05-31 21:22 ` [PATCH v2 1/2] ufs: pci: Add support MCQ for QEMU-based UFS Minwoo Im
2024-05-31 21:22 ` [PATCH v2 2/2] ufs: mcq: Prevent no I/O queue case for MCQ Minwoo Im
2024-06-04 21:25 ` [PATCH v2 0/2] ufs: pci: Add support UFSHCI 4.0 MCQ Minwoo Im
2024-06-04 22:17 ` Bart Van Assche
2024-06-05 2:17 ` Martin K. Petersen
2024-06-12 1:59 ` Martin K. Petersen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox