x/mobile/exp/audio/al: Flaky crash on Mac OS X #15075

hajimehoshi · 2016-04-02T18:39:10Z

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version go1.6 darwin/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/hajimehoshi/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GO15VENDOREXPERIMENT="1"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fno-common"
CXX="clang++"
CGO_ENABLED="1"

What did you do?
When running a program using x/mobile/exp/audio/al, this crashes on alc.go.

In my case, when running github.com/hajimehoshi/go-inovation on Mac OS X, I sometimes get the below crash. I'm afraid I couldn't get a minimum case to reproduce this crash. Looks like there is a case where unsafe.Pointer can't accept the value 0x18.

runtime: writebarrierptr *0x45ec598 = 0x18
fatal error: bad pointer in write barrier

runtime stack:
runtime.throw(0x4337840, 0x1c)
        /usr/local/go/src/runtime/panic.go:530 +0x90
runtime.writebarrierptr.func1()
        /usr/local/go/src/runtime/mbarrier.go:140 +0xb3
runtime.systemstack(0xc82001c000)
        /usr/local/go/src/runtime/asm_amd64.s:291 +0x79
runtime.mstart()
        /usr/local/go/src/runtime/proc.go:1048

goroutine 6 [running]:
runtime.systemstack_switch()
        /usr/local/go/src/runtime/asm_amd64.s:245 fp=0xc820038b90 sp=0xc820038b88
runtime.writebarrierptr(0x45ec598, 0x18)
        /usr/local/go/src/runtime/mbarrier.go:141 +0x97 fp=0xc820038bc0 sp=0xc820038b90
golang.org/x/mobile/exp/audio/al.OpenDevice(0x0, 0x0)
        /Users/hajimehoshi/go/src/golang.org/x/mobile/exp/audio/al/alc.go:53 +0x47c fp=0xc820038c78 sp=0xc820038bc0
github.com/hajimehoshi/ebiten/exp/audio.startPlaying(0xc790000, 0xc8200fc000, 0xac44, 0xc790000, 0x0, 0x0)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/exp/audio/audio_openal.go:44 +0x36 fp=0xc820038d58 sp=0xc820038c78
github.com/hajimehoshi/ebiten/exp/audio.NewContext(0xac44, 0x4522070)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/exp/audio/audio.go:125 +0x151 fp=0xc820038e18 sp=0xc820038d58
main.initAudio.func1(0xc820016540)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/go-inovation/audio.go:33 +0x70 fp=0xc820038fa8 sp=0xc820038e18
runtime.goexit()
        /usr/local/go/src/runtime/asm_amd64.s:1998 +0x1 fp=0xc820038fb0 sp=0xc820038fa8
created by main.initAudio
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/go-inovation/audio.go:68 +0x53

