sonic-net / sonic-buildimage

Scripts which perform an installable binary image build for SONiC
Other
746 stars 1.44k forks source link

[DPU][ACL][scale] The DPU doesn't support the scale of 4k tags with 24k IPs in each #17714

Open congh-nvidia opened 11 months ago

congh-nvidia commented 11 months ago

Description

Steps to reproduce the issue:

In the DASH ACL tag test plan https://github.com/sonic-net/sonic-mgmt/pull/11094, it is required to test the ACL tag scale up to 4k tags with 24k IP addresses in each:

### Test case \#8 - ACL Tags Scale

#### Test objective

Verify scale functionality of ACL Tag. 1 rule, 4k tags with 24k prefixes each.

#### Test steps

- setup
    - Create 4k Tags with 24k IP addresses each.
    - Create a Rule with these Tags.
- validation
    - Send packet to random IP, which should be received as matched by IP address.
- teardown
    - Remove Rule.
    - Remove Tags.

When I was testing 4k tags with 2 IPs in each, found that the max lengt of IP SAI_ATTR_VALUE_TYPE_IP_PREFIX_LIST is limited to 4096:

Jan  4 15:59:08.951504 r-r740-08-bf3-sonic-01 ERR swss#orchagent: :- meta_genetic_validation_list: SAI_DASH_ACL_RULE_ATTR_SIP:SAI_ATTR_VALUE_TYPE_IP_PREFIX_LIST list count 8192 > max list count 4096

Jan  4 15:59:08.951504 r-r740-08-bf3-sonic-01 ERR swss#orchagent: :- createRule: Failed to create ACL rule: -5, SAI_STATUS_INVALID_PARAMETER

Jan  4 15:59:08.951504 r-r740-08-bf3-sonic-01 ERR swss#orchagent: :- handleSaiCreateStatus: Encountered failure in create operation, exiting orchagent, SAI API: SAI_API_DASH_ACL, status: SAI_STATUS_INVALID_PARAMETER

Describe the results you received:

It is not possible to configure 4k tags with 24k IPs in each on the dpu.

Describe the results you expected:

Test should pass or the scale should be changed.

Output of show version:

bluefield.172-a6f196d52_Internal

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

volodymyrsamotiy commented 10 months ago

It should be triaged separately since it is DASH related issue. @prsunny will follow up.