netifd: Check device_claim return code in all cases

Fixes observed issue a tunnel interface is reported as up although device_claim failed

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Fix ipip6 tunnel memleak in case of tunnel creation failure

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Fix macvlan delete via netlink

Fix macvlan delete via netlink as netlink attribute IFLA_IFNAME data size was incorrect (size reject by the kernel) and NLM_F_REQUEST flag was missing.
In addition some minor improvements (attribute IFLA_INFO_KIND can be left out as RTM_DELLINK does not require the attribute)

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Fix vlan delete via netlink

Fix vlan delete via netlink as netlink attribute IFLA_IFNAME data size was incorrect.
In addition some minor improvements.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

Reorder alfabetically device type matching as they are growing in number

Signed-off-by: Gioacchino Mazzurco <gmazzurco89@gmail.com>

Add vlan 802.1q/802.1ad support as netifd devices

At moment netifd supports just 802.1q vlan, you can configure them using a concise but "hacky"
syntax using an interface config section, with this patch netifd acquire the capability
of configuring 802.1ad and 802.1q vlan using config device sections, so you can define a vlan device
plus interface with something like this:

config device 'test'
        option type '8021ad'
        option name 'test'
        option ifname 'eth0'
        option vid '1000'

config interface 'testif'
        option ifname 'test'
        option proto 'none'
        option auto '1'

old syntax for 802.1q keeps working so no retrocompatibility problems,
to keep retrocompatibility means also that user must not use name/ifname like eth0.2
for devices declared with the new style because this would trigger the "old style"
when interface config section is parsed

Signed-off-by: Gioacchino Mazzurco <gmazzurco89@gmail.com>

Add qtcreator and kdevelop4 project files to .gitignore

Signed-off-by: Gioacchino Mazzurco <gmazzurco89@gmail.com>

netifd: process reload calls immediately instead of going through a timer

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

Add support for onlink-flags for IPv4 routes

Signed-off-by: Steven Barth <steven@midlink.org>

add a interface. prefix to ubus_notify calls

Signed-off-by: John Crispin <blogic@openwrt.org>

macvlan: fix missing reference to device_attr_list

At moment macvlan devices ignore options inherited as device like mtu.

Thanks to suggestions provided by Felix i have done this patch that fix this
iussue by adding the missing reference to device_attr_list in
macvlan_attr_list

Signed-off-by: Gioacchino Mazzurco <gmazzurco89@gmail.com>

Rewrite ipip6-tunnel setup to use netlink and add support for FMRs

Signed-off-by: Steven Barth <steven@midlink.org>

Fix a spurious segfault when tearing down interfaces

Signed-off-by: Steven Barth <steven@midlink.org>

wireless: fix cancelling setup

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

system-linux: always set the ifindex for hotplug devices on init

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: Fix node version set after free

Fixes an issue where a bridge member will be removed from the bridge
upon an interface ifup as the bridge node version -1 is overwritten
by vlist_add while the new created bridge member pointer is freed in
bridge_member_update

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Check interface state only when main device is set during interface_change_config

Fixes a regression issue introduced by commit d2a33f3f0fe704e4396fa2ada08401cb955ba7cb for device less protocol handlers.
An active interface using a deviceless protocol handler will be be teared down when the interface config is checked upon
an update as the interface link and enabled parameters are unset as no underlying device is present (eg tunnel interfaces)

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Effectively apply configured route mtu

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Remove useless route table parameter check in update_proto_route

Not necessary since route table parameter is used as key element in route_cmp

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Don't assume routes are always applied in all cases

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Apply interface metric on configured interface gateway parameters

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

device: prevent interface up/down calls for external devices

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

interface: clear iface->available before changing main_dev

Fixes bogus interface restart loop on interface_remove_link

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

interface: use interface_set_main_dev() from interface_remove_link

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

interface: do not set l3_dev along with main_dev anymore

Made unnecessary by recent fixes to l3_dev handling

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

interface-ip: fix misuse of IN6_IS_ADDR_LINKLOCAL on a netifd specific union

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: fix a use-after-free issue on wireless config update

After vlist_add() the device could have been freed

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: Add interface config support to enable/disable IPv6 in the kernel per device

Main use case is being able to disable IPv6 on (a) WAN interface(s) when only IPv4 connectivity is offered or 6rd is used.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

