From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=209.132.183.28; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 4A66F211E82E0 for ; Mon, 25 Mar 2019 11:33:27 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 6A9F688AA9; Mon, 25 Mar 2019 18:33:26 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-188.rdu2.redhat.com [10.10.120.188]) by smtp.corp.redhat.com (Postfix) with ESMTP id E8F595F9CD; Mon, 25 Mar 2019 18:33:24 +0000 (UTC) To: "Kinney, Michael D" , "Wu, Hao A" , "edk2-devel@lists.01.org" Cc: "Justen, Jordan L" , Ard Biesheuvel , "Ni, Ray" References: <20190325052853.11220-1-hao.a.wu@intel.com> <20190325052853.11220-3-hao.a.wu@intel.com> From: Laszlo Ersek Message-ID: <2a2efa72-9d30-dad5-d97f-bcb8f4abe837@redhat.com> Date: Mon, 25 Mar 2019 19:33:18 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.26]); Mon, 25 Mar 2019 18:33:26 +0000 (UTC) Subject: Re: [PATCH v2 2/3] OvmfPkg: Add an Super IO bus driver X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 25 Mar 2019 18:33:28 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 03/25/19 18:30, Kinney, Michael D wrote: > Hi Laszlo, > > I do not think content added before April 9, 2019 > should use the new license type. We need to let the > 30-day review period complete and make sure all feedback > is resolved. Good point. > We will handle files added between the edk2-stable201903 > and April 9, 2019 in a final patch series with an easy > way for all maintainers to see what has changed between > those two points. Hm. From the reviewer side, this is not optimal. The patch set (and the individual patches themselves) are pretty big, and doing incremental reviews on them is taxing. Regardless of whether the incremental review needs to target an updated "full" patch set, or just an incremental patch set (for new files), the reviewer needs to re-evaluate whether something is now missed, after the introduction of new files. Instead, I'd prefer a "lock" period for OvmfPkg and ArmVirtPkg, between (a) my next (hopefully, final) review for the license conversion patches, and (b) the pushing of those patches. For that, I see two options: - We could delay Hao's work (and all other patches that add files to OvmfPkg and ArmVirtPkg files) until after April 9. We can of course collaborate on feature / bugfix patches meanwhile, it's just that the final versions of *those* should be reposted with updated license blocks. Incrementally reviewing *those* changes feels a lot easier to me. - Alternatively, I could delay my next (hopefully, final) review of the license conversion patches until reasonably close to April 9, until which "review point" new files could be added freely, to OvmfPkg and ArmVirtPkg. (This wouldn't eliminate the "lock period", just make it shorter for contributors.) IOW, this is similar to the stabilization period / feature freezes, just much more intrusive, because everything has to be switched at the same moment. I'd like to reach an understanding on our approach before I start reviewing "[edk2] [PATCH V2] Change EDK II to BSD+Patent License". Thanks Laszlo