this happens because umount calls into statfs() which in turn waits for MDS2 stopped by the test:
stop mds2 || error "stop mds2 failed"
umount $MOUNT

statfs() is supposed to be "lazy" as per Andreas's comment in skype, but it is not for connections to MDSs.