On Tue, Dec 04, 2018 at 12:50:09PM +0300, Kirill A. Shutemov wrote:

On Tue, Dec 04, 2018 at 09:21:45AM +0000, Peter Zijlstra wrote:

quoted

On Mon, Dec 03, 2018 at 11:39:58PM -0800, Alison Schofield wrote:

quoted

+struct mktme_hw_program_info {
+	struct mktme_key_program *key_program;
+	unsigned long status;
+};
+
+/* Program a KeyID on a single package. */
+static void mktme_program_package(void *hw_program_info)
+{
+	struct mktme_hw_program_info *info = hw_program_info;
+	int ret;
+
+	ret = mktme_key_program(info->key_program);
+	if (ret != MKTME_PROG_SUCCESS)
+		WRITE_ONCE(info->status, ret);

What's the purpose of that WRITE_ONCE()?

[I suggested the code to Alison.]

Yes, you're right. Simple assignment will do.

Will do. Thanks!

-- 
 Kirill A. Shutemov