[U-Boot] [PATCH 00/28] Add link-local addressing support

Joe Hershberger joe.hershberger at ni.com
Fri Jan 20 01:53:03 CET 2012


Link-local support allows the bootloader to get a usable network address
regardless of network environment.  Given that bootloaders may not always
have access to a DHCP server, this functionality makes accessing it in
those environments easier.  It is also referred to as zero-configuration IP.

The following script allows the network to be configured for DHCP, link-local,
and static IP addresses configurably by the environment using "dhcpenabled"
and "linklocalenabled".

#define CONFIG_EXTRA_ENV_SETTINGS \
	"ipconfigcmd=if test \\\"$dhcpenabled\\\" -ne 0;"		\
		"then "							\
			"dhcpfail=0;dhcp || dhcpfail=1;"		\
		"else "							\
			"dhcpfail=-1;"					\
		"fi;"							\
		"if test \\\"$linklocalenabled\\\" -ne 0 -a "		\
			"\\\"$dhcpfail\\\" -ne 0;"			\
		"then "							\
			"linklocal;"					\
			"llfail=0;"					\
		"else "							\
			"llfail=-1;"					\
		"fi;"							\
		"if test \\\"$llfail\\\" -ne 0 -a "			\
			"\\\"$dhcpfail\\\" -ne 0; "			\
		"then "							\
			"setenv ipaddr $sipaddr; "			\
			"setenv netmask $snetmask; "			\
			"setenv gatewayip $sgatewayip; "		\
		"fi;\0"							\

Some refactoring of net/ was done to make adding link-local support cleaner
as well as generally improving the readability and organization of the code.

Tested on da850_am18xxevm
Tested on MPC8313ERDB_33

check-patch
	No warnings or errors (depends on ignoring CONSIDER_KSTRTO)

MAKEALL -a powerpc
	No additional warnings or errors beyond master

MAKEALL -a arm
	No additional warnings or errors beyond master

git-test-sequence
	Cleanly bisectable
	Code-size affects on MPC8313ERDB_33 shown below

> cat sizeofMCP8313
make BUILD_DIR=/tmp/u-boot-build/ ARCH=powerpc CROSS_COMPILE=powerpc-linux-gnu- USE_PRIVATE_LIBGG=yes distclean > /dev/null && \
make BUILD_DIR=/tmp/u-boot-build/ ARCH=powerpc CROSS_COMPILE=powerpc-linux-gnu- MPC8313ERDB_33_config > /dev/null && \
make BUILD_DIR=/tmp/u-boot-build/ ARCH=powerpc CROSS_COMPILE=powerpc-linux-gnu- USE_PRIVATE_LIBGG=yes > /dev/null && \
powerpc-linux-gnu-size /tmp/u-boot-build/u-boot

> git-test-sequence origin/master.. './sizeofMCP8313'
Testing ./sizeofMCP8313
HEAD is now at 4dbe8d8 net: Remove volatile from all of net except the eth driver interface
   text	   data	    bss	    dec	    hex	filename
 264917	  24880	  42104	 331901	  5107d	/tmp/u-boot-build/ppc/u-boot
Passed: 90ea672853753b5a4244db60d095e5f99cdcfe7d.
Testing ./sizeofMCP8313
HEAD is now at 92893c4 net: cosmetic: checkpatch compliance
   text	   data	    bss	    dec	    hex	filename
 264917	  24880	  42104	 331901	  5107d	/tmp/u-boot-build/ppc/u-boot
Passed: 12b0b6e953406a1ed98d1c4dd9701078d1dfe00d.
Testing ./sizeofMCP8313
HEAD is now at d6c19a3 net: Move mv64x6x_eth_initialize() to board_eth_init()
   text	   data	    bss	    dec	    hex	filename
 264917	  24880	  42104	 331901	  5107d	/tmp/u-boot-build/ppc/u-boot
Passed: cf9f493ab5d08a282ec9905797fa58d82b15714d.
Testing ./sizeofMCP8313
HEAD is now at 52403e8 net: Make the MAC-seeded random number generator available to /net
   text	   data	    bss	    dec	    hex	filename
 264917	  24880	  42104	 331901	  5107d	/tmp/u-boot-build/ppc/u-boot
Passed: a92c9c9e8f68337a78ee2d7c6775299d7c057b52.
Testing ./sizeofMCP8313
HEAD is now at ff86c81 net: Move CDP out of net.c
   text	   data	    bss	    dec	    hex	filename
 264917	  24880	  42104	 331901	  5107d	/tmp/u-boot-build/ppc/u-boot
Passed: 9bf01065cb04c34e43cc5d56162aa2dac9d61f3e.
Testing ./sizeofMCP8313
HEAD is now at da1998d net: Move ARP out of net.c
   text	   data	    bss	    dec	    hex	filename
 265045	  24928	  42104	 332077	  5112d	/tmp/u-boot-build/ppc/u-boot
Passed: 621d67b8c6126ef6510d0a2b2c06ce410cd60327.
Testing ./sizeofMCP8313
HEAD is now at 408b8ab net: Move PING out of net.c
   text	   data	    bss	    dec	    hex	filename
 265237	  24976	  42096	 332309	  51215	/tmp/u-boot-build/ppc/u-boot
Passed: e8aaa7382de39137c91809ba2e121f87338627d5.
Testing ./sizeofMCP8313
HEAD is now at 1b3ed14 net: Move RARP receive logic out of net.c
   text	   data	    bss	    dec	    hex	filename
 265237	  24976	  42096	 332309	  51215	/tmp/u-boot-build/ppc/u-boot
Passed: 7a8216874def295337178da67078758cbf909681.
Testing ./sizeofMCP8313
HEAD is now at 50eb857 net: cosmetic: Un-typedef variables in net
   text	   data	    bss	    dec	    hex	filename
 265237	  24976	  42096	 332309	  51215	/tmp/u-boot-build/ppc/u-boot
Passed: b679739af8ffc6f9a31264c175077f4ac96c236d.
Testing ./sizeofMCP8313
HEAD is now at 22259f9 net: cosmetic: Improve variable names and code readability
   text	   data	    bss	    dec	    hex	filename
 265237	  24976	  42096	 332309	  51215	/tmp/u-boot-build/ppc/u-boot
Passed: 834a569897337636fe861797625fbdbbe1373f75.
Testing ./sizeofMCP8313
HEAD is now at 5340a18 net: Refactor IP, UPD, and ICMP header writing functions
   text	   data	    bss	    dec	    hex	filename
 265237	  24972	  42096	 332305	  51211	/tmp/u-boot-build/ppc/u-boot
Passed: 20b84dcfa7d355010cacc3493e1e86521b00879f.
Testing ./sizeofMCP8313
HEAD is now at 275e230 net: Refactor packet length computations
   text	   data	    bss	    dec	    hex	filename
 265221	  24972	  42096	 332289	  51201	/tmp/u-boot-build/ppc/u-boot
Passed: d078d84d959dbb40a67baa16e8b0d210e6472c79.
Testing ./sizeofMCP8313
HEAD is now at c1214f2 net: Refactor bootp packet length computations
   text	   data	    bss	    dec	    hex	filename
 265221	  24972	  42096	 332289	  51201	/tmp/u-boot-build/ppc/u-boot
Passed: 3ff5dfd46c8f70ec2304a150a606871b7504cf42.
Testing ./sizeofMCP8313
HEAD is now at 9cff376 net: Refactor ping recieve handler
   text	   data	    bss	    dec	    hex	filename
 265093	  24968	  42096	 332157	  5117d	/tmp/u-boot-build/ppc/u-boot
Passed: e531ec8d3986f45d30862eacce73b0db55e2fb13.
Testing ./sizeofMCP8313
HEAD is now at e47e5ac net: Refactor to call NetSendPacket() instead of calling eth_send()
   text	   data	    bss	    dec	    hex	filename
 265093	  24968	  42096	 332157	  5117d	/tmp/u-boot-build/ppc/u-boot
Passed: 4a78d3ab2514c8d00bc18978f7a489a800a3a77b.
Testing ./sizeofMCP8313
HEAD is now at 9bcb672 net: Refactor to protect access to the NetState variable
   text	   data	    bss	    dec	    hex	filename
 265109	  24956	  42096	 332161	  51181	/tmp/u-boot-build/ppc/u-boot
Passed: 8ecfa86d86fe2624cb509c41fbaec8dcf13a9945.
Testing ./sizeofMCP8313
HEAD is now at a2c79e3 net: Refactor to separate the UDP handler from the ARP handler
   text	   data	    bss	    dec	    hex	filename
 265317	  24956	  42112	 332385	  51261	/tmp/u-boot-build/ppc/u-boot
