Skip to content

Commit

Permalink
[image_config]: Update DHCP rate-limit for mgmt TOR devices (sonic-ne…
Browse files Browse the repository at this point in the history
…t#17630)

* [image_config]: Update DHCP rate-limit for mgmt TOR devices

    Change DHCP rate limit(queue4,group3) in SONiC copp configuration to 300 PPS
    for mgmt TORs while keeping the rate limit at 100 PPS for other topologies.

    Why I did it:
    Some mgmt TORs based on Marvell ASIC do not support 100 PPS CIR, so that led
    to these devices silently dropping DHCP packets.

    Microsoft ADO: **25820076**

    How to verify it:
    Send DHCP broadcast packets to an M0 DUT and verify that they are trapped to
    CPU at 300 PPS. On non-mgmt devices, the packets should be trapped at CIR of
    100 PPS. Also ran sonic-mgmt dhcp_relay test and confirmed that it passes.

Signed-off-by: Prabhat Aravind <paravind@microsoft.com>
  • Loading branch information
prabhataravind authored and mssonicbld committed Jan 3, 2024
1 parent 27c1e9b commit 595053f
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions files/image_config/copp/copp_cfg.j2
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,13 @@
"queue": "4",
"meter_type":"packets",
"mode":"sr_tcm",
{% if DEVICE_METADATA is defined and DEVICE_METADATA['localhost'] is defined and DEVICE_METADATA['localhost']['type'] is defined and 'Mgmt' in DEVICE_METADATA['localhost']['type'] %}
"cir":"300",
"cbs":"300",
{% else %}
"cir":"100",
"cbs":"100",
{% endif %}
"red_action":"drop"
},
"queue1_group1": {
Expand Down

0 comments on commit 595053f

Please sign in to comment.