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

mkdwarfs 0.5.0 crashes at creating images #39

Closed
Kron4ek opened this issue Apr 5, 2021 · 6 comments
Closed

mkdwarfs 0.5.0 crashes at creating images #39

Kron4ek opened this issue Apr 5, 2021 · 6 comments
Labels
bug Something isn't working
Milestone

Comments

@Kron4ek
Copy link

Kron4ek commented Apr 5, 2021

Unlike the previous time, now it crashes with any compression level.

** Aborted at 1617652815 (Unix time, try 'date -d @1617652815') ***
*** Signal 4 (SIGILL) (0x4fd686) received by PID 9296 (pthread TID 0x2c0e1c0) (linux TID 9296) (code: illegal operand), stack trace: ***
Illegal instruction (core dumped)

mkdwarfs-0.5.0-coredump.zip

This is with the static 0.5.0 binary from the releases page. Manually compiled dynamically linked build works fine.

@mhx mhx added the bug Something isn't working label Apr 6, 2021
@mhx
Copy link
Owner

mhx commented Apr 6, 2021

Thanks again for the report!

I wish I had one of those CPUs for testing...

This time it's not as obvious as before. The routine in which it crashes is not fancy at all. I reckon the crash goes away if you specifiy --pack-metadata=none.

Could you please try this version that's built with gcc instead of clang? mkdwarfs-0.5.0-gcc.gz

@mhx
Copy link
Owner

mhx commented Apr 6, 2021

Ah, don't worry about testing the new binary (it likely won't work), I think I've found the issue. A new library I'm using sets -march=native in its Makefile and that's obviously a bad idea for statically linked binaries...

@Kron4ek
Copy link
Author

Kron4ek commented Apr 6, 2021

The new binary crashes too. And you are right, --pack-metadata=none fixes the crash.

@mhx
Copy link
Owner

mhx commented Apr 6, 2021

Okay, could you please give mkdwarfs-0.5.1pre.gz a try before I'm pushing a new release?

@mhx mhx added this to the v0.5.1 milestone Apr 6, 2021
@Kron4ek
Copy link
Author

Kron4ek commented Apr 6, 2021

This build works without issues, thank you.

@mhx
Copy link
Owner

mhx commented Apr 6, 2021

Thanks for your help! New release is out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants