Merge ~smb/ubuntu/+source/iproute2:ubuntu/eoan-devel into ubuntu/+source/iproute2:debian/experimental

Proposed by Christian Ehrhardt 
Status: Merged
Approved by: Andreas Hasenack
Approved revision: ea4e2e1c9714c3c6d410ceff9786308b6980102b
Merge reported by: Christian Ehrhardt 
Merged at revision: f66e6703fea8112f60470c6c83bf0ad2828ec6c7
Proposed branch: ~smb/ubuntu/+source/iproute2:ubuntu/eoan-devel
Merge into: ubuntu/+source/iproute2:debian/experimental
Diff against target: 849 lines (+690/-44)
8 files modified
debian/changelog (+254/-3)
debian/control (+2/-1)
debian/patches/1000-ubuntu-poc-fan-driver.patch (+74/-0)
debian/patches/1001-ubuntu-poc-fan-driver-v3.patch (+134/-0)
debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch (+178/-0)
debian/patches/1005-lib-suppress-error-msg-when-filling-the-cache.patch (+40/-0)
debian/patches/series (+8/-1)
dev/null (+0/-39)
Reviewer Review Type Date Requested Status
Andreas Hasenack Approve
Christian Ehrhardt  (community) Approve
Review via email: mp+369140@code.launchpad.net

This proposal supersedes a proposal from 2019-06-17.

Description of the change

Rebased to Debian experimental as that is even closer to the target kernel version for Eoan.

To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote : Posted in a previous version of this proposal

The MP here is against the wrong target - since you rebased to debian/experimental the merge target here would be "debian/experimental"
Further it would be helpful to give new branches names, but I can still review from checking out your branch thou ...

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I did the re-target for you

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Minor changelog bikeshedding inline

review: Needs Information
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

You'd not need to carry the old commits and then revert them.
Those can just be deleted, as combined they are a no-op.

Since this is functionally equivalent this is just an FYI.
You could as well squash them into "nothing" before the next rebase.
But since dropping them now just makes life easier down the road I'd recommend to do so.

The rest LGTM, do you have a PPA to build-test and test it ahead of an upload?

review: Needs Fixing
Revision history for this message
Stefan Bader (smb) wrote :

Will additionally upload to https://launchpad.net/~smb/+archive/ubuntu/eoan. I had done test sbuild and ran ubuntu-fan dep8 tests with the new iproute2 installed in the testbed as integration test.

Revision history for this message
Christian Ehrhardt  (paelzer) :
Revision history for this message
Stefan Bader (smb) wrote :

On 03.07.19 12:24, Christian Ehrhardt  wrote:
>
>
> Diff comments:
>
>> diff --git a/debian/changelog b/debian/changelog
>> index 190b8b2..000d6f7 100644
>> --- a/debian/changelog
>> +++ b/debian/changelog
>> @@ -1,3 +1,24 @@
>> +iproute2 (5.1.0-1ubuntu1) eoan; urgency=medium
>> +
>> + * Merge with Debian experimental. Remaining changes:
>> + - Add: Ubuntu Fan driver (V2/3/4) patches
>> + - d/p/1000-ubuntu-poc-fan-driver.patch
>> + Ubuntu FAN driver support V2 (LP 1439706)
>> + - d/p/1001-ubuntu-poc-fan-driver-v3.patch
>> + Ubuntu FAN driver support V3 (LP 1470091)
>> + - d/p/1002-ubuntu-poc-fan-driver-vxlan.patch
>> + Ubuntu FAN driver support VXLAN V4.
>> + - Drop: debian/patches/0001-Add-moo-feature.patch
>> + - Refresh: Ubuntu Fan driver (V2/3/4) patches
>
> That I didn't mean.
> Let me outline how the usual standard would look like and you will see:
>
> + * Merge with Debian experimental. Remaining changes:
> + - Ubuntu Fan driver (V2/3/4) patches
> + - d/p/1000-ubuntu-poc-fan-driver.patch
> + Ubuntu FAN driver support V2 (LP 1439706)
> + - d/p/1001-ubuntu-poc-fan-driver-v3.patch
> + Ubuntu FAN driver support V3 (LP 1470091)
> + - d/p/1002-ubuntu-poc-fan-driver-vxlan.patch
> + Ubuntu FAN driver support VXLAN V4.
> + - Drop: debian/patches/0001-Add-moo-feature.patch
> + * Added changes:
> + - Refresh: Ubuntu Fan driver (V2/3/4) patches

Ok, so it is more how one looks at it. Most of the time when merging, the
refresh was again needed. So either you look at it as part of the remaining
changes from that perspective or it is something added (everytime).

But I can modify the changelog. I fear it will be a re-occuring todo.

