Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MdeModulePkg: SataController binding driver can check for DevicePath Protocol before starting (Bugzilla Bug 4858) #10542

Open
tianocore-issues opened this issue Oct 1, 2024 · 3 comments
Labels
package:mdemodulepkg priority:low Little to no impact. No urgency to fix. type:bug Something isn't working

Comments

@tianocore-issues
Copy link

This issue was created automatically with bugzilla2github

Bugzilla Bug 4858

Date: 2024-10-01T12:07:46+00:00
From: Paul Chang <>
To: Paul Chang <>
CC: @lgao4

Last updated: 2024-10-24T04:38:12+00:00

@tianocore-issues
Copy link
Author

Comment 23415

Date: 2024-10-01 12:07:46 +0000
From: Paul Chang <>

  • Industry Specification: ---
  • Release Observed: EDK II Master
  • Releases to Fix: EDK II Master
  • Target OS: ---
  • Bugzilla Assignee(s): Paul Chang <>

Created attachment 1541
Propsed patch for reference

Ph52a PCIE to SATA card inserted on Intel MTL/ARL causes system hanging with AMI code base. The issue is not there on previous Intel platform RPL because Intel uses SataControl code in its platform package with the fix attached.

Root cause of this issue is because Ph52a's driver only uses DevicePath protocol alone and EDK2 driver only uses PciIo protocol alone. Both drivers start and try to manage SATA controller.

To avoid seeing same issue with difference card vendor in the future, we are hoping EDK2 group can take this fix generically.

Provide AMI Internal ticket EIP803584 for reference.

Attachment: BZ-1541-SataController_Patch_20241001.7z

@tianocore-issues
Copy link
Author

Comment 23484

Date: 2024-10-23 20:54:15 +0000
From: @lgao4

Paul Chang: please create edk2 pull request to make this change.

@tianocore-issues
Copy link
Author

Comment 23495

Date: 2024-10-24 04:38:12 +0000
From: Paul Chang <>

Hi gaoliming,

I have created pull request below.
#6371

Thanks
Paul

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:mdemodulepkg priority:low Little to no impact. No urgency to fix. type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant