Date: Sat, 9 Jul 2016 21:51:50 -0700 From: "K. Macy" <kmacy@freebsd.org> To: "bugzilla-noreply@freebsd.org" <bugzilla-noreply@freebsd.org> Cc: "freebsd-fs@freebsd.org" <freebsd-fs@freebsd.org> Subject: Re: [Bug 203864] ZFS deadlock between zfs send, zfs rename and ctrl-C Message-ID: <CAHM0Q_OrwrvtWdhUGNi1Dpjuy1eV=x==RQzSm8C8BwY8gnTXvg@mail.gmail.com> In-Reply-To: <bug-203864-3630-juE3XsyRUC@https.bugs.freebsd.org/bugzilla/> References: <bug-203864-3630@https.bugs.freebsd.org/bugzilla/> <bug-203864-3630-juE3XsyRUC@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
On Saturday, July 9, 2016, <bugzilla-noreply@freebsd.org> wrote: > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=203864 > > --- Comment #17 from Andriy Gapon <avg@FreeBSD.org> --- > Another observation is that the current code seems to be broken in a > couple of > places as well. > > 1. dsl_dataset_rename_snapshot_sync_impl() uses ddrsa_fsname to construct > the > old and new snapshot names, but this would be wrong in the case of a > recursive > snpashot rename (zfs rename -r x@snap1 x@snap2) as ddrsa_fsname always > points > to the name of the top level filesystem. > > 2. dsl_dataset_promote_sync(): as far as I can see, oldname is never > populated, > so the code can do arbitrary renames. Have you tried enabling opensolaris witness? > > -- > You are receiving this mail because: > You are the assignee for the bug. > _______________________________________________ > freebsd-fs@freebsd.org <javascript:;> mailing list > https://lists.freebsd.org/mailman/listinfo/freebsd-fs > To unsubscribe, send any mail to "freebsd-fs-unsubscribe@freebsd.org > <javascript:;>" >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAHM0Q_OrwrvtWdhUGNi1Dpjuy1eV=x==RQzSm8C8BwY8gnTXvg>