diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2010-02-13 22:25:51 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-16 18:55:17 -0500 |
commit | 02291680ffba92e5b5865bc0c5e7d1f3056b80ec (patch) | |
tree | fde485582de465b211cc907710a23ef8ca84408e /net/ipv4 | |
parent | 9e3f8063a72a946f2ba29068b34190436573dffe (diff) |
net ipv4: Decouple ipv4 interface parameters from binary sysctl numbers
Stop using the binary sysctl enumeartion in sysctl.h as an index into
a per interface array. This leads to unnecessary binary sysctl number
allocation, and a fragility in data structure and implementation
because of unnecessary coupling.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
-rw-r--r-- | net/ipv4/devinet.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c index cd71a390839..b1eddee9bf9 100644 --- a/net/ipv4/devinet.c +++ b/net/ipv4/devinet.c | |||
@@ -64,20 +64,20 @@ | |||
64 | 64 | ||
65 | static struct ipv4_devconf ipv4_devconf = { | 65 | static struct ipv4_devconf ipv4_devconf = { |
66 | .data = { | 66 | .data = { |
67 | [NET_IPV4_CONF_ACCEPT_REDIRECTS - 1] = 1, | 67 | [IPV4_DEVCONF_ACCEPT_REDIRECTS - 1] = 1, |
68 | [NET_IPV4_CONF_SEND_REDIRECTS - 1] = 1, | 68 | [IPV4_DEVCONF_SEND_REDIRECTS - 1] = 1, |
69 | [NET_IPV4_CONF_SECURE_REDIRECTS - 1] = 1, | 69 | [IPV4_DEVCONF_SECURE_REDIRECTS - 1] = 1, |
70 | [NET_IPV4_CONF_SHARED_MEDIA - 1] = 1, | 70 | [IPV4_DEVCONF_SHARED_MEDIA - 1] = 1, |
71 | }, | 71 | }, |
72 | }; | 72 | }; |
73 | 73 | ||
74 | static struct ipv4_devconf ipv4_devconf_dflt = { | 74 | static struct ipv4_devconf ipv4_devconf_dflt = { |
75 | .data = { | 75 | .data = { |
76 | [NET_IPV4_CONF_ACCEPT_REDIRECTS - 1] = 1, | 76 | [IPV4_DEVCONF_ACCEPT_REDIRECTS - 1] = 1, |
77 | [NET_IPV4_CONF_SEND_REDIRECTS - 1] = 1, | 77 | [IPV4_DEVCONF_SEND_REDIRECTS - 1] = 1, |
78 | [NET_IPV4_CONF_SECURE_REDIRECTS - 1] = 1, | 78 | [IPV4_DEVCONF_SECURE_REDIRECTS - 1] = 1, |
79 | [NET_IPV4_CONF_SHARED_MEDIA - 1] = 1, | 79 | [IPV4_DEVCONF_SHARED_MEDIA - 1] = 1, |
80 | [NET_IPV4_CONF_ACCEPT_SOURCE_ROUTE - 1] = 1, | 80 | [IPV4_DEVCONF_ACCEPT_SOURCE_ROUTE - 1] = 1, |
81 | }, | 81 | }, |
82 | }; | 82 | }; |
83 | 83 | ||
@@ -1360,7 +1360,7 @@ int ipv4_doint_and_flush(ctl_table *ctl, int write, | |||
1360 | { \ | 1360 | { \ |
1361 | .procname = name, \ | 1361 | .procname = name, \ |
1362 | .data = ipv4_devconf.data + \ | 1362 | .data = ipv4_devconf.data + \ |
1363 | NET_IPV4_CONF_ ## attr - 1, \ | 1363 | IPV4_DEVCONF_ ## attr - 1, \ |
1364 | .maxlen = sizeof(int), \ | 1364 | .maxlen = sizeof(int), \ |
1365 | .mode = mval, \ | 1365 | .mode = mval, \ |
1366 | .proc_handler = proc, \ | 1366 | .proc_handler = proc, \ |
@@ -1381,7 +1381,7 @@ int ipv4_doint_and_flush(ctl_table *ctl, int write, | |||
1381 | 1381 | ||
1382 | static struct devinet_sysctl_table { | 1382 | static struct devinet_sysctl_table { |
1383 | struct ctl_table_header *sysctl_header; | 1383 | struct ctl_table_header *sysctl_header; |
1384 | struct ctl_table devinet_vars[__NET_IPV4_CONF_MAX]; | 1384 | struct ctl_table devinet_vars[__IPV4_DEVCONF_MAX]; |
1385 | char *dev_name; | 1385 | char *dev_name; |
1386 | } devinet_sysctl = { | 1386 | } devinet_sysctl = { |
1387 | .devinet_vars = { | 1387 | .devinet_vars = { |
@@ -1503,7 +1503,7 @@ static struct ctl_table ctl_forward_entry[] = { | |||
1503 | { | 1503 | { |
1504 | .procname = "ip_forward", | 1504 | .procname = "ip_forward", |
1505 | .data = &ipv4_devconf.data[ | 1505 | .data = &ipv4_devconf.data[ |
1506 | NET_IPV4_CONF_FORWARDING - 1], | 1506 | IPV4_DEVCONF_FORWARDING - 1], |
1507 | .maxlen = sizeof(int), | 1507 | .maxlen = sizeof(int), |
1508 | .mode = 0644, | 1508 | .mode = 0644, |
1509 | .proc_handler = devinet_sysctl_forward, | 1509 | .proc_handler = devinet_sysctl_forward, |
@@ -1547,7 +1547,7 @@ static __net_init int devinet_init_net(struct net *net) | |||
1547 | if (tbl == NULL) | 1547 | if (tbl == NULL) |
1548 | goto err_alloc_ctl; | 1548 | goto err_alloc_ctl; |
1549 | 1549 | ||
1550 | tbl[0].data = &all->data[NET_IPV4_CONF_FORWARDING - 1]; | 1550 | tbl[0].data = &all->data[IPV4_DEVCONF_FORWARDING - 1]; |
1551 | tbl[0].extra1 = all; | 1551 | tbl[0].extra1 = all; |
1552 | tbl[0].extra2 = net; | 1552 | tbl[0].extra2 = net; |
1553 | #endif | 1553 | #endif |