Skip to content

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

Open
siddharth-nair-dtdl wants to merge 1 commit into
rdkcentral: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 into
rdkcentral: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
@AkhilaReddyK7
Copy link
Copy Markdown

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

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