diff options
| author |   Fabio Massimo Di Nitto <fabbione@ubuntu.com> | 2007-11-16 09:50:40 +0000 |
|---|---|---|
| committer |   Steven Whitehouse <swhiteho@redhat.com> | 2008-01-25 08:08:06 +0000 |
| commit | 1a2781cfa5ed8eb82bb311d684f268c1822dae69 (patch) | |
| tree | d65a551d4e0b8b12365fd150fe70cfd9f14f80e2 /fs/gfs2/log.c | |
| parent | [GFS2] tidy up error message (diff) | |
| download | linux-dev-1a2781cfa5ed8eb82bb311d684f268c1822dae69.tar.xz linux-dev-1a2781cfa5ed8eb82bb311d684f268c1822dae69.zip | |
[GFS2] Fix runtime issue with UP kernels
The issue is indeed UP vs SMP and it is totally random.
spin_is_locked() is a bad assertion because there is no correct answer on UP.
on UP spin_is_locked() has to return either one value or another, always.
This means that in my setup I am lucky enough to trigger the issue and your you
are lucky enough not to.
the patch in attachment removes the bogus calls to BUG_ON and according to David
(in CC and thanks for the long explanation on the problem) we can rely upon
things like lockdep to find problem that might be trying to catch.
Signed-off-by: Fabio M. Di Nitto <fabbione@ubuntu.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | fs/gfs2/log.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/gfs2/log.c b/fs/gfs2/log.c index 4dcc7a8cda22..96dcf050e6c9 100644 --- a/fs/gfs2/log.c +++ b/fs/gfs2/log.c @@ -92,8 +92,6 @@ static void gfs2_ail1_start_one(struct gfs2_sbd *sdp, struct gfs2_ail *ai) struct buffer_head *bh; int retry; - BUG_ON(!spin_is_locked(&sdp->sd_log_lock)); - do { retry = 0; |