Revision tags: release/5.2.1_cvs, release/5.2.1, release/5.2.0_cvs, release/5.2.0, release/4.9.0_cvs, release/4.9.0 |
|
#
7144b1d4 |
| 22-Jun-2003 |
Don Lewis <truckman@FreeBSD.org> |
Don't check the state of the vnode interlock if the specification says that the lock should not be checked.
Skip the lock assertion checks for *vpp or any other pointer to a vnode pointer if vpp (or
Don't check the state of the vnode interlock if the specification says that the lock should not be checked.
Skip the lock assertion checks for *vpp or any other pointer to a vnode pointer if vpp (or equivalent) is NULL.
show more ...
|
#
86ed8962 |
| 20-Jun-2003 |
Stefan Eßer <se@FreeBSD.org> |
The assertions generated to test VFS locking never included checks for vnodes reached through double indirection (i.e. **vpp). This is worked-around by special-casing the identifier "vpp" (adding one
The assertions generated to test VFS locking never included checks for vnodes reached through double indirection (i.e. **vpp). This is worked-around by special-casing the identifier "vpp" (adding one level of indirection).
The alternative fix mentioned in the PR had required substantial changes to this script.
In case there are locking violations that had been hidden without this patch, they may suddenly show up, now ...
This change does not affect code compiled without DEBUG_VFS_LOCKS.
PR: kern/46652
show more ...
|
Revision tags: release/5.1.0_cvs, release/5.1.0, release/4.8.0_cvs, release/4.8.0, release/5.0.0_cvs, release/5.0.0, release/4.7.0_cvs |
|
#
6423c943 |
| 26-Sep-2002 |
Jeff Roberson <jeff@FreeBSD.org> |
- Move ASSERT_VOP_*LOCK* functionality into functions in vfs_subr.c - Make the VI asserts more orthogonal to the rest of the asserts by using a new, common vfs_badlock() function and adding a 'st
- Move ASSERT_VOP_*LOCK* functionality into functions in vfs_subr.c - Make the VI asserts more orthogonal to the rest of the asserts by using a new, common vfs_badlock() function and adding a 'str' arg. - Adjust generated ASSERTS to match the new prototype. - Adjust explicit ASSERTS to match the new prototype.
show more ...
|
#
15597470 |
| 23-Sep-2002 |
Jeff Roberson <jeff@FreeBSD.org> |
- Automatically generate and insert KTR points in all VOPs.
|
#
71ea4ba5 |
| 21-Aug-2002 |
Jeff Roberson <jeff@FreeBSD.org> |
- Add two new debugging macros: ASSERT_VI_LOCKED and ASSERT_VI_UNLOCKED - Use the new VI asserts in place of the old mtx_assert checks. - Add the VI asserts to the automated lock checking in the VO
- Add two new debugging macros: ASSERT_VI_LOCKED and ASSERT_VI_UNLOCKED - Use the new VI asserts in place of the old mtx_assert checks. - Add the VI asserts to the automated lock checking in the VOP calls. The interlock should not be held across vops with a few exceptions. - Add the vop_(un)lock_{pre,post} functions to assert that interlock is held when LK_INTERLOCK is set.
show more ...
|
Revision tags: release/4.6.2_cvs, release/4.6.2 |
|
#
cf409a0a |
| 30-Jul-2002 |
Jeff Roberson <jeff@FreeBSD.org> |
- Add automatic post vop debug checks. These work in both the success and failure cases.
|
Revision tags: release/4.6.1 |
|
#
5c08ffed |
| 07-Jul-2002 |
Jeff Roberson <jeff@FreeBSD.org> |
- Use 'options DEBUG_VFS_LOCKS' instead of the DEBUG_ALL_VFS_LOCKS environment variable to enable the lock verifiction code.
|
#
d1376090 |
| 06-Jul-2002 |
Jeff Roberson <jeff@FreeBSD.org> |
Add a new configuration directive that inserts calls to debugging functions in the VOP inlines. This is intended to replace the simple locking specifications for calls that have more complicated beh
Add a new configuration directive that inserts calls to debugging functions in the VOP inlines. This is intended to replace the simple locking specifications for calls that have more complicated behavior such as rename and lookup.
The syntax of the new entries is: #! name pre/post function
If the function is marked 'pre' it is executed prior to calling the VOP and takes a pointer to a struct vop_{name}_args as it's only parameter.
If the function is marked 'post' it is executed after the VOP call and takes a pointer to a struct vop_{name}_args as it's first parameter and the integer return value from the vop as the second paramter.
show more ...
|
Revision tags: release/4.6.0_cvs |
|
#
ef317162 |
| 04-May-2002 |
Jeff Roberson <jeff@FreeBSD.org> |
include systm.h in vnode_if.c so that panic is defined when we're doing DEBUG_ALL_VFS_LOCKS.
|
#
fda3199f |
| 01-Mar-2002 |
David E. O'Brien <obrien@FreeBSD.org> |
Return vnode_if back to its AWK roots. It became a Perl script in rev 1.20. This removes one more dependence on perl for the kernel build.
|
Revision tags: release/4.5.0_cvs, release/4.4.0_cvs |
|
#
a262ae82 |
| 04-Jan-2002 |
Mike Silbersack <silby@FreeBSD.org> |
Throw the $FreeBSD$s back in, properly escaping them.
|
#
91ea78c5 |
| 04-Jan-2002 |
Mike Silbersack <silby@FreeBSD.org> |
Remove $FreeBSD$s from previous commit; perl thinks that they're something to be interpreted. Urk.
|
#
cd6fdcb9 |
| 03-Jan-2002 |
Mike Silbersack <silby@FreeBSD.org> |
Solve vnode_if.pl's identity crisis; make sure that it refers to itself as vnode_if.pl instead of vnode_if.sh.
PR: 33509 MFC after: 3 weeks
|
#
b40ce416 |
| 12-Sep-2001 |
Julian Elischer <julian@FreeBSD.org> |
KSE Milestone 2 Note ALL MODULES MUST BE RECOMPILED make the kernel aware that there are smaller units of scheduling than the process. (but only allow one thread per process at this time). This is fu
KSE Milestone 2 Note ALL MODULES MUST BE RECOMPILED make the kernel aware that there are smaller units of scheduling than the process. (but only allow one thread per process at this time). This is functionally equivalent to teh previousl -current except that there is a thread associated with each process.
Sorry john! (your next MFC will be a doosie!)
Reviewed by: peter@freebsd.org, dillon@freebsd.org
X-MFC after: ha ha ha ha
show more ...
|
Revision tags: release/4.3.0_cvs, release/4.3.0 |
|
#
ec4dff5e |
| 23-Mar-2001 |
Alfred Perlstein <alfred@FreeBSD.org> |
replace calls to non-existant bail() subroutine with calls to the die() builtin function.
|
#
7ca7bbb3 |
| 06-Dec-2000 |
Peter Wemm <peter@FreeBSD.org> |
Simplify this a bit so that it doesn't have to generate silly redundant __P() prototypes when an ansi-style static inline is a prototype already. Since vnode_if.[ch] are generated on the fly, there a
Simplify this a bit so that it doesn't have to generate silly redundant __P() prototypes when an ansi-style static inline is a prototype already. Since vnode_if.[ch] are generated on the fly, there are no CVS diffs to mess up.
show more ...
|
Revision tags: release/4.2.0, release/4.1.1_cvs, release/4.1.0 |
|
#
35b1da80 |
| 27-Jun-2000 |
Alfred Perlstein <alfred@FreeBSD.org> |
remove crufty exec stuff, perl is in the base system make it work with warnings on (there was some harmless use of uninitialized variables) make it work with 'use strict'
Approved by: peter
|
Revision tags: release/3.5.0_cvs, release/4.0.0_cvs, release/3.4.0_cvs |
|
#
6f77b2de |
| 12-Dec-1999 |
Peter Wemm <peter@FreeBSD.org> |
Use a seperate -c and -h mode. The vnode_if.c file is compiled only into the kernel while the vnode_if.h header is a bunch of inlines to call the code that is in the kernel. Generating the .h file o
Use a seperate -c and -h mode. The vnode_if.c file is compiled only into the kernel while the vnode_if.h header is a bunch of inlines to call the code that is in the kernel. Generating the .h file on the fly is kinda bogus because it has to match the one compiled into the kernel.
IMHO we should have kern/vnode_if.c and sys/vnode_if.h committed in the tree but that's another battle.
show more ...
|
#
3176a7fe |
| 26-Sep-1999 |
Eivind Eklund <eivind@FreeBSD.org> |
Update this to be able to output ASSERT_VOP_(UN)LOCKED() based on the lock specifications in kern/vnode_if.src. At present, this do not distinguish between exclusive and shared locks, and the kernel
Update this to be able to output ASSERT_VOP_(UN)LOCKED() based on the lock specifications in kern/vnode_if.src. At present, this do not distinguish between exclusive and shared locks, and the kernel is so full of bugs in this area that running with auto-generation of assertions enabled makes DEBUG_VFS_LOCKS totally useless for anybody that has used it for anything prior to outputting automated assertions. Due to this, I made vnode_if.sh only output locking assertions if you have the environment variable DEBUG_ALL_VFS_LOCKS set to "YES". In order to actually use the assertions, you need to also add "options DEBUG_VFS_LOCKS" to your kernel config file.
Urged to commit by: phk
show more ...
|
Revision tags: release/3.3.0_cvs |
|
#
c3aac50f |
| 28-Aug-1999 |
Peter Wemm <peter@FreeBSD.org> |
$Id$ -> $FreeBSD$
|
#
f9c8cab5 |
| 17-Jun-1999 |
Kirk McKusick <mckusick@FreeBSD.org> |
Add a vnode argument to VOP_BWRITE to get rid of the last vnode operator special case. Delete special case code from vnode_if.sh, vnode_if.src, umap_vnops.c, and null_vnops.c.
|
Revision tags: release/3.2.0 |
|
#
4adbda97 |
| 28-Feb-1999 |
Bruce Evans <bde@FreeBSD.org> |
Declare static __inline functions as __inline in their forward declaration.
Fixed some comments.
Fixed a staticization botch.
|
Revision tags: release/3.1.0, release/3.0.0, release/2.2.8 |
|
#
4e61198e |
| 10-Nov-1998 |
Peter Wemm <peter@FreeBSD.org> |
Make the vnode opv vector construction fully dynamic. Previously we leaked memory on each unload and were limited to items referenced in the kernel copy of vnode_if.c. Now a kernel module is free t
Make the vnode opv vector construction fully dynamic. Previously we leaked memory on each unload and were limited to items referenced in the kernel copy of vnode_if.c. Now a kernel module is free to create it's own VOP_FOO() routines and the rest of the system will happily deal with it, including passthrough layers like union/umap/etc.
Have VFS_SET() call a common vfs_modevent() handler rather than inline duplicating the common code all over the place.
Have VNODEOP_SET() have the vnodeops removed at unload time (assuming a module) so that the vop_t ** vector is reclaimed.
Slightly adjust the vop_t ** vectors so that calling slot 0 is a panic rather than a page fault. This could happen if VOP_something() was called without *any* handlers being present anywhere (including in vfs_default.c). slot 1 becomes the default vector for the vnodeop table.
TODO: reclaim zones on unload (eg: nfs code)
show more ...
|
Revision tags: release/2.2.7 |
|
#
fd5d1124 |
| 04-Jul-1998 |
Julian Elischer <julian@FreeBSD.org> |
VOP_STRATEGY grows an (struct vnode *) argument as the value in b_vp is often not really what you want. (and needs to be frobbed). more cleanups will follow this. Reviewed by: Bruce Evans <bde@freebs
VOP_STRATEGY grows an (struct vnode *) argument as the value in b_vp is often not really what you want. (and needs to be frobbed). more cleanups will follow this. Reviewed by: Bruce Evans <bde@freebsd.org>
show more ...
|
Revision tags: release/2.2.6 |
|
#
214279ce |
| 20-Dec-1997 |
Bruce Evans <bde@FreeBSD.org> |
Use __inline instead of inline to prevent pedantic compiler warnings.
|