From nobody Fri Dec 19 15:01:57 2025 Received: from mail-pl1-f201.google.com (mail-pl1-f201.google.com [209.85.214.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 584F51FC7F1 for ; Wed, 4 Jun 2025 17:46:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749059164; cv=none; b=fTj2KmVBbbKgNDVEei7GzzGUtH4mExIzByZC+o7qHeGacWs12e7n3pNy22X3cUcDnaNCbPDQo/xvQAgs9NtsAss98LHb8V2BBWkVPcMxlAv/F+O9WxgytYi1RDG+9ZY9YqPovU5l6fV6Nm+fAL7Jk9nKTySzUdM9URfEZdk7FZ0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749059164; c=relaxed/simple; bh=l/r16VO14AuoTPhHYx1IhtNWRI73iyFImyAUkeVPBI0=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Content-Type; b=vGJEPw1NFWeRR/3LnHO2xyAzUoBoPc3tD24hs2bk4sLm2Sop5gQRRMDAYvo8oCRY1j3lbIgiKBM4QE9kRhHjMSHt84/5TraS2ljUOoe2IfYi3xErb+fdaEyhdnIziPH/hySam4DgvRbxU5uiCSYVQpYz0VUbSyiMZwIU5thg3oo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=wuxJs9Oo; arc=none smtp.client-ip=209.85.214.201 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="wuxJs9Oo" Received: by mail-pl1-f201.google.com with SMTP id d9443c01a7336-23592f5fcb0so1022965ad.3 for ; Wed, 04 Jun 2025 10:46:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1749059161; x=1749663961; darn=vger.kernel.org; h=to:from:subject:message-id:references:mime-version:in-reply-to:date :from:to:cc:subject:date:message-id:reply-to; bh=B8tuEWlyOsSZ3VWtxSAzfgMOdxNTH4pB9RszdQHfYhc=; b=wuxJs9OohDW4lQF0jMy3TAyejzjvXLOIqdVkebSsCXnIOTe7cu72rmSOrLpbui46vU oLV6yincNi7rAuHOet3780XE1yNAF3BYFuzqPUOsA6m1i/1grN+y94GyJfsdXjmw2wA/ /Qwt1Y0eRJeBDwR0jZLELFxUqNpuKJB1EU+GnBADPk2XSJojYY0guZYkLO0nXvOZlqzi tv+KTBCMRj41b/ywtd4mw6O3JhVGWD+1KKSTXWs9HxLub2OKo9X+oIxpExcuZ21F5sSF kGA3rEV0QLKSbJ1SsR6b/TOvGBeNgFWcxWgdfJa0TcEKtM2hIhT7CeziZ/YpJllhEe1b l6wQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1749059161; x=1749663961; h=to:from:subject:message-id:references:mime-version:in-reply-to:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=B8tuEWlyOsSZ3VWtxSAzfgMOdxNTH4pB9RszdQHfYhc=; b=N7UbLIG9rReM9+/lbipxmB7dlnG46eRw+BLbka9wE95B6s9Hm9qV1/eACsNEvqU14o ChAdxG4Tn5IODL9/mLSdwzVoloiJ/1CBX6zNKceopEVC958wNkJ/nvVgRxv1xNC4Jzrz 7+dIoopDMHkX2ye6udk/XeqzMIMoFuOTWujTNC8SyOJneF5SLjVshCSo6dHa+FFsQYIu WbsFBHFoJw7G2pOvLiqiIwKoG5P0qktpJh6eQsRScnpix67ayW1kM6ceA5VepLvr+hee vqIGXt+gEmzr4moviM7jnkWLmo0FilL7qbvocn9PJxBjASmd8EmUoZBzp6RD4PxeGS2N gi8A== X-Forwarded-Encrypted: i=1; AJvYcCUmCspQ8Ut1UX9iyz3Fg4Y4QgFZFg/S6hcO/0+MLngvqqsv9On5t2wWLHeQdymi5Y8C6Uo23ggpeSkicKo=@vger.kernel.org X-Gm-Message-State: AOJu0YwNhmFxzatoE5oBFIq4Clmo9vMaBYxFYK18dydGVX7ollm7JHlI TGSP6mGyE92rkNenV4NzifS3LVdFykpUi0DGJLx7ZfkFpqmU0cKh34BlCnE6WB/XPhIFyM9CCQZ rgPB54h1M1w== X-Google-Smtp-Source: AGHT+IFUdQJpYyNZoW3LNov2nnp6DBt7iCUI1cgeGKklZf5D9ju5lKohUdpptZ+KpaNA/8E19pxOS5sBSnwz X-Received: from pjbpa10.prod.google.com ([2002:a17:90b:264a:b0:312:dbc:f731]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a17:903:298f:b0:233:ab04:27a with SMTP id d9443c01a7336-235e128b23amr51021535ad.53.1749059161671; Wed, 04 Jun 2025 10:46:01 -0700 (PDT) Date: Wed, 4 Jun 2025 10:45:36 -0700 In-Reply-To: <20250604174545.2853620-1-irogers@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20250604174545.2853620-1-irogers@google.com> X-Mailer: git-send-email 2.50.0.rc0.604.gd4ff7b7c86-goog Message-ID: <20250604174545.2853620-3-irogers@google.com> Subject: [PATCH v4 02/10] perf target: Separate parse_uid into its own function From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , Kan Liang , James Clark , Dapeng Mi , Thomas Richter , Veronika Molnarova , Chun-Tse Shao , Leo Yan , Hao Ge , Howard Chu , Weilin Wang , Levi Yun , "Dr. David Alan Gilbert" , Gautam Menghani , Tengda Wu , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Allow parse_uid to be called without a struct target. Rather than have two errors, remove TARGET_ERRNO__USER_NOT_FOUND and use TARGET_ERRNO__INVALID_UID as the handling is identical. Signed-off-by: Ian Rogers --- tools/perf/util/target.c | 22 ++++++++++++---------- tools/perf/util/target.h | 3 ++- 2 files changed, 14 insertions(+), 11 deletions(-) diff --git a/tools/perf/util/target.c b/tools/perf/util/target.c index 0f383418e3df..f3ad59ccfa99 100644 --- a/tools/perf/util/target.c +++ b/tools/perf/util/target.c @@ -94,15 +94,13 @@ enum target_errno target__validate(struct target *targe= t) return ret; } =20 -enum target_errno target__parse_uid(struct target *target) +uid_t parse_uid(const char *str) { struct passwd pwd, *result; char buf[1024]; - const char *str =3D target->uid_str; =20 - target->uid =3D UINT_MAX; if (str =3D=3D NULL) - return TARGET_ERRNO__SUCCESS; + return UINT_MAX; =20 /* Try user name first */ getpwnam_r(str, &pwd, buf, sizeof(buf), &result); @@ -115,16 +113,22 @@ enum target_errno target__parse_uid(struct target *ta= rget) int uid =3D strtol(str, &endptr, 10); =20 if (*endptr !=3D '\0') - return TARGET_ERRNO__INVALID_UID; + return UINT_MAX; =20 getpwuid_r(uid, &pwd, buf, sizeof(buf), &result); =20 if (result =3D=3D NULL) - return TARGET_ERRNO__USER_NOT_FOUND; + return UINT_MAX; } =20 - target->uid =3D result->pw_uid; - return TARGET_ERRNO__SUCCESS; + return result->pw_uid; +} + +enum target_errno target__parse_uid(struct target *target) +{ + target->uid =3D parse_uid(target->uid_str); + + return target->uid !=3D UINT_MAX ? TARGET_ERRNO__SUCCESS : TARGET_ERRNO__= INVALID_UID; } =20 /* @@ -142,7 +146,6 @@ static const char *target__error_str[] =3D { "BPF switch overriding UID", "BPF switch overriding THREAD", "Invalid User: %s", - "Problems obtaining information for user %s", }; =20 int target__strerror(struct target *target, int errnum, @@ -171,7 +174,6 @@ int target__strerror(struct target *target, int errnum, break; =20 case TARGET_ERRNO__INVALID_UID: - case TARGET_ERRNO__USER_NOT_FOUND: snprintf(buf, buflen, msg, target->uid_str); break; =20 diff --git a/tools/perf/util/target.h b/tools/perf/util/target.h index 2ee2cc30340f..e082bda990fb 100644 --- a/tools/perf/util/target.h +++ b/tools/perf/util/target.h @@ -48,12 +48,13 @@ enum target_errno { =20 /* for target__parse_uid() */ TARGET_ERRNO__INVALID_UID, - TARGET_ERRNO__USER_NOT_FOUND, =20 __TARGET_ERRNO__END, }; =20 enum target_errno target__validate(struct target *target); + +uid_t parse_uid(const char *str); enum target_errno target__parse_uid(struct target *target); =20 int target__strerror(struct target *target, int errnum, char *buf, size_t = buflen); --=20 2.50.0.rc0.604.gd4ff7b7c86-goog