Re: [PATCH v7 2/7] perf config: Add '--system' and '--user' options to select which config file is used

2015-10-21 Thread TaeWoong Song
Hi, Namhyung Thanks for your review. > On Oct 21, 2015, at 1:20 PM, Namhyung Kim wrote: > > On Sun, Oct 04, 2015 at 04:35:05PM +0900, Taeung Song wrote: >> Which config file is used is decided in only perf_config(). >> And a perf-config command depend on perf_config() to list >> config variable

Re: [PATCH v4 4/5] perf config: Add a option 'list-all' to perf-config

2015-08-09 Thread Taewoong Song
> On Aug 9, 2015, at 12:01 PM, Namhyung Kim wrote: > > On Fri, Aug 07, 2015 at 10:12:02AM +0900, taeung wrote: >> Hi, Namhyung >> >> On 07/27/2015 05:48 PM, Namhyung Kim wrote: >>> On Mon, Jul 27, 2015 at 12:58:30AM +0900, Taeung Song wrote: A option 'list-all' is to display both current c

Re: [PATCH v4 4/5] perf config: Add a option 'list-all' to perf-config

2015-08-08 Thread Taewoong Song
Hi, Namhyung > On Aug 7, 2015, at 10:12 AM, taeung wrote: > > Hi, Namhyung > > On 07/27/2015 05:48 PM, Namhyung Kim wrote: >> On Mon, Jul 27, 2015 at 12:58:30AM +0900, Taeung Song wrote: >>> A option 'list-all' is to display both current config variables and >>> all possible config variables w

Re: [PATCH v3 2/5] perf config: Add functions which can get or set perf config variables.

2015-07-19 Thread Taewoong Song
> On Jul 12, 2015, at 11:33 PM, Jiri Olsa wrote: > > On Sun, Jul 12, 2015 at 02:10:56PM +0900, Taeung Song wrote: >> This patch consists of functions >> which can get, set specific config variables. >> For the syntax examples, >> >> perf config [options] [section.name[=value] ...] >> >> di

Re: [PATCH v3 4/5] perf mem: Fill in the missing freeing a session after an error occur

2015-07-01 Thread Taewoong Song
I’m sorry. I made a mistake. So I sent again patch mail as [PATCH RESEND v3 4/5] Thanks, Taeung > On Jul 1, 2015, at 10:09 PM, Jiri Olsa wrote: > > On Wed, Jul 01, 2015 at 08:59:48PM +0900, Taeung Song wrote: >> When an error occur a error value is just returned >> without freeing the session.

Re: [PATCH 2/2] perf tools: Add a option 'remove' to perf-config and features which get or set a config variable.

2015-04-12 Thread Taewoong Song
Hi, jirka > On Apr 10, 2015, at 6:31 PM, Jiri Olsa wrote: > > On Thu, Apr 09, 2015 at 11:26:31PM +0900, Taeung Song wrote: >> This patch consists of adding functions >> which get, set or remove a specific config variable. >> For the syntax examples, >> >>perf config [options] [section.subke