From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnaldo Carvalho de Melo Subject: Re: [RFC PATCH net-next 1/4] perf tools: Enable pre-event inherit setting by config terms Date: Wed, 28 Oct 2015 10:44:20 -0300 Message-ID: <20151028134420.GA17877@kernel.org> References: <1446029705-199659-1-git-send-email-wangnan0@huawei.com> <1446029705-199659-2-git-send-email-wangnan0@huawei.com> <20151028132126.GD5216@krava.brq.redhat.com> <20151028134213.GC18670@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Wang Nan , ast@kernel.org, brendan.d.gregg@gmail.com, jolsa@kernel.org, lizefan@huawei.com, pi3orama@163.com, davem@davemloft.net, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Alexei Starovoitov , Peter Zijlstra To: Jiri Olsa Return-path: Content-Disposition: inline In-Reply-To: <20151028134213.GC18670@kernel.org> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Em Wed, Oct 28, 2015 at 10:42:13AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Oct 28, 2015 at 02:21:26PM +0100, Jiri Olsa escreveu: > > On Wed, Oct 28, 2015 at 10:55:02AM +0000, Wang Nan wrote: > > > @@ -55,6 +56,7 @@ struct perf_evsel_config_term { > > > bool time; > > > char *callgraph; > > > u64 stack_user; > > > + u64 inherit; > > > > seems like bool would be enough > > Ok, will change this and move it to a more suitable place member > alignment wise. Nah, switched it to bool, but no need to move it around, that is an union... > Can I, with this change, slap an Acked-by: jirka? > > - Arnaldo