Date: Mon, 11 Feb 2019 20:13:16 -0500 From: Patrick Kelsey <pkelsey@freebsd.org> To: John Baldwin <jhb@freebsd.org> Cc: rgrimes@freebsd.org, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org Subject: Re: svn commit: r344027 - in stable/12/sys: dev/vmware/vmxnet3 modules/vmware/vmxnet3 net Message-ID: <CAD44qMVVMNOs3%2BKTf2AWEsbhdRTnykMOVTc4VOTJ6%2Bsd4k_PpQ@mail.gmail.com> In-Reply-To: <62d2dcc1-5bde-1eda-6d9f-82138932cb36@FreeBSD.org> References: <201902120026.x1C0Q1FW073337@pdx.rh.CN85.dnsmgr.net> <62d2dcc1-5bde-1eda-6d9f-82138932cb36@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Feb 11, 2019 at 8:08 PM John Baldwin <jhb@freebsd.org> wrote: > On 2/11/19 4:26 PM, Rodney W. Grimes wrote: > >> Author: pkelsey > >> Date: Mon Feb 11 23:24:39 2019 > >> New Revision: 344027 > >> URL: https://svnweb.freebsd.org/changeset/base/344027 > >> > >> Log: > >> MFC r343291: > >> Convert vmx(4) to being an iflib driver. > > > > I strongly object to this MFC, given the current number > > of 12.0 RELEASE related iflib problems we have it is > > foolish of us to iflib any more drivers in 12.0 > > This isn't the release branch though and presumably we have some time > before > 12.1 ships. If there are reports of vmx(4) breakage on stable before 12.1 > we could always revert this commit then? > > I've heard of some EN's for 12.0 for iflib fixes. Are those fixes in > stable/12 > yet or are we still waiting for them to land in HEAD and/or be merged? > iflib.c is currently the same between head and stable/12. I've found and fixed a number of iflib bugs by developing the iflib version of the vmx(4) driver, and it's also being fielded in a product. I'm also aware that not all current driver problems are necessarily iflib problems. I think we'd be better off letting this version of vmx(4) ride it out in stable/12 until such time as we discover an actual horror that we then feel we need to react to in some way other than just going ahead and fixing it. -Patrick
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAD44qMVVMNOs3%2BKTf2AWEsbhdRTnykMOVTc4VOTJ6%2Bsd4k_PpQ>