Date: Mon, 11 Feb 2019 21:07:26 -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: <CAD44qMWE8711oTE%2BjCw24_ciAA7cBKrB0Md6oTy5tenOQNX=gg@mail.gmail.com> In-Reply-To: <CAD44qMVVMNOs3%2BKTf2AWEsbhdRTnykMOVTc4VOTJ6%2Bsd4k_PpQ@mail.gmail.com> References: <201902120026.x1C0Q1FW073337@pdx.rh.CN85.dnsmgr.net> <62d2dcc1-5bde-1eda-6d9f-82138932cb36@FreeBSD.org> <CAD44qMVVMNOs3%2BKTf2AWEsbhdRTnykMOVTc4VOTJ6%2Bsd4k_PpQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Feb 11, 2019 at 8:13 PM Patrick Kelsey <pkelsey@freebsd.org> wrote: > > > 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. > > John, Which is to say, I second your motion to proceed with normal process. As one would reasonably expect, this driver didn't fall out of the sky yesterday. It was completed on 15 November 2018 and has been undergoing multi-party testing since then. I doubt it's bug free (nor was the last one), but when bugs have turned up in the driver or in iflib at any point along the way, convergence to root cause and a fix has been same-day, so I don't think there's anything that walks like an emergency or quacks like an emergency related to this commit. -Patrick
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAD44qMWE8711oTE%2BjCw24_ciAA7cBKrB0Md6oTy5tenOQNX=gg>