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

Deacresed Heap Crashes (Reopen PR) #2531

Open
wants to merge 45 commits into
base: master
Choose a base branch
from

Conversation

TheFocusMan
Copy link
Contributor

Open a new one because the last one is closed
Refrence: #2488

quajak
quajak previously approved these changes Dec 22, 2022
Copy link
Member

@quajak quajak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, it looks like all kernels are failing due to some timeout.

@TheFocusMan
Copy link
Contributor Author

Thanks, it looks like all kernels are failing due to some timeout.

Yep that is the problem

@quajak
Copy link
Member

quajak commented Dec 23, 2022

Are you able to reproduce the timeouts locally?

@TheFocusMan
Copy link
Contributor Author

Are you able to reproduce the timeouts locally?

Yes the problem is in the fat test file system:
Cosmos.Kernel.Tests.Fat

@quajak
Copy link
Member

quajak commented Dec 30, 2022

Where you able to figure out what kind of error it is? Stack corruption, issue with virtual methods or unhandled exception for example?

@TheFocusMan
Copy link
Contributor Author

Where you able to figure out what kind of error it is? Stack corruption, issue with virtual methods or unhandled exception for example?

No i cannot find out what kind of error it , but According to the log it look like that it's stopped on the code of create or delete file method, Most of the chances is can be unhandled exception.

@terminal-cs
Copy link
Contributor

I fixed native memory myself in #2597 because this PR seems dead.

@selkij
Copy link
Contributor

selkij commented Apr 24, 2023

So this PR can be closed?

@terminal-cs
Copy link
Contributor

So this PR can be closed?

Not exactly, it still fixes other things. Though it's causing issues

@pleasenoban
Copy link
Contributor

is this pr still active?

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

Successfully merging this pull request may close these issues.

7 participants