net/i40e: remove memset in i40evf_dev_info_get
Checks
Commit Message
i40evf_dev_info_get clears dev_info when data has already
been set by the calling function. Remove the call to
memset() to fix the problem.
Fixes: 4861cde46116 ("i40e: new poll mode driver")
Cc: helin.zhang@intel.com
Signed-off-by: David Harton <dharton@cisco.com>
---
drivers/net/i40e/i40e_ethdev_vf.c | 1 -
1 file changed, 1 deletion(-)
Comments
Hi,
Thanks for the patch, but there's another patch which has been accepted for the issue.
https://patches.dpdk.org/patch/40682/
Best Regards,
Beilei
> -----Original Message-----
> From: David Harton [mailto:dharton@cisco.com]
> Sent: Thursday, June 28, 2018 9:16 AM
> To: Xing, Beilei <beilei.xing@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: dev@dpdk.org; David Harton <dharton@cisco.com>; Zhang, Helin
> <helin.zhang@intel.com>
> Subject: [PATCH] net/i40e: remove memset in i40evf_dev_info_get
>
> i40evf_dev_info_get clears dev_info when data has already been set by the
> calling function. Remove the call to
> memset() to fix the problem.
>
> Fixes: 4861cde46116 ("i40e: new poll mode driver")
> Cc: helin.zhang@intel.com
>
> Signed-off-by: David Harton <dharton@cisco.com>
> ---
> drivers/net/i40e/i40e_ethdev_vf.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/net/i40e/i40e_ethdev_vf.c
> b/drivers/net/i40e/i40e_ethdev_vf.c
> index 804e445..86b38d2 100644
> --- a/drivers/net/i40e/i40e_ethdev_vf.c
> +++ b/drivers/net/i40e/i40e_ethdev_vf.c
> @@ -2182,7 +2182,6 @@ i40evf_dev_info_get(struct rte_eth_dev *dev,
> struct rte_eth_dev_info *dev_info) {
> struct i40e_vf *vf = I40EVF_DEV_PRIVATE_TO_VF(dev->data-
> >dev_private);
>
> - memset(dev_info, 0, sizeof(*dev_info));
> dev_info->max_rx_queues = vf->vsi_res->num_queue_pairs;
> dev_info->max_tx_queues = vf->vsi_res->num_queue_pairs;
> dev_info->min_rx_bufsize = I40E_BUF_SIZE_MIN;
> --
> 2.10.3.dirty
@@ -2182,7 +2182,6 @@ i40evf_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
{
struct i40e_vf *vf = I40EVF_DEV_PRIVATE_TO_VF(dev->data->dev_private);
- memset(dev_info, 0, sizeof(*dev_info));
dev_info->max_rx_queues = vf->vsi_res->num_queue_pairs;
dev_info->max_tx_queues = vf->vsi_res->num_queue_pairs;
dev_info->min_rx_bufsize = I40E_BUF_SIZE_MIN;