From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.groups.io with SMTP id smtpd.web10.53381.1673526254985414504 for ; Thu, 12 Jan 2023 04:24:15 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=BxpA6PRb; spf=pass (domain: redhat.com, ip: 170.10.129.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673526253; 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=qZlUsO1xaXygRaGhQXYIuzdu4lmuQnHeP5WAeu+FjyE=; b=BxpA6PRbe3e0R212I92OnqxxNdluDN/X2oQKcjUf+/sSpIFzp7NZ7s0Xm8cMH6VsGDBYcb vHVoj8tYZJzGMlUJaDWN4SLYkcrKweKSxMDagypq4TG5c0R0264vRPfkuzki1Rd9io9PjL O2E6jvw9WE49ARGhJ7obtFXLpfkYfgQ= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-331-iGbgl69_MOq0RUk_G3utoA-1; Thu, 12 Jan 2023 07:24:12 -0500 X-MC-Unique: iGbgl69_MOq0RUk_G3utoA-1 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id F1610811E9C; Thu, 12 Jan 2023 12:24:11 +0000 (UTC) Received: from [10.39.192.93] (unknown [10.39.192.93]) by smtp.corp.redhat.com (Postfix) with ESMTPS id E61F2492C14; Thu, 12 Jan 2023 12:24:09 +0000 (UTC) Message-ID: <75ed5959-fbb2-3343-e7d7-dfa17b4a2615@redhat.com> Date: Thu, 12 Jan 2023 13:24:08 +0100 MIME-Version: 1.0 Subject: Re: [edk2-devel] [PATCH 3/3] MdeModulePkg: Notify BeforeExitBootServices in CoreExitBootServices To: dionnaglaze@google.com Cc: Gerd Hoffmann , James Bottomley , "Yao, Jiewen" , Tom Lendacky , Ard Biesheuvel , "Xu, Min M" , Andrew Fish , "Ni, Ray" , Moritz Fischer , edk2-devel-groups-io , "Kinney, Michael D" References: <20221108164616.3251967-1-dionnaglaze@google.com> <20221108164616.3251967-4-dionnaglaze@google.com> From: "Laszlo Ersek" In-Reply-To: X-Scanned-By: MIMEDefang 3.1 on 10.11.54.9 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 1/12/23 00:08, Dionna Glaze via groups.io wrote: > On Thu, Nov 10, 2022 at 8:55 AM Kinney, Michael D > wrote: >> >> Hi Dionna, >> >> https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process >> >> I think you are at step 13. If you have not done so already, please update the >> commit messages with all the Reviewed-by and Acked-by tags and make sure your >> branch is rebased against the latest edk2/master and update the PR with that >> content and verify that all EDK II CI checks pass. >> >> At that point, it is the responsibility of the EDK II Maintainers to review >> the final state of the PR and set the "push" label if everything looks correct. >> >> At this moment, we are in Soft Freeze and will be in Hard Freeze for the next >> 2 weeks. If this is considered critical for the stable tag release, then >> please send a request to Liming with justification for stable tag. Otherwise, >> it will be merged after the stable tag release. >> >> https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning >> >> Thanks, >> >> Mike >> > > Hi Mike, my PR was closed without comment > https://github.com/tianocore/edk2/pull/3623 so I rebased and created a > new PR after the holidays and hard freeze. I hope this isn't > considered bad practice https://github.com/tianocore/edk2/pull/3885 > I closed the PR -- similarly to how I manually closed 400+ stale PRs then -- because it was not a maintainer-submitted PR with the "push" label set. In other words, it was just a personal build, for triggering a CI run. And, in fact regardlessly of whether it was a "push"-labeled maintainer PR or just a personal PR, the PR was almost two months old. Clearly stale and abandoned -- per edk2 contribution workflow, anyway. Please excuse me for not explaining this in a comment on the PR, but (as I said above) I closed 400+ stale PRs manually within 10-15 minutes on the github.com WebUI. The necessary muscle memory training didn't allow for individual comments. (I actually tried scripting the mass-closure at first, with the "gh" utility, but something in the github.com data model was broken, and the server wouldn't allow me to close those PRs with the utility, even though I had authenticated the utility under my account, with a complete set of scopes -- see my report at .) Regarding your new PR: it's again good for a personal CI run only. If it completes fine, please post the patches to the mailing list, using git-send-email. (From browsing recent list traffic, it seems that your colleague Moritz Fischer has posted patches like that to the list; please consider consulting with Moritz regarding the git setup (SMTP server etc) withing Google. Cc'ing Moritz now.) When sending the patches like that, please CC the maintainers and reviewers that are supposed to review them. Once they are happy with the series, one of them will submit a PR with them, and set the "push" label on the PR. When the CI run succeeds, the mergify bot will merge *that* PR automatically. Laszlo