From nobody Sat Feb 7 18:15:45 2026 Received: from out-181.mta0.migadu.com (out-181.mta0.migadu.com [91.218.175.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3A8E521422E for ; Mon, 10 Mar 2025 19:24:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741634642; cv=none; b=VX8sG4zRbigb3LTDt9IlfGcLW15gQ4FFb5E16WUNjGE73bxpoRrmpnhu3nmPVji+fk9vOBAjwahPCxAYntKXWc1sQk0+91W9QKZq3OVB3WmEDpNHnZNZvO4y+HBVYBAyvgvfANO6I9hd670+bPYkW4/zGwppe4rB4Sdqlm37FG8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741634642; c=relaxed/simple; bh=IBF/Hw53mk/kxLoojQzHTGPDpYoSnM0LJgRfCo+CXxw=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=LI7L7pKmBofdiXej+568Qk1MsIi/gssBazaYQhGihWgL22haocDZwsgbmcH0g1c48BhucpaUtuuv87P2Z8X6aqMgPAx6hwaqDBFOuDx70aYTb3KEaMctF+jLAxV77J36i3dBTvngRdhjINuqCblydfZJi0qLPoDdcVSm4ZRVsis= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=eQpaMRWJ; arc=none smtp.client-ip=91.218.175.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="eQpaMRWJ" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1741634628; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=rXMAaVXrQ0MJdauHGwB3sTIki68VqN7FkX+wHWA/QGE=; b=eQpaMRWJdqK5ySjcyDP+SLpV1lXS11PaIH3FnlTh4oxD0DVUY75Lun3XLlM/taAhwReIDB JRVotSfoCiV0MZRnLcMnOQFd0hOoLxxlTMYQV0kA7vhfn9qfiyuSnCO5M86kAtdt4EFKqn CWoZYANnYOy8nEZQICYkEwBjzTOSy0M= From: Thorsten Blum To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , "Liang, Kan" Cc: Thorsten Blum , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] perf/core: Remove size arguments when calling strscpy() Date: Mon, 10 Mar 2025 20:23:35 +0100 Message-ID: <20250310192336.442994-1-thorsten.blum@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Migadu-Flow: FLOW_OUT Content-Type: text/plain; charset="utf-8" The size parameter is optional and strscpy() automatically determines the length of the destination buffer using sizeof() if the argument is omitted. This makes the explicit sizeof() calls unnecessary. Furthermore, KSYM_NAME_LEN is equal to sizeof(name) and can also be removed. Remove them to shorten and simplify the code. Signed-off-by: Thorsten Blum --- kernel/events/core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/events/core.c b/kernel/events/core.c index 823aa0824916..e4b071fc7d74 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c @@ -8589,7 +8589,7 @@ static void perf_event_comm_event(struct perf_comm_ev= ent *comm_event) unsigned int size; =20 memset(comm, 0, sizeof(comm)); - strscpy(comm, comm_event->task->comm, sizeof(comm)); + strscpy(comm, comm_event->task->comm); size =3D ALIGN(strlen(comm)+1, sizeof(u64)); =20 comm_event->comm =3D comm; @@ -9033,7 +9033,7 @@ static void perf_event_mmap_event(struct perf_mmap_ev= ent *mmap_event) } =20 cpy_name: - strscpy(tmp, name, sizeof(tmp)); + strscpy(tmp, name); name =3D tmp; got_name: /* @@ -9457,7 +9457,7 @@ void perf_event_ksymbol(u16 ksym_type, u64 addr, u32 = len, bool unregister, ksym_type =3D=3D PERF_RECORD_KSYMBOL_TYPE_UNKNOWN) goto err; =20 - strscpy(name, sym, KSYM_NAME_LEN); + strscpy(name, sym); name_len =3D strlen(name) + 1; while (!IS_ALIGNED(name_len, sizeof(u64))) name[name_len++] =3D '\0'; --=20 2.48.1