Subject: Re: CVS commit: src/sys/sys
To: None <yamt@mwd.biglobe.ne.jp, christos@astron.com>
From: Mindaugas R. <rmind@NetBSD.org>
List: source-changes
Date: 01/24/2008 20:03:24
yamt@mwd.biglobe.ne.jp (YAMAMOTO Takashi) wrote:
> > Module Name:	src
> > Committed By:	rmind
> > Date:		Mon Jan 21 00:27:24 UTC 2008
> > 
> > Modified Files:
> > 	src/sys/sys: types.h
> > 
> > Log Message:
> > Move the cpuid_t type-defintion out of the _NETBSD_SOURCE condition.
> > Should fix the compilation of various applications.  Discussed with
> > <martin>.
> > 
> > 
> > To generate a diff of this commit:
> > cvs rdiff -r1.78 -r1.79 src/sys/sys/types.h
> > 
> > Please note that diffs are not public domain; they are subject to the
> > copyright notices on the relevant files.
> 
> doesn't it contradict the purpose of _NETBSD_SOURCE?

christos@astron.com (Christos Zoulas) wrote:
> Yes, and this makes visible cpuid_t which is not an any standard that
> I know off. Anyway, I had committed the fix in pthread.h, not to include
> it unless _NETBSD_SOURCE was defined.

If I understand this paper correctly, this is not true (see end of the table):
http://www.opengroup.org/onlinepubs/009695399/functions/xsh_chap02_02.html#tag_02_02_02

The problematic case is that sys/pset.h is included in pthread.h, because
defintion of cpuset_t is needed. How about moving this to sys/sched.h?

-- 
Best regards,
Mindaugas
www.NetBSD.org