<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<br>
<div class="moz-cite-prefix">Le 18/05/2022 à 09:42, Abhishek Singh
Tomar a écrit :<br>
</div>
<blockquote type="cite"
cite="mid:20220518074304.42497-1-abhishek@linux.ibm.com">
<pre class="moz-quote-pre" wrap="">These patches contains self test for PLDM. These self test
are not hardware dependent as they bypass hardware dependent code.
They tries to test codeflow for following PLDM message types
1. PLDM FILE I/O
2. PLDM BIOS
3. PLDM PLATFORM
4. PLDM FRU
This patch set should be applied on top of these previous sets:
- Implement virtual flash content using PLDM.
- Import external libraries for MCTP/PLDM protocols.
- Implement MCTP and PLDM features.
- Complete PLDM responder and enable PLDM support
</pre>
</blockquote>
<br>
<font size="2">Hi Abhishek,<br>
<br>
I think we should go further with the tool PLDM test. For the time<br>
being, PLDM test "just" responds to some Skiboot PLDM requests<br>
and the responses <span class="VIiyi" lang="en"><span
class="JLqJ4b ChMk0b" data-language-for-alternatives="en"
data-language-to-translate-into="fr" data-phrase-index="0"
data-number-of-phrases="1"><span class="Q4iAWc">are modeled on
the Skiboot requests so it is<br>
difficult to fault the host code.<br>
<br>
So, </span></span></span><span class="VIiyi" lang="en"><span
class="JLqJ4b ChMk0b" data-language-for-alternatives="en"
data-language-to-translate-into="fr" data-phrase-index="0"
data-number-of-phrases="1"><span class="Q4iAWc"><span
class="VIiyi" lang="en"><span class="JLqJ4b ChMk0b"
data-language-for-alternatives="en"
data-language-to-translate-into="fr"
data-phrase-index="0" data-number-of-phrases="1"><span
class="Q4iAWc">it would be, perhaps, more judicious,
among others, to look<br>
at the model of test hiomap. The idea </span></span></span></span></span></span></font><span
class="VIiyi" lang="en"><span class="JLqJ4b ChMk0b"
data-language-for-alternatives="en"
data-language-to-translate-into="fr" data-phrase-index="0"
data-number-of-phrases="1"><span class="Q4iAWc"><span
class="VIiyi" lang="en"><span class="JLqJ4b ChMk0b"
data-language-for-alternatives="en"
data-language-to-translate-into="fr" data-phrase-index="0"
data-number-of-phrases="1"><span class="Q4iAWc"><span
class="VIiyi" lang="en"><span class="JLqJ4b ChMk0b
C1N51c" data-language-for-alternatives="en"
data-language-to-translate-into="fr"
data-phrase-index="0" data-number-of-phrases="1"><span
class="Q4iAWc"><font size="2">is to have a more
open<br>
model which, in addition to what you did, also
tests for error cases.<br>
<br>
Thanks</font><br>
<br>
</span></span></span></span></span></span></span></span></span>
<blockquote type="cite"
cite="mid:20220518074304.42497-1-abhishek@linux.ibm.com">
<pre class="moz-quote-pre" wrap="">
Signed-off-by: Abhishek Singh Tomar <a class="moz-txt-link-rfc2396E" href="mailto:abhishek@linux.ibm.com"><abhishek@linux.ibm.com></a>
Abhishek Singh Tomar (9):
core/pldm/test : Implement PLDM self test common api
core/pldm/test : Initialize pldm file I/O test
core/pldm/test : Write file with PLDM file I/O message
core/pldm/test : Read file with PLDM file I/O message
core/pldm/test : Implement pldm bios specification test
core/pldm/test : Initialize pldm platform specification test
core/pldm/test : Implement pldm platform restart and power off request
test
core/pldm/test : Implement PLDM FRU table self test
core/pldm/test : Retrieve the BMC information with the FRU PLDM
message
core/pldm/pldm.h | 2 +-
core/pldm/test/Makefile.check | 46 +++
core/pldm/test/common/test-pldm-common.c | 191 +++++++++++
core/pldm/test/test-pldm-bios.c | 289 +++++++++++++++++
core/pldm/test/test-pldm-fileio.c | 321 +++++++++++++++++++
core/pldm/test/test-pldm-fru.c | 327 +++++++++++++++++++
core/pldm/test/test-pldm-platform.c | 386 +++++++++++++++++++++++
7 files changed, 1561 insertions(+), 1 deletion(-)
create mode 100644 core/pldm/test/Makefile.check
create mode 100644 core/pldm/test/common/test-pldm-common.c
create mode 100644 core/pldm/test/test-pldm-bios.c
create mode 100644 core/pldm/test/test-pldm-fileio.c
create mode 100644 core/pldm/test/test-pldm-fru.c
create mode 100644 core/pldm/test/test-pldm-platform.c
</pre>
</blockquote>
<br>
</body>
</html>