On 7 April 2017 at 06:13, Roi Dayan <[email protected]> wrote:
> From: Paul Blakey <[email protected]>
>
> netdev vports are backed by actualy netdev at the kernel
> level, so they can use the common netdev-tc offloads interface
> for flow offloading (if enabled).
>
> Signed-off-by: Paul Blakey <[email protected]>
> Signed-off-by: Simon Horman <[email protected]>
> Reviewed-by: Roi Dayan <[email protected]>
> ---
<snip>
> @@ -779,11 +783,37 @@ get_stats(const struct netdev *netdev, struct
> netdev_stats *stats)
> return 0;
> }
>
> -
Please don't drop the ^L, they're used to separate the file into
logically separate sections.
> +#ifdef __linux__
> +static int
> +netdev_vport_get_ifindex__(const struct netdev *netdev_)
> +{
> + char buf[NETDEV_VPORT_NAME_BUFSIZE];
> + const char *name = netdev_vport_get_dpif_port(netdev_, buf, sizeof(buf));
> +
> + return linux_get_ifindex(name);
Why link directly against linux implementation, ignoring that netdev
provides netdev_get_ifindex() ?
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
