From nobody Wed Oct 8 10:58:51 2025 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (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 3EA7128D8E6 for ; Mon, 30 Jun 2025 14:36:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.44 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751294177; cv=none; b=EN9yuyP6isVSFsSXV2wpTn0WE0bLFruGlbEO6qTG7F/+jyQmPI/vZME1Vz1MJFLkTKxVN4i5W8cWNjtp+ktIE9LLzAnYa9mQAOYJpPDl635J+fBNPIW5ZOwxRWVL6yAu13WchsxD111B0uUDt2ocZxr6GIzyRLg37ZsfID6zp+g= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751294177; c=relaxed/simple; bh=PirEDsPKzKZfF4lma/U6ABELtX2SsjTdnouDkKmFdHs=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=FsTNmed2HnZWm0/Owa1S7uE0gsEnkykWUTHC1s6WtxjwjFrME+xht/pUa8gCZvxNrOwgg0TmyyyO0hkclDMIjBS9q5Wlvw04VTAPIeGIl7gX8jstPJ9ABnOPMTEqyr7ywggRaL8IQZ4Kun/v7gS3g/pcPayq1G1gsxL1+6G+OzA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=Fdc/Ksoy; arc=none smtp.client-ip=209.85.128.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="Fdc/Ksoy" Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-450cfb79177so26328555e9.0 for ; Mon, 30 Jun 2025 07:36:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1751294173; x=1751898973; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=mtpZAHSxzBc52yNVlVB1MC/4zpT1HtcP/IJyN7kdpFo=; b=Fdc/KsoyxO07X+4I1KFRHthpL1FloKU88LNyvrysXSZ5opWS9U46fOH7pTYeeNg+bM O5euxNbL4LzB15ObmkRDTqfwIjomyuDaXf7kNRlqGwKTB+mssv+5CBeEkM9Jt5XoTJTf OZ3qaA79sKVFGUR3E2562pQcYRXMq9D7x5DFdNhtuQnz9VUcIzdFEoMU0DVDG4/4l8oi OOHTkmgoqfWyHwv9G5d+RLAOFPNICijIIeFzB+YMVtzqix7uo/Lt1AFh3Kp9UEeEEFyj TqT7+5w54JakWldteR6PHa76I9/qrhdkrrQMx7n3+IXlbqQ8VdZ8+EPcget2iGDOysXH NVdQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751294173; x=1751898973; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=mtpZAHSxzBc52yNVlVB1MC/4zpT1HtcP/IJyN7kdpFo=; b=hcLrQOWmju3SXAujFxKF1xHNZNgYw1shjWJ0nMcsS65Nux/weFzOhZEfUKs1YfT8AO AzwLv3xBZh4nsrv82Bq9ai97ppMj/4qas+Ydcn5X1VJCyQpv6owDnH8K8DJQbLwpP2x9 VlfcxqBvzSJc7go8ZxSwJzmBzcoa5rqid2BKFFUhrtjNeZirfqbHp2uT6mV6c4g3xvox afFkLgrdS7gl/Ta01pB6+T366aV8izWfmq2WmMal//LJDjM/EFQnsv0F20HQJfJNxXFo wMXDUPR+x75qSjm8gvIPIi68WuboOTJKvrH7HhAFPkVEWEAuGcf+tO/16kS9PAnau9nQ X1EA== X-Forwarded-Encrypted: i=1; AJvYcCUAqjQ7EAw4LH0yu+Dkw5mIkAiTY4VfQIhAem7a7zMpxPH6aLKCCf7NTGvGBAHEytiWOgZwj8vKaq2oaBw=@vger.kernel.org X-Gm-Message-State: AOJu0Yyy/iCUn8cemT+Y7uS06Iva9IFRNw6iXHvsTvf9sMTkab8T811K +tFDvc8hfpZ2Lh0M7rTgeXi8SDMQtDWisYsEuEYHQOBsbLsdEWIApy2FwEV+gDlWcl8= X-Gm-Gg: ASbGncvfEFxY/71mABZdj1vpY9UgIx1toB1Bczmgg5ZVkgTNCX8IdoiL3p1jEZr3fYm j3ybQK7u13KI4NemXwOSgOb9y1LERpBQj2iOLvJmTzoMFjO4NKB8+yyGwufkhVGkXvSMlsLsGvQ 8sPlCb5yOBFPhgJpy68xfrADGJoJKUWn13YBbEFctK2W4uNIydk2izP8qv7o8deaDpjmVzHyn63 RZjf+raNEB9HDzHOt2cqfE117shRX8xrJJz06hurqn2fnp9b+9qJysRm2MOTI+fw2Ye2RG6g5wR wFHAm7VEpuXoVXI069gVotvveMdPPCfvwNo21YrKNZ28kNQjNdFFWTCmvW7bDw== X-Google-Smtp-Source: AGHT+IE65xiXePmfkTNzrE3cu741b/HUQtQk5TEnp05EileMIiACkLUMMykov2/RQ02qlLbUlc+wEw== X-Received: by 2002:a05:600c:1549:b0:450:d019:263 with SMTP id 5b1f17b1804b1-4538ee5d63emr156951575e9.18.1751294173517; Mon, 30 Jun 2025 07:36:13 -0700 (PDT) Received: from zovi.suse.cz ([193.86.92.181]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4538a4064b1sm139691695e9.29.2025.06.30.07.36.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jun 2025 07:36:13 -0700 (PDT) From: Petr Pavlu To: Luis Chamberlain , Petr Pavlu , Sami Tolvanen , Daniel Gomez Cc: linux-modules@vger.kernel.org, linux-kernel@vger.kernel.org, Steven Rostedt , Masami Hiramatsu Subject: [PATCH 4/5] tracing: Replace MAX_PARAM_PREFIX_LEN with MODULE_NAME_LEN Date: Mon, 30 Jun 2025 16:32:35 +0200 Message-ID: <20250630143535.267745-5-petr.pavlu@suse.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250630143535.267745-1-petr.pavlu@suse.com> References: <20250630143535.267745-1-petr.pavlu@suse.com> 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 Content-Type: text/plain; charset="utf-8" Use the MODULE_NAME_LEN definition in module_exists() to obtain the maximum size of a module name, instead of using MAX_PARAM_PREFIX_LEN. The values are the same but MODULE_NAME_LEN is more appropriate in this context. MAX_PARAM_PREFIX_LEN was added in commit 730b69d22525 ("module: check kernel param length at compile time, not runtime") only to break a circular dependency between module.h and moduleparam.h, and should mostly be limited to use in moduleparam.h. Signed-off-by: Petr Pavlu Cc: Steven Rostedt Cc: Masami Hiramatsu Acked-by: Steven Rostedt (Google) Reviewed-by: Daniel Gomez --- As a side note, I suspect the function module_exists() would be better replaced with !!find_module() + RCU locking, but that is a separate issue. kernel/trace/trace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 95ae7c4e5835..b9da0c4661a0 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -10373,7 +10373,7 @@ bool module_exists(const char *module) { /* All modules have the symbol __this_module */ static const char this_mod[] =3D "__this_module"; - char modname[MAX_PARAM_PREFIX_LEN + sizeof(this_mod) + 2]; + char modname[MODULE_NAME_LEN + sizeof(this_mod) + 2]; unsigned long val; int n; =20 --=20 2.49.0