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

EmulatorPkg IA32 Segfault in late Dxe (Bugzilla Bug 2668) #8458

Closed
tianocore-issues opened this issue Apr 8, 2020 · 13 comments
Closed

EmulatorPkg IA32 Segfault in late Dxe (Bugzilla Bug 2668) #8458

tianocore-issues opened this issue Apr 8, 2020 · 13 comments
Labels
package:emulatorpkg priority:medium Moderate impact. Should be prioritized over lower priority issues. type:bug Something isn't working

Comments

@tianocore-issues
Copy link

This issue was created automatically with bugzilla2github

Bugzilla Bug 2668

Date: 2020-04-08T12:18:55+00:00
From: @spbrogan
To: guomin.jiang
CC: brian.delgado, nobody

See also: https://bugzilla.tianocore.org/show_bug.cgi?id=2639
See also: https://bugzilla.tianocore.org/show_bug.cgi?id=2638
Last updated: 2022-03-03T04:59:23+00:00

@tianocore-issues
Copy link
Author

Comment 12150

Date: 2020-04-08 12:18:55 +0000
From: @spbrogan

  • Industry Specification: ---
  • Release Observed: EDK II Master
  • Releases to Fix: EDK II Master
  • Target OS: Linux
  • Bugzilla Assignee(s): guomin.jiang

After getting IA32 to build on Ubuntu 18.04 it now segfaults when trying to run. It fails in late DXE which is different than https://bugzilla.tianocore.org/show_bug.cgi?id=2639.

Maybe I am missing some graphics configuration because i am using a headless VM but regardless this should not cause Segmentation fault.

Azure Pipeline run here (build and boot logs as artifacts): https://dev.azure.com/tianocore/edk2-ci-play/_build/results?buildId=5555&view=logs&j=340bd6f2-95b1-514a-e5d4-c88ff61ededb&t=2f930dd8-d4fd-573a-2635-738b56f3c93e

Boot Log ends like

INFO - [BdsDxe] Locate Variable Lock protocol - Success
INFO - [Variable] Lock: 8BE4DF61-93CA-11D2-AA0D-00E098032B8C:PlatformLangCodes Success
INFO - [Variable] Lock: 8BE4DF61-93CA-11D2-AA0D-00E098032B8C:LangCodes Success
INFO - [Variable] Lock: 8BE4DF61-93CA-11D2-AA0D-00E098032B8C:BootOptionSupport Success
INFO - [Variable] Lock: 8BE4DF61-93CA-11D2-AA0D-00E098032B8C:HwErrRecSupport Success
INFO - [Variable] Lock: 8BE4DF61-93CA-11D2-AA0D-00E098032B8C:OsIndicationsSupported Success
INFO - Variable Driver Auto Update Lang, Lang:eng, PlatformLang:en Status: Success
INFO - [Variable] Lock: 8BE4DF61-93CA-11D2-AA0D-00E098032B8C:PlatformRecovery0000 Success
INFO - PROGRESS CODE: V03051005 I0
INFO - [Variable]END_OF_DXE is signaled
INFO - Initialize variable error flag (FF)
INFO - InstallProtocolInterface: 9842073D-95D9-9F49-BD3F-2E29525125DF 4412AB90
INFO - InstallProtocolInterface: 09576E91-6D3F-11D2-8E39-00A0C969723B 4412AA10
INFO - InstallProtocolInterface: 453368F6-7C85-434A-A98A-72D1B7FFA926 4412AD9C
INFO - InstallProtocolInterface: 9042A9DE-23DC-4A38-96FB-7ADED080516A 4411301C
INFO - InstallProtocolInterface: 387477C1-69C7-11D2-8E39-00A0C969723B 4411302C
INFO - InstallProtocolInterface: 31878C87-0B75-11D5-9A4F-0090273FC14D 44113038
INFO - InstallProtocolInterface: DD9E7534-7762-4698-8C14-F58517A625AA 44113490
INFO - uga: cannot connect to X server
INFO - GraphicsConsole video resolution 800 x 600
INFO - Graphics - Mode 0, Column = 80, Row = 25
INFO - Graphics - Mode 1, Column = 0, Row = 0
INFO - Graphics - Mode 2, Column = 100, Row = 31
INFO - Graphics Console Started, Mode: -1
INFO - InstallProtocolInterface: 387477C2-69C7-11D2-8E39-00A0C969723B 44129D9C
INFO - InstallProtocolInterface: D3B36F2C-D551-11D4-9A46-0090273FC14D 0
INFO - InstallProtocolInterface: 9042A9DE-23DC-4A38-96FB-7ADED080516A 441931E8
INFO - InstallProtocolInterface: 982C298B-F4FA-41CB-B838-77AA688FB839 441931CC
INFO - Segmentation fault (core dumped)

