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

[DropCounter]Fix the ordering issue with dropcounter #40

Closed
wants to merge 1 commit into from

Conversation

dgsudharsan
Copy link
Owner

What I did
During the removal of drop counters, the counter is deleted first followed by removing it from flexcounter. This would result in a race condition where counterpoll might try to access the counter after it is deleted.

Apr 22 21:55:33.131134 r-panther-01 INFO python[125700]: ansible-command Invoked with _raw_params=config dropcounters delete TEST warn=True _uses_shell=False stdin_add_newline=True strip_empty_ends=True argv=None chdir=None executable=None creates=None removes=None stdin=None
Apr 22 21:55:33.611079 r-panther-01 NOTICE syncd#SDK: [SAI_DEBUG_COUNTER.NOTICE] ./src/mlnx_sai_debug_counter.c[1817]- mlnx_remove_debug_counter: Remove DEBUG_COUNTER [OID:0x300000055] [ID:0]
Apr 22 21:55:33.634416 r-panther-01 ERR syncd#SDK: [SAI_DEBUG_COUNTER.ERR] ./src/mlnx_sai_debug_counter.c[390]- mlnx_debug_counter_db_idx_to_data: Debug counter at index 0 is removed or not created yet
Apr 22 21:55:33.634416 r-panther-01 ERR syncd#SDK: :- collectData: Failed to get stats of Switch Debug Counter 0x100000021: -1
Apr 22 21:55:33.635328 r-panther-01 NOTICE swss#orchagent: :- uninstallDebugCounter: Successfully deleted drop counter TEST

Why I did it
To fix the error log appearing when drop counter is removed. The SAI object is removed during object destroy. Hence destroying the counter after flex counter is disabled.

https://github.com/sonic-net/sonic-swss/blob/67e03128dedc061c51da7ccd7fd716f00d974bb3/orchagent/debug_counter/drop_counter.cpp#L101

How I verified it
The existing sonic-swss UT should verify the drop counters functionality.

Details if related

@dgsudharsan dgsudharsan closed this May 1, 2024
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

Successfully merging this pull request may close these issues.

2 participants