OSDN Git Service

Merge branch 'net-if_arp-use-define-instead-of-hard-coded-value'
authorDavid S. Miller <davem@davemloft.net>
Sat, 22 Sep 2018 02:22:32 +0000 (19:22 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sat, 22 Sep 2018 02:22:32 +0000 (19:22 -0700)
commitb4d0782440ed38c14ff145ead241d5fcc95a37b7
tree60fdc2678ff4aabc4906b5d2bb24738cf3ad7596
parenta42055e8d2c30d4decfc13ce943d09c7b9dad221
parent6a12709da354ea149fdf86c4c9aba5b5033e9cf2
Merge branch 'net-if_arp-use-define-instead-of-hard-coded-value'

HÃ¥kon Bugge says:

====================
net: if_arp: use define instead of hard-coded value

Struct arpreq contains the name of the device. All other places in the
kernel, the define IFNAMSIZ is used to designate its size. But in
if_arp.h, a literal constant is used.

As it could be good reasons to use constants instead of the defines in
include files under uapi, it seems to be OK to use the define here,
without opening a can of worms in user-land.

This because if_arp.h includes netdevice.h, which also uses
IFNAMSIZ. For the distros I have checked, this also holds true for the
use-land side.

The series also fixes some incorrect indents.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>