FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Cluster Development

 
 
LinkBack Thread Tools
 
Old 01-09-2008, 02:06 PM
Patrick Caulfeld
 
Default 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>
 
Old 01-10-2008, 09:21 AM
Steven Whitehouse
 
Default dlm close othercons

Hi,

Now in the -nmw git tree. Thanks,

Steve.

On Wed, 2008-01-09 at 15:06 +0000, Patrick Caulfeld wrote:
> 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>
 
Old 01-18-2008, 05:46 AM
"Fabio M. Di Nitto"
 
Default dlm close othercons

On Wed, 9 Jan 2008, Patrick Caulfeld wrote:


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>



http://git.kernel.org/?p=linux/kernel/git/steve/gfs2-2.6-nmw.git;a=commitdiff;h=89f0bb2873fcaa8e57831dd3f0c 013cabbd99012

this doesn't look right...

shouldn't we free the con->other _before_ we free con?

That's what we were doing before applying all that patch set that removed
the original kmem_cache_free...


Fabio

--
I'm going to make him an offer he can't refuse.
 
Old 01-18-2008, 07:27 AM
Patrick Caulfeld
 
Default dlm close othercons

Fabio M. Di Nitto wrote:
> On Wed, 9 Jan 2008, Patrick Caulfeld wrote:
>
>> 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>
>>
>
> http://git.kernel.org/?p=linux/kernel/git/steve/gfs2-2.6-nmw.git;a=commitdiff;h=89f0bb2873fcaa8e57831dd3f0c 013cabbd99012
>
>
> this doesn't look right...
>
> shouldn't we free the con->other _before_ we free con?
>
> That's what we were doing before applying all that patch set that
> removed the original kmem_cache_free...
>


ARGH! you're right, of course. Patch to fix is attached.

Thanks Fabio, I'm glad that someone reads my patches ... I just wish it
had been me ;-)

Patrick

Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
 
Old 01-18-2008, 07:34 AM
"Fabio M. Di Nitto"
 
Default dlm close othercons

On Fri, 18 Jan 2008, Patrick Caulfeld wrote:


Fabio M. Di Nitto wrote:

On Wed, 9 Jan 2008, Patrick Caulfeld wrote:


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>



http://git.kernel.org/?p=linux/kernel/git/steve/gfs2-2.6-nmw.git;a=commitdiff;h=89f0bb2873fcaa8e57831dd3f0c 013cabbd99012


this doesn't look right...

shouldn't we free the con->other _before_ we free con?

That's what we were doing before applying all that patch set that
removed the original kmem_cache_free...




ARGH! you're right, of course. Patch to fix is attached.

Thanks Fabio, I'm glad that someone reads my patches ... I just wish it
had been me ;-)

Patrick

Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>



Signed-off-by: Fabio M. Di Nitto <fabbione@ubuntu.com>

Fabio

--
I'm going to make him an offer he can't refuse.
 

Thread Tools




All times are GMT. The time now is 06:31 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org