Page MenuHomeFreeBSD
Differential D8051

Remove ifa_list, use ifa_link instead
ClosedPublic
Actions

Authored by kevlo on Sep 27 2016, 1:14 PM.
Tags
None
Referenced Files
F136638779: D8051.diff
Tue, Nov 18, 4:57 PM
F136634052: D8051.id.diff
Tue, Nov 18, 4:26 PM
F136593689: D8051.id20741.diff
Tue, Nov 18, 12:17 PM
F136593493: D8051.id20768.diff
Tue, Nov 18, 12:15 PM
F136582422: D8051.diff
Tue, Nov 18, 11:10 AM
F136579001: D8051.id20768.diff
Tue, Nov 18, 10:51 AM
F136576303: D8051.id20741.diff
Tue, Nov 18, 10:31 AM
F136576289: D8051.id.diff
Tue, Nov 18, 10:30 AM
View All 83 Files
Subscribers
None

Details

Reviewers
rrs
tuexen
Commits
rS306407: Remove ifa_list, use ifa_link (structure field) instead.
Summary

There are few source code (only pf and sctp) that use ifa_list.
This patch uses ifa_link (structure field) instead of ifa_list
(macro alias for it) and removes ifa_list.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kevlo updated this revision to Diff 20741.Sep 27 2016, 1:14 PM
kevlo retitled this revision from to Remove ifa_list, use ifa_link instead.
kevlo updated this object.
kevlo edited the test plan for this revision. (Show Details)
kevlo added reviewers: rrs, tuexen.
kevlo added a comment.Sep 27 2016, 1:17 PM

Forgot to mention, prefer if_addrhead (FreeBSD) to if_addrlist (BSD compat)
naming for the interface address list while I'm here.

tuexen accepted this revision.Sep 27 2016, 3:11 PM
tuexen edited edge metadata.

Drop me a note when you commit it and indicate if you will MFC it.

This revision is now accepted and ready to land.Sep 27 2016, 3:11 PM
Closed by commit rS306407: Remove ifa_list, use ifa_link (structure field) instead. (authored by kevlo). · Explain WhySep 28 2016, 1:29 PM
This revision was automatically updated to reflect the committed changes.
kevlo added a comment.Sep 28 2016, 1:31 PM
In D8051#166553, @tuexen wrote:

Drop me a note when you commit it and indicate if you will MFC it.

Committed in r306407, thanks.

Revision Contents
Changeset List

PathSize
sys/
net/
3 lines
netinet/
4 lines
netpfil/
pf/
2 lines

Diff 20741

View Options

sys/net/if_var.h

Loading...
View Options

sys/netinet/sctp_bsd_addr.c

Loading...
View Options

sys/netpfil/pf/pf_if.c

Loading...