Skip to content

F1 NR-DC#254

Open
CedricRouxEurecom wants to merge 19 commits into
duranta-project:developfrom
CedricRouxEurecom:f1-nrdc
Open

F1 NR-DC#254
CedricRouxEurecom wants to merge 19 commits into
duranta-project:developfrom
CedricRouxEurecom:f1-nrdc

Conversation

@CedricRouxEurecom

@CedricRouxEurecom CedricRouxEurecom commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This is an F1AP-based NR-DC implementation.

Refer to commits for more details.

Anything can be changed according to the reviews, it was done over a long period of time, some code from six months ago may be bad now, some choices may be bad.

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 1, 2026 •
edited
Loading

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

@durantabot

durantabot commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

CI Build: #588 | Not performing CI due to the absence of one of the following mandatory labels:

  • documentation Documentation-related changes or updates. (don't perform any stages)
  • BUILD-ONLY Execute build stages only for code improvements with no impact on 4G/5G functionality. (execute only build stages)
  • 4G-LTE Perform 4G Tests (perform 4G tests)
  • 5G-NR Perform 5G Tests (perform 5G tests)
  • nrUE Run only 5G UE tests (physims included, LDPC tests excluded). (perform only 5G-UE related tests including physims excluding LDPC tests)
  • CI Perform both 4G and 5G tests (perform both 4G and 5G tests)

@CedricRouxEurecom

CedricRouxEurecom commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

i can't add a label, does not work

@sgarg00 sgarg00 added the 5G-NR Perform 5G Tests label Jul 1, 2026
@rorsc rorsc self-requested a review July 1, 2026 08:53
@rorsc rorsc changed the title F1 nrdc F1 NR-DC Jul 1, 2026
@CedricRouxEurecom

CedricRouxEurecom commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

so i can't add a label myslef?

@CedricRouxEurecom

CedricRouxEurecom commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

github is a very nice website, me feels i will love work with this

@durantabot

durantabot commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

CI Build: #591 | Failed on the following stages:

@durantabot

durantabot commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

CI Build: #592 | Failed on the following stages:

@CedricRouxEurecom

CedricRouxEurecom commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

i have no idea what's wrong with OAI-CN5G-COTS-UE-Test, sorry I can't do anything to fix

@sgarg00

sgarg00 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

i have no idea what's wrong with OAI-CN5G-COTS-UE-Test, sorry I can't do anything to fix

The cots-ue is disabled currently as Dogmatix is dead. We will replace it soon.

@durantabot

durantabot commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

CI Build: #596 | Failed on the following stages:

@CedricRouxEurecom

CedricRouxEurecom commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

by the way i can't add "milestone", this wonderful website does not give me a clickable zone in the right "milestone" display stuff

to me it's on "review can start" state

if i need to configure something to have the clickable something, please tell me

i love github

@sgarg00 sgarg00 added this to the REVIEW_CAN_START milestone Jul 2, 2026
@sgarg00

sgarg00 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

by the way i can't add "milestone", this wonderful website does not give me a clickable zone in the right "milestone" display stuff

to me it's on "review can start" state

if i need to configure something to have the clickable something, please tell me

i love github

We have sent an invite to you as a collaborator

@CedricRouxEurecom CedricRouxEurecom force-pushed the f1-nrdc branch 2 times, most recently from b78637d to 88cde7a Compare July 7, 2026 08:21
@durantabot

durantabot commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

CI Build: #692 | Failed on the following stages:

@durantabot

durantabot commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

CI Build: #693 | Failed on the following stages:

@CedricRouxEurecom
simple reference counting shared pointer
dbb94d7 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
time_manager based timer/timeout
2a25e05 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
cleanup MAX_MEAS_XX
ab404a5 
Those values exist in both LTE and NR, let's rename to NR_MAX_MEAS_XX and make
them known to NR-UE only, they are not used in the gNB.

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
add a module to properly allocate measurement IDs
e1914a0 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
first step of NR-DC: ask for UE capabilities to check for NR-DC support
35ef016 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
add a timeout mechanism for NR-DC processing
72afe2b 
nr_ulsim and nr_dlsim need time_management_core to compile

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
second step of NR-DC: receive UE capabilities, configure measurements
f8597d4 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: CU sends F1 UE Context Setup Request
c9ac9f2 
send F1 UE Context Setup Request to DU when getting measurement from UE
to transfer DRB 1 (hardcoded for the moment, maybe change in the future)

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: DU handles F1 UE Context Setup Request
12c0f47 
- create the UE
- send F1 UE Context Setup Response to the CU

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: transfer bearer to secondary DU
b608a8e 
Plus some modifications here and there.

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: handle PDU session release
cb5e337 
Samsung S25 does a PDU session release request when we do "airplane mode",
we have to deal with it (if not: crash of DU).

Code is not clean.

According to 38.331, it's forbidden to have SRB2 without DRB. So when the
UE asks to remove the last DRB present, we should do what? RRRelease?

With S25 in standard SA mode, we remove DRB + RLC entity + forward PDU
session release coming from core network, all of this in one RRC
Reconfiguration. Trying to do the same with NR-DC miserably fails. If I
understand 38.331 correctly, to remove the RLC that lives in the SCG DU
I have to embed an mrdc-SecondaryCellGroupConfig into the RRC Reconfiguration
one. This mrdc-SecondaryCellGroupConfig contains an RRC Reconfiguration
that contains a secondaryCellGroup. This secondaryCellGroup contains
rlc-BearerToReleaseList with the LCID of the RLC entity to remove.
This is to mimic the SA mode. But it fails. So instead, we only send
the core network PDU session release. Then the S25 is happy and follows
with a Deregistration.

But testing with a quectel module (without quectel-CM, only minicom), and
doing:
    at+cgact=1,1     (activate a PDU session)
    at+cgact=0,1     (release it)
    at+cgact=1,1     (activate again)
fails for the second activation. When the quectel module gets the RRC
Reconfiguration that activates the second PDU session, it rejects it and
does RRC Reestablishment. Probably because we did not release the DRB
and the RLC entity in the previous step.

So be it for the time being, act/deact/act is not supported.

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: handle UE context release
9010883 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: handle SCG Failure Information
da5b7aa 
When the UE sends an SCG Failure Information, we remove the NR-DC
setup and put back the SCG DRB to the MCG.

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: release the UE only when it's configured
d8270b0 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: configuration in conf file
eac6c41 
add this block (at top, outside of any other block) to the CU
configuration file:

nrdc = {
  combinations = (
    { mcg = 77, scg = 261 },
    { mcg = 78, scg = 257 },
  )
}

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: get NR-DC parameters from configuration file
3fc1cee 
All configured NR-DC combinations are searched, the first one that
is possible is chosen. A combination is possible if the MCG cell is
the one where the UE is connected and the SCG cell exists (a DU
using the SCG band is connected to the CU).

Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: check band combinations in UE capabilities
60c2481 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
NR-DC: add documentation
14276ac 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@CedricRouxEurecom
fix compilation of nr_dlbench
198bd44 
Signed-off-by: Cedric Roux <cedric.roux@eurecom.fr>
@durantabot

durantabot commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

CI Build: #801 | Failed on the following stages:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rorsc rorsc Awaiting requested review from rorsc

Assignees

@CedricRouxEurecom CedricRouxEurecom

Labels

5G-NR Perform 5G Tests

Projects

None yet

Milestone

REVIEW_CAN_START

Development

Successfully merging this pull request may close these issues.

3 participants

@CedricRouxEurecom @durantabot @sgarg00