Re: [PATCH] x86 UEFI: Fix broken build for UEFI

[Alexandru Elisei <alexandru.elisei@arm.com>]

Hi,

On Thu, Feb 10, 2022 at 05:20:44PM +0800, Zhenzhong Duan wrote:
> UEFI loads EFI applications to dynamic runtime addresses, so it requires
> all applications to be compiled as PIC (position independent code).
> 
> The new introduced single-step #DB tests series bring some compile time
> absolute address, fixed it with RIP relative address.

With this patch the error:

ld: x86/debug.o: relocation R_X86_64_32S against `.text' can not be used when making a shared object; recompile with -fPIC

disappears and I can now build kvm-unit-tests for x86_64 when configured
with --target-efi:

Tested-by: Alexandru Elisei <alexandru.elisei@arm.com>

Thanks,
Alex

> 
> Fixes: 9734b4236294 ("x86/debug: Add framework for single-step #DB tests")
> Fixes: 6bfb9572ec04 ("x86/debug: Test IN instead of RDMSR for single-step #DB emulation test")
> Fixes: bc0dd8bdc627 ("x86/debug: Add single-step #DB + STI/MOVSS blocking tests")
> Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
> ---
>  x86/debug.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/x86/debug.c b/x86/debug.c
> index 20ff8ebacc16..13d1f9629e1d 100644
> --- a/x86/debug.c
> +++ b/x86/debug.c
> @@ -145,7 +145,7 @@ static unsigned long singlestep_basic(void)
>  		"and $~(1<<8),%%rax\n\t"
>  		"1:push %%rax\n\t"
>  		"popf\n\t"
> -		"lea 1b, %0\n\t"
> +		"lea 1b(%%rip), %0\n\t"
>  		: "=r" (start) : : "rax"
>  	);
>  	return start;
> @@ -186,7 +186,7 @@ static unsigned long singlestep_emulated_instructions(void)
>  		"movl $0x3fd, %%edx\n\t"
>  		"inb %%dx, %%al\n\t"
>  		"popf\n\t"
> -		"lea 1b,%0\n\t"
> +		"lea 1b(%%rip),%0\n\t"
>  		: "=r" (start) : : "rax", "ebx", "ecx", "edx"
>  	);
>  	return start;
> @@ -223,7 +223,7 @@ static unsigned long singlestep_with_sti_blocking(void)
>  		"1:and $~(1<<8),%%rax\n\t"
>  		"push %%rax\n\t"
>  		"popf\n\t"
> -		"lea 1b,%0\n\t"
> +		"lea 1b(%%rip),%0\n\t"
>  		: "=r" (start_rip) : : "rax"
>  	);
>  	return start_rip;
> @@ -259,7 +259,7 @@ static unsigned long singlestep_with_movss_blocking(void)
>  		"and $~(1<<8),%%rax\n\t"
>  		"1: push %%rax\n\t"
>  		"popf\n\t"
> -		"lea 1b,%0\n\t"
> +		"lea 1b(%%rip),%0\n\t"
>  		: "=r" (start_rip) : : "rax"
>  	);
>  	return start_rip;
> @@ -302,7 +302,7 @@ static unsigned long singlestep_with_movss_blocking_and_icebp(void)
>  		"1:and $~(1<<8),%%rax\n\t"
>  		"push %%rax\n\t"
>  		"popf\n\t"
> -		"lea 1b,%0\n\t"
> +		"lea 1b(%%rip),%0\n\t"
>  		: "=r" (start) : : "rax"
>  	);
>  	return start;
> @@ -346,7 +346,7 @@ static unsigned long singlestep_with_movss_blocking_and_dr7_gd(void)
>  		"and $~(1<<8),%%rax\n\t"
>  		"push %%rax\n\t"
>  		"popf\n\t"
> -		"lea 1b,%0\n\t"
> +		"lea 1b(%%rip),%0\n\t"
>  		: "=r" (start_rip) : : "rax"
>  	);
>  	return start_rip;
> -- 
> 2.25.1
>