From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) (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 EF05E3D86 for ; Thu, 2 Jun 2022 21:12:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1654204373; x=1685740373; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=EG4fFmbtoez27DKDle4Ouq6qSysWshoV4GNB3C6WPzQ=; b=O1wx20eSJLt5DqoShEsgcrVtpCPrMNclORmXeHSgRI7/merNgRjYIewe kJgkc5jpyLpZWODiIPDYRze6JD8AOOpi6TNG/Ddsy0lsUx+YSTOWP37kE 1N7u4KLWSdcjxHJAreC2i7TwWQJrHiSsOLAqMpKX+2FEAg5w+Y9Li26du 6aICp6G5NjZvBuVmArs+PQNQyPHqK63MlsaT7ASXL14EPVHJotWHwC9m7 md6RrnEE+Kez0CXwqeNGiYCf5lelcSKiNoqScaq2I2ZS+O5Pk3Du98RWK fwTAlxB7QYpTQTAwZfdi189Ov4Jg/UwswXGx55hfgAf+JBduZ+bkNA2BO g==; X-IronPort-AV: E=McAfee;i="6400,9594,10366"; a="255946640" X-IronPort-AV: E=Sophos;i="5.91,272,1647327600"; d="scan'208";a="255946640" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Jun 2022 14:12:53 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.91,272,1647327600"; d="scan'208";a="634232611" Received: from lkp-server01.sh.intel.com (HELO 60dabacc1df6) ([10.239.97.150]) by fmsmga008.fm.intel.com with ESMTP; 02 Jun 2022 14:12:51 -0700 Received: from kbuild by 60dabacc1df6 with local (Exim 4.95) (envelope-from ) id 1nws7e-0005Rx-8h; Thu, 02 Jun 2022 21:12:50 +0000 Date: Fri, 3 Jun 2022 05:12:35 +0800 From: kernel test robot To: Jeff Xie , rostedt@goodmis.org Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org, mingo@redhat.com, mhiramat@kernel.org, zanussi@kernel.org, linux-kernel@vger.kernel.org, Jeff Xie Subject: Re: [PATCH v11 1/4] trace: Add trace any kernel object Message-ID: <202206030515.4lqqkb3W-lkp@intel.com> References: <20220602164518.286295-2-xiehuan09@gmail.com> Precedence: bulk X-Mailing-List: llvm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220602164518.286295-2-xiehuan09@gmail.com> Hi Jeff, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on b39181f7c6907dc66ff937b74758671fa6ba430c] url: https://github.com/intel-lab-lkp/linux/commits/Jeff-Xie/trace-Introduce-objtrace-trigger-to-trace-the-kernel-object/20220603-004723 base: b39181f7c6907dc66ff937b74758671fa6ba430c config: i386-randconfig-a013 (https://download.01.org/0day-ci/archive/20220603/202206030515.4lqqkb3W-lkp@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project b364c76683f8ef241025a9556300778c07b590c2) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/intel-lab-lkp/linux/commit/765253f020469f94856aedc5a3fe5444e1e8f4e8 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Jeff-Xie/trace-Introduce-objtrace-trigger-to-trace-the-kernel-object/20220603-004723 git checkout 765253f020469f94856aedc5a3fe5444e1e8f4e8 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/gpu/drm/qxl/ kernel/trace/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot All warnings (new ones prefixed by >>): >> kernel/trace/trace_object.c:336:61: warning: format specifies type 'long' but the argument has type 'unsigned int' [-Wformat] pr_err("the size of the %s should be:%ld\n", field->name, sizeof(void *)); ~~~ ^~~~~~~~~~~~~~ %u include/linux/printk.h:489:33: note: expanded from macro 'pr_err' printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__) ~~~ ^~~~~~~~~~~ include/linux/printk.h:446:60: note: expanded from macro 'printk' #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__) ~~~ ^~~~~~~~~~~ include/linux/printk.h:418:19: note: expanded from macro 'printk_index_wrap' _p_func(_fmt, ##__VA_ARGS__); \ ~~~~ ^~~~~~~~~~~ 1 warning generated. vim +336 kernel/trace/trace_object.c 297 298 static int 299 event_object_trigger_parse(struct event_command *cmd_ops, 300 struct trace_event_file *file, 301 char *glob, char *cmd, char *param_and_filter) 302 { 303 struct event_trigger_data *trigger_data; 304 struct objtrace_trigger_data *obj_data; 305 struct ftrace_event_field *field; 306 char *objtrace_cmd, *arg; 307 char *param, *filter; 308 int ret; 309 bool remove; 310 311 remove = event_trigger_check_remove(glob); 312 313 /* 314 * separate the param and the filter: 315 * objtrace:add:OBJ[:COUNT] [if filter] 316 */ 317 ret = event_trigger_separate_filter(param_and_filter, ¶m, &filter, true); 318 if (ret) 319 return ret; 320 321 objtrace_cmd = strsep(¶m, ":"); 322 if (!objtrace_cmd || strcmp(objtrace_cmd, OBJTRACE_CMD_ADD)) { 323 pr_err("error objtrace command\n"); 324 return -EINVAL; 325 } 326 327 arg = strsep(¶m, ":"); 328 if (!arg) 329 return -EINVAL; 330 331 field = trace_find_event_field(file->event_call, arg); 332 if (!field) 333 return -EINVAL; 334 335 if (field->size != sizeof(void *)) { > 336 pr_err("the size of the %s should be:%ld\n", field->name, sizeof(void *)); 337 return -EINVAL; 338 } 339 340 if (remove && !field_exist(file, cmd_ops, field->name)) 341 return -ENOENT; 342 343 obj_data = kzalloc(sizeof(*obj_data), GFP_KERNEL); 344 if (!obj_data) 345 return -ENOMEM; 346 347 obj_data->field = field; 348 obj_data->tr = file->tr; 349 snprintf(obj_data->objtrace_cmd, OBJTRACE_CMD_LEN, objtrace_cmd); 350 351 trigger_data = event_trigger_alloc(cmd_ops, cmd, param, obj_data); 352 if (!trigger_data) { 353 kfree(obj_data); 354 return -ENOMEM; 355 } 356 if (remove) { 357 event_trigger_unregister(cmd_ops, file, glob+1, trigger_data); 358 kfree(obj_data); 359 kfree(trigger_data); 360 return 0; 361 } 362 363 ret = event_trigger_parse_num(param, trigger_data); 364 if (ret) 365 goto out_free; 366 367 ret = event_trigger_set_filter(cmd_ops, file, filter, trigger_data); 368 if (ret < 0) 369 goto out_free; 370 371 ret = event_trigger_register(cmd_ops, file, glob, trigger_data); 372 if (ret) 373 goto out_free; 374 375 return ret; 376 377 out_free: 378 event_trigger_reset_filter(cmd_ops, trigger_data); 379 kfree(obj_data); 380 kfree(trigger_data); 381 return ret; 382 } 383 -- 0-DAY CI Kernel Test Service https://01.org/lkp