Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-14 Thread Arnaldo Carvalho de Melo
Em Thu, Dec 14, 2017 at 01:08:45PM +0100, Thomas-Mich Richter escreveu: > On 12/12/2017 06:12 PM, Arnaldo Carvalho de Melo wrote: > > Em Tue, Dec 12, 2017 at 05:24:43PM +0100, Michael Petlan escreveu: > >> Hey Arnaldo, this one did not remove the s390x hack and also, won't work > >> on arm. Please

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-14 Thread Thomas-Mich Richter
On 12/12/2017 06:12 PM, Arnaldo Carvalho de Melo wrote: > Em Tue, Dec 12, 2017 at 05:24:43PM +0100, Michael Petlan escreveu: >> Hey Arnaldo, this one did not remove the s390x hack and also, won't work >> on arm. Please use the one I just have sent, few seconds ago... > > Ok, I have you latest in,

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-12 Thread Arnaldo Carvalho de Melo
Em Tue, Dec 12, 2017 at 05:24:43PM +0100, Michael Petlan escreveu: > Hey Arnaldo, this one did not remove the s390x hack and also, won't work > on arm. Please use the one I just have sent, few seconds ago... Ok, I have you latest in, with the cset log I wrote, which clarifies that this is not a ar

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-12 Thread Michael Petlan
On Tue, 12 Dec 2017, Arnaldo Carvalho de Melo wrote: [...] > the point here is not to add any new patch for perf trace, etc, but find > a way to fix just this test, so I think this works: > > # evts=$(echo $(perf list syscalls:sys_enter_open* |& egrep 'open(at)? ' | > sed -r 's/.*sys_enter_([a-z]

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-12 Thread Michael Petlan
Hey Arnaldo, this one did not remove the s390x hack and also, won't work on arm. Please use the one I just have sent, few seconds ago... On Tue, 12 Dec 2017, Arnaldo Carvalho de Melo wrote: > Em Tue, Dec 12, 2017 at 11:57:07AM -0300, Arnaldo Carvalho de Melo escreveu: > > So the patch below does t

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-12 Thread Arnaldo Carvalho de Melo
Em Tue, Dec 12, 2017 at 11:57:07AM -0300, Arnaldo Carvalho de Melo escreveu: > So the patch below does the trick for me, can you please check if does > for you? So I've put this with a long explanation at: https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/commit/?h=tmp.perf/core&id=b

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-12 Thread Arnaldo Carvalho de Melo
Em Tue, Dec 12, 2017 at 04:05:23AM +0100, Michael Petlan escreveu: > On Mon, 11 Dec 2017, Arnaldo Carvalho de Melo wrote: > > It is not working here: > > Hmm, I think I got it. > > The following construction: > > evts=`perf list syscalls:sys_enter_open* |& egrep 'open(at)? ' | sed -r > 's/.*sys

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-11 Thread Michael Petlan
On Mon, 11 Dec 2017, Arnaldo Carvalho de Melo wrote: > It is not working here: Hmm, I think I got it. The following construction: evts=`perf list syscalls:sys_enter_open* |& egrep 'open(at)? ' | sed -r 's/.*sys_enter_(.*) +\[.*/-e \1/'` ... expands to: "-e open -e openat" ... which is a form

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-11 Thread Arnaldo Carvalho de Melo
Em Mon, Dec 11, 2017 at 01:43:12PM -0200, Arnaldo de Melo escreveu: > Em Fri, Dec 08, 2017 at 06:48:17PM +0100, Michael Petlan escreveu: > > Hi Arnaldo, so I have tried what you've suggested and looks good. > > Patch is attached. Sorry for not posting it in-text, but I need to > > fix my mail clien

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-11 Thread Arnaldo de Melo
Em Fri, Dec 08, 2017 at 06:48:17PM +0100, Michael Petlan escreveu: > Hi Arnaldo, so I have tried what you've suggested and looks good. > Patch is attached. Sorry for not posting it in-text, but I need to > fix my mail client first, since it screwes some patches up due to > flowed-text... > Cheers,