Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 21 Sep 2007 02:55:50 -0700 (PDT)
From:      Jeff Roberson <jroberson@chesapeake.net>
To:        Kostik Belousov <kostikbel@gmail.com>
Cc:        cvs-src@FreeBSD.org, Jeff Roberson <jeff@FreeBSD.org>, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/kern kern_fork.c kern_proc.c sched_4bsd.c sched_ule.c src/sys/sys proc.h
Message-ID:  <20070921025509.M547@10.0.0.1>
In-Reply-To: <20070921043733.GJ79542@deviant.kiev.zoral.com.ua>
References:  <200709210410.l8L4ANB5044030@repoman.freebsd.org> <20070921043733.GJ79542@deviant.kiev.zoral.com.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 21 Sep 2007, Kostik Belousov wrote:

> On Fri, Sep 21, 2007 at 04:10:23AM +0000, Jeff Roberson wrote:
>> jeff        2007-09-21 04:10:23 UTC
>>
>>   FreeBSD src repository
>>
>>   Modified files:
>>     sys/kern             kern_fork.c kern_proc.c sched_ule.c
>>                          sched_4bsd.c
>>     sys/sys              proc.h
>>   Log:
>>    - Redefine p_swtime and td_slptime as p_swtick and td_slptick.  This
>>      changes the units from seconds to the value of 'ticks' when swapped
>>      in/out.  ULE does not have a periodic timer that scans all threads in
>>      the system and as such maintaining a per-second counter is difficult.
>>    - Change computations requiring the unit in seconds to subtract ticks
>>      and divide by hz.  This does make the wraparound condition hz times
>>      more frequent but this is still in the range of several months to
>>      years and the adverse effects are minimal.
>>
>>   Approved by:    re
>>
>>   Revision  Changes    Path
>>   1.282     +1 -0      src/sys/kern/kern_fork.c
>>   1.252     +3 -2      src/sys/kern/kern_proc.c
>>   1.105     +20 -17    src/sys/kern/sched_4bsd.c
>>   1.207     +5 -7      src/sys/kern/sched_ule.c
>>   1.491     +2 -2      src/sys/sys/proc.h
> It seems that at least the sys/vm/ was missed from the commit.

yes I caught it when I rebuilt my tree.

Thanks,
Jeff

>
> cc -c -O -pipe -march=pentiumpro -std=c99 -g -Wall -Wredundant-decls -Wnested-ex
> terns -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign -fformat-extensions -nostdinc  -I. -I/usr/bsd/src/sys -I/usr/bsd/src/sys/contrib/altq -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common -finline-limit=8000 --param inline-unit-growth=100 --param large-function-growth=1000  -mno-align-long-strings -mpreferred-stack-boundary=2  -mno-mmx -mno-3dnow -mno-sse -mno-sse2 -mno-sse3 -ffreestanding -Werror  /usr/bsd/src/sys/vm/vm_glue.c
> /usr/bsd/src/sys/vm/vm_glue.c: In function 'faultin':
> /usr/bsd/src/sys/vm/vm_glue.c:639: error: 'struct proc' has no member named 'p_swtime'
> /usr/bsd/src/sys/vm/vm_glue.c: In function 'scheduler':
> /usr/bsd/src/sys/vm/vm_glue.c:700: error: 'struct proc' has no member named 'p_swtime'
> /usr/bsd/src/sys/vm/vm_glue.c:700: error: 'struct thread' has no member named 'td_slptime'
> ....
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070921025509.M547>