linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* A suggestion for the netfilter part of the sources
@ 2003-03-04 17:26 David Laganière
  2003-03-04 18:20 ` Harald Welte
  0 siblings, 1 reply; 4+ messages in thread
From: David Laganière @ 2003-03-04 17:26 UTC (permalink / raw)
  To: linux-kernel

Hi!

Since a couple of new kernel versions already, I use to modify two files 
related to the netfilter part to be able to add more
ports for the IRC NAT module. I was wondering if you could definitively 
apply those modifications to the kernel sources.
Here are my two modifications:

In /usr/src/linux-2.4.20/net/ipv4/netfilter:
I change "#define MAX_PORTS 8" to "#define MAX_PORTS 15" in both 
"ip_conntrack_irc.c" and "ip_nat_irc.c".

I make those modifications to be able to add more ports than only 8 when 
loading the modules as there are actually
a lot more IRC ports than 8 (ex: 6660-6669, 7000, that's already 11).

I'd greatly appreciate a reply even though my suggestion is not a good one.

Thanks.

David Laganière
Network/System Administrator
Securinet Systems


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: A suggestion for the netfilter part of the sources
  2003-03-04 17:26 A suggestion for the netfilter part of the sources David Laganière
@ 2003-03-04 18:20 ` Harald Welte
  2003-03-04 22:56   ` Dominik Kubla
  0 siblings, 1 reply; 4+ messages in thread
From: Harald Welte @ 2003-03-04 18:20 UTC (permalink / raw)
  To: David Laganière; +Cc: linux-kernel, Netfilter Development Mailinglist

[-- Attachment #1: Type: text/plain, Size: 1742 bytes --]

On Tue, Mar 04, 2003 at 12:26:32PM -0500, David Laganière wrote:
 
> Since a couple of new kernel versions already, I use to modify two files 
> related to the netfilter part to be able to add more
> ports for the IRC NAT module. I was wondering if you could definitively 
> apply those modifications to the kernel sources.

We (the netfilter developers) thought that for the usual case, 8 ports
should be a reasonable compiletime-limit.  I know, especially for IRC,
this largely depends on the number of IRC networks and servers you want
to support...

> Here are my two modifications:
> 
> In /usr/src/linux-2.4.20/net/ipv4/netfilter:
> I change "#define MAX_PORTS 8" to "#define MAX_PORTS 15" in both 
> "ip_conntrack_irc.c" and "ip_nat_irc.c".

yes, this is the (documented) way to compile with support for more ports

> I'd greatly appreciate a reply even though my suggestion is not a good one.

The suggestion is neither 'good' nor 'bad'.  Nobody has (until now)
asked us to raise this value, eight seems to be enough for most people.

As long as your proposal is not backed by more other users who think the
default should be raised, I'd rather leave it the way it currently is.

btw: further discussion should happen at
netfilter-devel@lists.netfilter.org

> David Laganière
> Network/System Administrator
> Securinet Systems

-- 
- Harald Welte <laforge@netfilter.org>             http://www.netfilter.org/
============================================================================
  "Fragmentation is like classful addressing -- an interesting early
   architectural error that shows how much experimentation was going
   on while IP was being designed."                    -- Paul Vixie

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: A suggestion for the netfilter part of the sources
  2003-03-04 18:20 ` Harald Welte
@ 2003-03-04 22:56   ` Dominik Kubla
  2003-03-04 23:27     ` Harald Welte
  0 siblings, 1 reply; 4+ messages in thread
From: Dominik Kubla @ 2003-03-04 22:56 UTC (permalink / raw)
  To: Harald Welte, David Laganière
  Cc: linux-kernel, Netfilter Development Mailinglist

On Tuesday 04 March 2003 19:20, Harald Welte wrote:

>
> The suggestion is neither 'good' nor 'bad'.  Nobody has (until now)
> asked us to raise this value, eight seems to be enough for most people.
>
> As long as your proposal is not backed by more other users who think the
> default should be raised, I'd rather leave it the way it currently is.
>

Since this is meant to be tunable, how about turning it into a configuration 
option (with 8 being the default)? I guess that would solve this problem 
quite nicely.

Regards,
  Dominik
-- 
Why should George W. Bush care what the American people think?
After all they did not vote for him.


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: A suggestion for the netfilter part of the sources
  2003-03-04 22:56   ` Dominik Kubla
@ 2003-03-04 23:27     ` Harald Welte
  0 siblings, 0 replies; 4+ messages in thread
From: Harald Welte @ 2003-03-04 23:27 UTC (permalink / raw)
  To: Dominik Kubla
  Cc: David Laganière, linux-kernel, Netfilter Development Mailinglist

[-- Attachment #1: Type: text/plain, Size: 865 bytes --]

On Tue, Mar 04, 2003 at 11:56:42PM +0100, Dominik Kubla wrote:
 
> Since this is meant to be tunable, how about turning it into a configuration 
> option (with 8 being the default)? I guess that would solve this problem 
> quite nicely.

well, if you think the 'netfilter configuration' submenu doesn't already
have enough config options ;)

SCNR.

Anyway, yes, this would be acceptable.  Patches are welcome, otherwise
it will end up on my TODO list.

> Regards,
>   Dominik

-- 
- Harald Welte <laforge@netfilter.org>             http://www.netfilter.org/
============================================================================
  "Fragmentation is like classful addressing -- an interesting early
   architectural error that shows how much experimentation was going
   on while IP was being designed."                    -- Paul Vixie

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2003-03-04 23:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-03-04 17:26 A suggestion for the netfilter part of the sources David Laganière
2003-03-04 18:20 ` Harald Welte
2003-03-04 22:56   ` Dominik Kubla
2003-03-04 23:27     ` Harald Welte

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).