It looks good and It did not break anything in the general dns, dhcp etc. functionality as much as I ran it. Thanks. Anything will go in the contrib directory about this ! ?
cheers Sushil. On Mon, Jun 27, 2011 at 9:17 PM, Simon Kelley <si...@thekelleys.org.uk>wrote: > Sushil Agrawal wrote: > > [sorry if you have received multiple copies of this email; facing > > problems in posting > > to the list; big size etc ... ] > > > > > > Attached is the modified patch after the feedback. It is also > > pasted at the end of this email. > > > > Following is the summary of the changes: > > > > 1. dnsmasq-2.57/dnsmasq.conf. > > example: > > 2. dnsmasq-2.57/man/dnsmasq.8 > > added example and description for following version of dhcp-boot > > > > #dhcp-boot=/var/ftpd/pxelinux.0,boothost,tftp_server_name > > > > 3. > > dnsmasq-2.57/src/dnsmasq.h > > added a new field 'tftp_sname' in struct dhcp_boot > > for the tftp server name. > > > > 4. > > dnsmasq-2.57/src/option.c > > added code to process the tftp_servername option. > > > > 5. > > dnsmasq-2.57/src/rfc2131.c > > added code to call cache_find_by_name() to get an IP address > > of the tftp server given its name. cache_find_by_name() > > automatically returns these addresses in cycle. > > > > > > tested with: > > > > dhcp-boot=pxelinux.0,test2,test2 > > in the dnsmasq.conf file > > > > --dhcp-boot=pxelinux.0,test2,test2 > > on the command line > > > > --dhcp-boot=pxelinux.0,,test2 > > on the command line. > > > > In these examples test2 was considered as tftp server name and > > so it was looked up in the cache and one ip address was returned. > > > > > > Thanks for that. I integrated your patch and tweaked it a bit, please > could you test > > > http://www.thekelleys.org.uk/dnsmasq/test-releases/dnsmasq-2.58test15.tar.gz > > and make sure I didn't break anything? > > > Cheers, > > Simon. > >
