lkml.org 
[lkml]   [2018]   [Oct]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v11 2/3]: perf record: enable asynchronous trace writing
From
Date
Hi,

On 08.10.2018 13:50, Jiri Olsa wrote:
> On Mon, Oct 08, 2018 at 09:17:11AM +0300, Alexey Budankov wrote:
>
> SNIP
>
>> -
>> +#ifdef HAVE_AIO_SUPPORT
>> + lseek(trace_fd, off, SEEK_SET);
>> +#endif
>> /*
>> * Mark the round finished in case we wrote
>> * at least one event.
>> @@ -650,6 +799,9 @@ record__switch_output(struct record *rec, bool at_exit)
>> /* Same Size: "2015122520103046"*/
>> char timestamp[] = "InvalidTimestamp";
>>
>> +#ifdef HAVE_AIO_SUPPORT
>> + record__mmap_read_sync(rec);
>> +#endif
>
> same, please define the record__mmap_read_sync dummy
> so we can skip that #ifdef #endif

Accepted.

>
>> record__synthesize(rec, true);
>> if (target__none(&rec->opts.target))
>> record__synthesize_workload(rec, true);
>> @@ -1157,6 +1309,10 @@ static int __cmd_record(struct record *rec, int argc, const char **argv)
>> record__synthesize_workload(rec, true);
>>
>> out_child:
>> +
>> +#ifdef HAVE_AIO_SUPPORT
>> + record__mmap_read_sync(rec);
>> +#endif
>
> ditto

Accepted.

Thanks,
Alexey

>
> thanks,
> jirka
>
> SNIP
>

\
 
 \ /
  Last update: 2018-10-08 14:18    [W:0.085 / U:5.664 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site