Original cpuset.h for review
Thomas Doerfler
Thomas.Doerfler at embedded-brains.de
Tue Oct 15 06:00:36 UTC 2013
Hi Joel,
just some comments from me:
1.) In the typedef for cpu_set_t, it might be better to use the
_NCPUWORDS macro instead of duplicating its expansion
2.) In the operations _OR, _AND etc, you used "(_set)" instead of
"(_src2)" for the second operand
3.) The term "(_cpu)/_NCPUBITS" is duplicated to get the index into the
cpuset table. This might be done through a macro (__cpu_set_idx ?) aswell.
wkr,
Thomas.
Am 14.10.2013 20:47, schrieb Joel Sherrill:
> Hi
>
> This has had limited review but should close if you ignore
> that I need to add some comments.
>
> This will go into the RTEMS specific subdirectory of
> newlib and be accompanied by prototypes of various
> pthread methods that use this. This is just the
> first step.
>
> Comments.
>
>
>
> _______________________________________________
> rtems-devel mailing list
> rtems-devel at rtems.org
> http://www.rtems.org/mailman/listinfo/rtems-devel
>
--
--------------------------------------------
embedded brains GmbH
Thomas Doerfler
Dornierstr. 4
D-82178 Puchheim
Germany
email: Thomas.Doerfler at embedded-brains.de
Phone: +49-89-18 94 741-12
Fax: +49-89-18 94 741-09
PGP: Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
More information about the devel
mailing list