[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4B201AB8.3090904@gmail.com>
Date: Wed, 09 Dec 2009 22:46:32 +0100
From: Eric Dumazet <eric.dumazet@...il.com>
To: David Miller <davem@...emloft.net>
CC: ilpo.jarvinen@...sinki.fi, shemminger@...tta.com,
netdev@...r.kernel.org
Subject: Re: [PATCH] net: dst_release() cleanup
Le 09/12/2009 05:35, David Miller a écrit :
>
> Using side effects in a debugging macro, that's always asking for
> trouble. Every time I see this thing I'm going to do a double
> take on it.
>
> We can pull the return value into an 'int' with a descriptive name
> such as "orig_dst_refcnt", and also use WARN() to make a descriptive
> error message for kerneloops.org to log if it triggers.
>
> Ok Eric?
Sure, I'll submit this when net-next-2.6 reopens :)
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists