Project

General

Profile

Task #1281

Close idle DatagramFace

Added by Junxiao Shi over 6 years ago. Updated over 6 years ago.

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

100%

Estimated time:
4.00 h

Description

If nothing is received on an accepted unicast DatagramFace for at least a configurable duration, close this face.


Related issues

Follows NFD - Task #1189: UdpFace implementationClosedGiulio Grassi

Actions
#1

Updated by Junxiao Shi over 6 years ago

  • Description updated (diff)

In ccnd 0.7.2 this is implemented as:

  1. If a face is initiated by this node (not accepted), set a PERMANENT flag.
  2. Every time a packet is received, increment the recently received packets counter.
  3. Periodically, enumerate all faces, delete face if it does not have PERMANENT flag and the recently received packets counter is zero. Also clear the recently received packets counter on all faces.

Therefore, an accepted face that receives zero packets between two periodical procedure executions would be deleted.

#2

Updated by Giulio Grassi over 6 years ago

  • % Done changed from 0 to 10

Do we need the "recently received packets counter" for other purpose different from this task? Do we need the information about the number of the recently received packets for something else? Otherwise I will simply use a bool instead of a counter, since the only thing that matters is if n_receivedPacket > 0.

Should sending operation extend the lifetime of the face?

#3

Updated by Junxiao Shi over 6 years ago

We don't need the counter for other propose.

Only receive operation extends lifetime.

Lifetime control only applies to accepted unicast faces. You may need a flag on the DatagramFace to indicate whether it's accepted.

If FaceMgmt is called to create the same face as an accepted face, this flag should be unset.

#4

Updated by Giulio Grassi over 6 years ago

  • Status changed from New to Code review
  • % Done changed from 10 to 100
#5

Updated by Giulio Grassi over 6 years ago

  • Status changed from Code review to Closed

Also available in: Atom PDF