From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id CEFD9941AE4 for ; Tue, 30 Jul 2024 15:08:20 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=dStbGmyimwqJulUDC60go4dWC4aJnQhVGzgTsksUY1s=; c=relaxed/simple; d=groups.io; h=Date:Mime-Version:Message-ID:Subject:From:To:Cc:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type; s=20240206; t=1722352100; v=1; b=xf1TTQDt9m1Pp68KYsmYCuAWYrU3XG8BES4G2sRNwW0qTQ1MEPLi+y+jk+3ksawlI7gV6HFB RdvE2LGTmMAo/kIrgdlfqwfpLCIyR4P3hAIbcVrnO3pxX5Pvsb91l7hhk86HeTAtgX9+21RA85E R+StQlYyQCe4FgNrRyvg3LVTiaxWD3QGkqY+cpPHWLGMFeJv4/teusKAtXafCzBTOusesxi14tv 2ex3En7HFE+J9kIk4yZ6wLAt7tnrmFYd9ggcGqgrY1HhoDBTW1vcrvcluDxZOGJgvAjM8EG76t2 CcArIh2XErAy7i6UAzW9UW8mKZzrxC7B8O3p16wm1AMFQ== X-Received: by 127.0.0.2 with SMTP id 2otqYY7687511xG0QumZMvOT; Tue, 30 Jul 2024 08:08:19 -0700 X-Received: from mail-wm1-f73.google.com (mail-wm1-f73.google.com [209.85.128.73]) by mx.groups.io with SMTP id smtpd.web10.42097.1722199482554094848 for ; Sun, 28 Jul 2024 13:44:42 -0700 X-Received: by mail-wm1-f73.google.com with SMTP id 5b1f17b1804b1-42816aacabcso10041695e9.1 for ; Sun, 28 Jul 2024 13:44:42 -0700 (PDT) X-Gm-Message-State: byq4TWm899pTolEuER1Y3RsOx7686176AA= X-Google-Smtp-Source: AGHT+IFFhsf9EEO9wCZ0NCWNeJpZyCnIfK5LMuNe93psu3LnnEwne0WfFTQCKx3cbTQbee5Wk1SsTV1X X-Received: from palermo.c.googlers.com ([fda3:e722:ac3:cc00:28:9cb1:c0a8:118a]) (user=ardb job=sendgmr) by 2002:adf:f34c:0:b0:368:12ef:92d1 with SMTP id ffacd0b85a97d-36b5d0894a4mr7583f8f.9.1722199480347; Sun, 28 Jul 2024 13:44:40 -0700 (PDT) Date: Sun, 28 Jul 2024 22:44:27 +0200 Mime-Version: 1.0 Message-ID: <20240728204437.4064847-1-ardb+git@google.com> Subject: [edk2-devel] [PATCH edk2-platforms v2 0/8] RPi: Drop EmbeddedPkg reset runtime From: "Ard Biesheuvel via groups.io" To: devel@edk2.groups.io Cc: Ard Biesheuvel , Leif Lindholm , Jeremy Linton Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Tue, 30 Jul 2024 08:08:17 -0700 Resent-From: ardb+git@google.com Reply-To: devel@edk2.groups.io,ardb+git@google.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="UTF-8" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=xf1TTQDt; dmarc=pass (policy=none) header.from=groups.io; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io From: Ard Biesheuvel The EmbeddedPkg runtime DXE is being retired in favour of the generic one in MdeModulePkg which is actually being maintained. RPi uses this driver and the associated EfiResetSystemLib, of which it has an implementation with value-add for reset notification. So this logic needs to be moved elsewhere and hooked up to the generic protocols that implement the same. Changes since v1: - boot tested - add patch to fix pre-existing issue that causes a crash when DmaLib attempts to set EFI_MEMORY_XP on allocated buffers - add a patch to force the correct dispatch order for the varstore related drivers - fix line endings Cc: Leif Lindholm Cc: Jeremy Linton Ard Biesheuvel (8): Platform/RaspberryPi: Mark RAM regions as write/execute protectable Platform/RaspberryPi: Fix line endings in DSCs Platform/RaspberryPi: Use depex based dispatch order for varstore Platform/RaspberryPi/VarBlockServiceDxe: Refactor DumpVars event handler Platform/RaspberryPi/VarBlockServiceDxe: Register for reset notification Platform/RaspberryPi/PlatformBootManagerLib: Reimplement reset hook Platform/RaspberryPi: Switch to generic reset runtime Platform/RaspberryPi: Drop platform specific EfiResetSystemLib Platform/RaspberryPi/RaspberryPi.dec | 1 - Platform/RaspberryPi/RPi3/RPi3.dsc | 15 +- Platform/RaspberryPi/RPi4/RPi4.dsc | 15 +- Platform/RaspberryPi/RPi3/RPi3.fdf | 2 +- Platform/RaspberryPi/RPi4/RPi4.fdf | 2 +- Platform/RaspberryPi/Drivers/VarBlockServiceDxe/VarBlockServiceDxe.inf | 6 +- Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf | 6 + Platform/RaspberryPi/Library/ResetLib/ResetLib.inf | 45 ------ Platform/RaspberryPi/Drivers/VarBlockServiceDxe/VarBlockServiceDxe.c | 65 ++++++--- Platform/RaspberryPi/Library/MemoryInitPeiLib/MemoryInitPeiLib.c | 2 + Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c | 77 ++++++++++ Platform/RaspberryPi/Library/ResetLib/ResetLib.c | 151 -------------------- 12 files changed, 156 insertions(+), 231 deletions(-) delete mode 100644 Platform/RaspberryPi/Library/ResetLib/ResetLib.inf delete mode 100644 Platform/RaspberryPi/Library/ResetLib/ResetLib.c -- 2.46.0.rc1.232.g9752f9e123-goog -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#120100): https://edk2.groups.io/g/devel/message/120100 Mute This Topic: https://groups.io/mt/107626473/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=-