Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Map tc 1, 2, 5, and 6 back to pg 0 #2650

Merged
merged 5 commits into from
Mar 8, 2019
Merged

Map tc 1, 2, 5, and 6 back to pg 0 #2650

merged 5 commits into from
Mar 8, 2019

Conversation

wendani
Copy link
Contributor

@wendani wendani commented Mar 6, 2019 •
edited
Loading

Lossy traffic does not need to be mapped to different ingress PGs. They can all share the same ingress PG.

Signed-off-by: Wenda Ni wenni@microsoft.com

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@wendani
Map tc 2 and 6 back to pg 1
ec10b24 
Signed-off-by: Wenda Ni <wenni@microsoft.com>
@wendani
Map tc 5 back to pg 1
1a6d21d 
Signed-off-by: Wenda Ni <wenni@microsoft.com>
@wendani wendani changed the title Map tc 2 and 6 back to pg 1 Map tc 2, 5, and 6 back to pg 1 Mar 6, 2019
wendani added 2 commits March 6, 2019 21:08
@wendani
Dettach pg 5 from lossy profile
3bbcd4e 
Signed-off-by: Wenda Ni <wenni@microsoft.com>
@wendani
Update s6100 qos and buffer test case
d1c94da 
Signed-off-by: Wenda Ni <wenni@microsoft.com>
@andriymoroz-mlnx andriymoroz-mlnx mentioned this pull request Mar 7, 2019
Open
@wendani
Map lossy traffic all to pg 0
adbbfef 
Signed-off-by: Wenda Ni <wenni@microsoft.com>
@lguohan
Copy link
Collaborator

lguohan commented Mar 7, 2019

@wendani , please also change your pr title.

@lguohan
Copy link
Collaborator

lguohan commented Mar 7, 2019

please also state the reason why you make this change?

