From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf0-f67.google.com (mail-lf0-f67.google.com [209.85.215.67]) by dpdk.org (Postfix) with ESMTP id 2ECEE1B226 for ; Sat, 11 Nov 2017 19:56:06 +0100 (CET) Received: by mail-lf0-f67.google.com with SMTP id o66so6118872lfg.0 for ; Sat, 11 Nov 2017 10:56:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=ZCvXBggHUGQnUYfuG8wV9TMa2MRJADnDqHmf5te56Es=; b=Epyg7F6jxZJzZ7epLoOKOcMQSiSZ1stc2mzOPO7EriTJfYroeSV5WV5ihRfEqdmYM6 Ll0On3XBiwCDsTHigO3Lx17sZRH4tYz68OY70OUwLRJbEkRG0Yh4rxNIZ+l8t14DxUPk NfKfSuzYIUtyVtlk6XdiM15eJ8vjCqJhx1pEE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=ZCvXBggHUGQnUYfuG8wV9TMa2MRJADnDqHmf5te56Es=; b=jrE9TBMi79Y1PydkypGKrKj6QQTF8CQzvtqf0P96HvpG8QtxXwqBzHzTGWzHiH2Pxq GOXvrhK0r5+4MiZXABOkbcJSoYbjpG39zZKRFyJXpDp7ZxE87+wg5gY+1SnVZNaPMbgk ZyasTg0QzNZwrynrKLEUlHZjONaZ+rMX2qPqFyheGg3zhk8vy/f3m5ZEzI4s2xqs/Q0D mtaWoFrH1+U/Bfmb1mQltyQAbaeN1zVqOf98g/y+Jv4d7Q5x3KNbH7IJeN52LbM9m0mC +TtehLQuRRmc/kWXvmW6njqXC61ZazPapZhoI++bFr7xWiT51s94XqbGeBn44sRtkQ3b 2tog== X-Gm-Message-State: AJaThX78gj3SjzEW7x6eUb0oq3wYPJ3KazL8itvONENZThWnIxlzHC4S fiC2+DbqHXd/XV7eopLGfVxNVg== X-Google-Smtp-Source: AGs4zMbrSsa5kkVtUs8cXDoZzJBDTn4A3+cOxH3ci30lMnQkyHwKd1x1I7gC3kI8HuR8r84apYVlJA== X-Received: by 10.25.43.9 with SMTP id r9mr1304565lfr.128.1510426566551; Sat, 11 Nov 2017 10:56:06 -0800 (PST) Received: from rad-H81M-S1.semihalf.local (31-172-191-173.noc.fibertech.net.pl. [31.172.191.173]) by smtp.gmail.com with ESMTPSA id x90sm2342107ljb.86.2017.11.11.10.56.05 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Sat, 11 Nov 2017 10:56:05 -0800 (PST) From: Radoslaw Biernacki To: dev@dpdk.org, david.hunt@intel.com Cc: stable@dpdk.org, alan.carew@intel.com, pablo.de.lara.guarch@intel.com Date: Sat, 11 Nov 2017 19:55:07 +0100 Message-Id: <1510426507-28245-4-git-send-email-radoslaw.biernacki@linaro.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1510426507-28245-1-git-send-email-radoslaw.biernacki@linaro.org> References: <1508161628-4265-1-git-send-email-radoslaw.biernacki@linaro.org> <1510426507-28245-1-git-send-email-radoslaw.biernacki@linaro.org> Subject: [dpdk-stable] [PATCH 3/3] power: check if userspace governor is available X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 11 Nov 2017 18:56:07 -0000 Since for new Intel CPU's kernel use intel_pstate driver, which does not offer userspace governor, it is wise to check the userspace governor availability before trying to perform governor switch. The outcome from this patch is direct information for user about the root cause of failure during the rte_power_acpi_cpufreq_init(). Signed-off-by: Radoslaw Biernacki --- lib/librte_power/rte_power_acpi_cpufreq.c | 74 +++++++++++++++++++++++++++++++ 1 file changed, 74 insertions(+) diff --git a/lib/librte_power/rte_power_acpi_cpufreq.c b/lib/librte_power/rte_power_acpi_cpufreq.c index f811bd3..0d9a2a5 100644 --- a/lib/librte_power/rte_power_acpi_cpufreq.c +++ b/lib/librte_power/rte_power_acpi_cpufreq.c @@ -65,6 +65,11 @@ #define POWER_CONVERT_TO_DECIMAL 10 #define POWER_GOVERNOR_USERSPACE "userspace" +#define POWER_SCALING_DRIVER_INTEL_PSTATE "intel_pstate" +#define POWER_SYSFILE_SCALING_DRIVER \ + "/sys/devices/system/cpu/cpu%u/cpufreq/scaling_driver" +#define POWER_SYSFILE_AVAIL_GOVERNOR \ + "/sys/devices/system/cpu/cpu%u/cpufreq/scaling_available_governors" #define POWER_SYSFILE_GOVERNOR \ "/sys/devices/system/cpu/cpu%u/cpufreq/scaling_governor" #define POWER_SYSFILE_AVAIL_FREQ \ @@ -139,6 +144,70 @@ set_freq_internal(struct rte_power_info *pi, uint32_t idx) return 1; } +/* check /sys file for presence of given string */ +static int +power_check_sysfile_string(const char *fullpath, const char *string) +{ + int fd; + int count; + int ret = -1; + char buf[BUFSIZ]; + + fd = open(fullpath, O_RDONLY); + if (fd < 0) { + RTE_LOG(ERR, POWER, "Failed to open %s\n", fullpath); + return ret; + } + + count = read(fd, buf, sizeof(buf)); + if (count < 0) { + RTE_LOG(ERR, POWER, "Failed to read from %s\n", fullpath); + goto out; + } + buf[min((size_t)count, sizeof(buf)-1)] = '\0'; + + if (strstr(buf, string)) + ret = 0; /* substring found */ + +out: + if (close(fd)) { + RTE_LOG(ERR, POWER, "Error while closing file %s\n", fullpath); + return -1; + } + + return ret; +} + +/* check if userspace governor is offered by the kernel */ +static int power_check_gov_userspace(unsigned int lcore_id) +{ + char fullpath[PATH_MAX]; + + snprintf(fullpath, sizeof(fullpath), + POWER_SYSFILE_AVAIL_GOVERNOR, lcore_id); + if (power_check_sysfile_string( + fullpath, POWER_GOVERNOR_USERSPACE)) { + + RTE_LOG(ERR, POWER, "Userspace governor is not available\n"); + + snprintf(fullpath, sizeof(fullpath), + POWER_SYSFILE_SCALING_DRIVER, lcore_id); + if (!power_check_sysfile_string( + fullpath, POWER_SCALING_DRIVER_INTEL_PSTATE)) { + + RTE_LOG(WARNING, POWER, + POWER_SCALING_DRIVER_INTEL_PSTATE "was detected" + " which does not offer userspace power governor" + ". \"intel_pstate=disable\" needs to be added " + "to the kernel boot parameters.\n"); + } + + return -1; + } + + return 0; +} + /** * It is to check the current scaling governor by reading sys file, and then * set it into 'userspace' if it is not by writing the sys file. The original @@ -154,6 +223,11 @@ power_set_governor(unsigned int lcore_id, const char *new_gov, char *old_gov, char buf[BUFSIZ]; char fullpath[PATH_MAX]; + /* check if userspace governor is available */ + if (power_check_gov_userspace(lcore_id)) + return ret; + + /* store current governor and set userspace governor */ snprintf(fullpath, sizeof(fullpath), POWER_SYSFILE_GOVERNOR, lcore_id); fd = open(fullpath, O_RDWR); -- 2.7.4