diff options
| author | Steven Whitehouse <swhiteho@redhat.com> | 2009-05-19 10:01:18 +0100 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2009-05-19 10:01:18 +0100 |
| commit | fe64d517df0970a68417184a12fcd4ba0589cc28 (patch) | |
| tree | d977f214fdf6ba96254cfbf6683e8583ecebe504 /fs/gfs2/recovery.h | |
| parent | 9582d41135c0d362f04ed6bf3dc8d693a7eafee2 (diff) | |
| download | op-kernel-dev-fe64d517df0970a68417184a12fcd4ba0589cc28.zip op-kernel-dev-fe64d517df0970a68417184a12fcd4ba0589cc28.tar.gz | |
GFS2: Umount recovery race fix
This patch fixes a race condition where we can receive recovery
requests part way through processing a umount. This was causing
problems since the recovery thread had already gone away.
Looking in more detail at the recovery code, it was really trying
to implement a slight variation on a work queue, and that happens to
align nicely with the recently introduced slow-work subsystem. As a
result I've updated the code to use slow-work, rather than its own home
grown variety of work queue.
When using the wait_on_bit() function, I noticed that the wait function
that was supplied as an argument was appearing in the WCHAN field, so
I've updated the function names in order to produce more meaningful
output.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/recovery.h')
| -rw-r--r-- | fs/gfs2/recovery.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/recovery.h b/fs/gfs2/recovery.h index a8218ea..1616ac2 100644 --- a/fs/gfs2/recovery.h +++ b/fs/gfs2/recovery.h @@ -28,7 +28,7 @@ extern void gfs2_revoke_clean(struct gfs2_sbd *sdp); extern int gfs2_find_jhead(struct gfs2_jdesc *jd, struct gfs2_log_header_host *head); extern int gfs2_recover_journal(struct gfs2_jdesc *gfs2_jd); -extern int gfs2_recoverd(void *data); +extern struct slow_work_ops gfs2_recover_ops; #endif /* __RECOVERY_DOT_H__ */ |
