From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [v10][PATCH 06/16] hvmloader/pci: Try to avoid placing BARs in RMRRs Date: Tue, 21 Jul 2015 10:42:09 +0100 Message-ID: <55AE13F1.3090406@citrix.com> References: <1437373023-14884-1-git-send-email-tiejun.chen@intel.com> <1437373023-14884-7-git-send-email-tiejun.chen@intel.com> <55ACF7EF020000780009305B@mail.emea.novell.com> <55ACEEFB.1050707@citrix.com> <55AD004F.7060905@intel.com> <55AD0155.1020108@citrix.com> <55AD1EEB0200007800093246@mail.emea.novell.com> <55AD0690.9000801@intel.com> <55AD291B0200007800093427@mail.emea.novell.com> <55AD9805.60803@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <55AD9805.60803@intel.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Chen, Tiejun" , Jan Beulich Cc: George Dunlap , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 07/21/2015 01:53 AM, Chen, Tiejun wrote: >>> Okay, I regenerate this patch online. And I just hope its good to be >>> acked here: >> >> Provided it also works, >> Reviewed-by: Jan Beulich >> > > Why is this marked as Acked-by this time? The same question is raised to > another hvmloader patch as well. > > This really makes me confused since you're the key maintainer associated > to this, and I remember you also gave me Acked-by to the first hvmloader > patch. I know this solution is always argued, so does this mean you > still don't think this is good to go in the tree in your perspective, so > you want to leave this Acked-by to other maintainers, right? > > And what about patch #7, hvmloader/e820: construct guest e820 table, is > this also not fine to you? Acked-by means basically, "I have no objections to this going in." For a patch to be committed it needs at least one maintainer to Ack it, and (in general) no one else who objects. (A maintainer can override someone's objection.) But it doesn't mean that Jan has done a thorough review, or even that he's read it (he may Ack it just based on other people's Reviewed-by's). Reviewed-by means that he *has* done a thorough technical review, and he thinks it's ready to go in. In particular, Reviewed-by means "While there may be things that could be improved with this submission, I believe that it is, at this time, (1) a worthwhile modification, and (2) free of known issues which would argue against its inclusion." [1] So Reviewed-by is stronger than Acked-by. -George [1] Section 13, https://www.kernel.org/doc/Documentation/SubmittingPatches -George