Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-11-16 Thread Luiz Capitulino
On Tue, 15 Nov 2016 16:38:19 -0500 Steven Rostedt wrote: > On Fri, 28 Oct 2016 17:14:56 -0400 > Luiz Capitulino wrote: > > > > > > > > cpu_set(ret_mask, i) sets bits from begin to end in uint64_t cpumask > > > from below. > > > > > > What happens

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-11-16 Thread Luiz Capitulino
On Tue, 15 Nov 2016 16:38:19 -0500 Steven Rostedt wrote: > On Fri, 28 Oct 2016 17:14:56 -0400 > Luiz Capitulino wrote: > > > > > > > > cpu_set(ret_mask, i) sets bits from begin to end in uint64_t cpumask > > > from below. > > > > > > What happens if this range is greater than 64? We already

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-11-15 Thread Steven Rostedt
On Fri, 28 Oct 2016 17:14:56 -0400 Luiz Capitulino wrote: > > > > cpu_set(ret_mask, i) sets bits from begin to end in uint64_t cpumask > > from below. > > > > What happens if this range is greater than 64? We already have boxes > > that run this with 80 CPUs. Looks to

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-11-15 Thread Steven Rostedt
On Fri, 28 Oct 2016 17:14:56 -0400 Luiz Capitulino wrote: > > > > cpu_set(ret_mask, i) sets bits from begin to end in uint64_t cpumask > > from below. > > > > What happens if this range is greater than 64? We already have boxes > > that run this with 80 CPUs. Looks to be out of memory range

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Steven Rostedt
On Fri, 28 Oct 2016 17:15:57 -0400 Luiz Capitulino wrote: > On Fri, 28 Oct 2016 15:50:10 -0400 > Steven Rostedt wrote: > > > On Fri, 28 Oct 2016 15:49:22 -0400 > > Steven Rostedt wrote: > > > > > Sorry it took so long to look

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Steven Rostedt
On Fri, 28 Oct 2016 17:15:57 -0400 Luiz Capitulino wrote: > On Fri, 28 Oct 2016 15:50:10 -0400 > Steven Rostedt wrote: > > > On Fri, 28 Oct 2016 15:49:22 -0400 > > Steven Rostedt wrote: > > > > > Sorry it took so long to look at this, but I finally got around to > > > it ;-) > > > > > >

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Luiz Capitulino
On Fri, 28 Oct 2016 15:50:10 -0400 Steven Rostedt wrote: > On Fri, 28 Oct 2016 15:49:22 -0400 > Steven Rostedt wrote: > > > Sorry it took so long to look at this, but I finally got around to > > it ;-) > > > > I did apply patches 1 and 2. I'm

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Luiz Capitulino
On Fri, 28 Oct 2016 15:50:10 -0400 Steven Rostedt wrote: > On Fri, 28 Oct 2016 15:49:22 -0400 > Steven Rostedt wrote: > > > Sorry it took so long to look at this, but I finally got around to > > it ;-) > > > > I did apply patches 1 and 2. I'm hoping to get them out tomorrow, or > while I'm

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Luiz Capitulino
On Fri, 28 Oct 2016 15:49:22 -0400 Steven Rostedt wrote: > Sorry it took so long to look at this, but I finally got around to > it ;-) > > > On Fri, 7 Oct 2016 12:47:11 -0400 > Luiz Capitulino wrote: > > > With --cpu-list you can do: > > > > #

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Luiz Capitulino
On Fri, 28 Oct 2016 15:49:22 -0400 Steven Rostedt wrote: > Sorry it took so long to look at this, but I finally got around to > it ;-) > > > On Fri, 7 Oct 2016 12:47:11 -0400 > Luiz Capitulino wrote: > > > With --cpu-list you can do: > > > > # trace-cmd record --cpu-list 1,4,10-15 [...]

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Steven Rostedt
On Fri, 28 Oct 2016 15:49:22 -0400 Steven Rostedt wrote: > Sorry it took so long to look at this, but I finally got around to > it ;-) > I did apply patches 1 and 2. I'm hoping to get them out tomorrow, or while I'm at plumbers. -- Steve

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Steven Rostedt
On Fri, 28 Oct 2016 15:49:22 -0400 Steven Rostedt wrote: > Sorry it took so long to look at this, but I finally got around to > it ;-) > I did apply patches 1 and 2. I'm hoping to get them out tomorrow, or while I'm at plumbers. -- Steve

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Steven Rostedt
Sorry it took so long to look at this, but I finally got around to it ;-) On Fri, 7 Oct 2016 12:47:11 -0400 Luiz Capitulino wrote: > With --cpu-list you can do: > > # trace-cmd record --cpu-list 1,4,10-15 [...] > > Which is much more human friendly than -M. > >

Re: [PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-28 Thread Steven Rostedt
Sorry it took so long to look at this, but I finally got around to it ;-) On Fri, 7 Oct 2016 12:47:11 -0400 Luiz Capitulino wrote: > With --cpu-list you can do: > > # trace-cmd record --cpu-list 1,4,10-15 [...] > > Which is much more human friendly than -M. > > Signed-off-by: Luiz

[PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-07 Thread Luiz Capitulino
With --cpu-list you can do: # trace-cmd record --cpu-list 1,4,10-15 [...] Which is much more human friendly than -M. Signed-off-by: Luiz Capitulino --- Documentation/trace-cmd-record.1.txt | 4 + trace-record.c | 138

[PATCH 3/3] trace-cmd record: add --cpu-list option

2016-10-07 Thread Luiz Capitulino
With --cpu-list you can do: # trace-cmd record --cpu-list 1,4,10-15 [...] Which is much more human friendly than -M. Signed-off-by: Luiz Capitulino --- Documentation/trace-cmd-record.1.txt | 4 + trace-record.c | 138 +++ 2 files