Project

General

Profile

Actions

Task #1209

closed

Task #1205: NDNLP

NDNLP fragmentation and reassembly on Ethernet multicast face

Added by Junxiao Shi about 10 years ago. Updated over 9 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 1 (0 open1 closed)

Related to NFD - Task #1191: EthernetFace implementationClosedDavide Pesavento

Actions
Actions #1

Updated by Junxiao Shi about 10 years ago

  • Parent task set to #1205
Actions #2

Updated by Junxiao Shi about 10 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
Actions #3

Updated by Junxiao Shi about 10 years ago

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

Updated by Davide Pesavento about 10 years ago

  • Assignee set to Davide Pesavento
Actions #5

Updated by Junxiao Shi almost 10 years ago

  • Target version deleted (v0.2)

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

Actions #6

Updated by Junxiao Shi almost 10 years ago

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

Updated by Junxiao Shi over 9 years ago

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

Updated by Junxiao Shi over 9 years ago

  • Description updated (diff)
Actions #9

Updated by Junxiao Shi over 9 years ago

  • Target version set to v0.3

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

Actions #10

Updated by Davide Pesavento over 9 years ago

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

Actions #11

Updated by Davide Pesavento over 9 years ago

  • Assignee changed from Davide Pesavento to Matteo Sammarco
Actions #12

Updated by Davide Pesavento over 9 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?

Actions #13

Updated by Junxiao Shi over 9 years ago

Each EthernetFace needs a single ndnlp::Slicer.

Actions #14

Updated by Davide Pesavento over 9 years ago

  • % Done changed from 0 to 20
Actions #15

Updated by Davide Pesavento over 9 years ago

We will resume working on this task next week.

Actions #16

Updated by Matteo Sammarco over 9 years ago

  • % Done changed from 20 to 30
Actions #17

Updated by Matteo Sammarco over 9 years ago

  • % Done changed from 30 to 40
Actions #18

Updated by Davide Pesavento over 9 years ago

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

Updated by Junxiao Shi over 9 years ago

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

Actions #20

Updated by Matteo Sammarco over 9 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.

Actions #21

Updated by Junxiao Shi over 9 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.

Actions #22

Updated by Davide Pesavento over 9 years ago

  • % Done changed from 70 to 100
Actions #23

Updated by Junxiao Shi over 9 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF