On 04/18/2011 12:45 AM, Ray Hinchliffe (RH) wrote:
> The patch works by reading from /proc/<pid>/task/<tid>/stat. If this in not
> possible it will report the fix messages as before.
> Given that SystemProcessorPerformanceInformation uses /proc/stat I expect this
Thanks for the patch, however there are several issues with it:
- Don't attach changes to automatically generated files (server/trace.c,
server/request.h, etc). Only server/protocol.def should be included, which
you didn't even modify.
- Keep formatting of the original file you modifying. No whitespace only
changes, no funky new formatting styles.
- Please use Windows types instead of system types for integers (ULONG
instead of unsigned long)
> + sprintf( buf, "/proc/%u/task/%u/stat", reply->unix_pid, reply->unix_tid );
Please don't call any functions in the server call context. Also you not
using this buffer if server call fails, or if the thread is the same.
Vitaliy.