Re: [PATCH V6 11/20] rtla/osnoise: Add the hist mode
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH V6 11/20] rtla/osnoise: Add the hist mode
- From: Daniel Bristot de Oliveira <bristot@xxxxxxxxxx>
- Date: Fri, 29 Oct 2021 18:04:15 +0200
- Cc: Ingo Molnar <mingo@xxxxxxxxxx>, Tom Zanussi <zanussi@xxxxxxxxxx>, Masami Hiramatsu <mhiramat@xxxxxxxxxx>, Juri Lelli <juri.lelli@xxxxxxxxxx>, Clark Williams <williams@xxxxxxxxxx>, John Kacur <jkacur@xxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>, linux-rt-users@xxxxxxxxxxxxxxx, linux-trace-devel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
- In-reply-to: <20211027142918.487cd143@gandalf.local.home>
- References: <cover.1635284863.git.bristot@kernel.org> <fede3c3805cb7a70bd7c8d60ba2c5ce290cc6e1e.1635284863.git.bristot@kernel.org> <YXmWTOXHYXFduayt@geo.homenetwork> <20211027142918.487cd143@gandalf.local.home>
- User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0
On 10/27/21 20:29, Steven Rostedt wrote:
> On Thu, 28 Oct 2021 02:11:24 +0800
> Tao Zhou <tao.zhou@xxxxxxxxx> wrote:
>
>>> + /* one histogram for IRQ and one for thread, per cpu */
>>> + for (cpu = 0; cpu < nr_cpus; cpu++) {
>>> + data->hist[cpu].samples = calloc(1, sizeof(*data->hist) * (entries + 1));
>> @samples is a pointer to int and used for int array. The "sizeof(*data->hist)"
>> should be "sizeof(int)" or am I totally wrong.
> Or standard approach is:
>
> data->hist[cpu].samples = calloc(1, sizeof(*data->hist->samples) * (entries + 1));
Fixed in the next version, thanks!
-- Daniel
[Index of Archives]
[Linux USB Development]
[Linux USB Development]
[Linux Audio Users]
[Yosemite Hiking]
[Linux Kernel]
[Linux SCSI]