On Sun, Jul 26, 2015 at 4:45 PM, Vivien Didelot <vivien.dide...@savoirfairelinux.com> wrote: > This patch replaces the vid_begin and vid_end members of the > switchdev_obj_vlan structure for a single vid member. This way, the VID > range abstraction is restricted to switchdev, not exposed to drivers. > > The main benefice to do so is to allow the prepare phase to be called > for each VID, not only once for the whole range. > > For example, when adding VLANs 2-5, a switch chip may not be able to add > hardware VLAN 2 due to some bridge restriction (thus must return > -EOPNOTSUPP), while VLAN 3-5 are allowed.
Hi Vivien, Since the netlink request (for example vlan add) includes the range, I'm not seeing how we can response with success for the satisfied vlans in the range, and also respond with an error for the unsatisfied vlans in the range. In other words, from the netlink msgs perspective, we need to treat a vlan range as all-or-nothing. So in your example, if hw can't add vlan 2, we fail the entire request to add range 2-5. This is where the prepare phase checks to make sure the entire request can be satisfied before committing to hw. -scott -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
