-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Program memory and connection number problems #4505
Comments
We cannot reproduce this. Please fill the whole issue template and also provide information about which kinds of upstreams you're using. Also, how do you determine the amount of |
我采用的是dns分流文件 但是多人用的时候close_wait请求过多 注:单人用没有感觉,多人用才会 dns分流文件 同时我已经修改了sysctl |
Thanks for the info, we'll inspect the code and see if we leak any conns. |
这个是我自己的/etc/sysctl.conf配置文件 重要的是 我已经调整sysctl.conf文件好了一点点 |
Merge in DNS/dnsproxy from 4505-tls-conn-leak to master Updates AdguardTeam/AdGuardHome#4505. Squashed commit of the following: commit 4a4b52f Author: Eugene Burkov <E.Burkov@AdGuard.COM> Date: Thu Apr 21 15:51:47 2022 +0300 upstream: close conn on err
Did you update the binary before restarting? Because if so, your screenshot shows that the problem is fixed. |
I actually compiled the above one and ran it for an hour before restarting the program. At the beginning of the program is ok, time began to occupy the port. |
@Potterli20, @fernvenue, this is unrelated to this issue, but I've noticed you two downvoting each other and, sometimes, other posters as well. I don't know why you two do that, but could you please stop? That confuses newcomers, like in #4503, and just generally doesn't improve the quality of conversations in issues. Thanks. |
Could you look through the |
@ainar-g I have no idea, but I do upvoted for you, if that confuse you so sorry and I will stop using any emoji in this project. Edited: I have checked and removed all emojis as much as possible, my apologies for that. |
Upvotes and other reactions are fine, but is regarded as a fairly negative thing, and it's better to not use it unless you also provide a comment regarding the reason. Again, thanks for understanding. |
This question feels like a long one. My current program ADH restarts every 6 hours and DNSProxy upstream restarts every 15 minutes. I don't have a lot of data to offer, but CLOSE_Wait has a big impact on Linux. My upstream agent can be found there. |
There is a negative state itself, from this issue I do not want to talk #4316 |
Oh, right. If DNS writes udp:// protocol, TCP is preferred |
AdguardTeam/dnsproxy#230 |
@Potterli20, hello again. What exact setup are you checking? We've only pushed the fix into dnsproxy master branch, so that AGH's behavior has no changes yet. Have you also built the AGH from source with dnsproxy module replaced? Also, have you tried the dnsproxy as a single resolver? Thanks. |
It is entirely possible that different network environments uncover different bugs in our implementations. We'll keep looking for them. Thanks for all the info you're providing so far. |
I have always used DNSProxy as a pure DNS, not for AD blocking. I have been using CLOSE_wait for a few weeks and dNSproxy is used to stream files with simple DNS. |
But I am using the Chinese network and the international network are the same problem. Close_wait occurs whenever there are too many requests |
I'm actually checking for close_WAIT. The close_wait problem is a problem for Linux |
@Potterli20, to what value the |
Adh does not set Max, only cache |
@Potterli20, in AGH's configuration file there is a field called Could you please try to set both of them into |
This is affected, users will also feel stuck |
这个是我本地上游的dns,全部都在另一个机器上处理 |
Dnsproxy and ADH have a common problem, the program memory leak, request IP most close_wait leads to crazy connection, one hour connection number has gone to more than 3W, resulting in network slow down
dnsproxy和adh有个通病,程序内存泄露,请求ip大多数close_wait导致程序疯狂连接,一个小时连接数已经去到3w多,导致网络变慢
Now it's 30 minutes plus 30 watts
The text was updated successfully, but these errors were encountered: