画像タブが大量にある状態で「複数の画像を閉じる」を実行すると、ウインドウが長時間フリーズする

[ma8ma]

画像タブが大量にある状態で「複数の画像を閉じる」を実行すると、ウインドウが長時間フリーズする

バグの説明

大量の画像タブを開いた状態で、右クリックメニューの「複数の画像を閉じる」→「全ての画像」を実行すると、1000個以上のタブを閉じる間、ウインドウが操作を受け付けなくなり、長時間フリーズします。

再現の方法

⚠ 注意: この操作を実行すると、ウインドウが長時間フリーズします。強制終了が必要になる可能性があるため、事前にデータを保存してください。

1. JDimを起動します。
2. 以下のいずれかの方法で、画像を最低1000個以上開きます。
   • 画像が貼られているスレッドを開き、画像URLをクリックする。
   • メニューバーの「ファイル」＞「URLを開く」、または Ctrl+O を押してダイアログを開き、画像URLを入力して「開く」を押す。
3. 画像タブを右クリックし、「複数の画像を閉じる」＞「全ての画像」を選択します。
4. ウインドウがマウスやキーボードの入力を受け付けなくなり、フリーズします。
5. 数分程度待つとフリーズ状態が解除され、画像タブがすべて閉じます。

やりたかったこと・期待する結果

画像タブを閉じている間もウインドウが応答し、他の操作（例：新しいタブを開く、設定を変更するなど）ができること。

スクリーンショット

画面表示の問題はないため、スクリーンショットはありません。

動作環境

[バージョン] JDim 0.14.0-alpha20250215(git:947216f464)
[ディストリ ] Ubuntu 24.10 (x86_64)
[パッケージ] バイナリ/ソース( <配布元> )
[ DE／WM ] KDE (Wayland)
[　gtkmm 　] 3.24.9
[　glibmm 　] 2.66.7
[　TLS lib　] GnuTLS 3.8.6
[オプション ] '--with-alsa''--with-pangolayout'
[ そ の 他 ] 

追加の情報

右クリックメニューの「複数の画像を閉じる」のサブメニューには「全ての画像」以外の項目もあります。
これらの項目も1000個以上の画像を閉じることができるため、同様の操作でフリーズすることを確認しました。

正確な計測はしていませんが、画像タブの数が増えるほどフリーズ時間が長くなる傾向があります。閉じるタブの数が2倍になっても、フリーズ時間が2倍になるとは限らず、非線形な関係があるように感じられます。例えば、タブ500個を閉じる場合と1000個を閉じる場合では、フリーズ時間は単純に2倍になるのではなく、大幅に延長されました。

[ma8ma]

master (947216f) で gprof を使って実行プロファイルを計測しました。

時間を多く消費する関数のうち上位30個を抜粋します。

動作環境

[バージョン] JDim 0.14.0-alpha20250215(git:947216f464)
[ディストリ ] Ubuntu 24.10 (x86_64)
[パッケージ] バイナリ/ソース( <配布元> )
[ DE／WM ] KDE (Wayland)
[　gtkmm 　] 3.24.9
[　glibmm 　] 2.66.7
[　TLS lib　] GnuTLS 3.8.6
[オプション ] '--with-pangolayout''--enable-gprof'
[ そ の 他 ]
- コンパイラー最適化なし

何もしないときのフラットプロファイル（比較用）

1. 画像0枚のキャッシュで起動する
2. ウインドウが表示されたら、JDimを終了する

Details

Flat profile:

