[Bug 1656] GSoC AIO patch

[bugzilla-daemon at rtems.org]

https://www.rtems.org/bugzilla/show_bug.cgi?id=1656

--- Comment #19 from Joel Sherrill <joel.sherrill at oarcorp.com> 2010-08-16 09:39:50 CDT ---
(In reply to comment #18)
> Now, once again with some modifications applied, 
> testsuites/psxtests/psxaio01 and testsuites/psxtests/psxaio02 
> are imported into RTEMS-CVS as well.

aio_init() appears to use a posix thread, mutex, and condition variable.
These should be accounted for in confdefs.h and in doc/user/conf.t.
A configuration variable like CONFIGURE_ENABLE_POSIX_AIO and then propagate
the requirements into the various objects. The GNAT/Ada configuration should
be a good guide.

> As before, no actual testing yet.

I just did a coverage run on erc32.  Both psxaio01 and psxaio02 failed.

Obviously both should pass as a minimum.  But the aio code needs to have
100% code coverage so getting these to run is a minimum.  With the addition
of AIO code and these tests failing, we went from 72 bytes (18 instructions)
uncovered to this much just in the aio code (bytes).

1772 aio_misc.c
28 aio_suspend.c
28 aio_fsync.c
12 aio_write.c
12 aio_read.c

So submit a patch against the cvs head to get these tests running and then
we can see what is not exercised by your tests.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.