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

Deck がメモリリークしている可能性がある #5467

Open
acid-chicken opened this issue Sep 30, 2019 · 25 comments
Open

Deck がメモリリークしている可能性がある #5467

acid-chicken opened this issue Sep 30, 2019 · 25 comments
Labels
⚠️bug? This might be a bug packages/frontend Client side specific issue/PR 🐢Performance Efficiency related issue/PR ❓needs more investigation A bug whose causes are unknown

Comments

@acid-chicken
Copy link
Member

Home Deck
2GB 13GB
image image

Environment

A Misskey fork based 10.102.4 (w/o deck behavior changes)
Firefox 70.0b10 (x86_64)

@acid-chicken acid-chicken added ❓needs more investigation A bug whose causes are unknown packages/frontend Client side specific issue/PR ⚠️bug? This might be a bug labels Sep 30, 2019
@syuilo
Copy link
Member

syuilo commented Sep 30, 2019

Vueにはメモリリークのバグがあるけれどそれかな

@syuilo
Copy link
Member

syuilo commented Sep 30, 2019

Deckに限らずメモリリークはあるけどDeckはコンポーネントの数が多いからそれが顕著に現れるものと思われる

@acid-chicken
Copy link
Member Author

わからない
とりあえず GC/CC では回収されないし、Home で一ヶ月ぐらいほっておいても元気にしてる自環が Deck にした途端数時間程度でメモリを使い切るようになった。

@syuilo
Copy link
Member

syuilo commented Sep 30, 2019

Vueが関係してるとしたら「UIの動きを減らす」オプションを有効にするとメモリリークは消えるはず

@acid-chicken
Copy link
Member Author

とりあえずしばらく試してみます

@tamaina
Copy link
Member

tamaina commented Sep 30, 2019

「UIの動きを減らす」オプションはメモリリーク対策で導入したと記憶しています。

ref https://github.com/syuilo/misskey/blob/develop/src/client/app/desktop/views/components/notes.vue#L17

@acid-chicken
Copy link
Member Author

それは知っているけど、たかがトランジション箇所が数倍に増えるだけで数時間まで縮まるのは現実的な悪化ではない。

@tamaina
Copy link
Member

tamaina commented Sep 30, 2019

でも昔からDeckはもとからこんなもんだったよ

@acid-chicken

This comment has been minimized.

@tamaina

This comment has been minimized.

@tamaina
Copy link
Member

tamaina commented Sep 30, 2019

こっちだ
Maybe related to #1746

@acid-chicken
Copy link
Member Author

それは Home の話では?

@tamaina
Copy link
Member

tamaina commented Sep 30, 2019

そう、なぜかHomeの話になってるけど原因たぶん同じ

@acid-chicken
Copy link
Member Author

それは知っているけど、たかがトランジション箇所が数倍に増えるだけで数時間まで縮まるのは現実的な悪化ではない。

なので確定ではない

@acid-chicken
Copy link
Member Author

まあとりあえず一日ぐらいお待ちください

@syuilo
Copy link
Member

syuilo commented Sep 30, 2019

どんなオブジェクトがメモリに溜まっているのかと、UIの動きを減らすオプションを有効にするとリークしなくなるかを調べればわかりそう

@tamaina
Copy link
Member

tamaina commented Sep 30, 2019

私の環境だと、
UIの動く設定でDeck数時間放置で使い物にならなくなってた
→動かない設定ONにすればDeck数時間放置でも大丈夫になってる
のでどうかなぁと

@AyaMorisawa AyaMorisawa added the 🐢Performance Efficiency related issue/PR label Sep 30, 2019
@acid-chicken
Copy link
Member Author

やっぱトランジション関係なさそう
image

@tamaina
Copy link
Member

tamaina commented Sep 30, 2019

ウィジェットは置いていますか?あったら消して放置してみてほしい(ウィジェットはパフォーマンスの問題が起きやすそう)
あとは万が一のことを考えてウィジェットもタイムラインも何もない状態で放置してみてみるのも実験としてアリかも

@acid-chicken
Copy link
Member Author

ウィジェットはホームに置いてるもののサブセットなので比較する意味はないかと。(何かしらのウィジェットがメモリ解放機能を持っていない限り)

@syuilo
Copy link
Member

syuilo commented Sep 30, 2019

やっぱトランジション関係なさそう
image

その図だと詳細がよくわからないので後で確認してみます

@tamaina
Copy link
Member

tamaina commented Oct 1, 2019

うーん、トランジション抜きDeckで9時間放置して2000投稿ほどためてみたけど、私の環境(Firefox 70.0b10, Misskey v11.33.0)だとメモリがもりもりする感じがない…

@SanMurakami
Copy link
Contributor

少なくとも自分の環境ではバックグラウンドタブにしない限りMisskey開いたまま仕事に行って帰ってきてもChrome全体でメモリ3~6GB程度しか使ってないですね。

ちなみに、過去に体験したメモリリークの現象でかなり似ている事例としては、CPUの性能が足りない(or バックグラウンドで性能が制限されている)時にループ処理や常時接続関連のラグによりGCが正常にできずにメモリリークしたと言うことがありました。

今回の件がそれと断定できるわけではないですが、挙動的にはかなり似ているので視野に入れてみてもいいかもしれません。

また、時間とマシンスペックに余裕がある方はフォアグラウンドで常時開いて同じような症状が発生するか確認してみてほしいです。

@yszkst
Copy link
Contributor

yszkst commented Feb 4, 2023

手元の環境で

  • Windows11/Chrome 109
  • UIをデフォルト設定
  • misskey.ioでTLを開いてバックグランドタブで10分放置

にしたときメモリ使用量が

  • 連合のとき +760MB
  • ソーシャル +150MB

ほど増えました。そのまま放置で増え続けます。

他でも再現するようならちょっと無視するには大きいような気がします。
Firefoxでも同様でした。

image

短時間しか見ていませんが、フォアグラウンドでも増えているように見えます。ただ、解放処理が動きながら減って増えてを繰り返しながらバックグラウンド時よりはるかにゆっくりです。

関連(でしょうか) #6385

@tamaina

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug packages/frontend Client side specific issue/PR 🐢Performance Efficiency related issue/PR ❓needs more investigation A bug whose causes are unknown
Projects
None yet
Development

No branches or pull requests

6 participants