Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1396491 > unrolled thread

[char-misc-next 1/2] mei: don't use wake_up_interruptible for wr_ctrl

Started byTomas Winkler <tomas.winkler@intel.com>
First post2016-05-08 23:10 +0200
Last post2016-05-08 23:10 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel


Contents

  [char-misc-next 1/2] mei: don't use wake_up_interruptible for wr_ctrl Tomas Winkler <tomas.winkler@intel.com> - 2016-05-08 23:10 +0200

#1396491 — [char-misc-next 1/2] mei: don't use wake_up_interruptible for wr_ctrl

FromTomas Winkler <tomas.winkler@intel.com>
Date2016-05-08 23:10 +0200
Subject[char-misc-next 1/2] mei: don't use wake_up_interruptible for wr_ctrl
Message-ID<rwE7E-6aJ-9@gated-at.bofh.it>
From: Alexander Usyskin <alexander.usyskin@intel.com>

wr_ctrl waiters are none interruptible, so should be waken up
with call to wake_up and not to wake_up_interruptible.

This fixes commit:
7ff4bdd ("mei: fix waiting for wr_ctrl for corner cases.")

Signed-off-by: Alexander Usyskin <alexander.usyskin@intel.com>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
---
 drivers/misc/mei/client.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/misc/mei/client.c b/drivers/misc/mei/client.c
index eed254da63a8..641c1a566687 100644
--- a/drivers/misc/mei/client.c
+++ b/drivers/misc/mei/client.c
@@ -730,7 +730,7 @@ static void mei_cl_wake_all(struct mei_cl *cl)
 	/* synchronized under device mutex */
 	if (waitqueue_active(&cl->wait)) {
 		cl_dbg(dev, cl, "Waking up ctrl write clients!\n");
-		wake_up_interruptible(&cl->wait);
+		wake_up(&cl->wait);
 	}
 }
 
-- 
2.5.5

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web