The .align directive has a different behaviour on 32-bit arm than on
other platforms. Use .balign instead, which has consistent behaviour on
all architectures.
Reported by: kib
Fixes: 6b82130e6c9a ("clock: Add a long ticks variable, ticksl")
Differential D48430 Authored by markj on Jan 11 2025, 5:00 PM.
Details Summary The .align directive has a different behaviour on 32-bit arm than on Reported by: kib Test Plan Booted an armv7 kernel in qemu, checked the alignment of the ticksl symbol.
Diff Detail
Event TimelineHerald added subscribers: olce, imp. · View Herald TranscriptJan 11 2025, 5:00 PM2025-01-11 17:00:26 (UTC+0) Harbormaster completed remote builds in B61675: Diff 149136.Jan 11 2025, 5:00 PM2025-01-11 17:00:29 (UTC+0) This revision is now accepted and ready to land.Jan 11 2025, 5:03 PM2025-01-11 17:03:51 (UTC+0) Closed by commit rG159d29d39482: clock: Use .balign to align ticksl (authored by markj). · Explain WhyJan 11 2025, 5:46 PM2025-01-11 17:46:52 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 149138 sys/kern/subr_ticks.S
|