rhashtable: add a note for grow and shrink decision functions
authorYing Xue <ying.xue@windriver.com>
Mon, 12 Jan 2015 06:52:24 +0000 (14:52 +0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 13 Jan 2015 19:01:00 +0000 (14:01 -0500)
As commit c0c09bfdc415 ("rhashtable: avoid unnecessary wakeup for
worker queue") moves condition statements of verifying whether hash
table size exceeds its maximum threshold or reaches its minimum
threshold from resizing functions to resizing decision functions,
we should add a note in rhashtable.h to indicate the implementation
of what the grow and shrink decision function must enforce min/max
shift, otherwise, it's failed to take min/max shift's set watermarks
into effect.

Signed-off-by: Ying Xue <ying.xue@windriver.com>
Cc: Thomas Graf <tgraf@suug.ch>
Acked-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/rhashtable.h

index 7b9bd77ed684fe1b2f3fdeda3d84babc40c41570..9570832ab07cbf633c409abdf7f1bc2ef5be5d4f 100644 (file)
@@ -79,6 +79,10 @@ struct rhashtable;
  * @obj_hashfn: Function to hash object
  * @grow_decision: If defined, may return true if table should expand
  * @shrink_decision: If defined, may return true if table should shrink
+ *
+ * Note: when implementing the grow and shrink decision function, min/max
+ * shift must be enforced, otherwise, resizing watermarks they set may be
+ * useless.
  */
 struct rhashtable_params {
        size_t                  nelem_hint;