<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/lib/efi_selftest, branch v2019.04</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/lib/efi_selftest?h=v2019.04</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/lib/efi_selftest?h=v2019.04'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2019-03-27T05:32:27Z</updated>
<entry>
<title>efi_selftest: avoid double free in dp utilities test</title>
<updated>2019-03-27T05:32:27Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-03-26T04:56:31Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=d0826507e2d24974cb99bfa8cac39d7684107994'/>
<id>urn:sha1:d0826507e2d24974cb99bfa8cac39d7684107994</id>
<content type='text'>
Avoid duplicate FreePool() in unit test for the device patch utilities
protocol.

Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
</content>
</entry>
<entry>
<title>efi_selftest: fix test_hii_string_get_string()</title>
<updated>2019-03-20T17:16:53Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-03-19T19:08:46Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=d0bd87612f410a723d5ddb3001e805485e3efb4f'/>
<id>urn:sha1:d0bd87612f410a723d5ddb3001e805485e3efb4f</id>
<content type='text'>
The check testing the string result of get_string() returned the wrong
result. The result was ignored.

Use efi_st_strcmp_16_8() for the string comparison.

Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
</content>
</entry>
<entry>
<title>efi_loader: fix build error for freestanding.o</title>
<updated>2019-03-14T20:03:14Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-03-11T21:04:51Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=e616100eab84464f09baa93b3fe5c15087a385c1'/>
<id>urn:sha1:e616100eab84464f09baa93b3fe5c15087a385c1</id>
<content type='text'>
Since commit f51a226436a87 ("efi_loader: provide freestanding library") in
parallel builds errors

    lib/efi_selftest/../efi_loader/efi_freestanding.o:
    file not recognized: File truncated

occur. Obviously make cannot correctly sequence parallel builds with a
dependency like ../efi_loader/efi_freestanding.o.

Fixes: f51a226436a87 ("efi_loader: provide freestanding library")
Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
</content>
</entry>
<entry>
<title>efi_selftest: LoadImage from file device path</title>
<updated>2019-02-16T14:42:19Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-01-09T18:04:14Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=d787caddf1ac69c3845992a435eafca3637b1fac'/>
<id>urn:sha1:d787caddf1ac69c3845992a435eafca3637b1fac</id>
<content type='text'>
Provide a unit test that calls LoadImage() with a file device path and
executes the application via StartImage().

Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
</content>
</entry>
<entry>
<title>efi_selftest: fix memory allocation in HII tests</title>
<updated>2019-02-16T14:42:19Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-02-12T20:38:02Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ee3c8ba85525cea5b004c4bd1262c75e461e3eb3'/>
<id>urn:sha1:ee3c8ba85525cea5b004c4bd1262c75e461e3eb3</id>
<content type='text'>
In efi_selftest we are in EFI land. We cannot call U-Boot library
functions malloc() and free() but should use the boot time services
instead.

Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
Reviewed-by: Alexander Graf &lt;agraf@suse.de&gt;
</content>
</entry>
<entry>
<title>efi_selftest: do not use efi_free_pool()</title>
<updated>2019-02-16T14:42:19Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-02-12T20:50:45Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=afa17aa23ffcd62b2233281b603cb95510655dce'/>
<id>urn:sha1:afa17aa23ffcd62b2233281b603cb95510655dce</id>
<content type='text'>
In efi_selftest we are in EFI land. We should not use U-Boot library
functions but boot time services for memory management.

Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
Reviewed-by: Alexander Graf &lt;agraf@suse.de&gt;
</content>
</entry>
<entry>
<title>efi_loader: Make HII a config option</title>
<updated>2019-02-13T08:40:06Z</updated>
<author>
<name>Alexander Graf</name>
<email>agraf@suse.de</email>
</author>
<published>2019-02-11T14:24:00Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=5fbb28958becc2e725d2ee14a35c3b2f0918c62f'/>
<id>urn:sha1:5fbb28958becc2e725d2ee14a35c3b2f0918c62f</id>
<content type='text'>
Heinrich ran into issues with HII and iPXE which lead to #SErrors on
his Odroid-C2 system. We definitely do not want to regress just yet,
so let's not expose the HII protocols by default.

Instead, let's make it a config option that people can play with
This way, we can stabilize the code in tree without breaking any
users.

Once someone figures out, why this breaks iPXE (probably a NULL
dereference), we can enable it by default.

Reported-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
Signed-off-by: Alexander Graf &lt;agraf@suse.de&gt;

---

v1 -&gt; v2:

  - Remove HII selftest as well

v2 -&gt; v3:

  - Make config option
</content>
</entry>
<entry>
<title>efi_selftest: fix HII tests</title>
<updated>2019-02-13T08:40:06Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-01-23T19:36:27Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f38753d2ef5dabcca78b1a70f2903e7f49d5d846'/>
<id>urn:sha1:f38753d2ef5dabcca78b1a70f2903e7f49d5d846</id>
<content type='text'>
efi_st_printf() does not support format code %ld. Anyway the format code
for size_t would be %zu which isn't supported either.

We do not want any divisions to avoid invalid references to integer
arithmetic routines, cf.
https://gcc.gnu.org/onlinedocs/gccint/Integer-library-routines.html.

As a simple remedy remove the noisy messages from the output.
They are not relevant for automated testing.

Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
Signed-off-by: Alexander Graf &lt;agraf@suse.de&gt;
</content>
</entry>
<entry>
<title>efi_selftest: add HII database protocols test</title>
<updated>2019-02-13T08:40:06Z</updated>
<author>
<name>AKASHI Takahiro</name>
<email>takahiro.akashi@linaro.org</email>
</author>
<published>2019-01-21T03:13:01Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=4c4fb10da294ab46196c6f1daf8621f2451c88a5'/>
<id>urn:sha1:4c4fb10da294ab46196c6f1daf8621f2451c88a5</id>
<content type='text'>
This efi_selftest tests HII database protocol and HII string protocol.

Signed-off-by: AKASHI Takahiro &lt;takahiro.akashi@linaro.org&gt;
Signed-off-by: Alexander Graf &lt;agraf@suse.de&gt;
</content>
</entry>
<entry>
<title>efi_loader: do not use symbolic links</title>
<updated>2019-02-13T08:40:06Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2019-01-21T18:30:42Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6c75db07f4115904a460540c8956db2087cceb1f'/>
<id>urn:sha1:6c75db07f4115904a460540c8956db2087cceb1f</id>
<content type='text'>
Symbolic links are not supported on all file systems, e.g. not on FAT. So
it is not wise to use them in our source tree.

Use a qualified path to refer to lib/efi_loader/efi_freestanding.c in
scripts/Makefile.lib instead.

Reported-by: Alexander Graf &lt;agraf@suse.de&gt;
Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
[agraf: Fix build with O=]
Signed-off-by: Alexander Graf &lt;agraf@suse.de&gt;
</content>
</entry>
</feed>
