[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 12/15] hw/nvme: Initialize capability structures for prima
From: |
Klaus Jensen |
Subject: |
Re: [PATCH v2 12/15] hw/nvme: Initialize capability structures for primary/secondary controllers |
Date: |
Wed, 24 Nov 2021 09:04:31 +0100 |
On Nov 16 16:34, Łukasz Gieryk wrote:
> With four new properties:
> - sriov_v{i,q}_flexible,
> - sriov_max_v{i,q}_per_vf,
> one can configure the number of available flexible resources, as well as
> the limits. The primary and secondary controller capability structures
> are initialized accordingly.
>
> Since the number of available queues (interrupts) now varies between
> VF/PF, BAR size calculation is also adjusted.
>
> Signed-off-by: Łukasz Gieryk <lukasz.gieryk@linux.intel.com>
> ---
> hw/nvme/ctrl.c | 138 ++++++++++++++++++++++++++++++++++++++++---
> hw/nvme/nvme.h | 4 ++
> include/block/nvme.h | 5 ++
> 3 files changed, 140 insertions(+), 7 deletions(-)
>
> diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c
> index f8f5dfe204..f589ffde59 100644
> --- a/hw/nvme/ctrl.c
> +++ b/hw/nvme/ctrl.c
> @@ -6358,13 +6444,40 @@ static void nvme_init_state(NvmeCtrl *n)
> n->starttime_ms = qemu_clock_get_ms(QEMU_CLOCK_VIRTUAL);
> n->aer_reqs = g_new0(NvmeRequest *, n->params.aerl + 1);
>
> - list->numcntl = cpu_to_le16(n->params.sriov_max_vfs);
> - for (i = 0; i < n->params.sriov_max_vfs; i++) {
> + list->numcntl = cpu_to_le16(max_vfs);
> + for (i = 0; i < max_vfs; i++) {
> sctrl = &list->sec[i];
> sctrl->pcid = cpu_to_le16(n->cntlid);
> }
>
> cap->cntlid = cpu_to_le16(n->cntlid);
> + cap->crt = NVME_CRT_VQ | NVME_CRT_VI;
> +
> + if (pci_is_vf(&n->parent_obj)) {
> + cap->vqprt = cpu_to_le16(1 + n->conf_ioqpairs);
> + } else {
> + cap->vqprt = cpu_to_le16(1 + n->params.max_ioqpairs -
> + n->params.sriov_vq_flexible);
> + cap->vqfrt = cpu_to_le32(n->params.sriov_vq_flexible);
> + cap->vqrfap = cap->vqfrt;
> + cap->vqgran = cpu_to_le16(NVME_VF_RES_GRANULARITY);
> + cap->vqfrsm = n->params.sriov_max_vq_per_vf ?
> + cpu_to_le16(n->params.sriov_max_vq_per_vf) :
> + cap->vqprt;
That this defaults to VQPRT doesn't seem right. It should default to
VQFRT. Does not make sense to report a maximum number of assignable
flexible resources that are bigger than the number of flexible resources
available.
signature.asc
Description: PGP signature
- [PATCH v2 06/15] hw/nvme: Add support for Primary Controller Capabilities, (continued)
- [PATCH v2 06/15] hw/nvme: Add support for Primary Controller Capabilities, Łukasz Gieryk, 2021/11/16
- [PATCH v2 10/15] hw/nvme: Remove reg_size variable and update BAR0 size calculation, Łukasz Gieryk, 2021/11/16
- [PATCH v2 08/15] hw/nvme: Implement the Function Level Reset, Łukasz Gieryk, 2021/11/16
- [PATCH v2 07/15] hw/nvme: Add support for Secondary Controller List, Łukasz Gieryk, 2021/11/16
- [PATCH v2 09/15] hw/nvme: Make max_ioqpairs and msix_qsize configurable in runtime, Łukasz Gieryk, 2021/11/16
- [PATCH v2 03/15] pcie: Add helpers to the SR/IOV API, Łukasz Gieryk, 2021/11/16
- [PATCH v2 11/15] hw/nvme: Calculate BAR attributes in a function, Łukasz Gieryk, 2021/11/16
- [PATCH v2 12/15] hw/nvme: Initialize capability structures for primary/secondary controllers, Łukasz Gieryk, 2021/11/16
- Re: [PATCH v2 12/15] hw/nvme: Initialize capability structures for primary/secondary controllers,
Klaus Jensen <=
- [PATCH v2 14/15] docs: Add documentation for SR-IOV and Virtualization Enhancements, Łukasz Gieryk, 2021/11/16
- [PATCH v2 15/15] hw/nvme: Update the initalization place for the AER queue, Łukasz Gieryk, 2021/11/16
- [PATCH v2 13/15] hw/nvme: Add support for the Virtualization Management command, Łukasz Gieryk, 2021/11/16
- Re: [PATCH v2 00/15] hw/nvme: SR-IOV with Virtualization Enhancements, Klaus Jensen, 2021/11/24