Skip to content

Commit

Permalink
---
Browse files Browse the repository at this point in the history
yaml
---
r: 280382
b: refs/heads/master
c: ec01247
h: refs/heads/master
v: v3
  • Loading branch information
Tejun Heo committed Nov 21, 2011
1 parent 123b2a9 commit 25ee524
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
2 changes: 1 addition & 1 deletion [refs]
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
---
refs/heads/master: 839e3407d90a810318d17c17ceb3d5928a910704
refs/heads/master: ec012476af73a1a8a82565a915e9b48c2e337878
13 changes: 7 additions & 6 deletions trunk/drivers/usb/storage/usb.c
Original file line number Diff line number Diff line change
Expand Up @@ -831,24 +831,25 @@ static int usb_stor_scan_thread(void * __us)

dev_dbg(dev, "device found\n");

set_freezable_with_signal();
set_freezable();

/*
* Wait for the timeout to expire or for a disconnect
*
* We can't freeze in this thread or we risk causing khubd to
* fail to freeze, but we can't be non-freezable either. Nor can
* khubd freeze while waiting for scanning to complete as it may
* hold the device lock, causing a hang when suspending devices.
* So we request a fake signal when freezing and use
* interruptible sleep to kick us out of our wait early when
* freezing happens.
* So instead of using wait_event_freezable(), explicitly test
* for (DONT_SCAN || freezing) in interruptible wait and proceed
* if any of DONT_SCAN, freezing or timeout has happened.
*/
if (delay_use > 0) {
dev_dbg(dev, "waiting for device to settle "
"before scanning\n");
wait_event_interruptible_timeout(us->delay_wait,
test_bit(US_FLIDX_DONT_SCAN, &us->dflags),
delay_use * HZ);
test_bit(US_FLIDX_DONT_SCAN, &us->dflags) ||
freezing(current), delay_use * HZ);
}

/* If the device is still connected, perform the scanning */
Expand Down

0 comments on commit 25ee524

Please sign in to comment.