@tianocore-issues
Copy link
Author

Comment 12214

Date: 2020-04-14 21:40:22 +0000
From: nobody <>

Guomin: please check it.

@tianocore-issues
Copy link
Author

Comment 12408

Date: 2020-05-05 23:19:30 +0000
From: guomin.jiang

Sorry for replying it till now, i will check it this week.

@tianocore-issues
Copy link
Author

Comment 12472

Date: 2020-05-11 00:46:07 +0000
From: guomin.jiang

I am afraid that i have no time for it until August.

@tianocore-issues
Copy link
Author

Comment 13043

Date: 2020-07-05 11:09:25 +0000
From: Brian Delgado <<brian.delgado>>

Disabling compiler optimizations worked around this issue for me (although my crash was elsewhere).

This link mentions the workaround: https://edk2.groups.io/g/discuss/topic/67981202#76

I added -O0 as shown below and it loaded properly.


!include NetworkPkg/Network.dsc.inc

[BuildOptions]

Disable deprecated APIs.

__*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES -O0

@tianocore-issues
Copy link
Author

Comment 13283

Date: 2020-07-31 20:38:28 +0000
From: guomin.jiang

(In reply to Brian Delgado from comment #4)
> Disabling compiler optimizations worked around this issue for me (although
> my crash was elsewhere).
>
> This link mentions the workaround:
> https://edk2.groups.io/g/discuss/topic/67981202#76
>
> I added -O0 as shown below and it loaded properly.
>
> ------------
> !include NetworkPkg/Network.dsc.inc
>
> [BuildOptions]
> #
> # Disable deprecated APIs.
> #
> __*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES -O0

It is no helpful, but still thank yo.

@tianocore-issues
Copy link
Author

Comment 18353

Date: 2022-02-14 01:51:17 +0000
From: guomin.jiang

Will check it.

@tianocore-issues
Copy link
Author

Comment 18421

Date: 2022-02-21 22:42:48 +0000
From: guomin.jiang

sent change https://edk2.groups.io/g/devel/message/86842 for review

Thanks
Guomin

@tianocore-issues
Copy link
Author

Comment 18471

Date: 2022-03-01 00:14:51 +0000
From: guomin.jiang

Sent change for review:

[edk2-devel] [PATCH 0/2] Avoid Emulator Segmentation fault: https://edk2.groups.io/g/devel/message/86842

[edk2-devel] [PATCH 1/2] MdeModulePkg/GraphicsConsoleDxe: Check status to make sure no error: https://edk2.groups.io/g/devel/message/86843

[edk2-devel] [PATCH 2/2] EmulatorPkg/EmuGopDxe: Set ModeInfo after Open successfully: https://edk2.groups.io/g/devel/message/86844

@tianocore-issues
Copy link
Author

Comment 18472

Date: 2022-03-01 04:26:01 +0000
From: guomin.jiang

Sent V2 change

[edk2-devel] [PATCH v2 0/2] Avoid Emulator Segmentation fault: https://edk2.groups.io/g/devel/message/87138

[edk2-devel] [PATCH v2 1/2] MdeModulePkg/GraphicsConsoleDxe: Check status to make sure no error: https://edk2.groups.io/g/devel/message/87139

[edk2-devel] [PATCH v2 2/2] EmulatorPkg/EmuGopDxe: Set ModeInfo after Open successfully: https://edk2.groups.io/g/devel/message/87140

Thanks

@tianocore-issues
Copy link
Author

Comment 18512

Date: 2022-03-02 19:52:51 +0000
From: guomin.jiang

Sent V3 change

[edk2-devel] [PATCH v3 0/2] Avoid Emulator Segmentation fault: https://edk2.groups.io/g/devel/message/87190

[edk2-devel] [PATCH v3 1/2] MdeModulePkg/GraphicsConsoleDxe: Check status to make sure no error: https://edk2.groups.io/g/devel/message/87191

[edk2-devel] [PATCH v3 2/2] EmulatorPkg/EmuGopDxe: Set ModeInfo after Open successfully: https://edk2.groups.io/g/devel/message/87192

Thanks
Guomin

@tianocore-issues
Copy link
Author

Comment 18524

Date: 2022-03-03 04:53:17 +0000
From: guomin.jiang

Merged

b422b0f...dc39554

Guomin
Thanks

@tianocore-issues
Copy link
Author

Comment 18526

Date: 2022-03-03 04:59:23 +0000
From: guomin.jiang

*** Bug 2639 has been marked as a duplicate of this bug. ***

leiflindholm added a commit to leiflindholm/edk2 that referenced this issue Feb 3, 2025
We've migrated bugzilla to github, so update the references to specific
bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

Signed-off-by: Leif Lindholm <[email protected]>
leiflindholm added a commit to leiflindholm/edk2 that referenced this issue Feb 3, 2025
We've migrated bugzilla to github, so update the references to specific
bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

Signed-off-by: Leif Lindholm <[email protected]>
leiflindholm added a commit to leiflindholm/edk2 that referenced this issue Feb 3, 2025
We've migrated bugzilla to github, so update the references to specific
bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

Signed-off-by: Leif Lindholm <[email protected]>
leiflindholm added a commit to leiflindholm/edk2 that referenced this issue Feb 3, 2025
We've migrated bugzilla to github, so update the references to specific
bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

Signed-off-by: Leif Lindholm <[email protected]>
leiflindholm added a commit to leiflindholm/edk2 that referenced this issue Feb 3, 2025
We've migrated bugzilla to github, so update the references to specific
bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

Signed-off-by: Leif Lindholm <[email protected]>
leiflindholm added a commit to leiflindholm/edk2 that referenced this issue Feb 3, 2025
We've migrated bugzilla to github, so update the references to specific
bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

Signed-off-by: Leif Lindholm <[email protected]>
leiflindholm added a commit to leiflindholm/edk2 that referenced this issue Feb 3, 2025
We've migrated bugzilla bugs to github issues, so update the references
to specific bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

And drop bugzilla.tianocore.org from "references".
The server is no longer in use, and the certificate has expired,
so stop telling people to go there.

Signed-off-by: Leif Lindholm <[email protected]>
mergify bot pushed a commit that referenced this issue Feb 4, 2025
We've migrated bugzilla bugs to github issues, so update the references
to specific bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
#8453
#8458

And drop bugzilla.tianocore.org from "references".
The server is no longer in use, and the certificate has expired,
so stop telling people to go there.

Signed-off-by: Leif Lindholm <[email protected]>
niruiyu pushed a commit to niruiyu/edk2 that referenced this issue Feb 19, 2025
(cherry picked from commit 258f2d1)

We've migrated bugzilla bugs to github issues, so update the references
to specific bugs called out in the top-level readme.

Also delete the references to BZ2661 and BZ2668 which are both *closed*:
tianocore#8453
tianocore#8458

And drop bugzilla.tianocore.org from "references".
The server is no longer in use, and the certificate has expired,
so stop telling people to go there.

Signed-off-by: Leif Lindholm <[email protected]>
(cherry picked from commit 258f2d1)

Change-Id: Ic99ba325907b9fd94bc15e71c46886f406e78a95
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:emulatorpkg priority:medium Moderate impact. Should be prioritized over lower priority issues. type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant