[NET]: Elminate spurious print_mac() calls.
authorDavid S. Miller <davem@davemloft.net>
Tue, 19 Feb 2008 00:50:22 +0000 (16:50 -0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 19 Feb 2008 00:50:22 +0000 (16:50 -0800)
Patrick McHardy notes that print_mac() can get invoked
even if the result it unused (f.e. as an argument to
pr_debug() when DEBUG is not defined).

Mark this function as "__pure" to eliminate this problem.

Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/if_ether.h

index 7a1e011b8a2cda33a140acbc747aeb369f200f97..42dc6a3571ec78a94d1de22dcae6e4a3b3ceece5 100644 (file)
@@ -129,7 +129,7 @@ extern ssize_t sysfs_format_mac(char *buf, const unsigned char *addr, int len);
 /*
  *     Display a 6 byte device address (MAC) in a readable format.
  */
-extern char *print_mac(char *buf, const unsigned char *addr);
+extern __pure char *print_mac(char *buf, const unsigned char *addr);
 #define MAC_BUF_SIZE   18
 #define DECLARE_MAC_BUF(var) char var[MAC_BUF_SIZE] __maybe_unused