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

out of memory #568

Open
lastmjs opened this issue Jan 27, 2024 · 9 comments
Open

out of memory #568

lastmjs opened this issue Jan 27, 2024 · 9 comments

Comments

@lastmjs
Copy link

lastmjs commented Jan 27, 2024

First off, my JavaScript environment is very custom and different than the web or Node.js. We are running in a wasm32-wasi environment with QuickJS, using wasmedge-quickjs, and Wasmtime on the Internet Computer.

That being said, the strange thing is that I had this working about a week ago, but now it doesn't work. We've been doing a lot of low-level changes to our JS environment (we're working at the engine/platform level), but I just don't understand what could have caused this.

The simple problem is that we get an error just trying to initialize the database: out of memory

Here's our code:

import { Server } from 'azle';
import express from 'express';
import initSqlJs from 'sql.js/dist/sql-asm.js';

export default Server(async () => {
    const SQL = await initSqlJs({
        locateFile: (filename) => `/dist/${filename}`
    });

    const db = new SQL.Database();

    const app = express();

    app.get('/', (req, res) => {
        res.send('Hello world');
    });

    return app.listen();
});

We have to use asm as our environment does not have a Wasm interpreter. I've tried sql-asm-memory-growth.js as well and I get the same error. I've even gone back multiple versions to maybe 1.7.0 and the error is always there.

Any ideas on what could cause this? I've also tried adding:

            TOTAL_STACK: 4_000_000_000,
            TOTAL_MEMORY: 4_000_000_000,
            FAST_MEMORY: 4_000_000_000

to the initSqlJs options.

@lovasoa
Copy link
Member

lovasoa commented Jan 27, 2024

Do you have an easy and minimal way to reproduce your issue?

@kripken , any idea?

@lovasoa
Copy link
Member

lovasoa commented Jan 28, 2024

Are you sure /dist/${filename} is what you need? Did you check if the files were loaded correctly in your custom environment.

@lovasoa
Copy link
Member

lovasoa commented Jan 28, 2024

Compiling sqlite to js and quickjs to wasm to run one on the other strikes me as bizarre and inefficient. Why not compile sqlite to wasm directly if you have access to a wasm runtime ?

@lastmjs
Copy link
Author

lastmjs commented Jan 29, 2024

Do you have an easy and minimal way to reproduce your issue?

@kripken , any idea?

I could get you a reproduction maybe after our next release, but it's not going to be "easy" because you would have to setup our dev environment.

@lastmjs
Copy link
Author

lastmjs commented Jan 29, 2024

Are you sure /dist/${filename} is what you need? Did you check if the files were loaded correctly in your custom environment.

I don't think this matters in our case at all since we are using sql-asm.js, am I correct? I've tried changing that path and even removing it entirely and the behavior is the same.

Though I'm not really sure what that path is supposed to do, when I logged it it was looking for a .wasm file I believe, so seemed not applicable.

@lastmjs
Copy link
Author

lastmjs commented Jan 29, 2024

Compiling sqlite to js and quickjs to wasm to run one on the other strikes me as bizarre and inefficient. Why not compile sqlite to wasm directly if you have access to a wasm runtime ?

We're basically building a Node/Deno for the Internet Computer which is a Wasm runtime, and we are trying to allow developers to npm install any package and just use it. So we would like SQL.js to work without modifications.

It was working...alas this bug that I can't seem to get around now.

@kripken
Copy link
Collaborator

kripken commented Jan 29, 2024

@kripken , any idea?

Sorry, no - an internal out of memory error is too general.

I'd recommend the usual ideas, though, as before:

  • Try a debug build (-sASSERTIONS)
  • Try a sanitizers build (say, -fsanitize=undefined,address)
  • Bisect on emscripten releases

@lovasoa
Copy link
Member

lovasoa commented Jan 30, 2024

@lastmjs , what does the debug build output for you ?

@linhttbk97
Copy link

I got the same issue when upgrade from 1.6.0 -> 1.8.0 only on safari 15.6.1
on chrome, the issue is gone

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

4 participants