250569 2023-01-13 06:10:35 -0800 Terminated worker memory leak 2024-10-16 08:10:53 -0700 1 1 1 Unclassified WebKit JavaScriptCore Safari Technology Preview Unspecified Unspecified RESOLVED CONFIGURATION CHANGED https://bugs.webkit.org/show_bug.cgi?id=272813 InRadar P2 Normal --- 1 jaffathecake webkit-unassigned ahmad.saleem792 alonzakai ap bfulgham calm cdumez ivan.curic mark.lam samddenty sam.verschueren steven webkit-bug-importer youennf ysuzuki oldest_to_newest 1925320 0 jaffathecake 2023-01-13 06:10:35 -0800 Not sure this is the correct component. This bug is something to do with workers/js/wasm. 1. Go to https://safari-worker-memory-leak-demo.netlify.app 2. Click "draw using threads", and wait for the result. Repeat this step around 20 times. I start getting "rangeerror: Out of memory". Each run of "draw using threads" spins up a worker, which in turn starts other workers. The root worker is terminated once the operation is complete. I wonder if the inner workers are not being correctly collected when the parent worker terminates. 1925416 1 ahmad.saleem792 2023-01-13 12:04:25 -0800 Is this an issue STP160 or before? Because this landed - https://github.com/WebKit/WebKit/commit/0987e578a6a8813156f74ebbf01490e4d31aa8f8 Related to workers termination, which can cause this? 1925430 2 ahmad.saleem792 2023-01-13 13:08:05 -0800 (In reply to Jake Archibald from comment #0) > Not sure this is the correct component. This bug is something to do with > workers/js/wasm. > > 1. Go to https://safari-worker-memory-leak-demo.netlify.app > 2. Click "draw using threads", and wait for the result. Repeat this step > around 20 times. > > I start getting "rangeerror: Out of memory". > > Each run of "draw using threads" spins up a worker, which in turn starts > other workers. The root worker is terminated once the operation is complete. > > I wonder if the inner workers are not being correctly collected when the > parent worker terminates. Just to update - I can reproduce this in Safari Technology Preview 161 and get 'Unhandled Promise Rejection: RangeError: Out of memory' in Console. 1925659 3 jaffathecake 2023-01-15 05:26:01 -0800 Might be noteworthy: refreshing the page doesn't seem to release the memory either. 1926877 4 webkit-bug-importer 2023-01-20 06:11:17 -0800 <rdar://problem/104473516> 1927313 5 ahmad.saleem792 2023-01-22 13:23:45 -0800 (In reply to Ahmad Saleem from comment #1) > Is this an issue STP160 or before? Because this landed - > https://github.com/WebKit/WebKit/commit/ > 0987e578a6a8813156f74ebbf01490e4d31aa8f8 > > Related to workers termination, which can cause this? I did local revert on WebKit trunk by undoing changes of this commit and build WebKit release and use Comment 0 test case but I still got "rangeerror: Out of memory", so I think this change is not culprit or regression (if this is regression). Just wanted to update my testing result since I was conscious of my change, so I wanted to confirm, it is not caused by it. 1941488 6 sam.verschueren 2023-03-15 01:23:23 -0700 Just want to step in here to say that I see the exact same behaviour when testing StackBlitz WebContainers (https://blog.stackblitz.com/posts/introducing-webcontainers/) in the latest Safari Technology Preview. We spin up quite some worker (in combination with wasm) and after running some projects and refreshing the page, I also see the `RangeError: Out of Memory`. Just like Jake indicated, refreshing the page does not release the memory and I basically have to close the browser entirely. When the error happens, and I try to create a memory object in the DevTools console like this, it also throws. > new WebAssembly.Memory({initial:10, maximum:65535, shared:true}) However, if I set the `maximum` to something much lower like `10`, it works. So I tried constructing those memory objects in a loop and see if it crashes at some point, but I was able to easily create 300.000 of those objects. 1949843 7 samddenty 2023-04-19 14:10:19 -0700 Until a fix for this lands, this can be worked around by doing the following: 1. Create a cloudflare worker with the following: ```js export default { async fetch(request, env) { let url try { url = new URL(decodeURIComponent(new URL(request.url).search.slice(1))) } catch { return new Response("must provide a url", { status: 404 }) } url.searchParams.set('memory-freed', 1) return Response.redirect(url, 302); } } ``` 2. Server-side: On the page using WASM & Workers, if the URL doesn't contain the query string `memory-freed` do a 302 redirect to `https://WORKER_NAME.USER.workers.dev/?${url_with_query_string}` 3. Worker with above code will redirect back to page with `memory-freed=1` in the URL 4. Server-side: use middleware that detects the `memory-freed=1` in requests, and inject at the top of the document the following: ``` <script>{ const url = new URL(location.href); url.searchParams.delete('memory-freed'); history.replaceState(null, '', url) }</script> ``` 5. TADA, this bug is worked around! 2009267 8 ivan.curic 2024-01-31 04:48:04 -0800 (In reply to Jake Archibald from comment #0) > Not sure this is the correct component. This bug is something to do with > workers/js/wasm. > > 1. Go to https://safari-worker-memory-leak-demo.netlify.app > 2. Click "draw using threads", and wait for the result. Repeat this step > around 20 times. > > I start getting "rangeerror: Out of memory". @Jake, could you share the source code for this? On iPhone 13, running iOS 16.7.1 I am unable to run even a single render using 4 threads. After booting up https://rreverser.com/wasm-bindgen-rayon-demo/ in another tab (which doesn't exhibit this issue), your demo runs out of memory during the wasm load. Might be related to https://bugs.webkit.org/show_bug.cgi?id=222097 2009269 9 jaffathecake 2024-01-31 04:53:49 -0800 I agree that https://bugs.webkit.org/show_bug.cgi?id=222097 sounds the same. Ugh, sorry, but I don't have access to the source code. It was just a quick hack on a laptop I no longer own. 2042865 10 ysuzuki 2024-06-24 20:31:50 -0700 Hi! Thanks, I tried it on STP and no longer reproduces. 2068376 11 ivan.curic 2024-10-16 06:47:44 -0700 This issue is still present in 18.0.1: https://www.youtube.com/watch?v=4W_ScbuM0HI 2068395 12 mark.lam 2024-10-16 08:10:53 -0700 (In reply to Ivan Čurić from comment #11) > This issue is still present in 18.0.1: > > https://www.youtube.com/watch?v=4W_ScbuM0HI Ivan, if you believe there is still an issue, please file a new bug and include details of: 1. What the issue is. 2. Steps to reproduce the issue, and what are the telltale signs of the issue manifesting (actual behavior) vs not (expected behavior). Posting a video in a bug that is already resolved will probably not get any action because (1) this bug is already resolved, and (2) no one else knows what you're intending to show with that video.