interface-event: make eventnames static and const

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: Rework hotplug event queueing in case of congestion

In case of event congestion (e.g. when several interfaces become active in a short notice) :
-hotplug events will be handled on a first come first served basis (before it was lifo)
-drop a new ifupdate event in favour of an already queued ifup event (before the ifup event was overwritten by the ifupdate event resulting in some hotplug scripts "missing" the ifup event)

Additonal the event flow has been documented

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>
Signed-off-by: Joeri Barbarien <joeri.barbarien@gmail.com>

netifd: Check rt_table parameter when comparing routes

Allows to add identical routes in different routing tables

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Set prefix assignment enabled if the prefix adress can be installed

No need to install policy routes if the prefix address cannot be installed; clean up of the policy routes
and the prefix address only needs to happen when the prefix assignment was active.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Mark interface as dynamic after it has been added in the interface list

Adding an interface in the interface list in case of a config update will override the node version of the old interface and thus overriding the dynamic interface marker.
In case of config update interface_set_dynamic is done on an invalid interface pointer as the new interface pointer has been freed.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: fix IPv6 Link-local DNS server

interface name must be added to Link-local IPv6
DNS servers, otherwise they are unusable.

Signed-off-by: Ulrich Weber <uw@ocedo.com>

config: add support for disabling interfaces

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

ubus: check the l3_dev pointer before dereferencing it

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

interface: ensure that l3_dev gets set whenever the proto handler marks the interface as up

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

interface: add support for a parameter to force setup without a carrier on the underlying device

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

proto-static: ensure that the l3 device is properly set

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

proto-shell: fix a regression with handling multiple proto updates for the same interface

Preserve the device when keep == true

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

proto-shell: ensure that l3_dev is always set

A link-down event clears l3_dev, so even if the proto handler does not
provide a new ifname, it still needs to be set explicitly

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

interface: always delete l3 dev on proto down

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

system-dummy: indicate link on present devices

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

config: ignore config parser errors

This helps with keeping routers reachable when there are typos in the
network config.

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

ubus: use list_empty() instead of the compat wrapper

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: Reload proto on topology change

Introduce a new device event "topology change" that gets signaled
by bridges on adding/removing members.

On "topology changes" the proto handlers are requested to "renew"
which is most useful for DHCP.

Signed-off-by: Helmut Schaa <helmut.schaa@googlemail.com>

ubus: fix displaying interface data items

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: Check interface autostart parameter when handling interface state changes

Fixes an issue when 2 interfaces make use of the same ifname (device) and one of the interfaces has autostart disabled; bringing up the other interface would also start the interface for which autostart is disabled (link_state and enabled will both be true for the autostart disabled interface and thus a setup will be launched)

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

uci: Allow setting the UCI config path on the command line

Add a -c switch which allows setting an alternative UCI path instead of using
the libuci default.

Signed-off-by: Matthias Schiffer <mschiffer@universe-factory.net>

Revert "ubus events: don't incorrectly announce an interface as being up"

Made obsolete by b114b86c70766f67f503077ad1de013c17fbf366

This reverts commit 4d1a597f65bbb49aa843112812d5ca929478c706.

netifd: Set interface state in teardown state before launching an interface event

Interface state needs to be set in teardown state before launching an interface event otherwise the up state will be reported as true in the ubus interface notify message

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Assign interface metric to route metric when route is created

Interface metric needs to be assigned to the route metric parameter at route creation time. Otherwise if the interface metric is different from 0 route_cmp will wrongly conclude the routes are different.
In this case the route will be added/deleted and could end up with the route missing in the kernel depending on the add/delete order.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Fix bridge MTU setting when a bridge member is added

Reapply bridge mtu setting as adding a bridge member will override the bridge mtu in the kernel

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

proto-shell: defer init until after command line option processing

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: fix bridge reloading issue

Check in the right order to decide if the bridge have to be set up.
Additionally we are consistent with setting it down.

Signed-off-by: Emanuel Taube <emanuel.taube@gmail.com>

wireless: defer init until after command line option processing

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

ubus events: don't incorrectly announce an interface as being up

Signed-off-by: Steven Barth <steven@midlink.org>

netifd: Link layer state awareness support on interface level

The link layer state is monitored for a given interface; an interface will be setup
when both enabled and link layer active. Likewise an interface will be teared down
when either disabled or link layer down.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Link layer state awareness support on device level

