Project

General

Profile

Task #1209

Task #1205: NDNLP

NDNLP fragmentation and reassembly on Ethernet multicast face

Added by Junxiao Shi over 5 years ago. Updated almost 5 years ago.

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

100%

Estimated time:
6.00 h

Description

Add fragmentation in Ethernet sending path.
Each EthernetFace needs a single ndnlp::Slicer.

Add reassembly in Ethernet receiving path.
EthernetFace must distinguish the sender address, and create one ndnlp::PartialMessageStore per sender.
If a sender has been idle for some configuration duration, destroy its PartialMessageStore.

Modify link layer byte counters to reflect NDNLP byte counts.
In the receiving path, the byte counter should include all received NDNLP packets, even if a network layer packet could not be reassembled due to a lost fragment.


Related issues

Related to NFD - Task #1191: EthernetFace implementationClosed

History

#1 Updated by Junxiao Shi over 5 years ago

  • Parent task set to #1205

#2 Updated by Junxiao Shi over 5 years ago

  • Subject changed from NDNLP service on datagram channel to NDNLP fragmentation and reassembly on Ethernet multicast face
  • Description updated (diff)
  • Assignee set to Davide Pesavento

#3 Updated by Junxiao Shi over 5 years ago

  • Assignee deleted (Davide Pesavento)
  • Target version changed from v0.1 to v0.2

#4 Updated by Davide Pesavento over 5 years ago

  • Assignee set to Davide Pesavento

#5 Updated by Junxiao Shi over 5 years ago

  • Target version deleted (v0.2)

20140514 call decides it's not urgent because nobody is blocked on this feature.

#6 Updated by Junxiao Shi about 5 years ago

  • Blocks Task #1780: Link layer byte count in EthernetFace added

#7 Updated by Junxiao Shi about 5 years ago

  • Blocks deleted (Task #1780: Link layer byte count in EthernetFace)

#8 Updated by Junxiao Shi about 5 years ago

  • Description updated (diff)

#9 Updated by Junxiao Shi about 5 years ago

  • Target version set to v0.3

20140910 conference call decides to include this feature in v0.3

#10 Updated by Davide Pesavento about 5 years ago

Sounds good to me. Either Matteo or I will start working on this task as soon as possible.

#11 Updated by Davide Pesavento about 5 years ago

  • Assignee changed from Davide Pesavento to Matteo Sammarco

#12 Updated by Davide Pesavento almost 5 years ago

  • Status changed from New to In Progress

What's the relationship between ndnlp::Slicer and the face? Should we create a new instance of Slicer for each packet that has to be fragmented, or should there be just one instance, member of EthernetFace, with the same lifetime as the face object?

#13 Updated by Junxiao Shi almost 5 years ago

Each EthernetFace needs a single ndnlp::Slicer.

#14 Updated by Davide Pesavento almost 5 years ago

  • % Done changed from 0 to 20

#15 Updated by Davide Pesavento almost 5 years ago

We will resume working on this task next week.

#16 Updated by Matteo Sammarco almost 5 years ago

  • % Done changed from 20 to 30

#17 Updated by Matteo Sammarco almost 5 years ago

  • % Done changed from 30 to 40

#18 Updated by Davide Pesavento almost 5 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 40 to 70

#19 Updated by Junxiao Shi almost 5 years ago

Technically this feature conflicts with #2222, but I'll wait for this to merge before start implementation of #2222.

#20 Updated by Matteo Sammarco almost 5 years ago

Hello Junxiao,
I didn't get you last update since on gerrit you wrote "It needs to go into ndn-cxx. This Change is blocked until then."
Thanks.

#21 Updated by Junxiao Shi almost 5 years ago

I didn't get you last update since on gerrit you wrote "It needs to go into ndn-cxx. This Change is blocked until then."

Please commit std::hash<ndn::Ethernet::Address> specialization code into ndn-cxx repository.

#22 Updated by Davide Pesavento almost 5 years ago

  • % Done changed from 70 to 100

#23 Updated by Junxiao Shi almost 5 years ago

  • Status changed from Code review to Closed

Also available in: Atom PDF