Skip to content

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

Open
siddharth-nair-dtdl wants to merge 1 commit intordkcentral:developfrom
siddharth-nair-dtdl:RDKBWIFI-202
Open

RDKCOM-5580: RDKBDEV-3433 RDKBWIFI-202 Implementation of Traffic Separation#263
siddharth-nair-dtdl wants to merge 1 commit intordkcentral:developfrom
siddharth-nair-dtdl:RDKBWIFI-202

Conversation

@siddharth-nair-dtdl
Copy link
Copy Markdown

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

@siddharth-nair-dtdl siddharth-nair-dtdl requested review from a team as code owners March 16, 2026 10:25
@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/utopia/263/rdkb/components/opensource/ccsp/Utopia

  • Commit: 250ee7c

Report detail: gist'

@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/utopia/263/rdkb/components/opensource/ccsp/Utopia

  • Commit: 250ee7c

Report detail: gist'

@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

b'## WARNING: A Blackduck scan failure has been waived

A prior failure has been upvoted

  • Upvote reason: OK

  • Commit: 250ee7c
    '

@siddharth-nair-dtdl siddharth-nair-dtdl marked this pull request as draft March 17, 2026 06:10
@siddharth-nair-dtdl siddharth-nair-dtdl marked this pull request as ready for review March 17, 2026 06:10

bridge vlan del vid 1 dev wifi0
bridge vlan del vid 1 dev wifi0.1
bridge vlan del vid 1 dev wifi02
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there seems to be a typo as wifi02, there is already wifi0.2 in the below line

iptables -I INPUT -i brlan0.$vlan_mesh -p udp --dport 67:68 -j ACCEPT
iptables -I INPUT -i brlan0.$vlan_mesh -p udp --sport 67:68 -j ACCEPT
iptables -I INPUT -i brlan0.$vlan_iot -p udp --dport 67:68 -j ACCEPT
iptables -I INPUT -i brlan0.$vlan_iot -p udp --sport 67:68 -j ACCEPT
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation to be checked above

/*const char* SERVICE_CUSTOM_EVENTS[] = {
"lan-status|/etc/utopia/service.d/service_xhs.sh",
"multinet_2-status|/etc/utopia/service.d/service_xhs.sh",
"ipv4_5-status|/etc/utopia/service.d/service_xhs.sh",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation to be corrected

@sundar2k sundar2k changed the title RDKBWIFI-202: Implementation of Traffic Separation RDKBWIFI-202: RDKBDEV-3433: Implementation of Traffic Separation Apr 23, 2026
@pradeeptakdas pradeeptakdas changed the title RDKBWIFI-202: RDKBDEV-3433: Implementation of Traffic Separation RDKCOM-5580: RDKBDEV-3433 RDKBWIFI-202 Implementation of Traffic Separation Apr 23, 2026
@pradeeptakdas
Copy link
Copy Markdown

Hi @siddharth-nair-dtdl

Few places indentation need to be corrected, Can you please fix the review comments?

Reason for change: Added traffic seperation TLV implementation via single bridge
Test Procedure: Verify build is successfull and check if traffic seperation is functional
Risks: Medium
Priority: P2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Contribution from community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants