Update to Linux 5.10.179#328
Conversation
One patch needed some changes; all other patches are unchanged. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
paulmenzel
left a comment
paulmenzel
left a comment
There was a problem hiding this comment.
Thank you for doing this update.
It’d be great if you mentioned the devices you tested this on.
I wouldn’t put a dot/period at the end of commit messages summaries.
| KVERSION_SHORT ?= 5.10.0-23-$(KERNEL_ABI_MINOR_VERSION) | ||
| KVERSION ?= $(KVERSION_SHORT)-amd64 | ||
| KERNEL_VERSION ?= 5.10.179 | ||
| KERNEL_SUBVERSION ?= 1 |
There was a problem hiding this comment.
What is the subversion, and why change it?
There was a problem hiding this comment.
The subversion refers to the Debian version number. In this case, version 5.10.179-3 from Debian is being used.
| -abiname: 18 | ||
| +abiname: 18-2 | ||
| -abiname: 23 | ||
| +abiname: 23-2 |
There was a problem hiding this comment.
It’d be great if you elaborated in the commit message, why 23.
There was a problem hiding this comment.
ABI 23 is the ABI version number that Debian is using. This typically gets incremented by one for each minor kernel update that Debian does (i.e. going from 5.10.179 to the current 5.10.191).
There was a problem hiding this comment.
Indeed. Please reference the Debian package you base your change on. For example I found different versions: version 5.10.179+3~deb10u1 and 5.10.179+5~deb10u1, where the change-log of the latter contains:
Changes:
linux-signed-5.10-arm64 (5.10.179+5~deb10u1) buster-security; urgency=high
.
- Sign kernel from linux-5.10 5.10.179-5~deb10u1
.- Rebuild for buster:
- Change ABI number to 0.deb10.24
There was a problem hiding this comment.
I think the versions you found are for Buster (hinted by the ~deb10u1 at the end, which is used for backports).
I've updated the main PR description to include the version number and a link to the publishing of that version.
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
paulmenzel
left a comment
paulmenzel
left a comment
There was a problem hiding this comment.
Feel free to address the nits I mentioned, but all in all, it looks great. Thank you for updating the description.
Remove Mellanox patches that are not required for 5.10.179
## How I did it Depends on sonic-net/sonic-linux-kernel#328 and sonic-net/saibcm-modules#12. #### How to verify it Verified that the image boots up, BGP comes up, and a basic warm-reboot works on VS, broadcom, and mellanox.
|
Backport ADO: part of 24592132 |
* Update to Linux 5.10.179 One patch needed some changes; all other patches are unchanged. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Fix kversion Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Update to 5.10.179-3. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Disable 6 Mellanox patches as those changes are already included Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Remove Mellanox patches that are not required for 5.10.179 --------- Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> Co-authored-by: Vivek Reddy <vkarri@nvidia.com>
|
@saiarcot895 this PR failed compile with 202305, could you submit separate PR to 202305? |
This reverts commit e64669d.
|
Hi @StormLiangMS, for backporting, there would need to be additional separate changes on the sonic-buildimage repo side. I'll create manual PRs for both. |
* Update to Linux 5.10.179 One patch needed some changes; all other patches are unchanged. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Fix kversion Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Update to 5.10.179-3. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Disable 6 Mellanox patches as those changes are already included Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Remove Mellanox patches that are not required for 5.10.179 --------- Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> Co-authored-by: Vivek Reddy <vkarri@nvidia.com>
* Update to Linux 5.10.179 One patch needed some changes; all other patches are unchanged. * Fix kversion * Update to 5.10.179-3. * Disable 6 Mellanox patches as those changes are already included * Remove Mellanox patches that are not required for 5.10.179 --------- Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> Co-authored-by: Vivek Reddy <vkarri@nvidia.com>
* Update to Linux 5.10.179 One patch needed some changes; all other patches are unchanged. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Fix kversion Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Update to 5.10.179-3. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Disable 6 Mellanox patches as those changes are already included Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Remove Mellanox patches that are not required for 5.10.179 --------- Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> Co-authored-by: Vivek Reddy <vkarri@nvidia.com>
* Update to Linux 5.10.179 One patch needed some changes; all other patches are unchanged. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Fix kversion Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Update to 5.10.179-3. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Disable 6 Mellanox patches as those changes are already included Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> * Remove Mellanox patches that are not required for 5.10.179 --------- Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> Co-authored-by: Vivek Reddy <vkarri@nvidia.com>
* Update to Linux 5.10.179 One patch needed some changes; all other patches are unchanged. * Fix kversion * Update to 5.10.179-3. * Disable 6 Mellanox patches as those changes are already included * Remove Mellanox patches that are not required for 5.10.179 --------- Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com> Co-authored-by: Vivek Reddy <vkarri@nvidia.com>
5 participants
Update from 5.10.140 to 5.10.179 (published as linux 5.10.179-3 by Debian). This brings in a number of bug and security fixes.
Update our ABI version number to 23-2 to match the ABI version 23 used by Debian for 5.10.179.