Tianocore/edk2.git/MdeModulePkg/Bus/Scsi, branch dependabot/github_actions/actions/github-script-9

MdeModulePkg,MdePkg: Send I/O without FUA if necessary

2026-05-06T08:04:12+00:00

Commit 8c654bb3ec sets FUA bit forcefully in SCSI sync I/O to avoid possible data loss. However, it may cause issues for the storage without the support FUA and WCE. For example, Windows Server 2025 guest fails to boot from vhost-scsi block device in such scenario. The booting Error code is 0xc0000185, the SCSI command fails with the following, Mode Sense Key: EFI_SCSI_SK_ILLEGAL_REQUEST Additional Sense Code: EFI_SCSI_ASC_INVALID_FIELD This patch gets the FUA and WCE support of the storage and sets FUA conditionally. Continuous-integration-options: PatchCheck.ignore-multi-package Signed-off-by: Annie Li

MdeModulePkg/ScsiDiskDxe: Check Write Caching and FUA support

2026-05-06T08:04:12+00:00

Check Write Caching and FUA support of the storage, then save the result if both are disabled. Signed-off-by: Annie Li

MdeModulePkg: Replace include guards with #pragma once

2026-02-23T21:01:28+00:00

Replace traditional `#ifndef`/`#define`/`#endif` include guards with `#pragma` once. `#pragma once` is a widely supported preprocessor directive that prevents header files from being included multiple times. It is supported by all toolchains used to build edk2: GCC, Clang/LLVM, and MSVC. Compared to macro-based include guards, `#pragma once`: - Eliminates the risk of macro name collisions or copy/paste errors where two headers inadvertently use the same guard macro. - Eliminate inconsistency in the way include guard macros are named (e.g., some files use `__FILE_H__`, others use `FILE_H_`, etc.). - Reduces boilerplate (three lines replaced by one). - Avoids polluting the macro namespace with guard symbols. - Can improve build times as the preprocessor can skip re-opening the file entirely, rather than re-reading it to find the matching `#endif` ("multiple-include optimization"). - Note that some compilers may already optimize traditional include guards, by recognzining the idiomatic pattern. This change is made acknowledging that overall portability of the code will technically be reduced, as `#pragma once` is not part of the C/C++ standards. However, this is considered acceptable given: 1. edk2 already defines a subset of supported compilers in BaseTools/Conf/tools_def.template, all of which have supported `#pragma once` for over two decades. 2. There have been concerns raised to the project about inconsistent include guard naming and potential macro collisions. Approximate compiler support dates: - MSVC: Supported since Visual C++ 4.2 (1996) - GCC: Supported since 3.4 (2004) (http://gnu.ist.utl.pt/software/gcc/gcc-3.4/changes.html) - Clang (LLVM based): Since initial release in 2007 Signed-off-by: Michael Kubacki

MdeModulePkg/Bus: Remove ScsiPassThruProtocolGuid

2025-11-23T18:40:59+00:00

The legacy ScsiPassThruProtocol has been removed as per the UEFI 2.10A specification. This commit, removes references to ScsiPassThruProtocol and transitions the implementation to only use ExtScsiPassThruProtocol, which provides enhanced support for device enumeration, target/LUN discovery, and broader SCSI compatibility. Cc: Sachin Ganesh Signed-off-by: Gowtham M

MdeModulePkg/Scsi:Change Index and MaxRetry from UINT8 to UINTN

2025-11-22T07:16:33+00:00

Change Index and MaxRetry to UINTN in ScsiDiskInquiryDevice(). Signed-off-by: Revathy

MdeModulePkg: ScsiDiskDxe: Query Write Protected State

2025-10-16T12:25:52+00:00

Currently, only SCSI attached CDROMs are marked as read only SCSI devices. Write failures have been seen to occur when a read only disk is attached as a SCSI device, because the device is never queried for its write protected state and the disk is listed as writeable. This adds a new mode sense query to determine if a SCSI disk is write protected. If so, the proper BlockIO media property is set to read only, to ensure that all layers know this is a read only disk. Signed-off-by: Oliver Smith-Denny

MdeModulePkg: Update ReceiveData and SendData function description

2024-03-13T02:48:38+00:00

AtaBusDxe, NvmExpressDxe, ScsiDiskDxe and EmmcDxe is used to back the EFI_STORAGE_SECURITY_COMMAND_PROTOCOL, update the parameter 'MediaId' description for the protocol function ReceiveData and SendData as described in UEFI Spec 2.10 section 13.14. Cc: Liming Gao Cc: Michael D Kinney Signed-off-by: Qingyu Shang Reviewed-by: Liming Gao

MdeModulePkg: Update code to be more C11 compliant by using func

2023-04-10T14:19:57+00:00

__FUNCTION__ is a pre-standard extension that gcc and Visual C++ among others support, while __func__ was standardized in C99. Since it's more standard, replace __FUNCTION__ with __func__ throughout MdeModulePkg. Signed-off-by: Rebecca Cran Reviewed-by: Michael D Kinney Reviewed-by: Ard Biesheuvel

MdeModulePkg: Consume new alignment-related macros

2023-04-01T23:11:44+00:00

This patch substitutes the macros that were renamed in the first patch with the new, shared alignment macros. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Vitaly Cheptsov Signed-off-by: Marvin Häuser Signed-off-by: Gerd Hoffmann Reviewed-by: Hao A Wu Reviewed-by: Michael D Kinney

MdeModulePkg: Rename IS_ALIGNED macros to avoid name collisions