[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] correct some register return values for vxm
From: |
Miao Yan |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] correct some register return values for vxmnet3 |
Date: |
Tue, 22 Dec 2015 10:44:34 +0800 |
Hi Dmitry,
2015-12-22 1:19 GMT+08:00 Dmitry Fleytman <address@hidden>:
> Hello Miao,
>
> While patches look good technically, I cannot find any code snippets that
> prove correctness of these changes.
Linux driver does not read those registers currently,
not sure about Windows version.
> How do you know this is the correct behaviour? Could you please extend commit
> messages with corresponding references?
This behavior can be observed by modifying linux driver to read those registers
at probe time (in vmxnet3_device_probe) and kernel log will have the following
message (running on esxi server, of source):
[ 198.427389] VMware vmxnet3 virtual NIC driver - version 1.2.0.0-k-NAPI
[ 198.428863] vmxnet3 0000:03:00.0: # of Tx queues : 1, # of Rx queues : 1
[ 198.559625] vmxnet3 DID lo: 0x7b0, high: 0x1, dev_info: 0x0
[ 198.561151] vmxnet3 0000:03:00.0: irq 72 for MSI/MSI-X
Here DID_LO is the pci device id, DID_HIGH is 0x1 and
GET_DEV_EXTRA_INFO returns 0.
putting above into commit message should be enough ?
>
> Thanks,
> Dmitry
>
>> On 21 Dec 2015, at 13:06 PM, Miao Yan <address@hidden> wrote:
>>
>> Qemu vmxnet3 emulation doesn't recognize VMXNET3_CMD_GET_DID_LO,
>> VMXNET3_CMD_GET_DID_HI and VMXNET3_CMD_GET_DEV_EXTRA_INFO command and
>> returns -1 on all of them.
>>
>> This patchset makes them return correct values.
>>
>> Miao Yan (3):
>> net/vmxnet3: return 1 on device activation failure
>> net/vmxnet3: return correct value for VMXNET3_CMD_GET_DID command
>> net/vmxnet3: return correct value for VMXNET3_CMD_GET_DEV_EXTRA_INFO
>>
>> hw/net/vmxnet3.c | 11 ++++++++++-
>> 1 file changed, 10 insertions(+), 1 deletion(-)
>>
>> --
>> 1.9.1
>>
>
[Qemu-devel] [PATCH 3/3] net/vmxnet3: return correct value for VMXNET3_CMD_GET_DEV_EXTRA_INFO, Miao Yan, 2015/12/21
Re: [Qemu-devel] [PATCH 0/3] correct some register return values for vxmnet3, Dmitry Fleytman, 2015/12/21
- Re: [Qemu-devel] [PATCH 0/3] correct some register return values for vxmnet3,
Miao Yan <=