On Tue, Oct 27, 2020 at 11:22:57PM +0000, Ophir Munk wrote:
> From: Tal Shnaiderman <tal...@nvidia.com>
> 
> Get the list of MAC addresses and verify if the input mac parameter
> already exists. If not - return -ENOTSUP (as Windows does not support
> adding new MAC addresses). If the MAC address exists (EEXIST) return 0
> (the equivalent of Linux implementation of this API).
> 
> Signed-off-by: Tal Shnaiderman <tal...@nvidia.com>
> ---
>  drivers/net/mlx5/windows/mlx5_os.c | 36 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 36 insertions(+)
> 
> diff --git a/drivers/net/mlx5/windows/mlx5_os.c 
> b/drivers/net/mlx5/windows/mlx5_os.c
> index b0cc9f3..6e27474 100644
> --- a/drivers/net/mlx5/windows/mlx5_os.c
> +++ b/drivers/net/mlx5/windows/mlx5_os.c
> @@ -183,6 +183,42 @@ mlx5_os_mac_addr_remove(struct rte_eth_dev *dev, 
> uint32_t index)
>  }
>  
> + *
> + * @return
> + *   0 on success, a negative errno value otherwise
> + */
> +int
> +mlx5_os_mac_addr_add(struct rte_eth_dev *dev, struct rte_ether_addr *mac,
> +                  uint32_t index)
> +{
> +     (void)index;
> +     struct rte_ether_addr lmac;
> +
> +     if (mlx5_get_mac(dev, &lmac.addr_bytes)) {
> +             DRV_LOG(ERR,
> +                     "port %u cannot get MAC address, is mlx5_en"
> +                     " loaded? (errno: %s)",
> +                     dev->data->port_id, strerror(rte_errno));
> +             return rte_errno;

Should it be "return -rte_errno" here?

> +     }
> +     if (memcmp(&lmac, mac, sizeof(struct rte_ether_addr))) {
> +             DRV_LOG(ERR,
> +                     "adding new mac address to device is unsupported");
> +             return -ENOTSUP;
> +     }
> +     return 0;
> +}
> +
> +/**
>   * Modify a VF MAC address
>   * Currently it has no support under Windows.
>   *
> -- 
> 2.8.4

Reply via email to