Re: [linux-lvm] reiserfs panic

[Andreas Dilger <adilger@turbolabs.com>]

On Nov 30, 2001  07:44 -0500, Ed Tomlinson wrote:
> Wonder if this is the result of something not getting into cvs?  Mid October
> Chris fixed a bunch of snapshot problems.  One was with full snapshots...
> He posted a fix here, some one else posted an alternate.  Bet neither has 
> been put into cvs.  Here is what I am using with 'rc4ish'.  Does it help?

Hmm, I thought I had put this into CVS.  Note that my version put the
checks into lvm_snapshot_remap_block() rather than requiring the caller
to check it.  It looks like it is there in my CVS.

> @@ -1142,7 +1142,8 @@
>  
>  	/* we must redo lvm_snapshot_remap_block in order to avoid a
>  	   race condition in the gap where no lock was held */
> -	if (!lvm_snapshot_remap_block(&rdev, &rsector, pe_start, lv) &&
> +	if (lv->lv_block_exception && 
> +	    !lvm_snapshot_remap_block(&rdev, &rsector, pe_start, lv) &&
>  	    !lvm_snapshot_COW(rdev, rsector, pe_start, rsector, vg, lv))
>  		lvm_write_COW_table_block(vg, lv);

Cheers, Andreas
--
Andreas Dilger
http://sourceforge.net/projects/ext2resize/
http://www-mddsp.enel.ucalgary.ca/People/adilger/