On 9/12/12 8:40 AM, Andi Kleen wrote:
- while(!done) sleep(1); + char piddir[40]; + if (target.pid && access("/proc", X_OK) == 0) + snprintf(piddir, sizeof piddir, "/proc/%d", atoi(target.pid));
else path has piddir not set. Also as Namyhung pointed out target.pid can have multiple pids in it, so this fails if a user specified -p pid1,pid2 -- it will only check if the first process died.
Perhaps a more generic perf_target__is_alive function is needed that can check if the given tid(s) or pid(s) have terminated.
David -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/