From: Maxim Levitsky <mlevitsk@redhat.com>
To: Max Reitz <mreitz@redhat.com>,
Klaus Birkelund Jensen <klaus@birkelund.eu>,
qemu-block@nongnu.org
Cc: Keith Busch <keith.busch@intel.com>,
Kevin Wolf <kwolf@redhat.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Qemu-block] [PATCH] nvme: do not advertise support for unsupported arbitration mechanism
Date: Sat, 15 Jun 2019 21:08:35 +0300 [thread overview]
Message-ID: <5154aa1deb2c8a2d51f575f3b89a1a3b2cb7c9b3.camel@redhat.com> (raw)
In-Reply-To: <639bcbb8-fabc-9bb8-b11d-909af4ed9cdc@redhat.com>
On Fri, 2019-06-14 at 22:39 +0200, Max Reitz wrote:
> On 06.06.19 11:25, Klaus Birkelund Jensen wrote:
> > The device mistakenly reports that the Weighted Round Robin with Urgent
> > Priority Class arbitration mechanism is supported.
> >
> > It is not.
>
> I believe you based on the fact that there is no “weight” or “priority”
> anywhere in nvme.c, and that it does not evaluate the Arbitration
> Mechanism Selected field.
>
> > Signed-off-by: Klaus Birkelund Jensen <klaus.jensen@cnexlabs.com>
> > ---
> > hw/block/nvme.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/hw/block/nvme.c b/hw/block/nvme.c
> > index 30e50f7a3853..415b4641d6b4 100644
> > --- a/hw/block/nvme.c
> > +++ b/hw/block/nvme.c
> > @@ -1383,7 +1383,6 @@ static void nvme_realize(PCIDevice *pci_dev, Error **errp)
> > n->bar.cap = 0;
> > NVME_CAP_SET_MQES(n->bar.cap, 0x7ff);
> > NVME_CAP_SET_CQR(n->bar.cap, 1);
> > - NVME_CAP_SET_AMS(n->bar.cap, 1);
>
> I suppose the better way would be to pass 0, so it is more explicit, I
> think.
>
> (Just removing it looks like it may have just been forgotten.)
>
> Max
>
> > NVME_CAP_SET_TO(n->bar.cap, 0xf);
> > NVME_CAP_SET_CSS(n->bar.cap, 1);
> > NVME_CAP_SET_MPSMAX(n->bar.cap, 4);
> >
>
>
Yea. no way that this driver supports WRRU and I haven't noticed it.
Just checked again to be sure.
To be honest, after some quick look,
that driver doesn' even really support the regular Round-Robin as it uses a timer to process a specific queue,
kicked up by a doorbell write :-)
Acked-by: Maxim Levitsky <mlevitsk@redhat.com>
Best regards,
Maxim Levitsky
next prev parent reply other threads:[~2019-06-15 18:10 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-06 9:25 [Qemu-devel] [PATCH] nvme: do not advertise support for unsupported arbitration mechanism Klaus Birkelund Jensen
2019-06-14 20:39 ` Max Reitz
2019-06-15 18:08 ` Maxim Levitsky [this message]
2019-06-17 6:54 ` Klaus Birkelund
2019-06-17 12:25 ` Max Reitz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5154aa1deb2c8a2d51f575f3b89a1a3b2cb7c9b3.camel@redhat.com \
--to=mlevitsk@redhat.com \
--cc=keith.busch@intel.com \
--cc=klaus@birkelund.eu \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).