[PATCH 3/5] uas: rename work list lock + list field
This patch prepares for the addition of another list and renames the
work list lock and the list_head field in struct uas_cmd_info.
Signed-off-by: Gerd Hoffmann
---
drivers/usb/storage/uas.c | 50 +++
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/drivers/usb/storage/uas.c b/drivers/usb/storage/uas.c
index fc08ee9..db09bda 100644
--- a/drivers/usb/storage/uas.c
+++ b/drivers/usb/storage/uas.c
@@ -77,7 +77,7 @@ struct uas_cmd_info {
struct urb *cmd_urb;
struct urb *data_in_urb;
struct urb *data_out_urb;
- struct list_head list;
+ struct list_head work;
};
/* I hate forward declarations, but I actually have a loop */
@@ -89,7 +89,7 @@ static void uas_configure_endpoints(struct uas_dev_info
*devinfo);
static void uas_free_streams(struct uas_dev_info *devinfo);
static DECLARE_WORK(uas_work, uas_do_work);
-static DEFINE_SPINLOCK(uas_work_lock);
+static DEFINE_SPINLOCK(uas_lists_lock);
static LIST_HEAD(uas_work_list);
static void uas_unlink_data_urbs(struct uas_dev_info *devinfo,
@@ -124,11 +124,11 @@ static void uas_do_work(struct work_struct *work)
unsigned long flags;
int err;
- spin_lock_irq(_work_lock);
+ spin_lock_irq(_lists_lock);
list_replace_init(_work_list, );
- spin_unlock_irq(_work_lock);
+ spin_unlock_irq(_lists_lock);
- list_for_each_entry_safe(cmdinfo, temp, , list) {
+ list_for_each_entry_safe(cmdinfo, temp, , work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -139,10 +139,10 @@ static void uas_do_work(struct work_struct *work)
cmdinfo->state &= ~IS_IN_WORK_LIST;
spin_unlock_irqrestore(>lock, flags);
if (err) {
- list_del(>list);
- spin_lock_irq(_work_lock);
- list_add_tail(>list, _work_list);
- spin_unlock_irq(_work_lock);
+ list_del(>work);
+ spin_lock_irq(_lists_lock);
+ list_add_tail(>work, _work_list);
+ spin_unlock_irq(_lists_lock);
schedule_work(_work);
}
}
@@ -155,12 +155,12 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
struct list_head list;
unsigned long flags;
- spin_lock_irq(_work_lock);
+ spin_lock_irq(_lists_lock);
list_replace_init(_work_list, );
- spin_unlock_irq(_work_lock);
+ spin_unlock_irq(_lists_lock);
spin_lock_irqsave(>lock, flags);
- list_for_each_entry_safe(cmdinfo, temp, , list) {
+ list_for_each_entry_safe(cmdinfo, temp, , work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -178,10 +178,10 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
uas_try_complete(cmnd, __func__);
} else {
/* not our uas device, relink into list */
- list_del(>list);
- spin_lock_irq(_work_lock);
- list_add_tail(>list, _work_list);
- spin_unlock_irq(_work_lock);
+ list_del(>work);
+ spin_lock_irq(_lists_lock);
+ list_add_tail(>work, _work_list);
+ spin_unlock_irq(_lists_lock);
}
}
spin_unlock_irqrestore(>lock, flags);
@@ -288,10 +288,10 @@ static void uas_xfer_data(struct urb *urb, struct
scsi_cmnd *cmnd,
cmdinfo->state |= direction | SUBMIT_STATUS_URB;
err = uas_submit_urbs(cmnd, cmnd->device->hostdata, GFP_ATOMIC);
if (err) {
- spin_lock(_work_lock);
- list_add_tail(>list, _work_list);
+ spin_lock(_lists_lock);
+ list_add_tail(>work, _work_list);
cmdinfo->state |= IS_IN_WORK_LIST;
- spin_unlock(_work_lock);
+ spin_unlock(_lists_lock);
schedule_work(_work);
}
}
@@ -694,10 +694,10 @@ static int uas_queuecommand_lck(struct scsi_cmnd *cmnd,
spin_unlock_irqrestore(>lock, flags);
return SCSI_MLQUEUE_DEVICE_BUSY;
}
- spin_lock(_work_lock);
- list_add_tail(>list, _work_list);
+ spin_lock(_lists_lock);
+ list_add_tail(>work, _work_list);
cmdinfo->state |= IS_IN_WORK_LIST;
- spin_unlock(_work_lock);
+ spin_unlock(_lists_lock);
schedule_work(_work);
}
[PATCH 3/5] uas: rename work list lock + list field
This patch prepares for the addition of another list and renames the
work list lock and the list_head field in struct uas_cmd_info.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
drivers/usb/storage/uas.c | 50 +++
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/drivers/usb/storage/uas.c b/drivers/usb/storage/uas.c
index fc08ee9..db09bda 100644
--- a/drivers/usb/storage/uas.c
+++ b/drivers/usb/storage/uas.c
@@ -77,7 +77,7 @@ struct uas_cmd_info {
struct urb *cmd_urb;
struct urb *data_in_urb;
struct urb *data_out_urb;
- struct list_head list;
+ struct list_head work;
};
/* I hate forward declarations, but I actually have a loop */
@@ -89,7 +89,7 @@ static void uas_configure_endpoints(struct uas_dev_info
*devinfo);
static void uas_free_streams(struct uas_dev_info *devinfo);
static DECLARE_WORK(uas_work, uas_do_work);
-static DEFINE_SPINLOCK(uas_work_lock);
+static DEFINE_SPINLOCK(uas_lists_lock);
static LIST_HEAD(uas_work_list);
static void uas_unlink_data_urbs(struct uas_dev_info *devinfo,
@@ -124,11 +124,11 @@ static void uas_do_work(struct work_struct *work)
unsigned long flags;
int err;
- spin_lock_irq(uas_work_lock);
+ spin_lock_irq(uas_lists_lock);
list_replace_init(uas_work_list, list);
- spin_unlock_irq(uas_work_lock);
+ spin_unlock_irq(uas_lists_lock);
- list_for_each_entry_safe(cmdinfo, temp, list, list) {
+ list_for_each_entry_safe(cmdinfo, temp, list, work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -139,10 +139,10 @@ static void uas_do_work(struct work_struct *work)
cmdinfo-state = ~IS_IN_WORK_LIST;
spin_unlock_irqrestore(devinfo-lock, flags);
if (err) {
- list_del(cmdinfo-list);
- spin_lock_irq(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
- spin_unlock_irq(uas_work_lock);
+ list_del(cmdinfo-work);
+ spin_lock_irq(uas_lists_lock);
+ list_add_tail(cmdinfo-work, uas_work_list);
+ spin_unlock_irq(uas_lists_lock);
schedule_work(uas_work);
}
}
@@ -155,12 +155,12 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
struct list_head list;
unsigned long flags;
- spin_lock_irq(uas_work_lock);
+ spin_lock_irq(uas_lists_lock);
list_replace_init(uas_work_list, list);
- spin_unlock_irq(uas_work_lock);
+ spin_unlock_irq(uas_lists_lock);
spin_lock_irqsave(devinfo-lock, flags);
- list_for_each_entry_safe(cmdinfo, temp, list, list) {
+ list_for_each_entry_safe(cmdinfo, temp, list, work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -178,10 +178,10 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
uas_try_complete(cmnd, __func__);
} else {
/* not our uas device, relink into list */
- list_del(cmdinfo-list);
- spin_lock_irq(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
- spin_unlock_irq(uas_work_lock);
+ list_del(cmdinfo-work);
+ spin_lock_irq(uas_lists_lock);
+ list_add_tail(cmdinfo-work, uas_work_list);
+ spin_unlock_irq(uas_lists_lock);
}
}
spin_unlock_irqrestore(devinfo-lock, flags);
@@ -288,10 +288,10 @@ static void uas_xfer_data(struct urb *urb, struct
scsi_cmnd *cmnd,
cmdinfo-state |= direction | SUBMIT_STATUS_URB;
err = uas_submit_urbs(cmnd, cmnd-device-hostdata, GFP_ATOMIC);
if (err) {
- spin_lock(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
+ spin_lock(uas_lists_lock);
+ list_add_tail(cmdinfo-work, uas_work_list);
cmdinfo-state |= IS_IN_WORK_LIST;
- spin_unlock(uas_work_lock);
+ spin_unlock(uas_lists_lock);
schedule_work(uas_work);
}
}
@@ -694,10 +694,10 @@ static int uas_queuecommand_lck(struct scsi_cmnd *cmnd,
spin_unlock_irqrestore(devinfo-lock, flags);
return SCSI_MLQUEUE_DEVICE_BUSY;
}
- spin_lock(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
+
[PATCH 3/5] uas: rename work list lock + list field
This patch prepares for the addition of another list and renames the
work list lock and the list_head field in struct uas_cmd_info.
Signed-off-by: Gerd Hoffmann
---
drivers/usb/storage/uas.c | 50 +++
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/drivers/usb/storage/uas.c b/drivers/usb/storage/uas.c
index f89202f..a63972a 100644
--- a/drivers/usb/storage/uas.c
+++ b/drivers/usb/storage/uas.c
@@ -77,7 +77,7 @@ struct uas_cmd_info {
struct urb *cmd_urb;
struct urb *data_in_urb;
struct urb *data_out_urb;
- struct list_head list;
+ struct list_head work;
};
/* I hate forward declarations, but I actually have a loop */
@@ -89,7 +89,7 @@ static void uas_configure_endpoints(struct uas_dev_info
*devinfo);
static void uas_free_streams(struct uas_dev_info *devinfo);
static DECLARE_WORK(uas_work, uas_do_work);
-static DEFINE_SPINLOCK(uas_work_lock);
+static DEFINE_SPINLOCK(uas_lists_lock);
static LIST_HEAD(uas_work_list);
static void uas_unlink_data_urbs(struct uas_dev_info *devinfo,
@@ -124,11 +124,11 @@ static void uas_do_work(struct work_struct *work)
unsigned long flags;
int err;
- spin_lock_irq(_work_lock);
+ spin_lock_irq(_lists_lock);
list_replace_init(_work_list, );
- spin_unlock_irq(_work_lock);
+ spin_unlock_irq(_lists_lock);
- list_for_each_entry_safe(cmdinfo, temp, , list) {
+ list_for_each_entry_safe(cmdinfo, temp, , work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -139,10 +139,10 @@ static void uas_do_work(struct work_struct *work)
cmdinfo->state &= ~IS_IN_WORK_LIST;
spin_unlock_irqrestore(>lock, flags);
if (err) {
- list_del(>list);
- spin_lock_irq(_work_lock);
- list_add_tail(>list, _work_list);
- spin_unlock_irq(_work_lock);
+ list_del(>work);
+ spin_lock_irq(_lists_lock);
+ list_add_tail(>work, _work_list);
+ spin_unlock_irq(_lists_lock);
schedule_work(_work);
}
}
@@ -155,12 +155,12 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
struct list_head list;
unsigned long flags;
- spin_lock_irq(_work_lock);
+ spin_lock_irq(_lists_lock);
list_replace_init(_work_list, );
- spin_unlock_irq(_work_lock);
+ spin_unlock_irq(_lists_lock);
spin_lock_irqsave(>lock, flags);
- list_for_each_entry_safe(cmdinfo, temp, , list) {
+ list_for_each_entry_safe(cmdinfo, temp, , work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -178,10 +178,10 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
uas_try_complete(cmnd, __func__);
} else {
/* not our uas device, relink into list */
- list_del(>list);
- spin_lock_irq(_work_lock);
- list_add_tail(>list, _work_list);
- spin_unlock_irq(_work_lock);
+ list_del(>work);
+ spin_lock_irq(_lists_lock);
+ list_add_tail(>work, _work_list);
+ spin_unlock_irq(_lists_lock);
}
}
spin_unlock_irqrestore(>lock, flags);
@@ -288,10 +288,10 @@ static void uas_xfer_data(struct urb *urb, struct
scsi_cmnd *cmnd,
cmdinfo->state |= direction | SUBMIT_STATUS_URB;
err = uas_submit_urbs(cmnd, cmnd->device->hostdata, GFP_ATOMIC);
if (err) {
- spin_lock(_work_lock);
- list_add_tail(>list, _work_list);
+ spin_lock(_lists_lock);
+ list_add_tail(>work, _work_list);
cmdinfo->state |= IS_IN_WORK_LIST;
- spin_unlock(_work_lock);
+ spin_unlock(_lists_lock);
schedule_work(_work);
}
}
@@ -694,10 +694,10 @@ static int uas_queuecommand_lck(struct scsi_cmnd *cmnd,
spin_unlock_irqrestore(>lock, flags);
return SCSI_MLQUEUE_DEVICE_BUSY;
}
- spin_lock(_work_lock);
- list_add_tail(>list, _work_list);
+ spin_lock(_lists_lock);
+ list_add_tail(>work, _work_list);
cmdinfo->state |= IS_IN_WORK_LIST;
- spin_unlock(_work_lock);
+ spin_unlock(_lists_lock);
schedule_work(_work);
}
[PATCH 3/5] uas: rename work list lock + list field
This patch prepares for the addition of another list and renames the
work list lock and the list_head field in struct uas_cmd_info.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
drivers/usb/storage/uas.c | 50 +++
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/drivers/usb/storage/uas.c b/drivers/usb/storage/uas.c
index f89202f..a63972a 100644
--- a/drivers/usb/storage/uas.c
+++ b/drivers/usb/storage/uas.c
@@ -77,7 +77,7 @@ struct uas_cmd_info {
struct urb *cmd_urb;
struct urb *data_in_urb;
struct urb *data_out_urb;
- struct list_head list;
+ struct list_head work;
};
/* I hate forward declarations, but I actually have a loop */
@@ -89,7 +89,7 @@ static void uas_configure_endpoints(struct uas_dev_info
*devinfo);
static void uas_free_streams(struct uas_dev_info *devinfo);
static DECLARE_WORK(uas_work, uas_do_work);
-static DEFINE_SPINLOCK(uas_work_lock);
+static DEFINE_SPINLOCK(uas_lists_lock);
static LIST_HEAD(uas_work_list);
static void uas_unlink_data_urbs(struct uas_dev_info *devinfo,
@@ -124,11 +124,11 @@ static void uas_do_work(struct work_struct *work)
unsigned long flags;
int err;
- spin_lock_irq(uas_work_lock);
+ spin_lock_irq(uas_lists_lock);
list_replace_init(uas_work_list, list);
- spin_unlock_irq(uas_work_lock);
+ spin_unlock_irq(uas_lists_lock);
- list_for_each_entry_safe(cmdinfo, temp, list, list) {
+ list_for_each_entry_safe(cmdinfo, temp, list, work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -139,10 +139,10 @@ static void uas_do_work(struct work_struct *work)
cmdinfo-state = ~IS_IN_WORK_LIST;
spin_unlock_irqrestore(devinfo-lock, flags);
if (err) {
- list_del(cmdinfo-list);
- spin_lock_irq(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
- spin_unlock_irq(uas_work_lock);
+ list_del(cmdinfo-work);
+ spin_lock_irq(uas_lists_lock);
+ list_add_tail(cmdinfo-work, uas_work_list);
+ spin_unlock_irq(uas_lists_lock);
schedule_work(uas_work);
}
}
@@ -155,12 +155,12 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
struct list_head list;
unsigned long flags;
- spin_lock_irq(uas_work_lock);
+ spin_lock_irq(uas_lists_lock);
list_replace_init(uas_work_list, list);
- spin_unlock_irq(uas_work_lock);
+ spin_unlock_irq(uas_lists_lock);
spin_lock_irqsave(devinfo-lock, flags);
- list_for_each_entry_safe(cmdinfo, temp, list, list) {
+ list_for_each_entry_safe(cmdinfo, temp, list, work) {
struct scsi_pointer *scp = (void *)cmdinfo;
struct scsi_cmnd *cmnd = container_of(scp,
struct scsi_cmnd, SCp);
@@ -178,10 +178,10 @@ static void uas_abort_work(struct uas_dev_info *devinfo)
uas_try_complete(cmnd, __func__);
} else {
/* not our uas device, relink into list */
- list_del(cmdinfo-list);
- spin_lock_irq(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
- spin_unlock_irq(uas_work_lock);
+ list_del(cmdinfo-work);
+ spin_lock_irq(uas_lists_lock);
+ list_add_tail(cmdinfo-work, uas_work_list);
+ spin_unlock_irq(uas_lists_lock);
}
}
spin_unlock_irqrestore(devinfo-lock, flags);
@@ -288,10 +288,10 @@ static void uas_xfer_data(struct urb *urb, struct
scsi_cmnd *cmnd,
cmdinfo-state |= direction | SUBMIT_STATUS_URB;
err = uas_submit_urbs(cmnd, cmnd-device-hostdata, GFP_ATOMIC);
if (err) {
- spin_lock(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
+ spin_lock(uas_lists_lock);
+ list_add_tail(cmdinfo-work, uas_work_list);
cmdinfo-state |= IS_IN_WORK_LIST;
- spin_unlock(uas_work_lock);
+ spin_unlock(uas_lists_lock);
schedule_work(uas_work);
}
}
@@ -694,10 +694,10 @@ static int uas_queuecommand_lck(struct scsi_cmnd *cmnd,
spin_unlock_irqrestore(devinfo-lock, flags);
return SCSI_MLQUEUE_DEVICE_BUSY;
}
- spin_lock(uas_work_lock);
- list_add_tail(cmdinfo-list, uas_work_list);
+
