From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Google-Smtp-Source: AB8JxZokaCv0v6Tm79SgXTAyfG8Ux992bJXr5ofOeNxJplAp2SA0m7Vmz+OeIETd10nrVmpDMsSh ARC-Seal: i=1; a=rsa-sha256; t=1526282265; cv=none; d=google.com; s=arc-20160816; b=r4xopM7qhALCYBIGn4qfZgBy/fNxQxEdgPKCIDfwJVGmvBByER2mHpSQtk+PpUH+9f xEsDlJM/HbLYSzfBNKOCHZOkq6UPBLIzNooK/+sEoESKOcGgOcl+ezrzUH1TMLG06dUV EwLT54vig1A6D+S/SyDou1tlaaeW304WnZFZQN91gkVJSjvXW2FN634D5TkheQH9hpRA 5oBp6UdiQKUlAK+Zg7zvceccWvYR9fB7O8Rf5bUgYdmHPJekW6WnrNqbUmpNRJkQTjDG mmvm9qp9Kb19OkBSQZHr4Lu3QO6KpZFwqTZQUMV3U0KS7e/a9FhSofnky/3qutHrpuqu Jllw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:user-agent:references:in-reply-to:message-id:date :subject:cc:to:from:dkim-signature:arc-authentication-results; bh=/smfZpcxT1kQfhyCtt5/H/WflCJMh2Kn0LfUSxWGqEg=; b=WwfIbquc7hl6vEGLbniPZgJQ5cEyZWQC0aIhvzMWCeWCQpbHrHO/OiPPbo4hSsiPv+ mM+TifphXoCzuKOvp6G6AfnMtZqfi6ESwu6lXZcbhFTTzX9sNmxajhrmvYtO/s2CIs8z KSr4iI0ysFZfjZ715wULk5tYrHIssxqkKq7TJezhED6r7GZNhfAmbyIkWFS8NCK82L2I FJmjmjgYVChO9vOdPmAEmWEiDG8C47DuSmeNgVGv9vZgFG6FMd2lH02Y0vgvahFjsL83 u3n3B4QZ131/UNYGCpVNkSH8DFz85yPpUEjylp0CABhaIWsZdy6Eani9iZcd8Cdw8WnB PjYQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=0ncM31WV; spf=pass (google.com: domain of srs0=ywzk=ib=linuxfoundation.org=gregkh@kernel.org designates 198.145.29.99 as permitted sender) smtp.mailfrom=SRS0=ywzk=IB=linuxfoundation.org=gregkh@kernel.org Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=0ncM31WV; spf=pass (google.com: domain of srs0=ywzk=ib=linuxfoundation.org=gregkh@kernel.org designates 198.145.29.99 as permitted sender) smtp.mailfrom=SRS0=ywzk=IB=linuxfoundation.org=gregkh@kernel.org From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Dan Carpenter , "Peter Zijlstra (Intel)" , Alexander Shishkin , Arnaldo Carvalho de Melo , Jiri Olsa , Linus Torvalds , Stephane Eranian , Thomas Gleixner , Vince Weaver , Ingo Molnar Subject: [PATCH 4.9 32/36] perf/x86: Fix possible Spectre-v1 indexing for hw_perf_event cache_* Date: Mon, 14 May 2018 08:49:06 +0200 Message-Id: <20180514064806.733966440@linuxfoundation.org> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180514064804.252823817@linuxfoundation.org> References: <20180514064804.252823817@linuxfoundation.org> User-Agent: quilt/0.65 X-stable: review MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-LABELS: =?utf-8?b?IlxcU2VudCI=?= X-GMAIL-THRID: =?utf-8?q?1600422453930408065?= X-GMAIL-MSGID: =?utf-8?q?1600422952486758272?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: 4.9-stable review patch. If anyone has any objections, please let me know. ------------------ From: Peter Zijlstra commit ef9ee4ad38445a30909c48998624861716f2a994 upstream. > arch/x86/events/core.c:319 set_ext_hw_attr() warn: potential spectre issue 'hw_cache_event_ids[cache_type]' (local cap) > arch/x86/events/core.c:319 set_ext_hw_attr() warn: potential spectre issue 'hw_cache_event_ids' (local cap) > arch/x86/events/core.c:328 set_ext_hw_attr() warn: potential spectre issue 'hw_cache_extra_regs[cache_type]' (local cap) > arch/x86/events/core.c:328 set_ext_hw_attr() warn: potential spectre issue 'hw_cache_extra_regs' (local cap) Userspace controls @config which contains 3 (byte) fields used for a 3 dimensional array deref. Reported-by: Dan Carpenter Signed-off-by: Peter Zijlstra (Intel) Cc: Cc: Alexander Shishkin Cc: Arnaldo Carvalho de Melo Cc: Jiri Olsa Cc: Linus Torvalds Cc: Peter Zijlstra Cc: Stephane Eranian Cc: Thomas Gleixner Cc: Vince Weaver Signed-off-by: Ingo Molnar Signed-off-by: Greg Kroah-Hartman --- arch/x86/events/core.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) --- a/arch/x86/events/core.c +++ b/arch/x86/events/core.c @@ -303,17 +303,20 @@ set_ext_hw_attr(struct hw_perf_event *hw config = attr->config; - cache_type = (config >> 0) & 0xff; + cache_type = (config >> 0) & 0xff; if (cache_type >= PERF_COUNT_HW_CACHE_MAX) return -EINVAL; + cache_type = array_index_nospec(cache_type, PERF_COUNT_HW_CACHE_MAX); cache_op = (config >> 8) & 0xff; if (cache_op >= PERF_COUNT_HW_CACHE_OP_MAX) return -EINVAL; + cache_op = array_index_nospec(cache_op, PERF_COUNT_HW_CACHE_OP_MAX); cache_result = (config >> 16) & 0xff; if (cache_result >= PERF_COUNT_HW_CACHE_RESULT_MAX) return -EINVAL; + cache_result = array_index_nospec(cache_result, PERF_COUNT_HW_CACHE_RESULT_MAX); val = hw_cache_event_ids[cache_type][cache_op][cache_result];