[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Cluster-devel] [DLM][PATCH] lowcomms init fixes



On Mon, Apr 23, 2007 at 11:19:52AM +0100, Patrick Caulfield wrote:
> Thanks for finding & fixing those bugs Josef.
> I've changed the patch a little, making the comparison always <= max_nodeid, I
> prefer max_modeid to be what it says it as and not 1 more than the max node ID.
> 
> I'm a little concerned about the need for idr_init() in lowcomms_close(). Surely
> if the idr was properly cleaned out it would not be necessary - though somehow
> it seems to be, Do you know what's actually happening here ?
> 
> Anyway, here's the modified patch. it seems to work for me, but I'd be grateful
> if you could run it through your tests too, cos I'm supposed to be working on
> something else at the moment!
> 
> Steve, can you wait till Josef has tested this patch before adding it to the git
> tree? thanks.
> 

K I tested it and it works great, thanks much Patrick.

Josef


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]