diff options
author | Jay Fenlason <fenlason@redhat.com> | 2007-10-08 17:00:29 -0400 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2007-10-16 18:00:10 -0400 |
commit | 83431cba3d847fc2296d3f38ce7feb623a1cfc45 (patch) | |
tree | c5ce3bb5ea97d624c15331f63087b8373fad4040 /drivers/firewire | |
parent | c1b91ce49158d607666dbef5b8d53cd77d397589 (diff) |
firewire: fw-cdev: reorder wakeup vs. spinlock
Signed-off-by: Jay Fenlason <fenlason@redhat.com>
Prompted by https://bugzilla.redhat.com/show_bug.cgi?id=323411
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/firewire')
-rw-r--r-- | drivers/firewire/fw-cdev.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/firewire/fw-cdev.c b/drivers/firewire/fw-cdev.c index 290d9f036199..60f1a8924a95 100644 --- a/drivers/firewire/fw-cdev.c +++ b/drivers/firewire/fw-cdev.c | |||
@@ -143,11 +143,10 @@ static void queue_event(struct client *client, struct event *event, | |||
143 | event->v[1].size = size1; | 143 | event->v[1].size = size1; |
144 | 144 | ||
145 | spin_lock_irqsave(&client->lock, flags); | 145 | spin_lock_irqsave(&client->lock, flags); |
146 | |||
147 | list_add_tail(&event->link, &client->event_list); | 146 | list_add_tail(&event->link, &client->event_list); |
148 | wake_up_interruptible(&client->wait); | ||
149 | |||
150 | spin_unlock_irqrestore(&client->lock, flags); | 147 | spin_unlock_irqrestore(&client->lock, flags); |
148 | |||
149 | wake_up_interruptible(&client->wait); | ||
151 | } | 150 | } |
152 | 151 | ||
153 | static int | 152 | static int |