[Cluster-devel] [GFS2 PATCH] gfs2: truncate glock address space pages during evict

[Bob Peterson]

----- Original Message -----
> Bob,
> 
> 
> not sure why, but this patch breaks xfstests generic/311 and generic/467.

Hm, that's odd. Can you tell me how it misbehaves?
fwiw, both 311 and 467 work for me, no hangs, panics, etc.

> Also,
> don't we want to initialize metamapping earlier and also call
> truncate_inode_pages_final on it before we detach the glock?

Yes, this is a great concern now you mention it.
I've created a 5-patch set that refactors function gfs2_evict_inode
to actually make it readable. I'll test and post that next week.
 
> (Mind that truncate_inode_pages_final leaves AS_EXITING set in
> metamapping->flags, so we'll also need to do somthing about that.)

Yes, that is another concern. I'll wait and see if Johannes W. responds
to your query and we'll go from there.

Regards,

Bob Peterson