Re: [virtio-dev] Re: [PATCH 1/2] transport-pci: Improve config msix vector description

["Zhu, Lingshan" <lingshan.zhu@intel.com>]

On 4/19/2023 3:05 PM, Michael S. Tsirkin wrote:
> On Tue, Apr 11, 2023 at 10:42:19PM +0300, Parav Pandit wrote:
>> config_msix_vector is the register that holds the MSI-X vector number
>> for receiving configuration change related interrupts.
>>
>> It is not "for MSI-X".
>>
>> Hence, replace the confusing text with appropriate one.
>>
>> Reviewed-by: Max Gurtovoy <mgurtovoy@nvidia.com>
>> Signed-off-by: Parav Pandit <parav@nvidia.com>
>>
>> ---
>> changelog:
>> v0->v1:
>> - added 'here'
>> - dropped 'receiving'
>> ---
>>   transport-pci.tex | 3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/transport-pci.tex b/transport-pci.tex
>> index 42be072..cb9fadd 100644
>> --- a/transport-pci.tex
>> +++ b/transport-pci.tex
>> @@ -343,7 +343,8 @@ \subsubsection{Common configuration structure layout}\label{sec:Virtio Transport
>>           Driver Feature Bits selected by \field{driver_feature_select}.
>>   
>>   \item[\field{config_msix_vector}]
>> -        The driver sets the Configuration Vector for MSI-X.
>> +        The driver writes an MSI-X vector number here for Configuration change
>> +        interrupts.
> Hmm I think it's "the vector", not an. And the order of words
> is unusual. We also don't normally use "writes here". Would
>
> 	Used by the driver to set the MSI-X vector number for Configuration change interrupts.
>
> be equally acceptable?
Is it better also explain what is a MSI-X vector number here? Like 
saying the offset in the table or the slot number?

Thanks,
Zhu Lingshan
>
>
>>   \item[\field{num_queues}]
>>           The device specifies the maximum number of virtqueues supported here.
>> -- 
>> 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
>


---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org