lkml.org 
[lkml]   [2006]   [Apr]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [patch 11/26] IPOB: Move destructor from neigh->ops to neigh_param
From
Date
    David> You were using an interface in an unintended way.

There were a lot of opportunities to suggest a better way or even just
raise the alarm when IPoIB was first being reviewed. And I don't
remember anyone giving any guidance or insight into the neighbour
destructor design the three or four times Michael raised the issue of
the IPoIB crash and posted this patch for review....

David> Do you know %100 for certain that moving that callback to a
David> different location won't break anything?

Of course it's not %100 certain, but it definitely fixes a panic in
IPoIB, and the clip.c change looks "obviously correct."

If this patch is too risky for -stable, that's fine. But let's be
clear that it _does_ fix a panic people hit in practice, and as far as
I know it doesn't break the ATM build

- R.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-04-05 02:45    [W:0.099 / U:0.680 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site