Re: [edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
Hi Robert, Thank you for the confirmation. The patch is upstreamed through the below commit: https://github.com/tianocore/edk2-test/commit/1d181ad0d82520c099435ff08a8e829b44f493cc With Warm Regards, Edhay -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96307): https://edk2.groups.io/g/devel/message/96307 Mute This Topic: https://groups.io/mt/94007106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
Hi G Edhaya, Thank you for taking the time to review this patch and issue. I have tested the changes you have made and they pass locally and resolve the issues we are experiencing with this test. Thanks, Robert Wood -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96172): https://edk2.groups.io/g/devel/message/96172 Mute This Topic: https://groups.io/mt/94007106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
Hi Robert, In the present patch the verification of elapsed time is failing. The patch will need additional changes as below (bolded): diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c index ad72646b..f3d189bd 100644 --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c @@ -827,7 +827,7 @@ BBTestStallInterfaceTest ( StartTime = Epoch; OldTpl = gtBS->RaiseTPL (TplArray[Index]); Status = gtBS->Stall ( - 1000 + 400^M ); gtBS->RestoreTPL (OldTpl); if (gtRT->GetTime (, NULL) != EFI_SUCCESS) @@ -845,7 +845,7 @@ BBTestStallInterfaceTest ( (Index == 1? \ gMiscBootServicesBBTestFunctionAssertionGuid021: \ gMiscBootServicesBBTestFunctionAssertionGuid022), - L"BS.Stall - 10 seconds", *+ L"BS.Stall - 4 seconds",^M* L"%a:%d:Status - %r, TPL - %d", __FILE__, (UINTN)__LINE__, @@ -853,8 +853,8 @@ BBTestStallInterfaceTest ( TplArray[Index] ); SecondsElapsed = GetSecondsElapsed (, ); - if ((SecondsElapsed <= 10 + MAX_SECOND_MARGIN) && - (SecondsElapsed >= 10 - MAX_SECOND_MARGIN)) { *+ if ((SecondsElapsed <= 4 + MAX_SECOND_MARGIN) &&^M* *+ (SecondsElapsed >= 4 - MAX_SECOND_MARGIN)) {^M* AssertionType = EFI_TEST_ASSERTION_PASSED; } else { AssertionType = EFI_TEST_ASSERTION_FAILED; diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h index 9e98ec01..4f8eaa4c 100644 --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h @@ -46,7 +46,7 @@ typedef struct _RESET_DATA { { 0xA6033499, 0xE4AF, 0x44f5, {0x9D, 0x16, 0x30, 0x78, 0xD8, 0x61, 0x32, 0x28 }} #define TPL_ARRAY_SIZE 3 -#define MAX_SECOND_MARGIN 2 +#define MAX_SECOND_MARGIN 1^M // // Change size from TPL_ARRAY_SIZE to TPL_ARRAY_SIZE + 1 I have verified the new changes and it executes fine. Also uploaded the changes to bugzilla 4105 – SctPkg: Stall_Func test can cause disk timeouts and testing failure (tianocore.org) ( https://bugzilla.tianocore.org/show_bug.cgi?id=4105 ) You could please also check and reply. With Warm Regards, Edhay -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96129): https://edk2.groups.io/g/devel/message/96129 Mute This Topic: https://groups.io/mt/94007106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
Thank you Samer for creating the ticket on Bugzilla. The next step by the forum is to build SCT with the patch at devel@edk2.groups.io | [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func ( https://edk2.groups.io/g/devel/topic/94007106#94539 ) and run regression on three reference hardwares. Once verified, we will upstream the patch. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#95864): https://edk2.groups.io/g/devel/message/95864 Mute This Topic: https://groups.io/mt/94007106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
BZ ticket created: https://bugzilla.tianocore.org/show_bug.cgi?id=4105 Patch was at: https://edk2.groups.io/g/discuss/message/1115 From: devel@edk2.groups.io On Behalf Of G Edhaya Chandran via groups.io Sent: Thursday, October 6, 2022 10:37 AM To: Robert Wood ; devel@edk2.groups.io Subject: Re: [edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func On Fri, Sep 30, 2022 at 05:28 AM, Robert Wood wrote: MiscBootServicesBBTestFunction.c Hi Robert, Can you please also raise a Bugzilla ticket for this issue here: Bug List (tianocore.org)<https://bugzilla.tianocore.org/buglist.cgi?component=UEFI-SCT_id=22841=EDK2%20Test=---> Please do attach the failure logs in the ticket. This issue was discussed in the forum. May we know how the value of 400 was arrived for the new Stall value? With Warm Regards, Edhay IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#95861): https://edk2.groups.io/g/devel/message/95861 Mute This Topic: https://groups.io/mt/94007106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
On Fri, Sep 30, 2022 at 05:28 AM, Robert Wood wrote: > > MiscBootServicesBBTestFunction.c Hi Robert, Can you please also raise a Bugzilla ticket for this issue here: Bug List (tianocore.org) ( https://bugzilla.tianocore.org/buglist.cgi?component=UEFI-SCT_id=22841=EDK2%20Test=--- ) Please do attach the failure logs in the ticket. This issue was discussed in the forum. May we know how the value of 400 was arrived for the new Stall value? With Warm Regards, Edhay -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94794): https://edk2.groups.io/g/devel/message/94794 Mute This Topic: https://groups.io/mt/94007106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
The Stall_Func test on the highest TPL causes issues with the disk IO by blocking interrupts. This blocking can cause disk corruption through IO timeouts. Since this doesn't seem to be the intent of the test this revision reduces the stall interval from 10 seconds to 4 and adjusts the delta tolerance in scale. Signed-off-by: Robert Wood Cc: Samer El-Haj-Mahmoud Cc: G Edhaya Chandran Cc: Barton Gao Cc: Carolyn Gjertsen --- uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h | 2 +- uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h index 9e98ec013c74..4f8eaa4c70ea 100644 --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h @@ -46,7 +46,7 @@ typedef struct _RESET_DATA { { 0xA6033499, 0xE4AF, 0x44f5, {0x9D, 0x16, 0x30, 0x78, 0xD8, 0x61, 0x32, 0x28 }} #define TPL_ARRAY_SIZE 3 -#define MAX_SECOND_MARGIN 2 +#define MAX_SECOND_MARGIN 1 // // Change size from TPL_ARRAY_SIZE to TPL_ARRAY_SIZE + 1 diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c index ad72646bada2..f831ed6fbccc 100644 --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c @@ -827,7 +827,7 @@ BBTestStallInterfaceTest ( StartTime = Epoch; OldTpl = gtBS->RaiseTPL (TplArray[Index]); Status = gtBS->Stall ( - 1000 + 400 ); gtBS->RestoreTPL (OldTpl); if (gtRT->GetTime (, NULL) != EFI_SUCCESS) -- 2.38.0.rc1.362.ged0d419d3c-goog -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94757): https://edk2.groups.io/g/devel/message/94757 Mute This Topic: https://groups.io/mt/94137117/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[edk2-devel] [PATCH v1 1/1] MiscBootServices: Stall_Func: Reduces the stall interval for Stall_Func
The Stall_Func test on the highest TPL causes issues with the disk IO by blocking interrupts. This blocking can cause disk corruption through IO timeouts. Since this doesn't seem to be the intent of the test this revision reduces the stall interval from 10 seconds to 4 and adjusts the delta tolerance in scale. Signed-off-by: Robert Wood Cc: Samer El-Haj-Mahmoud --- uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h | 2 +- uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h index 9e98ec013c74..4f8eaa4c70ea 100644 --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestMain.h @@ -46,7 +46,7 @@ typedef struct _RESET_DATA { { 0xA6033499, 0xE4AF, 0x44f5, {0x9D, 0x16, 0x30, 0x78, 0xD8, 0x61, 0x32, 0x28 }} #define TPL_ARRAY_SIZE 3 -#define MAX_SECOND_MARGIN 2 +#define MAX_SECOND_MARGIN 1 // // Change size from TPL_ARRAY_SIZE to TPL_ARRAY_SIZE + 1 diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c index ad72646bada2..f831ed6fbccc 100644 --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MiscBootServices/BlackBoxTest/MiscBootServicesBBTestFunction.c @@ -827,7 +827,7 @@ BBTestStallInterfaceTest ( StartTime = Epoch; OldTpl = gtBS->RaiseTPL (TplArray[Index]); Status = gtBS->Stall ( - 1000 + 400 ); gtBS->RestoreTPL (OldTpl); if (gtRT->GetTime (, NULL) != EFI_SUCCESS) -- 2.38.0.rc1.362.ged0d419d3c-goog -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94539): https://edk2.groups.io/g/devel/message/94539 Mute This Topic: https://groups.io/mt/94007106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-