Skip to content

Improvements to STP/RSTP#459

Open
ManiAm wants to merge 27 commits intoinet-framework:masterfrom
ManiAm:stp
Open

Improvements to STP/RSTP#459
ManiAm wants to merge 27 commits intoinet-framework:masterfrom
ManiAm:stp

Conversation

@ManiAm
Copy link
Contributor

@ManiAm ManiAm commented Sep 15, 2019

@ZoltanBojthe
Copy link
Contributor

I try to run fingerprint tests for STP/RSTP simulations: (on topic/ManiAm-stp branch in inet repo)

These simulations generates errors:

  • /examples/ieee8021d/ -f omnetpp.ini -c STP-LargeNet -r 0 (runtime error with exitcode=1: <!> Error: Logic error: frame BPDU from higher layer has local MAC address as dest (AA-AA-AA-00-00-02) -- in module (inet::EtherMac) LargeNet.switch2.eth[3].mac (id=139), at t=79.000001348s, event #57894)
  • /examples/ieee8021d/ -f omnetpp.ini -c STP-LinkReconnect -r 0 (runtime error with exitcode=1: <!> Error: Logic error: frame BPDU from higher layer has local MAC address as dest (AA-AA-AA-00-00-03) -- in module (inet::EtherMac) LargeNet.switch3.eth[2].mac (id=178), at t=79.000001348s, event #59736)

All other STP/RSTP simulation reports FAILED fingerprint values

@ManiAm
Copy link
Contributor Author

ManiAm commented Oct 14, 2019

Ok, I am going to take a look at it.

@ManiAm
Copy link
Contributor Author

ManiAm commented Oct 28, 2019

Still haven't found any free time to look into this.

I think the failed fingerprint is expected. In the original INET code, each switch is using the MAC address of its first Ethernet interface as its Switch MAC. I do not know what is the logic/rationale behind this, but in almost all implementations, each switch has a base MAC port that is unique. When you power up a switch, you will see its base MAC address getting printed on the console. In my PL, I have added the base MAC port concept and each switch is using it for communication. Hence the change in fingerprint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants