[Cluster-devel] [GFS2] Fix runtime issue with UP kernels

2007-11-16 Thread Fabio Massimo Di Nitto
Hi Steven, the patch fixes the issues we discussed on the mailing list before and IRC. 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.

Re: [Cluster-devel] [PATCH] gfs2: tidy up error message

2007-11-16 Thread Fabio Massimo Di Nitto
Signed-off-by: Fabio M. Di Nitto [EMAIL PROTECTED] David Teigland wrote: Print error with log_error() to be consistent with others. Signed-off-by: David Teigland [EMAIL PROTECTED] --- fs/gfs2/locking/dlm/mount.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git

[Cluster-devel] Re: [GFS2] Fix runtime issue with UP kernels

2007-11-16 Thread David Miller
From: Steven Whitehouse [EMAIL PROTECTED] Date: Fri, 16 Nov 2007 10:37:43 + Ah, all becomes clear :-) Thanks for chasing down this issue, the patch is in my git tree now. I guess I must have had some other option turned on when I did my UP build that caused this not to happen, Spinlock

[Cluster-devel] [PATCH] gfs2: tidy up error message

2007-11-16 Thread David Teigland
Print error with log_error() to be consistent with others. Signed-off-by: David Teigland [EMAIL PROTECTED] --- fs/gfs2/locking/dlm/mount.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/fs/gfs2/locking/dlm/mount.c b/fs/gfs2/locking/dlm/mount.c index ab30102..f2efff4

[Cluster-devel] small inconsistency presenting fs stats between GFS1 and GFS2

2007-11-16 Thread Fabio Massimo Di Nitto
Hi guys, this is purely cosmetic and I didn't prepare a patch but see this: given a 4GB block device (just as an example): /dev/nbd2 3,9G 518M 3,4G 14% /mnt/gfs2 /dev/nbd1 3,1G 20K 3,1G 1% /mnt/gfs you can see that gfs1 masks the device size to hide the

Re: [Cluster-devel] small inconsistency presenting fs stats between GFS1 and GFS2

2007-11-16 Thread Wendy Cheng
Fabio Massimo Di Nitto wrote: Hi guys, this is purely cosmetic and I didn't prepare a patch but see this: given a 4GB block device (just as an example): /dev/nbd2 3,9G 518M 3,4G 14% /mnt/gfs2 /dev/nbd1 3,1G 20K 3,1G 1% /mnt/gfs you can see that gfs1 masks the

[Cluster-devel] cluster/gfs2/fsck metawalk.c pass1.c pass1c.c ...

2007-11-16 Thread rpeterso
CVSROOT:/cvs/cluster Module name:cluster Branch: RHEL5 Changes by: [EMAIL PROTECTED] 2007-11-16 23:22:23 Modified files: gfs2/fsck : metawalk.c pass1.c pass1c.c pass2.c pass5.c Log message: Resolves: bz 382581: GFS2: gfs2_fsck: buffer still