Re: [PATCH v15 13/15] pvqspinlock: Only kick CPU at unlock time

From: Waiman Long
Date: Thu Apr 09 2015 - 18:06:26 EST


On 04/09/2015 03:57 PM, Peter Zijlstra wrote:
On Mon, Apr 06, 2015 at 10:55:48PM -0400, Waiman Long wrote:

@@ -219,24 +236,30 @@ static void pv_wait_node(struct mcs_spinlock *node)
}

/*
+ * Called after setting next->locked = 1& lock acquired.
+ * Check if the the CPU has been halted. If so, set the _Q_SLOW_VAL flag
+ * and put an entry into the lock hash table to be waken up at unlock time.
*/
-static void pv_kick_node(struct mcs_spinlock *node)
+static void pv_scan_next(struct qspinlock *lock, struct mcs_spinlock *node)
I'm not too sure about that name change..

It is because the function will no longer kick the cpu. So I change the name to reflect that. I am not good at naming. Please let me know if you have a better name.

Cheers,
Longman

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