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

[Mellanox] Fix the memory leak in mlnx-sfpd (201807) #3131

Merged
merged 1 commit into from
Jul 10, 2019
Merged

[Mellanox] Fix the memory leak in mlnx-sfpd (201807) #3131

merged 1 commit into from
Jul 10, 2019

Conversation

stephenxs
Copy link
Collaborator

- What I did
Fix the memory leak in mlnx-sfpd.

- How I did it
Fix the memory leak in mlnx-sfpd::sx_recv.

- How to verify it
reset an SFP module, and then:

  1. observe /proc//maps and check whether [heap] increases.
  2. observe syslog and check whether SFP module plugin/out event has been recorded.

- Description for the changelog
[Mellanox] Fix the memory leak in mlnx-sfpd

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

…vention new_<type_name> should be release explicitly via delete_<type_name>.
@lguohan lguohan merged commit 0e0d678 into sonic-net:201807 Jul 10, 2019
@stephenxs stephenxs deleted the fix-mlnx-sfpd-leak-201807 branch July 10, 2019 21:49
mssonicbld added a commit that referenced this pull request Jan 26, 2024
…atically (#17911)

#### Why I did it
src/sonic-utilities
```
* b3d856bf - (HEAD -> master, origin/master, origin/HEAD) Add all SKUs to the generic config update list (#3131) (7 hours ago) [Stephen Sun]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Feb 5, 2024
…atically (#18025)

#### Why I did it
src/sonic-utilities
```
* 2046e66c - (HEAD -> 202311, origin/202311) Reduce generate_dump mem usage for cores (#3052) (3 days ago) [davidm-arista]
* fbd6c916 - Disable Key Validation feature during sonic-installation for Cisco Platforms (#3115) (3 days ago) [selvipal]
* 88c027f0 - [Techsupport]Adding more FRR and BGP dumps (#3118) (3 days ago) [Sudharsan Dhamal Gopalarathnam]
* 555ecf64 - [chassis]: Support show ip bgp summary to display without error when no external neighbors are configured on chassis LC (#3099) (3 days ago) [Arvindsrinivasan Lakshmi Narasimhan]
* 1515edcb - [db_migrator]Remove route migration (#3068) (3 days ago) [Sudharsan Dhamal Gopalarathnam]
* 8862c114 - Modify teamd retry count script to base BGP status on default BGP status (#3069) (3 days ago) [Saikrishna Arcot]
* f4b5ef21 - Add all SKUs to the generic config update list (#3131) (3 days ago) [Stephen Sun]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Feb 16, 2024
…atically (#18106)

#### Why I did it
src/sonic-utilities
```
* 1c5c134b - (HEAD -> 202305, origin/202305) Add all SKUs to the generic config update list (#3131) (4 hours ago) [Stephen Sun]
```
#### 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants