From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from ws5-mx01.kavi.com (ws5-mx01.kavi.com [34.193.7.191]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EF89AC77B73 for ; Wed, 19 Apr 2023 07:09:09 +0000 (UTC) Received: from lists.oasis-open.org (oasis.ws5.connectedcommunity.org [10.110.1.242]) by ws5-mx01.kavi.com (Postfix) with ESMTP id 62DEF42BCA for ; Wed, 19 Apr 2023 07:09:09 +0000 (UTC) Received: from lists.oasis-open.org (oasis-open.org [10.110.1.242]) by lists.oasis-open.org (Postfix) with ESMTP id 55147986435 for ; Wed, 19 Apr 2023 07:09:09 +0000 (UTC) Received: from host09.ws5.connectedcommunity.org (host09.ws5.connectedcommunity.org [10.110.1.97]) by lists.oasis-open.org (Postfix) with QMQP id 487A2986416; Wed, 19 Apr 2023 07:09:09 +0000 (UTC) Mailing-List: contact virtio-dev-help@lists.oasis-open.org; run by ezmlm List-ID: Sender: Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Received: from lists.oasis-open.org (oasis-open.org [10.110.1.242]) by lists.oasis-open.org (Postfix) with ESMTP id 37CFE98641B for ; Wed, 19 Apr 2023 07:09:04 +0000 (UTC) X-Virus-Scanned: amavisd-new at kavi.com X-MC-Unique: GHSFo0BHPVucSy-cLCvKPQ-1 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1681888140; x=1684480140; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=qiSj/Y4eGfRgj/W9qU3/++JWC/iNvWA7Oj55ofNwirc=; b=YZXZbdvXY15HFT7+zXKLFN6UcvKq+XDB7Qhgua8oH4mUFI+fmZVSlWXYfr8t9+3hHr A02MceHxe/bc2KIMo3ruRgWp9lPCfBGGjXyIkMsJqeV9Rl/CB6BO2+1Tqs9ZdAj2ViLd sCszJJYUAq+ObkuQHjyJqct7MZW8P8ArP9n12lYV6qCOJZaEwVz07nuqFoloaZt2aVBL V83bz7IAXn0OzKi/37U3RKXXNwFdBTi0WDA4Bpk/0Sut4nnjCU97le9C0oltiUD67614 kIvzM3pMD68RwAQ2aY/ItreIUrE9NekwkmlkcsOzR2WUTtglEkOtppiuGV7fMfuhCOE6 BiZw== X-Gm-Message-State: AAQBX9cnp6z11huXglH2xAaGNlsUi+0nTcsPyz9dB6hV4c/OoOynFW0J +eLXk7/E3ZQv92kSxOKlEy3SOXiGUCHihTr50Yi94jJI4xFTpZr+KmhRAS5pngGobQfOJ15PiOE y3GU7jBD0l+1AxRfyZ51qRrGoYRq6 X-Received: by 2002:adf:fc49:0:b0:2f8:d1e6:e721 with SMTP id e9-20020adffc49000000b002f8d1e6e721mr3742073wrs.51.1681888140546; Wed, 19 Apr 2023 00:09:00 -0700 (PDT) X-Google-Smtp-Source: AKy350YBkNgVFv/kdJd4Xec6hZHZOSDIWf2NwCF9565xDGOO5dGJxN3wvz9s4JJl9USFlTzeJWL65g== X-Received: by 2002:adf:fc49:0:b0:2f8:d1e6:e721 with SMTP id e9-20020adffc49000000b002f8d1e6e721mr3742052wrs.51.1681888140217; Wed, 19 Apr 2023 00:09:00 -0700 (PDT) Date: Wed, 19 Apr 2023 03:08:53 -0400 From: "Michael S. Tsirkin" To: Parav Pandit Cc: virtio-dev@lists.oasis-open.org, pasic@linux.ibm.com, cohuck@redhat.com, virtio-comment@lists.oasis-open.org, shahafs@nvidia.com, Max Gurtovoy Message-ID: <20230419030840-mutt-send-email-mst@kernel.org> References: <20230411194220.752355-1-parav@nvidia.com> <20230411194220.752355-3-parav@nvidia.com> <20230419030514-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 In-Reply-To: <20230419030514-mutt-send-email-mst@kernel.org> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Subject: [virtio-dev] Re: [PATCH 2/2] transport-pci: Improve queue msix vector register desc On Wed, Apr 19, 2023 at 03:08:30AM -0400, Michael S. Tsirkin wrote: > On Tue, Apr 11, 2023 at 10:42:20PM +0300, Parav Pandit wrote: > > queue_msix_vector register is for receiving interrupts from the device > > for the virtqueue. > > > > "for MSI-X" is confusing term. > > > > Also it is the register that driver "writes" to, similar to > > many other registers such as queue_desc, queue_driver etc. > > > > Hence, replace the verb from use to write. > > > > Signed-off-by: Parav Pandit > > Reviewed-by: Max Gurtovoy > > --- > > changelog: > > v0->v1: > > - added 'here' > > - dropped 'receiving' > > --- > > transport-pci.tex | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/transport-pci.tex b/transport-pci.tex > > index cb9fadd..00669b5 100644 > > --- a/transport-pci.tex > > +++ b/transport-pci.tex > > @@ -367,7 +367,7 @@ \subsubsection{Common configuration structure layout}\label{sec:Virtio Transport > > A 0 means the queue is unavailable. > > > > \item[\field{queue_msix_vector}] > > - The driver uses this to specify the queue vector for MSI-X. > > + The driver writes an MSI-X vector number here for virtqueue interrupts. > > Here it's *the* specific queue not just some queue. And in fact > we call them virtqueue notifications, not interrupts elsewhere. > So how about: > > Used by the driver to set the MSI-X vector number for the virtqueue notifications. or even "the virtqueue notification interrupts". > > > \item[\field{queue_enable}] > > The driver uses this to selectively prevent the device from executing requests from this virtqueue. > > -- > > 2.26.2 --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org