Date: Sun, 3 Jan 2016 10:30:16 -0800 From: NGie Cooper <yaneurabeya@gmail.com> To: Mariusz Zaborski <oshogbo@FreeBSD.org> Cc: FreeBSD-Stable ML <freebsd-stable@freebsd.org>, Marcel Moolenaar <marcel@xcllnt.net> Subject: Re: [HEADS UP] "options ZFS" broken on stable/10; please use zfs(4) module for the time being Message-ID: <67AA1D00-44B5-4F45-8699-C9BEA5713D17@gmail.com> In-Reply-To: <CAGOYWV9f1YziMqFzit4yyhvDje7z18S5V1eUU5vvdEReMW5m%2BQ@mail.gmail.com> References: <AF4A00EA-0E83-43D3-B7F3-2296DBEA66DC@gmail.com> <CAGOYWV9f1YziMqFzit4yyhvDje7z18S5V1eUU5vvdEReMW5m%2BQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Jan 3, 2016, at 06:42, Mariusz Zaborski <oshogbo@FreeBSD.org> = wrote: >=20 > Hi, >=20 > Did you merge the changes made in = ./sys/cddl/compat/opensolaris/sys/nvpair.h and = https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D201356 ? Those twos = should repair the problem. >=20 > Thanks for doing all the work around merging it. Hi Mariusz! I figured out the missing revisions to get things working again = on stable/10 (the revisions I listed below were the sticking point to = getting this working 100%). There are a lot of changes around pci(9) / = nv(9) that I=E2=80=99ll be putting out for review soon, so the backport = of pci_iov(9) and nv(9) can be completed (I need to do some more testing = soon and some careful inspection to ensure KBI guarantees and the like = are maintained). All in all I=E2=80=99m hoping that this can be fixed =E2=80=9Csoon= =E2=80=9D (sometime in the next week or so). Thanks! -NGie r293093 | ngie | 2016-01-03 01:11:55 -0800 (Sun, 03 Jan 2016) | 21 lines MFC r279437,r284107: r279437 (by rstone): Allow Illumos code to co-exist with nv(9) Suggested by: pjd r284107 (by avg): compat nvpair.h: make sure that the names are mangled only for kernel Currently there is no good reason to mangle the userland API. The change was introduced in eac1d566b46edef765754203bef22c75c1699966, r279437. Also see https://reviews.freebsd.org/D1881. I am still convinced that nv should not have introduced intentionally conflicting API. Discussed with: rstone=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?67AA1D00-44B5-4F45-8699-C9BEA5713D17>