diff options
author | phk <phk@FreeBSD.org> | 2005-01-13 07:53:01 +0000 |
---|---|---|
committer | phk <phk@FreeBSD.org> | 2005-01-13 07:53:01 +0000 |
commit | 4d9781b3dbece651d7be988939f2c166bc7e8562 (patch) | |
tree | e7e985f51392cbb38e54bd0ff0579b79648afa2c /sys/fs/umapfs/umap_vnops.c | |
parent | 93aaafbe168e1ffcbe302959df6d2898d8e3d894 (diff) | |
download | FreeBSD-src-4d9781b3dbece651d7be988939f2c166bc7e8562.zip FreeBSD-src-4d9781b3dbece651d7be988939f2c166bc7e8562.tar.gz |
Change the generated VOP_ macro implementations to improve type checking
and KASSERT coverage.
After this check there is only one "nasty" cast in this code but there
is a KASSERT to protect against the wrong argument structure behind
that cast.
Un-inlining the meat of VOP_FOO() saves 35kB of text segment on a typical
kernel with no change in performance.
We also now run the checking and tracing on VOP's which have been layered
by nullfs, umapfs, deadfs or unionfs.
Add new (non-inline) VOP_FOO_AP() functions which take a "struct
foo_args" argument and does everything the VOP_FOO() macros
used to do with checks and debugging code.
Add KASSERT to VOP_FOO_AP() check for argument type being
correct.
Slim down VOP_FOO() inline functions to just stuff arguments
into the struct foo_args and call VOP_FOO_AP().
Put function pointer to VOP_FOO_AP() into vop_foo_desc structure
and make VCALL() use it instead of the current offsetoff() hack.
Retire vcall() which implemented the offsetoff()
Make deadfs and unionfs use VOP_FOO_AP() calls instead of
VCALL(), we know which specific call we want already.
Remove unneeded arguments to VCALL() in nullfs and umapfs bypass
functions.
Remove unused vdesc_offset and VOFFSET().
Generally improve style/readability of the generated code.
Diffstat (limited to 'sys/fs/umapfs/umap_vnops.c')
-rw-r--r-- | sys/fs/umapfs/umap_vnops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/fs/umapfs/umap_vnops.c b/sys/fs/umapfs/umap_vnops.c index 010e291..c7803f0 100644 --- a/sys/fs/umapfs/umap_vnops.c +++ b/sys/fs/umapfs/umap_vnops.c @@ -197,7 +197,7 @@ umap_bypass(ap) * Call the operation on the lower layer * with the modified argument structure. */ - error = VCALL(*(vps_p[0]), descp->vdesc_offset, ap); + error = VCALL(ap); /* * Maintain the illusion of call-by-value |