Re: [PATCH net-next 4/5] rocker: add offload_fwd_mark support

[Simon Horman <simon.horman@netronome.com>]

Hi Scott,

On Wed, Jul 08, 2015 at 04:16:42PM -0700, sfeldma@gmail.com wrote:
> From: Scott Feldman <sfeldma@gmail.com>
> 
> If device flags ingress packet as "fwd offload", mark the
> skb->offlaod_fwd_mark using the ingress port's dev->offlaod_fwd_mark.  This
> will be the hint to the kernel that this packet has already been forwarded
> by device to egress ports matching skb->offlaod_fwd_mark.
> 
> For rocker, derive port dev->offlaod_fwd_mark based on device switch ID and
> port ifindex.  If port is bridged, use the bridge ifindex rather than the
> port ifindex.
> 
> Signed-off-by: Scott Feldman <sfeldma@gmail.com>

[snip]

> diff --git a/drivers/net/ethernet/rocker/rocker.c b/drivers/net/ethernet/rocker/rocker.c
> index c005167..a4ced91 100644
> --- a/drivers/net/ethernet/rocker/rocker.c
> +++ b/drivers/net/ethernet/rocker/rocker.c

[snip] 

> @@ -4938,11 +4944,13 @@ static int rocker_probe_port(struct rocker *rocker, unsigned int port_number)
>  	}
>  	rocker->ports[port_number] = rocker_port;
>  
> +	switchdev_port_fwd_mark_set(rocker_port->dev, NULL, false);
> +
>  	rocker_port_set_learning(rocker_port, SWITCHDEV_TRANS_NONE);
>  
>  	err = rocker_port_ig_tbl(rocker_port, SWITCHDEV_TRANS_NONE, 0);
>  	if (err) {
> -		dev_err(&pdev->dev, "install ig port table failed\n");
> +		netdev_err(rocker_port->dev, "install ig port table failed\n");
>  		goto err_port_ig_tbl;
>  	}

It is unclear to me how the above *_err change and...

>  
> @@ -4962,6 +4970,7 @@ err_untagged_vlan:
>  	rocker_port_ig_tbl(rocker_port, SWITCHDEV_TRANS_NONE,
>  			   ROCKER_OP_FLAG_REMOVE);
>  err_port_ig_tbl:
> +	rocker->ports[port_number] = NULL;
>  	unregister_netdev(dev);
>  err_register_netdev:
>  	free_netdev(dev);

... the above ports[] change relate to the rest of this patch.

[snip]