Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New assembler crashes on M1 in x64 #96

Closed
haraldsteinlechner opened this issue Aug 25, 2022 · 14 comments
Closed

New assembler crashes on M1 in x64 #96

haraldsteinlechner opened this issue Aug 25, 2022 · 14 comments

Comments

@haraldsteinlechner
Copy link
Member

Seems new after updates, in PRo3D on Mac M1 running in rosetta we get:
Screenshot 2022-08-25 at 09 09 07

@krauthaufen this is a new assembler which got active, right? could one switch to real ARM64 yet? pro3d-space/PRo3D#197

@krauthaufen
Copy link
Member

So far everything worked on ARM64, so I think we could give it a try

@haraldsteinlechner
Copy link
Member Author

haraldsteinlechner commented Aug 25, 2022

it is a lot of work to get native arm64 though (pro3d side, other libs etc). could be step 2. ideas why x64 is broken? it might also be broken on native x64 Macs?

@haraldsteinlechner
Copy link
Member Author

seems like eiswriteable is not available in jitmem. is this just a wrong version being used or something?

@krauthaufen
Copy link
Member

Looks like it, i added the entrypoint in one of the last versions

@krauthaufen
Copy link
Member

Maybe there was a broken version which got resolved at your end? Is there a newer one?

@haraldsteinlechner
Copy link
Member Author

i is 5.2.12 rendering. if there is a newer one maybe @laurafritz or @hyazinthh could upgrade. i need to hand this one over... thanks you all

@haraldsteinlechner
Copy link
Member Author

@haraldsteinlechner
Copy link
Member Author

ah and another info. on win this exact build works...

@krauthaufen
Copy link
Member

Maybe related: aardvark-platform/aardvark.docs#34

@haraldsteinlechner
Copy link
Member Author

fix for amd64 Mac is in the assembler package, not sure about deployment though

@haraldsteinlechner
Copy link
Member Author

status:

  • aardvark.assembler 0.0.7, the embedded version has eiswriteable
  • pro3d is using 0.0.7
  • yet is seems to load a native lib which is packaged in aardvark.rendering according to this output
 6: [Introspection] loaded native library jitmem.dylib from /var/folders/ls/fc80rh9d40vbtn23g92pvcjh0000gn/T/aardvark-native/Aardvark.Rendering.GL-04c7c946-bb69-d9ba-c36b-1d07d22d1550-64/libjitmem.dylib
 6: [Introspection] loaded native library jitmem from jitmem.dylib
 6: ERROR
 6:   [Client] 2: rendering faulted with System.TypeInitializationException: The type initializer for 'Aardvark.Assembler.JitMem' threw an exception.
 6:    ---> System.TypeInitializationException: The type initializer for '<StartupCode$Aardvark-Assembler>.$JitMem' threw an exception.
 6:    ---> System.EntryPointNotFoundException: Unable to find an entry point named 'eiswritable' in shared library 'jitmem'.
 6:      at Aardvark.Assembler.JitMem.eiswritable()
 6:      at <StartupCode$Aardvark-Assembler>.$JitMem..cctor()
 6:      --- End of inner exception stack trace ---
 6:      at Aardvark.Assembler.JitMem..cctor()
 6:      --- End of inner exception stack trace ---

shouldn't this one be unpacked from aardvark.assembler?� it seems to be using rendering 5.2.12. could rendering upgrade just fix this one? pro3d-space/PRo3D#227

@haraldsteinlechner
Copy link
Member Author

it seems like this aardvark.rendering version is taking jitmem.dylib from aardvark.rendering instead of aardvark.assembler - 5.3 prerelease track could fix this issue, unfortunately pro3d is blocked. what is the last eiswritable free version to downforce rendering? other option would be to backport the new assembler to non 5.3 or upgrade pro3d to 5.3

haraldsteinlechner added a commit to pro3d-space/PRo3D that referenced this issue Sep 8, 2022
@krauthaufen
Copy link
Member

The rendering dylib shouldn't exist anymore, so we can just remove it and publish a new minor rendering version?

@haraldsteinlechner
Copy link
Member Author

yes, was already done, created a new package. works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants