Bug 1284259

yeah seems like that the newer smbios format is the issue, seabios.git doesn't show any signs of handling it. I mailed the seabios list about it

Is anyone working on a solution?

I don't think so. For reference here was the thread:

http://www.seabios.org/pipermail/seabios/2015-November/010028.html

Created attachment 1113947 [details]
Machine UUID format match - SeaBIOS 1.9

Created attachment 1113948 [details]
Machine UUID format match - SeaBIOS 1.8

Created attachment 1113950 [details]
SeaBIOS 1.9 UUID OK


diff --git a/src/fw/biostables.c b/src/fw/biostables.c
index cb74396..52d7730 100644
--- a/src/fw/biostables.c
+++ b/src/fw/biostables.c
@@ -312,9 +312,9 @@ display_uuid(void)
                    "-%02x%02x"
                    "-%02x%02x"
                    "-%02x%02x%02x%02x%02x%02x\n"
-                   , uuid[ 0], uuid[ 1], uuid[ 2], uuid[ 3]
-                   , uuid[ 4], uuid[ 5]
-                   , uuid[ 6], uuid[ 7]
+                   , uuid[ 3], uuid[ 2], uuid[ 1], uuid[ 0]
+                   , uuid[ 5], uuid[ 4]
+                   , uuid[ 7], uuid[ 6]
                    , uuid[ 8], uuid[ 9]
                    , uuid[10], uuid[11], uuid[12], uuid[13], uuid[14], uuid[15]);
             return;

We can't unconditionally change it, the format is dependent on the smbios version. I sent a patch:

http://www.seabios.org/pipermail/seabios/2016-January/010311.html

(In reply to poma from comment #6)
> Created attachment 1113950 [details]
> SeaBIOS 1.9 UUID OK
> 
> 
> diff --git a/src/fw/biostables.c b/src/fw/biostables.c
> index cb74396..52d7730 100644
> --- a/src/fw/biostables.c
> +++ b/src/fw/biostables.c
> @@ -312,9 +312,9 @@ display_uuid(void)
>                     "-%02x%02x"
>                     "-%02x%02x"
>                     "-%02x%02x%02x%02x%02x%02x\n"
> -                   , uuid[ 0], uuid[ 1], uuid[ 2], uuid[ 3]
> -                   , uuid[ 4], uuid[ 5]
> -                   , uuid[ 6], uuid[ 7]
> +                   , uuid[ 3], uuid[ 2], uuid[ 1], uuid[ 0]
> +                   , uuid[ 5], uuid[ 4]
> +                   , uuid[ 7], uuid[ 6]
>                     , uuid[ 8], uuid[ 9]
>                     , uuid[10], uuid[11], uuid[12], uuid[13], uuid[14],
> uuid[15]);
>              return;


Reported-by: poma <poma>
Tested-by: poma <poma>
Suggested-by: poma <poma>

"biostables: Support SMBIOS 2.6+ UUID format"
http://code.coreboot.org/p/seabios/source/commit/2e11d58

(In reply to poma from comment #9)
> "biostables: Support SMBIOS 2.6+ UUID format"
> http://code.coreboot.org/p/seabios/source/commit/2e11d58

Signed-off-by: Cole Robinson <crobinso>

13) Using Reported-by:, Tested-by:, Reviewed-by:, Suggested-by: and Fixes:

~~~

"The Reported-by tag gives credit to people who find bugs and report them and it
hopefully inspires them to help us again in the future."

= hopefully inspires them to help us again in the future =

~~~

"A Tested-by: tag indicates that the patch has been successfully tested (in
some environment) by the person named.  This tag informs maintainers that
some testing has been performed, provides a means to locate testers for
future patches, and ensures credit for the testers."

= informs maintainers that some testing has been performed =

~~~

"A Suggested-by: tag indicates that the patch idea is suggested by the person
named and ensures credit to the person for the idea. Please note that this
tag should not be added without the reporter's permission, especially if the
idea was not posted in a public forum. That said, if we diligently credit our
idea reporters, they will, hopefully, be inspired to help us again in the
future."

= credit our idea reporters, they will, hopefully, be inspired to help us again in the future =

~~~

Ref.
https://www.kernel.org/doc/Documentation/SubmittingPatches

Thank you very much for your support!

I apologize for not adding a Reported-by tag, it wasn't deliberate, I just didn't think of it.

I'm not going to add Tested-by tags on someone else's behalf. If you want to report your test results, I suggest responding to the actual mail thread in the future. Aside from that you never indicated you actually tested the patch that went upstream

Suggested-by is partly applicable here since you helpfully identified the original seabios bug, but you didn't suggest the actual committed solution which takes more elements into account. So that's a toss up

There's also the problem that you don't report your real name. Probably doesn't matter for these extra meta tags, but at least qemu rejects aliases for Signed-off-by: . Something to consider

If you want to ensure full proper credit in the future, I suggest engaging directly with upstream since you obviously have the capabilities to write and and test patches. That said I will try to be more thoughtful when similar situations arise in the future and give proper credit

Setting to POST (meaning there's a patch available upstream)

After a whole discussion upstream, guess what has been done, nooooothiiiiing.

Therefore, because the original patch arises from here,
simple "thanks" will do, man.

Thank you for filing a helpful bug report and for pushing this along

Pushing 1.9.1 to f24, so reassigning this bug. F23 and earlier aren't on seabios 1.9 series

seabios-1.9.1-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-30f49486cc

seabios-1.9.1-1.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-30f49486cc

seabios-1.9.1-1.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.