[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v22 17/17] i386: move cpu_load_efer into sysemu-only section

From: Claudio Fontana
Subject: Re: [PATCH v22 17/17] i386: move cpu_load_efer into sysemu-only section of cpu.h
Date: Thu, 25 Feb 2021 10:02:38 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0

On 2/25/21 5:28 AM, Richard Henderson wrote:
> On 2/24/21 5:34 AM, Claudio Fontana wrote:
>> cpu_load_efer is now used only for sysemu code.
>> Therefore, make this inline function not visible anymore
>> in CONFIG_USER_ONLY builds.
>> Signed-off-by: Claudio Fontana <cfontana@suse.de>
>> ---
>>  target/i386/cpu.h | 31 ++++++++++++++++---------------
>>  1 file changed, 16 insertions(+), 15 deletions(-)
> Perhaps move to cpu-internal.h?  It is not used outside of target/i386/.
> Or declared in cpu-internal.h and placed in cpu-sysemu.c?  I don't see that
> it's particularly performance sensitive either.
> But one way or the other,
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> r~

cpu-sysemu.c (and cpu.c) seems now to be about cpu class, model, properties, 
and relative functions. Maybe worth writing it down in the header..

the file that seems to contain pertinent content now is "helper.c", which maybe 
should be renamed..



reply via email to

[Prev in Thread] Current Thread [Next in Thread]