From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 0F1E22F32 for ; Thu, 19 Oct 2023 13:22:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=none Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1CB8CC433CD; Thu, 19 Oct 2023 13:22:09 +0000 (UTC) Date: Thu, 19 Oct 2023 09:22:07 -0400 From: Steven Rostedt To: kernel test robot Cc: Tom Zanussi , oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: kernel/trace/trace_kprobe.c:952: warning: Excess function parameter 'args' description in '__kprobe_event_gen_cmd_start' Message-ID: <20231019092207.7d2fa545@gandalf.local.home> In-Reply-To: <202310190437.paI6LYJF-lkp@intel.com> References: <202310190437.paI6LYJF-lkp@intel.com> X-Mailer: Claws Mail 3.19.1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: oe-kbuild-all@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Thu, 19 Oct 2023 04:33:41 +0800 kernel test robot wrote: > All warnings (new ones prefixed by >>): > > >> kernel/trace/trace_kprobe.c:952: warning: Excess function parameter 'args' description in '__kprobe_event_gen_cmd_start' > >> kernel/trace/trace_kprobe.c:1017: warning: Excess function parameter 'args' description in '__kprobe_event_add_fields' > > > vim +952 kernel/trace/trace_kprobe.c > > 929 > 930 /** > 931 * __kprobe_event_gen_cmd_start - Generate a kprobe event command from arg list > 932 * @cmd: A pointer to the dynevent_cmd struct representing the new event > 933 * @name: The name of the kprobe event > 934 * @loc: The location of the kprobe event > 935 * @kretprobe: Is this a return probe? > 936 * @args: Variable number of arg (pairs), one pair for each field > 937 * > 938 * NOTE: Users normally won't want to call this function directly, but > 939 * rather use the kprobe_event_gen_cmd_start() wrapper, which automatically > 940 * adds a NULL to the end of the arg list. If this function is used > 941 * directly, make sure the last arg in the variable arg list is NULL. > 942 * > 943 * Generate a kprobe event command to be executed by > 944 * kprobe_event_gen_cmd_end(). This function can be used to generate the > 945 * complete command or only the first part of it; in the latter case, > 946 * kprobe_event_add_fields() can be used to add more fields following this. > 947 * > 948 * Return: 0 if successful, error otherwise. > 949 */ > 950 int __kprobe_event_gen_cmd_start(struct dynevent_cmd *cmd, bool kretprobe, > 951 const char *name, const char *loc, ...) > > 952 { So what is the proper kerneldoc way to express variable length arguments? -- Steve