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.

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

Rate limiting not happening for ARP #3080

Open
mfriyas opened this issue Jun 25, 2019 · 1 comment
Open

Rate limiting not happening for ARP #3080

mfriyas opened this issue Jun 25, 2019 · 1 comment
Assignees

Comments

@mfriyas
Copy link

mfriyas commented Jun 25, 2019

Issue Description:-

               ARP rate limiting to CPU in dell platforms was set as 600 in copp.config.json .

Am sending arp request from ixia to sonic with more than 600 pps secs. Sonic device not dropping the packets when it reaches the rate-limit as 600. it processes all the packets to the cpu and gave arp reply to ixia .This happens when am increases the ixia stream to 1000pps,2000pps,5000pps,7000pps,10000pps. For all this scenarios rate-limiting not works as expected.

JSON output from swss container:-

{
"COPP_TABLE:trap.group.arp": {
"trap_ids": "arp_req,arp_resp,neigh_discovery",
"trap_action":"copy",
"trap_priority":"4",
"queue": "4",
"meter_type":"packets",
"mode":"sr_tcm",
"cir":"600",
"cbs":"600",
"red_action":"drop"
},
"OP": "SET"
},

Device outputs:-

1500 pps

root@sonic-z9264f-04:/home/admin# bcmcmd "show c" | grep "cpu"
PERQ_PKT(4).cpu0 : 57,060,468 +1,725 1,493/s
PERQ_BYTE(4).cpu0 : 3,880,111,824 +117,300 101,526/s
root@sonic-z9264f-04:/home/admin#

2500 pps

root@sonic-z9264f-04:/home/admin# bcmcmd "show c" | grep "cpu"
PERQ_PKT(4).cpu0 : 57,121,736 +61,268 2,500/s
PERQ_BYTE(4).cpu0 : 3,884,278,048 +4,166,224 170,004/s

5000 pps

root@sonic-z9264f-04:/home/admin# bcmcmd "show c" | grep "cpu"
PERQ_PKT(4).cpu0 : 57,243,989 +5,794 5,026/s
PERQ_BYTE(4).cpu0 : 3,892,591,252 +393,992 341,760/s

7000 pps

root@sonic-z9264f-04:/home/admin# bcmcmd "show c" | grep "cpu"
PERQ_PKT(4).cpu0 : 57,425,961 +17,120 7,003/s
PERQ_BYTE(4).cpu0 : 3,904,965,348 +1,164,160 476,190/s
root@sonic-z9264f-04:/home/admin#

10000 pps

root@sonic-z9264f-04:/home/admin# bcmcmd "show c" | grep "cpu"
PERQ_PKT(4).cpu0 : 57,666,198 +240,237 10,203/s
PERQ_BYTE(4).cpu0 : 3,921,301,464 +16,336,116 693,784/s

@bryantkwu
Copy link

I think this is a problem with broadcom SAI.
When I enter bcmsh, and add some actions to the FP rule of ARP, for example:
“fp action add 0x7 YpDrop” and “fp action add 0x7 GpCopyToCpu”, and then reinstall.
After that the function will be normal.

mssonicbld added a commit that referenced this issue Apr 19, 2024
…lly (#18334)

#### Why I did it
src/sonic-swss
```
* 9e183a65 - (HEAD -> master, origin/master, origin/HEAD) Add bookworm build to the PR checkers (#3114) (2 days ago) [Saikrishna Arcot]
* ec463953 - [twamporch] Explicitly initialize local variable (#3115) (2 days ago) [xiaodong hu]
* 19410232 - Add force option for fabric port unisolate command (#3089) (7 days ago) [jfeng-arista]
* 774973d9 - [dash] fix ENI admin state update (#3081) (8 days ago) [Yakiv Huryk]
* 56ba6de3 - [p4orch] Clang format change. (#3096) (10 days ago) [mint570]
* f6dd9caa - Introduce a new role for DPU-NPU Interconnect (10 days ago) [Vivek]
* 0f707757 - [portsorch] Handle TRANSCEIVER_INFO table on warm boot (#3087) (10 days ago) [Stepan Blyshchak]
* cbc263a1 - [EVPN]Fix fpmsyncd crash when EVPN type5 is received with bgp fib suppression enabled (#3101) (13 days ago) [Sudharsan Dhamal Gopalarathnam]
* 4af61167 - Fix oper FEC retrieval after warmboot (#3100) (13 days ago) [Sudharsan Dhamal Gopalarathnam]
* 2a586154 - [Fdbsyncd] Adding extern_learn flag with fdb entry so Kernel doesn't age out (#2985) (2 weeks ago) [KISHORE KUNAL]
* 465399e9 - T2-VOQ-VS: Fix iBGP bringup issue (#3053) (2 weeks ago) [Deepak Singhal]
* dc9eae87 - Clang format change. (#3080) (2 weeks ago) [mint570]
* 95a74750 - [orchagent] TWAMP Light orchagent implementation (#2927) (2 weeks ago) [xiaodong hu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants