On 10/4/23 18:31, Taylor Beebe wrote:
>
> On 10/4/23 1:46 AM, Gerd Hoffmann wrote:
>> On Fri, Sep 29, 2023 at 12:52:35PM -0700, Taylor Beebe wrote:
>>> I can also update ArmVirtPkg to disable execution protection
>>> for EfiLoaderData by default until fw_cfg parsing
>>> support is added to ArmVirt
On 10/5/23 08:31, Nhi Pham via groups.io wrote:
> Hi Ard, Oliver,
>
> I'm investigating the crash on grub2/shim loader due to the added
> EFI_MEMORY_ATTRIBUTE_PROTOCOL when rebasing. I found this interesting
> patch and went through on the discussion, I am still not sure the
> conclusion on this p
Hi,
> On x86, PEI can use IO ports (no page tables), but that's not available
> on ARM. I don't recall off-hand where / when exactly page tables are set
> up during ArmVirtQemu boot.
It also changed roughly one year ago, Ard updated the arm platform setup
to turn on paging much earlier (for w^x
Shouldn't we update the PLDM protocol's 'PldmSubmit' function to
receive 'MctpSrcEID'/'MctpDestEID' as incoming parameters? Or maybe
add some 'PldmInit' function to set those parameters for further PLDM
communication?
Because right now the MCTP EIDs are fixed to PCDs with no flexibility
https://git
On Thu, Oct 05, 2023 at 10:23:25AM +0200, Laszlo Ersek wrote:
> On 10/5/23 08:31, Nhi Pham via groups.io wrote:
> > Hi Ard, Oliver,
> >
> > I'm investigating the crash on grub2/shim loader due to the added
> > EFI_MEMORY_ATTRIBUTE_PROTOCOL when rebasing. I found this interesting
> > patch and went
Hi,
> > > An Arm compatible PEIM instance of QemuFwCfgLib will need to be created.
> > > I'm happy to look into it, but I don't want to hang up this patch series
> > > on
> > > that addition. Instead, I'll set the protection policy for ArmVirtPkg to
> > > the equivalent of the new GrubCompat pr
On Wed, Oct 04, 2023 at 18:46:58 +, Michael D Kinney wrote:
> I do not prefer special comments for one static analyzer.
>
> Is there an alternative design/implementation of this code
> to make it more readable and not trigger any static analysis
> false positives?
Not the tidiest, but could r
Hi Tuan,
Thank you for this patch.
These changes look good to me.
Reviewed-by: Sami Mujawar
Regards,
Sami Mujawar
On 28/09/2023 10:14 pm, Tuan Phan wrote:
Update entry point library for Arm to use the new architecture independent
StandaloneMmCpu driver.
Signed-off-by: Tuan Phan
Reviewed-
Also I see that the 'PldmProtocolCommon' code contains array of
expected response sizes for every supported PLDM command:
https://github.com/changab/edk2-platforms/blob/5af7ca1555863288bd2887ca465ad07d74b9868e/Features/ManageabilityPkg/Universal/PldmProtocol/Common/PldmProtocolCommon.c#L24
I'm not
I was thinking more about it and I think the right solution would be
to move these checks to the EDKII_PLDM_SMBIOS_TRANSFER_PROTOCOL code
(i.e.
https://github.com/changab/edk2-platforms/blob/MCTP_OVER_KCS_UPDATE/Features/ManageabilityPkg/Universal/PldmSmbiosTransferDxe/PldmSmbiosTransferDxe.c)
Be
On 10/5/23 12:23, Gerd Hoffmann wrote:
> Hi,
>
An Arm compatible PEIM instance of QemuFwCfgLib will need to be created.
I'm happy to look into it, but I don't want to hang up this patch series on
that addition. Instead, I'll set the protection policy for ArmVirtPkg to
the equ
*Reminder: TianoCore edk2-test Bug Triage Meeting*
*When:*
Thursday, October 5, 2023
10:00pm to 11:00pm
(UTC+08:00) Asia/Shanghai
*Where:*
https://armltd.zoom.us/j/91247522013?pwd=ei9nUndTbG9oWEROS2M1aVREZkpiQT09&from=addon
*Organizer:* Edhaya Chandran edhaya.chand...@arm.com (
edhaya.chand...@
*TianoCore edk2-test Bug Triage Meeting*
*When:*
Thursday, October 5, 2023
10:00pm to 11:00pm
(UTC+08:00) Asia/Shanghai
*Where:*
https://armltd.zoom.us/j/91247522013?pwd=ei9nUndTbG9oWEROS2M1aVREZkpiQT09&from=addon
*Organizer:* Edhaya Chandran edhaya.chand...@arm.com (
edhaya.chand...@arm.com?su
The solution is upstreamed thorugh the commit:
https://github.com/tianocore/edk2-test/commit/81dfa8d53d4290366ae41e1f4c2ed6d6c5016c07
Thank you all.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#109357): https://edk2.groups.io/g/devel/m
Sorry for many messages today, but I have one more observation.
I think the MCTP code should set specific 'AdditionalTransferError'
for the caller when it fails on each of the response header checks:
https://github.com/changab/edk2-platforms/blob/5af7ca1555863288bd2887ca465ad07d74b9868e/Features/M
__builtin_return_address returns a pointer, not a string. Fix the STACK
FAULT message in BaseStackCheckLib appropriately.
Signed-off-by: Jake Garver
---
MdePkg/Library/BaseStackCheckLib/BaseStackCheckGcc.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/MdePkg/Library/Base
AmlCodeGenRdQWordMemory's and AmlCodeGenRdDWordMemory's Cacheable
and MemoryRangeType parameters treat specific values as having
specific meanings as defined by the spec. This change adds enums to map
those meanings to their corresponding values.
Signed-off-by: Jeshua Smith
---
Notes:
v2: based
On Wed, Oct 4, 2023 at 9:01 PM MrChromebox wrote:
>
> Add the ClockRate field to the UNIVERSAL_PAYLOAD_SERIAL_PORT_INFO
> struct, so that the field can be used by UefiPayloadPkg to properly
> set up the serial port on boards using a non-standard clock rate.
>
> Signed-off-by: Matt DeVillier
> Cha
https://bugzilla.tianocore.org/show_bug.cgi?id=4461
1. The function GetRandomNumber128() returns a boolean TRUE/FALSE.
The existing code was getting data in EFI_STATUS.0==FALSE==EFI_SUCCESS
Updated code to check for the return value before proceeding further.
2. The 'seed' from the GetRandomN
19 matches
Mail list logo
