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
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>