From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [205.139.110.61]) by mx.groups.io with SMTP id smtpd.web10.132185.1598008017637152693 for ; Fri, 21 Aug 2020 04:06:57 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=YO13cbT/; spf=pass (domain: redhat.com, ip: 205.139.110.61, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1598008016; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=JeRet3OxVGw7aMb2pXn3GCysDiHtDUX9USWrspAxI8M=; b=YO13cbT//3oLXNybrAcqF5Xkw8/hmM+3z8oiOyvtPfXQ+lbUQ96hyBf9oCvFoVH3U7CsEs j0+5PRe9y0QMJ6dFnQaOtiXEObbIMptM0jbbAftmnZDXJ2WtijW2KWj0JTTE6FlHT44j+h UYsWYiKx6lbojbsinxWbhMhXnYXB/TU= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-400-6EC4VeKgO5SKB7YHu_Hs0g-1; Fri, 21 Aug 2020 07:06:52 -0400 X-MC-Unique: 6EC4VeKgO5SKB7YHu_Hs0g-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id B1BDE807340; Fri, 21 Aug 2020 11:06:50 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-113-164.ams2.redhat.com [10.36.113.164]) by smtp.corp.redhat.com (Postfix) with ESMTP id 8C0D219C78; Fri, 21 Aug 2020 11:06:48 +0000 (UTC) Subject: Re: Patch List for 202008 stable tag To: "Gao, Liming" , Leif Lindholm , "afish@apple.com" , "Kinney, Michael D" , "Guptha, Soumya K" , "Chang, Abner (HPS SW/FW Technologist)" , Vladimir Olovyannikov , Tom Lendacky Cc: "devel@edk2.groups.io" References: From: "Laszlo Ersek" Message-ID: <55f8dfee-ef2e-b04a-f1cb-4fef1931a7c3@redhat.com> Date: Fri, 21 Aug 2020 13:06:47 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0.002 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit On 08/20/20 13:30, Gao, Liming wrote: > Hi Stewards and all: > I collect current patch lists in devel mail list. Those patch contributors request to add them for 202008 stable tag. Because we have enter into Soft Feature Freeze, I want to collect your feedback for them. If any patches are missing, please reply this mail to add them. > > Feature List: > https://edk2.groups.io/g/devel/message/63767 [PATCH] EmbeddedPkg/libfdt: Add strncmp macro to use AsciiStrnCmp > [Liming] This patch pass code review after Soft Feature Freeze (SFF) starts. According to SFF definition, it should not be merged for this stable tag. But, the patch submitter says this patch is important to RISC-V community. To catch it for this stable tag, Laszlo proposed the solution to defer SFF start day from 2020-08-14 to 2020-08-21, then hard feature freeze and release date will also defer one week. Any patches those pass review before new SFF start day can be merged. @ Stewards, please give your comments to defer this stable tag release by one week. Thank you, very nice summary. As stated earlier, I'm OK with 1 week (or even two weeks, if needed) extension. I invite Andrew, Leif and Mike to comment. > > https://edk2.groups.io/g/devel/message/63348 [PATCH v5 1/1] ShellPkg/DynamicCommand: add HttpDynamicCommand > [Liming] This patch has collected the review comment. New version will be sent. I have no information how important it is. @Vladimir, does this patch must catch this stable tag? If yes, can you give the reason? This is a feature addition. While it's a super useful feature, it needs to mature (mainly from the coding style perspective, as I gather). Posting of new versions and review are on-going. Most likely material for the next stable tag. In case we extend the deadlines, and Maciej and the ShellPkg maintainers approve one of the upcoming versions until the new SFF, then yes, we can merge that too. > Bug List: > https://edk2.groups.io/g/devel/message/64383 [PATCH 1/1] UefiCpuPkg/MpInitLib: Always initialize the DoDecrement variable > [Liming] This patch has collected the review comment. New version will be sent. This is a bugfix. Certainly a build fix on CLANGPDB. I tried to do some (lightweight) analysis to see whether the bug is a functional one as well (i.e. whether the bad code path that CLANG warns about can actually occur in practice), but in the end I didn't want to spend much time on it, as the build breakage needs to be fixed anyway. So, this patch (more precisely, v2 of this patch) would even qualify during the hard feature freeze (as it's clearly a bugfix). > > https://edk2.groups.io/g/devel/message/50406 [PATCH 1/1] MdePkg/Include: Add missing definitions of SMBIOS type 42h in SmBios.h > [Liming] This patch passed review early. But, it is not merged. I will merge it. Right, it was approved on 2019-Nov-18. If it still applies cleanly, it should be merged, before we enter the Hard Feature Freeze. Well... now that I'm writing this, we have already entered the HFF (as planned originally) and the patch doesn't seem to be upstream yet. So unless we extend, I don't think this patch qualifies. (On the other hand -- it seems like we really should extend.) Thanks! Laszlo