Re: Added unicast support to SNTP client

[Andrew Lunn <andrew at lunn dot ch>]

On Wed, Oct 15, 2003 at 10:39:29AM -0700, Dan Jakubiec wrote:
> I've attached an SNTP patch that addresses Andrew
> Lunn's comments about my previous SNTP unicast
> enhancements.  This patch includes:
> 
> 1) SNTP documentation updates
> 
> 2) SNTP test updates to include unicast mode testing
> 
> 3) Support for extracting NTP server information from
> DHCP
> 
> 4) Removed NTP_UDP_PORT constant from sntp.h

Thanks for the patch. I made a few tweaks.

There was a problem that the DHCP code would modify the list of
servers while sntp could be using it. I got around this by
deregistering the existing servers before building the new list.

I added code where the list of options are printed so that it know
knows about the TAG_NTP_SERVER option.

I also made a few tweaks to the documentation.

What would be nice is if the call the cyg_sntp_set_servers would break
the main thread out of select and so imeadiately use the new list of
servers. Something to add to the TODO list.

Attached i what i committed.

         Andrew

Attachment: sntp.diff
Description: Text document