Re: CVS commit: src/sys

2010-03-18 Thread David Young
On Thu, Mar 18, 2010 at 08:27:58PM +0200, Antti Kantee wrote:
> On Mon Mar 15 2010 at 16:25:01 +0200, Antti Kantee wrote:
> > On Sun Mar 14 2010 at 14:30:07 -0500, David Young wrote:
> > > On my console and in /var/log/messages, I find
> > > 
> > >   WARNING: module error: use -f to reinstate builtin module "compat"
> > > 
> > > For example, whenever I type my login:
> > > 
> > >   login: dyoung
> > >   WARNING: module error: use -f to reinstate builtin module "compat"
> > >   Password:
> > 
> > hmm, something is unloading compat.  I wonder if builtins can be
> > autounloaded now (shouldn't be allowed to happen).  I didn't notice this
> > behaviour on a fresh anita installation, so i'm not sure what it's about.
> 
> This should be fixed now.  Autounload (disable) of builtins was attempted
> only if resources were short -- i guess you have a low-memory setup.

Heh, low-memory.  I guess it's all relative. :-) I saw the warning
on 128MB, on 512MB, and on 1024MB machines.  I first noticed it on
the 512MB machine very shortly after boot (i.e., before I'd knowingly
created memory pressure with build.sh or something).

Thanks so much for fixing it!  I'll give the new kernel a spin, soon.

Dave

-- 
David Young OJC Technologies
dyo...@ojctech.com  Urbana, IL * (217) 278-3933


Re: CVS commit: src/sys

2010-03-18 Thread Antti Kantee
On Mon Mar 15 2010 at 16:25:01 +0200, Antti Kantee wrote:
> On Sun Mar 14 2010 at 14:30:07 -0500, David Young wrote:
> > On my console and in /var/log/messages, I find
> > 
> >   WARNING: module error: use -f to reinstate builtin module "compat"
> > 
> > For example, whenever I type my login:
> > 
> >   login: dyoung
> >   WARNING: module error: use -f to reinstate builtin module "compat"
> >   Password:
> 
> hmm, something is unloading compat.  I wonder if builtins can be
> autounloaded now (shouldn't be allowed to happen).  I didn't notice this
> behaviour on a fresh anita installation, so i'm not sure what it's about.

This should be fixed now.  Autounload (disable) of builtins was attempted
only if resources were short -- i guess you have a low-memory setup.

On a tangent, when doing the original change, I wanted to require
something like modunload -f to disable builtins, but there is no way to
pass flags to modctl() in unload.


Re: CVS commit: src/external/cddl/osnet/dev/dtrace/i386

2010-03-18 Thread Matthias Scheler
On Thu, Mar 18, 2010 at 11:44:40PM +0900, Izumi Tsutsui wrote:
> > Modified Files:
> > src/external/cddl/osnet/dev/dtrace/i386: dtrace_subr.c
> > 
> > Log Message:
> > Fix the build:
> > 1.) "ifdef-out" unused functions and global variables.
> > 2.) Include "x86/include/cpu_counter.h" for amd64 and i386 to get the
> > prototype of "cpu_frequency".
> 
> Probably this should be rewritten to use MI timecounter(9)?

Well, it seems to be an architecture specific file anyway. But I don't
know the details, I just "mechanically" fixed the build.

Kind regards

-- 
Matthias Scheler  http://zhadum.org.uk/


Re: CVS commit: src/external/cddl/osnet/dev/dtrace/i386

2010-03-18 Thread Izumi Tsutsui
> Module Name:  src
> Committed By: tron
> Date: Thu Mar 18 11:00:03 UTC 2010
> 
> Modified Files:
>   src/external/cddl/osnet/dev/dtrace/i386: dtrace_subr.c
> 
> Log Message:
> Fix the build:
> 1.) "ifdef-out" unused functions and global variables.
> 2.) Include "x86/include/cpu_counter.h" for amd64 and i386 to get the
> prototype of "cpu_frequency".

Probably this should be rewritten to use MI timecounter(9)?
---
Izumi Tsutsui