lkml.org 
[lkml]   [1999]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] linux/net/ipv4/arp.c, kernel 2.0.36 (& 2.0.37-pre9)
Date
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=US-ASCII">
<META NAME="Generator" CONTENT="MS Exchange Server version 5.5.2448.0">
<TITLE>RE: [PATCH] linux/net/ipv4/arp.c, kernel 2.0.36 (&amp; 2.0.37-pre9)</TITLE>
</HEAD>
<BODY>

<P><FONT SIZE=2 FACE="Arial">Are there any &quot;official&quot; sites for posting patches like this (i.e., patches that don't make it into the main kernel)?&nbsp; It might help others avoid working on previously-solved problems, as well as help those who know little about kernel hacking.</FONT></P>

<P><FONT SIZE=2 FACE="Arial">Brian</FONT>
</P>
<UL>
<P><FONT SIZE=1 FACE="MS Sans Serif">----------</FONT>
<BR><B><FONT SIZE=1 FACE="MS Sans Serif">From:</FONT></B> &nbsp; <FONT SIZE=1 FACE="MS Sans Serif">alan@lxorguk.ukuu.org.uk[SMTP:alan@lxorguk.ukuu.org.uk]</FONT>
<BR><B><FONT SIZE=1 FACE="MS Sans Serif">Sent:</FONT></B> &nbsp; <FONT SIZE=1 FACE="MS Sans Serif">Monday, March 29, 1999 1:25 PM</FONT>
<BR><B><FONT SIZE=1 FACE="MS Sans Serif">To:</FONT></B> &nbsp;&nbsp;&nbsp; <FONT SIZE=1 FACE="MS Sans Serif">pavel@bug.ucw.cz</FONT>
<BR><B><FONT SIZE=1 FACE="MS Sans Serif">Cc:</FONT></B> &nbsp;&nbsp;&nbsp; <FONT SIZE=1 FACE="MS Sans Serif">alan@lxorguk.ukuu.org.uk; bmoyle@redcreek.com; linux-kernel@vger.rutgers.edu</FONT>
<BR><B><FONT SIZE=1 FACE="MS Sans Serif">Subject:</FONT></B> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; <FONT SIZE=1 FACE="MS Sans Serif">Re: [PATCH] linux/net/ipv4/arp.c, kernel 2.0.36&nbsp; (&amp; 2.0.37-pre9)</FONT>
</P>

<P><FONT SIZE=2 FACE="Arial">&gt; &gt; The kernel knows perfectly well that its silly to send packets to yourself </FONT>
<BR><FONT SIZE=2 FACE="Arial">&gt; &gt; via external interfaces. This patch isn't needed. The bug is your routing</FONT>
<BR><FONT SIZE=2 FACE="Arial">&gt; &gt; table and trying to set up this configuration.</FONT>
<BR><FONT SIZE=2 FACE="Arial">&gt; </FONT>
<BR><FONT SIZE=2 FACE="Arial">&gt; Alan, this bitten me long time ago, too. External loopback is very</FONT>
<BR><FONT SIZE=2 FACE="Arial">&gt; nice for testing...</FONT>
</P>

<P><FONT SIZE=2 FACE="Arial">There are two reasons Im not putting this in</FONT>
</P>

<P><FONT SIZE=2 FACE="Arial">1. Its a very specialised need. The patch is fine if you need it add it - its</FONT>
<BR><FONT SIZE=2 FACE="Arial">more specialised than other &quot;not needed&quot; patches people keep out of the</FONT>
<BR><FONT SIZE=2 FACE="Arial">main tree. </FONT>
</P>

<P><FONT SIZE=2 FACE="Arial">2. Its on a common code path. Sum the number of times that path is taken</FONT>
<BR><FONT SIZE=2 FACE="Arial">by the number of users of Linux versus the 3 or 4 who actually need it.</FONT>
</P>

<P><FONT SIZE=2 FACE="Arial">There isnt anything wrong with the patch, its just not relevant to the</FONT>
<BR><FONT SIZE=2 FACE="Arial">userbase as a whole and its on a regularly executed path</FONT>
</P>

<P><FONT SIZE=2 FACE="Arial">Alan</FONT>
</P>
</UL>
</BODY>
</HTML>
\
 
 \ /
  Last update: 2005-03-22 13:51    [W:0.892 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site