From owner-svn-src-all@freebsd.org Fri Mar 9 16:09:49 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 772ACF351D2; Fri, 9 Mar 2018 16:09:49 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 1FB537E6C7; Fri, 9 Mar 2018 16:09:48 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id w29G9kCY065168; Fri, 9 Mar 2018 08:09:46 -0800 (PST) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id w29G9kI1065167; Fri, 9 Mar 2018 08:09:46 -0800 (PST) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201803091609.w29G9kI1065167@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r330451 - in stable/11/sys: dev/iwm dev/otus dev/usb/wlan net80211 In-Reply-To: To: Kyle Evans Date: Fri, 9 Mar 2018 08:09:46 -0800 (PST) CC: "Rodney W. Grimes" , svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers , svn-src-stable-11@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 09 Mar 2018 16:09:49 -0000 > On Fri, Mar 9, 2018 at 9:28 AM, Rodney W. Grimes > wrote: > >> On Wed, Mar 07, 2018 at 02:53:49PM -0800, Eitan Adler wrote: > >> > On 7 March 2018 at 09:37, John Baldwin wrote: > >> > > ... > >> > > I suspect many of these changes for iwm, etc. are all intertwined > >> > > so I'm not sure if you can leave out individual ones. > >> > > >> > Possibly. I do have iwm working on my laptop though. I also know of > >> > one open PR assigned to me w.r.t. a model I don't own. I'll be > >> > addressing it some time this week. > >> > >> I often have mixed feelings when I see lots of similar changes (i.e. > >> that make up for better hardware support, esp. on a laptop) MFCed. > >> I'd rather see laptop users run -CURRENT and leave -STABLE branches > >> for very conservative (server?) users who can't/don't want to afford > >> the risks of running -CURRENT or require ABI stability in a really > >> long run, rather than binge-merging things. :-) > >> > >> By default it should be -CURRENT all over; it's a very good thing > >> that we as a Project ourselves are doing this as part of our own > >> dogfood eating strategy. > > > > As a data point just last night a person came into #freebsd irc > > channel with a none working wireless nic on a desktop, he was > > attemtping to use a Realteak RTL8192EU on 11.1-RELEASE. > > > > Someone had already told him to try urtwn driver, which in > > -current is the right driver and supports this device. > > > > It did not work for him. This is about when I came in to the > > discussion, and helped to confirm that -current did infact > > have support for this device, and that this supported had > > existed in -current for 16 months, and that this support > > would not be a simple grab a couple driver files and build > > it on 11.1. > > > > The commit to add this support involves 46 files, which 18 > > of are new files. > > > > My feelings are if this driver has been in -current for > > 16 months why is it NOT in -stable yet? I know part of > > the answer "its not a simple merge". But as a project > > this is egg on our face. We can merge a new very complicated > > change to a our boot code, within a few months (thank you > > kevans for that work), but we can not merge back a > > device driver in 16? > > > > I had the same questions- this exact same person had hopped over to > #freebsd-wifi and I had walked through this same process, identifying > it as "not MFC-able" at this point because so many commits having been > left un-merged prior to it. I've already recently gone through the fun > of catching up on one and a half years worth of unmerged work in > stand, I'm not really prepared to do it again quite yet. But but but.. you did it so well the first time!!! :-) I can fully appreciate that you do not desire to do another massive merge. > It felt pretty bad having to tell him that his only option here was to > either hop on -CURRENT + rtwn(4) or grab a stable/11 supported NIC- > especially since stable/11 is still supposed to be supported for > another three (3!) years, and I had to leave before I could help him > walk through getting it setup on -CURRENT properly. When I left the #freebsd channel he was downloading, and others there are capable of helping him get up and running. One thing that did come up while discussing some of the issues with merging head to stable was it might be usefull if we add yet another marker line: Changes ABI/KABI: yes to the commit messages, at least if marked you can be pretty sure that merging is going to involve additional work, lack of this mark would not guarantee you didnt have an ABI issue as they are easy to overlook and you would still need to look for those types of problems. I have been told by someone there are some tools that can mechanically look for ABI changes. -- Rod Grimes rgrimes@freebsd.org