This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] add get_nprocs, get_nprocs_conf


On Nov 12 04:10, Yaakov S wrote:
> On 12/11/2009 03:44, Corinna Vinschen wrote:
> >In this case I'm rather surprised that these very GNU/Linux specific
> >things are *not* in a linux/sysinfo.h file.  But it doesn't hurt to keep
> >that in line with Linux, right?
> 
> In that case, here is a patch which declares directly in sys/sysinfo.h.
> 
> 
> Yaakov

> 2009-11-12  Yaakov Selkowitz  <yselkowitz@users.sourceforge.net>
> 
> 	* sysconf.cc (get_nprocs, get_nprocs_conf): New functions.
> 	* cygwin.din: Export them.
> 	* include/sys/sysinfo.h: New header.
> 	(get_nprocs, get_nprocs_conf): Declare.
> 	* include/cygwin/version.h (CYGWIN_VERSION_API_MINOR): Bump.
> 	* posix.sgml: Mention them as GNU extensions.

Thanks, I applied the change with a few changes.

First of all, I took the opportunity to add get_phys_pages and
get_avphys_pages as well so we only have to bump the API minor
number once.

Second, I added the C++ guards to sys/sysinfo.h, as Vaclav pointeed out.

Third, a tiny formatting change:

> +extern "C" int
> +get_nprocs_conf (void)
> +{
> +  return get_nproc_values(_SC_NPROCESSORS_CONF);
                           ^^^
                           Please add a space in front of an opening
                           parenthesis.

Finally, I added the new API to the "What's new" section in the User's
Guide as well.


Thanks again,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]