On 7/31/15 4:59 AM, Kaixu Xia wrote:
According to the comments from Daniel, rewrite part of the bpf_prog_array map code and make it more generic. So the new perf_event_array map type can reuse most of code with bpf_prog_array map and add fewer lines of special code.Tested the samples/bpf/tracex5 after this patch: $ sudo ./tracex5 ... dd-1051 [000] d... 26.682903: : mmap dd-1051 [000] d... 26.698348: : syscall=102 (one of get/set uid/pid/gid) dd-1051 [000] d... 26.703892: : read(fd=0, buf=000000000078c010, size=512) dd-1051 [000] d... 26.705847: : write(fd=1, buf=000000000078c010, size=512) dd-1051 [000] d... 26.707914: : read(fd=0, buf=000000000078c010, size=512) dd-1051 [000] d... 26.710988: : write(fd=1, buf=000000000078c010, size=512) dd-1051 [000] d... 26.711865: : read(fd=0, buf=000000000078c010, size=512) dd-1051 [000] d... 26.712704: : write(fd=1, buf=000000000078c010, size=512) ...
the commit log doesn't really need these test results. Also first paragraph is meaningless without prior history. Please reword it, so it clearly says what this patch is doing and why. Anyone reading it years later should be able to understand. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

