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

Backport new kernel patches #195

Merged
merged 3 commits into from
Feb 25, 2021
Merged

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Feb 5, 2021

0024-mlxsw-core-Fix-memory-leak-on-module-removal.patch torvalds/linux@adc80b6 @v5.10-rc2
0025-platform-x86-mlx-platform-Remove-PSU-EEPROM-configur.patch torvalds/linux@2bf5046 @v5.11-rc1
0026-platform-x86-mlx-platform-Remove-PSU-EEPROM-configur.patch torvalds/linux@912b341 @v5.11-rc1

regression tests have been performed against these patches on the Mellanox platform, no issue found.

Signed-off-by: Kebo Liu kebol@nvidia.com

Copy link
Contributor

@paulmenzel paulmenzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the backport. Additionally to the comments, please mention in the merge/pull request description, how you tested this.

@paulmenzel
Copy link
Contributor

Also, at least the last patch is in Linux master since v5.11-rc1. Have you sent them for inclusion into the stable series, at least 5.10, so we can drop the patches, once updating to the latest 5.10.y release?

…al.patch

Signed-off-by: Kebo Liu <kebol@nvidia.com>
…M-configur.patch

Signed-off-by: Kebo Liu <kebol@nvidia.com>
…M-configur.patch

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@keboliu
Copy link
Collaborator Author

keboliu commented Feb 20, 2021

Also, at least the last patch is in Linux master since v5.11-rc1. Have you sent them for inclusion into the stable series, at least 5.10, so we can drop the patches, once updating to the latest 5.10.y release?

I can talk to the author of these patches to see the what we can do.

@keboliu
Copy link
Collaborator Author

keboliu commented Feb 23, 2021

@paulmenzel could you please review again?

@liat-grozovik
Copy link
Collaborator

@ paulmenzel can you please mark your comments as resolved and approve? we would like to move forward with the merge and have it on both 202012 and master.

@liat-grozovik
Copy link
Collaborator

@lguohan can you please review as well?

@liat-grozovik
Copy link
Collaborator

@paulmenzel and @lguohan kindly help to review and merge?
this one is holding us for having bugs fixes in master/202012. please also add the label for 202012 merge if there is already new branch for it.

@lguohan lguohan merged commit 2fcd4e3 into sonic-net:master Feb 25, 2021
@paulmenzel
Copy link
Contributor

Sorry for not responding earlier. I thought, you can mark the comments as resolved once you implemented the change. @lguohan, thank you for taking care of this.

lguohan pushed a commit to lguohan/sonic-linux-kernel that referenced this pull request Mar 10, 2021
0024-mlxsw-core-Fix-memory-leak-on-module-removal.patch torvalds/linux@adc80b6 @v5.10-rc2
0025-platform-x86-mlx-platform-Remove-PSU-EEPROM-configur.patch torvalds/linux@2bf5046 @v5.11-rc1
0026-platform-x86-mlx-platform-Remove-PSU-EEPROM-configur.patch torvalds/linux@912b341 @v5.11-rc1

regression tests have been performed against these patches on the Mellanox platform, no issue found.

Signed-off-by: Kebo Liu <kebol@nvidia.com>
lguohan pushed a commit that referenced this pull request Mar 10, 2021
0024-mlxsw-core-Fix-memory-leak-on-module-removal.patch torvalds/linux@adc80b6 @v5.10-rc2
0025-platform-x86-mlx-platform-Remove-PSU-EEPROM-configur.patch torvalds/linux@2bf5046 @v5.11-rc1
0026-platform-x86-mlx-platform-Remove-PSU-EEPROM-configur.patch torvalds/linux@912b341 @v5.11-rc1

regression tests have been performed against these patches on the Mellanox platform, no issue found.

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@keboliu keboliu deleted the hw-mgmt-7.0010.2000 branch October 28, 2023 03:36
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.

4 participants