Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 22 Aug 2018 07:42:53 -0700
From:      Chuck Tuffli <chuck@freebsd.org>
To:        lists@eitanadler.com
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r338182 - in head: sbin/nvmecontrol sys/cam/nvme sys/dev/mpr sys/dev/nvme sys/sys usr.sbin/bhyve
Message-ID:  <CAKAYmMJE2eR34worTS8=-eTs=MgTptdGYBbX_X6xCqQpgSzrZg@mail.gmail.com>
In-Reply-To: <CAF6rxgmz0DDBDnkEtVpX=qnvGg27E=E_9P1D9B%2BgR0UE%2BU9kjg@mail.gmail.com>
References:  <201808220429.w7M4TOsT095655@repo.freebsd.org> <CAF6rxgmz0DDBDnkEtVpX=qnvGg27E=E_9P1D9B%2BgR0UE%2BU9kjg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Aug 22, 2018 at 6:10 AM Eitan Adler <lists@eitanadler.com> wrote:
>
> On Tue, 21 Aug 2018 at 21:29, Chuck Tuffli <chuck@freebsd.org> wrote:
> >
> > Author: chuck
> > Date: Wed Aug 22 04:29:24 2018
> > New Revision: 338182
> > URL: https://svnweb.freebsd.org/changeset/base/338182
> >
> > Log:
> >   Make NVMe compatible with the original API
>
> This breaks sysutils/smartmontools. Would you be willing to patch the
> port and/or submitting a diff upstream?

Technically, the original big-endian changes broke smartmontools, but
I understand your point and am happy to provide a patch. Since I don't
have a sense for how much backwards compatibility they want, I've
contacted the maintainer to give them a heads up that this change
happened and will see where they want to go.

--chuck



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAKAYmMJE2eR34worTS8=-eTs=MgTptdGYBbX_X6xCqQpgSzrZg>