Skip to content

x86: Add 'l' suffix to btr/bts instructions#224

Merged
82marbag merged 3 commits into
KernelTestFramework:mainlinefrom
cjams:asm-fix
Nov 9, 2021
Merged

x86: Add 'l' suffix to btr/bts instructions#224
82marbag merged 3 commits into
KernelTestFramework:mainlinefrom
cjams:asm-fix

Conversation

@cjams
Copy link
Copy Markdown
Contributor

@cjams cjams commented Nov 2, 2021

Add an explicit suffix to the btr/bts instructions within
atomic_test_and_{re,}set_bit. This addition silences the
following assembler warnings:

ktf/include/arch/x86/atomic.h:62: Warning: no instruction mnemonic suffix given and no register operands; using default for bts' ktf/include/arch/x86/atomic.h:74: Warning: no instruction mnemonic suffix given and no register operands; using default for btr'

Signed-off-by: Connor Davis connojd@amazon.com

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add an explicit suffix to the btr/bts instructions within
atomic_test_and_{re,}set_bit. This addition silences the
following assembler warnings:

ktf/include/arch/x86/atomic.h:62: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
ktf/include/arch/x86/atomic.h:74: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'

Signed-off-by: Connor Davis <connojd@amazon.com>
@82marbag 82marbag requested a review from a team as a code owner November 9, 2021 16:15
@82marbag 82marbag merged commit 5902578 into KernelTestFramework:mainline Nov 9, 2021
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.

3 participants