-
Notifications
You must be signed in to change notification settings - Fork 91
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
Serial console is not re-connected after SerialIo tests (Bugzilla Bug 3866) #229
Comments
Comment 18597Date: 2022-03-11 06:54:26 +0000
SCT disconnects serial console (TerminalDxe) before running SerialIo tests and does not connect it back once execution as completed. As a result, it's impossible to run SCT via the serial console. |
Comment 18777Date: 2022-04-07 10:19:18 +0000 Hi Felix, |
Comment 18889Date: 2022-04-21 18:11:02 +0000 (In reply to Edhay from comment #1) Edhay, The logs look fine, no errors, warnings or anything like this is reported. |
Comment 18933Date: 2022-05-05 10:13:44 +0000 Hi Felix, |
Comment 18948Date: 2022-05-11 04:55:56 +0000 Hi Barton, In SCT tool source, before performing EfiSerialIoProtocol test ,SCT code invokes OpenProtocol() on SerialIo Handle with EXCLUSIVE attribute. As Terminal driver already opened SerialIo protocol with BY_DRIVER attribute, it will get disconnected as EXCLUSIVE attribute is passed for OpenProtocol(). After completion of test, CloseProtocol() is invoked but no attempt made to reconnect the SerialIo Handle. After closing the protocol, Serial I/O protocol handle should be connected recursively. Same issue was reproduced with PciSioSerialDxe & TerminalDxe from EDK2 source. SCT code: Referred edk2-test-edk2-test-stable202108 source code. |
Comment 19022Date: 2022-06-02 10:20:05 +0000 Thank you Sundaresan for your inputs. |
Comment 19929Date: 2022-11-03 10:55:07 +0000 @sundaresan: Can AMI provide a patch for the solution? |
Comment 21761Date: 2023-07-21 09:52:52 +0000 Created attachment 1385
Attachment: BZ-1385-SCT_SerialIo.zip |
Comment 21762Date: 2023-07-21 09:54:55 +0000 Comment on attachment 1385 Attached the changes to save the driver binding handle that binds the SerialIo handle before SCT tools exclusive take ownership of the SerialIo handle. After SerialIo test, passed the saved driver binding handle to ConnectController in order to connect back serial Io handle to the old driver. Unit Test: Verified after SCT serial Io test, SerialIo handle got connected to the Terminal driver and verified redirection works fine. Attachment: BZ-1385-SCT_SerialIo.zip |
Comment 22518Date: 2024-02-04 13:31:18 +0000 Thank you, Gayathri for the patch and verification results. Could you please send the patch in https://edk2.groups.io/g/devel for review. |
Comment 22657Date: 2024-02-27 01:11:12 +0000 Hi Gayathri, Could you please provide the same. Please refer to https://github.com/tianocore/tianocore.github.io/wiki/Laszlo%27s-unkempt-git-guide-for-edk2-contributors-and-maintainers |
This issue was created automatically with bugzilla2github
Bugzilla Bug 3866
Date: 2022-03-11T06:54:26+00:00
From: Felix <>
To: Gayathri Thunuguntla <>
CC: edhaya.chandran, gaojie, gayathrit, sundaresans
Last updated: 2024-02-27T01:11:12+00:00
The text was updated successfully, but these errors were encountered: