Re: [edk2-devel] MOV64 and CLANGPDB

2025-02-14 Thread Michael Kubacki via groups.io
Thanks https://github.com/tianocore/edk2/pull/10760 is updated. I don't have a strong need for it to make the stable tag. I have a local solution that can be used in the meantime. On 2/14/2025 5:48 AM, Leif Lindholm wrote: On Fri, 14 Feb 2025 at 01:54, Michael Kubacki wrote: That w

Re: [edk2-devel] MOV64 and CLANGPDB

2025-02-13 Thread Michael Kubacki via groups.io
That worked. Let me know what you'd like to do with https://github.com/tianocore/edk2/pull/10760. --- Before (original code) ...ModuleEntryPoint.:58:12: error: immediate must be an integer in range [0, 65535]. movz x9, (-1) >> 48, lsl #48 ; movk x9, ((-1) >> 32) & 0x, lsl #32 ; mov

Re: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg failure

2024-12-18 Thread Michael Kubacki via groups.io
e new names. Understanding this is tangential to a change dropping IA32 VS EmulatorPkg in the pipeline but that could be considered as well and I could add that if desired. Thanks, Michael On 12/18/2024 3:44 PM, Michael Kubacki wrote: Hi Mike, I've also had trouble reproducing it co

Re: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg failure

2024-12-18 Thread Michael Kubacki via groups.io
Mike -Original Message- From: devel@edk2.groups.io On Behalf Of Michael Kubacki via groups.io Sent: Wednesday, December 18, 2024 8:48 AM To: devel@edk2.groups.io; o...@linux.microsoft.com; leif.lindh...@oss.qualcomm.com; Kinney, Michael D Cc: Oliver Smith-Denny Subject: Re: [edk2-devel] CI blo

Re: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg failure

2024-12-18 Thread Michael Kubacki via groups.io
t have permission to do so, it has to be done by an admin, Mike Kinney or Michael Kubacki I believe will do it. The plan is to also change the name of the GCC5 pipeline (Gerd requested this) and instead have more generically named pipelines so when they are updated in the future we don't have th

Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow

2024-08-19 Thread Michael Kubacki
For simplicity of testing on the fork, I'd just create a PAT in your repo with the necessary permissions and use that as the secret there. Then, substitute it with the app secret in the edk2-platforms PR. You can test almost everything that way except the app token generation itself. But, that

Re: 回复: [edk2-devel] [RFC] Transitioning from Bugzilla to GitHub Issues

2024-08-16 Thread Michael Kubacki
On 8/14/2024 11:49 PM, Michael D Kinney wrote: -Original Message- From: Michael Kubacki Sent: Wednesday, August 14, 2024 7:30 PM To: devel@edk2.groups.io; gaolim...@byosoft.com.cn; Kinney, Michael D ; r...@edk2.groups.io Cc: 'Andrew Fish' ; 'Leif Lindholm' ; '

Re: 回复: [edk2-devel] [RFC] Transitioning from Bugzilla to GitHub Issues

2024-08-14 Thread Michael Kubacki
Hi Liming, I apologize for the delayed response. Thanks, Michael On 8/6/2024 1:34 AM, gaoliming via groups.io wrote: Michael: Thanks for your great effort. This is a good direction to consolidate code and issue together. My feedbacks are below: 1. BugZilla supports code first process. Wha

Re: [edk2-devel] [RFC] Transitioning from Bugzilla to GitHub Issues

2024-07-31 Thread Michael Kubacki
I made updates to the RFC on the GitHub Discussion page with rev tracking. https://github.com/tianocore/edk2/discussions/5926 On 7/31/2024 1:56 PM, Kinney, Michael D wrote: -Original Message- From: Michael Kubacki Sent: Wednesday, July 31, 2024 7:42 AM To: devel@edk2.groups.io

Re: [edk2-devel] [RFC] Transitioning from Bugzilla to GitHub Issues

2024-07-31 Thread Michael Kubacki
Proposed Process 1. Create GitHub issue templates for the following categories that include basic automation of validation for input. * Bug Report * Fields: * Title * Packages Affected Would this be free form text or a multi select list of possib

[edk2-devel] Quick Change to Improve edk2 CI Time

2024-07-26 Thread Michael Kubacki
TLDR: Pipelines should be relatively faster soon --- As you know, PR gates can take a long time in edk2. While there are various improvements that could be made, I want to highlight a change in the queue now, it's impact, and how it will help. Pipelines are composed of jobs which are compose

[edk2-devel] CodeQL Failures in PR Checks

2024-07-26 Thread Michael Kubacki
TLDR: If CodeQL failed in a PR. Rebase your PR branch on master and push. --- edk2 uses an application called the CodeQL CLI to run CodeQL in CI builds. https://github.com/github/codeql-cli-binaries/ A fixed version of the application is used to allow controlled updates where the version is t

Re: [edk2-devel] edk2 Automatic PR Reviewers

2024-07-24 Thread Michael Kubacki
It looks like the permissions will need adjusted. That will come in a follow up change. On 7/24/2024 10:35 PM, Michael Kubacki wrote: A change was just pushed to edk2 to automatically add reviewers to pull request based on those listed in Maintainers.txt. For more details and in case of any

[edk2-devel] edk2 Automatic PR Reviewers

2024-07-24 Thread Michael Kubacki
A change was just pushed to edk2 to automatically add reviewers to pull request based on those listed in Maintainers.txt. For more details and in case of any issues, I put some information on this wiki page: https://github.com/tianocore/tianocore.github.io/wiki/Automatic-Pull-Request-Reviewer

[edk2-devel] [RFC] Transitioning from Bugzilla to GitHub Issues

2024-07-19 Thread Michael Kubacki
Hello, A topic over the last few weeks in the TianoCore Tools & CI meeting has been establishing a transition from Bugzilla to GitHub issues. I've drafted the following RFC to help explain the rationale and a proposed process to do so. I think it is easier to read with formatting, so I've pl

Re: [edk2-devel] [edk2-platforms][PATCH v1 1/2] IntelSiliconPkg: Compiler and linker fixes

2024-07-18 Thread Michael Kubacki
, July 17, 2024 10:55 AM To: devel@edk2.groups.io Cc: Ni, Ray ; Chaganty, Rangasai V ; S, Ashraf Ali ; Lautner, Kenneth Subject: [edk2-platforms][PATCH v1 1/2] IntelSiliconPkg: Compiler and linker fixes From: Michael Kubacki Some code is currently not being built in the package. This fixes GCC

[edk2-devel] [edk2-platforms][PATCH v2 2/2] IntelSiliconPkg: Add missing components to build

2024-07-18 Thread Michael Kubacki
From: Michael Kubacki Adds libraries and modules missing from the [Components] section of IntelSiliconPkg.dsc. Cc: Ray Ni Cc: Rangasai V Chaganty Cc: Ashraf Ali S Signed-off-by: Michael Kubacki Reviewed-by: S, Ashraf Ali --- Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc | 9

[edk2-devel] [edk2-platforms][PATCH v2 1/2] IntelSiliconPkg: Compiler and linker fixes

2024-07-18 Thread Michael Kubacki
From: Michael Kubacki Some code is currently not being built in the package. This fixes GCC compiler errors and MSVC linker failures in the code so the package can build with these modules included. Cc: Ray Ni Cc: Rangasai V Chaganty Cc: Ashraf Ali S Co-authored-by: Ken Lautner Signed-off

[edk2-devel] [edk2-platforms][PATCH v2 0/2] IntelSiliconPkg: Add missing components

2024-07-18 Thread Michael Kubacki
From: Michael Kubacki Some libraries and modules are missing from the build. The first patch fixes compiler and linker failures. The second patch adds the missing components to IntelSiliconPkg.dsc. A PR with these changes is available here: https://github.com/tianocore/edk2-platforms/pull/171

[edk2-devel] [edk2-platforms][PATCH v1 2/2] IntelSiliconPkg: Add missing components to build

2024-07-17 Thread Michael Kubacki
From: Michael Kubacki Adds libraries and modules missing from the [Components] section of IntelSiliconPkg.dsc. Cc: Ray Ni Cc: Rangasai V Chaganty Cc: Ashraf Ali S Signed-off-by: Michael Kubacki --- Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc | 9 + 1 file changed, 9

[edk2-devel] [edk2-platforms][PATCH v1 1/2] IntelSiliconPkg: Compiler and linker fixes

2024-07-17 Thread Michael Kubacki
From: Michael Kubacki Some code is currently not being built in the package. This fixes GCC compiler errors and MSVC linker failures in the code so the package can build with these modules included. Cc: Ray Ni Cc: Rangasai V Chaganty Cc: Ashraf Ali S Co-authored-by: Ken Lautner Signed-off

[edk2-devel] [edk2-platforms][PATCH v1 0/2] IntelSiliconPkg: Add missing components

2024-07-17 Thread Michael Kubacki
From: Michael Kubacki Some libraries and modules are missing from the build. The first patch fixes compiler and linker failures. The second patch adds the missing components to IntelSiliconPkg.dsc. A PR with these changes is available here: https://github.com/tianocore/edk2-platforms/pull/171

Re: [edk2-devel] github impact:breaking-change

2024-07-09 Thread Michael Kubacki
The tweaks look good to me. Thanks, Michael On 7/9/2024 11:03 AM, Leif Lindholm wrote: Hi Michael, On Tue, Jul 09, 2024 at 10:44:46 -0400, Michael Kubacki wrote: Hi Leif, Thanks for raising this. The label was intended to be set from the PR template which has an explanation: https

Re: [edk2-devel] github impact:breaking-change

2024-07-09 Thread Michael Kubacki
Hi Leif, Thanks for raising this. The label was intended to be set from the PR template which has an explanation: https://github.com/tianocore/edk2/blob/master/.github/pull_request_template.md If the breaking change box is checked, it will be added automatically. I added a description in Git

Re: [edk2-devel] Topic for Monday Tools, CI, Code Base construction meeting

2024-06-24 Thread Michael Kubacki
Hi Rebecca, I'm covering hosting for the meeting today. I've created a discussion with your agenda topics here https://github.com/tianocore/edk2/discussions/5818. Thanks, Michael On 6/23/2024 1:39 PM, Rebecca Cran wrote: Add edk2-devel On 6/23/24 11:38 AM, Rebecca Cran wrote: I have two i

Re: [edk2-devel] [PATCH v1 1/1] CodeQL: Update from 2.16.1 to 2.17.3

2024-06-03 Thread Michael Kubacki
t the latest (incompatible) queries. https://github.com/tianocore/edk2/pull/5720 On 5/20/2024 9:40 PM, Michael Kubacki wrote: Just wanted to send a reminder that I recommend https://github.com/tianocore/edk2/pull/5667 be merged to unblock CI for those that are currently trying to test changes. T