Each sample counts as 0.01 seconds.
  %   cumulative   self              self     total
 time   seconds   seconds    calls  ms/call  ms/call  name
 20.00      0.01     0.01  2124821     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::size() const
 20.00      0.02     0.01   656469     0.00     0.00  bool std::operator==<char, std::char_traits<char>, std::allocator<char> >(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)
 20.00      0.03     0.01   111261     0.00     0.00  std::_List_iterator<JDLIB::ConfData>::operator*() const
 20.00      0.04     0.01    10377     0.00     0.00  Gtk::TreeModelColumnBase::type() const
 20.00      0.05     0.01     4050     0.00     0.00  std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > >::_M_realloc_append<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&&)::_Guard::~_Guard()
  0.00      0.05     0.00  1860894     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_data() const
  0.00      0.05     0.00   550919     0.00     0.00  std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> >::get() const
  0.00      0.05     0.00   550919     0.00     0.00  std::__uniq_ptr_impl<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> >::_M_ptr() const
  0.00      0.05     0.00   550919     0.00     0.00  std::_Head_base<0ul, DBTREE::BoardBase*, false>::_M_head(std::_Head_base<0ul, DBTREE::BoardBase*, false> const&)
  0.00      0.05     0.00   550919     0.00     0.00  std::_Tuple_impl<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> >::_M_head(std::_Tuple_impl<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> > const&)
  0.00      0.05     0.00   550919     0.00     0.00  DBTREE::BoardBase* const& std::__get_helper<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> >(std::_Tuple_impl<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> > const&)
  0.00      0.05     0.00   550919     0.00     0.00  std::tuple_element<0ul, std::tuple<DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> > >::type const& std::get<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> >(std::tuple<DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> > const&)
  0.00      0.05     0.00   550863     0.00     0.00  __gnu_cxx::__aligned_membuf<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > >::_M_ptr()
  0.00      0.05     0.00   550863     0.00     0.00  __gnu_cxx::__aligned_membuf<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > >::_M_addr()
  0.00      0.05     0.00   550863     0.00     0.00  std::_List_node<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > >::_M_valptr()
  0.00      0.05     0.00   549833     0.00     0.00  std::operator!=(std::_List_iterator<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > > const&, std::_List_iterator<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > > const&)
  0.00      0.05     0.00   549141     0.00     0.00  std::char_traits<char>::assign(char&, char const&)
  0.00      0.05     0.00   548846     0.00     0.00  std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> >::operator->() const
  0.00      0.05     0.00   548791     0.00     0.00  std::_List_iterator<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > >::operator*() const
  0.00      0.05     0.00   548736     0.00     0.00  std::_List_iterator<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > >::operator++()
  0.00      0.05     0.00   544647     0.00     0.00  DBTREE::BoardBase::get_path_board[abi:cxx11]() const
  0.00      0.05     0.00   466752     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_is_local() const
  0.00      0.05     0.00   466752     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_local_data() const
  0.00      0.05     0.00   466752     0.00     0.00  std::__ptr_traits_ptr_to<char const*, char const, false>::pointer_to(char const&)
  0.00      0.05     0.00   466752     0.00     0.00  char const* std::__addressof<char const>(char const&)
  0.00      0.05     0.00   466752     0.00     0.00  char const* std::addressof<char const>(char const&)
  0.00      0.05     0.00   446845     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_length(unsigned long)
  0.00      0.05     0.00   406751     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_set_length(unsigned long)
  0.00      0.05     0.00   376217     0.00     0.00  std::__new_allocator<char>::~__new_allocator()
  0.00      0.05     0.00   241469     0.00     0.00  std::__ptr_traits_ptr_to<char*, char, false>::pointer_to(char&)

フリーズ時のフラットプロファイル

1. 画像1035枚のキャッシュで起動する
2. ウインドウが表示されたら、画像が読み込まれて画像ビューに表示されるまで待つ
3. 画像タブの右クリックメニューから「複数の画像を閉じる」→「全ての画像」を選択する
4. ウインドウがフリーズするので操作を受け付けるようになるまで待つ
5. フリーズが解除されたら、JDimを終了する

Details

Flat profile:

