Re: NS doesn't write CBR and rtProtoDV events in the same simulation(fwd)

[Lloyd Wood <l.wood@eim.surrey.ac.uk>]

Forwarding Liang's added multipath changes (below) to the list. Can we
fix per-node setting of multipath routing while we're at it?

The below patch still allows the traditional 

Node set multiPath_ 1

to work, as far as I can see with a test of my multipath DV networks,
so it shouldn't be harmful;

$node set multiPath_ 1 

should now also be working properly (I didn't test that, not having a
test scenario to hand, hint hint). It's just a question of setting
multiPath_ in the right place...

thanks,

L.

fix something, mail a patch and description to the list, move on.

<L.Wood@surrey.ac.uk>PGP<http://www.ee.surrey.ac.uk/Personal/L.Wood/>

minimal diff to patch ns-node.tcl v1.68 based on Liang's comments
below:

<PRE>

56a57
>         $self instvar multiPath_
71a73
>       set multiPath_ [$class set multiPath_]
591,592c593
<       $self instvar multiPath_ routes_ rtObject_
<       set multiPath_ [$class set multiPath_]
---
>       $self instvar routes_ rtObject_

</PRE>


---------- Forwarded message ----------
Date: Mon, 14 Feb 2000 13:49:47 -0500 (EST)
From: "Guo, Liang" <guol@cs.bu.edu>
To: Lloyd Wood <l.wood@eim.surrey.ac.uk>
Subject: Re: NS doesn't write CBR and rtProtoDV events in the same simulation

On Mon, 14 Feb 2000, Lloyd Wood wrote:

> Correct me if I'm wrong, but didn't you have to make changes to ns to
> get per-node multipath (for some nodes only) working properly?
> 
> I'm just wondering if any changes to CVS are required there...
> 
> thanks,


Yes, you are right, I forgot to put this into the modification:

in ns/tcl/lib/ns-node.tcl, you need to do the following things:

1. in line 53 

Node instproc init args {
  ...
  ###     added by Liang Guo on 11/09/99
        $self instvar multiPath_
 
  ...

        $self cmd addr $address_; # new by tomh

### added by Liang Guo, for "I-only-want-it-partially" setting

        set multiPath_ [$class set multiPath_]
}


2. in line 590, 

#
# Node support for detailed dynamic routing
#
Node instproc init-routing rtObject {
#### Modified by Liang Guo on 11/08/99, for "I-only-want-it-partially"
        $self instvar routes_ rtObject_
#       $self instvar multiPath_  routes_ rtObject_

#### The following two lines makes no sense, multiPath_ should be
####     set individually for each node, so we can't reset it to
####     the class default value, commented out by Liang Guo 11/09/99
#       set multiPath_ [$class set multiPath_]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     Comment this line out!



I think that will work. Thanks Lloyd for pointing this out.

Liang.