@wendani wendani changed the title Map tc 2, 5, and 6 back to pg 1 Map tc 2, 5, and 6 back to pg 0 Mar 7, 2019
@wendani wendani changed the title Map tc 2, 5, and 6 back to pg 0 Map tc 1, 2, 5, and 6 back to pg 0 Mar 7, 2019
@lguohan lguohan merged commit f9c9fa8 into sonic-net:master Mar 8, 2019
yxieca pushed a commit that referenced this pull request Mar 19, 2019
@wendani @yxieca
[qos]: Map tc 1, 2, 5, and 6 back to pg 0 (#2650)
f720c2a 
Lossy traffic does not need to be mapped to different ingress PGs. They can all share the same ingress PG.

Signed-off-by: Wenda Ni <wenni@microsoft.com>
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 10, 2019
@wendani @tiantianlv
[qos]: Map tc 1, 2, 5, and 6 back to pg 0 (sonic-net#2650)
8c974a2 
Lossy traffic does not need to be mapped to different ingress PGs. They can all share the same ingress PG.

Signed-off-by: Wenda Ni <wenni@microsoft.com>
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 10, 2019
@wendani @tiantianlv
[qos]: Map tc 1, 2, 5, and 6 back to pg 0 (sonic-net#2650)
1678ca4 
Lossy traffic does not need to be mapped to different ingress PGs. They can all share the same ingress PG.

Signed-off-by: Wenda Ni <wenni@microsoft.com>
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 10, 2019
@wendani @tiantianlv
[qos]: Map tc 1, 2, 5, and 6 back to pg 0 (sonic-net#2650)
d94942c 
Lossy traffic does not need to be mapped to different ingress PGs. They can all share the same ingress PG.

Signed-off-by: Wenda Ni <wenni@microsoft.com>
@devpatha devpatha mentioned this pull request Feb 2, 2023
Closed
8 tasks
yxieca added a commit to yxieca/sonic-buildimage that referenced this pull request Feb 2, 2023
@yxieca
[202205][swss][utilities][linkmgrd][sairedis][platform-common] advanc…
cb9f620 
…e submodule head

swss:
* 79223f0 2023-01-25 | [bfdorch] add local discriminator to state DB (sonic-net#2629) (HEAD -> 202205) [Baorong Liu]
* 64940d4 2023-01-25 | Vxlan tunnel endpoint custom monitoring APPL DB table. (sonic-net#2589) [siqbal1986]
* 0930529 2023-02-02 | Merge pull request sonic-net#2650 from devpatha/202205 (github/202205) [Kevin Wang]
* 8f5884f 2023-02-01 | Remove counters for Mux ACL [Devesh P Pathak]

utilities:
* 557adfcc 2023-02-01 | show logging CLI support for logs stored in tmpfs (sonic-net#2641) (HEAD -> 202205) [mihirpat1]
* 28d7bb26 2023-01-31 | Making 'show feature autorestart' more resilient to missing auto_restart config in CONFIG_DB (sonic-net#2592) [kartik-arista]
* e0b3b4ab 2023-01-06 | [masic] 'show interfaces counters' reminds to use '-d all' option to check for internal links (sonic-net#2466) [wenyiz2021]
* 5613277d 2023-01-27 | [show_bfd] add local discriminator in show bfd command (sonic-net#2625) [Baorong Liu]

linkmgrd:
* 8803690 2023-01-31 | [active-active] force switch when config == active && target state == active (sonic-net#164) (HEAD -> 202205) [Jing Zhang]

sairedis:
* ad3d4d6 2023-02-01 | [sai_failure_dump]Invoking dump during SAI failure (sonic-net#1198) (HEAD -> 202205) [Sudharsan Dhamal Gopalarathnam]

platform-common:
* 6fd8992 2023-01-30 | Add missing sys imports (sonic-net#343) (HEAD -> 202205) [spilkey-cisco]
* 1fbc864 2023-01-25 | Change get_tx_bias return type to list (sonic-net#342) [mihirpat1]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@yxieca yxieca mentioned this pull request Feb 2, 2023
Merged
yxieca added a commit that referenced this pull request Feb 3, 2023
@yxieca
[202205][swss][utilities][linkmgrd][sairedis][platform-common] advanc…
4355911 
…e submodule head (#13606)

swss:
* 79223f0 2023-01-25 | [bfdorch] add local discriminator to state DB (#2629) (HEAD -> 202205) [Baorong Liu]
* 64940d4 2023-01-25 | Vxlan tunnel endpoint custom monitoring APPL DB table. (#2589) [siqbal1986]
* 0930529 2023-02-02 | Merge pull request #2650 from devpatha/202205 (github/202205) [Kevin Wang]
* 8f5884f 2023-02-01 | Remove counters for Mux ACL [Devesh P Pathak]

utilities:
* 557adfcc 2023-02-01 | show logging CLI support for logs stored in tmpfs (#2641) (HEAD -> 202205) [mihirpat1]
* 28d7bb26 2023-01-31 | Making 'show feature autorestart' more resilient to missing auto_restart config in CONFIG_DB (#2592) [kartik-arista]
* e0b3b4ab 2023-01-06 | [masic] 'show interfaces counters' reminds to use '-d all' option to check for internal links (#2466) [wenyiz2021]
* 5613277d 2023-01-27 | [show_bfd] add local discriminator in show bfd command (#2625) [Baorong Liu]

linkmgrd:
* 8803690 2023-01-31 | [active-active] force switch when config == active && target state == active (#164) (HEAD -> 202205) [Jing Zhang]

sairedis:
* ad3d4d6 2023-02-01 | [sai_failure_dump]Invoking dump during SAI failure (#1198) (HEAD -> 202205) [Sudharsan Dhamal Gopalarathnam]

platform-common:
* 6fd8992 2023-01-30 | Add missing sys imports (#343) (HEAD -> 202205) [spilkey-cisco]
* 1fbc864 2023-01-25 | Change get_tx_bias return type to list (#342) [mihirpat1]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan lguohan left review comments

@liat-grozovik liat-grozovik liat-grozovik approved these changes

@andriymoroz-mlnx andriymoroz-mlnx andriymoroz-mlnx approved these changes

@yxieca yxieca yxieca approved these changes

Assignees
No one assigned
Labels
Enhancement ➕ Included in 201811 Branch Request for 201811 Branch :shipit:
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@wendani @lguohan @yxieca @andriymoroz-mlnx @liat-grozovik