Each sample counts as 0.01 seconds.
  %   cumulative   self              self     total           
 time   seconds   seconds    calls  ms/call  ms/call  name    
  8.13      0.10     0.10  2589927     0.00     0.00  std::__uniq_ptr_impl<SKELETON::View, std::default_delete<SKELETON::View> >::_M_ptr() const
  8.13      0.20     0.10     2266     0.04     0.08  IMAGE::ImageAdmin::update_status_of_all_views()
  6.50      0.28     0.08 48501739     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::size() const
  5.69      0.35     0.07  2430115     0.00     0.00  auto DBTREE::Root::get_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, int)::{lambda(auto:1 const&)#1}::operator()<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > >(std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > const&) const
  4.88      0.41     0.06     2277     0.03     0.05  IMAGE::ImageAdmin::switch_img(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)
  4.88      0.47     0.06                             _init
  3.25      0.51     0.04  2996918     0.00     0.00  std::__uniq_ptr_impl<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> >::_M_ptr() const
  3.25      0.55     0.04  2577533     0.00     0.00  std::_List_iterator<std::unique_ptr<SKELETON::View, std::default_delete<SKELETON::View> > >::operator++()
  3.25      0.59     0.04     2278     0.02     0.04  IMAGE::ImageAdmin::get_view(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)
  1.63      0.61     0.02  2996913     0.00     0.00  std::_Tuple_impl<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> >::_M_head(std::_Tuple_impl<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> > const&)
  1.63      0.63     0.02  2589813     0.00     0.00  std::unique_ptr<SKELETON::View, std::default_delete<SKELETON::View> >::get() const
  1.63      0.65     0.02   558124     0.00     0.00  void std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_construct<char*>(char*, char*, std::forward_iterator_tag)
  1.63      0.67     0.02   161299     0.00     0.00  CONTROL::get_id(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)
  0.81      0.68     0.01 30926960     0.00     0.00  std::char_traits<char>::length(char const*)
  0.81      0.69     0.01 30557144     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_data() const
  0.81      0.70     0.01 29487185     0.00     0.00  bool std::operator==<char, std::char_traits<char>, std::allocator<char> >(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, char const*)
  0.81      0.71     0.01  8229047     0.00     0.00  std::char_traits<char>::assign(char&, char const&)
  0.81      0.72     0.01  7404411     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_length(unsigned long)
  0.81      0.73     0.01  7337093     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_is_local() const
  0.81      0.74     0.01  7336408     0.00     0.00  char const* std::addressof<char const>(char const&)
  0.81      0.75     0.01  7112963     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_set_length(unsigned long)
  0.81      0.76     0.01  4838228     0.00     0.00  CONTROL::MouseKeyItem::get_id() const
  0.81      0.77     0.01  2999244     0.00     0.00  std::operator!=(std::_List_iterator<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > > const&, std::_List_iterator<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > > const&)
  0.81      0.78     0.01  2996912     0.00     0.00  std::tuple_element<0ul, std::tuple<DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> > >::type const& std::get<0ul, DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> >(std::tuple<DBTREE::BoardBase*, std::default_delete<DBTREE::BoardBase> > const&)
  0.81      0.79     0.01  2993467     0.00     0.00  std::_List_iterator<std::unique_ptr<DBTREE::BoardBase, std::default_delete<DBTREE::BoardBase> > >::operator++()
  0.81      0.80     0.01  2580740     0.00     0.00  __gnu_cxx::__aligned_membuf<std::unique_ptr<SKELETON::View, std::default_delete<SKELETON::View> > >::_M_ptr()
  0.81      0.81     0.01  1953922     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_get_allocator() const
  0.81      0.82     0.01  1951015     0.00     0.00  IMAGE::ImageAdmin::focus_out_all()::{lambda(Gtk::Widget&)#1}::operator()(Gtk::Widget&) const
  0.81      0.83     0.01  1949607     0.00     0.00  SKELETON::View::focus_out()
  0.81      0.84     0.01  1009972     0.00     0.00  std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_Alloc_hider::_Alloc_hider(char*, std::allocator<char> const&)

画像ビュー関連で時間を多く消費する関数

Each sample counts as 0.01 seconds.
  %   cumulative   self              self     total
 time   seconds   seconds    calls  ms/call  ms/call  name
  8.13      0.20     0.10     2266     0.04     0.08  IMAGE::ImageAdmin::update_status_of_all_views()
  4.88      0.41     0.06     2277     0.03     0.05  IMAGE::ImageAdmin::switch_img(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)
  3.25      0.59     0.04     2278     0.02     0.04  IMAGE::ImageAdmin::get_view(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)

[ma8ma]

#1529 の修正によって、複数の画像タブを閉じている間もウィンドウの操作が可能になりました。
また、#1530 の修正によって IMAGE::ImageAdmin::update_status_of_all_views() のプロファイルの負荷が低減しました。

プロファイリングの手順について：

修正前と修正後のプロファイルデータを作成して比較します。

1. gprof を有効にしてビルドした JDim を用意します。
2. 約1000個の画像タブを開くキャッシュディレクトリを用意します。
3. 用意したキャッシュディレクトリを指定して JDim を起動します。

   JDIM_CACHE=/path/to/jdim_cache_image1000 ./builddir/src/jdim
   

4. JDim が起動してメインウィンドウが表示されたら終了します。終了には時間がかかる場合があります。
5. プロファイルデータを作成します。

   gprof ./builddir/src/jdim gmon.out > jdim.gprof
   

gprof を利用して計測した Flat profile の結果（上記が修正前、下記が修正後）：

Flat profile:

Each sample counts as 0.01 seconds.
  %   cumulative   self              self     total           
 time   seconds   seconds    calls  ms/call  ms/call  name    
  9.38      0.09     0.09     1132     0.08     0.16  IMAGE::ImageAdmin::update_status_of_all_views()
  0.00      0.93     0.00     1131     0.00     0.00  IMAGE::ImageAdmin::update_status_of_all_views()

前回開いていたURLを復元する処理の中で、ビューの再描画処理がスキップされるようになり、IMAGE::ImageAdmin::update_status_of_all_views() の実行時間の割合がほぼゼロになっています。
ただ、実行時間の割合が減少したものの、JDim の起動時間については体感上の差は確認できませんでした。

今後は、IMAGE::ImageAdmin::switch_img() や IMAGE::ImageAdmin::get_view() を改善し、起動時間、終了時間、画像タブを閉じる時間の変化を確認する予定です。

[ma8ma]

今週予定していた、多量の画像を開いているときのJDimのパフォーマンス改善に関する更新は、来週以降に延期します。

試したこと（改善が見られなかったもの）

• IMAGE::ImageAdmin::switch_img() を修正して、処理の負荷を低減しましたが、処理時間に明確な改善は見られませんでした。
• GUI要素を格納するコンテナーを std::list から std::map や std::unordered_map に変更し、要素へのアクセス負荷を低減する方法を試しましたが、処理時間は改善しませんでした。

試したこと（改善が見られたが、問題があったもの）

• 複数の画像を閉じる操作で、画像の非表示と削除の処理を分割し、削除を後で行う方法を試したところ、画像タブを閉じる速度は向上しました。
  • ただし、画像の読み込みが終わらないうちにウィンドウを閉じたとき、segfaultが発生することを確認したため、現状ではこの修正は採用できません。この問題については引き続き調査し、解決策を検討します。