Since set_current_state() contains a memory barrier in it,

an additional barrier isn't needed.

Signed-off-by: Namhyung Kim <namhyung@gmail.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>

diff --git a/drivers/md/dm.c b/drivers/md/dm.c
index 52a8fd8eb17f08c3a02632ea57901d7c5041d7d6..2fe3017ba97cfdf808cddccad3c22b948ee8676a 100644 (file)
--- a/drivers/md/dm.c
+++ b/drivers/md/dm.c
@@ -2326,7 +2326,6 @@ static int dm_wait_for_completion(struct mapped_device *md, int interruptible)
        while (1) {
                set_current_state(interruptible);
 
-               smp_mb();
                if (!md_in_flight(md))
                        break;