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

9.0 binaries exit with "Illegal instruction" on Windows 10 #15

Closed
jedfrechette opened this issue Sep 19, 2016 · 4 comments
Closed

9.0 binaries exit with "Illegal instruction" on Windows 10 #15

jedfrechette opened this issue Sep 19, 2016 · 4 comments

Comments

@jedfrechette
Copy link

The version 9.0 binaries for PoissonRecon and SSDRecon from:

http://www.cs.jhu.edu/~misha/Code/PoissonRecon/Version9.0

crash on Windows 10 with an Illegal Instruction. After fixing issue #14 locally I'm able to produce a debug build that does not crash and runs as expected. However, my local release builds are crashing as well. The exception is thrown at line 241 in Ply.h.

@mkazhdan
Copy link
Owner

Can you provide some more information.

I ran the binaries on my machine and things seemed OK.

Do you get this problem if you run the executable without any parameters?

If this is input-dependent, would you be able to share the .ply file you are running with (and provide the command line arguments you are using)?

Are you running on a 32-bit or 64-bit build of Windows 10? (The executables are compiled for 64-bit and should not work on a 32-bit OS.)

-- Misha

From: jedfrechette [mailto:notifications@github.com]
Sent: Monday, September 19, 2016 4:25 PM
To: mkazhdan/PoissonRecon PoissonRecon@noreply.github.com
Subject: [mkazhdan/PoissonRecon] 9.0 binaries exit with "Illegal instruction" on Windows 10 (#15)

The version 9.0 binaries for PoissonRecon and SSDRecon from:

http://www.cs.jhu.edu/~misha/Code/PoissonRecon/Version9.0 http://www.cs.jhu.edu/%7Emisha/Code/PoissonRecon/Version9.0

crash on Windows 10 with an Illegal Instruction. After fixing issue #14 #14 locally I'm able to produce a debug build that does not crash and runs as expected. However, my local release builds are crashing as well. The exception is thrown at line 241 in Ply.h.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #15 , or mute the thread https://github.com/notifications/unsubscribe-auth/AKXc5bBVudIeLpW7TW7qqiVsLEnf5KkPks5qru-ogaJpZM4KA7K0 . https://github.com/notifications/beacon/AKXc5RnZjJy_POZFa6RiX2OCsqYe8zInks5qru-ogaJpZM4KA7K0.gif

@jedfrechette
Copy link
Author

I tested with the same result on two machines, both are 64-bit Windows 10 and have vcredist 2015 installed.Running without parameters doesn't make a difference, both binaries still crash immediately. Notably, the SurfaceTrimmer from both your distribution and my local build runs fine.Best, --Jed Frechette From: notifications@github.comSent: September 19, 2016 6:26 PMTo: PoissonRecon@noreply.github.comReply-to: reply@reply.github.comCc: jedfrechette@gmail.com; author@noreply.github.comSubject: Re: [mkazhdan/PoissonRecon] 9.0 binaries exit with "Illegal instruction" on Windows 10 (#15) Can you provide some more information.

I ran the binaries on my machine and things seemed OK.

Do you get this problem if you run the executable without any parameters?

If this is input-dependent, would you be able to share the .ply file you are running with (and provide the command line arguments you are using)?

Are you running on a 32-bit or 64-bit build of Windows 10? (The executables are compiled for 64-bit and should not work on a 32-bit OS.)

-- Misha

From: jedfrechette [mailto:notifications@github.com]
Sent: Monday, September 19, 2016 4:25 PM
To: mkazhdan/PoissonRecon PoissonRecon@noreply.github.com
Subject: [mkazhdan/PoissonRecon] 9.0 binaries exit with "Illegal instruction" on Windows 10 (#15)

The version 9.0 binaries for PoissonRecon and SSDRecon from:

http://www.cs.jhu.edu/~misha/Code/PoissonRecon/Version9.0 http://www.cs.jhu.edu/%7Emisha/Code/PoissonRecon/Version9.0

crash on Windows 10 with an Illegal Instruction. After fixing issue #14 #14 locally I'm able to produce a debug build that does not crash and runs as expected. However, my local release builds are crashing as well. The exception is thrown at line 241 in Ply.h.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #15 , or mute the thread https://github.com/notifications/unsubscribe-auth/AKXc5bBVudIeLpW7TW7qqiVsLEnf5KkPks5qru-ogaJpZM4KA7K0 . https://github.com/notifications/beacon/AKXc5RnZjJy_POZFa6RiX2OCsqYe8zInks5qru-ogaJpZM4KA7K0.gif

—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

@mkazhdan
Copy link
Owner

My co-author recommends trying the following:

Under the “release” build, go to:

            “Properties” -> “C/C++” -> “Code Generation”

And switch the “Enable Enhanced Instruction Set” to “Streaming SIMD Extensions 2”.

It may be the case that the hardware does not support the more recent instruction sets. (You shouldn’t be seeing the problem in “debug” mode because enhanced instruction sets are disabled for that build.)

-- Misha

From: jedfrechette [mailto:notifications@github.com]
Sent: Monday, September 19, 2016 8:38 PM
To: mkazhdan/PoissonRecon PoissonRecon@noreply.github.com
Cc: mkazhdan misha@cs.jhu.edu; Comment comment@noreply.github.com
Subject: Re: [mkazhdan/PoissonRecon] 9.0 binaries exit with "Illegal instruction" on Windows 10 (#15)

I tested with the same result on two machines, both are 64-bit Windows 10 and have vcredist 2015 installed.Running without parameters doesn't make a difference, both binaries still crash immediately. Notably, the SurfaceTrimmer from both your distribution and my local build runs fine.Best, --Jed Frechette From: notifications@github.comSent mailto:notifications@github.comSent : September 19, 2016 6:26 PMTo: PoissonRecon@noreply.github.comReply-to mailto:PoissonRecon@noreply.github.comReply-to : reply@reply.github.comCc mailto:reply@reply.github.comCc : jedfrechette@gmail.com mailto:jedfrechette@gmail.com ; author@noreply.github.comSubject mailto:author@noreply.github.comSubject : Re: [mkazhdan/PoissonRecon] 9.0 binaries exit with "Illegal instruction" on Windows 10 (#15) Can you provide some more information.

I ran the binaries on my machine and things seemed OK.

Do you get this problem if you run the executable without any parameters?

If this is input-dependent, would you be able to share the .ply file you are running with (and provide the command line arguments you are using)?

Are you running on a 32-bit or 64-bit build of Windows 10? (The executables are compiled for 64-bit and should not work on a 32-bit OS.)

-- Misha

From: jedfrechette [mailto:notifications@github.com]
Sent: Monday, September 19, 2016 4:25 PM
To: mkazhdan/PoissonRecon <PoissonRecon@noreply.github.com mailto:PoissonRecon@noreply.github.com >
Subject: [mkazhdan/PoissonRecon] 9.0 binaries exit with "Illegal instruction" on Windows 10 (#15)

The version 9.0 binaries for PoissonRecon and SSDRecon from:

http://www.cs.jhu.edu/~misha/Code/PoissonRecon/Version9.0 http://www.cs.jhu.edu/%7Emisha/Code/PoissonRecon/Version9.0

crash on Windows 10 with an Illegal Instruction. After fixing issue #14 #14 locally I'm able to produce a debug build that does not crash and runs as expected. However, my local release builds are crashing as well. The exception is thrown at line 241 in Ply.h.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #15 , or mute the thread https://github.com/notifications/unsubscribe-auth/AKXc5bBVudIeLpW7TW7qqiVsLEnf5KkPks5qru-ogaJpZM4KA7K0 . https://github.com/notifications/beacon/AKXc5RnZjJy_POZFa6RiX2OCsqYe8zInks5qru-ogaJpZM4KA7K0.gif

—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub #15 (comment) , or mute the thread https://github.com/notifications/unsubscribe-auth/AKXc5ZHo7DqstFXoRkT1CxEItCJ16zwxks5qryr_gaJpZM4KA7K0 . https://github.com/notifications/beacon/AKXc5UyPRkiiuxqYcDv-qDB-mLWQb8j3ks5qryr_gaJpZM4KA7K0.gif

@jedfrechette
Copy link
Author

The AVX2 instruction set was the issue. Both machines I was testing on were older E5s that only supported AVX but not AVX2.

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