On 12/18/2014 02:49 PM, 尹杰 wrote: > Hi all, > > I am a operating system researcher in Tsinghua University. I find that > there is a missing "{" in the definition of struct read_format's inner > struct definition in the function perf_event_open > <http://man7.org/linux/man-pages/man2/perf_event_open.2.html>'s document : > > struct read_format { > u64 nr; /* The number of events */ > u64 time_enabled; /* if PERF_FORMAT_TOTAL_TIME_ENABLED */ > u64 time_running; /* if PERF_FORMAT_TOTAL_TIME_RUNNING */ > struct > u64 value; /* The value of the event */ > u64 id; /* if PERF_FORMAT_ID */ > } values[nr]; > }; > > > should be > > struct read_format { > u64 nr; /* The number of events */ > u64 time_enabled; /* if PERF_FORMAT_TOTAL_TIME_ENABLED */ > u64 time_running; /* if PERF_FORMAT_TOTAL_TIME_RUNNING */ > struct { > u64 value; /* The value of the event */ > u64 id; /* if PERF_FORMAT_ID */ > } values[nr]; > }; Hello Jie Yin, Thanks for the report. I fixed the page as you suggest. Cheers, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/ -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html