On 8/21/2019 3:16 AM, Myers, Charles wrote:
> From: Charles Myers <charles.my...@spirent.com>
> 
> When the mtu_set() function is not implemented, rte_eth_dev_set_mtu()
> fails with -ENOTSUP and mtu is not stored in the mtu field in the
> rte_eth_dev_data.  This causes the mtu in Vmxnet3_MiscConf which is
> shared with hypervisor to always be set to 1500.
> 
> This may cause issues receiving jumbo frames on Enhanced Data Path
> N-VDS.
> 
> Signed-off-by: Charles Myers <charles.my...@spirent.com>

Reviewed-by: Ferruh Yigit <ferruh.yi...@intel.com>

Applied to dpdk-next-net/master, thanks.


> @@ -1205,6 +1209,18 @@ vmxnet3_dev_supported_ptypes_get(struct rte_eth_dev 
> *dev)
>  }
>  
>  static int
> +vmxnet3_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
> +{

'mtu' is unused and giving build error:
error: unused parameter ‘mtu’ [-Werror=unused-parameter]

Fixed while merging:
 +vmxnet3_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu __rte_unused)


Reply via email to