diff options
| author |   Patrick Caulfeld <pcaulfie@redhat.com> | 2008-01-09 15:06:27 +0000 |
|---|---|---|
| committer |   David Teigland <teigland@redhat.com> | 2008-01-29 17:17:32 -0600 |
| commit | 39bd4177ddbeb4c86e854d3d5c4a6a26088e601e (patch) | |
| tree | 80bebc23aac81cc56dbd33d81dcb9ffbbf2582dc /fs/dlm | |
| parent | dlm: use dlm prefix on alloc and free functions (diff) | |
| download | linux-dev-39bd4177ddbeb4c86e854d3d5c4a6a26088e601e.tar.xz linux-dev-39bd4177ddbeb4c86e854d3d5c4a6a26088e601e.zip | |
dlm: close othercons
This patch addresses a problem introduced with the last round of
lowcomms patches where the 'othercon' connections do not get freed when
the DLM shuts down.
This results in the error message
"slab error in kmem_cache_destroy(): cache `dlm_conn': Can't free all
objects"
and the DLM cannot be restarted without a system reboot.
See bz#428119
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Fabio M. Di Nitto <fabbione@ubuntu.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm')
| -rw-r--r-- | fs/dlm/lowcomms.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index 57728448f1b9..7c1e5e5cccd8 100644 --- a/fs/dlm/lowcomms.c +++ b/fs/dlm/lowcomms.c @@ -1437,6 +1437,8 @@ void dlm_lowcomms_stop(void) con = __nodeid2con(i, 0); if (con) { close_connection(con, true); + if (con->othercon) + kmem_cache_free(con_cache, con->othercon); kmem_cache_free(con_cache, con); } } |