Description:
In scenarios where VNFs and PNFs are deployed on different machines within a network, it's crucial to maintain synchronization and ensure smooth operation. This commit addresses the need to keep the VNF slot ahead of 2 (as required for network deployment), while ensuring that PNFs wait appropriately to avoid overtaking the slot. Specifically, this involves measuring slot duration and implementing a wait mechanism to ensure that PNFs do not exceed a slot duration of 300 microseconds. This ensures proper coordination and timing between VNFs and PNFs, contributing to the overall stability and performance of the network deployment.

This reverts commit 697fa11fd12d8c0645c358de2ff2b3c360fff8be.

This reverts commit 318508f3.

This reverts commit a04bf851.

- Removed unused global variable `pnf_p7_recv`.
- Refactored `recv_and_save_pnf_p7` function to be inline within `pnf_p7_slot_ind`.
- Removed unnecessary computations and commented-out code related to slot shifting.
- Simplified slot buffer handling by removing redundant assignments and commented-out code.
- Renamed and organized function calls related to handling DL/UL TTI, DCI, and TX data requests.
- Cleaned up debug print statements and unused variables.

Description:
- Refactored the handling of PNF (Physical Network Function) and UE (User Equipment) RRC (Radio Resource Control) interactions.
- Removed commented-out code related to slot shift in PNF handling.
- Modified the logic to apply slot shift to the incoming sfn_sf in PNF handling.
- Adjusted the logic to correctly handle wrap-around when applying slot shift in PNF handling.
- Streamlined the processing of DL (Downlink) TTI (Transmission Time Interval) requests in PNF handling.
- Updated the handling of UL (Uplink) TTI requests in PNF handling.
- Enhanced the processing of UL DCI (Uplink Downlink Control Information) requests in PNF handling.
- Improved the handling of TX (Transmit) data requests in PNF handling.
- Modified logging levels in UE RRC handling for BCCH (Broadcast Control Channel) data indication.
- Added logging of BCCH BCH (Broadcast Channel) indication type in UE RRC handling.

Changes:
- Removed commented-out code related to slot shift.
- Modified PNF handling to correctly apply slot shift and handle wrap-around.
- Refactored DL and UL TTI request processing in PNF handling.
- Updated logging levels and added additional logging in UE RRC handling.

Files Modified:
- nfapi/oai_integration/nfapi_pnf.c
- nfapi/open-nFAPI/pnf/src/pnf_p7.c
- openair2/RRC/NR_UE/rrc_UE.c

Description:

Currently, the UE is consistently showing Harq round stats for Downlink as 0/0/0, indicating a problem with the downlink communication.
This issue is leading to an inability to ping the UE successfully.
Investigating and addressing the root cause of the Harq round stats being stuck at 0/0/0 is essential to restore proper downlink communication and enable successful pinging of the UE.
Once resolved, this fix will enhance the overall functionality and reliability of the system.

In the patch 71e254, an issue was identified where the number of slots in the TDD table was hardcoded inconsistently during packing and unpacking. This led to a mismatch in the number of slots processed, causing errors during execution.

Upon further investigation, it was found that the number of slots in the TDD table should be determined dynamically based on the value of mu, derived from the configured tdd-UL-DL-ConfigurationCommon.

To address this issue, this commit introduces dynamic allocation of memory for the TDD table during unpacking based on the value of mu. Additionally, the packing process now calculates the number of slots to pack dynamically, ensuring consistency between packing and unpacking.

This change resolves the discrepancy in the number of slots processed during packing and unpacking, ensuring that the PNF receives a complete and accurate TDD table.

This reverts commit 71e2546c.

This commit refactors the DL TTI request handling in the PNF P7 module and enhances code clarity.

Changes made:
- Updated the definition of `dl_tti_req` in the `nfapi_pnf_interface.h` header file to directly store the `nfapi_nr_dl_tti_request_t` structure instead of a pointer.
- Refactored DL TTI request processing in the `pnf_handle_dl_tti_request` function in `pnf_p7.c` to simplify the code structure and improve readability.
- Introduced a static function, `cp_nr_dl_tti_req`, to copy DL TTI request structures, improving code modularity and reducing duplication.
- Updated DL TTI request handling to utilize the refactored logic.
- Removed unnecessary deallocation of DL TTI request in error handling path to prevent double deallocation.

This commit refactors the UL TTI request handling in the PNF P7 module and improves the clarity of function logic.

Changes made:
- Refactored UL TTI request handling to simplify code structure and improve readability.
- Introduced a static function, `cp_nr_ul_tti_req`, to copy UL TTI request structures, improving code modularity and reducing duplication.
- Updated UL TTI request processing to utilize the refactored logic.
- Removed unnecessary deallocation of UL TTI request in error handling path to prevent double deallocation.

Additionally, the commit enhances comments and debug traces for better understanding of the code flow and debugging.

The function now utilizes simpler variable names and logic to determine if a given NR P7 request falls within the timing window.

The logic for determining if a request is within the timing window is as follows:
- The function calculates the absolute difference between the current and received SFN slots, taking into account the possibility of wraparound.
- If the absolute difference is greater than half of the maximum SFN slot value, it subtracts this difference from the maximum SFN slot value to get the actual difference.
- The function then checks if this difference is less than or equal to the specified timing window. If it is, the request is considered to be within the window.

Additionally, the commit updates the function signature to return a boolean value for better readability and consistency.

Changes made:
- Simplified variable names for readability
- Improved logic for handling wraparound scenarios
- Updated function signature to return a boolean value

- Process ul_dci whenever the number of ul_dci PDUs in the buffer > 0.
- Replace dynamic allocation on the heap with stack allocation.
- New function, **`cp_nr_ul_dci_req`**, to copy the **`ul_dci_req`**.

This reverts commit a05184e8.

This reverts commit f79d48e6.

This reverts commit 8201dc81.

Move the code "msgTx->num_ul_pdcch set to zero" out of the for-loop, even though it is always 0 or 1, the for-loop will execute at most once.

This reverts commit 251ebcf6.

- nr_fill_ulsch
- nr_fill_pucch
- nr_fill_prach

From VNF oai_nfapi_ul_tti_req to PNF save slot_buffer to tx_func nr_fill_pucch to rx_func nr_decode_pucch0 finish

from [VNF] oai_nfapi_dl_tti_req to [PNF] nr_schedule_dl_tti_req, including PDCCH, PDSCH, SSB, and CSI_RS.

The PMI needs to be zero.
The prgs_list index always should be zero.
prgs_list[0] needs to be zero. If iperf is used, it will abort, and the value will be unexpected, such as 5466, 37669, 5980.
The problem may lie in nFAPI pack/unpack.