public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ben Warren <biggerbadderben@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Duplicate IP address check
Date: Sun, 06 Jan 2008 16:30:19 -0500	[thread overview]
Message-ID: <4781486B.7030305@gmail.com> (raw)
In-Reply-To: <4780C0EA.8040701@gmail.com>

gvb.uboot wrote:
> Ben Warren wrote:
>> Hi Dan,
>>
>> Dan Batryn wrote:
>>> Due to a mis-configuration of our DHCP server there was an
>>> overlapping range of DHCP assigned IP address and units configured
>>> to have IP addresses statically assigned. I was surprised to find
>>> that UBOOT did not complain of a duplicate IP address while trying
>>> to boot via the network. Looking through the source I cannot see
>>> any code to perform the typical ARP for yourself check that I have
>>> seen before. Could someone please tell me if I have overlooked
>>> something or is this feature missing?
>>>
>> There is no explicit check for duplicate IP assignment. U-boot's 
>> networking code is intentionally minimalist and thus lacks many such 
>> features. Feel free to provide a patch and a convincing argument of 
>> why it's needed.
>>
>> regards,
>> Ben
>
> Ditto on the patch.
>
> WRT the convincing argument, I think the bar would be *really* low on 
> this one. :-) I would consider a convincing argument to be "The nearly 
> universal convention of performing an ARP to verify that the 
> DHCP-assigned IP address is unused is missing."
Sure, this is a no-brainer. But, it shouldn't be limited to DHCP since 
having a dynamic address trouncing a static one is only one trouble 
scenario. Who knows, there may be people out there who object to the 
extra milliseconds that this ARP will add to boot time (not me).

regards,
Ben

  reply	other threads:[~2008-01-06 21:30 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-04 22:01 [U-Boot-Users] Duplicate IP address check Dan Batryn
2008-01-06  4:40 ` Ben Warren
2008-01-06 11:52   ` gvb.uboot
2008-01-06 21:30     ` Ben Warren [this message]
2008-01-07  0:09       ` gvb.uboot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4781486B.7030305@gmail.com \
    --to=biggerbadderben@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox