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@rtems.org
http://www.rtems.org/mailman/listinfo/rtems-devel



--
--------------------------------------------
embedded brains GmbH
Thomas Doerfler
Dornierstr. 4
D-82178 Puchheim
Germany
email: thomas.doerf...@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.
_______________________________________________
rtems-devel mailing list
rtems-devel@rtems.org
http://www.rtems.org/mailman/listinfo/rtems-devel

Reply via email to