Re: [edk2-devel] GitHub PR Code Review process now active

2024-05-30 Thread Michael Kubacki
@edk2.groups.io On Behalf Of Michael Kubacki Sent: Wednesday, May 29, 2024 5:51 PM To: Kinney, Michael D ; devel@edk2.groups.io; Rebecca Cran ; kra...@redhat.com; Kubacki, Michael Subject: Re: [edk2-devel] GitHub PR Code Review process now active I've updated the wiki process to clarify that mainta

Re: [edk2-devel] GitHub PR Code Review process now active

2024-05-29 Thread Michael Kubacki
he list of future enhancements when resources are available to help. We need to focus on a process that works until those automations can be deployed and supported. Mike -Original Message----- From: Michael Kubacki Sent: Wednesday, May 29, 2024 12:25 PM To: devel@edk2.groups.io; Kinney,

Re: [edk2-devel] libspdm Breaking Builds

2024-05-29 Thread Michael Kubacki
Since we already reached agreement in libspdm to move its cmocka submodule to the gitlab mirror in https://github.com/DMTF/libspdm/issues/2707 and that was merged in https://github.com/tianocore/edk2/pull/5697. I'd like to suggest we go with option 2 as well. This is impacting our ability to

Re: [edk2-devel] GitHub PR Code Review process now active

2024-05-29 Thread Michael Kubacki
Mike, I agree that automatically adding reviewers would be helpful. Do you think we could add a CODEOWNERS file now to assist with this? Benefits being: 1. CODEOWNERS is low overhead in that GitHub already supports it. 2. We do not need to require CODEOWNER enforcement yet. We can simply use

Re: [edk2-devel] [edk2-rfc] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-24 Thread Michael Kubacki
roject for this at https://github.com/orgs/tianocore/projects/5 which it looks like Michael Kubacki created last year. I suspect it's obsolete though, since it looks like the tasks haven't been updated recently. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent

Re: [edk2-devel] libspdm Breaking Builds

2024-05-22 Thread Michael Kubacki
We did not want to clone openssl twice. I though stuart config specifies which submodules to clone. Can’t we skip all the submodules within libspdm to fix CI? Can't devs choose to not use --recursive? Mike -Original Message----- From: Michael Kubacki Sent: Wednesday, May 22, 202

Re: [edk2-devel] libspdm Breaking Builds

