diff options
| author | Jeff Moyer <[email protected]> | 2007-12-04 23:45:02 -0800 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2007-12-05 09:21:18 -0800 |
| commit | e00ba3dae077f54cfd2af42e939a618caa7a3bca (patch) | |
| tree | eb2f29fe8df7cb29be7d1bdda8dabe825f30f182 | |
| parent | e3c0ac04f980750a368f7cd5f1b8d1d2cdc1f735 (diff) | |
aio: only account I/O wait time in read_events if there are active requests
On 2.6.24, top started showing 100% iowait on one CPU when a UML instance was
running (but completely idle). The UML code sits in io_getevents waiting for
an event to be submitted and completed.
Fix this by checking ctx->reqs_active before scheduling to determine whether
or not we are waiting for I/O.
Signed-off-by: Jeff Moyer <[email protected]>
Cc: Zach Brown <[email protected]>
Cc: Miklos Szeredi <[email protected]>
Cc: Jeff Dike <[email protected]>
Cc: "Rafael J. Wysocki" <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -rw-r--r-- | fs/aio.c | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -1161,7 +1161,12 @@ retry: ret = 0; if (to.timed_out) /* Only check after read evt */ break; - io_schedule(); + /* Try to only show up in io wait if there are ops + * in flight */ + if (ctx->reqs_active) + io_schedule(); + else + schedule(); if (signal_pending(tsk)) { ret = -EINTR; break; |