goroutine 1 [runnable, locked to thread]:
github.com/hajimehoshi/ebiten/internal/graphics/opengl.(*Context).NewTexture(0xc8200e0000, 0x200, 0x800, 0xc8205fc000, 0x400000, 0x400000, 0x2600, 0x4d04458, 0xc82000ad50, 0x40b7f41)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/internal/graphics/opengl/context_desktop.go:132
github.com/hajimehoshi/ebiten/internal/graphics.NewTextureFromImage(0xc8200e0000, 0x4d04608, 0xc820010400, 0x2600, 0xc8200ff000, 0x0, 0x0)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/internal/graphics/texture.go:82 +0x3be
github.com/hajimehoshi/ebiten.NewImageFromImage(0x4d04608, 0xc820010400, 0x0, 0xc820010400, 0x0, 0x0)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/image.go:228 +0x7a
github.com/hajimehoshi/ebiten/ebitenutil.NewImageFromFile(0xc82000e880, 0x1c, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/ebitenutil/loadimage.go:37 +0x1b6
main.Run(0x0, 0x0)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/go-inovation/main.go:443 +0x30e
main.main()
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/go-inovation/main.go:457 +0x1c

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
        /usr/local/go/src/runtime/asm_amd64.s:1998 +0x1

goroutine 5 [chan receive, locked to thread]:
github.com/hajimehoshi/ebiten/internal/graphics/opengl.(*Context).Loop(0xc8200e0000)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/internal/graphics/opengl/context_desktop.go:83 +0x55
github.com/hajimehoshi/ebiten/internal/ui.Init.func1(0xc820012280, 0xc820016120)
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/internal/ui/ui_glfw.go:79 +0xe4
created by github.com/hajimehoshi/ebiten/internal/ui.Init
        /Users/hajimehoshi/go/src/github.com/hajimehoshi/ebiten/internal/ui/ui_glfw.go:80 +0x362
exit status 2

What did you expect to see?
No crash.
What did you see instead?
Crash.

The text was updated successfully, but these errors were encountered:

hajimehoshi · 2016-04-02T18:41:05Z

This might be related to #9191.

hajimehoshi · 2017-07-04T13:10:16Z

What's going on? If it is OK to use uintptr for the context and device values, I'd be glad to create a patch. However, this fix might be tricky. Does anyone have a better idea? Thanks,

davecheney · 2017-07-04T13:12:43Z

Please upgrade to Go 1.8.3

…

On Tue, Jul 4, 2017 at 11:10 PM, Hajime Hoshi ***@***.***> wrote: What's going on? If it is OK to use uintptr for the context and device values, I'd be glad to create a patch. However, this fix might be tricky. Does anyone have a better idea? Thanks, — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#15075 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA5coKRlKlc2COrDNG1VziI7h1PFhks5sKjm6gaJpZM4H-WLb> .

hajimehoshi · 2017-07-04T13:18:37Z

Ah, is this already fixed? If so, could you show me the related change to fix this issue? I'll try Go 1.8.3 anyway. Thanks.

davecheney · 2017-07-04T13:25:10Z

This, #17234, and others related to apple changing the gettimeofday syscall which causes widespread corruption with older go binaries. You need to upgrade to at least go 1.7.x which is the oldest supported release, but as long as you are doing that, upgrade to 1.8.3 so you get more than a month before we tell you to upgrade again. As a bonus you get faster gc, and more than a year's worth of bug fixes.

…

On Tue, Jul 4, 2017 at 11:18 PM, Hajime Hoshi ***@***.***> wrote: Ah, is this already fixed? If so, could you show me the related change to fix this issue? I'll try Go 1.8.3 anyway. Thanks. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#15075 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA3K1S1ebv5haqeV3wmTqDEVqed1aks5sKjuugaJpZM4H-WLb> .

hajimehoshi · 2017-07-04T13:33:24Z

OK, it looks like it is already fixed. I've tried x/mobile/exp/audio 10 times and nothing crashed.

TBH, I've already installed Go 1.8 and I've not tested this issue.

Thank you!

alistanis · 2017-07-04T13:38:53Z

Thanks @davecheney for the info on this, we've been using uintptrs on our own but it's good to know when/where this was fixed.

fastfading · 2017-07-13T02:09:15Z

I am using go 1.8.3 still crash at alcDestroyContext
just like
#17054

bradfitz added this to the Unreleased milestone Apr 7, 2016

emil10001 mentioned this issue Sep 10, 2016

x/mobile/exp/audio/al: Crash when running al.CloseDevice() on OSX #17054

Open

hajimehoshi mentioned this issue Apr 9, 2017

why not use gomobie native for ios/android hajimehoshi/ebiten#340

Closed

hajimehoshi mentioned this issue Jul 4, 2017

Fix player_ios.go to follow player_openal.go fix ebitengine/oto#18

Closed

hajimehoshi closed this as completed Jul 4, 2017

hajimehoshi mentioned this issue Dec 6, 2017

go-inovation sometimes crashes hajimehoshi/ebiten#457

Closed

golang locked and limited conversation to collaborators Jul 13, 2018

gopherbot added the FrozenDueToAge label Jul 13, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

x/mobile/exp/audio/al: Flaky crash on Mac OS X #15075

x/mobile/exp/audio/al: Flaky crash on Mac OS X #15075

hajimehoshi commented Apr 2, 2016

hajimehoshi commented Apr 2, 2016

hajimehoshi commented Jul 4, 2017

davecheney commented Jul 4, 2017 via email

hajimehoshi commented Jul 4, 2017

davecheney commented Jul 4, 2017 via email

hajimehoshi commented Jul 4, 2017

alistanis commented Jul 4, 2017

fastfading commented Jul 13, 2017

x/mobile/exp/audio/al: Flaky crash on Mac OS X #15075

x/mobile/exp/audio/al: Flaky crash on Mac OS X #15075

Comments

hajimehoshi commented Apr 2, 2016

hajimehoshi commented Apr 2, 2016

hajimehoshi commented Jul 4, 2017

davecheney commented Jul 4, 2017 via email

hajimehoshi commented Jul 4, 2017

davecheney commented Jul 4, 2017 via email

hajimehoshi commented Jul 4, 2017

alistanis commented Jul 4, 2017

fastfading commented Jul 13, 2017