Date: Tue, 8 Feb 2011 00:16:36 +0000 (UTC) From: Matthew D Fleming <mdf@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/dev/sio sio.c src/sys/kern kern_synch.c kern_sysctl.c subr_uio.c vfs_bio.c vfs_mount.c vfs_subr.c vfs_vnops.c src/sys/pc98/cbus sio.c src/sys/sys proc.h uio.h src/sys/ufs/ffs ffs_softdep.c Message-ID: <201102080017.p180H9bc081334@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
mdf 2011-02-08 00:16:36 UTC
FreeBSD src repository
Modified files:
sys/dev/sio sio.c
sys/kern kern_synch.c kern_sysctl.c subr_uio.c
vfs_bio.c vfs_mount.c vfs_subr.c
vfs_vnops.c
sys/pc98/cbus sio.c
sys/sys proc.h uio.h
sys/ufs/ffs ffs_softdep.c
Log:
SVN rev 218424 on 2011-02-08 00:16:36Z by mdf
Based on discussions on the svn-src mailing list, rework r218195:
- entirely eliminate some calls to uio_yeild() as being unnecessary,
such as in a sysctl handler.
- move should_yield() and maybe_yield() to kern_synch.c and move the
prototypes from sys/uio.h to sys/proc.h
- add a slightly more generic kern_yield() that can replace the
functionality of uio_yield().
- replace source uses of uio_yield() with the functional equivalent,
or in some cases do not change the thread priority when switching.
- fix a logic inversion bug in vlrureclaim(), pointed out by bde@.
- instead of using the per-cpu last switched ticks, use a per thread
variable for should_yield(). With PREEMPTION, the only reasonable
use of this is to determine if a lock has been held a long time and
relinquish it. Without PREEMPTION, this is essentially the same as
the per-cpu variable.
Revision Changes Path
1.476 +0 -1 src/sys/dev/sio/sio.c
1.322 +33 -2 src/sys/kern/kern_synch.c
1.214 +1 -1 src/sys/kern/kern_sysctl.c
1.5 +1 -23 src/sys/kern/subr_uio.c
1.600 +1 -1 src/sys/kern/vfs_bio.c
1.328 +1 -1 src/sys/kern/vfs_mount.c
1.805 +4 -4 src/sys/kern/vfs_subr.c
1.300 +2 -2 src/sys/kern/vfs_vnops.c
1.251 +0 -1 src/sys/pc98/cbus/sio.c
1.563 +4 -0 src/sys/sys/proc.h
1.44 +0 -2 src/sys/sys/uio.h
1.265 +1 -1 src/sys/ufs/ffs/ffs_softdep.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201102080017.p180H9bc081334>