Device layer is informed by netlink events regarding the link layer status. Link
layer status change results in a DEV_EVENT_LINK_UP/DEV_EVENT_LINK_DOWN broadcast
event for a given device.

Depends on uloop error callback patch.

Solves issue reported in https://dev.openwrt.org/ticket/14590

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>
Acked-by: Karl Vogel <karl.vogel@gmail.com>

Don't always assume routes & addresses are applied

Actually check netlink return values and remember failure.

Signed-off-by: Steven Barth <steven@midlink.org>

wireless: fix device config reload regression

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: add a "get_validate" methode to ubus

Signed-off-by: John Crispin <blogic@openwrt.org>

wireless: load and then ignore disabled devices

Signed-off-by: John Crispin <blogic@openwrt.org>

add validation rules to proto static

Signed-off-by: John Crispin <blogic@openwrt.org>

get_proto_handlers now also dumps the validation info

Signed-off-by: John Crispin <blogic@openwrt.org>

make netifd_handler_parse_config honour the luci validation passed from the backend scripts

Signed-off-by: John Crispin <blogic@openwrt.org>

add a helper to convert blobmsg types to luci validate rules

Signed-off-by: John Crispin <blogic@openwrt.org>

Work around warn_unused_result in recent gcc/libc

IPv6: work around dangling routes kernel issue

Fix source-restrictions of routes when no mask was provided

netifd : Fix ubus route src mask printing

Fix printing of route src mask attribute when it contains 3 digits

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd : Disable netlink auto ack

Disable netlink auto ack when doing a delete in the get callback
handler to avoid race conditions resulting into stalled message
on the netlink socket.

Solves issue reported in https://dev.openwrt.org/ticket/14590

Signed-off-by: Karl Vogel <karl.vogel@gmail.com>
Acked-by: Hans Dedecker <dedeckeh@gmail.com>

Don't add unnecessary NOP policy rules

Remove automatically assigned IPv6 routing table

This puts IPv6 routes back into the main table where they are expected.
Instead the newly-fixed source-address restrictions should be used.

Add indicator-flags to ubus and hotplug update-events

wireless: fix alignment of wireless config structs - fixes segfaults on arm

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: cancel the script check timer

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

device: prevent callbacks to device users that have been cleaned up already

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

Revert "netifd: Link layer state support on device level"

This reverts commit c439b52400978dd3799c66e1f632ee68d2c7c9eb.

Revert "netifd: Link layer state support on interface level"

This reverts commit 7335c3e19ea09674245dda3653d3f40705e68d6b.

Set dynamic interface flag after adding them so the version is preserved

wireless: remove the no_delete flag, remove virtual interfaces from the avl tree

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: fix handling of wdev->config_state, prevent handling of multiple state changes (reload + remove)

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: remove freed wireless devices from the vlist avl tree

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

IPv6: Remove IPv6 source-routing workaround (kernel is fixed)

Signed-off-by: Steven Barth <steven@midlink.org>

wireless: add back support for multiple networks per interface

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

netifd: Link layer state support on interface level

Patch implements handling of link layer support on interface level.
An interface will go into the setup state when it's enabled and the
underlying link state is enabled. Vice versa an interface will go to
the down state when it's either disabled or underlying link state is
disabled.
Testing has been done with PPP, IPoE, tunnel and static interfaces

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Link layer state support on device level

Patch implements link layer state awareness (aka carrier detection) in
netifd on device level.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Code style line/clean up

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

Tunnel don't fragment bit configurable

Make tunnel don't fragment bit configurable via UCI

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Apply tunnel MTU

Tunnel MTU is applied according to the tunnel MTU UCI parameter

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Don't export default route via UBUS when disabled

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Export ip4table/ip6table via UBUS

Export ip4table/ip6table parameters via UBUS to facilitate
route inspection and usage by other applications

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

netifd: Routing table parameter needs to be checked in route update

Routing table parameter needs to be checked in interface_update_proto_route; a route which
has an identical nexthop but a different routing table needs to be deleted and added in the
correct routing table.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>

wireless: make interfaces in status an array instead of an object

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: prepare bridge interfaces before passing them to the setup script

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: add an example driver cleanup implementation

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: fix getting the network_bridge variable

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: only process setup retries in setup/up state

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wireless: fix htmode handling

Signed-off-by: Felix Fietkau <nbd@openwrt.org>