linux-stable.git/net/ipv4/ipconfig.c, branch linux-4.3.y

net: ipv4: increase dhcp inter device timeout

2015-08-12T23:40:22+00:00

When a system has multiple ethernet devices and during DHCP
request (for using NFS), the system waits only for HZ/2 which is
500mS before switching to another interface for DHCP.

There are some routers (Ex: Trendnet routers) which responds to
DHCP request at about 560mS. When the system has only one
ethernet interface there is no issue as the timeout is 2S and the
dev xid doesn't changes and only retries.

But when the system has multiple Ethernet like DRA74x with CPSW
in dual EMAC mode, the DHCP response is dropped as the dev xid
changes while shifting to the next device. So changing inter
device timeout to HZ (which is 1S).

Signed-off-by: Mugunthan V N 
Signed-off-by: David S. Miller

ipv4: coding style: comparison for equality with NULL

2015-04-03T16:11:15+00:00

The ipv4 code uses a mixture of coding styles. In some instances check
for NULL pointer is done as x == NULL and sometimes as !x. !x is
preferred according to checkpatch and this patch makes the code
consistent by adopting the latter form.

No changes detected by objdiff.

Signed-off-by: Ian Morris 
Signed-off-by: David S. Miller

net: ipv4: handle DSA enabled master network devices

2015-01-19T20:45:10+00:00

The logic to configure a network interface for kernel IP
auto-configuration is very simplistic, and does not handle the case
where a device is stacked onto another such as with DSA. This causes the
kernel not to open and configure the master network device in a DSA
switch tree, and therefore slave network devices using this master
network devices as conduit device cannot be open.

This restriction comes from a check in net/dsa/slave.c, which is
basically checking the master netdev flags for IFF_UP and returns
-ENETDOWN if it is not the case.

Automatically bringing-up DSA master network devices allows DSA slave
network devices to be used as valid interfaces for e.g: NFS root booting
by allowing kernel IP autoconfiguration to succeed on these interfaces.

On the reverse path, make sure we do not attempt to close a DSA-enabled
device as this would implicitely prevent the slave DSA network device
from operating.

Signed-off-by: Florian Fainelli 
Signed-off-by: David S. Miller

ipv4: remove 0/NULL assignment on static

2014-11-04T20:09:52+00:00

static values are automatically initialized to 0

Signed-off-by: Fabian Frederick 
Signed-off-by: David S. Miller

ipv4: remove set but unused variable sha

2014-10-27T20:03:52+00:00

unsigned char *sha (source) was already in original git version
 but was never used.

Signed-off-by: Fabian Frederick 
Signed-off-by: David S. Miller

ipconfig: Use time_before

2014-08-22T19:23:11+00:00

The functions time_before, time_before_eq, time_after, and time_after_eq
are more robust for comparing jiffies against other values.

A simplified version of the Coccinelle semantic patch making this change
is as follows:

@change@
expression E1,E2;
@@
- jiffies - E1 < E2
+ time_before(jiffies, E1+E2)

Signed-off-by: Himangi Saraogi 
Acked-by: Julia Lawall 
Signed-off-by: David S. Miller

ipconfig: Only bootp paths should reference ic_dev_xid.

2014-07-10T05:25:18+00:00

It is only tested, and declared, in the bootp code.

So, in ic_dynamic() guard it's setting with IPCONFIG_BOOTP.

Signed-off-by: David S. Miller

ipconfig: move ic_dev_xid under IPCONFIG_BOOTP

2014-07-09T21:32:54+00:00

ic_dev_xid is only used in __init ic_bootp_recv under IPCONFIG_BOOTP
and __init ic_dynamic under IPCONFIG_DYNAMIC(which is itself defined
with the same IPCONFIG_BOOTP)

This patch fixes the following warning when IPCONFIG_BOOTP is not set:
>> net/ipv4/ipconfig.c:146:15: warning: 'ic_dev_xid' defined but not used [-Wunused-variable]
    static __be32 ic_dev_xid;  /* Device under configuration */

Reported-by: Fengguang Wu 
Cc: Fengguang Wu 
Cc: "David S. Miller" 
Cc: Alexey Kuznetsov 
Cc: netdev@vger.kernel.org
Signed-off-by: Fabian Frederick 
Signed-off-by: David S. Miller

ipconfig: add static to local variable

2014-07-08T18:30:33+00:00

ic_dev_xid is only used in ipconfig.c

Cc: "David S. Miller" 
Cc: Alexey Kuznetsov 
Cc: netdev@vger.kernel.org
Signed-off-by: Fabian Frederick 
Signed-off-by: David S. Miller

ipv4: ipconfig.c: add parentheses in an if statement

2014-02-14T05:14:23+00:00

Even if the 'time_before' macro expand with parentheses, the look is bad.

Signed-off-by: Francois-Xavier Le Bail 
Signed-off-by: David S. Miller