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.

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

Issue 15: mention checksums in hybrid simulations #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

teto
Copy link
Member

@teto teto commented Jan 18, 2016

In case of hybrid simulations Kernel drops packets (e.g. sends RST) if ns3 checksums are disabled.
Add a mention in documentation.

I am not sure it's the best place to put the mention ?

== EDIT
Hybrid simulations refer to simulations involving both ns and linux or BSD stacks. I would like to raise in the doc points of importance to get reasonable results.

  • ns Checksums need to be enabled
  • ns3 callbacks don't get called Properly relay ns3 callbacks #24
  • List ns attributes that need to change (Dunno if an ns3 page exist, for instance ns3 and linux TCP RTOs differ).

@thehajime
Copy link
Member

thanks for the PR.

I am not sure it's the best place to put the mention ?

I think it can be documented in the following file, either inside the limitation section or new one.

https://github.com/direct-code-execution/ns-3-dce/blob/master/doc/source/dce-user-tech.rst

In case of hybrid simulations Kernel drops packets (e.g. sends RST) if ns3 checksums are disabled.
Add a mention in documentation.
@teto
Copy link
Member Author

teto commented Jan 31, 2016

In fact I believe it makes sense to have an hybrid section because there are several tips/tricks onbe should be aware of such as :
#24
and also the routing interface is not unified which is a bit annoying.

@thehajime
Copy link
Member

sorry, I don't understand what a hybrid section is. care to elaborate ?

@teto
Copy link
Member Author

teto commented Feb 2, 2016

I am thinking of scenarios involving both ns3 and linux or BSD stacks. If
you can come up with a better name :)

2016-02-03 0:56 GMT+01:00 Hajime Tazaki [email protected]:

sorry, I don't understand what a hybrid section is. care to elaborate ?


Reply to this email directly or view it on GitHub
#31 (comment)
.

@thehajime
Copy link
Member

okay.

I think for the moment we can fix one-by-one: once we come up with the nice text then we can create a new section and put this patch to the section. is it fine for you ?

@teto
Copy link
Member Author

teto commented Feb 3, 2016

ok I'll add items to the section (dunno when though) and you deem it ready, we can merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants