Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Message-ID: <1607101812.24959.157.camel@suse.cz>
Subject: Re: [PATCH v4 RESEND] x86, sched: Use midpoint of max_boost and
 max_P for frequency invariance on AMD EPYC
From:   Giovanni Gherdovich <ggherdovich@suse.cz>
To:     Borislav Petkov <bp@alien8.de>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Len Brown <lenb@kernel.org>,
        "Rafael J . Wysocki" <rjw@rjwysocki.net>
Cc:     Nathan Fontenot <Nathan.Fontenot@amd.com>, x86@kernel.org,
        linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-acpi@vger.kernel.org
Date:   Fri, 04 Dec 2020 18:10:12 +0100
In-Reply-To: <20201204170326.4178-1-ggherdovich@suse.cz>
References: <20201112182614.10700-3-ggherdovich@suse.cz>
         <20201204170326.4178-1-ggherdovich@suse.cz>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Fri, 2020-12-04 at 18:03 +0100, Giovanni Gherdovich wrote:
> Frequency invariant accounting calculations need the ratio
> freq_curr/freq_max, but freq_max is unknown as it depends on dynamic power
> allocation between cores: AMD EPYC CPUs implement "Core Performance Boost".
> Three candidates are considered to estimate this value:
> [...]
> 
> Benchmarks are described in the next section.
> Tilde (~) means the value is the same as baseline.
> 
> - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
>             ondemand  perfgov  sugov-noinv  sugov-max  sugov-mid  sugov-P0  better if
> - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
> [...]

Hello,

this patch is currently merged in the tip tree, branch sched/core as
commit 46609527577d1def0af29ca5b56cffeeea771ada.

Unfortunately in the original commit message I used "----------" for
making table headers, and git dropped all the commit message after
that sign, i.e. the benchmark results and my signed-off-by.

In this "resend" I replaced the offending sign and the new commit
message should make it intact to the destination tree.


Thanks,
Giovanni