interp: ptrtoint integer size does not equal pointer size

[niaow]

I think this error was supposed to be used internally for rollback, but it seems that it replicates on AVR:

package main

import "fmt"

func printOne() {
        fmt.Println("one")
}

func main() {
        f := printOne
        f()
}

This example is pulled from #349 which I was checking on.

[niaow@finch tinygo]$ build/tinygo run -target arduino /tmp/bleh.go
error: interp: ptrtoint integer size does not equal pointer size

[dgryski]

Is this the issue that was mentioned in the patch for the conservative collector here: f870ed5 ?

[niaow]

No, it is not. Although idk if the patch might impact it.

[dgryski]

Is this patch then still something I should create a PR for?

diff --git a/src/runtime/gc_conservative.go b/src/runtime/gc_conservative.go
index cabd41427..faa4091a5 100644
--- a/src/runtime/gc_conservative.go
+++ b/src/runtime/gc_conservative.go
@@ -410,7 +410,7 @@ func markRoots(start, end uintptr) {
 		}
 	}
 
-	for addr := start; addr < end; addr += unsafe.Alignof(addr) {
+	for addr := start; addr+unsafe.Sizeof(unsafe.Pointer(nil)) <= end; addr += unsafe.Alignof(addr) {
 		root := *(*uintptr)(unsafe.Pointer(addr))
 		markRoot(addr, root)
 	}

[niaow]

I do not know. I will need to look into that again. That should not be related to this issue.

[niaow]

So the answer there is that yes that code still needs to be fixed but also my fix had a bug in it because markRoots already had another load-bearing bug.

[niaow]

For future reference, that patch has been turned into #2396

[dgryski]

This issue is also happening when trying to build https://github.com/google/open-location-code/, and I'm pretty sure that's a recent failure.

[aykevl]

This issue is most likely unrelated to any runtime GC that might happen. The interp package intercepts calls to runtime.alloc and replaces them with creating a new global.

I think this error was supposed to be used internally for rollback

No, what it actually shows is that there is a bug somewhere in the interp package. It is usually caused when something stores a pointer in a non-pointer field (such as int, uintptr, [4]byte, etc). It might be fixable by passing a correct layout parameter to runtime.alloc: there are many places where this isn't yet done (for example, in llvmutil.EmitPointerPack).

[niaow]

While working on #2458, I found that this can be triggered by testdata/json.go, testdata/stdlib.go, and testdata/testing.go on AVR.

[markoxley]

Not sure if this will help, but I had a similar issue until I removed all fmt and log references from my code

[dgryski]

(This is no longer crashing open-location-code/go either wasi or native.)

[dkegel-fastly]

#2565 includes a regression test for this, but doesn't run it yet from Makefile, as it is still failing.