From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 4BC46941943 for ; Wed, 6 Mar 2024 14:10:35 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=x0KRShN2Q4ewOorQkRhKgqeDsWSaDafIN21UUnJdkhM=; c=relaxed/simple; d=groups.io; h=From:To:Cc:References:In-Reply-To:Subject:Date:Message-ID:MIME-Version:Thread-Index:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding:Content-Language; s=20240206; t=1709734234; v=1; b=EHYsbJ9/WJNZYt41QnJqA0TWsnxmMWkVPCtA7ZY5upUDj1gGsBXWvn5nfqEsL8FCK8z8aMNm Ww9YZIxnImtG9hhjaOKhn/m1wrKmp8es3oNfwTvm1PKzX3aoblyFaaPftPxhPv5ercy7hzpvNap rYiRtRPoh3J5h/RJAOmQZD6JOdEJfIJlEzuRS/PRGSqfI28V9TR47YqDhTyS2ytvjuAdw4IaObd TjUypLt/4pfIkaM6nth7Rf2kzHStUIUMT/032cXNRjwm2bJgK17ly/O46cDY9f3aht5zXP9ICI7 ouH2IOp4297JnVZkF3xR1CWfyfa0obFl1j3xM+PlO9nlw== X-Received: by 127.0.0.2 with SMTP id RPcnYY7687511xJ6391Pvypo; Wed, 06 Mar 2024 06:10:34 -0800 X-Received: from cxsh.intel-email.com (cxsh.intel-email.com [121.46.250.151]) by mx.groups.io with SMTP id smtpd.web11.12556.1709734232633400449 for ; Wed, 06 Mar 2024 06:10:33 -0800 X-Received: from cxsh.intel-email.com (localhost [127.0.0.1]) by cxsh.intel-email.com (Postfix) with ESMTP id 8B954DDA7EA for ; Wed, 6 Mar 2024 22:10:30 +0800 (CST) X-Received: from localhost (localhost [127.0.0.1]) by cxsh.intel-email.com (Postfix) with ESMTP id 8725DDDA7C8 for ; Wed, 6 Mar 2024 22:10:30 +0800 (CST) X-Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by cxsh.intel-email.com (Postfix) with SMTP id 99CCFDDA7E3 for ; Wed, 6 Mar 2024 22:10:27 +0800 (CST) X-Received: from DESKTOPS6D0PVI ([114.93.194.54]) (envelope-sender ) by 192.168.6.13 with ESMTP(SSL) for ; Wed, 06 Mar 2024 22:10:24 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 114.93.194.54 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming via groups.io" To: , Cc: "'Michael D Kinney'" References: <6082ad3bafb377c7515e98ba0c5ff70acd236615.1708915895.git.qingyu.shang@intel.com> In-Reply-To: <6082ad3bafb377c7515e98ba0c5ff70acd236615.1708915895.git.qingyu.shang@intel.com> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIHY0IDIvM10gTWRlTW9kdWxlUGtnOiBVcGRhdGUgUmVjZWl2ZURhdGEgYW5kIFNlbmREYXRhIGZ1bmN0aW9uIGRlc2NyaXB0aW9u?= Date: Wed, 06 Mar 2024 06:10:33 -0800 Message-ID: <00f801da6fd0$0971e1f0$1c55a5d0$@byosoft.com.cn> MIME-Version: 1.0 Thread-Index: AQGPgotzNRwg0tBwITEHJPxCKnY49gGwBzgqsbMavGA= 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 Reply-To: devel@edk2.groups.io,gaoliming@byosoft.com.cn List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: eR0egWkLZyLVMEEYSS09a0q3x7686176AA= Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b="EHYsbJ9/"; dmarc=pass (policy=none) header.from=groups.io; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io Reviewed-by: Liming Gao > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: devel@edk2.groups.io =B4=FA=B1= =ED Qingyu > =B7=A2=CB=CD=CA=B1=BC=E4: 2024=C4=EA2=D4=C226=C8=D5 11:06 > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io > =B3=AD=CB=CD: Liming Gao ; Michael D Kinney > > =D6=F7=CC=E2: [edk2-devel] [PATCH v4 2/3] MdeModulePkg: Update ReceiveDat= a and > SendData function description >=20 > AtaBusDxe, NvmExpressDxe, ScsiDiskDxe and EmmcDxe is used to back the > EFI_STORAGE_SECURITY_COMMAND_PROTOCOL, update the parameter > 'MediaId' > description for the protocal function ReceiveData and SendData as described > in Uefi spec 2.10 section 13.14. >=20 > Cc: Liming Gao > Cc: Michael D Kinney > Signed-off-by: Qingyu > --- > MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.c | 8 > ++++++-- > MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h | 8 > ++++++-- > .../Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.c | 8 ++++++-- > .../Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.h | 8 ++++++-- > MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c | 8 ++++++-- > MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h | 8 ++++++-- > MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c | 12 > +++++++++--- > MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.h | 12 > +++++++++--- > 8 files changed, 54 insertions(+), 18 deletions(-) >=20 > diff --git a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.c > b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.c > index a729c087ff5a..67c6389bb8b3 100644 > --- a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.c > +++ b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.c > @@ -1483,7 +1483,9 @@ AtaDiskInfoWhichIde ( > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -1600,7 +1602,9 @@ AtaStorageSecurityReceiveData ( > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > diff --git a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > index 6bc345f7e777..8906947e28b3 100644 > --- a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > +++ b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBus.h > @@ -927,7 +927,9 @@ AtaDiskInfoWhichIde ( > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -1007,7 +1009,9 @@ AtaStorageSecurityReceiveData ( > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > diff --git a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.c > b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.c > index b6ed7f7031b3..e0a85c1cb34e 100644 > --- a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.c > +++ b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.c > @@ -1699,7 +1699,9 @@ TrustTransferNvmeDevice ( > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -1812,7 +1814,9 @@ NvmeStorageSecurityReceiveData ( > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > diff --git a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.h > b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.h > index 8cad15efd33a..5547e057e77a 100644 > --- a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.h > +++ b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressBlockIo.h > @@ -289,7 +289,9 @@ NvmeBlockIoFlushBlocksEx ( > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -369,7 +371,9 @@ NvmeStorageSecurityReceiveData ( > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > diff --git a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > index 670f70caf41f..e79110c14e11 100644 > --- a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > +++ b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c > @@ -1897,7 +1897,9 @@ Done: > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -2119,7 +2121,9 @@ Done: > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > diff --git a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > index 5a25b55c4952..49a00403695f 100644 > --- a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > +++ b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h > @@ -677,7 +677,9 @@ ScsiDiskEraseBlocks ( > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -751,7 +753,9 @@ ScsiDiskReceiveData ( > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > diff --git a/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c > b/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c > index 39b7c7ce6ef1..61fa740be505 100644 > --- a/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c > +++ b/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c > @@ -1267,7 +1267,9 @@ EmmcFlushBlocksEx ( > function shall return EFI_DEVICE_ERROR. >=20 > @param[in] This Indicates a pointer to > the calling context. > - @param[in] MediaId ID of the medium to > receive data from. > + @param[in] MediaId ID of the medium to > receive data from. If there is no > + block IO protocol > supported by the physical device, the > + value of MediaId is > undefined. > @param[in] Timeout The timeout, in 100ns > units, to use for the execution > of the security > protocol command. A Timeout value of 0 > means that this > function will wait indefinitely for the > @@ -1454,7 +1456,9 @@ EmmcSecurityProtocolInOut ( > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -1555,7 +1559,9 @@ EmmcSecurityProtocolIn ( > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > diff --git a/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.h > b/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.h > index 2568f0359df3..8c7e5d2675ff 100644 > --- a/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.h > +++ b/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.h > @@ -250,7 +250,9 @@ EmmcFlushBlocksEx ( > function shall return EFI_DEVICE_ERROR. >=20 > @param[in] This Indicates a pointer to > the calling context. > - @param[in] MediaId ID of the medium to > receive data from. > + @param[in] MediaId ID of the medium to > receive data from. If there is no > + block IO protocol > supported by the physical device, the > + value of MediaId is > undefined. > @param[in] Timeout The timeout, in 100ns > units, to use for the execution > of the security > protocol command. A Timeout value of 0 > means that this > function will wait indefinitely for the > @@ -337,7 +339,9 @@ EmmcSecurityProtocolInOut ( > function shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > @@ -417,7 +421,9 @@ EmmcSecurityProtocolIn ( > shall return EFI_DEVICE_ERROR. >=20 > @param This Indicates a pointer to the > calling context. > - @param MediaId ID of the medium to > receive data from. > + @param MediaId ID of the medium to > receive data from. If there is no > + block IO protocol supported > by the physical device, the > + value of MediaId is > undefined. > @param Timeout The timeout, in 100ns units, > to use for the execution > of the security protocol > command. A Timeout value of 0 > means that this function > will wait indefinitely for the > -- > 2.39.1.windows.1 >=20 >=20 >=20 >=20 >=20 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#116441): https://edk2.groups.io/g/devel/message/116441 Mute This Topic: https://groups.io/mt/104766056/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-