Thread (61 messages) 61 messages, 8 authors, 2012-10-29

Re: [PATCH v7 06/16] tracepoint: use new hashtable implementation

From: Tejun Heo <tj@kernel.org>
Date: 2012-10-29 19:13:06
Also in: dm-devel, linux-nfs, lkml, netdev

On Mon, Oct 29, 2012 at 03:09:36PM -0400, Sasha Levin wrote:
The other thing is whether hash_init() should be called for hashtables
that were created with DEFINE_HASHTABLE(). That point was raised by
Neil Brown last time this series went around, and it seems that no one
objected to the point that it should be consistent across the code.
Hmmm?  If something is DEFINE_XXX()'d, you definitely shouldn't be
calling XXX_init() on it.  That's how it is with most other abstract
data types and you need *VERY* strong rationale to deviate from that.

Thanks.

-- 
tejun

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help