Passed: f66f88edda9ec16b3de20a10faa1c6bb35f316c2.
Testing ./sizeofMCP8313
HEAD is now at 502deaa net: Add NetUpdateEther() to handle ARP or Ping replies (VLAN or SNAP)
   text	   data	    bss	    dec	    hex	filename
 265429	  24952	  42112	 332493	  512cd	/tmp/u-boot-build/ppc/u-boot
Passed: 3daff36dc0b5b65b8df7a3a7011f1e5f89eb5a9d.
Testing ./sizeofMCP8313
HEAD is now at 315bd4d net: Don't write the "serverip" env var if told not to in the config
   text	   data	    bss	    dec	    hex	filename
 265429	  24952	  42112	 332493	  512cd	/tmp/u-boot-build/ppc/u-boot
Passed: b0e7a73485c771eed710d6b9afd1e80e0ccc2cfa.
Testing ./sizeofMCP8313
HEAD is now at 82b1968 net: Fix compile warning if CONFIG_BOOTP_SERVERIP is not defined
   text	   data	    bss	    dec	    hex	filename
 265429	  24952	  42112	 332493	  512cd	/tmp/u-boot-build/ppc/u-boot
Passed: 121eb763232d5cf6a4e66f6c100713537f6036cf.
Testing ./sizeofMCP8313
HEAD is now at cebdefe net: Remove unneeded static allocation for MAC address in PingSend()
   text	   data	    bss	    dec	    hex	filename
 265397	  24944	  42104	 332445	  5129d	/tmp/u-boot-build/ppc/u-boot
Passed: 1366a9b1529e9752dd4a47d7c82545b13e11b461.
Testing ./sizeofMCP8313
HEAD is now at 8f856ad net: Fix net buffer initialization
   text	   data	    bss	    dec	    hex	filename
 265605	  24956	  42104	 332665	  51379	/tmp/u-boot-build/ppc/u-boot
Passed: 16cbef4189d5be4837c3de9dc2ac0bcf8d89fa75.
Testing ./sizeofMCP8313
HEAD is now at 002991b net: Refactor NetSendUDPPacket to share more code
   text	   data	    bss	    dec	    hex	filename
 265557	  24956	  42104	 332617	  51349	/tmp/u-boot-build/ppc/u-boot
Passed: 7f001d4025dc564fe2a96a1e32214e194b891bd1.
Testing ./sizeofMCP8313
HEAD is now at 314902e net: Don't copy every packet that waits for an ARP
   text	   data	    bss	    dec	    hex	filename
 265573	  24956	  42104	 332633	  51359	/tmp/u-boot-build/ppc/u-boot
Passed: 3be3cdd2224c88c88442e48c0f4f836d7be1d8ae.
Testing ./sizeofMCP8313
HEAD is now at 2a43805 net: Add option CONFIG_BOOTP_CAN_FAIL
   text	   data	    bss	    dec	    hex	filename
 265573	  24956	  42104	 332633	  51359	/tmp/u-boot-build/ppc/u-boot
Passed: 4d578ba84228e51a56249035207085f0845d3be8.
Testing ./sizeofMCP8313
HEAD is now at defb33c net: Add link-local addressing support
   text	   data	    bss	    dec	    hex	filename
 265577	  24956	  42104	 332637	  5135d	/tmp/u-boot-build/ppc/u-boot
Passed: cb6b29e82f292b5b27aec6f51f2f9d50e1f8b0f8.
Testing ./sizeofMCP8313
HEAD is now at 615c707 net: Work-around for brain-damaged Cisco routers with arp-proxy
   text	   data	    bss	    dec	    hex	filename
 265657	  24956	  42104	 332717	  513ad	/tmp/u-boot-build/ppc/u-boot
Passed: 63a3ddaf7e029bb7ec71b1edb42473ae6a208dec.
Testing ./sizeofMCP8313
HEAD is now at 7d5229d net: Allow filtering on debug traces in the net subsystem
   text	   data	    bss	    dec	    hex	filename
 265594	  24948	  42104	 332646	  51366	/tmp/u-boot-build/ppc/u-boot
Passed: 1cf14c686c5b60bfa48d8bfd6eb40e20ac0c378b.
All's well.


Joe Hershberger (28):
  net: Remove volatile from all of net except the eth driver interface
  net: cosmetic: checkpatch compliance
  net: Move mv64x6x_eth_initialize() to board_eth_init()
  net: Make the MAC-seeded random number generator available to /net
  net: Move CDP out of net.c
  net: Move ARP out of net.c
  net: Move PING out of net.c
  net: Move RARP receive logic out of net.c
  net: cosmetic: Un-typedef variables in net
  net: cosmetic: Improve variable names and code readability
  net: Refactor IP, UPD, and ICMP header writing functions
  net: Refactor packet length computations
  net: Refactor bootp packet length computations
  net: Refactor ping recieve handler
  net: Refactor to call NetSendPacket() instead of calling eth_send()
  net: Refactor to protect access to the NetState variable
  net: Refactor to separate the UDP handler from the ARP handler
  net: Add NetUpdateEther() to handle ARP or Ping replies (VLAN or
    SNAP)
  net: Don't write the "serverip" env var if told not to in the config
  net: Fix compile warning if CONFIG_BOOTP_SERVERIP is not defined
  net: Remove unneeded static allocation for MAC address in PingSend()
  net: Fix net buffer initialization
  net: Refactor NetSendUDPPacket to share more code
  net: Don't copy every packet that waits for an ARP
  net: Add option CONFIG_BOOTP_CAN_FAIL
  net: Add link-local addressing support
  net: Work-around for brain-damaged Cisco routers with arp-proxy
  net: Allow filtering on debug traces in the net subsystem

 README                               |   22 +-
 arch/powerpc/cpu/mpc8260/ether_fcc.c |   10 +-
 arch/powerpc/cpu/mpc8xx/fec.c        |    2 +-
 board/Marvell/db64360/db64360.c      |   12 +-
 board/Marvell/db64360/mv_eth.c       |   78 ++-
 board/Marvell/db64360/mv_eth.h       |   63 +--
 board/Marvell/db64460/db64460.c      |   12 +-
 board/Marvell/db64460/mv_eth.c       |   78 ++-
 board/Marvell/db64460/mv_eth.h       |   63 +--
 board/esd/cpci750/cpci750.c          |    6 +
 board/esd/cpci750/mv_eth.c           |   78 ++-
 board/esd/cpci750/mv_eth.h           |   63 +--
 board/prodrive/p3mx/mv_eth.c         |   78 ++-
 board/prodrive/p3mx/mv_eth.h         |   63 +--
 board/prodrive/p3mx/p3mx.c           |    6 +
 common/cmd_net.c                     |   34 +-
 common/main.c                        |    1 -
 drivers/net/netconsole.c             |  130 ++--
 include/common.h                     |    4 +
 include/configs/am3517_crane.h       |    2 +-
 include/configs/am3517_evm.h         |    2 +-
 include/configs/origen.h             |    2 +-
 include/net.h                        |  206 ++++---
 net/Makefile                         |    8 +-
 net/arp.c                            |  242 +++++++
 net/arp.h                            |   30 +
 net/bootp.c                          |  435 ++++++-------
 net/bootp.h                          |   60 +-
 net/cdp.c                            |  372 +++++++++++
 net/cdp.h                            |   18 +
 net/dns.c                            |   12 +-
 net/eth.c                            |   75 +--
 net/link_local.c                     |  340 ++++++++++
 net/link_local.h                     |   21 +
 net/net.c                            | 1152 +++++++++-------------------------
 net/net_rand.c                       |   68 ++
 net/net_rand.h                       |   18 +
 net/nfs.c                            |  306 +++++-----
 net/nfs.h                            |    2 +-
 net/ping.c                           |  119 ++++
 net/ping.h                           |   20 +
 net/rarp.c                           |   77 ++-
 net/rarp.h                           |    5 +-
 net/sntp.c                           |   37 +-
 net/sntp.h                           |    2 +-
 net/tftp.c                           |   50 +-
 net/tftp.h                           |    2 +-
 47 files changed, 2625 insertions(+), 1861 deletions(-)
 create mode 100644 net/arp.c
 create mode 100644 net/arp.h
 create mode 100644 net/cdp.c
 create mode 100644 net/cdp.h
 create mode 100644 net/link_local.c
 create mode 100644 net/link_local.h
 create mode 100644 net/net_rand.c
 create mode 100644 net/net_rand.h
 create mode 100644 net/ping.c
 create mode 100644 net/ping.h



More information about the U-Boot mailing list