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