* [Cluster-devel] [GFS2 Patch] fsck.gfs2 reported statfs error after gfs2_grow
@ 2010-12-07 18:58 Bob Peterson
2010-12-07 19:20 ` Steven Whitehouse
0 siblings, 1 reply; 2+ messages in thread
From: Bob Peterson @ 2010-12-07 18:58 UTC (permalink / raw)
To: cluster-devel.redhat.com
Hi,
When you do gfs2_grow it failed to take the very last
rgrp into account when adding up the new free space due
to an off-by-one error. It was not reading the last
rgrp from the rindex because of a check for "<=" that
should have been "<". Therefore, fsck.gfs2 was finding
(and fixing) an error with the system statfs file.
Regards,
Bob Peterson
Red Hat GFS
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
--
fs/gfs2/rgrp.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c
index 25dbe5c..7293ea2 100644
--- a/fs/gfs2/rgrp.c
+++ b/fs/gfs2/rgrp.c
@@ -500,7 +500,7 @@ u64 gfs2_ri_total(struct gfs2_sbd *sdp)
for (rgrps = 0;; rgrps++) {
loff_t pos = rgrps * sizeof(struct gfs2_rindex);
- if (pos + sizeof(struct gfs2_rindex) >= i_size_read(inode))
+ if (pos + sizeof(struct gfs2_rindex) > i_size_read(inode))
break;
error = gfs2_internal_read(ip, &ra_state, buf, &pos,
sizeof(struct gfs2_rindex));
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Cluster-devel] [GFS2 Patch] fsck.gfs2 reported statfs error after gfs2_grow
2010-12-07 18:58 [Cluster-devel] [GFS2 Patch] fsck.gfs2 reported statfs error after gfs2_grow Bob Peterson
@ 2010-12-07 19:20 ` Steven Whitehouse
0 siblings, 0 replies; 2+ messages in thread
From: Steven Whitehouse @ 2010-12-07 19:20 UTC (permalink / raw)
To: cluster-devel.redhat.com
Hi,
Now in the -nmw git tree. Thanks,
Steve.
On Tue, 2010-12-07 at 13:58 -0500, Bob Peterson wrote:
> Hi,
>
> When you do gfs2_grow it failed to take the very last
> rgrp into account when adding up the new free space due
> to an off-by-one error. It was not reading the last
> rgrp from the rindex because of a check for "<=" that
> should have been "<". Therefore, fsck.gfs2 was finding
> (and fixing) an error with the system statfs file.
>
> Regards,
>
> Bob Peterson
> Red Hat GFS
>
> Signed-off-by: Bob Peterson <rpeterso@redhat.com>
> --
> fs/gfs2/rgrp.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c
> index 25dbe5c..7293ea2 100644
> --- a/fs/gfs2/rgrp.c
> +++ b/fs/gfs2/rgrp.c
> @@ -500,7 +500,7 @@ u64 gfs2_ri_total(struct gfs2_sbd *sdp)
> for (rgrps = 0;; rgrps++) {
> loff_t pos = rgrps * sizeof(struct gfs2_rindex);
>
> - if (pos + sizeof(struct gfs2_rindex) >= i_size_read(inode))
> + if (pos + sizeof(struct gfs2_rindex) > i_size_read(inode))
> break;
> error = gfs2_internal_read(ip, &ra_state, buf, &pos,
> sizeof(struct gfs2_rindex));
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-12-07 19:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-12-07 18:58 [Cluster-devel] [GFS2 Patch] fsck.gfs2 reported statfs error after gfs2_grow Bob Peterson
2010-12-07 19:20 ` Steven Whitehouse
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.