[Cluster-devel] [PATCH] GFS2: kernel changes to support new gfs2_grow command (Try 3)

[Robert Peterson <rpeterso@redhat.com>]

David Teigland wrote:
 > So the use of glock_is_held _is_ part of an assertion, not part of an
> algorithm which I was worried about before.  We should only ever get to
> this spot with a shared glock, right? (rindex_hold takes it).  So a plain
> old assertion that the glock is shared at the beginning would be ok, but
> this particular check doesn't make sense to me.

See previous email.
 
> Why is this needed now if it wasn't before?

Again, this goes back to the partial rinedx entries left on a page in
the middle of an rindex update through the meta_fs.  We don't want
to read in a partial entry, we want to ignore it.

Regards,

Bob Peterson
Red Hat Cluster Suite