diff options
author | Christoph Hellwig <hch@infradead.org> | 2008-07-18 03:11:46 -0400 |
---|---|---|
committer | Niv Sardi <xaiki@debian.org> | 2008-07-28 02:59:25 -0400 |
commit | 9f8868ffb39c2f80ba69df4552cb530b6634f646 (patch) | |
tree | fba09366faf55ee039cdbd91dff78b7d87a86d0b /fs/xfs/linux-2.6/xfs_sysctl.c | |
parent | 136f8f21b6d564f553abe6130127d16fb50432d3 (diff) |
[XFS] streamline init/exit path
Currently the xfs module init/exit code is a mess. It's farmed out over a
lot of function with very little error checking. This patch makes sure we
propagate all initialization failures properly and clean up after them.
Various runtime initializations are replaced with compile-time
initializations where possible to make this easier. The exit path is
similarly consolidated.
There's now split out function to create/destroy the kmem zones and
alloc/free the trace buffers. I've also changed the ktrace allocations to
KM_MAYFAIL and handled errors resulting from that.
And yes, we really should replace the XFS_*_TRACE ifdefs with a single
XFS_TRACE..
SGI-PV: 976035
SGI-Modid: xfs-linux-melb:xfs-kern:31354a
Signed-off-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: Niv Sardi <xaiki@sgi.com>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Diffstat (limited to 'fs/xfs/linux-2.6/xfs_sysctl.c')
-rw-r--r-- | fs/xfs/linux-2.6/xfs_sysctl.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/fs/xfs/linux-2.6/xfs_sysctl.c b/fs/xfs/linux-2.6/xfs_sysctl.c index bb997d75c05c..7dacb5bbde3f 100644 --- a/fs/xfs/linux-2.6/xfs_sysctl.c +++ b/fs/xfs/linux-2.6/xfs_sysctl.c | |||
@@ -259,15 +259,17 @@ static ctl_table xfs_root_table[] = { | |||
259 | {} | 259 | {} |
260 | }; | 260 | }; |
261 | 261 | ||
262 | void | 262 | int |
263 | xfs_sysctl_register(void) | 263 | xfs_sysctl_register(void) |
264 | { | 264 | { |
265 | xfs_table_header = register_sysctl_table(xfs_root_table); | 265 | xfs_table_header = register_sysctl_table(xfs_root_table); |
266 | if (!xfs_table_header) | ||
267 | return -ENOMEM; | ||
268 | return 0; | ||
266 | } | 269 | } |
267 | 270 | ||
268 | void | 271 | void |
269 | xfs_sysctl_unregister(void) | 272 | xfs_sysctl_unregister(void) |
270 | { | 273 | { |
271 | if (xfs_table_header) | 274 | unregister_sysctl_table(xfs_table_header); |
272 | unregister_sysctl_table(xfs_table_header); | ||
273 | } | 275 | } |