Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Wed, 19 Oct 2016 15:06:34 -0300 Arnaldo Carvalho de Melo wrote: > Em Wed, Oct 19, 2016 at 03:05:48PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Wed, Oct 19, 2016 at 02:48:45PM -0300, Arnaldo Carvalho de Melo > > escreveu: > > > Em Tue, Oct 18, 2016 at 11:29:53AM

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Wed, 19 Oct 2016 15:06:34 -0300 Arnaldo Carvalho de Melo wrote: > Em Wed, Oct 19, 2016 at 03:05:48PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Wed, Oct 19, 2016 at 02:48:45PM -0300, Arnaldo Carvalho de Melo > > escreveu: > > > Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Wed, 19 Oct 2016 15:05:48 -0300 Arnaldo Carvalho de Melo wrote: > Some are the compiler not grokking logic where the compiler gets > confused with logic that tests one variable to use another and thinks it > is using garbage (uninitialized stuff), I tried to follow the

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Wed, 19 Oct 2016 15:05:48 -0300 Arnaldo Carvalho de Melo wrote: > Some are the compiler not grokking logic where the compiler gets > confused with logic that tests one variable to use another and thinks it > is using garbage (uninitialized stuff), I tried to follow the logic and > I think

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Wed, 19 Oct 2016 14:48:45 -0300 Arnaldo Carvalho de Melo wrote: > Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > > On Tue, 18 Oct 2016 11:01:09 +0900 > > Namhyung Kim wrote: > > > > > Hi Honggyu, > > > > > > You need to CC

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Wed, 19 Oct 2016 14:48:45 -0300 Arnaldo Carvalho de Melo wrote: > Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > > On Tue, 18 Oct 2016 11:01:09 +0900 > > Namhyung Kim wrote: > > > > > Hi Honggyu, > > > > > > You need to CC relevant maintainers when you send patches

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Em Wed, Oct 19, 2016 at 02:48:45PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > > On Tue, 18 Oct 2016 11:01:09 +0900 > > Namhyung Kim wrote: > > > > > Hi Honggyu, > > > > > > You need to CC relevant

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Em Wed, Oct 19, 2016 at 02:48:45PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > > On Tue, 18 Oct 2016 11:01:09 +0900 > > Namhyung Kim wrote: > > > > > Hi Honggyu, > > > > > > You need to CC relevant maintainers when you send

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Em Wed, Oct 19, 2016 at 03:05:48PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Oct 19, 2016 at 02:48:45PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > > > On Tue, 18 Oct 2016 11:01:09 +0900 > > > Namhyung Kim

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Em Wed, Oct 19, 2016 at 03:05:48PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Oct 19, 2016 at 02:48:45PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > > > On Tue, 18 Oct 2016 11:01:09 +0900 > > > Namhyung Kim wrote: > >

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > On Tue, 18 Oct 2016 11:01:09 +0900 > Namhyung Kim wrote: > > > Hi Honggyu, > > > > You need to CC relevant maintainers when you send patches to LKML. > > For the libtraceevent, they are Arnaldo and Steven.

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Em Tue, Oct 18, 2016 at 11:29:53AM -0400, Steven Rostedt escreveu: > On Tue, 18 Oct 2016 11:01:09 +0900 > Namhyung Kim wrote: > > > Hi Honggyu, > > > > You need to CC relevant maintainers when you send patches to LKML. > > For the libtraceevent, they are Arnaldo and Steven. You can use > >

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Tue, 18 Oct 2016 11:01:09 +0900 Namhyung Kim wrote: > Hi Honggyu, > > You need to CC relevant maintainers when you send patches to LKML. > For the libtraceevent, they are Arnaldo and Steven. You can use > scripts/get_maintainer.pl for this job later. In addition

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

On Tue, 18 Oct 2016 11:01:09 +0900 Namhyung Kim wrote: > Hi Honggyu, > > You need to CC relevant maintainers when you send patches to LKML. > For the libtraceevent, they are Arnaldo and Steven. You can use > scripts/get_maintainer.pl for this job later. In addition running >

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Hi Honggyu, You need to CC relevant maintainers when you send patches to LKML. For the libtraceevent, they are Arnaldo and Steven. You can use scripts/get_maintainer.pl for this job later. In addition running scripts/checkpatch.pl before sending patches is a good habit. Arnaldo and Steve,

Re: [PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Hi Honggyu, You need to CC relevant maintainers when you send patches to LKML. For the libtraceevent, they are Arnaldo and Steven. You can use scripts/get_maintainer.pl for this job later. In addition running scripts/checkpatch.pl before sending patches is a good habit. Arnaldo and Steve,

[PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Since current traceevent somehow does not have an optimization flag, this patch just adds -O2 to optimize its code. Signed-off-by: Honggyu Kim --- tools/lib/traceevent/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/3] tools lib traceevent: Add -O2 option to traceevent

Since current traceevent somehow does not have an optimization flag, this patch just adds -O2 to optimize its code. Signed-off-by: Honggyu Kim --- tools/lib/traceevent/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/lib/traceevent/Makefile