[edk2-devel] [PATCH] MdeModulePkg: Fix port multiplier port in AhciPei PEIM

Fri Jun 2 18:11:55 UTC 2023

[AMD Official Use Only - General]

Hi Abner,

Thank you for your feedback. I updated the commit and re-create the PR as below:
https://github.com/tianocore/edk2/pull/4471

Regards,
Neo

________________________________
From: Chang, Abner <Abner.Chang at amd.com>
Sent: Friday, June 2, 2023 11:31 AM
To: Hsueh, Hong-Chih (Neo) <Hong-Chih.Hsueh at amd.com>; Wu, Hao A <hao.a.wu at intel.com>; He, Jiangang <Jiangang.He at amd.com>; devel at edk2.groups.io <devel at edk2.groups.io>
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg: Fix port multiplier port in AhciPei PEIM

[AMD Official Use Only - General]

Hi Leo,

Please add Hao’s RB in the commit message below your signed-off-by, thus we know this patch has been reviewed.

I also suggest to update your commit subject to “MdeModulePkg/Bus: Fix port multiplier port in AhciPei PEIM”.

Please resend the PR with above updates, then Hao will add “Push” label to this PR once your change passed CI.

Thanks

Abner

From: Hsueh, Hong-Chih (Neo) <Hong-Chih.Hsueh at amd.com>
Sent: Friday, June 2, 2023 10:37 PM
To: Wu, Hao A <hao.a.wu at intel.com>; He, Jiangang <Jiangang.He at amd.com>; devel at edk2.groups.io
Cc: Chang, Abner <Abner.Chang at amd.com>
Subject: Re: [edk2-devel] [PATCH] MdeModulePkg: Fix port multiplier port in AhciPei PEIM

[AMD Official Use Only - General]

Hi Hao,

Thank you for your review.

I already created a pull request for this commit, may I know how to proceed to merge it into master?

https://github.com/tianocore/edk2/pull/4424

Regards,

Neo

________________________________

From: Wu, Hao A <hao.a.wu at intel.com<mailto:hao.a.wu at intel.com>>
Sent: Thursday, June 1, 2023 10:14 PM
To: He, Jiangang <Jiangang.He at amd.com<mailto:Jiangang.He at amd.com>>; devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>>; Hsueh, Hong-Chih (Neo) <Hong-Chih.Hsueh at amd.com<mailto:Hong-Chih.Hsueh at amd.com>>
Cc: Chang, Abner <Abner.Chang at amd.com<mailto:Abner.Chang at amd.com>>
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg: Fix port multiplier port in AhciPei PEIM

[AMD Official Use Only - General]

Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.

Thanks.
Reviewed-by: Hao A Wu <hao.a.wu at intel.com<mailto:hao.a.wu at intel.com>>

Best Regards,
Hao Wu

> -----Original Message-----
> From: He, Jiangang <Jiangang.He at amd.com<mailto:Jiangang.He at amd.com>>
> Sent: Wednesday, May 31, 2023 10:49 PM
> To: Wu, Hao A <hao.a.wu at intel.com<mailto:hao.a.wu at intel.com>>; devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Hsueh, Hong-
> Chih (Neo) <Hong-Chih.Hsueh at amd.com<mailto:Hong-Chih.Hsueh at amd.com>>
> Cc: Chang, Abner <Abner.Chang at amd.com<mailto:Abner.Chang at amd.com>>
> Subject: RE: [edk2-devel] [PATCH] MdeModulePkg: Fix port multiplier port in
> AhciPei PEIM
>
> [AMD Official Use Only - General]
>
> We did crisis recovery and Opal HD password unlock from S3 resume from
> SATA HD test on two different version of AHCI host controllers.
>
> Thanks,
> Jiangang
> -----Original Message-----
> From: Wu, Hao A <hao.a.wu at intel.com<mailto:hao.a.wu at intel.com>>
> Sent: Tuesday, May 30, 2023 10:30 PM
> To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Hsueh, Hong-Chih (Neo) <Hong-
> Chih.Hsueh at amd.com<mailto:Chih.Hsueh at amd.com>>
> Cc: He, Jiangang <Jiangang.He at amd.com<mailto:Jiangang.He at amd.com>>; Chang, Abner
> <Abner.Chang at amd.com<mailto:Abner.Chang at amd.com>>
> Subject: RE: [edk2-devel] [PATCH] MdeModulePkg: Fix port multiplier port in
> AhciPei PEIM
>
> Caution: This message originated from an External Source. Use proper
> caution when opening attachments, clicking links, or responding.
>
>
> Thanks, the code changes look good to me.
> May I know what tests have been performed for the patch?
>
> Best Regards,
> Hao Wu
>
> > -----Original Message-----
> > From: devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>> On Behalf Of Neo
> > Hsueh via groups.io
> > Sent: Wednesday, May 24, 2023 1:07 AM
> > To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>
> > Cc: jiangang.he at amd.com<mailto:jiangang.he at amd.com>; abner.chang at amd.com<mailto:abner.chang at amd.com>; Neo Hsueh <Hong-
> > Chih.Hsueh at amd.com<mailto:Chih.Hsueh at amd.com>>
> > Subject: [edk2-devel] [PATCH] MdeModulePkg: Fix port multiplier port
> > in AhciPei PEIM
> >
> > If there is no port multiplier, PortMultiplierPort should be converted
> > to 0 to follow AHCI spec.
> > The same logic already applied in AtaAtapiPassThruDxe driver.
> >
> > Signed-off-by: Neo Hsueh <Hong-Chih.Hsueh at amd.com<mailto:Hong-Chih.Hsueh at amd.com>>
> > ---
> >  MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> >
> > diff --git a/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c
> > b/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c
> > index cd55272c96..7bd04661d0 100644
> > --- a/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c
> > +++ b/MdeModulePkg/Bus/Ata/AhciPei/AhciPeiPassThru.c
> > @@ -3,6 +3,7 @@
> >    mode at PEI phase.
> >
> >    Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> > +  Copyright (C) 2023 Advanced Micro Devices, Inc. All rights
> > + reserved.<BR>
> >
> >    SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > @@ -92,6 +93,15 @@ AhciPassThruExecute (  {
> >    EFI_STATUS  Status;
> >
> > +  if (PortMultiplierPort == 0xFFFF) {
> > +    //
> > +    // If there is no port multiplier, PortMultiplierPort will be 0xFFFF
> > +    // according to UEFI spec. Here, we convert its value to 0 to follow
> > +    // AHCI spec.
> > +    //
> > +    PortMultiplierPort = 0;
> > +  }
> > +
> >    switch (Packet->Protocol) {
> >      case EFI_ATA_PASS_THRU_PROTOCOL_ATA_NON_DATA:
> >        Status = AhciNonDataTransfer (
> > --
> > 2.40.0.windows.1
> >
> >
> >
> > 
> >

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#105673): https://edk2.groups.io/g/devel/message/105673
Mute This Topic: https://groups.io/mt/99093106/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230602/059cabad/attachment-0001.htm>