AdGuardVPNForWindows icon indicating copy to clipboard operation
AdGuardVPNForWindows copied to clipboard

High RAM consumption

Open QuiXMaDe opened this issue 1 year ago • 9 comments

Please answer the following questions for yourself before submitting an issue.

  • [X] I checked the knowledge base and found no answer
  • [X] I checked to make sure that this issue has not already been filed

AdGuard VPN version

2.4.0 (1475)

Environment

- OS: Windows 11 (23H2)
- Other network-level software: no

Which DNS server do you use?

Default

Custom DNS

No response

Use Wintun

  • [ ] yes, I do

Issue Details

Steps to reproduce:

  1. Starting Adguard with connecting to Stockholm
  2. Watching YouTube via Yandex.Browser
  3. YouTube stops loading videos and the program starts "eating" RAM
  4. All system crashing with next error: Application pop-up: Windows - Low Virtual Memory : The system is running low on virtual memory. To ensure that Windows continues to function properly, increase the size of the virtual memory paging file. For more information, see the built-in help. P.S. RAM: 16GB; Pagefile: 8GB+16GB

Expected Behavior

No response

Actual Behavior

All applications and the system are crashing

Screenshots

Screenshot 1

image image

Additional Information

No response

QuiXMaDe avatar Sep 13 '24 19:09 QuiXMaDe

Hi @QuiXMaDe,

Unfortunately, we were unable to reproduce this issue on our end. Could you please try to record the application logs? We need them to diagnose and troubleshoot this issue.

Here's what we need you to do:

  1. Go to Settings → App settings → Logging level → Record everything.
  2. Reproduce the problem, then remember the exact time it happened.
  3. Open Support tab → Export logs and system info.
  4. Send the archive to [email protected]: - include [windows] keyword and ISSUE_NUMBER in the subject of your email - specify the exact time when the issue occurred

AlexandrPkhm avatar Sep 16 '24 09:09 AlexandrPkhm

@QuiXMaDe Could you please let us know if you were able to reproduce the issue and record the logs?

AlexandrPkhm avatar Sep 19 '24 09:09 AlexandrPkhm

@AlexandrPkhm, Sent a letter to the address you provided. image

QuiXMaDe avatar Sep 19 '24 10:09 QuiXMaDe

@QuiXMaDe We need a memory dump of the affected processes for further investigation. To create a memory dump, please follow the steps described in our Knowledge Base article - How to create a dump file, but instead of the Adguard.exe process, please select the AdGuard VPN process, which consumes RAM.

Please send the generated dump to [email protected] and make sure to include [windows] keyword and ISSUE_NUMBER in the subject of your email.

AlexandrPkhm avatar Sep 26 '24 14:09 AlexandrPkhm

@AlexandrPkhm I don't know whether the client provided you with a memory dump or not, but in my case (issue #782) the problem was cured by removing the Google search engine from the exceptions. When you add the Google search engine to the exceptions, it adds about 400 domains. I suspect that the service in this case does not have time to filter every request and starts mindlessly caching something.

NotSoWell avatar Nov 20 '24 15:11 NotSoWell

@QuiXMaDe, hi! Could you please try this build and let us know if it helps? https://uploads.adguard.org/AdGuardVpn-v2.4.1622.2.exe

vankos avatar Dec 09 '24 10:12 vankos

I have same issue with consumption and google exception too By comment suggestion ill remove it and watch consumption Usually crash happens after awake from sleep mode or using youtube I have full logs (if needed) P.S.: works perfectly

Screenshot

2024-12-19 16 20 24

entasi avatar Dec 19 '24 16:12 entasi

+1 To this one. Will remove google.com from the exclusions to try and see if it fixes anything. In my case it's happening sometimes right after I start the PC

tabakaevga avatar Dec 27 '24 10:12 tabakaevga

Update: After 3 weeks of getting into high RAM consumption occasionally it now does it regularly, no matter the exclusions. Up to 5 times a day. Any ETA on the fix?

P.S. I've done some decompiling of the App DLLs with Jetbrains dotPeek (nothing criminal here - it doesn't show actual implementation source code, only interfaces) and it appears that a C# List is used for exclusion storage. I hope it gets converted to a HashSet at some point in the implementation before the need for filtering because if it's searching through a List then it's probably too slow for some other task and that's where it leaks probably

tabakaevga avatar Jan 17 '25 15:01 tabakaevga

Hi everyone, Could you please try last nightly? There have been some changes in the native libraries that might potentially fix the problem.

vankos avatar May 14 '25 12:05 vankos

I am closing the issue. If the high consumption occurs again, feel free to reopen it or create a new one.

vankos avatar May 26 '25 13:05 vankos