Integration tests rhel-8.9 #1362
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Joe Lawrence <[email protected]>
Shrink the size of the sub-section header below the OpenEuler section, reducing confusion as to what section that is a part of, as it is only necessary for OpenEuler machines. Signed-off-by: Ryan Sullivan <[email protected]>
Signed-off-by: Joe Lawrence <[email protected]>
Signed-off-by: Joe Lawrence <[email protected]>
Signed-off-by: Ryan Sullivan <[email protected]>
PowerPC implements alternative instructions with __ftr_fixup and
__ftr_alt_* sections, for example net/core/dev.o:
Disassembly of section __ftr_alt_97:
0000000000000000 <__ftr_alt_97>:
0: 3c 00 40 7c pause_short
4: 3c 00 40 7c pause_short
8: 3c 00 40 7c pause_short
c: 3c 00 40 7c pause_short
Disassembly of section __ftr_fixup:
0000000000000000 <__ftr_fixup>:
0: 00 00 00 00 .long 0x0
4: 00 00 04 00 .long 0x40000
...
10: R_PPC64_REL64 .text.napi_busy_loop+0x1cc
18: R_PPC64_REL64 .text.napi_busy_loop+0x1dc
20: R_PPC64_REL64 __ftr_alt_97+0x20
28: R_PPC64_REL64 __ftr_alt_97+0x2c
34: 00 00 04 00 .long 0x40000
...
40: R_PPC64_REL64 .text.napi_busy_loop+0x3c0
48: R_PPC64_REL64 .text.napi_busy_loop+0x3d0
50: R_PPC64_REL64 __ftr_alt_97+0x24
58: R_PPC64_REL64 __ftr_alt_97+0x30
64: 00 00 04 00 .long 0x40000
...
70: R_PPC64_REL64 .text.napi_threaded_poll+0x170
78: R_PPC64_REL64 .text.napi_threaded_poll+0x180
80: R_PPC64_REL64 __ftr_alt_97+0x28
88: R_PPC64_REL64 __ftr_alt_97+0x34
94: 00 00 04 00 .long 0x40000
...
a0: R_PPC64_REL64 .text.net_rx_action+0x1a0
a8: R_PPC64_REL64 .text.net_rx_action+0x1b0
b0: R_PPC64_REL64 __ftr_alt_97+0x2c
b8: R_PPC64_REL64 __ftr_alt_97+0x38
Like .altinstr_{aux,replacement}, __ftr_alt sections don't have symbols
associated with them and may skip kpatch_replace_sections_syms()'s rela
mangling. As __ftr_fixup references __ftr_alt_* directly,
kpatch_process_special_sections() will already include __ftr_alt_* as
part of the "group" handling in kpatch_regenerate_special_section().
Reported-by: Zhijun Wang <[email protected]>
Signed-off-by: Joe Lawrence <[email protected]>
Signed-off-by: Ryan Sullivan <[email protected]>
|
Hmm, something looks weird with the history of this branch. I looks like it branched off master, separately cherry-picked some commits, then merged master back onto it. I would have thought that would have dropped the redundant commits from the list, but the webUI shows an additional change to create-diff-object.c. To avoid this, I would just force push with a fresh branch from main and pick e1a8b28 ("test/integration/rhel-8.9: add kernel-4.18.0-492.el8 tests") to the new branch. That should be all we want to merge onto master. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contains updated integration tests for rhel-8.9