From owner-svn-src-all@freebsd.org Fri Mar 15 16:18:15 2019 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 A005215255BD; Fri, 15 Mar 2019 16:18:15 +0000 (UTC) (envelope-from pfg@FreeBSD.org) Received: from smtp.freebsd.org (smtp.freebsd.org [IPv6:2610:1c1:1:606c::24b:4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 44EE46EE55; Fri, 15 Mar 2019 16:18:15 +0000 (UTC) (envelope-from pfg@FreeBSD.org) Received: from [192.168.0.2] (unknown [181.52.72.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client did not present a certificate) (Authenticated sender: pfg) by smtp.freebsd.org (Postfix) with ESMTPSA id 142D71D4D1; Fri, 15 Mar 2019 16:18:13 +0000 (UTC) (envelope-from pfg@FreeBSD.org) Subject: Re: svn commit: r345171 - head/usr.sbin/bhyve To: rgrimes@freebsd.org, araujo@freebsd.org Cc: Ian Lepore , Chuck Tuffli , src-committers , svn-src-all , svn-src-head References: <201903151603.x2FG3pIp030819@gndrsh.dnsmgr.net> From: Pedro Giffuni Organization: FreeBSD Message-ID: <6f9466bd-a282-e329-dd7c-13aef8deadaf@FreeBSD.org> Date: Fri, 15 Mar 2019 11:18:12 -0500 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.5.3 MIME-Version: 1.0 In-Reply-To: <201903151603.x2FG3pIp030819@gndrsh.dnsmgr.net> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US X-Rspamd-Queue-Id: 44EE46EE55 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.94 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; REPLY(-4.00)[]; NEURAL_HAM_SHORT(-0.94)[-0.944,0] X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 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, 15 Mar 2019 16:18:15 -0000 On 15/03/2019 11:03, Rodney W. Grimes wrote: >> Em sex, 15 de mar de 2019 ?s 22:12, Ian Lepore escreveu: >> >>> On Thu, 2019-03-14 at 19:31 -0700, Rodney W. Grimes wrote: >>>>> Author: chuck >>>>> Date: Fri Mar 15 02:11:28 2019 >>>>> New Revision: 345171 >>>>> URL: https://svnweb.freebsd.org/changeset/base/345171 >>>>> >>>>> Log: >>>>> Fix bhyve PCIe capability emulation >>>>> >>>>> PCIe devices starting with version 1.1 must set the Role-Based >>>>> Error >>>>> Reporting bit. >>>>> >>>>> And while we're in the neighborhood, generalize the code >>>>> assigning the >>>>> device type. >>>>> >>>>> Reviewed by: imp, araujo, rgrimes >>>>> Approved by: imp (mentor) >>>>> MFC after: 1 week >>>>> Differential Revision: https://reviews.freebsd.org/D19580 >>>> This code requires maintainer approval before a commit, >>>> though this was well reviewed that doesnt exclude it >>>> from the MAINTAINERS entry. >>>> >>> Where exactly does it say that in MAINTAINERS? As another victim of >>> this sort of drive-by lynching after making a trivial bhyve change I >>> pretty seriously object to a vague and meaningless entry in MAINTAINERS >>> being used to pounce on anyone who dares to touch the precious bhyve >>> code. >>> >> There is a new entry on MAINTAINERS: >> https://svnweb.freebsd.org/base?view=revision&revision=344631 >> >> >>> There is no mention of bhyve in MAINTAINERS, for usr.sbin or elsewhere. >>> There is an entry for vmm(4), which to me does not say anything about >>> bhyve, yet somehow everybody is supposed to know what it means and >>> what-all territory it covers? >>> >>> IMO, this sort of hyper-proprietary pouncing on everyone who dares >>> change a single line of code is not productive. It is HIGHLY de- >>> motivating. Large sweeping design changes are one thing, but pouncing >>> on every tiny minor commit is just not helpful. >>> >> +1 >> >> I got so frustrated with it recently that I have decided to don't >> contribute with bhyve anymore, perhaps even with FreeBSD. >> I still have some people under mentorship that I intend to finish and then >> probably I will phase out. > Your failure to get reviews, and infact even abandon one that had > negative advice as to the validity of your suggested change and > committing it anyway is more likely the cause here. I will have to add  to the choir here: getting reviews is not mandatory and I hope they will never be. Reviewed code is likely to have less errors but sometimes we just need to get things done. Look, for example, the case of the automounting daemon amd(8), which we have been planning to remove for some years: your unfortunate intervention stopped it from getting removed from the system for how many months(?) stopping progress there altogether. Pedro. > You also committed code with no review at all that had to be reverted > after the bugs it caused were found by an external down stream consumers > of the bhyve code. > > You had code reverted by core due to a external attribution request, > which had you been attending the bi monthly bhyve calls you would of > known was an issue. > > I would suggest these are the reasons your feeling angry, and that > I infact tried to reach out to jhb to discuss some of these earlier > but that reach out was never returned. I under stand your frustration, > you are just wanting to do with best thing you can for the project > and bhyve, can we try to find a better resolution to this situation > than your exit? > >>> -- Ian >>> >>>> Leave it for now, I am sure jhb or thyco are fine with it, >>>> this is just a heads up FYI for future commits. >>>> >>>> Bhyve code has been and still is under a fairly tight >>>> MAINTAINER status. >>>> >>>>> Modified: >>>>> head/usr.sbin/bhyve/pci_emul.c >>>>> >>>>> Modified: head/usr.sbin/bhyve/pci_emul.c >>>>> =================================================================== >>>>> =========== >>>>> --- head/usr.sbin/bhyve/pci_emul.c Fri Mar 15 02:11:27 2019 (r3 >>>>> 45170) >>>>> +++ head/usr.sbin/bhyve/pci_emul.c Fri Mar 15 02:11:28 2019 (r3 >>>>> 45171) >>>>> @@ -953,7 +953,10 @@ pci_emul_add_pciecap(struct pci_devinst *pi, >>>>> int type) >>>>> bzero(&pciecap, sizeof(pciecap)); >>>>> >>>>> pciecap.capid = PCIY_EXPRESS; >>>>> - pciecap.pcie_capabilities = PCIECAP_VERSION | >>>>> PCIEM_TYPE_ROOT_PORT; >>>>> + pciecap.pcie_capabilities = PCIECAP_VERSION | type; >>>>> + /* Devices starting with version 1.1 must set the RBER bit */ >>>>> + if (PCIECAP_VERSION >= 1) >>>>> + pciecap.dev_capabilities = PCIEM_CAP_ROLE_ERR_RPT; >>>>> pciecap.link_capabilities = 0x411; /* gen1, x1 */ >>>>> pciecap.link_status = 0x11; /* gen1, x1 */ >>>>> >>>>> >>>>> >>>> >>> >>> >> -- >> >> -- >> Marcelo Araujo (__)araujo@FreeBSD.org >> \\\'',)http://www.FreeBSD.org \/ \ ^ >> Power To Server. .\. /_)