> On Jan 7, 2016, at 6:01 PM, Dan McDonald <[email protected]> wrote:
> 
> 
> Here's the webrev of just the code change:
> 
>        http://kebe.com/~danmcd/webrevs/son-of-4986/
> 
> And the comments in the one changed file mostly explain things from this 
> thread.  The affected function, dsl_dataset_clone_swap_check_impl(), is ONLY 
> called by dmu_recv_end_check.  Because it's checking for refquota, we can 
> insert the slack mentioned previously in the mail thread into the check.  I 
> have some long comments to explain things -- I'd appreciate those being 
> scrutinized carefully as well.

I've updated the above webrev to include:

        - ZFS Test changes for this situation.

        - An additional fix to have an ASSERT()ion not trip.

I'm running the ZFS Test Suite as I type this, just to make sure.  The new 
zfs_receive_012 appears to work.  I need to go test stock as well to get a 
baseline.

Appreciate your reviews!
Dan
_______________________________________________
developer mailing list
[email protected]
http://lists.open-zfs.org/mailman/listinfo/developer

Reply via email to