[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAE4R7bDt2JM_ZXTrqq9Hz3ZDJbfb1=_06b8LftE-3-BBsONp_Q@mail.gmail.com>
Date: Sat, 21 Mar 2015 20:21:08 -0700
From: Scott Feldman <sfeldma@...il.com>
To: Roopa Prabhu <roopa@...ulusnetworks.com>
Cc: "David S. Miller" <davem@...emloft.net>,
Jiří Pírko <jiri@...nulli.us>,
Netdev <netdev@...r.kernel.org>
Subject: Re: [PATCH net-next v6] switchdev: fix stp update API to work with
layered netdevices
On Sat, Mar 21, 2015 at 10:27 AM, <roopa@...ulusnetworks.com> wrote:
> From: Roopa Prabhu <roopa@...ulusnetworks.com>
>
> make it same as the netdev_switch_port_bridge_setlink/dellink
> api (ie traverse lowerdevs to get to the switch port).
>
> removes "WARN_ON(!ops->ndo_switch_parent_id_get)" because
> direct bridge ports can be stacked netdevices (like bonds
> and team of switch ports) which may not implement this ndo.
>
> v2 to v3:
> - remove changes to bond and team. Bring back the
> transparently following lowerdevs like i initially
> had for setlink/getlink
> (http://www.spinics.net/lists/netdev/msg313436.html)
> dave and scott feldman also seem to prefer it be that
> way and move to non-transparent way of doing things
> if we see a problem down the lane.
>
> v3 to v4:
> - fix ret initialization
>
> v4 to v5:
> - return err on first failure (scott feldman)
>
> v5 to v6:
> - change variable name (err) and initialize to
> -EOPNOTSUPP (scott feldman).
>
> Signed-off-by: Roopa Prabhu <roopa@...ulusnetworks.com>
Acked-by: Scott Feldman <sfeldma@...il.com>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists