Date: Mon, 9 Nov 2009 20:29:10 +0000 (UTC) From: Roman Divacky <rdivacky@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/amd64/isa isa_dma.c src/sys/dev/fdc fdc.c src/sys/dev/ieee488 ibfoo.c src/sys/i386/isa isa_dma.c Message-ID: <200911092030.nA9KU1ov037089@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
rdivacky 2009-11-09 20:29:10 UTC
FreeBSD src repository
Modified files:
sys/amd64/isa isa_dma.c
sys/dev/fdc fdc.c
sys/dev/ieee488 ibfoo.c
sys/i386/isa isa_dma.c
Log:
SVN rev 199104 on 2009-11-09 20:29:10Z by rdivacky
Make isa_dma functions MPSAFE by introducing its own private lock. These
functions are selfcontained (ie. they touch only isa_dma.c static variables
and hardware) so a private lock is sufficient to prevent races. This changes
only i386/amd64 while there are also isa_dma functions for ia64/sparc64.
Sparc64 are ones empty stubs and ia64 ones are unused as ia64 does not
have isa (says marcel).
This patch removes explicit locking of Giant from a few drivers (there
are some that requires this but lack ones - this patch fixes this) and
also removes the need for implicit locking of Giant from attach routines
where it's provided by newbus.
Approved by: ed (mentor, implicit)
Reviewed by: jhb, attilio (glanced by)
Tested by: Giovanni Trematerra <giovanni.trematerra gmail com>
IA64 clue: marcel
Revision Changes Path
1.23 +83 -27 src/sys/amd64/isa/isa_dma.c
1.326 +0 -6 src/sys/dev/fdc/fdc.c
1.7 +0 -8 src/sys/dev/ieee488/ibfoo.c
1.20 +83 -27 src/sys/i386/isa/isa_dma.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200911092030.nA9KU1ov037089>