>
>> + * Dropped changes:
>> + - d/p/1005-ubuntu-fix-testsuite-kenv.patch (upstream)
>> + - d/p/1006-ubuntu-iprule-fix-output.patch (upstream
>> + - d/p/ss-review-ssfilter.patch (upstream)
>> + - Set debhelper dependency to >= 10 (upstream)
>> + - Set compat level 10 (upstream)
>> +
>> + -- Stefan Bader <email address hidden> Mon, 17 Jun 2019 11:15:33 +0200
>> +
>> iproute2 (5.1.0-1) experimental; urgency=medium
>>
>> * Update upstream source from tag 'upstream/5.1.0'
>
>

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

It is your style decision and I explained the usual look, nothing I'd nack you on.
Due to that also I should add my +1 as I think you are good (and style decisions are up to you as uploader).

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1, I see what Christian means with adding and dropping patches, that complicates the review a bit. If you like, maybe we can get together at a sprint in the future to go over how we usually do these merges, and if you would like to follow that style.

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Sponsoring ea4e2e1c9714c3c6d410ceff9786308b6980102b

$ git push pkg upload/5.1.0-1ubuntu1
Enumerating objects: 71, done.
Counting objects: 100% (71/71), done.
Delta compression using up to 2 threads
Compressing objects: 100% (22/22), done.
Writing objects: 100% (65/65), 14.74 KiB | 1.84 MiB/s, done.
Total 65 (delta 45), reused 58 (delta 42)
To ssh://git.launchpad.net/~usd-import-team/ubuntu/+source/iproute2
 * [new tag] upload/5.1.0-1ubuntu1 -> upload/5.1.0-1ubuntu1

$ dput ubuntu ../iproute2_5.1.0-1ubuntu1_source.changes
Checking signature on .changes
gpg: ../iproute2_5.1.0-1ubuntu1_source.changes: Valid signature from AC983EB5BF6BCBA9
Checking signature on .dsc
gpg: ../iproute2_5.1.0-1ubuntu1.dsc: Valid signature from AC983EB5BF6BCBA9
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading iproute2_5.1.0-1ubuntu1.dsc: done.
  Uploading iproute2_5.1.0.orig.tar.xz: done.
  Uploading iproute2_5.1.0-1ubuntu1.debian.tar.xz: done.
  Uploading iproute2_5.1.0-1ubuntu1_source.buildinfo: done.
  Uploading iproute2_5.1.0-1ubuntu1_source.changes: done.
Successfully uploaded packages.

e445254... by Stefan Bader

Add: d/p/1005-lib-suppress-error-msg-when-filling-the-cache.patch

This suppresses an RTLINK level message to stderr that could
happen when creating interfaces. The result was ok but this
did break scripts (and testing).

Signed-off-by: Stefan Bader <email address hidden>

f66e670... by Stefan Bader

Release: 5.1.0-1ubuntu2

Signed-off-by: Stefan Bader <email address hidden>

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

This migrated, closing forgotten MP

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 190b8b2..50f33c6 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,31 @@
6+iproute2 (5.1.0-1ubuntu2) eoan; urgency=medium
7+
8+ * Add: d/p/1005-lib-suppress-error-msg-when-filling-the-cache.patch
9+ Suppress meaningless error message during interface setup.
10+
11+ -- Stefan Bader <stefan.bader@canonical.com> Wed, 10 Jul 2019 16:38:52 +0200
12+
13+iproute2 (5.1.0-1ubuntu1) eoan; urgency=medium
14+
15+ * Merge with Debian experimental. Remaining changes:
16+ - Add: Ubuntu Fan driver (V2/3/4) patches
17+ - d/p/1000-ubuntu-poc-fan-driver.patch
18+ Ubuntu FAN driver support V2 (LP 1439706)
19+ - d/p/1001-ubuntu-poc-fan-driver-v3.patch
20+ Ubuntu FAN driver support V3 (LP 1470091)
21+ - d/p/1002-ubuntu-poc-fan-driver-vxlan.patch
22+ Ubuntu FAN driver support VXLAN V4.
23+ - Drop: debian/patches/0001-Add-moo-feature.patch
24+ - Refresh: Ubuntu Fan driver (V2/3/4) patches
25+ * Dropped changes:
26+ - d/p/1005-ubuntu-fix-testsuite-kenv.patch (upstream)
27+ - d/p/1006-ubuntu-iprule-fix-output.patch (upstream
28+ - d/p/ss-review-ssfilter.patch (upstream)
29+ - Set debhelper dependency to >= 10 (upstream)
30+ - Set compat level 10 (upstream)
31+
32+ -- Stefan Bader <stefan.bader@canonical.com> Mon, 17 Jun 2019 11:15:33 +0200
33+
34 iproute2 (5.1.0-1) experimental; urgency=medium
35
36 * Update upstream source from tag 'upstream/5.1.0'
37@@ -70,6 +98,43 @@ iproute2 (4.18.0-2) unstable; urgency=medium
38
39 -- Luca Boccassi <bluca@debian.org> Tue, 28 Aug 2018 16:55:44 +0100
40
41+iproute2 (4.18.0-1ubuntu3) eoan; urgency=medium
42+
43+ * d/p/ss-review-ssfilter.patch: fixed issue with ss and ssfilter
44+ where ssfilter rejects single expressions if enclosed in
45+ braces (LP: #1831775)
46+
47+ -- Rafael David Tinoco <rafaeldtinoco@ubuntu.com> Wed, 05 Jun 2019 21:26:00 -0300
48+
49+iproute2 (4.18.0-1ubuntu2) cosmic; urgency=low
50+
51+ * d/p/1006-ubuntu-iprule-fix-output.patch
52+ Fix destination prefix output of "ip rule show" (LP: #1789431).
53+
54+ -- Stefan Bader <stefan.bader@canonical.com> Tue, 28 Aug 2018 16:38:47 +0200
55+
56+iproute2 (4.18.0-1ubuntu1) cosmic; urgency=medium
57+
58+ * Merge with Debian unstable. Remaining changes:
59+ - Add: Ubuntu Fan driver (V2/3/4) patches
60+ - debian/patches/1000-ubuntu-poc-fan-driver.patch
61+ Ubuntu FAN driver support V2 (LP 1439706)
62+ - debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
63+ Ubuntu FAN driver support V3 (LP 1470091)
64+ - debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
65+ Ubuntu FAN driver support VXLAN V4.
66+ - Drop: debian/patches/0001-Add-moo-feature.patch
67+ - set compat level 10 (no further changes needed)
68+ - Refresh: Ubuntu Fan driver (V2/3/4) patches
69+ Patches updated to apply against upstream 4.18 release.
70+ * Added changes:
71+ - debian/patches/1005-ubuntu-fix-testsuite-kenv.patch
72+ Fix testsuite failure if there are too many kernel config
73+ options.
74+ - Set debhelper dependency to >= 10 (no further changes needed)
75+
76+ -- Stefan Bader <stefan.bader@canonical.com> Wed, 22 Aug 2018 09:54:04 +0200
77+
78 iproute2 (4.18.0-1) experimental; urgency=medium
79
80 * Add Brazilian Portuguese translation for Debconf template. Thanks
81@@ -100,6 +165,22 @@ iproute2 (4.17.0-1) experimental; urgency=medium
82
83 -- Luca Boccassi <bluca@debian.org> Sat, 09 Jun 2018 17:45:11 +0100
84
85+iproute2 (4.16.0-4ubuntu1) cosmic; urgency=medium
86+
87+ * Merge with Debian unstable. Remaining changes:
88+ - debian/patches/1000-ubuntu-poc-fan-driver.patch [refreshed]
89+ Ubuntu FAN driver support V2 (LP 1439706)
90+ - debian/patches/1001-ubuntu-poc-fan-driver-v3.patch [refreshed]
91+ Ubuntu FAN driver support V3 (LP 1470091)
92+ - debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch [refreshed]
93+ Ubuntu FAN driver support VXLAN V4.
94+ - drop debian/patches/0001-Add-moo-feature.patch
95+ - set compat level 10 (no further changes needed)
96+ * Dropped (upstream)
97+ - d/p/netns-allow-negative-nsid: permit negative nsid using "auto".
98+
99+ -- Stefan Bader <stefan.bader@canonical.com> Mon, 30 Jul 2018 16:55:35 +0200
100+
101 iproute2 (4.16.0-4) unstable; urgency=medium
102
103 * Backport patch to fix regression in v6 automated parsing.
104@@ -152,6 +233,32 @@ iproute2 (4.15.0-3) unstable; urgency=medium
105
106 -- Luca Boccassi <bluca@debian.org> Mon, 12 Mar 2018 22:46:24 +0000
107
108+iproute2 (4.15.0-2ubuntu1) bionic; urgency=medium
109+
110+ * Merge with Debian unstable. Remaining changes:
111+ - debian/patches/1000-ubuntu-poc-fan-driver.patch
112+ Ubuntu FAN driver support V2 (LP 1439706)
113+ - debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
114+ Ubuntu FAN driver support V3 (LP 1470091)
115+ - debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
116+ Ubuntu FAN driver support VXLAN V4
117+ - drop debian/patches/0001-Add-moo-feature.patch
118+ - set compat level 10 (no further changes needed)
119+ - Add: d/p/netns-allow-negative-nsid: permit negative nsid using "auto"
120+ * Dropped changes (in Debian):
121+ - follow upstream dropping the old ip documentation (of 1999)
122+ - d/rules: consider DEB_*_GNU_TYPE from env
123+ - d/control: bump debhelper dependency and Standards-Version
124+ - drop 0003-fix-segfault-with-iptables-1.6.patch (upstream)
125+ - d/control: fix lintian priority-extra-is-replaced-by-priority-optional
126+ - d/t/control: d/t/testsuite.sh: add iproute2 testsuite as autopkgtest
127+ * Dropped changes (upstream):
128+ - d/p/fix-testsuite-broken-pipes.patch: avoid broken pipe errors in tests
129+ * Added changes:
130+ - Refresh: d/p/*ubuntu-poc-fan-driver*.patch to latest upstream
131+
132+ -- Stefan Bader <stefan.bader@canonical.com> Mon, 26 Feb 2018 16:21:30 +0100
133+
134 iproute2 (4.15.0-2) unstable; urgency=medium
135
136 * Update upstream git repository URL. (Closes: #888804)
137@@ -202,6 +309,58 @@ iproute2 (4.14.1-1) unstable; urgency=medium
138
139 -- Luca Boccassi <bluca@debian.org> Sat, 23 Dec 2017 14:55:10 +0100
140
141+iproute2 (4.14.1-0ubuntu3) bionic; urgency=medium
142+
143+ [ Christian Brauner ]
144+ * d/p/netns-allow-negative-nsid: permit negative nsid using "auto".
145+
146+ -- Robie Basak <robie.basak@ubuntu.com> Thu, 15 Feb 2018 15:03:56 +0000
147+
148+iproute2 (4.14.1-0ubuntu2) bionic; urgency=medium
149+
150+ * d/p/1002-ubuntu-poc-fan-driver-vxlan.patch: update to fix fan-map
151+ processing in vxlan_parse_opt in new version of iproute2.
152+
153+ -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Mon, 11 Dec 2017 12:13:50 +0100
154+
155+iproute2 (4.14.1-0ubuntu1) bionic; urgency=medium
156+
157+ [ Christian Ehrhardt ]
158+ * Merge from Debian unstable AND latest upstream (LP: #1726730)
159+ (LP: #1735032). Remaining changes:
160+ - debian/patches/1000-ubuntu-poc-fan-driver.patch
161+ Ubuntu FAN driver support V2 (LP 1439706)
162+ - debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
163+ Ubuntu FAN driver support V3 (LP 1470091)
164+ - debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
165+ Ubuntu FAN driver support VXLAN V4.
166+ * Dropped changes (upstream now)
167+ - Fix ip maddr show (LP 1732032):
168+ - d/p/1003-ip-maddr-fix-igmp-parsing.patch: fix igmp parsing when iface is
169+ long
170+ - d/p/1004-ip-maddr-avoid-uninitialized-data.patch: avoid accessing
171+ uninitialized data
172+ - d/p/1005-ip-maddr-fix-filtering-by-device.patch: fix filtering
173+ by device
174+ - d/p/1006-fix-undeclared-UINT16_MAX.patch: FTBFS: tc: include stdint.h
175+ explicitly for UINT16_MAX (LP 1735158)
176+ * Added changes
177+ - drop 0003-fix-segfault-with-iptables-1.6.patch (upstream)
178+ - update d/p/*ubuntu-poc-fan-driver*.patch to latest upstream
179+ - d/control: fix lintian priority-extra-is-replaced-by-priority-optional
180+ - d/t/control: d/t/testsuite.sh: add iproute2 testsuite as autopkgtest
181+ - d/p/fix-testsuite-broken-pipes.patch: avoid broken pipe errors in tests
182+
183+ [ Julian Wollrath <jwollrath@web.de> ]
184+ * Added changes based on suggested 4.14.1-0 changes for Debian in bug 879854
185+ - dropped d/p/0001-Add-moo-feature.patch (isn't needed)
186+ - follow upstream dropping the old ip documentation (of 1999)
187+ - d/rules: consider DEB_*_GNU_TYPE from env
188+ - d/control: bump debhelper dependency and Standards-Version
189+ - set compat level 10 (no further changes needed)
190+
191+ -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Thu, 30 Nov 2017 14:17:25 +0100
192+
193 iproute2 (4.13.0-1) unstable; urgency=medium
194
195 * Merge tag 'upstream/4.13.0' (Closes: #879854)
196@@ -232,6 +391,44 @@ iproute2 (4.9.0-2) unstable; urgency=medium
197
198 -- Andreas Henriksson <andreas@fatal.se> Thu, 14 Sep 2017 11:01:46 +0200
199
200+iproute2 (4.9.0-1ubuntu3) bionic; urgency=medium
201+
202+ * Fix ip maddr show (LP: #1732032):
203+ - d/p/1003-ip-maddr-fix-igmp-parsing.patch: fix igmp parsing when iface is
204+ long
205+ - d/p/1004-ip-maddr-avoid-uninitialized-data.patch: avoid accessing
206+ uninitialized data
207+ - d/p/1005-ip-maddr-fix-filtering-by-device.patch: fix filtering
208+ by device
209+ * d/p/1006-fix-undeclared-UINT16_MAX.patch: FTBFS: tc: include stdint.h
210+ explicitly for UINT16_MAX (LP: #1735158)
211+
212+ -- Andreas Hasenack <andreas@canonical.com> Wed, 29 Nov 2017 09:35:46 -0200
213+
214+iproute2 (4.9.0-1ubuntu2) artful; urgency=medium
215+
216+ * No-change rebuild against latest iptables
217+
218+ -- Jeremy Bicha <jbicha@ubuntu.com> Sun, 18 Jun 2017 23:59:24 -0400
219+
220+iproute2 (4.9.0-1ubuntu1) zesty; urgency=medium
221+
222+ * Merge from Debian unstable. Remaining changes:
223+ - debian/patches/1000-ubuntu-poc-fan-driver.patch
224+ Ubuntu FAN driver support V2 (LP: #1439706)
225+ - debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
226+ Ubuntu FAN driver support V3 (LP: #1470091)
227+ - debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
228+ Ubuntu FAN driver support VXLAN V4.
229+ * Drop the following, fixed in debian:
230+ - d/p/1003-fix-variable-in-libnetlink.patch (LP: #1522371)
231+ - Add TIPC support (LP: #1508225):
232+ - d/control: add libmnl-dev to BDs, to include TIPC support.
233+ - d/iproute2.install: add tipc binary.
234+ * Rebased FAN quilt patches on to latest upstream code.
235+
236+ -- Jon Grimm <jon.grimm@canonical.com> Fri, 10 Feb 2017 14:15:55 -0600
237+
238 iproute2 (4.9.0-1) unstable; urgency=medium
239
240 * New upstream release, tested by Julian Wollrath.
241@@ -296,6 +493,34 @@ iproute2 (4.6.0-1) unstable; urgency=medium
242
243 -- Andreas Henriksson <andreas@fatal.se> Sat, 09 Jul 2016 13:17:22 +0200
244
245+iproute2 (4.3.0-1ubuntu3) xenial; urgency=medium
246+
247+ * Add d/p/1003-fix-variable-in-libnetlink.patch (LP: #1522371), fix a variable name
248+ collision issuing a segfault; from upstream.
249+ * Add TIPC support (LP: #1508225):
250+ - d/control: add libmnl-dev to BDs, to include TIPC support.
251+ - d/iproute2.install: add tipc binary.
252+
253+ -- Pierre-André MOREY <pierre-andre.morey@canonical.com> Tue, 05 Apr 2016 09:43:44 +0200
254+
255+iproute2 (4.3.0-1ubuntu2) xenial; urgency=medium
256+
257+ * No-change rebuild for new iptables.
258+
259+ -- Matthias Klose <doko@ubuntu.com> Sun, 21 Feb 2016 20:00:55 +0100
260+
261+iproute2 (4.3.0-1ubuntu1) xenial; urgency=low
262+
263+ * Merge from Debian unstable. Remaining changes:
264+ - debian/patches/1000-ubuntu-poc-fan-driver.patch
265+ Ubuntu FAN driver support V2 (LP: #1439706)
266+ - debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
267+ Ubuntu FAN driver support V3 (LP: #1470091)
268+ - debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
269+ Ubuntu FAN driver support VXLAN V4.
270+
271+ -- Andy Whitcroft <apw@ubuntu.com> Tue, 22 Dec 2015 09:32:32 +0000
272+
273 iproute2 (4.3.0-1) unstable; urgency=medium
274
275 * Add debian/gbp.conf
276@@ -307,6 +532,23 @@ iproute2 (4.3.0-1) unstable; urgency=medium
277
278 -- Andreas Henriksson <andreas@fatal.se> Thu, 26 Nov 2015 18:34:59 +0100
279
280+iproute2 (4.1.1-1ubuntu2) xenial; urgency=low
281+
282+ * debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
283+ Ubuntu FAN drvier support VXLAN V4.
284+
285+ -- Andy Whitcroft <apw@ubuntu.com> Tue, 17 Nov 2015 13:29:03 +0000
286+
287+iproute2 (4.1.1-1ubuntu1) wily; urgency=medium
288+
289+ * Merge with Debian, remaining changes (LP: #1448800):
290+ - debian/patches/1000-ubuntu-poc-fan-driver.patch
291+ Ubuntu FAN driver support V2 (LP: #1439706)
292+ - debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
293+ Ubuntu FAN driver support V3 (LP: #1470091)
294+
295+ -- Andy Whitcroft <apw@ubuntu.com> Wed, 19 Aug 2015 09:59:35 +0100
296+
297 iproute2 (4.1.1-1) unstable; urgency=medium
298
299 * [be9f298] Imported Upstream version 4.1.1
300@@ -323,6 +565,18 @@ iproute2 (4.0.0-1) unstable; urgency=medium
301
302 -- Alexander Wirt <formorer@debian.org> Wed, 24 Jun 2015 10:01:45 +0200
303
304+iproute2 (3.16.0-2ubuntu2) wily; urgency=low
305+
306+ * Update Ubuntu FAN support to V3. (LP: #1470091)
307+
308+ -- Andy Whitcroft <apw@canonical.com> Mon, 29 Jun 2015 16:59:24 +0100
309+
310+iproute2 (3.16.0-2ubuntu1) vivid; urgency=low
311+
312+ * Add Ubuntu FAN support. (LP: #1439706)
313+
314+ -- Andy Whitcroft <apw@ubuntu.com> Tue, 24 Feb 2015 14:45:19 +0000
315+
316 iproute2 (3.16.0-2) unstable; urgency=medium
317
318 * Cherry-pick upstream commit f1b66ff8
319@@ -1442,6 +1696,3 @@ iproute (961225-1) unstable; urgency=low
320
321 -- Tom Lees <tom@lpsg.demon.co.uk> Mon, 30 Dec 1996 11:12:23 +0000
322
323-Local variables:
324-mode: debian-changelog
325-End:
326diff --git a/debian/control b/debian/control
327index c042a1b..e24afa8 100644
328--- a/debian/control
329+++ b/debian/control
330@@ -1,7 +1,8 @@
331 Source: iproute2
332 Section: net
333 Priority: optional
334-Maintainer: Alexander Wirt <formorer@debian.org>
335+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
336+XSBC-Original-Maintainer: Alexander Wirt <formorer@debian.org>
337 Uploaders: Luca Boccassi <bluca@debian.org>
338 Homepage: https://wiki.linuxfoundation.org/networking/iproute2
339 Vcs-Browser: https://salsa.debian.org/debian/iproute2.git
340diff --git a/debian/patches/0001-Add-moo-feature.patch b/debian/patches/0001-Add-moo-feature.patch
341deleted file mode 100644
342index b046590..0000000
343--- a/debian/patches/0001-Add-moo-feature.patch
344+++ /dev/null
345@@ -1,39 +0,0 @@
346-From: Alexander Wirt <formorer@debian.org>
347-Date: Mon, 10 Jun 2013 11:47:00 +0200
348-Subject: Add moo feature
349-
350----
351- ip/ip.c | 15 +++++++++++++++
352- 1 file changed, 15 insertions(+)
353-
354---- a/ip/ip.c
355-+++ b/ip/ip.c
356-@@ -69,6 +69,20 @@ static int do_help(int argc, char **argv
357- return 0;
358- }
359-
360-+static int do_moo(int argc, char **argv)
361-+{
362-+
363-+fprintf(stderr,
364-+"\n"
365-+" _ __ ___ ___ ___\n"
366-+"| '_ ` _ \\ / _ \\ / _ \\\n"
367-+"| | | | | | (_) | (_) |\n"
368-+"|_| |_| |_|\\___/ \\___/\n"
369-+"\n\n"
370-+"P.S. no real cows were harmed for this moo\n");
371-+ exit(1);
372-+}
373-+
374- static const struct cmd {
375- const char *cmd;
376- int (*func)(int argc, char **argv);
377-@@ -103,6 +117,7 @@ static const struct cmd {
378- { "vrf", do_ipvrf},
379- { "sr", do_seg6 },
380- { "help", do_help },
381-+ { "moo", do_moo },
382- { 0 }
383- };
384-
385diff --git a/debian/patches/1000-ubuntu-poc-fan-driver.patch b/debian/patches/1000-ubuntu-poc-fan-driver.patch
386new file mode 100644
387index 0000000..f3aaec2
388--- /dev/null
389+++ b/debian/patches/1000-ubuntu-poc-fan-driver.patch
390@@ -0,0 +1,74 @@
391+Description: POC fan driver support
392+ POC Fan driver support
393+Author: Jay Vosburgh <jay.vosburgh@canonical.com>
394+
395+Index: iproute2/ip/link_iptnl.c
396+===================================================================
397+--- iproute2.orig/ip/link_iptnl.c
398++++ iproute2/ip/link_iptnl.c
399+@@ -91,6 +91,7 @@ static int iptunnel_parse_opt(struct lin
400+ struct rtattr *iptuninfo[IFLA_IPTUN_MAX + 1];
401+ int len;
402+ inet_prefix saddr, daddr, ip6rdprefix, ip6rdrelayprefix;
403++ inet_prefix underlay;
404+ __u8 pmtudisc = 1;
405+ __u8 tos = 0;
406+ __u16 iflags = 0;
407+@@ -106,6 +107,7 @@ static int iptunnel_parse_opt(struct lin
408+
409+ inet_prefix_reset(&saddr);
410+ inet_prefix_reset(&daddr);
411++ inet_prefix_reset(&underlay);
412+
413+ inet_prefix_reset(&ip6rdprefix);
414+ inet_prefix_reset(&ip6rdrelayprefix);
415+@@ -219,6 +221,9 @@ get_failed:
416+ } else if (strcmp(*argv, "remote") == 0) {
417+ NEXT_ARG();
418+ get_addr(&daddr, *argv, AF_INET);
419++ } else if (strcmp(*argv, "underlay") == 0) {
420++ NEXT_ARG();
421++ get_addr(&underlay, *argv, AF_INET);
422+ } else if (strcmp(*argv, "local") == 0) {
423+ NEXT_ARG();
424+ get_addr(&saddr, *argv, AF_INET);
425+@@ -360,6 +365,10 @@ get_failed:
426+ }
427+ }
428+
429++ if (is_addrtype_inet(&underlay))
430++ addattr_l(n, 1024, IFLA_IPTUN_FAN_UNDERLAY,
431++ underlay.data, underlay.bytelen);
432++
433+ return 0;
434+ }
435+
436+@@ -398,6 +407,14 @@ static void iptunnel_print_opt(struct li
437+ tnl_print_endpoint("remote", tb[IFLA_IPTUN_REMOTE], AF_INET);
438+ tnl_print_endpoint("local", tb[IFLA_IPTUN_LOCAL], AF_INET);
439+
440++ if (tb[IFLA_IPTUN_FAN_UNDERLAY]) {
441++ inet_prefix addr;
442++
443++ if(!get_addr_rta(&addr, tb[IFLA_IPTUN_FAN_UNDERLAY], AF_INET))
444++ print_string(PRINT_ANY, "underlay", "underlay %s ",
445++ format_host(AF_INET, addr.bytelen, addr.data));
446++ }
447++
448+ if (tb[IFLA_IPTUN_LINK]) {
449+ __u32 link = rta_getattr_u32(tb[IFLA_IPTUN_LINK]);
450+
451+Index: iproute2/include/uapi/linux/if_tunnel.h
452+===================================================================
453+--- iproute2.orig/include/uapi/linux/if_tunnel.h
454++++ iproute2/include/uapi/linux/if_tunnel.h
455+@@ -77,6 +77,9 @@ enum {
456+ IFLA_IPTUN_ENCAP_DPORT,
457+ IFLA_IPTUN_COLLECT_METADATA,
458+ IFLA_IPTUN_FWMARK,
459++
460++ IFLA_IPTUN_FAN_UNDERLAY = 32,
461++
462+ __IFLA_IPTUN_MAX,
463+ };
464+ #define IFLA_IPTUN_MAX (__IFLA_IPTUN_MAX - 1)
465diff --git a/debian/patches/1001-ubuntu-poc-fan-driver-v3.patch b/debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
466new file mode 100644
467index 0000000..d12b88e
468--- /dev/null
469+++ b/debian/patches/1001-ubuntu-poc-fan-driver-v3.patch
470@@ -0,0 +1,134 @@
471+Description: Fan driver support v3
472+ Fan driver support v3
473+Author: Jay Vosburgh <jay.vosburgh@canonical.com>
474+Index: iproute2/include/uapi/linux/if_tunnel.h
475+===================================================================
476+--- iproute2.orig/include/uapi/linux/if_tunnel.h
477++++ iproute2/include/uapi/linux/if_tunnel.h
478+@@ -79,6 +79,7 @@ enum {
479+ IFLA_IPTUN_FWMARK,
480+
481+ IFLA_IPTUN_FAN_UNDERLAY = 32,
482++ IFLA_IPTUN_FAN_MAP = 33,
483+
484+ __IFLA_IPTUN_MAX,
485+ };
486+@@ -164,6 +165,21 @@ enum {
487+
488+ #define IFLA_VTI_MAX (__IFLA_VTI_MAX - 1)
489+
490++enum {
491++ IFLA_FAN_UNSPEC,
492++ IFLA_FAN_MAPPING,
493++ __IFLA_FAN_MAX,
494++};
495++
496++#define IFLA_FAN_MAX (__IFLA_FAN_MAX - 1)
497++
498++struct ip_tunnel_fan_map {
499++ __be32 underlay;
500++ __be32 overlay;
501++ __u16 underlay_prefix;
502++ __u16 overlay_prefix;
503++};
504++
505+ #define TUNNEL_CSUM __cpu_to_be16(0x01)
506+ #define TUNNEL_ROUTING __cpu_to_be16(0x02)
507+ #define TUNNEL_KEY __cpu_to_be16(0x04)
508+Index: iproute2/ip/link_iptnl.c
509+===================================================================
510+--- iproute2.orig/ip/link_iptnl.c
511++++ iproute2/ip/link_iptnl.c
512+@@ -70,6 +70,42 @@ static void iptunnel_print_help(struct l
513+ " MARK := { 0x0..0xffffffff }\n"
514+ );
515+ }
516++static int fan_parse_map(int *argcp, char ***argvp, struct nlmsghdr *n)
517++{
518++ inet_prefix underlay, overlay;
519++ struct ip_tunnel_fan_map map;
520++ struct rtattr *nest;
521++ char **argv = *argvp;
522++ int argc = *argcp;
523++
524++ nest = addattr_nest(n, 1024, IFLA_IPTUN_FAN_MAP);
525++ while (argc > 0) {
526++ char *colon = strchr(*argv, ':');
527++
528++ if (!colon)
529++ break;
530++ *colon = '\0';
531++
532++ if (get_prefix(&overlay, *argv, AF_INET))
533++ invarg("invalid fan-map overlay", *argv);
534++ if (get_prefix(&underlay, colon + 1, AF_INET))
535++ invarg("invalid fan-map underlay", colon + 1);
536++
537++ memcpy(&map.underlay, underlay.data, 4);
538++ map.underlay_prefix = underlay.bitlen;
539++ memcpy(&map.overlay, overlay.data, 4);
540++ map.overlay_prefix = overlay.bitlen;
541++
542++ argc--, argv++;
543++
544++ addattr_l(n, 1024, IFLA_FAN_MAPPING, &map, sizeof(map));
545++ }
546++ addattr_nest_end(n, nest);
547++
548++ *argcp = argc;
549++ *argvp = argv;
550++ return 0;
551++}
552+
553+ static int iptunnel_parse_opt(struct link_util *lu, int argc, char **argv,
554+ struct nlmsghdr *n)
555+@@ -224,6 +260,10 @@ get_failed:
556+ } else if (strcmp(*argv, "underlay") == 0) {
557+ NEXT_ARG();
558+ get_addr(&underlay, *argv, AF_INET);
559++ } else if (strcmp(*argv, "fan-map") == 0) {
560++ NEXT_ARG();
561++ if (fan_parse_map(&argc, &argv, n))
562++ invarg("invalid fan-map", *argv);
563+ } else if (strcmp(*argv, "local") == 0) {
564+ NEXT_ARG();
565+ get_addr(&saddr, *argv, AF_INET);
566+@@ -372,6 +412,28 @@ get_failed:
567+ return 0;
568+ }
569+
570++static void fan_print_map(FILE *f, struct rtattr *attr)
571++{
572++ char b1[INET_ADDRSTRLEN], b2[INET_ADDRSTRLEN];
573++ struct ip_tunnel_fan_map *m;
574++ struct rtattr *i;
575++ int rem;
576++ int p;
577++
578++ fprintf(f, "fan-map ");
579++
580++ rem = RTA_PAYLOAD(attr);
581++ for (i = RTA_DATA(attr); RTA_OK(i, rem); i = RTA_NEXT(i, rem)) {
582++ p = RTA_PAYLOAD(i);
583++ m = RTA_DATA(i);
584++ fprintf(f, "%s/%d:%s/%d ",
585++ rt_addr_n2a_r(AF_INET, p, &m->overlay, b1, INET_ADDRSTRLEN),
586++ m->overlay_prefix,
587++ rt_addr_n2a_r(AF_INET, p, &m->underlay, b2, INET_ADDRSTRLEN),
588++ m->underlay_prefix);
589++ }
590++}
591++
592+ static void iptunnel_print_opt(struct link_util *lu, FILE *f, struct rtattr *tb[])
593+ {
594+ char s2[64];
595+@@ -415,6 +477,9 @@ static void iptunnel_print_opt(struct li
596+ format_host(AF_INET, addr.bytelen, addr.data));
597+ }
598+
599++ if (tb[IFLA_IPTUN_FAN_MAP])
600++ fan_print_map(f, tb[IFLA_IPTUN_FAN_MAP]);
601++
602+ if (tb[IFLA_IPTUN_LINK]) {
603+ __u32 link = rta_getattr_u32(tb[IFLA_IPTUN_LINK]);
604+
605diff --git a/debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch b/debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
606new file mode 100644
607index 0000000..b5b3b0c
608--- /dev/null
609+++ b/debian/patches/1002-ubuntu-poc-fan-driver-vxlan.patch
610@@ -0,0 +1,178 @@
611+Description: Fan driver support VXLAN (p4)
612+ Fan driver setup support for vxlan interfaces.
613+
614+Index: iproute2/include/uapi/linux/if_tunnel.h
615+===================================================================
616+--- iproute2.orig/include/uapi/linux/if_tunnel.h
617++++ iproute2/include/uapi/linux/if_tunnel.h
618+@@ -173,7 +173,7 @@ enum {
619+
620+ #define IFLA_FAN_MAX (__IFLA_FAN_MAX - 1)
621+
622+-struct ip_tunnel_fan_map {
623++struct ifla_fan_map {
624+ __be32 underlay;
625+ __be32 overlay;
626+ __u16 underlay_prefix;
627+Index: iproute2/ip/iplink_vxlan.c
628+===================================================================
629+--- iproute2.orig/ip/iplink_vxlan.c
630++++ iproute2/ip/iplink_vxlan.c
631+@@ -15,7 +15,10 @@
632+ #include <net/if.h>
633+ #include <linux/ip.h>
634+ #include <linux/if_link.h>
635++#include <linux/types.h>
636+ #include <arpa/inet.h>
637++#include <linux/in6.h>
638++#include <linux/if_tunnel.h>
639+
640+ #include "rt_names.h"
641+ #include "utils.h"
642+@@ -73,6 +76,45 @@ static void check_duparg(__u64 *attrs, i
643+ duparg2(key, argv);
644+ }
645+
646++static int fan_parse_map(int *argcp, char ***argvp, struct nlmsghdr *n)
647++{
648++ inet_prefix underlay, overlay;
649++ struct ifla_fan_map map;
650++ struct rtattr *nest;
651++ char **argv = *argvp;
652++ int argc = *argcp;
653++
654++ nest = addattr_nest(n, 1024, IFLA_VXLAN_FAN_MAP);
655++ while (argc > 0) {
656++ char *colon = strchr(*argv, ':');
657++
658++ if (!colon) {
659++ PREV_ARG();
660++ break;
661++ }
662++ *colon = '\0';
663++
664++ if (get_prefix(&overlay, *argv, AF_INET))
665++ invarg("invalid fan-map overlay", *argv);
666++ if (get_prefix(&underlay, colon + 1, AF_INET))
667++ invarg("invalid fan-map underlay", colon + 1);
668++
669++ memcpy(&map.underlay, underlay.data, 4);
670++ map.underlay_prefix = underlay.bitlen;
671++ memcpy(&map.overlay, overlay.data, 4);
672++ map.overlay_prefix = overlay.bitlen;
673++
674++ argc--, argv++;
675++
676++ addattr_l(n, 1024, IFLA_FAN_MAPPING, &map, sizeof(map));
677++ }
678++ addattr_nest_end(n, nest);
679++
680++ *argcp = argc;
681++ *argvp = argv;
682++ return 0;
683++}
684++
685+ static int vxlan_parse_opt(struct link_util *lu, int argc, char **argv,
686+ struct nlmsghdr *n)
687+ {
688+@@ -330,6 +372,11 @@ static int vxlan_parse_opt(struct link_u
689+ } else if (!matches(*argv, "gpe")) {
690+ check_duparg(&attrs, IFLA_VXLAN_GPE, *argv, *argv);
691+ addattr_l(n, 1024, IFLA_VXLAN_GPE, NULL, 0);
692++ } else if (!matches(*argv, "fan-map")) {
693++ check_duparg(&attrs, IFLA_VXLAN_FAN_MAP, *argv, *argv);
694++ NEXT_ARG();
695++ if (fan_parse_map(&argc, &argv, n))
696++ invarg("invalid fan-map", *argv);
697+ } else if (matches(*argv, "help") == 0) {
698+ explain();
699+ return -1;
700+@@ -406,6 +453,28 @@ static int vxlan_parse_opt(struct link_u
701+ return 0;
702+ }
703+
704++static void fan_print_map(FILE *f, struct rtattr *attr)
705++{
706++ char b1[INET_ADDRSTRLEN], b2[INET_ADDRSTRLEN];
707++ struct ifla_fan_map *m;
708++ struct rtattr *i;
709++ int rem;
710++ int p;
711++
712++ fprintf(f, "fan-map ");
713++
714++ rem = RTA_PAYLOAD(attr);
715++ for (i = RTA_DATA(attr); RTA_OK(i, rem); i = RTA_NEXT(i, rem)) {
716++ p = RTA_PAYLOAD(i);
717++ m = RTA_DATA(i);
718++ fprintf(f, "%s/%d:%s/%d ",
719++ rt_addr_n2a_r(AF_INET, p, &m->overlay, b1, INET_ADDRSTRLEN),
720++ m->overlay_prefix,
721++ rt_addr_n2a_r(AF_INET, p, &m->underlay, b2, INET_ADDRSTRLEN),
722++ m->underlay_prefix);
723++ }
724++}
725++
726+ static void vxlan_print_opt(struct link_util *lu, FILE *f, struct rtattr *tb[])
727+ {
728+ __u32 vni;
729+@@ -466,6 +535,9 @@ static void vxlan_print_opt(struct link_
730+ }
731+ }
732+
733++ if (tb[IFLA_VXLAN_FAN_MAP])
734++ fan_print_map(f, tb[IFLA_VXLAN_FAN_MAP]);
735++
736+ if (tb[IFLA_VXLAN_LOCAL]) {
737+ __be32 addr = rta_getattr_u32(tb[IFLA_VXLAN_LOCAL]);
738+
739+Index: iproute2/ip/link_iptnl.c
740+===================================================================
741+--- iproute2.orig/ip/link_iptnl.c
742++++ iproute2/ip/link_iptnl.c
743+@@ -70,10 +70,11 @@ static void iptunnel_print_help(struct l
744+ " MARK := { 0x0..0xffffffff }\n"
745+ );
746+ }
747++
748+ static int fan_parse_map(int *argcp, char ***argvp, struct nlmsghdr *n)
749+ {
750+ inet_prefix underlay, overlay;
751+- struct ip_tunnel_fan_map map;
752++ struct ifla_fan_map map;
753+ struct rtattr *nest;
754+ char **argv = *argvp;
755+ int argc = *argcp;
756+@@ -82,8 +83,10 @@ static int fan_parse_map(int *argcp, cha
757+ while (argc > 0) {
758+ char *colon = strchr(*argv, ':');
759+
760+- if (!colon)
761++ if (!colon) {
762++ PREV_ARG();
763+ break;
764++ }
765+ *colon = '\0';
766+
767+ if (get_prefix(&overlay, *argv, AF_INET))
768+@@ -415,7 +418,7 @@ get_failed:
769+ static void fan_print_map(FILE *f, struct rtattr *attr)
770+ {
771+ char b1[INET_ADDRSTRLEN], b2[INET_ADDRSTRLEN];
772+- struct ip_tunnel_fan_map *m;
773++ struct ifla_fan_map *m;
774+ struct rtattr *i;
775+ int rem;
776+ int p;
777+Index: iproute2/include/uapi/linux/if_link.h
778+===================================================================
779+--- iproute2.orig/include/uapi/linux/if_link.h
780++++ iproute2/include/uapi/linux/if_link.h
781+@@ -533,6 +533,7 @@ enum {
782+ IFLA_VXLAN_GPE,
783+ IFLA_VXLAN_TTL_INHERIT,
784+ IFLA_VXLAN_DF,
785++ IFLA_VXLAN_FAN_MAP = 33,
786+ __IFLA_VXLAN_MAX
787+ };
788+ #define IFLA_VXLAN_MAX (__IFLA_VXLAN_MAX - 1)
789diff --git a/debian/patches/1005-lib-suppress-error-msg-when-filling-the-cache.patch b/debian/patches/1005-lib-suppress-error-msg-when-filling-the-cache.patch
790new file mode 100644
791index 0000000..acb2e59
792--- /dev/null
793+++ b/debian/patches/1005-lib-suppress-error-msg-when-filling-the-cache.patch
794@@ -0,0 +1,40 @@
795+Description: lib: suppress error msg when filling the cache
796+ Before the patch:
797+ $ ip netns add foo
798+ $ ip link add name veth1 address 2a:a5:5c:b9:52:89 type veth peer \
799+ name veth2 address 2a:a5:5c:b9:53:90 netns foo
800+ RTNETLINK answers: No such device
801+ RTNETLINK answers: No such device
802+ .
803+ But the command was successful. This may break script. Let's remove those
804+ error messages.
805+ .
806+ Fixes: 55870dfe7f8b ("Improve batch and dump times by caching link lookups")
807+ Reported-by: Philippe Guibert <philippe.guibert@6wind.com>
808+ Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
809+ Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
810+ (cherry picked from 757837230a654d39623d0b90882b695a2facd107)
811+ Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
812+Author: Nicolas Dichtel <nicolas.dichtel@6wind.com>
813+Origin: upstream, https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=757837230a654d39623d0b90882b695a2facd107
814+Last-Update: 2019-07-10
815+---
816+ lib/ll_map.c | 2 +-
817+ 1 file changed, 1 insertion(+), 1 deletion(-)
818+
819+diff --git a/lib/ll_map.c b/lib/ll_map.c
820+index 2d7b65dc..e0ed54bf 100644
821+--- a/lib/ll_map.c
822++++ b/lib/ll_map.c
823+@@ -177,7 +177,7 @@ static int ll_link_get(const char *name, int index)
824+ addattr_l(&req.n, sizeof(req), IFLA_IFNAME, name,
825+ strlen(name) + 1);
826+
827+- if (rtnl_talk(&rth, &req.n, &answer) < 0)
828++ if (rtnl_talk_suppress_rtnl_errmsg(&rth, &req.n, &answer) < 0)
829+ goto out;
830+
831+ /* add entry to cache */
832+--
833+2.17.1
834+
835diff --git a/debian/patches/series b/debian/patches/series
836index 7616c7e..9dec7b5 100644
837--- a/debian/patches/series
838+++ b/debian/patches/series
839@@ -1,2 +1,9 @@
840-0001-Add-moo-feature.patch
841 0004-sync-iptables-header.patch
842+
843+
844+
845+1000-ubuntu-poc-fan-driver.patch
846+1001-ubuntu-poc-fan-driver-v3.patch
847+1002-ubuntu-poc-fan-driver-vxlan.patch
848+
849+1005-lib-suppress-error-msg-when-filling-the-cache.patch

Subscribers

People subscribed via source and target branches