On Mon, 2012-07-23 at 10:32 +0400, Igor Zhbanov wrote: > Hi Chris! > Thank you for your answer. > > Chris Ferron wrote: > > On 07/20/2012 12:06 AM, Igor Zhbanov wrote: > >> > >> Linaro has made only few commits after forking from mainline. > >> > >> They are: > >> > >> 1) adding __NR_perf_event_open define > > Please insure this is fresh code authored by you and detailed if you > > submit, and I will be happy to take it. > Yes, this is just a system call number copied from unistd.h for ARM > platform. > Without it PowerTOP will not know what system call number > __NR_perf_event_open has on ARM. Why can't it take __NR_perf_event_open from the libc headers? That is, why can't the whole ifdef under /* some people have stale headers */ go? Powertop is rather demanding with regards to the environment anyhow and magic constants are never fun. /MF