RDKCOM-5580: RDKBDEV-3433 RDKBWIFI-202 Implementation of Traffic Separation

[siddharth-nair-dtdl]

Reason for change: Added traffic seperation TLV implementation via single bridge
Test Procedure: Verify build is successfull and check if traffic seperation is functional
Dependency: rdkcentral/utopia#263
Risks: Medium
Priority: P2

[Copilot]

Pull request overview

Implements EasyMesh “Traffic Separation” policy support end-to-end in the webconfig pipeline, including schema additions, JSON encode/decode, EasyMesh translation, and applying resulting VLAN/SSID settings to VAP configuration.

Changes:

• Added traffic_separation_policy structures to em_config_t.
• Implemented JSON encoding/decoding for “Traffic Separation Policy” in the webconfig EM policy object.
• Added translation from EasyMesh policy to internal EM config and applied VLAN/SSID values via a new EM subdoc apply path.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 14 comments.

Show a summary per file

File	Description
source/webconfig/wifi_encoder.c	Encodes Traffic Separation policy into the EM “Policy” JSON.
source/webconfig/wifi_decoder.c	Decodes Traffic Separation policy from EM “Policy” JSON into em_config_t.
source/webconfig/wifi_easymesh_translator.c	Copies Traffic Separation policy from EasyMesh structures into internal policy config.
source/core/wifi_ctrl_webconfig.c	Adds EM subdoc apply handler that updates VAP SSID/VLAN and syscfg + service restart.
include/wifi_base.h	Introduces Traffic Separation policy structs/limits and wires them into em_config_t.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sundaresh-k-oss]

Hi @amarnathhullur -We have made the changes suggested and replied to other comments.

    We have built the latest changes and OneWifi is crashing. After Factory reset, the box is recovered and OneWifi is not getting crashed (in some scenarios). The box is not consistent with each reboot. 

     For traffic separation TLV parsing, the agent is not getting parsed traffic separation properly. The issue is due to the vendor specific tlv parsing done in the file: src/em/policy_cfg/em_policy_cfg.cpp. We have commented the changes for vendor specific tlv and issue was resolved. That is due to adding the break as the vendor_tlv->num is zero in this case, control is coming out of while loop. Because of this, the control is coming out of TLV and some of the lines are not executed. 

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 8 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Nikita-Hakai]

Hi @siddharth-nair-dtdl, can you confirm if the copilot comments are addressed.

[AkhilaReddyK7]

@siddharth-nair-dtdl / @sundaresh-k-oss
Can you please address the review comments and update the PR.

[sundaresh-k-oss]

@AkhilaReddyK7 - We have addressed the comments, tested and updated the PR . Can you please review the changes ?

[Nikita-Hakai]

Internal build success: https://gerrit.teamccp.com/#/c/956554/

[Nikita-Hakai]

@siddharth-nair-dtdl and @sundaresh-k-oss , can you please address the build failure for:

1. <syscfg/syscfg.h> as suggested by copilot above
2. Build failures(Workflow) indicated as part of this active PR

[sundaresh-k-oss]

@Nikita-Hakai -

1. <syscfg/syscfg.h> as suggested by copilot above

• The header file is already present as part of the Pull Request in the file source/core/wifi_ctrl_webconfig.c .

2. Build failures(Workflow) indicated as part of this active PR.

• Fixed the error and pushed the changes. Can you please check and let us know if the build is successful?

[Nikita-Hakai]

@Nikita-Hakai -

1. <syscfg/syscfg.h> as suggested by copilot above

• The header file is already present as part of the Pull Request in the file source/core/wifi_ctrl_webconfig.c .

2. Build failures(Workflow) indicated as part of this active PR.

• Fixed the error and pushed the changes. Can you please check and let us know if the build is successful?

Hi @siddharth-nair-dtdl , still seeing the build error.

[sundaresh-k-oss]

@Nikita-Hakai - Fixed the build errors. Can you please check from your side ?

[Nikita-Hakai]

@Nikita-Hakai - Fixed the build errors. Can you please check from your side ?

Hi @sundaresh-k-oss , The build is still failing as part of the github workflow for both RPI and BPI.

[sundaresh-k-oss]

Hi @Nikita-Hakai-
The issue is due to syscfg.h file not getting included. We have added the same code changes and tested in Banana PI RDK-B builds and no issues are there since utopia component is present and compiled in RDK-B. The issue is seen in openwrt build in banana pi since utopia component is not added or compiled in RDK-B.

[Nikita-Hakai]

Hi @Nikita-Hakai- The issue is due to syscfg.h file not getting included. We have added the same code changes and tested in Banana PI RDK-B builds and no issues are there since utopia component is present and compiled in RDK-B. The issue is seen in openwrt build in banana pi since utopia component is not added or compiled in RDK-B.

Agreed that it does not fail for RDKB BPI builds, however we wont be able to merge the PR if the build errors for BPI and RPI are not resolved, as the github workflows are based on RPI/BPI builds.