Re: CVS commit: src

[Andrew Doran <ad%NetBSD.org@localhost>]

On Tue, Apr 29, 2008 at 07:50:27PM +0900, Izumi Tsutsui wrote:

> ad%NetBSD.org@localhost wrote:
> 
> > Module Name:        src
> > Committed By:       ad
> > Date:               Mon Apr 28 15:36:01 UTC 2008
> > 
> > Modified Files:
> >     src/sys/dev: lockstat.c lockstat.h
> >     src/sys/kern: kern_exit.c kern_lock.c kern_lwp.c kern_ras.c
> >         kern_softint.c kern_synch.c kern_turnstile.c subr_percpu.c
> >         subr_pool.c
> >     src/sys/sys: cpu.h lwp.h sched.h userret.h
> >     src/usr.sbin/lockstat: main.c
> > 
> > Log Message:
> > Add MI code to support in-kernel preemption.
>  :
> > cvs rdiff -r1.89 -r1.90 src/sys/sys/lwp.h
> 
> Now <sys/lwp.h> refers curlwp which is declared in <sys/proc.h>,
> but <sys/proc.h> already includes <sys/lwp.h> in it for struct lwp.
> 
> Then we have to include <sys/proc.h> in <sys/lwp.h> and
> also include <sys/lwp.h> before <sys/proc.h> in each source
> if curlwp is not defined in <machine/cpu.h>.
> 
> Is there any good way to resolve this dependency?

I moved it into lwp.h which I think should fix the problem, since
proc.h needs lwp.h.

Thanks,
Andrew