Bug 63 – flag needed in an internet node to distinguish routers from hosts

Bug 63 - flag needed in an internet node to distinguish routers from hosts


Summary:	flag needed in an internet node to distinguish routers from hosts

Status:	RESOLVED FIXED

Product:	ns-3
Classification:	Unclassified
Component:	routing
Version:	ns-3-dev
Hardware:	All All

Importance:	P2 minor
Assigned To:	Tom Henderson

URL:
Keywords:

Depends on:
Blocks:	384
	Show dependency tree / graph

Reported:	2007-08-06 00:49 EDT by Tom Henderson
Modified:	2009-06-26 04:57 EDT (History)
CC List:	3 users (show)

See Also:

Attachments
tested patch (26.14 KB, patch) 2009-06-25 16:08 EDT, Tom Henderson	Details \| Diff
additional patch for global routing (1.77 KB, patch) 2009-06-25 16:11 EDT, Tom Henderson	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Tom Henderson 2007-08-06 00:49:38 EDT

discussion from the list:
>> - whether to add some kind of flag in an InternetNode that is equivalent
>> >> to /proc/sys/net/ipv4/ip_forward , so that every InternetNode is not
>> >> necessarily a router.
> >
> >What is the use-case for that ? Is it not enough to not create a
> >forwarding table for that node ? If not, I suspect that we could
> >trivially add:
> >
> >Ipv4::EnableForwarding ()
> >Ipv4::DisableForwarding ()
> >Ipv4::IsForwardingEnabled ()
> >
> >

Issue left for further study.

Comment 1 Tom Henderson 2009-02-16 00:58:29 EST

sliding to ns-3.5

Comment 2 Tom Henderson 2009-06-25 16:08:17 EDT

Created attachment 483 [details]
tested patch

Passing this to release mgr. for consideration; note that it extends the Ipv4 API.  I tested this on some example scripts; both the operation of the attribute and some of the underlying SetForwarding() calls.

Comment 3 Tom Henderson 2009-06-25 16:11:40 EDT

Created attachment 484 [details]
additional patch for global routing

This patch updates global routing to be aware of this interface state and disable the advertisement of links for which forwarding is disabled.  Should be applied on top of previous patch.

Comment 4 Mathieu Lacage 2009-06-26 04:57:25 EDT

changeset 0e15594f67f3