From owner-svn-src-head@freebsd.org Fri Mar 15 17:35:13 2019 Return-Path: Delivered-To: svn-src-head@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 06F12152804D; Fri, 15 Mar 2019 17:35:13 +0000 (UTC) (envelope-from araujobsdport@gmail.com) Received: from mail-lf1-x12f.google.com (mail-lf1-x12f.google.com [IPv6:2a00:1450:4864:20::12f]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 642D072F0F; Fri, 15 Mar 2019 17:35:12 +0000 (UTC) (envelope-from araujobsdport@gmail.com) Received: by mail-lf1-x12f.google.com with SMTP id g7so7458222lfh.10; Fri, 15 Mar 2019 10:35:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:reply-to:from:date:message-id :subject:to:cc; bh=5W92w+e/fa7L0oeaGNIsuyV+h/SuNYawDf9nFbwDKcs=; b=TuABeWgDiR3+xaQ+TWt+oXac2LKgZoY8Kz0gu311ecxPcAHYh2GLNJtL8MWL14uMwC yys8Z5EK6sZIP3we5XP0GXpumB46NOZYcULF3oYPxEa0PpER2OnmZrsmJTiEOelZ51FQ rIvY5/vWAVpVf4bfN4H7V4bmA2jYjMC5MZeAKw51vPJqHah81GYa9xiJY+hF6Y/iCWfJ rWpx1BPlehHJrYZzQdDSqvWgqudE1+MkEC0GFA26Q03xhvjBKOifUKbdk6uhc30XrreE N7eTwcj2nTc0Fx1kn9pj/gos5ioi3jqN41Xh7hxqBnMyp7sRexGb/69YTXNw5uk6zG+q wjGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:reply-to :from:date:message-id:subject:to:cc; bh=5W92w+e/fa7L0oeaGNIsuyV+h/SuNYawDf9nFbwDKcs=; b=WpCYP7Mp9h2t52310Ca6duicxBSMDKDSDcoPtlCyZ9jV6z1T1OA+bkQIwhgFycxZ7N urhPUNFXHdtSvNa1cxGInbhKU5KT7ltefIReHtAYAZM1vvWEaGtO5zkXCPG2x32awf4F iYEAw76D+vl5wMXyESnQqHXL7bzdGW385dKEunP8dd01Dlg6Uemkzay3QonzAU5/XOgH uQISWSCH+EdOxVZZdLZPbQgUQEIcEXbfan790bLiqOogYtZwgkPv4wYMtV+QsVeiUydG ezxerp63NgU9M1c9wCUU6Twsrobq5AoqVnA981DvTndufuwTib6rdti4nZY+hvPCDivN Yo+A== X-Gm-Message-State: APjAAAXrbY/XvJ2Jjv2aWVE9w7b1UylEqU9Nd4WHTqNNfD68FJF7eFJL iP+ge9MPEjoN+K8XQBsuKuts28eINFO317wLlIDfjiga X-Google-Smtp-Source: APXvYqywX/x6UCzJfY7um0SgevBEIUyERBrl5z/8EFrKm96PkZ+WChKzg/bi4vJvbYdCCPm/05s1ZgFUaZr97KWYwBI= X-Received: by 2002:a19:ed19:: with SMTP id y25mr2998675lfy.80.1552671310571; Fri, 15 Mar 2019 10:35:10 -0700 (PDT) MIME-Version: 1.0 References: <201903151727.x2FHR4ls031294@gndrsh.dnsmgr.net> In-Reply-To: <201903151727.x2FHR4ls031294@gndrsh.dnsmgr.net> Reply-To: araujo@freebsd.org From: Marcelo Araujo Date: Sat, 16 Mar 2019 01:34:58 +0800 Message-ID: Subject: Re: svn commit: r345171 - head/usr.sbin/bhyve To: "Rodney W. Grimes" Cc: Warner Losh , Chuck Tuffli , src-committers , svn-src-all , svn-src-head X-Rspamd-Queue-Id: 642D072F0F X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.97 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_SHORT(-0.97)[-0.967,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0] Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Mar 2019 17:35:13 -0000 Em s=C3=A1b, 16 de mar de 2019 =C3=A0s 01:27, Rodney W. Grimes < freebsd@gndrsh.dnsmgr.net> escreveu: > > On Fri, Mar 15, 2019 at 9:56 AM Rodney W. Grimes < > freebsd@gndrsh.dnsmgr.net> > > wrote: > > > > > > On Thu, Mar 14, 2019 at 8:32 PM Rodney W. Grimes < > > > freebsd@gndrsh.dnsmgr.net> > > > > 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-Base= d > > > 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. > > > > > > > > > > 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. > > > > > > > > > > > > > There is no such thing as a hard lock in FreeBSD. This sounds like > you > > > are > > > > advocating for that, but that's not the case. > > > > > > > > Stop this stupid nitpicking for single line commits. We don't have > that > ^^^^^^ > > Thank you for calling my actions stupid, in effect demoralizing me with > the label that includes. I may nit pick, but I never call people degradi= ng > names on a public list. > > Also it only takes a single like to make a bug or problem, > it would help to not consider single line changes any less or > any more important or potentially damaging. > > > > > culture any more and it's really pissing a lot of people off. > > > > > > > > The MAINTAINERS file even says this: > > > > > > > > Please note that the content of this file is strictly advisory. > > > > > > > > And the entry for bhyve doesn't say things are mandatory, just > requested. > > > > > > > > Jumping on people's case like this, for a review you yourself were > on and > > > > approved but made no mention of seeking further review / approval, = is > > > > demotivating and toxic. Please stop. > > > > > > I explicitly DID add jhb to the review. > > > I also explicitly did not mark the bhyve# box that is added by > > > the hearald rules. > > > > > > I did not jump on him, I informed him of the entry, and told him to > leave > > > it. > > > You how ever have infact jumped on me, repeatedly, if you want to tal= k > > > about > > > discouraging tones of behavior I suggest you look at yourself as well= . > > > > > > > Weird that so many other people in the thread read it the same way that= I > > did, and not only this time. That suggests that you're not aware that > your > > behavior is annoying others, which I contend is a problem that needs > > looking into. > > If you have a pre conceived notion or opinion about anything I say, > and I contend that many do, you well always hear it in that tone. This i= s > the rose colored glasses problem. I can not fix that what you hear is > not what I said. > > I speak frankly and without political or other polish to my words, > which at times do make them sound harsh or overly direct. I think > we both actually do that, and, imho, that is better than trying to > sugar coat stuff and be all polite and indirect about things. > > > As always, I'm open to constructive, actionable feedback about my > actions. > > I have tried above. > > > Warner > > > > Warner > > > > > head/usr.sbin/bhyve/pci_emul.c > > > > > > > > > > > > Modified: head/usr.sbin/bhyve/pci_emul.c > > > > > > > > > > > > > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > > > > > > --- head/usr.sbin/bhyve/pci_emul.c Fri Mar 15 02:11:27 2019 > > > > > (r345170) > > > > > > +++ head/usr.sbin/bhyve/pci_emul.c Fri Mar 15 02:11:28 2019 > > > > > (r345171) > > > > > > @@ -953,7 +953,10 @@ pci_emul_add_pciecap(struct pci_devinst > *pi, int > > > > > type) > > > > > > bzero(&pciecap, sizeof(pciecap)); > > > > > > > > > > > > pciecap.capid =3D PCIY_EXPRESS; > > > > > > - pciecap.pcie_capabilities =3D PCIECAP_VERSION | > > > PCIEM_TYPE_ROOT_PORT; > > > > > > + pciecap.pcie_capabilities =3D PCIECAP_VERSION | type; > > > > > > + /* Devices starting with version 1.1 must set the RBER bi= t > */ > > > > > > + if (PCIECAP_VERSION >=3D 1) > > > > > > + pciecap.dev_capabilities =3D PCIEM_CAP_ROLE_ERR_R= PT; > > > > > > pciecap.link_capabilities =3D 0x411; /* gen1, x1 */ > > > > > > pciecap.link_status =3D 0x11; /* gen1, x1 */ > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > Rod Grimes > > > > > rgrimes@freebsd.org > > > > > > > > > > > > > > > > -- > > > Rod Grimes > > > rgrimes@freebsd.org > > > > > -- > Rod Grimes > rgrimes@freebsd.org > > Rod, again, I don't really want to be rude!!! But have you tried to write a blog or something like that with the ideas that you have instead to write an email and press send? Sometimes we are eager to reply every each email, but if we wait for a while that desire disappears and we will realize if we send that email or not would not change anything. I'm not trying to be sarcastic here, my point is, somebody commit something and someone else replies in public is because there is something really wrong and needs everybody attention. Maybe if you really think there is something wrong, first reply in private to that person and then escalete to public if the matters were not solved. Best, -- --=20 Marcelo Araujo (__)araujo@FreeBSD.org \\\'',)http://www.FreeBSD.org \/ \ ^ Power To Server. .\. /_)