Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-07-09 Thread Namhyung Kim
Hi David, 2013-07-09 오후 11:21, David Ahern 쓴 글: On 7/9/13 1:41 AM, Namhyung Kim wrote: Hi Arnaldo, You may want to merge this patch too. :) He did. See 77d03596 and the note: [ combined patch removing the -f usage in various sub-commands, such as 'perf sched', etc, by Namhyung Kim ]

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-07-09 Thread David Ahern
On 7/9/13 1:41 AM, Namhyung Kim wrote: Hi Arnaldo, You may want to merge this patch too. :) He did. See 77d03596 and the note: [ combined patch removing the -f usage in various sub-commands, such as 'perf sched', etc, by Namhyung Kim ] David -- To unsubscribe from this list: send the

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-07-09 Thread Namhyung Kim
Hi Arnaldo, You may want to merge this patch too. :) Thanks, Namhyung 2013-06-27 오후 1:25, Namhyung Kim 쓴 글: From: Namhyung Kim The commit bf3da4014a0c ("perf record: Remove -f/--force option") got rid of -f option from perf record. But this option was used internally by various sub-command

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-28 Thread Ingo Molnar
* David Ahern wrote: > On 6/28/13 9:37 AM, Ingo Molnar wrote: > > > >* David Ahern wrote: > > > >>On 6/28/13 3:47 AM, Jiri Olsa wrote: > >I thought -f was the implied default for ages? > > OK.. I've been dutifully typing it all this while :-) > >>> > >>>The '-f' option in record co

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-28 Thread David Ahern
On 6/28/13 9:37 AM, Ingo Molnar wrote: * David Ahern wrote: On 6/28/13 3:47 AM, Jiri Olsa wrote: I thought -f was the implied default for ages? OK.. I've been dutifully typing it all this while :-) The '-f' option in record command had no affect.. myabe it got depreceated when we started

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-28 Thread Ingo Molnar
* David Ahern wrote: > On 6/28/13 3:47 AM, Jiri Olsa wrote: > >>>I thought -f was the implied default for ages? > >> > >>OK.. I've been dutifully typing it all this while :-) > > > >The '-f' option in record command had no affect.. myabe it got > >depreceated when we started to backup perf.data

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-28 Thread David Ahern
On 6/28/13 3:47 AM, Jiri Olsa wrote: I thought -f was the implied default for ages? OK.. I've been dutifully typing it all this while :-) The '-f' option in record command had no affect.. myabe it got depreceated when we started to backup perf.data to perf.data.old..? Way back in 2010, 2.6.

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-28 Thread Jiri Olsa
On Thu, Jun 27, 2013 at 12:47:50PM +0200, Peter Zijlstra wrote: > On Thu, Jun 27, 2013 at 12:39:31PM +0200, Ingo Molnar wrote: > > > > * Peter Zijlstra wrote: > > > > > On Thu, Jun 27, 2013 at 01:25:20PM +0900, Namhyung Kim wrote: > > > > From: Namhyung Kim > > > > > > > > The commit bf3da4014

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-27 Thread Peter Zijlstra
On Thu, Jun 27, 2013 at 12:39:31PM +0200, Ingo Molnar wrote: > > * Peter Zijlstra wrote: > > > On Thu, Jun 27, 2013 at 01:25:20PM +0900, Namhyung Kim wrote: > > > From: Namhyung Kim > > > > > > The commit bf3da4014a0c ("perf record: Remove -f/--force option") got > > > rid of -f option from pe

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-27 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Thu, Jun 27, 2013 at 01:25:20PM +0900, Namhyung Kim wrote: > > From: Namhyung Kim > > > > The commit bf3da4014a0c ("perf record: Remove -f/--force option") got > > rid of -f option from perf record. But this option was used > > internally by various sub-commands s

Re: [PATCH] perf tools: Fixup for removing -f option in perf record

2013-06-27 Thread Peter Zijlstra
On Thu, Jun 27, 2013 at 01:25:20PM +0900, Namhyung Kim wrote: > From: Namhyung Kim > > The commit bf3da4014a0c ("perf record: Remove -f/--force option") got > rid of -f option from perf record. But this option was used > internally by various sub-commands so they wouldn't work anymore. > Also up