<feed xmlns='http://www.w3.org/2005/Atom'>
<title>efi-boot-shim.git/include/console.h, branch rolling</title>
<subtitle> (mirror of https://github.com/vyos/efi-boot-shim.git)
</subtitle>
<id>https://git.amelek.net/vyos/efi-boot-shim.git/atom?h=rolling</id>
<link rel='self' href='https://git.amelek.net/vyos/efi-boot-shim.git/atom?h=rolling'/>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/'/>
<updated>2025-02-24T20:22:28+00:00</updated>
<entry>
<title>Save the debug and error logs in mok-variables</title>
<updated>2025-02-24T20:22:28+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2025-02-19T16:04:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=d972515e608e4898c4a34a5895648135d4ac5aec'/>
<id>urn:sha1:d972515e608e4898c4a34a5895648135d4ac5aec</id>
<content type='text'>
This changes our debug and error logging to save the entire logs into
mok-variables as "shim-dbg.txt" and "shim-log.txt".

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
<entry>
<title>test: Make our fake dprintf be a statement.</title>
<updated>2023-06-23T21:12:58+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2023-04-28T18:47:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=f132655f458035aa2ed6b9e3a5ae4c04b13c9311'/>
<id>urn:sha1:f132655f458035aa2ed6b9e3a5ae4c04b13c9311</id>
<content type='text'>
In a few places we put dprintf() at places where the compiler will get
confused if it isn't a block or a statement.

Obviously, it should be a statement, so this makes it one.

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
<entry>
<title>Rename 'msecs' to 'usecs' to avoid potential confusion</title>
<updated>2023-06-21T18:19:51+00:00</updated>
<author>
<name>Kamil Aronowski</name>
<email>kamil.aronowski@yahoo.com</email>
</author>
<published>2023-04-12T16:50:12+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=549d34691d68518e55c2edd6e759b19de7f8ddef'/>
<id>urn:sha1:549d34691d68518e55c2edd6e759b19de7f8ddef</id>
<content type='text'>
The function msleep uses gBS-&gt;Stall which waits for a specified number
of microseconds.

Reference: https://edk2-docs.gitbook.io/edk-ii-uefi-driver-writer-s-guide/5_uefi_services/51_services_that_uefi_drivers_commonly_use/517_stall

This reference even mentions an example sleeping for 10 microseconds: // Wait 10 uS. Notice the letter 'u'.

Therefore it's a good idea to call the function 'usleep' rather than
'msleep', so no one confuses it with milliseconds, and to change the
argument name to match as well.

Signed-off-by: Kamil Aronowski &lt;kamil.aronowski@yahoo.com&gt;
</content>
</entry>
<entry>
<title>Skip testing msleep()</title>
<updated>2023-06-21T18:18:25+00:00</updated>
<author>
<name>Kamil Aronowski</name>
<email>kamil.aronowski@yahoo.com</email>
</author>
<published>2023-05-08T07:28:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=f7a4338f1b5ef03dca83ce44075e9d6e5897e037'/>
<id>urn:sha1:f7a4338f1b5ef03dca83ce44075e9d6e5897e037</id>
<content type='text'>
In preparation for renaming msleep() to usleep(), in some cases tests
were failing due to a mismatch between our declaration of the usleep()
function and what is being provided by unistd.h. This change simply
makes our function declared only when not in a unit test environment.

Signed-off-by: Kamil Aronowski &lt;kamil.aronowski@yahoo.com&gt;
</content>
</entry>
<entry>
<title>console: add a clear_screen() primitive</title>
<updated>2021-10-12T14:40:13+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2021-09-16T20:46:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=35ca373d20fbeeb80aff2202077d614bc89575c0'/>
<id>urn:sha1:35ca373d20fbeeb80aff2202077d614bc89575c0</id>
<content type='text'>
Several places in e.g. MokManager and our console library use
ST-&gt;ConOut-&gt;ClearScreen directly, without checking for the existence of
a console output device.

This patch adds function to our console library to do that correctly,
instead of using the bug-prone ad hoc implementation everywhere.

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
<entry>
<title>tests: Disable some logging when SHIM_UNIT_TEST is enabled</title>
<updated>2021-09-07T21:05:04+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2021-08-16T21:24:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=e5a406b4655f75bb1a691acc73a18ac69cc87cb0'/>
<id>urn:sha1:e5a406b4655f75bb1a691acc73a18ac69cc87cb0</id>
<content type='text'>
None of this should ever actually get called when we're running any of
the unit tests we've got, but some older compilers (i.e. Centos 7's gcc)
fail to remove some of the intermediate functions, and that causes a
link error with the functions they call.

This patch makes the top level call go away as well, so that the
intermediates never have linkage to the underlying implementation
functions.

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
<entry>
<title>More va_* work</title>
<updated>2021-03-12T09:15:01+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2021-03-11T21:48:44+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=4457d79ce0ea638e7732f5529bf13849e290940d'/>
<id>urn:sha1:4457d79ce0ea638e7732f5529bf13849e290940d</id>
<content type='text'>
Be much more explicit about exactly which va_* stuff comes from which
ABI in both shim and gnu-efi.  This fixes the problem where we see:

| (null):0:(null)() v-&gt;name:"(null)" v-&gt;rtname:"(null)"
| (null):0:(null)() v-&gt;data_size:0 v-&gt;data:0x0

and similar messages where everything is NULL.

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
<entry>
<title>Add some test cases, and make "make test" actually work.</title>
<updated>2021-03-10T20:54:20+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2021-03-09T16:56:31+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=bbdfa72a0a5f8d5a8dd4a47e67195504a22ece5b'/>
<id>urn:sha1:bbdfa72a0a5f8d5a8dd4a47e67195504a22ece5b</id>
<content type='text'>
Note the one test case I'm not 100% sure about.  Someone let me know.

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
<entry>
<title>Fix stdarg to work the same everywhere.</title>
<updated>2021-03-10T20:54:20+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2021-03-09T19:40:03+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=9beca885c29c77bb901547321a5ce6fd3c9c8ee3'/>
<id>urn:sha1:9beca885c29c77bb901547321a5ce6fd3c9c8ee3</id>
<content type='text'>
This gets us the same working definition for VA_* va_* etc everywhere,
and it's the same definition edk2 is using.

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
<entry>
<title>Restructure our includes.</title>
<updated>2021-03-10T20:54:20+00:00</updated>
<author>
<name>Peter Jones</name>
<email>pjones@redhat.com</email>
</author>
<published>2021-03-09T16:42:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.amelek.net/vyos/efi-boot-shim.git/commit/?id=f033a1da9f4c3acf7e3dfef906d01e348b6fcf42'/>
<id>urn:sha1:f033a1da9f4c3acf7e3dfef906d01e348b6fcf42</id>
<content type='text'>
This re-structures our includes so we can be sure everything is always
including all the system headers in a uniform, predictable way.

Temporarily it also adds a bunch of junk at all the places we use
variadic functions to specifically pick either the MS (cdecl) or ELF
ABIs.

I'm not 100% sure that's all correct (see later patch) but it's enough
to allow this to build.

Signed-off-by: Peter Jones &lt;pjones@redhat.com&gt;
</content>
</entry>
</feed>
