On Mon, 2020-06-01 at 01:27 +0300, Timo Lindfors wrote:
> On Mon, 1 Jun 2020, Timo Lindfors wrote:
> > printk(TBOOT_INFO"start=%p tag_type=%d start->type=%d start->size=%d\n",
> >       start,
> >       tag_type,
> >       start->type,
> >       start->size);
> 
> On warm boot this prints just
> 
> TBOOT: start=0x0x10008 tag_type=17 start->type=3031684 start->size=-2147418113
> TBOOT: start=0x0x80020008 tag_type=17 start->type=0 start->size=0
> 

That looks like memory corruption... Does it work when you remove all
SINITs except the good one?

Could you please apply following patch and send me a log?

One more test - please remove 'memory' option in 'logging' parameter
from TBOOT command line in grub.cfg and check if that helps.

Thanks,
Lukasz

diff -r 1f912c52b1cc tboot/common/loader.c
--- a/tboot/common/loader.c     Sat May 23 20:32:48 2020 +0300
+++ b/tboot/common/loader.c     Mon Jun 01 17:17:01 2020 +0200
@@ -1907,10 +1907,11 @@
         return;
     } else {
         struct mb2_tag *start = (struct mb2_tag *)(lctx->addr + 8);
-        printk(TBOOT_INFO"MB2 dump, size %d\n", *(uint32_t *)lctx->addr);
+        printk(TBOOT_INFO"MB2 dump, size %d addr %p\n", *(uint32_t 
*)lctx->addr,
+               lctx->addr);
         while (start != NULL){
-            printk(TBOOT_INFO"MB2 tag found of type %d size %d ", 
-                   start->type, start->size);
+            printk(TBOOT_INFO"MB2 tag found of type %d size %d addr %p ", 
+                   start->type, start->size, start);
             switch (start->type){
             case MB2_TAG_TYPE_CMDLINE:
             case MB2_TAG_TYPE_LOADER_NAME:
@@ -1924,6 +1925,8 @@
                 {
                     struct mb2_tag_module *ts = 
                         (struct mb2_tag_module *) start;
+                    printk(TBOOT_INFO"mod_start 0x%x, mod_end 0x%x ",
+                           ts->mod_start, ts->mod_end);
                     printk_long(ts->cmdline);
                 }
                 break;
diff -r 1f912c52b1cc tboot/common/tboot.c
--- a/tboot/common/tboot.c      Sat May 23 20:32:48 2020 +0300
+++ b/tboot/common/tboot.c      Mon Jun 01 17:17:01 2020 +0200
@@ -369,6 +369,8 @@
        print_loader_ctx(g_ldr_ctx);
     */
 
+    print_loader_ctx(g_ldr_ctx);
+
     /* clear resume vector on S3 resume so any resets will not use it */
     if ( !is_launched() && s3_flag )        
set_s3_resume_vector(&_tboot_shared.acpi_sinfo, 0);





_______________________________________________
tboot-devel mailing list
tboot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tboot-devel

Reply via email to