On 10/15/18 11:17 AM, Alex Bennée wrote: > > Richard Henderson <richard.hender...@linaro.org> writes: > >> On 10/5/18 8:48 AM, Alex Bennée wrote: >>> To avoid overly long lines in trace events I've split the event into >>> tlb_set_page and tlb_set_page_attrs. >> >> I don't think this split is a good idea. It allows data from different >> threads >> to be interleaved. > > Won't we soon have a lock serialising this?
For each individual trace entry, surely. If you split the trace in two, you leave a race in the middle. > It was mainly to shut-up checkpatch because the trace-events don't seem > to do multi-line. I would just ignore that. r~