Project

General

Profile

Task #4887

ASF Strategy should estimate RTO using TCP’s algorithm (RFC6298) instead of NFD’s RtoEstimator

Added by Ashlesh Gawande about 1 year ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Category:
Forwarding
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

#1

Updated by Davide Pesavento about 1 year ago

We have 3 different implementations of RttEstimator in 3 different places (ndn-cxx, ndn-tools, NFD). We should consolidate them.

For this task, I suggest fixing/improving the ndn-cxx implementation and then using that one in both NFD and ndncatchunks.

#2

Updated by Ernest McCracken about 1 year ago

ndn-tools and ndn-cxx implementations are exactly the same except that for ndn-tools has an afterMeasurement signal.

#3

Updated by Davide Pesavento about 1 year ago

Any progress here?

#4

Updated by Ernest McCracken about 1 year ago

  • Status changed from New to In Progress

Yes. I will open a gerrit code review soon. We fixed it by having Asf strat use ndn-cxx's rto estimator. We don't need any fixes to ndn-cxx's rto estimator but I will have to check to see what all touches NFD's rto estimator and what changes might be needed there

#5

Updated by Davide Pesavento 12 months ago

  • Status changed from In Progress to Code review
  • Target version set to v0.7
#6

Updated by Davide Pesavento 11 months ago

  • % Done changed from 0 to 60
#7

Updated by Davide Pesavento 10 months ago

  • % Done changed from 60 to 80
#8

Updated by Junxiao Shi 10 months ago

For AccessStrategy, it needed a state-copy operator, so that per-prefix RTT estimator can inherit from per-face RTT estimator state. The rationale is explained in NFD devguide.
If this operator cannot be supported, it's okay to drop it, but strategy version number must be incremented as this represents a behavior change.

#9

Updated by Davide Pesavento 10 months ago

  • Tracker changed from Feature to Task
  • Status changed from Code review to Closed
  • % Done changed from 80 to 100

Also available in: Atom PDF