fix(amf): Added Service Accept handling in case of Service Request with uplink data status#15330
Merged
fix(amf): Added Service Accept handling in case of Service Request with uplink data status#15330
Conversation
Contributor
|
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
Contributor
Bazel unit-test results 1 files ± 0 86 suites - 146 3m 13s ⏱️ -45s Results for commit 51fd05e. ± Comparison against base commit dd43cff. This pull request removes 314 tests.♻️ This comment has been updated with latest results. |
Contributor
Bazel unit-test results --config=production 1 files ± 0 86 suites - 43 3m 9s ⏱️ -3s Results for commit 51fd05e. ± Comparison against base commit dd43cff. This pull request removes 193 tests.♻️ This comment has been updated with latest results. |
Contributor
Bazel unit-test results --config=asan 1 files ± 0 86 suites - 43 3m 7s ⏱️ -27s Results for commit 51fd05e. ± Comparison against base commit dd43cff. This pull request removes 193 tests.♻️ This comment has been updated with latest results. |
Akshayp77
force-pushed
the
service_accept_handling
branch
from
6361f2e to
0f6b6c0
Compare
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Akshayp77
force-pushed
the
service_accept_handling
branch
from
0f6b6c0 to
9c616bf
Compare
Akshayp77
force-pushed
the
service_accept_handling
branch
from
9c616bf to
4d315eb
Compare
…th uplink data status Signed-off-by: Akshayp77 <[email protected]>
Akshayp77
force-pushed
the
service_accept_handling
branch
from
f5f2343 to
11a58da
Compare
lucasgonze
pushed a commit
to lucasgonze/magma
that referenced
this pull request
Feb 29, 2024
…th uplink data status (magma#15330) Signed-off-by: Akshayp77 <[email protected]> Co-authored-by: panyogesh <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Added Service Accept code In case If the Uplink data status IE is included in the Service Request message.
Test Plan
Tested with Ue-Ran-Sim.
Steps followed:
-Bring up 5g call flow.
-Restart Gnb.
Additional Information
As per the 3gpp spec 24.501
If the Uplink data status IE is included in the SERVICE REQUEST message and the UE is: a) not in NB-N1 mode; or b) in NB-N1 mode and the UE does not indicate a request to have user-plane resources established for a number of PDU sessions that exceeds the UE's maximum number of supported user-plane resources; the AMF shall: a) indicate the SMF to re-establish the user-plane resources for the corresponding PDU sessions; b) include the PDU session reactivation result IE in the SERVICE ACCEPT message to indicate the user-plane resources re-establishment result of the PDU sessions for which the UE requested to re-establish the user-plane resources; and c) determine the UE presence in LADN service area and forward the UE presence in LADN service area towards the SMF, if the corresponding PDU session is a PDU session for LADN.
Packet2610.zip
Security Considerations