New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FS#1452 - [netifd] gretap instantiation fails on and after commit 1cd76e2 on "master" #6421
Comments
jeffsf: Looking at the ''netifd'' commits, it appears that the change in behavior was introduced at the commit shown below. While a patch to reverse this commit "resolves" the symptom, it is not clear that it is the //cause// of the problem. The change in ''netifd'' may cause ''NULL'' to be returned in some cases, rather than ''iface'' which may cause change in behavior elsewhere.
The "reversal" also removes the symptoms on recent ''master'' (but again may cause other breakage I am not aware of)
|
jeffsf: Difference between "working" and "non-working" first show up to me in the call from within ''gre.sh'' that works down to On a "bad" build, the ''ubus'' call returns 4 ''UBUS_STATUS_NOT_FOUND'' On a "good" build, the ''ubus'' call returns 0 This likely comes from ''netifd_add_host_route()'' in ''ubus.c'' The following is a work-around, as I don't know what logic there might be around the NULL being returned
|
nbd: Fixed in r6552-d290024c42, thanks. |
jeffsf: Thanks!! |
jeffsf:
====Summary====
Prior to the indicated commit, gretap interfaces could be initialized through ''/etc/config/network''
After the indicated commit, gretap interfaces fail to be configured.
gretap interfaces can be initialized "by hand" using ''ip'' from ''ip-full'' in both cases
====Impact====
Unable to utilize gretap interfaces with UCI-based configuration
====To Replicate====
config interface 'gt'
option proto 'gretap'
option ipaddr '192.168.1.1'
option peeraddr '192.168.1.2'
option force_link '1'
root@OpenWrt:~# logread | fgrep gt Tue Mar 13 12:35:25 2018 daemon.notice netifd: Interface 'gt' is setting up now Tue Mar 13 12:35:26 2018 daemon.notice netifd: Interface 'gt' is now down
====Expected Behavior====
13: gre4t-gt99@NONE: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1280 qdisc fq_codel state UNKNOWN qlen 1000
link/ether ba:e1:b8:b7:fa:f5 brd ff:ff:ff:ff:ff:ff
root@OpenWrt:~# logread | fgrep gt
Tue Mar 13 12:17:18 2018 daemon.notice netifd: Interface 'gt' is setting up now
Tue Mar 13 12:17:19 2018 daemon.notice netifd: Interface 'gt' is now up
Tue Mar 13 12:17:19 2018 daemon.notice netifd: tunnel 'gre4t-gt' link is up
Tue Mar 13 12:17:20 2018 user.notice firewall: Reloading firewall due to ifup of gt (gre4t-gt)
====Potential Cause====
commit 1cd76e2d85d86356868db731a5cacfb84150b2a1
Author: Felix Fietkau nbd@nbd.name
Date: Tue Mar 13 13:33:04 2018 +0100
The text was updated successfully, but these errors were encountered: