From nobody Wed Dec 17 12:44:28 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 48B95C4167B for ; Mon, 27 Nov 2023 22:09:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233348AbjK0WJh (ORCPT ); Mon, 27 Nov 2023 17:09:37 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46784 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233570AbjK0WJ0 (ORCPT ); Mon, 27 Nov 2023 17:09:26 -0500 Received: from mail-yw1-x114a.google.com (mail-yw1-x114a.google.com [IPv6:2607:f8b0:4864:20::114a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F09FDD5A for ; Mon, 27 Nov 2023 14:09:32 -0800 (PST) Received: by mail-yw1-x114a.google.com with SMTP id 00721157ae682-5c994880223so71177537b3.0 for ; Mon, 27 Nov 2023 14:09:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1701122972; x=1701727772; darn=vger.kernel.org; h=to:from:subject:references:mime-version:message-id:in-reply-to:date :from:to:cc:subject:date:message-id:reply-to; bh=ujtQ3QhQpwDoNxv/mBiBRV+6eibZuimyXiHKFflebSQ=; b=COmBgx4L1//l84yqThO705JA0rnfhKDL7zGmyRiscQFiJ8m1D2SSwPk4sAaUUMvmTg 80yYZQ8UEmRQjs7qeURMKM5phtmMkOF8smhRPOaKSdtwjrmHXNuvWNyg1A/JcrpQUkHz BZZydtTZ+jmUEHm2/dNE3v2ovnfCD972cm2w+yQrrTM45qO1DlKSF5R8K/vUn7g094r0 5ZrgHetW1dEjIHlPjT9TxpKQPx65u1ebrAUpzsszB0gxzVdP2nA3Lva4sRZyd1stXWv9 UajzHr0cAB1lMbHqK63cs6NeCyjOJbryjQ35cioSjPuwu6tc1BBTzXOOC1H4051lteww mapQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1701122972; x=1701727772; h=to:from:subject:references:mime-version:message-id:in-reply-to:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ujtQ3QhQpwDoNxv/mBiBRV+6eibZuimyXiHKFflebSQ=; b=J0he0skL6xrhckmnpndv10p6luf+7adSKSIJTFJpRbxUJySjXARWT4+DkjizUuxdxo ag6WWPOktgtcJ8WBdXU4/b72xfLoDGzIhyWoOBn52PlJ+0C1qu8Uli3sB30EO1tU/GKI zFnqC4hWAd8UhNYVnmrBjlRTecFSwFRjXVkpsWUp6o+rA4n1tvBzyUbiqVYvZ7yt18fE p5reV0XK7ZV4uFLS03o80qSfsRq9qzlqYk2tICc+G/+WM2XZxFLpO1KZQiKOJOIlL1QX Egz/xu50Vv3ldbsJdmjlEicMPJyFtd/BmeFCSWS/4eMb2u6v0BrYr8frogJdCTjfJf+o bhZg== X-Gm-Message-State: AOJu0Yw4ts0TLYxXYgmTfilLRVND/Qu8TMQ630UfGuBRFp/MhCSqwM/W SUxNejY/htMlGbhWnKq1zFFtGngN2nMt X-Google-Smtp-Source: AGHT+IE15NVm0P2tzub/k7bVxns1jsuHZLV+W+ubNfzDivu1+mCTMHwctkrWJwb3d1cSHMdlaehOprszJXNj X-Received: from irogers.svl.corp.google.com ([2620:15c:2a3:200:829:6e77:9093:f39b]) (user=irogers job=sendgmr) by 2002:a05:690c:3105:b0:5cc:4276:1c6b with SMTP id fb5-20020a05690c310500b005cc42761c6bmr362765ywb.4.1701122972182; Mon, 27 Nov 2023 14:09:32 -0800 (PST) Date: Mon, 27 Nov 2023 14:08:17 -0800 In-Reply-To: <20231127220902.1315692-1-irogers@google.com> Message-Id: <20231127220902.1315692-6-irogers@google.com> Mime-Version: 1.0 References: <20231127220902.1315692-1-irogers@google.com> X-Mailer: git-send-email 2.43.0.rc1.413.gea7ed67945-goog Subject: [PATCH v5 05/50] tools api fs: Avoid reading whole file for a 1 byte bool From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Ian Rogers , Adrian Hunter , Nick Terrell , Kan Liang , Andi Kleen , Kajol Jain , Athira Rajeev , Huacai Chen , Masami Hiramatsu , Vincent Whitchurch , "Steinar H. Gunderson" , Liam Howlett , Miguel Ojeda , Colin Ian King , Dmitrii Dolgov <9erthalion6@gmail.com>, Yang Jihong , Ming Wang , James Clark , K Prateek Nayak , Sean Christopherson , Leo Yan , Ravi Bangoria , German Gomez , Changbin Du , Paolo Bonzini , Li Dong , Sandipan Das , liuwenyu , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Guilherme Amadio Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" sysfs__read_bool used the first byte from a fully read file into a string. It then looked at the first byte's value. Avoid doing this and just read the first byte. Signed-off-by: Ian Rogers Acked-by: Namhyung Kim --- tools/lib/api/fs/fs.c | 24 +++++++++++++++--------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/tools/lib/api/fs/fs.c b/tools/lib/api/fs/fs.c index 004f2af5504b..337fde770e45 100644 --- a/tools/lib/api/fs/fs.c +++ b/tools/lib/api/fs/fs.c @@ -447,15 +447,22 @@ int sysfs__read_str(const char *entry, char **buf, si= ze_t *sizep) =20 int sysfs__read_bool(const char *entry, bool *value) { - char *buf; - size_t size; - int ret; + struct io io; + char bf[16]; + int ret =3D 0; + char path[PATH_MAX]; + const char *sysfs =3D sysfs__mountpoint(); + + if (!sysfs) + return -1; =20 - ret =3D sysfs__read_str(entry, &buf, &size); - if (ret < 0) - return ret; + snprintf(path, sizeof(path), "%s/%s", sysfs, entry); + io.fd =3D open(path, O_RDONLY); + if (io.fd < 0) + return -errno; =20 - switch (buf[0]) { + io__init(&io, io.fd, bf, sizeof(bf)); + switch (io__get_char(&io)) { case '1': case 'y': case 'Y': @@ -469,8 +476,7 @@ int sysfs__read_bool(const char *entry, bool *value) default: ret =3D -1; } - - free(buf); + close(io.fd); =20 return ret; } --=20 2.43.0.rc1.413.gea7ed67945-goog