From patchwork Mon Aug 7 06:37:45 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Patchwork-Id: 131699 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp1270114vqr; Mon, 7 Aug 2023 00:03:04 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFrgt1sTOH/P11FBL4wy97KNMBWtLauitXLTcvgCVwqXBmKOqX78jOGHygaQnuqGsVifgyU X-Received: by 2002:a17:90b:1e4f:b0:269:13f0:4b9a with SMTP id pi15-20020a17090b1e4f00b0026913f04b9amr6529535pjb.7.1691391783575; Mon, 07 Aug 2023 00:03:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691391783; cv=none; d=google.com; s=arc-20160816; b=0XFHoU2ioFRCpQ1qI1kRnjroKTx5DN/f02FrmJ+h5qxMnKtt8sCHqG/gkufjhUuitw c6bVPqsm6f0T/3MfnuQ/5/QAay/Y6iG8Erc6EIgLtgFwll95Tl5rh5oK3oUBT1txYijo /FjfCLSTmHrEEwvG7isezksuE9qMIOi4/E+k5W/bODzzBa8J59Drv4WhJaJrGSNYetUI i9xh8VmlG1EOWHSUf5oOP/1kaVjP7hNp7IpFH6WIHGnbyIbXluv39IsVlkqVBalh/9pW bgz9mNEEorlM6U3s0iXAxF+Vvd+FIa1dcZGgUl/egyRAXqeUchDEVSX2GupqkqCpuY5p AdWg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:message-id:content-transfer-encoding :mime-version:subject:date:from:dkim-signature; bh=l7K6nF90lbXHABIflOeHIaUCac9bblrokbU6aCeJgGc=; fh=UGgyWs5nXuk2bpyxEk3gnREM4i2EElZdMfCWXrqWwCQ=; b=CaBYpXwGNEtvOmB8/HKwXRR6HDemfhhMLW+jAwSTeTMNdF+gn69J1We2vuj1YuNIBr GWmNWp5y6dwF2HPn4HqV+llaiYCI0KK7lE9GCxrRPQNsGBvshGBSnnwNUcHVPHLWVpkC VXyom916G3ceX7VjSmuqFqAfmabFk9nxWPBA7GmyH2/HGlvhjwRcaEg8N5NFLvN3B0/a WDHxq1d8OWQJMOxCcW4YNkHzMIM/Lv0DvhaFQywtH312TSEAZkZb8UzYv3Z825qABl55 atgXLWlMYkBgUqk8jex5i63nUhPC4xaYzHWyitSmptHNieLLXDQvE6+8QreZ4pTSwgrm bjPw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@weissschuh.net header.s=mail header.b=lx40vcbj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id lr12-20020a17090b4b8c00b0026810b074afsi9207500pjb.3.2023.08.07.00.02.50; Mon, 07 Aug 2023 00:03:03 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@weissschuh.net header.s=mail header.b=lx40vcbj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230526AbjHGGiB (ORCPT + 99 others); Mon, 7 Aug 2023 02:38:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42868 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231293AbjHGGh5 (ORCPT ); Mon, 7 Aug 2023 02:37:57 -0400 Received: from todd.t-8ch.de (todd.t-8ch.de [IPv6:2a01:4f8:c010:41de::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F1EFA12A; Sun, 6 Aug 2023 23:37:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=weissschuh.net; s=mail; t=1691390273; bh=ni6N3FT+XqiJ1D10eJkiShr+T3NXxFp1tGK/dEGHs94=; h=From:Date:Subject:To:Cc:From; b=lx40vcbjL6vrGNekhAE+X6MIBtVcF/9QBHxUdBXVvnIPl077XuMuthI+I6UpBn9Pm a6c5WFAGFBJ+wFF2Z+rcd3fcaM9x+2Ur32Oi6Hzkdl/IAA7jazO1XipwrCDcW15213 VPz+6rJxzA7uDn1Ixp1PHT+pQtBUG65Z8OQnxKZM= From: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= Date: Mon, 07 Aug 2023 08:37:45 +0200 Subject: [PATCH] cpufreq: amd-pstate: fix global sysfs attribute type MIME-Version: 1.0 Message-Id: <20230807-amd-pstate-cfi-v1-1-0263daa13bc3@weissschuh.net> X-B4-Tracking: v=1; b=H4sIADiR0GQC/x3MQQqAIBBA0avErBswNbKuEi3ExppFJhoRhHdPW r7F/y9kSkwZpuaFRDdnPkNF1zbgdhs2Ql6rQQqphBED2mPFmC97ETrPaJSQejSu176HGsVEnp9 /OC+lfPGOUZhgAAAA To: Huang Rui , "Rafael J. Wysocki" , Viresh Kumar Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Jannik Glueckert , Bagas Sanjaya , Linux LLVM Build Support , Linux Power Management , Nick Desaulniers , Nathan Chancellor , Greg Kroah-Hartman , Huang Rui , stable@vger.kernel.org, =?utf-8?q?Thomas_Wei?= =?utf-8?q?=C3=9Fschuh?= X-Mailer: b4 0.12.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1691390272; l=3581; i=linux@weissschuh.net; s=20221212; h=from:subject:message-id; bh=ni6N3FT+XqiJ1D10eJkiShr+T3NXxFp1tGK/dEGHs94=; b=gECqiO43NLSBF6EL/FWzJyIhuHS2wygf/Ze967FjXMXfQkKO52PsscEBiQIOqW717HMIei6am PObnA9ZA3cQChixTd/K2OVlorY/QrC+f3/ueKDkn3eRIzK7qR+WIPtq X-Developer-Key: i=linux@weissschuh.net; a=ed25519; pk=KcycQgFPX2wGR5azS7RhpBqedglOZVgRPfdFSPB1LNw= X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773552830828917319 X-GMAIL-MSGID: 1773552830828917319 In commit 3666062b87ec ("cpufreq: amd-pstate: move to use bus_get_dev_root()") the "amd_pstate" attributes where moved from a dedicated kobject to the cpu root kobject. While the dedicated kobject expects to contain kobj_attributes the root kobject needs device_attributes. As the changed arguments are not used by the callbacks it works most of the time. However CFI will detect this issue: [ 4947.849350] CFI failure at dev_attr_show+0x24/0x60 (target: show_status+0x0/0x70; expected type: 0x8651b1de) ... [ 4947.849409] Call Trace: [ 4947.849410] [ 4947.849411] ? __warn+0xcf/0x1c0 [ 4947.849414] ? dev_attr_show+0x24/0x60 [ 4947.849415] ? report_cfi_failure+0x4e/0x60 [ 4947.849417] ? handle_cfi_failure+0x14c/0x1d0 [ 4947.849419] ? __cfi_show_status+0x10/0x10 [ 4947.849420] ? handle_bug+0x4f/0x90 [ 4947.849421] ? exc_invalid_op+0x1a/0x60 [ 4947.849422] ? asm_exc_invalid_op+0x1a/0x20 [ 4947.849424] ? __cfi_show_status+0x10/0x10 [ 4947.849425] ? dev_attr_show+0x24/0x60 [ 4947.849426] sysfs_kf_seq_show+0xa6/0x110 [ 4947.849433] seq_read_iter+0x16c/0x4b0 [ 4947.849436] vfs_read+0x272/0x2d0 [ 4947.849438] ksys_read+0x72/0xe0 [ 4947.849439] do_syscall_64+0x76/0xb0 [ 4947.849440] ? do_user_addr_fault+0x252/0x650 [ 4947.849442] ? exc_page_fault+0x7a/0x1b0 [ 4947.849443] entry_SYSCALL_64_after_hwframe+0x72/0xdc Reported-by: Jannik Glückert Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217765 Link: https://lore.kernel.org/lkml/c7f1bf9b-b183-bf6e-1cbb-d43f72494083@gmail.com/ Fixes: 3666062b87ec ("cpufreq: amd-pstate: move to use bus_get_dev_root()") Cc: stable@vger.kernel.org Signed-off-by: Thomas Weißschuh Reviewed-by: Greg Kroah-Hartman Reviewed-by: Nathan Chancellor --- Note: This was not tested with CFI as I don't have the toolchain available. Jannik, could you give it a spin? --- drivers/cpufreq/amd-pstate.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) --- base-commit: 52a93d39b17dc7eb98b6aa3edb93943248e03b2f change-id: 20230807-amd-pstate-cfi-8302498c54f5 Best regards, diff --git a/drivers/cpufreq/amd-pstate.c b/drivers/cpufreq/amd-pstate.c index 81fba0dcbee9..9a1e194d5cf8 100644 --- a/drivers/cpufreq/amd-pstate.c +++ b/drivers/cpufreq/amd-pstate.c @@ -1012,8 +1012,8 @@ static int amd_pstate_update_status(const char *buf, size_t size) return 0; } -static ssize_t show_status(struct kobject *kobj, - struct kobj_attribute *attr, char *buf) +static ssize_t status_show(struct device *dev, + struct device_attribute *attr, char *buf) { ssize_t ret; @@ -1024,7 +1024,7 @@ static ssize_t show_status(struct kobject *kobj, return ret; } -static ssize_t store_status(struct kobject *a, struct kobj_attribute *b, +static ssize_t status_store(struct device *a, struct device_attribute *b, const char *buf, size_t count) { char *p = memchr(buf, '\n', count); @@ -1043,7 +1043,7 @@ cpufreq_freq_attr_ro(amd_pstate_lowest_nonlinear_freq); cpufreq_freq_attr_ro(amd_pstate_highest_perf); cpufreq_freq_attr_rw(energy_performance_preference); cpufreq_freq_attr_ro(energy_performance_available_preferences); -define_one_global_rw(status); +static DEVICE_ATTR_RW(status); static struct freq_attr *amd_pstate_attr[] = { &amd_pstate_max_freq, @@ -1062,7 +1062,7 @@ static struct freq_attr *amd_pstate_epp_attr[] = { }; static struct attribute *pstate_global_attributes[] = { - &status.attr, + &dev_attr_status.attr, NULL };