From f8c3b26620e07aed333a57619a766b6a98b5e89f Mon Sep 17 00:00:00 2001 From: Vladislav Bolkhovitin Date: Fri, 2 Dec 2011 02:24:41 +0000 Subject: [PATCH] disk_work.result is set by disk_cmd_done() and read by disk_exec(). Since there is a complete_all() / wait_for_completion() pair between both accesses the volatile keyword is not necessary here. BSD-signed-off-by: Bart Van Assche git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@3945 d57e44dd-8a1f-0410-8b47-8ef2f437770f --- scst/src/dev_handlers/scst_disk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scst/src/dev_handlers/scst_disk.c b/scst/src/dev_handlers/scst_disk.c index f69786473..e4de2e181 100644 --- a/scst/src/dev_handlers/scst_disk.c +++ b/scst/src/dev_handlers/scst_disk.c @@ -363,7 +363,7 @@ static bool disk_on_sg_tablesize_low(struct scst_cmd *cmd) struct disk_work { struct scst_cmd *cmd; struct completion disk_work_cmpl; - volatile int result; + int result; unsigned int left; uint64_t save_lba; unsigned int save_len;