On Tue, 2018-11-06 at 17:46 +0000, Leif Lindholm wrote: > On Tue, Nov 06, 2018 at 02:17:41PM +0530, Sakar Arora wrote: > > From: John Powell <[email protected]> > > > > Tests SBBR requirements 3.4.1 and 3.4.4. It would be better if we add comments on how SBBR requirements 3.4.1 and 3.4.4 relate to the UEFI specification. Since this is being requested to be added as part of UEFI-SCT, it would need to be related to UEFI specification.
For all the SBBR patches, please correlate SBBR specification text to UEFI specification, so that it can be justified as UEFI-SCT test case. > > > > Change-Id: I0dd625851db65d04d2a7572a2d6838951c8f7c67 > > Signed-off-by: John Powell <[email protected]> > > --- > > uefi-sct/SctPkg/CommonGenFramework.sh | 3 + > > uefi-sct/SctPkg/Config/Data/Category.ini | 11 +- > > .../SbbrBootServices/BlackBoxTest/Guid.c | 65 +++ > > .../SbbrBootServices/BlackBoxTest/Guid.h | 76 +++ > > .../BlackBoxTest/SbbrBootServicesBBTest.inf | 85 ++++ > > .../BlackBoxTest/SbbrBootServicesBBTestFunction.c | 549 > > +++++++++++++++++++++ > > .../BlackBoxTest/SbbrBootServicesBBTestFunction.h | 85 ++++ > > .../BlackBoxTest/SbbrBootServicesBBTestMain.c | 149 ++++++ > > .../BlackBoxTest/SbbrBootServicesBBTestMain.h | 99 ++++ > > uefi-sct/SctPkg/UEFI/SBBR_SCT.dsc | 7 + > > 10 files changed, 1127 insertions(+), 2 deletions(-) > > create mode 100644 uefi- > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBox > > Test/Guid.c > > create mode 100644 uefi- > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBox > > Test/Guid.h > > create mode 100644 uefi- > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBox > > Test/SbbrBootServicesBBTest.inf > > create mode 100644 uefi- > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBox > > Test/SbbrBootServicesBBTestFunction.c > > create mode 100644 uefi- > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBox > > Test/SbbrBootServicesBBTestFunction.h > > create mode 100644 uefi- > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBox > > Test/SbbrBootServicesBBTestMain.c > > create mode 100644 uefi- > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBox > > Test/SbbrBootServicesBBTestMain.h > > > > diff --git a/uefi-sct/SctPkg/CommonGenFramework.sh b/uefi- > > sct/SctPkg/CommonGenFramework.sh > > index aecd21b..ab35c5a 100755 > > --- a/uefi-sct/SctPkg/CommonGenFramework.sh > > +++ b/uefi-sct/SctPkg/CommonGenFramework.sh > > @@ -364,6 +364,9 @@ then > > # Tests SBBR sections 3.3.1, 3.3.2, 3.3.3, and 3.3.4 > > cp > > $ProcessorType/SysEnvConfigBBTest.efi $Framework/ > > Test/ > NUL > > > > + # UEFI Boot Services - SBBR v1.0 3.4 > > + cp > > $ProcessorType/SbbrBootServicesBBTest.efi $Framework/ > > Test/ > NUL > > + > > # Runtime Services Tests - SBBR v1.0 Appendix B > > cp > > $ProcessorType/VariableServicesBBTest.efi $Framework/ > > Test/ > NUL > > cp > > $ProcessorType/TimeServicesBBTest.efi $Framework/ > > Test/ > NUL > > diff --git a/uefi-sct/SctPkg/Config/Data/Category.ini b/uefi- > > sct/SctPkg/Config/Data/Category.ini > > index 9576971..17ca639 100644 > > --- a/uefi-sct/SctPkg/Config/Data/Category.ini > > +++ b/uefi-sct/SctPkg/Config/Data/Category.ini > > @@ -145,8 +145,15 @@ Description = > > > > [Category Data] > > Revision = 0x00010000 > > -CategoryGuid = AFF115FB-387B-4C18-8C41-6AFC7F03BB90 > > -InterfaceGuid = AFF115FB-387B-4C18-8C41-6AFC7F03BB90 > > +CategoryGuid = 8dafd7a7-44d0-4cf4-9d28-9f87080fc091 > > +InterfaceGuid = E9EF7553-F833-4E56-96E8-38AE679523CC > > +Name = BootServicesTest\SbbrBootServices > > +Description = Tests SBBR boot services requirements. > > + > > +[Category Data] > > +Revision = 0x00010000 > > +CategoryGuid = AFF115FB-387B-4c18-8C41-6AFC7F03BB90 > > +InterfaceGuid = AFF115FB-387B-4c18-8C41-6AFC7F03BB90 > > Name = RuntimeServicesTest > > Description = > > Diff would be more clear if added test was added after existing test. > > / > Leif _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
