[dpdk-dev,4/7] virtio: Don't fill dev_info->driver_name

  This is overwritten in rte_eth_dev_info_get().

Signed-off-by: Jan Blunck <jblunck@infradead.org>
---
 drivers/net/virtio/virtio_ethdev.c | 4 ----
 1 file changed, 4 deletions(-)
  

On Sun, Nov 20, 2016 at 11:05 AM, Jan Blunck <jblunck@infradead.org> wrote:
> This is overwritten in rte_eth_dev_info_get().
>
> Signed-off-by: Jan Blunck <jblunck@infradead.org>
> ---
>  drivers/net/virtio/virtio_ethdev.c | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c
> index 079fd6c..741688e 100644
> --- a/drivers/net/virtio/virtio_ethdev.c
> +++ b/drivers/net/virtio/virtio_ethdev.c
> @@ -1624,10 +1624,6 @@ virtio_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
>         uint64_t tso_mask;
>         struct virtio_hw *hw = dev->data->dev_private;
>
> -       if (dev->pci_dev)
> -               dev_info->driver_name = dev->driver->pci_drv.driver.name;
> -       else
> -               dev_info->driver_name = "virtio_user PMD";
>         dev_info->max_rx_queues =
>                 RTE_MIN(hw->max_queue_pairs, VIRTIO_MAX_RX_QUEUES);
>         dev_info->max_tx_queues =
> --
> 2.7.4
>

I posted something similar [1], so looks good to me :-)

[1]: http://dpdk.org/dev/patchwork/patch/16991/
  

On Sun, Nov 20, 2016 at 4:22 PM, David Marchand
<david.marchand@6wind.com> wrote:
> On Sun, Nov 20, 2016 at 11:05 AM, Jan Blunck <jblunck@infradead.org> wrote:
>> This is overwritten in rte_eth_dev_info_get().
>>
>> Signed-off-by: Jan Blunck <jblunck@infradead.org>
>> ---
>>  drivers/net/virtio/virtio_ethdev.c | 4 ----
>>  1 file changed, 4 deletions(-)
>>
>> diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c
>> index 079fd6c..741688e 100644
>> --- a/drivers/net/virtio/virtio_ethdev.c
>> +++ b/drivers/net/virtio/virtio_ethdev.c
>> @@ -1624,10 +1624,6 @@ virtio_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
>>         uint64_t tso_mask;
>>         struct virtio_hw *hw = dev->data->dev_private;
>>
>> -       if (dev->pci_dev)
>> -               dev_info->driver_name = dev->driver->pci_drv.driver.name;
>> -       else
>> -               dev_info->driver_name = "virtio_user PMD";
>>         dev_info->max_rx_queues =
>>                 RTE_MIN(hw->max_queue_pairs, VIRTIO_MAX_RX_QUEUES);
>>         dev_info->max_tx_queues =
>> --
>> 2.7.4
>>
>
> I posted something similar [1], so looks good to me :-)
>
> [1]: http://dpdk.org/dev/patchwork/patch/16991/
>

Thanks for reviewing. Do we go with your proposal then? Do you plan to
follow up on the comments?
  

On Mon, Nov 21, 2016 at 5:34 PM, Jan Blunck <jblunck@infradead.org> wrote:
> On Sun, Nov 20, 2016 at 4:22 PM, David Marchand
>> I posted something similar [1], so looks good to me :-)
>>
>> [1]: http://dpdk.org/dev/patchwork/patch/16991/
>>
>
> Thanks for reviewing. Do we go with your proposal then? Do you plan to
> follow up on the comments?

Yes will do.
Out of curiosity how did you catch the qede driver ? Did you check all
other drivers ?
  

On Mon, Nov 21, 2016 at 5:49 PM, David Marchand
<david.marchand@6wind.com> wrote:
> On Mon, Nov 21, 2016 at 5:34 PM, Jan Blunck <jblunck@infradead.org> wrote:
>> On Sun, Nov 20, 2016 at 4:22 PM, David Marchand
>>> I posted something similar [1], so looks good to me :-)
>>>
>>> [1]: http://dpdk.org/dev/patchwork/patch/16991/
>>>
>>
>> Thanks for reviewing. Do we go with your proposal then? Do you plan to
>> follow up on the comments?
>
> Yes will do.
> Out of curiosity how did you catch the qede driver ? Did you check all
> other drivers ?
>

$ grep drivername -r drivers/