2024-05-22 Thread Michael Kubacki
n libspdm. Mike -Original Message----- From: Michael Kubacki Sent: Wednesday, May 22, 2024 1:40 PM To: devel@edk2.groups.io; Yao, Jiewen Cc: Kinney, Michael D Subject: libspdm Breaking Builds Recently, the libspdm submodule was added to SecurityPkg (SecurityPkg/DeviceSecurity/SpdmLib/l

[edk2-devel] libspdm Breaking Builds

2024-05-22 Thread Michael Kubacki
Recently, the libspdm submodule was added to SecurityPkg (SecurityPkg/DeviceSecurity/SpdmLib/libspdm) [1]. That submodule points to https://github.com/DMTF/libspdm.git. The libspdm repo contains a submodule for cmocka that is hosted on cryptomilk.org [2]. Cloning that submodule is failing to

Re: [edk2-devel] [PATCH v1 1/1] CodeQL: Update from 2.16.1 to 2.17.3

2024-05-20 Thread Michael Kubacki
-Original Message- From: Michael Kubacki Sent: Friday, May 17, 2024 4:28 PM To: Kinney, Michael D ; devel@edk2.groups.io; Andrew Fish ; Leif Lindholm ; Liming Gao Cc: Feng, Bob C ; Joey Vagedes ; Rebecca Cran ; Sean Brogan ; Chen, Christine Subject: Re: [PATCH v1 1/1] CodeQL: Update from 2.16.1

[edk2-devel] PR Process Wiki Updates

2024-05-20 Thread Michael Kubacki
In preparation for the transition to a pull request (PR) contribution process, I've updated the TianoCore wiki. - Current official TianoCore wiki: https://github.com/tianocore/tianocore.github.io/wiki/ - My fork with proposed changes: https://github.com/makubacki/tianocore.github.io/wiki -

[edk2-devel] [PATCH v3 1/1] MdeModulePkg: Add the EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE attribute

2024-05-17 Thread Michael Kubacki
From: Patrick Payne Add the EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE resource attribute as per the PI 1.8 spec. This flag is used to indicate that the memory should be treated as special purpose memory (SPM). Cc: Liming Gao Cc: Michael D Kinney Signed-off-by: Michael Kubacki --- Notes: V3

Re: [edk2-devel] [PATCH v1 1/1] CodeQL: Update from 2.16.1 to 2.17.3

2024-05-17 Thread Michael Kubacki
notes, I will follow up in the future for a change that should be able to lock the CodeQL query versions against the CLI version. Thanks, Michael On 5/17/2024 7:20 PM, Michael Kubacki wrote: To ensure CI is unblocked, I am going to add the push tag now. On 5/17/2024 6:31 PM, Michael Kubacki

Re: [edk2-devel] [PATCH v1 1/1] CodeQL: Update from 2.16.1 to 2.17.3

2024-05-17 Thread Michael Kubacki
To ensure CI is unblocked, I am going to add the push tag now. On 5/17/2024 6:31 PM, Michael Kubacki wrote: Thanks Mike. Are you okay with me completing the PR now? On 5/17/2024 5:31 PM, Kinney, Michael D wrote: Reviewed-by: Michael D Kinney -Original Message- From: mikub

Re: [edk2-devel] [PATCH v1 1/1] CodeQL: Update from 2.16.1 to 2.17.3

2024-05-17 Thread Michael Kubacki
; Liming Gao ; Kinney, Michael D ; Rebecca Cran ; Sean Brogan ; Chen, Christine Subject: [PATCH v1 1/1] CodeQL: Update from 2.16.1 to 2.17.3 From: Michael Kubacki This fixes an issue where the CodeQL queries currently fetched in the pipeline are incompatible with the current executable used. Update

[edk2-devel] [PATCH v1 1/1] CodeQL: Update from 2.16.1 to 2.17.3

2024-05-17 Thread Michael Kubacki
From: Michael Kubacki This fixes an issue where the CodeQL queries currently fetched in the pipeline are incompatible with the current executable used. Update to pick up functional and security fixes. See the following comparison for detailed differences: https://github.com/github/codeql-cli

[edk2-devel] [PATCH v2 1/1] MdePkg: Add the EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE attribute

2024-05-17 Thread Michael Kubacki
From: Patrick Payne Add the EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE resource attribute as per the PI 1.8 spec. This flag is used to indicate that the memory should be treated as special purpose memory (SPM). Cc: Liming Gao Cc: Michael D Kinney Signed-off-by: Michael Kubacki --- Notes: V2

[edk2-devel] [PATCH v1 1/1] Add the EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE attribute.

2024-05-16 Thread Michael Kubacki
From: Patrick Payne Add the EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE resource attribute as per the PI 1.8 spec. This flag is used to indicate that the memory should be treated as special purpose memory (SPM). Also adds GCD code that marks the SPM with the EFI_MEMORY_SP bit and sets the GCD memory

Re: [edk2-rfc] [edk2-devel] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-03 Thread Michael Kubacki
On 5/3/2024 4:38 PM, Michael D Kinney wrote: -Original Message- From: Kinney, Michael D Sent: Friday, May 3, 2024 1:13 PM To: Pedro Falcato Cc: r...@edk2.groups.io; devel@edk2.groups.io; Leif Lindholm ; Andrew Fish (af...@apple.com) ; Kinney, Michael D Subject: RE: [edk2-rfc] [edk2-

Re: [edk2-devel] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-03 Thread Michael Kubacki
Depends on: https://github.com/mdkinney/github-action-check-codeowners-maintainers Thanks, Mike -Original Message- From: Michael Kubacki Sent: Thursday, May 2, 2024 8:21 AM To: devel@edk2.groups.io; quic_llind...@quicinc.com; marcin.juszkiew...@linaro.org; Kinney, Michael D ; r.

Re: [edk2-devel] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-03 Thread Michael Kubacki
] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024 On 2024-05-02 04:08, Michael Kubacki wrote: Thank you for this proposal. We've been anticipating this change for years and are excited to help support it. Here's some items we'd like to raise for f

Re: [edk2-devel] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-03 Thread Michael Kubacki
On 5/2/2024 6:57 AM, Leif Lindholm wrote: On 2024-05-02 04:08, Michael Kubacki wrote: Thank you for this proposal. We've been anticipating this change for years and are excited to help support it. Here's some items we'd like to raise for feedback that we could help implem

Re: [edk2-devel] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-02 Thread Michael Kubacki
On 5/2/2024 6:34 AM, Leif Lindholm wrote: On 2024-05-02 07:33, Marcin Juszkiewicz wrote: W dniu 1.05.2024 o 19:43, Michael D Kinney via groups.io pisze: I would like to propose that TianoCore move all code review from email based code reviews to GitHub Pull Requests based code reviews. The pro

Re: [edk2-devel] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-02 Thread Michael Kubacki
On 5/2/2024 5:37 AM, Ard Biesheuvel wrote: On Wed, 1 May 2024 at 19:44, Michael D Kinney wrote: Hello, I would like to propose that TianoCore move all code review from email based code reviews to GitHub Pull Requests based code reviews. The proposed date to switch would be immediately after

Re: [edk2-devel] Proposal to switch TianoCore Code Review from email to GitHub Pull Requests on 5-24-2024

2024-05-01 Thread Michael Kubacki
Thank you for this proposal. We've been anticipating this change for years and are excited to help support it. Here's some items we'd like to raise for feedback that we could help implement. Many could likely be done in time for the transition. 1. Automate reviewers - We've discussed CODEOWN

Re: [edk2-devel] [PATCH v4 1/1] OvmfPkg/VirtHstiDxe: do not load driver in confidential guests

2024-04-24 Thread Michael Kubacki
, thank you Mike. Should I close/re-open my PR? Or should I keep waiting? Thank you Yao, Jiewen -Original Message- From: Kinney, Michael D Sent: Thursday, April 25, 2024 7:01 AM To: Yao, Jiewen ; devel@edk2.groups.io; Sean Brogan ; Michael Kubacki Cc: Gerd Hoffmann ; Ard Biesheuvel

Re: [edk2-devel] [PATCH v1 1/1] Update to CodeQL 2.16.1

2024-04-24 Thread Michael Kubacki
/24/2024 6:11 PM, Michael D Kinney wrote: Go ahead and add the push label. Mike -Original Message- From: devel@edk2.groups.io On Behalf Of Michael Kubacki Sent: Wednesday, April 24, 2024 2:55 PM To: devel@edk2.groups.io Cc: Feng, Bob C ; Liming Gao ; Rebecca Cran ; Sean Brogan ; Chen

Re: [edk2-devel] [PATCH v1 1/1] Update to CodeQL 2.16.1

2024-04-24 Thread Michael Kubacki
ichael On 4/24/2024 1:37 PM, Michael Kubacki wrote: From: Michael Kubacki Fixes current CI CodeQL failures due to old CodeQL CLI version. Updates CodeQL to work with the latest queries. Includes functional and security fixes within the CodeQL CLI binary. For more information on release d

[edk2-devel] [PATCH v1 1/1] Update to CodeQL 2.16.1

2024-04-24 Thread Michael Kubacki
From: Michael Kubacki Fixes current CI CodeQL failures due to old CodeQL CLI version. Updates CodeQL to work with the latest queries. Includes functional and security fixes within the CodeQL CLI binary. For more information on release details see: https://github.com/github/codeql-cli-binaries

Re: [edk2-devel] OVMF SMM Support

2024-04-08 Thread Michael Kubacki
Might be related to this issue https://gitlab.com/qemu-project/qemu/-/issues/2064. On 4/8/2024 4:33 AM, Wu, Jiaxin wrote: Hi Gerd, With below OVMF build and QEMU command, OVMF hangs after SendSmiIpi (mBspApicId) during SmmRelocateBases(), is there any issues with latest code to support SMM o

Re: [edk2-devel] [PATCH 4/9] MdeModulePkg/Variable: Add TCG SPDM device measurement update

2024-04-04 Thread Michael Kubacki
Can you please fix the following typo? "PcdEnableSpdmDeviceAuthenticaion" Thanks, Michael On 4/1/2024 10:31 PM, Wenxing Hou wrote: Add EV_EFI_SPDM_DEVICE_POLICY support for MeasureVariable. Cc: Liming Gao Cc: Jiewen Yao Signed-off-by: Wenxing Hou --- MdeModulePkg/MdeModulePkg.dec

Re: [edk2-devel] Call or topics for April TianoCore Community Meeting

2024-04-03 Thread Michael Kubacki
Hi Mike, Thanks for continuing to use that project. I haven't seen many updates but my understanding is that you've worked on some of the items since we last talked. Could you please take a look through and see what items you can update? Mostly these types of updates: - Check that current

[edk2-devel] [PATCH v2 1/1] .github/workflows/codeql.yml: Update actions being deprecated

2024-03-01 Thread Michael Kubacki
From: Michael Kubacki Currently CodeQL runs have the following warnings: Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-python@v4, actions/upload-artifact@v3, actions/cache@v3. For more information see: https://github.blog

Re: [edk2-devel] 回复: [PATCH v1] MdeModulePkg/Core/Pei: Improve the copy performance

2024-03-01 Thread Michael Kubacki
This looks fine. Reviewed-by: Michael Kubacki On 3/1/2024 7:58 AM, gaoliming wrote: I agree this change. It should have no negative impact. Reviewed-by: Liming Gao Michael: Have you any comments for this change? Thanks Liming -邮件原件- 发件人: Zhihao Li 发送时间: 2024年3月1日 15:12 收件人

Re: [edk2-devel] CodeQL Analysis in edk2

2024-02-27 Thread Michael Kubacki
On 2/27/2024 10:43 PM, Laszlo Ersek wrote: On 2/27/24 17:04, Michael Kubacki wrote: Hi Gerd, There is a way to suppress results explained here: https://github.com/tianocore/edk2/tree/master/BaseTools/Plugin/CodeQL#filter-patterns A real-world example is here: https://github.com/microsoft

Re: [edk2-devel] CodeQL Analysis in edk2

2024-02-27 Thread Michael Kubacki
Hi Gerd, There is a way to suppress results explained here: https://github.com/tianocore/edk2/tree/master/BaseTools/Plugin/CodeQL#filter-patterns A real-world example is here: https://github.com/microsoft/mu_basecore/blob/release/202311/CodeQlFilters.yml That can currently operate at the fi

Re: [edk2-devel] [PATCH v1 1/1] .github/workflows/codeql.yml: Update actions being deprecated

2024-02-27 Thread Michael Kubacki
being deprecated From: Michael Kubacki Currently CodeQL runs have the following warnings: Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-python@v4, actions/upload-artifact@v3, actions/cache@v3. For more information see: https

[edk2-devel] [PATCH v1 1/1] .github/workflows/codeql.yml: Update actions being deprecated

2024-02-26 Thread Michael Kubacki
From: Michael Kubacki Currently CodeQL runs have the following warnings: Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-python@v4, actions/upload-artifact@v3, actions/cache@v3. For more information see: https://github.blog

Re: [edk2-devel] [Patch 4/4] BaseTools/Scripts/PatchCheck: Error if commit modifies multiple packages

2024-02-23 Thread Michael Kubacki
Reviewed-by: Michael Kubacki On 2/18/2024 3:59 PM, Michael D Kinney wrote: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4679 Update PatchCheck.py to evaluate all the files modified in each commit and generate an error if: * A commit adds/modifies files in multiple package directories

[edk2-devel] [PATCH v1 1/1] BaseTools/GenFds: Resolve absolute workspace INF paths

2024-02-20 Thread Michael Kubacki
From: Michael Kubacki Currently, if an INF path is an absolute path on Linux (begins with "/"), the "/" character will be removed. If the path is an absolute system path, this creates an invalid path. An example of when this may be an issue is in external dependencies where

[edk2-devel] [PATCH v1 1/1] MdeModulePkg/TraceHubDebugSysTLib: Use wider type for loop comparisons

2024-02-20 Thread Michael Kubacki
From: Michael Kubacki Resolves a new CodeQL error due to the value being incremented in the loop being a narrower type than the variable it is being compared against. The variable is changed to a UINT32 type so it has the same width as the type it is being compared against. Issue explanation

[edk2-devel] [PATCH v1 1/1] MdeModulePkg/Bus/Usb/UsbNetwork: Check array index range before access

2024-02-20 Thread Michael Kubacki
From: Michael Kubacki Checks that an offset used to access array elements is within the expected range before accessing the array item. Cc: Liming Gao Cc: Ray Ni Cc: Rebecca Cran Cc: Richard Ho Signed-off-by: Michael Kubacki --- MdeModulePkg/Bus/Usb/UsbNetwork/UsbCdcEcm/UsbEcmFunction.c

[edk2-devel] [PATCH v1 1/1] PrmPkg/PrmInfo: Drop -r parameter

2024-01-10 Thread Michael Kubacki
From: Michael Kubacki The "-r" parameter was not added to the application so remove it from the help string. The standards section is also updated to point to the current specification location on uefi.org. Cc: Nate DeSimone Cc: Ankit Sinha Signed-off-by: Michael Kubacki -

[edk2-devel] [PATCH v1 1/1] FatPkg/FatPei: Check array offset before use

2023-12-12 Thread Michael Kubacki
From: Michael Kubacki Move the range check before array access to enforce the bounds as expected. Cc: Ray Ni Signed-off-by: Michael Kubacki --- FatPkg/FatPei/FatLiteApi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/FatPkg/FatPei/FatLiteApi.c b/FatPkg/FatPei

Re: [edk2-devel] [PATCH 1/2] UnitTestFrameworkPkg: Fix Google Test components with multiple files

2023-12-01 Thread Michael Kubacki
including previously-unreferenced constructors and making sure multi-file gtest EDK2 components work. BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4610 Signed-off-by: Pedro Falcato Cc: Michael D Kinney Cc: Michael Kubacki Cc: Sean Brogan --- Note: /WHOLEARCHIVE should work for VS2015 and newer. I

Re: [edk2-devel] [edk2-platforms][PATCH v1] Vlv2TbltDevicePkg: Fix DxeCore Build Failures

2023-11-30 Thread Michael Kubacki
Reviewed-by: Michael Kubacki On 11/30/2023 7:18 PM, Nate DeSimone wrote: Commit 7284c44 in edk2 introduces an incompatibility that causes any project that uses DxeMain.inf to fail to build. This is due to the addition of ImagePropertiesRecordLib, and a new added dependency on that library in

Re: [edk2-devel] [edk2-platforms][PATCH v1] QuarkPlatformPkg: Fix DxeCore Build Failures

2023-11-30 Thread Michael Kubacki
Reviewed-by: Michael Kubacki On 11/30/2023 7:18 PM, Nate DeSimone wrote: Commit 7284c44 in edk2 introduces an incompatibility that causes any project that uses DxeMain.inf to fail to build. This is due to the addition of ImagePropertiesRecordLib, and a new added dependency on that library in

[edk2-devel] [PATCH v1 1/1] .github/workflows/codeql.yml: Add emacs output

2023-11-29 Thread Michael Kubacki
From: Michael Kubacki Updates the workflow to also output files that can be loaded in emacs to show CodeQL issues (in addition to the existing SARIF output for standard SARIF viewers). The emacs files are in the SARIF zip file attached to each "CodeQL" run (https://github.com/tian

[edk2-devel] [PATCH v2 1/1] SecurityPkg/Tpm2CommandLib: Add new digest list copy and size functions

2023-11-28 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3267 Adds two new helper functions. Currently, a function exists in Tpm2CommandLib to copy a digest list to a buffer. A function to perform the inverse operation - copying from a buffer to a digest list is added. A

[edk2-devel] [PATCH v1 1/1] BaseTools/Conf/target.template: Use VS2019 as default tool chain

2023-11-28 Thread Michael Kubacki
From: Michael Kubacki Updates the default tool chain from VS2015x86 to VS2019. This is the VS tool chain used in CI and more likely to be installed on developer's systems. This is used in stuart commands when a toolchain is not explicitly specified. Cc: Bob Feng Cc: Liming Gao Cc: Re

[edk2-devel] [PATCH v1 1/1] SecurityPkg/Tpm2CommandLib: Add new digest list copy and size functions

2023-11-28 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3267 Adds two new helper functions. Currently, a function exists in Tpm2CommandLib to copy a digest list to a buffer. A function to perform the inverse operation - copying from a buffer to a digest list is added. A

[edk2-devel] [PATCH v2 1/1] ArmPkg/Drivers/CpuDxe: Use lower and upper attributes

2023-11-28 Thread Michael Kubacki
From: Michael Kubacki GetNextEntryAttribute() is currently applying a 64-bit mask (TT_ATTRIBUTES_MASK) to a 32-bit descriptor value (EntryType). The original descriptor was 64 bits containing the upper and lower attributes which are included in TT_ATTRIBUTES_MASK. The PrevEntryAttribute

[edk2-devel] [PATCH v2 0/1] ArmPkg/CpuDxe: Use upper and lower attributes

2023-11-28 Thread Michael Kubacki
From: Michael Kubacki This series contains two changes: 1. To fix a compiler warning with the current state of code. 2. To update the code to pass the integer width needed for a comparison to set EFI_MEMORY_XP in the GCD attribute returned for a given page attribute. v2 changes

[edk2-devel] [PATCH v1 1/1] .git-blame-ignore-revs: Ignore recent uncrustify commits

2023-11-27 Thread Michael Kubacki
From: Michael Kubacki Includes two recent Uncrustify formatting commits to prevent them from showing in git blame. Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael D Kinney Cc: Rebecca Cran Signed-off-by: Michael Kubacki --- .git-blame-ignore-revs | 4 1 file changed, 4 insertions

[edk2-devel] [PATCH v1 1/2] ArmPkg/Drivers/CpuDxe: Explicitly cast table entry

2023-11-27 Thread Michael Kubacki
From: Michael Kubacki GetNextEntryAttribute() assigns a 64-bit integer to 32-bit integers. This change explicitly casts the assigned value as UINT32 to prevent the following Visual Studio compiler warning: '=': conversion from 'UINT64' to 'UINT32', possible l

[edk2-devel] [PATCH v1 2/2] ArmPkg/Drivers/CpuDxe: Use lower and upper attributes

2023-11-27 Thread Michael Kubacki
From: Michael Kubacki GetNextEntryAttribute() is currently applying a 64-bit mask (TT_ATTRIBUTES_MASK) to a 32-bit descriptor value (EntryType). The original descriptor was 64 bits containing the upper and lower attributes which are included in TT_ATTRIBUTES_MASK. The PrevEntryAttribute

[edk2-devel] [PATCH v1 0/2] ArmPkg/CpuDxe: Use upper and lower attributes

2023-11-27 Thread Michael Kubacki
From: Michael Kubacki This series contains two changes: 1. To fix a compiler warning with the current state of code. 2. To update the code to pass the integer width needed for a comparison to set EFI_MEMORY_XP in the GCD attribute returned for a given page attribute. Cc: Leif Lindholm

[edk2-devel] [PATCH v1 1/1] ArmPkg/ArmMmuLib: Use function pointer type

2023-11-27 Thread Michael Kubacki
From: Michael Kubacki mReplaceLiveEntryFunc is a function pointer but assigned as a VOID* pointer: mReplaceLiveEntryFunc = *(VOID **)GET_GUID_HOB_DATA (Hob); This leads to the Visual Studio warning: nonstandard extension, function/data pointer conversion in expression This change

Re: [edk2-devel] [PATCH v2 4/5] MdeModulePkg/Bus/Pci/PciBusDxe: Fix NULL_RETURNS Coverity issue

2023-11-20 Thread Michael Kubacki
On 11/15/2023 5:10 AM, Laszlo Ersek wrote: On 11/14/23 20:37, Michael Kubacki wrote: On 11/14/2023 11:21 AM, Michael D Kinney wrote: Hi Ranbir, First I want to recognize your efforts to collect Coverity issues and propose changes to address them. I still disagree with adding CpuDealLoop

Re: [edk2-devel] question about PrmPkg

2023-11-17 Thread Michael Kubacki
On 11/17/2023 3:42 AM, Laszlo Ersek wrote: On 11/17/23 03:15, Yoshinoya wrote: Hi, I find there is a PrmPkg in udk source code. Based on its Readme.md, its goal is to offload smm code to sci os mechanisms. So, is there any actual use case on real platform now? It seems it's just a conceptional

Re: [edk2-devel] edk2 uncrustify update (73.0.8)?

2023-11-16 Thread Michael Kubacki
On 11/16/2023 3:29 AM, Pedro Falcato wrote: On Tue, Nov 14, 2023 at 3:01 PM Laszlo Ersek wrote: On 11/13/23 22:33, Pedro Falcato wrote: On Mon, Nov 13, 2023 at 8:37 PM Rebecca Cran wrote: On 11/13/2023 1:08 PM, Michael Kubacki wrote: Yes. I just did it. It is relatively minor and impacts

Re: [edk2-devel] CodeQL Analysis in edk2

2023-11-14 Thread Michael Kubacki
On 11/13/2023 8:42 AM, Laszlo Ersek wrote: sorry, unfinished thought: On 11/13/23 14:39, Laszlo Ersek wrote: - the "sarif emacs" output seems a bit broken, actually, so it's not usable. Consider the following entry from the original JSON file: }, { "ruleId" : "cpp/missing-null-te

[edk2-devel] [PATCH v1 3/3] OvmfPkg: Format with Uncrustify 73.0.8

2023-11-14 Thread Michael Kubacki
From: Michael Kubacki Cc: Ard Biesheuvel Cc: Corvin Köhne Cc: Gerd Hoffmann Cc: Jiewen Yao Cc: Rebecca Cran Signed-off-by: Michael Kubacki --- OvmfPkg/EmuVariableFvbRuntimeDxe/Fvb.c | 4 ++-- OvmfPkg/IncompatiblePciDeviceSupportDxe

[edk2-devel] [PATCH v1 2/3] EmulatorPkg: Format with Uncrustify 73.0.8

2023-11-14 Thread Michael Kubacki
From: Michael Kubacki Cc: Andrew Fish Cc: Ray Ni Signed-off-by: Michael Kubacki --- EmulatorPkg/FvbServicesRuntimeDxe/FvbInfo.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/EmulatorPkg/FvbServicesRuntimeDxe/FvbInfo.c b/EmulatorPkg/FvbServicesRuntimeDxe/FvbInfo.c

[edk2-devel] [PATCH v1 0/3] Update from Uncrustify 73.0.3 to 73.0.8

2023-11-14 Thread Michael Kubacki
From: Michael Kubacki This patch series updates to 73.0.8 Uncrustify release which brings the following changes: - A fix for preventing endless indentation in struct assignment. - Updates to code areas impacted by the change. This currently updates 5 files in two packages - EmulatorPkg

Re: [edk2-devel] [PATCH v2 4/5] MdeModulePkg/Bus/Pci/PciBusDxe: Fix NULL_RETURNS Coverity issue

2023-11-14 Thread Michael Kubacki
On 11/14/2023 11:21 AM, Michael D Kinney wrote: Hi Ranbir, First I want to recognize your efforts to collect Coverity issues and propose changes to address them. I still disagree with adding CpuDealLoop() for any static analysis issues. There have been previous discussions about adding a PA

Re: [edk2-devel] edk2 uncrustify update (73.0.8)?

2023-11-13 Thread Michael Kubacki
On 11/13/2023 3:37 PM, Rebecca Cran wrote: On 11/13/2023 1:08 PM, Michael Kubacki wrote: Yes. I just did it. It is relatively minor and impacts expected code areas. https://github.com/tianocore/edk2/pull/5043/files Could you update .git-blame-ignore-revs please? https

Re: [edk2-devel] edk2 uncrustify update (73.0.8)?

2023-11-13 Thread Michael Kubacki
On 11/13/2023 2:14 PM, Rebecca Cran via groups.io wrote: On 11/13/2023 5:29 AM, Marcin Juszkiewicz via groups.io wrote: Still a fan of adding edk2-uncrustify to BaseTools. If we are expected to use it then let it get installed at same moment as "build" command is. The issue with doing this is

Re: [edk2-devel] edk2 uncrustify update (73.0.8)?

2023-11-13 Thread Michael Kubacki
On 11/13/2023 2:07 PM, Pedro Falcato wrote: On Mon, Nov 13, 2023 at 11:58 AM Laszlo Ersek wrote: Hi Michael, recently I encountered an uncrustify failure on github. The reason was that my local uncrustify was *more recent* (73.0.8) than the one we use in edk2 CI (which is 73.0.3, per the edk

Re: [edk2-devel] edk2 uncrustify update (73.0.8)?

2023-11-13 Thread Michael Kubacki
On 11/13/2023 6:58 AM, Laszlo Ersek wrote: Hi Michael, recently I encountered an uncrustify failure on github. The reason was that my local uncrustify was *more recent* (73.0.8) than the one we use in edk2 CI (which is 73.0.3, per the edk2 file ".pytool/Plugin/UncrustifyCheck/uncrustify_ext_dep

Re: [edk2-devel] [PATCH v4 8/8] ReadMe.rst: Add CodeQL/analyze directory under other licenses

2023-11-03 Thread Michael Kubacki
On 11/3/2023 10:46 AM, Laszlo Ersek wrote: On 11/3/23 15:16, Michael Kubacki wrote: On 11/3/2023 9:06 AM, Laszlo Ersek wrote: On 11/2/23 21:03, Michael Kubacki wrote: From: Michael Kubacki The code in this directory is licensed under Apache License, Version 2.0. Therefore, the directory is

Re: [edk2-devel] [PATCH v4 8/8] ReadMe.rst: Add CodeQL/analyze directory under other licenses

2023-11-03 Thread Michael Kubacki
On 11/3/2023 9:06 AM, Laszlo Ersek wrote: On 11/2/23 21:03, Michael Kubacki wrote: From: Michael Kubacki The code in this directory is licensed under Apache License, Version 2.0. Therefore, the directory is listed under paths with licenses other than BSD-2-Clause Plus Patent. The directory

[edk2-devel] [PATCH v4 8/8] ReadMe.rst: Add CodeQL/analyze directory under other licenses

2023-11-02 Thread Michael Kubacki
From: Michael Kubacki The code in this directory is licensed under Apache License, Version 2.0. Therefore, the directory is listed under paths with licenses other than BSD-2-Clause Plus Patent. The directory link points to the complete Apache License, Version 2.0 on apache.org. Cc: Andrew Fish

[edk2-devel] [PATCH v4 7/8] BaseTools/Plugin/CodeQL: Enable 30 queries

2023-11-02 Thread Michael Kubacki
From: Michael Kubacki Updates the CodeQL queries opted into by edk2 to a set of queries from the standard CodeQL query package `codeql/cpp-queries`. After testing a large number of queries the included set here were found to be the most useful with the least number of false positives. Some

[edk2-devel] [PATCH v4 6/8] .pytool/CISettings: Enable CodeQL audit mode

2023-11-02 Thread Michael Kubacki
From: Michael Kubacki Since a large number of CodeQL queries are being enabled to identify issues that the community can collectively resolve, audit mode needs to be enabled to prevent the build from failing. In the future, this global audit mode can be disabled and individual packages can

  1   2   3   4   5   6   7   8   9   10   >