Date: Tue, 26 Feb 2019 19:27:41 -0800 From: Cy Schubert <Cy.Schubert@cschubert.com> To: Cy Schubert <Cy.Schubert@cschubert.com> Cc: Fatih Acar <fatih.acar@gandi.net>, Baptiste Daroussin <bapt@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org, jack@gandi.net, fatih@gandi.net Subject: Re: svn commit: r344569 - in head/cddl/contrib/opensolaris: cmd/zfs lib/libzfs/common Message-ID: <201902270327.x1R3RfXn062807@slippy.cwsent.com> In-Reply-To: Message from Cy Schubert <Cy.Schubert@cschubert.com> of "Tue, 26 Feb 2019 13:02:53 -0800." <201902262102.x1QL2rIo032528@slippy.cwsent.com>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <201902262102.x1QL2rIo032528@slippy.cwsent.com>, Cy Schubert writes: > In message <a69cf582-abc7-aff9-41c7-ed2a9e13e388@gandi.net>, Fatih Acar > writes: > > This is a multi-part message in MIME format. > > --------------6A54CE5B28D823DCB8C41577 > > Content-Type: text/plain; charset=utf-8 > > Content-Transfer-Encoding: 8bit > > > > On 2/26/19 5:36 PM, Cy Schubert wrote: > > > On February 26, 2019 8:11:31 AM PST, Baptiste Daroussin <bapt@FreeBSD.org > > > > wrote: > > >> On Tue, Feb 26, 2019 at 05:04:11PM +0100, Baptiste Daroussin wrote: > > >>> On Tue, Feb 26, 2019 at 07:48:27AM -0800, Cy Schubert wrote: > > >>>> On February 26, 2019 12:18:35 AM PST, Baptiste Daroussin > > >> <bapt@FreeBSD.org> wrote: > > >>>>> Author: bapt > > >>>>> Date: Tue Feb 26 08:18:34 2019 > > >>>>> New Revision: 344569 > > >>>>> URL: https://svnweb.freebsd.org/changeset/base/344569 > > >>>>> > > >>>>> Log: > > >>>>> Implement parallel mounting for ZFS filesystem > > >>>>> > > >>>>> It was first implemented on Illumos and then ported to ZoL. > > >>>>> This patch is a port to FreeBSD of the ZoL version. > > >>>>> This patch also includes a fix for a race condition that was > > >> amended > > >>>>> > > >>>>> With such patch Delphix has seen a huge decrease in latency of the > > >>>>> mount phase > > >>>>> (https://github.com/openzfs/openzfs/commit/a3f0e2b569 for > > >> details). > > >>>>> With that current change Gandi has measured improvments that are > > >> on par > > >>>>> with > > >>>>> those reported by Delphix. > > >>>>> > > >>>>> Zol commits incorporated: > > >>>> > > >>> https://github.com/zfsonlinux/zfs/commit/a10d50f999511d304f910852c7825c > 70 > > c9c9e303 > > >>>> > > >>> https://github.com/zfsonlinux/zfs/commit/e63ac16d25fbe991a356489c86d407 > 75 > > 67dfea21 > > >>>>> > > >>>>> Reviewed by: avg, sef > > >>>>> Approved by: avg, sef > > >>>>> Obtained from: ZoL > > >>>>> MFC after: 1 month > > >>>>> Relnotes: yes > > >>>>> Sponsored by: Gandi.net > > >>>>> Differential Revision: https://reviews.freebsd.org/D19098 > > >>>>> > > >>>>> Modified: > > >>>>> head/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c > > >>>>> head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs.h > > >>>>> head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_dataset.c > > >>>>> head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_impl.h > > >>>>> head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_mount.c > > >>>>> > > >>>>> Modified: head/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c > > >>>> > > >>> ======================================================================= > == > > ===== > [...] > > >>>>> > > >>>> > > >>>> This broke my systems, many filesystems fail to mount causing > > >> nullfs late mounts to fail. No details now until tonight. > > >>>> > > >>>> Suggest we back this out until it is properly tested. > > >>>> > > >>> > > >>> What fails to mount? what message? can you provide Gandi folks more > > >> informations > > >>> so they can fix? > > >>> > > >>> I will revert if we cannot have a quick fix but let s give them a > > >> chance to fix > > >>> first. > > >>> > > >> With the proper email in CC there is a better chance to reach at them > > >> :) > > >> > > >> Best regards, > > >> Bapt > > > > > > Sorry about that. I'm terribly frustrated as this broke my mail gateway, > ha > > ving to fix it using juiceSSH on my phone on the bus. Ssh on the phone make > s > > for a very grumpy Cy. > > > > > > I did bring my personal laptop to work, so I'll try to help out testing t > hi > > s at noon here and maybe look at it more. I'll help out any way I can. > > > > > > > > > > Sorry about all this... > > Could you try the attached patch, it should fix the issue. I don't > > understand how this regression happened, it's not present in ZoL... > > I'll check with Jack who worked on this when he's back from PTO. > > > > Thanks. > > > > -- > > Fatih ACAR > > Gandi > > fatih.acar@gandi.net > > > > --------------6A54CE5B28D823DCB8C41577 > > Content-Type: text/x-patch; > > name="mount.patch" > > Content-Transfer-Encoding: quoted-printable > > Content-Disposition: attachment; > > filename="mount.patch" > > > > Index: cddl/contrib/opensolaris/lib/libzfs/common/libzfs_mount.c > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > = > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > = > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > --- cddl/contrib/opensolaris/lib/libzfs/common/libzfs_mount.c (revisi > on 3= > > 44590) > > +++ cddl/contrib/opensolaris/lib/libzfs/common/libzfs_mount.c (workin > g co= > > py) > > @@ -1260,11 +1260,11 @@ > > if (*a =3D=3D '\0') > > return (-1); > > if (*b =3D=3D '\0') > > - return (-1); > > + return (1); > > if (*a =3D=3D '/') > > return (-1); > > if (*b =3D=3D '/') > > - return (-1); > > + return (1); > > return (*a < *b ? -1 : *a > *b); > > } > > =20 > > > > Thanks. This fixes this particular issue. Should we consider committing this? -- Cheers, Cy Schubert <Cy.Schubert@cschubert.com> FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201902270327.x1R3RfXn062807>