[OpenWrt-Devel] [PATCH 2/2] netifd: Don't call set_state for external device in device_claim

[Hans Dedecker]

On Tue, Sep 1, 2015 at 2:49 PM, Felix Fietkau <nbd at openwrt.org> wrote:

> On 2015-09-01 14:43, Hans Dedecker wrote:
> > The function set_state disable is not called for external devices in
> device_release
> > which means for external vlan/macvlan devices they won't be deleted.
> > As a result of this the set_state enable call for external devices by
> device_claim fails
> > as vlan/macvlan devices cannot be created since the device already
> exists in the kernel.
> > Therefore move the external device check from device_set_state to
> device_claim so
> > external vlan/macvlan devices are not created again and can also be
> external.
> Why/how do vlan/macvlan devices become external?
>
This use case is driven by an external application which is adding a vlan
device to the bridge.
Initially the vlan device is created as an internal device but not added to
the bridge; later added by an external application via ubus to the bridge.
In this scenario we hit the issue when doing network reload the vlan device
is not anymore an active member of the bridge as vlan set_state enable was
called which failed.
This is a bit similar to a wireless device which has uci device config;
initially it's considered as an internal device by netifd when loading the
config but becomes an external device when the wireless logic adds it to
the bridge.

Hans

>
> - Felix
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/openwrt-devel/attachments/20150901/2ec8bd3b/attachment.htm>
-------------- next part --------------
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel