Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp3292018imm; Tue, 17 Jul 2018 02:14:29 -0700 (PDT) X-Google-Smtp-Source: AAOMgpfxKce0dcD1e9Ffj7gUlpxbaZsetuAFgwgG92dt1OMF7UeX2f1y99O6/dMCqMUms8keSSWb X-Received: by 2002:a62:3601:: with SMTP id d1-v6mr846408pfa.41.1531818868955; Tue, 17 Jul 2018 02:14:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531818868; cv=none; d=google.com; s=arc-20160816; b=cvWQKv0xQOP5l2e+g6KCep/eri/qiqjJkcay0RdozXxy2H+qavMZW4tjADtQkDS0xR BCtCpJcF8PtwxOo9jg54JqGtnS1c5Yyslrvh35dWzMuNoyrEW2n/YdljMXgpujOMdx6m ZYU1ccX///Pdy3rP1l20UyBZ9oAc4r2Jg4a5JsnTVvfIXEfm1TMchDaBop7VdwOcs4Mx XhImDkkRdAbgO2fm5BT5/hV91lK6Jmi5TyH1pyZymB0yKOZLCq4cunV0CBpS0Zq8YHR1 hdQww9JFVkMiW+zRRU+0cWCVLUC7bQ0YijteJ7PU1hR2U7LQgVr1heF1G4gGcjUftoFU IZdw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=Usr+Y1oGFZuBqEYZPhchpxoJ2WocCZMAcu1vUnPLGaM=; b=KhipLpjp1RIeEYa+zLbO348Vslzf5nEtycd/D+7wm3FEuZY5O90MjTs4uSCQh4IHzs EKWf06IzewqKFrvOVatcEmOTijrrsV5s/cYZjxZtOt/zXy3LQyHvjV5AvOrtZgkfHgAD ThuNX50frmcN/+U8SQJ0ha7qYkqNuBT++3YWoOj3dN1gZnzZ1ZhmZq/TX8i4eq2MEab7 rHVpsHSX4WECyXPa3nBVXvNfOhfm0iDjdsakEsIpmBCC9gOMcbdJ6aq3d81VkAMikp6S xdqgPv/aGn6j1dgfYdQUxhatkGDr9JJlxMZbLQvsvnixQn5b06o1XqyqZmtiINrGe3He QMow== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id o19-v6si506042pgl.212.2018.07.17.02.14.14; Tue, 17 Jul 2018 02:14:28 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729715AbeGQJoG (ORCPT + 99 others); Tue, 17 Jul 2018 05:44:06 -0400 Received: from smtp.nue.novell.com ([195.135.221.5]:55584 "EHLO smtp.nue.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728089AbeGQJoF (ORCPT ); Tue, 17 Jul 2018 05:44:05 -0400 Received: from emea4-mta.ukb.novell.com ([10.120.13.87]) by smtp.nue.novell.com with ESMTP (TLS encrypted); Tue, 17 Jul 2018 11:12:26 +0200 Received: from suselix (nwb-a10-snat.microfocus.com [10.120.13.201]) by emea4-mta.ukb.novell.com with ESMTP (TLS encrypted); Tue, 17 Jul 2018 10:11:54 +0100 Date: Tue, 17 Jul 2018 11:11:52 +0200 From: Andreas Herrmann To: "Rafael J. Wysocki" Cc: "Rafael J. Wysocki" , Peter Zijlstra , Frederic Weisbecker , Viresh Kumar , Linux PM , Linux Kernel Mailing List Subject: Re: Commit 554c8aa8ecad causing severe performance degression with pcc-cpufreq Message-ID: <20180717091152.l4ixicbp6imvqtsr@suselix> References: <20180717065048.74mmgk4t5utjaa6a@suselix> <20180717085039.kqxwbkgruhj5qxtx@suselix> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170421 (1.8.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 17, 2018 at 11:06:29AM +0200, Rafael J. Wysocki wrote: > On Tue, Jul 17, 2018 at 10:50 AM, Andreas Herrmann wrote: > > [cut] > > > > > On balance before this commit users could use pcc-cpufreq but had > > already suboptimal performance (compared to say intel_pstate driver > > which can be used changing BIOS options). > > BTW, I wonder why you need to change the BIOS options for intel_pstate to load. I think this is because of (in intel_pstate_init()): /* * The Intel pstate driver will be ignored if the platform * firmware has its own power management modes. */ if (intel_pstate_platform_pwr_mgmt_exists()) return -ENODEV; > It should be initialized before pcc-cpufreq (according to their > respective initcall levels), so in theory intel_pstate should be used > by default on the affected systems anyway. > What BIOS settings need to be changed for that? Already answered in other mail. Andreas