Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Proper topology with both TCP and UDP apps sending packets. #7

Closed
wants to merge 36 commits into from
Closed

Proper topology with both TCP and UDP apps sending packets. #7

wants to merge 36 commits into from

Conversation

charliebarber
Copy link
Collaborator

@charliebarber charliebarber commented Mar 9, 2024 •
edited
Loading

Can you take a look at this and give a hand with setting up alternate links? Little bit confused by the queue index template.

Also getting a runtime error:

Creating Topology
FRRQueue:FRRQueue(0xaaab012c5020)
FRRQueue:FRRQueue(0xaaab012c5320)
FRRQueue:FRRQueue(0xaaab012c54a0)
FRRQueue:FRRQueue(0xaaab01404810)
FRRQueue:FRRQueue(0xaaab01405a50)
FRRQueue:FRRQueue(0xaaab01405fb0)
FRRQueue:FRRQueue(0xaaab01407230)
FRRQueue:FRRQueue(0xaaab014077a0)
FRRQueue:FRRQueue(0xaaab014089c0)
FRRQueue:FRRQueue(0xaaab01408f20)
FRRQueue:FRRQueue(0xaaab0140a340)
FRRQueue:FRRQueue(0xaaab0140a870)
FRRQueue:Enqueue(0xaaab012c5020, 0xaaab012be6e0)
Enqueued 0xaaab012be6e0 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab012c5020)
Popped 0xaaab012be6e0
FRRQueue:Enqueue(0xaaab012c5020, 0xaaab01417240)
Enqueued 0xaaab01417240 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab012c5020)
Popped 0xaaab01417240
FRRQueue:Enqueue(0xaaab012c54a0, 0xaaab01402eb0)
Enqueued 0xaaab01402eb0 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab012c54a0)
Popped 0xaaab01402eb0
FRRQueue:Enqueue(0xaaab012c5020, 0xaaab014085e0)
Enqueued 0xaaab014085e0 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab012c5020)
Popped 0xaaab014085e0
FRRQueue:Enqueue(0xaaab012c54a0, 0xaaab014042d0)
Enqueued 0xaaab014042d0 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab012c54a0)
Popped 0xaaab014042d0
FRRQueue:Enqueue(0xaaab014089c0, 0xaaab01389b40)
Enqueued 0xaaab01389b40 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab014089c0)
Popped 0xaaab01389b40
FRRQueue:Enqueue(0xaaab012c5020, 0xaaab0141aa80)
Enqueued 0xaaab0141aa80 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab012c5020)
Popped 0xaaab0141aa80
FRRQueue:Enqueue(0xaaab012c54a0, 0xaaab0141ab60)
Enqueued 0xaaab0141ab60 to 0 packets in queue.
FRRQueue:Dequeue(0xaaab012c54a0)
Popped 0xaaab0141ab60
FRRQueue:Enqueue(0xaaab014089c0, 0xaaab0141aaf0)
Congested Route, Rerouting packet: 0xaaab0141aaf0
NS_ASSERT failed, cond="m_ptr", msg="Attempted to dereference zero pointer", +2.035488000s 3 file=/usr/workspace/ns-allinone-3.41/ns-3.41/src/core/model/ptr.h, line=713
NS_FATAL, terminating
terminate called without an active exception
Command 'build/scratch/ns3.41-combined-default' died with <Signals.SIGABRT: 6>.

Otherwise this is the proper topology with the applications running

charliebarber and others added 15 commits March 4, 2024 16:36
@YousefEZ YousefEZ marked this pull request as draft March 12, 2024 01:12
@charliebarber charliebarber changed the title Congestion framework, sending UDP packets Proper topology with both TCP and UDP apps sending packets. Mar 12, 2024
YousefEZ
YousefEZ reviewed Mar 12, 2024
View reviewed changes
src/combined.cc Outdated
Comment on lines 204 to 212
// TODO: Need some help with setting alternate target
// setAlternateTarget<0>(devices01, getDevice<0>(devices02));
// setAlternateTarget<0>(devices02, getDevice<0>(devices01));

// setAlternateTarget<0>(devices12, getDevice<1>(devices01));
// setAlternateTarget<1>(devices01, getDevice<0>(devices12));

// setAlternateTarget<1>(devices02, getDevice<1>(devices12));
// setAlternateTarget<1>(devices12, getDevice<1>(devices02));
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs to be linked to the topology, and thats why the lfa policy is giving a seg fault. I will change it on my part to print but not abort the error

@charliebarber
Copy link
Collaborator Author

Closed due to relevant features implemented in other PR

@YousefEZ YousefEZ deleted the congestion-framework branch April 12, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YousefEZ YousefEZ YousefEZ left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@charliebarber @YousefEZ