mirrors/scst
1
0
Fork 0
mirror of https://github.com/SCST-project/scst.git synced 2026-05-28 09:20:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixes race on a scst_user device cleanup

Browse Source 
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@958 d57e44dd-8a1f-0410-8b47-8ef2f437770f
This commit is contained in:
Vladislav Bolkhovitin
2009-07-17 10:42:32 +00:00
parent 29da0878e3
commit 3ec348004f
1 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
scst/src/dev_handlers/scst_user.c
View File

@@ -2150,11 +2150,11 @@ out:
return;
}
static void dev_user_unjam_dev(struct scst_user_dev *dev)
static int dev_user_unjam_dev(struct scst_user_dev *dev)
__releases(&dev->cmd_lists.cmd_list_lock)
__acquires(&dev->cmd_lists.cmd_list_lock)
{
int i;
int i, res = 0;
struct scst_user_cmd *ucmd;
TRACE_ENTRY();
@@ -2168,6 +2168,8 @@ repeat:
struct list_head *head = &dev->ucmd_hash[i];
list_for_each_entry(ucmd, head, hash_list_entry) {
res++;
if (!ucmd->sent_to_user)
continue;
@@ -2192,8 +2194,8 @@ repeat:
spin_unlock_irq(&dev->cmd_lists.cmd_list_lock);
TRACE_EXIT();
return;
TRACE_EXIT_RES(res);
return res;
}
static int dev_user_process_reply_tm_exec(struct scst_user_cmd *ucmd,
@@ -3125,7 +3127,7 @@ out:
static int dev_user_process_cleanup(struct scst_user_dev *dev)
{
struct scst_user_cmd *ucmd;
int rc, res = 1;
int rc = 0, res = 1;
TRACE_ENTRY();
@@ -3133,9 +3135,13 @@ static int dev_user_process_cleanup(struct scst_user_dev *dev)
wake_up_all(&dev->cmd_lists.cmd_list_waitQ); /* just in case */
while (1) {
int rc1;
TRACE_DBG("Cleanuping dev %p", dev);
dev_user_unjam_dev(dev);
rc1 = dev_user_unjam_dev(dev);
if ((rc1 == 0) && (rc == -EAGAIN) && dev->cleanup_done)
break;
spin_lock_irq(&dev->cmd_lists.cmd_list_lock);
@@ -3146,9 +3152,7 @@ static int dev_user_process_cleanup(struct scst_user_dev *dev)
spin_unlock_irq(&dev->cmd_lists.cmd_list_lock);
if (rc == -EAGAIN) {
if (dev->cleanup_done)
break;
else {
if (!dev->cleanup_done) {
TRACE_DBG("No more commands (dev %p)", dev);
goto out;
}