Thanks for pointing out. A clarification - that kind of define check was in two places, see next. The patch you refer to is updating on the second condition. The first one should stay? ``` #if defined (__OS_darwin) || defined (__OS_freebsd) #include <sys/param.h> #include <sys/sysctl.h> #else #include <unistd.h> #endif int get_num_cpus() { int count = 0; #if defined (__OS_darwin) || defined (__OS_freebsd) int nm[2]; size_t len; ``` --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/24#issuecomment-69491839

The code was more or less found on various places on the net, here is a link I found searching now: * http://stackoverflow.com/questions/150355/programmatically-find-the-number-… It is a bit old, indeed, but shows that bsd was using the same way as mac os x. I didn't have a *BSD at hand to try, though. Maybe the GNU extension with _SC_NPROCESSORS_ONLN got into *BSD, but wonder if the includes are not different, as for Linux. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/24#issuecomment-69492771

Kamailio is written in C and don't want to mix it here. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/24#issuecomment-69493188

I tried to figure out if we need to adjust the condition for included files or let it as it is: ``` #if defined (__OS_darwin) || defined (__OS_freebsd) ``` A 'man sysconf' on darwin shows that it needs: ``` #include <unistd.h> ``` No *BSD around, but I guess it will be the same, therefore I think the guard for the includes needs to be updated as well. Would you have a possibility to test that it compiles? --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/24#issuecomment-69498862

Closed #24. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/24#event-217984186