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

adds features and fix some bugs #6

Open
wants to merge 45 commits into
base: master
Choose a base branch
from
Open

adds features and fix some bugs #6

wants to merge 45 commits into from

Conversation

yleintel
Copy link

This pull request enables the following features:

  1. support two-tier topologies;
  2. support various packet size;
  3. support taking the NS3 flow traces as the input.
  4. fix various bugs when worst case happens, i.e., control packets get dropped.
  5. using packet size instead of MTU to set pace time: _packet_drain_time.

yfle0707 and others added 30 commits December 23, 2022 16:47
@yfle0707
add 2-tier network topology and WebSearch workloads as the inputs
ca3b019
@yfle0707
fix the bugs when copy files that works for old versions of ndp, but …
fdddde0 
…does not work for the latest version
@yfle0707
add the support for packet size less than MTU and validate the RTS fu…
f0d57c9 
…nctions
@yfle0707
add notes to README
54eafe4
@yfle0707
gitignore
8f75e74
@yfle0707
fix the bugs when support the packet with size < MTU
76c1e87
@yfle0707
change the configuration to be a 100G NIC, 400G inter-switch links, l…
ee91625 
…ink delay as 1us.
@yfle0707
add debugging statements to understand the code.
76a4b06
@yleintel
the code can be compiled in MacOs, but not linux; add the compiling i…
06e685d 
…ssue on the ubuntu;
@yleintel
remove random drops, because it assumes that the packet size in the n…
4237cb8 
…etwork are exactly the same.
@yleintel
fix a bug to support packet size or message size less than MTU.
84e103d
@yleintel
add compilation script
7e28079
@yleintel
revert to the origianl NDP queue size
93dbfac
@yleintel
add the same output as the ns3 for fct
e6f99d3
@yleintel
support single path
d4d5b3d
@yleintel
adds code to support single path
5191757
@yleintel
set max simulation time to be 4 sec
28afc03
@yleintel
time scale of fct and ideal fct is different, fix it
2cffab5
@yleintel
1. fix the ReadInput exactly same as ns3; 2. leafspineTopology's ToR …
2baedd0 
…to server queue is wrong, fixed it; 3. the network topology is 10G
@yleintel
update the configuration to be 100Gbps
157de27
@yleintel
fix a bug on the pacer constructor
ca6a2a7
@yleintel
1. fix the name in leafspine topology; 2. fix the reverse route to ov…
3996278 
…ercome the limit that two nodes can not have more than 2 connections;
@yleintel
1. fix a bug when support with different size of packets; 2. it is po…
ce2e948 
…ssible that a PULL packet is inflight, a message finishes and a new message uses that previous connection, which might receive the old PULL packet; as pull packet has cum ack, this can make the new message's state incorrect; thus, we would reuse the old connection only when the connection has been idled for 1ms
@yleintel
only NDP data packets are allowed to be bounced.
45e267b
@yleintel
fix mtu input
0ba21d0
@yleintel
fix a bug mentioned in the ndp.cpp
751534c
@yleintel
fix the NdpSrc::pull_packets
bb3a46d
@yleintel
add simulation finish mark
0ae58c5
@yleintel
enable timeout retransmission; fix the single_path configuration bug.
aa99af1
@yleintel
update
6141428
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@yleintel @yfle0707