Re: [PATCH 4/5] Add DAD do dhclient-script

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



David Dillow wrote:
On Wed, 2009-06-24 at 20:08 +0200, Philippe Seewer wrote:
The manpage for dhclient-script says:

Before actually configuring the address, dhclient-script should
somehow ARP for it and exit with a nonzero status if it receives a
reply.

By using arping in dracut this is very easy, since arping has a
specific option to do just that.

--- a/modules.d/40network/dhclient-script
+++ b/modules.d/40network/dhclient-script
@@ -64,6 +64,7 @@ case $reason in
 	sleep 1
 	;;
     BOUND)
+	arping -q -D -c 2 -I $netif $new_ip_address || exit 1

Where's the addition of arping to 40network/install?

*sigh* Thanks for noticing, forgot to add it to the commit. Complete
Patch below

---
modules.d/40network/dhclient-script |    1 +
modules.d/40network/install         |    2 +-
2 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/modules.d/40network/dhclient-script b/modules.d/40network/dhclient-script
index 6df630d..3fb8da6 100755
--- a/modules.d/40network/dhclient-script
+++ b/modules.d/40network/dhclient-script
@@ -64,6 +64,7 @@ case $reason in
	sleep 1
	;;
    BOUND)
+	arping -q -D -c 2 -I $netif $new_ip_address || exit 1
setup_interface set | while read line; do
	    [ "${line#new_}" = "$line" ] && continue
diff --git a/modules.d/40network/install b/modules.d/40network/install
index 0b76cbd..513503c 100755
--- a/modules.d/40network/install
+++ b/modules.d/40network/install
@@ -1,5 +1,5 @@
#!/bin/bash
-dracut_install ip dhclient hostname
+dracut_install ip dhclient hostname arping
# Include wired net drivers, excluding wireless
for modname in $(find "/lib/modules/$kernel/kernel/drivers" -name '*.ko'); do
  if nm -uPA $modname | grep -q eth_type_trans; then
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux