Skip to content
Snippets Groups Projects

T2: minor fix for 4x4 processing

Merged Jaroslava Fiedlerova requested to merge t2-fix-4x4 into develop
All threads resolved!
  • correct max number of LDPC segments for T2 processing
  • use pointer arithmetics in retrieve_ldpc_enc_op() in T2 lib to optimize loop processing

Comparison of processing time measured in nr_dlsim for LDPC encoding performed by CPU, T2 and T2 with this MR.

testcase develop (8 cores) develop with T2 this MR with T2
SNR = 30, MCS = 5, 106 PRBs, 1 layer 184 us 37 us 31 us
SNR = 30, MCS = 15, 106 PRBs, 1 layer 243 us 61 us 48 us
SNR = 30, MCS = 25, 106 PRBs, 1 layer 248 us 89 us 69 us
SNR = 30, MCS = 5, 273 PRBs, 1 layer 256 us 68 us 53 us
SNR = 30, MCS = 15, 273 PRBs, 1 layer 308 us 135 us 101 us
SNR = 30, MCS = 25, 273 PRBs, 1 layer 337 us 202 us 153 us
SNR = 30, MCS = 5, 273 PRBs, 2 layers 334 us 126 us 125 us
SNR = 30, MCS = 15, 273 PRBs, 2 layers 365 us 246 us 181 us
SNR = 30, MCS = 25, 273 PRBs, 2 layers 394 us 386 us 273 us
SNR = 30, MCS = 5, 273 PRBs, 2 layers, 4x4 349 us 131 us 98 us
SNR = 30, MCS = 15, 273 PRBs, 2 layers, 4x4 355 us 250 us 186 us
SNR = 30, MCS = 25, 273 PRBs, 2 layers, 4x4 388 us 377 us 279 us
Edited by Jaroslava Fiedlerova

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Robert SchmidtRobert Schmidt 3 months ago (Dec 3, 2024 3:10pm UTC)

Merge details

  • Changes merged into develop with c942fe43.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Robert Schmidt
  • Robert Schmidt
  • Robert Schmidt
  • added 2 commits

    • 0021b696 - Optimize memory usage in decode_offload function
    • 0c354dbc - T2: Rework retrieve_ldpc_enc_op()

    Compare with previous version

  • added 1 commit

    • 2f278395 - T2: Rework retrieve_ldpc_enc_op()

    Compare with previous version

  • Robert Schmidt resolved all threads

    resolved all threads

  • requested review from @lthomas

  • added 5G-NR label

  • changed milestone to %REVIEW_CAN_START

  • Jaroslava Fiedlerova changed the description

    changed the description

  • Jaroslava Fiedlerova marked this merge request as draft

    marked this merge request as draft

  • Jaroslava Fiedlerova marked this merge request as ready

    marked this merge request as ready

  • beurdouc mentioned in merge request !2952 (merged)

    mentioned in merge request !2952 (merged)

  • changed milestone to %OK_TO_BE_MERGED

  • Robert Schmidt mentioned in commit a8081914

    mentioned in commit a8081914

  • Robert Schmidt mentioned in merge request !3134 (merged)

    mentioned in merge request !3134 (merged)

  • Robert Schmidt mentioned in commit c942fe43

    mentioned in commit c942fe43

  • Robert Schmidt merged with merge request !3134 (merged)

    merged with merge request !3134 (merged)

  • Please register or sign in to reply
    Loading