-
Notifications
You must be signed in to change notification settings - Fork 29
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
Fix downlink FAR applyaction #37
Fix downlink FAR applyaction #37
Conversation
@badhrinathpa @thakurajayL @krsna1729... Please review the patch |
@@ -523,11 +523,11 @@ func (dataPath *DataPath) ActivateTunnelAndPDR(smContext *SMContext, precedence | |||
nextDLTunnel := nextDLDest.DownLinkTunnel | |||
|
|||
DLFAR.ApplyAction = pfcpType.ApplyAction{ | |||
Buff: false, | |||
Buff: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why buffer true during establishment req ? Can we not keep it drop ? and during this time when UE has not sent any packet do we expect downlink packet to flow ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@deepaks2 : When is the buffering action is set back to Forward ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The buffering action rule is created by the SMF and associated with the FAR in order to apply a specific
buffering behavior for DL packets requested to be buffered, as described in clause 5.8.3 and clause 5.2.4
in TS 29.244 [65].
-
buffer is enabled in downlink packets by setting the BUFF flag and by optionally provisioning buffering parameters providing instructions on how to buffer the packets; (5.2.3 Forwarding Action Rule Handling)
-
5GC supports buffering of UE's data packets for deactivated PDU Sessions and UP tunnel management
(more details in 5.8.3 Explicit Buffer Management (in TS 29.244 [65].)) -
The buffering is mandatory in UPF but optional in SMF.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@deepaks2 : Since PFCP Modification request shall set action to forward and disable buffering, approving the pull request.
retest this please |
Thanks @UpadhyayaAnkur. I will re-test it |
…abilisment Signed-off-by: Deepak S <deepak.s@intel.com>
d299abb
to
cb3acff
Compare
retest this please |
@UpadhyayaAnkur which file is giving the FOSSA issue? Does it matter when not creating new files? |
Looks to be Fossa server issue. Observing timeout, nothing wrong with the changes. I have already approved code, please proceed with merge |
Set DownLink FAR applyAction to buffer & nocp during PFCP session establishment