Date: Thu, 05 Jul 2001 00:20:14 +0100 From: Brian Somers <brian@Awfulhak.org> To: Matt Dillon <dillon@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, John Baldwin <jhb@FreeBSD.org>, Alfred Perlstein <alfred@FreeBSD.org> Subject: Re: cvs commit: src/sys/alpha/alpha machdep.c mem.c trap.c vm_machdep.c src/sys/alpha/osf1 imgact_osf1.c osf1_misc.c src/sys/dev/md md.c src/sys/fs/procfs procfs_map.c procfs_mem.c src/sys/fs/specfs spec_vnops.c src/sys/i386/i386 busdma_machdep.c ... Message-ID: <200107042320.f64NKEN05839@hak.lan.Awfulhak.org> In-Reply-To: Message from Matt Dillon <dillon@FreeBSD.org> of "Wed, 04 Jul 2001 09:20:30 PDT." <200107041620.f64GKUW78489@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Unfortunately, even after these changes, I'm still seeing sig 4/10 on
my 1.7Ghz P4 doing a buildworld.
It *appears* that the problem started with version 1.156 of
swap_pager.c, but I can't be sure. I can't complete a buildworld with
version 1.155 because of lockrecursion and lock assertion problems.
buildworld works fine on pre vm_mtx kernels.
> dillon 2001/07/04 09:20:30 PDT
>
> Modified files:
> sys/alpha/alpha machdep.c mem.c trap.c vm_machdep.c
> sys/alpha/osf1 imgact_osf1.c osf1_misc.c
> sys/dev/md md.c
> sys/fs/procfs procfs_map.c procfs_mem.c
> sys/fs/specfs spec_vnops.c
> sys/i386/i386 busdma_machdep.c machdep.c mem.c pmap.c
> vm_machdep.c
> sys/i386/isa isa_dma.c
> sys/ia64/ia64 vm_machdep.c
> sys/kern imgact_aout.c imgact_elf.c init_main.c
> kern_exec.c kern_exit.c kern_fork.c
> kern_resource.c link_elf.c subr_blist.c
> sys_pipe.c sysv_shm.c uipc_syscalls.c
> vfs_bio.c vfs_cluster.c vfs_default.c
> vfs_subr.c vfs_syscalls.c
> sys/nfs nfs_bio.c nfs_subs.c
> sys/pc98/i386 machdep.c
> sys/pc98/pc98 isa_dma.c
> sys/powerpc/powerpc vm_machdep.c
> sys/sys mutex.h proc.h
> sys/ufs/ufs ufs_readwrite.c
> sys/vm default_pager.c device_pager.c
> phys_pager.c swap_pager.c vm.h vm_fault.c
> vm_glue.c vm_init.c vm_kern.c vm_map.c
> vm_map.h vm_meter.c vm_mmap.c vm_object.c
> vm_object.h vm_page.c vm_page.h
> vm_pageout.c vm_pager.c vm_pager.h
> vm_unix.c vm_zone.c vnode_pager.c
> Log:
> With Alfred's permission, remove vm_mtx in favor of a fine-grained approach
> (this commit is just the first stage). Also add various GIANT_ macros to
> formalize the removal of Giant, making it easy to test in a more piecemeal
> fashion. These macros will allow us to test fine-grained locks to a degree
> before removing Giant, and also after, and to remove Giant in a piecemeal
> fashion via sysctl's on those subsystems which the authors believe can
> operate without Giant.
>
> Revision Changes Path
> 1.134 +1 -3 src/sys/alpha/alpha/machdep.c
> 1.36 +3 -5 src/sys/alpha/alpha/mem.c
> 1.69 +3 -8 src/sys/alpha/alpha/trap.c
> 1.53 +10 -13 src/sys/alpha/alpha/vm_machdep.c
> 1.7 +3 -6 src/sys/alpha/osf1/imgact_osf1.c
> 1.17 +5 -5 src/sys/alpha/osf1/osf1_misc.c
> 1.37 +5 -6 src/sys/dev/md/md.c
> 1.29 +3 -7 src/sys/fs/procfs/procfs_map.c
> 1.51 +4 -9 src/sys/fs/procfs/procfs_mem.c
> 1.160 +3 -4 src/sys/fs/specfs/spec_vnops.c
> 1.23 +2 -1 src/sys/i386/i386/busdma_machdep.c
> 1.461 +3 -5 src/sys/i386/i386/machdep.c
> 1.92 +3 -9 src/sys/i386/i386/mem.c
> 1.283 +2 -8 src/sys/i386/i386/pmap.c
> 1.164 +12 -16 src/sys/i386/i386/vm_machdep.c
> 1.9 +6 -8 src/sys/i386/isa/isa_dma.c
> 1.21 +10 -13 src/sys/ia64/ia64/vm_machdep.c
> 1.69 +3 -9 src/sys/kern/imgact_aout.c
> 1.96 +5 -14 src/sys/kern/imgact_elf.c
> 1.174 +5 -5 src/sys/kern/init_main.c
> 1.133 +5 -7 src/sys/kern/kern_exec.c
> 1.130 +3 -3 src/sys/kern/kern_exit.c
> 1.116 +3 -4 src/sys/kern/kern_fork.c
> 1.84 +3 -3 src/sys/kern/kern_resource.c
> 1.39 +3 -9 src/sys/kern/link_elf.c
> 1.9 +2 -1 src/sys/kern/subr_blist.c
> 1.83 +8 -12 src/sys/kern/sys_pipe.c
> 1.62 +12 -14 src/sys/kern/sysv_shm.c
> 1.95 +5 -14 src/sys/kern/uipc_syscalls.c
> 1.283 +36 -78 src/sys/kern/vfs_bio.c
> 1.111 +7 -11 src/sys/kern/vfs_cluster.c
> 1.54 +6 -13 src/sys/kern/vfs_default.c
> 1.315 +7 -11 src/sys/kern/vfs_subr.c
> 1.195 +3 -3 src/sys/kern/vfs_syscalls.c
> 1.96 +9 -15 src/sys/nfs/nfs_bio.c
> 1.103 +3 -3 src/sys/nfs/nfs_subs.c
> 1.225 +3 -5 src/sys/pc98/i386/machdep.c
> 1.11 +5 -8 src/sys/pc98/pc98/isa_dma.c
> 1.54 +10 -13 src/sys/powerpc/powerpc/vm_machdep.c
> 1.32 +27 -1 src/sys/sys/mutex.h
> 1.170 +2 -1 src/sys/sys/proc.h
> 1.81 +6 -31 src/sys/ufs/ufs/ufs_readwrite.c
> 1.28 +2 -1 src/sys/vm/default_pager.c
> 1.52 +2 -1 src/sys/vm/device_pager.c
> 1.10 +5 -2 src/sys/vm/phys_pager.c
> 1.160 +32 -70 src/sys/vm/swap_pager.c
> 1.18 +1 -5 src/sys/vm/vm.h
> 1.122 +12 -39 src/sys/vm/vm_fault.c
> 1.116 +11 -36 src/sys/vm/vm_glue.c
> 1.28 +1 -7 src/sys/vm/vm_init.c
> 1.72 +13 -59 src/sys/vm/vm_kern.c
> 1.205 +53 -60 src/sys/vm/vm_map.c
> 1.65 +6 -12 src/sys/vm/vm_map.h
> 1.56 +2 -7 src/sys/vm/vm_meter.c
> 1.123 +20 -68 src/sys/vm/vm_mmap.c
> 1.194 +23 -50 src/sys/vm/vm_object.c
> 1.72 +21 -29 src/sys/vm/vm_object.h
> 1.165 +29 -50 src/sys/vm/vm_page.c
> 1.87 +15 -23 src/sys/vm/vm_page.h
> 1.180 +12 -36 src/sys/vm/vm_pageout.c
> 1.72 +5 -17 src/sys/vm/vm_pager.c
> 1.33 +6 -11 src/sys/vm/vm_pager.h
> 1.33 +23 -25 src/sys/vm/vm_unix.c
> 1.45 +4 -21 src/sys/vm/vm_zone.c
> 1.133 +18 -51 src/sys/vm/vnode_pager.c
--
Brian <brian@freebsd-services.com> <brian@Awfulhak.org>
http://www.freebsd-services.com/ <brian@[uk.]FreeBSD.org>
Don't _EVER_ lose your sense of humour ! <brian@[uk.]OpenBSD.org>
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200107042320.f64NKEN05839>