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.

Sign up for GitHub

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

Jump to bottom

Windows x86 UPBGE crashing at start #797

Closed
joelgomes1994 opened this issue Aug 19, 2018 · 5 comments
Closed

Windows x86 UPBGE crashing at start #797

joelgomes1994 opened this issue Aug 19, 2018 · 5 comments
Labels
bug

Comments

@joelgomes1994
Copy link

Hello! Windows build process is successful at both x86 and x64, but the x86 build won't start, printing only EXCEPTION_ACCESS_VIOLATION on the console when Blender starts. I tried to rebuild from scratch, but the problem persists.

I will try to revert some commits to see where this problem starts. In the meantime, could someone confirm the issue? Thanks.
@lordloki
Copy link
Member

lordloki commented Aug 20, 2018 •
edited
Loading

Confirmed.
Crash inside of gpu_nodes_tag function.
image

@lordloki lordloki added the bug label Aug 20, 2018
@panzergame
Copy link
Contributor

@lordloki : What kind of file are you using ?

@lordloki
Copy link
Member

Default cube

@joelgomes1994 joelgomes1994 mentioned this issue Sep 17, 2018
Closed
@lordloki lordloki mentioned this issue Sep 24, 2018
Closed
@lordloki
Copy link
Member

Joel, can you test this build? https://mega.nz/#!cttHlRiY!NNf4b8NO9k7tFUySVdP4K39Zd6iGUmupDG8qokfNrG4

It is from branch ge_bug_fix_797.
For my it is working

@joelgomes1994
Copy link
Author

@lordloki I just built the branch you said, it's working for me as well.

lordloki added a commit that referenced this issue Oct 4, 2018
@lordloki
UPBGE: Fix for #797
09aca70 
Set GPU_INT to 5 avoid buffer overrun
lordloki added a commit that referenced this issue Oct 6, 2018
@lordloki
UPBGE: Fix for #797
637bee8 
Introduce new array of GPUType sizes to make it extensible and adding more
comments too.
lordloki added a commit that referenced this issue Oct 9, 2018
@lordloki
UPBGE: Fix for #797
d438f1c 
Introduce a new array of GPUType sizes to avoid a buffer overrun.
The new array will allow make the system extensible easier.
More comments added.

P.D. It fixes the win32 crash
lordloki added a commit that referenced this issue Oct 9, 2018
@lordloki
UPBGE: Fix for #797 (#869)
683c104 
Introduce a new array of GPUType sizes to avoid a buffer overrun when we introduce new elements in GPUType.
The new array will allow make the system extensible easier.
More comments added.

P.D. It fixes the win32 crash.
@joelgomes1994 joelgomes1994 mentioned this issue Oct 10, 2018
Closed
lordloki added a commit that referenced this issue Oct 14, 2018
@lordloki
UPBGE: Additional fix for #797
f169853 
We use GPU_DATATYPE_SIZE to avoid pass info from outside of array
youle31 pushed a commit that referenced this issue May 26, 2019
@lordloki @youle31
UPBGE: Fix for #797 (#869)
c79a076 
Introduce a new array of GPUType sizes to avoid a buffer overrun when we introduce new elements in GPUType.
The new array will allow make the system extensible easier.
More comments added.

P.D. It fixes the win32 crash.
youle31 pushed a commit that referenced this issue May 26, 2019
@lordloki @youle31
UPBGE: Additional fix for #797
688b5ca 
We use GPU_DATATYPE_SIZE to avoid pass info from outside of array
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lordloki @panzergame @joelgomes1994