Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp4872084pxb; Sun, 7 Nov 2021 00:59:19 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz518Rfkl0aDl2NOFX96/o+pKZYT7L5NiLIkBbfRL+2SeZhMIgCLvKBuPTWtaZ0GueCA/ln X-Received: by 2002:a05:6638:190f:: with SMTP id p15mr17835740jal.82.1636271959589; Sun, 07 Nov 2021 00:59:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1636271959; cv=none; d=google.com; s=arc-20160816; b=NXiZpnvPMyTn/d8mryJMuZ7SU9xPJkO91MDk4zwq24AClrZShArYypjlslbaRZo35/ sZ1OXuUGrYpMv9pwvOdbnOxTT1NdUBizF8QZnmYL+oecMVVLI3RRVMVIprNfHkNKmE9Z WIaeDW70ylDa6zxYpVXidl+Za9L14ESfm18eQlTuoVDC5kiK0B831ixV5LWNkJc0Zmss DbpeZrWu9g/VblPrkWSCfHK82htAOw7D3dYUQXl30JoIEH+4YHG3mPgyUc1Ddmi57nac IM2g+/lnlfZHOvYAdV7f0Bsgi5j6uvek/JvLisMZpos4wTzChb8zSBmeIsBreJlg6VQ0 nrog== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=fIJBqPyIVe5yR0jB8wcOmWjWqm+98ECy0TUpADKCpyk=; b=bqD+7N9T0m15lKhXtJavMntTOL62Yy7ynEU6IC5VsqHCcu1xHrn7NI5GVRhVry10n6 /0HcqGP8zmGArQ896cKILyVfo3UD1J1kPhl7wSwRetQWXxCMlP25NdOanTGZNNfzJemY 7isxLS/v1XW7EIr8bL9zF0U+872wqmsYNciwUnSPkeB5AGBGfmEH75YlGzVKsZZImOaS mG36xeI68NmuQtPpg42nnXwGkYFwbUm3utyqZBQjA3bceQOvxwIqWQfWdWnL6aIFvqBq TS8ytC5ggzAGfiYuwazkO8VNI/jjI8dDVGhk+0/VCVLefsB14QePkcVRp0mQsbyUMoJQ 0uxQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=YFcAv2qB; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id h22si17908455jal.48.2021.11.07.00.59.06; Sun, 07 Nov 2021 00:59:19 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=YFcAv2qB; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234976AbhKFTzq (ORCPT + 99 others); Sat, 6 Nov 2021 15:55:46 -0400 Received: from mail.kernel.org ([198.145.29.99]:52350 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229727AbhKFTzp (ORCPT ); Sat, 6 Nov 2021 15:55:45 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 2C74A61054; Sat, 6 Nov 2021 19:53:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1636228384; bh=qOGVHV2slOC5KIVm+e1qOM8RcHuz8RRw18Wkv9dHLGA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=YFcAv2qBwo6SX9n6aQ+4q0Uyjo7i879ialfqOLPD4spS8Sweh3QPkd0s7uMyrBzPc KFVzmcJKhr1bl2YEelz/R+eA4nPyVqViwpSiPZ54170W2M11XXfqsXsiGh/0VlieI2 L8UUu19g47Wee1/lS59TkSVourVJbbWK4JFp3kNheT7hNrMf3Yb7elcRIfFZrqlgnx JLt6LVLDUXmOlwxl9joReuVpC4buZFNl12rt+hHCmuD8nvjNrZvA72ugusnXvexe/F sTQyaclbHzSzpfMJlDwqmY5wFXPcB9f3k6XFLKytIquuyt6QzlhkWxKsxT+GjJNTr8 YjQidxwULyKRQ== Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 38D31410A1; Sat, 6 Nov 2021 16:53:02 -0300 (-03) Date: Sat, 6 Nov 2021 16:53:02 -0300 From: Arnaldo Carvalho de Melo To: Denis Nikitin Cc: James Clark , linux-perf-users@vger.kernel.org, Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] perf inject: Add vmlinux and ignore-vmlinux arguments Message-ID: References: <20211018134844.2627174-1-james.clark@arm.com> <20211018134844.2627174-4-james.clark@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Url: http://acmel.wordpress.com Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Fri, Nov 05, 2021 at 09:19:10PM -0700, Denis Nikitin escreveu: > On Mon, Oct 18, 2021 at 6:49 AM James Clark wrote: > > > > Other perf tools allow specifying the path to vmlinux. Perf inject > > didn't have this argument which made some auxtrace workflows difficult. > > > > Also add ignore-vmlinux for consistency with other tools. > > > > Suggested-by: Denis Nitikin > > Signed-off-by: James Clark > > --- > > tools/perf/builtin-inject.c | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/tools/perf/builtin-inject.c b/tools/perf/builtin-inject.c > > index 6ad191e731fc..4261ad89730f 100644 > > --- a/tools/perf/builtin-inject.c > > +++ b/tools/perf/builtin-inject.c > > @@ -938,6 +938,10 @@ int cmd_inject(int argc, const char **argv) > > #endif > > OPT_INCR('v', "verbose", &verbose, > > "be more verbose (show build ids, etc)"), > > + OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name, > > + "file", "vmlinux pathname"), > > + OPT_BOOLEAN(0, "ignore-vmlinux", &symbol_conf.ignore_vmlinux, > > + "don't load vmlinux even if found"), > > I think we also need to update documentation at Documentation/perf-inject.txt I can replicate what is in 'perf report' where this came from, will do it now. > > OPT_STRING(0, "kallsyms", &symbol_conf.kallsyms_name, "file", > > "kallsyms pathname"), > > OPT_BOOLEAN('f', "force", &data.force, "don't complain, do it"), > > @@ -972,6 +976,9 @@ int cmd_inject(int argc, const char **argv) > > return -1; > > } > > > > + if (symbol__validate_sym_arguments()) > > + return -1; > > + > > if (inject.in_place_update) { > > if (!strcmp(inject.input_name, "-")) { > > pr_err("Input file name required for in-place updating\n"); > > -- > > 2.28.0 > > > > Tested-by: Denis Nikitin Thanks, - Arnaldo