From 218fd3fbb1876fcdbff9698cc2b334c20cba125a Mon Sep 17 00:00:00 2001 From: Bart Van Assche Date: Mon, 19 Dec 2011 16:11:43 +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 (merge r3945 from trunk). git-svn-id: http://svn.code.sf.net/p/scst/svn/branches/2.2.0.x@4011 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 bfda494ee..755d529e1 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;