diff mbox series

[v2] selftests: kvm: remove reassignment of non-absolute variables

Message ID 20201211012317.3722214-1-morbo@google.com (mailing list archive)
State New, archived
Headers show
Series [v2] selftests: kvm: remove reassignment of non-absolute variables | expand

Commit Message

Bill Wendling Dec. 11, 2020, 1:23 a.m. UTC
Clang's integrated assembler does not allow symbols with non-absolute
values to be reassigned. Modify the interrupt entry loop macro to be
compatible with IAS by using a label and an offset.

Cc: Jian Cai <caij2003@gmail.com>
Signed-off-by: Bill Wendling <morbo@google.com>
References: https://lore.kernel.org/lkml/20200714233024.1789985-1-caij2003@gmail.com/
---
 tools/testing/selftests/kvm/lib/x86_64/handlers.S | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Jim Mattson May 3, 2021, 6:37 p.m. UTC | #1
On Thu, Dec 10, 2020 at 7:58 PM Bill Wendling <morbo@google.com> wrote:
>
> Clang's integrated assembler does not allow symbols with non-absolute
> values to be reassigned. Modify the interrupt entry loop macro to be
> compatible with IAS by using a label and an offset.
>
> Cc: Jian Cai <caij2003@gmail.com>
> Signed-off-by: Bill Wendling <morbo@google.com>
> References: https://lore.kernel.org/lkml/20200714233024.1789985-1-caij2003@gmail.com/
Reviewed-by: Jim Mattson <jmattson@google.com>
Paolo Bonzini May 4, 2021, 7:22 a.m. UTC | #2
On 03/05/21 20:37, Jim Mattson wrote:
> On Thu, Dec 10, 2020 at 7:58 PM Bill Wendling <morbo@google.com> wrote:
>>
>> Clang's integrated assembler does not allow symbols with non-absolute
>> values to be reassigned. Modify the interrupt entry loop macro to be
>> compatible with IAS by using a label and an offset.
>>
>> Cc: Jian Cai <caij2003@gmail.com>
>> Signed-off-by: Bill Wendling <morbo@google.com>
>> References: https://lore.kernel.org/lkml/20200714233024.1789985-1-caij2003@gmail.com/
> Reviewed-by: Jim Mattson <jmattson@google.com>
> 

Queued, thanks.

Paolo
diff mbox series

Patch

diff --git a/tools/testing/selftests/kvm/lib/x86_64/handlers.S b/tools/testing/selftests/kvm/lib/x86_64/handlers.S
index aaf7bc7d2ce1..7629819734af 100644
--- a/tools/testing/selftests/kvm/lib/x86_64/handlers.S
+++ b/tools/testing/selftests/kvm/lib/x86_64/handlers.S
@@ -54,9 +54,9 @@  idt_handlers:
 	.align 8
 
 	/* Fetch current address and append it to idt_handlers. */
-	current_handler = .
+666 :
 .pushsection .rodata
-.quad current_handler
+	.quad 666b
 .popsection
 
 	.if ! \has_error