Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <bbde7529-eb64-5454-0984-bfdabac37b64@intel.com>
Date:   Mon, 10 Apr 2023 11:16:15 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.8.0
Subject: Re: [PATCH v7 23/33] x86/fred: let ret_from_fork() jmp to
 fred_exit_user when FRED is enabled
Content-Language: en-US
To:     Xin Li <xin3.li@intel.com>, linux-kernel@vger.kernel.org,
        x86@kernel.org, kvm@vger.kernel.org
Cc:     tglx@linutronix.de, mingo@redhat.com, bp@alien8.de,
        dave.hansen@linux.intel.com, hpa@zytor.com, peterz@infradead.org,
        andrew.cooper3@citrix.com, seanjc@google.com, pbonzini@redhat.com,
        ravi.v.shankar@intel.com, jiangshanlai@gmail.com,
        shan.kang@intel.com
References: <20230404102716.1795-1-xin3.li@intel.com>
 <20230404102716.1795-24-xin3.li@intel.com>
From:   Dave Hansen <dave.hansen@intel.com>
In-Reply-To: <20230404102716.1795-24-xin3.li@intel.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 4/4/23 03:27, Xin Li wrote:
> --- a/arch/x86/entry/entry_64.S
> +++ b/arch/x86/entry/entry_64.S
> @@ -299,7 +299,12 @@ SYM_CODE_START_NOALIGN(ret_from_fork)
>  	UNWIND_HINT_REGS
>  	movq	%rsp, %rdi
>  	call	syscall_exit_to_user_mode	/* returns with IRQs disabled */
> +#ifdef CONFIG_X86_FRED
> +	ALTERNATIVE "jmp swapgs_restore_regs_and_return_to_usermode", \
> +		    "jmp fred_exit_user", X86_FEATURE_FRED
> +#else
>  	jmp	swapgs_restore_regs_and_return_to_usermode
> +#endif

Does the #ifdef really buy us anything here?

I guess it might save a *TINY* amount of time at alternative processing
time.  But that doesn't really seem worth it.