public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Samer El-Haj-Mahmoud" <samer.el-haj-mahmoud@arm.com>
To: "Nelson, Eric" <eric.nelson@intel.com>,
	G Edhaya Chandran <Edhaya.Chandran@arm.com>,
	"gaojie@byosoft.com.cn" <gaojie@byosoft.com.cn>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
Subject: Re: Proposing a new area of the edk2-test repository
Date: Wed, 23 Jun 2021 20:55:40 +0000	[thread overview]
Message-ID: <PAXPR08MB6987B7B9B53F3AB76184CECB90089@PAXPR08MB6987.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <BY5PR11MB3943C6BF262E40A921D20479E9089@BY5PR11MB3943.namprd11.prod.outlook.com>

[-- Attachment #1: Type: text/plain, Size: 6895 bytes --]

+edk2 list

I am not against adding additional test tools to edk2-test. Just feel like there is a need to organize and have a strategy, rather than just use edk2-test as a dumping group of miscellaneous tools.

There is already a place for apps under https://github.com/tianocore/edk2/tree/master/MdeModulePkg/Application

We also have a number of EDK2 misc applications that use edk2-libc in https://github.com/tianocore/edk2-libc/tree/master/AppPkg/Applications

A couple of questions:

  *   Do you expect more apps from WinTestPkg to be contributed to TianoCore? And are they all around testing specific Windows requirements? If so, then having an edk2-test/WinTestPkg makes sense to me, as you will have a collection of useful testing app targeting specific area.
     *   But what about other OSes?
  *   If this is a one-off test app and other WinTestPkg apps are not going to be contributed, then does it make sense to put this under MdeModulePkg/Application ?



From: Nelson, Eric <eric.nelson@intel.com>
Sent: Wednesday, June 23, 2021 3:10 PM
To: G Edhaya Chandran <Edhaya.Chandran@arm.com>; gaojie@byosoft.com.cn
Cc: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
Subject: RE: Proposing a new area of the edk2-test repository


Hi Edhay,

Do you have any more questions?
What do you think of creating another directory in edk2-test, for other test apps, in addition to uefi-sct, such as ResumeOK.efi?

Thanks,
__e


From: Nelson, Eric
Sent: Tuesday, June 15, 2021 12:00 PM
To: G Edhaya Chandran <Edhaya.Chandran@arm.com<mailto:Edhaya.Chandran@arm.com>>; gaojie@byosoft.com.cn<mailto:gaojie@byosoft.com.cn>
Cc: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com<mailto:Samer.El-Haj-Mahmoud@arm.com>>
Subject: RE: Proposing a new area of the edk2-test repository


Hi Edhay,

ResumeOK.efi is a tool I wrote from the HelloWorld example, that validates Windows resume from S4 requirements, specifically that the memory-map run-time memory regions don't change, and secondly that PCI devices don't disappear from the system, both conditions would cause Windows to fail to resume from S4.

You install the tool to the root of the ESP, and set it as the default/top entry in the boot manager, and launch it.  (Disable Secure Boot.)

It runs warm, cold, and 60s ACPI RTC wake cycles, infinitely looking for errors.

ResumeOK.efi writes a file to the root of the ESP, ResumeOK.map, which contains the ACPI Facs->HardwareSignature, a list of the PCI devices in the system, and a copy of its memory map, from the first time it runs.

During each test pass, it runs a barrage of tests:


  1.  Free memory test - does the available memory match the memory map saved in ResumeOK.map
  2.  HW signature check - does the system still have the same HW signature as saved in the ResumeOK.map
  3.  Allocation test - all the available memory is allocated, and then the memory map is checked if the run-time regions match ResumeOK.map.

If any of the tests fail, then the new/missing PCI devices are listed (HW signature fail case), or the memory descriptor that changed, it's location, and current and previous type and size.

I have received permission from Intel to *try* to release the source under Edk2-test.

I've included a 64-bit binary, if you want to give it a test drive.

Make sure Secure Boot is off.
Also, it is required to manually delete any ResumeOK.map on the ESP, before beginning a new test pass.

The tool also supports a host of EFI Shell commands:

Resumeok.efi MEMMAP - displays Windows coalesced view of the current memory map
ResumeOK.efi ROKMAP - displays Windows coalesced view of the memory saved in ResumeOK.map
ResumeOK.efi RTDATA - displays an analysis of RT_Data pool usage
ResumeOK.efi NORESET - run one test pass, but suppress automatic SX cycling

These are the files that build it:

Edk2\WinTestPkg\Application
Edk2\WinTestPkg\WinTestPkg.dec
Edk2\WinTestPkg\WinTestPkg.dsc
Edk2\WinTestPkg\Application\ResumeOK
Edk2\WinTestPkg\Application\ResumeOK\AcpiTbl.c
Edk2\WinTestPkg\Application\ResumeOK\AcpiTbl.h
Edk2\WinTestPkg\Application\ResumeOK\AppSupport.c
Edk2\WinTestPkg\Application\ResumeOK\BitMap.c
Edk2\WinTestPkg\Application\ResumeOK\BitMap.h
Edk2\WinTestPkg\Application\ResumeOK\EfiFileLib.c
Edk2\WinTestPkg\Application\ResumeOK\EfiFileLib.h
Edk2\WinTestPkg\Application\ResumeOK\pci.c
Edk2\WinTestPkg\Application\ResumeOK\Pci.h
Edk2\WinTestPkg\Application\ResumeOK\ResumeOK.c
Edk2\WinTestPkg\Application\ResumeOK\ResumeOK.h
Edk2\WinTestPkg\Application\ResumeOK\ResumeOK.inf
Edk2\WinTestPkg\Application\ResumeOK\ResumeOK.uni
Edk2\WinTestPkg\Application\ResumeOK\ResumeOKExtra.uni
Edk2\WinTestPkg\Application\ResumeOK\RtData.c
Edk2\WinTestPkg\Application\ResumeOK\TimeBaseLib.c

Thanks,
__e


From: G Edhaya Chandran <Edhaya.Chandran@arm.com<mailto:Edhaya.Chandran@arm.com>>
Sent: Monday, June 14, 2021 9:36 PM
To: Nelson, Eric <eric.nelson@intel.com<mailto:eric.nelson@intel.com>>; gaojie@byosoft.com.cn<mailto:gaojie@byosoft.com.cn>
Cc: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com<mailto:Samer.El-Haj-Mahmoud@arm.com>>
Subject: RE: Proposing a new area of the edk2-test repository

Hi Eric,

    Thanks for reaching out to us.
Can we get more details of the tool?

Is this tool already open sourced or could you send us the basic documentation pertaining to it.

With Warm Regards,
Edhay


From: Nelson, Eric <eric.nelson@intel.com<mailto:eric.nelson@intel.com>>
Sent: 15 June 2021 04:23
To: gaojie@byosoft.com.cn<mailto:gaojie@byosoft.com.cn>; G Edhaya Chandran <Edhaya.Chandran@arm.com<mailto:Edhaya.Chandran@arm.com>>
Subject: Proposing a new area of the edk2-test repository


Hello SCT maintainers,

I'm looking to release source to a UEFI validation tool that has been a big hit with platform BIOS validation teams, so it can help other PC vendors.

My coworker Michael Kinney suggested I reach out to you directly about the idea of creating a new top level directory in the edk2-test repro for other test apps, and I could be maintainer.

What do you think of creating another directory in edk2-test, for other test apps, in addition to uefi-sct?

Thanks!
__e

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.
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.

[-- Attachment #2: Type: text/html, Size: 18374 bytes --]

       reply	other threads:[~2021-06-23 20:56 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <BY5PR11MB3943EFD67AA9709C7E103315E9319@BY5PR11MB3943.namprd11.prod.outlook.com>
     [not found] ` <DB9PR08MB660293DC749327D9896AA600E8309@DB9PR08MB6602.eurprd08.prod.outlook.com>
     [not found]   ` <BY5PR11MB39433B2895E68F64E830379DE9309@BY5PR11MB3943.namprd11.prod.outlook.com>
     [not found]     ` <BY5PR11MB3943C6BF262E40A921D20479E9089@BY5PR11MB3943.namprd11.prod.outlook.com>
2021-06-23 20:55       ` Samer El-Haj-Mahmoud [this message]
2021-06-23 21:42         ` Proposing a new area of the edk2-test repository Nelson, Eric
2021-06-24  4:25           ` Bret Barkelew
2021-07-09 20:11             ` Samer El-Haj-Mahmoud
2021-07-28 19:05               ` Nelson, Eric
2021-08-03 13:09                 ` Samer El-Haj-Mahmoud
2021-08-03 15:53                   ` Bret Barkelew

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=PAXPR08MB6987B7B9B53F3AB76184CECB90089@PAXPR08MB6987.eurprd08.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox