[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src/sys
On Sun, Jan 11, 2009 at 02:45:56AM +0000, Christos Zoulas wrote:
> Log Message:
> merge christos-time_t
A few things I have noticed so far:
- adjtime1() leaks timecounter_lock.
- #ifdef LFS doesn't belong under kern/ or in the compat module. The shim
should live in the LFS source. I realize that LFS syscalls are not
currently installed - this is a bug.
- The compat module now needs to be marked as depending on the aio module,
unless the compat shim lives in sys_aio.c. I'd tend toward putting it in
the compat mod but it could also live with aio. I'm suggesting the
opposite of the LFS case here since it's a basic kernel facility without
a big mess of depends and 100's kb of code behind it, unlike LFS.
- compat_mod.c needs an ifdef AIO for the case where we build a monolithic
kernel without AIO (and a -DAIO in the module makefile so it gets picked
up when building that way).
There is one example of ugly depends handling for the MODULE macro in
linux_mod.c (it's there to cater for monolithic builds and differerences
between architectures, because e.g. a.out doesn't make sense on NetBSD/foo).
Main Index |
Thread Index |