JakeStanger/ironbar

Fork 0

mirror of https://github.com/JakeStanger/ironbar.git synced 2026-07-11 06:15:21 +01:00

[GH-ISSUE #972] Possible memory leak with cairo widgets #301

New issue

Closed

opened 2026-05-22 21:52:30 +01:00 by JakeStanger · 4 comments

JakeStanger commented

2026-05-22 21:52:30 +01:00

Owner

Originally created by @kafeinnet on GitHub (May 12, 2025).
Original GitHub issue: https://github.com/JakeStanger/ironbar/issues/972

Describe the bug

If I add a cairo widget, the memory usage of the ironbar process increase quite quickly.
The process is OOM-killed after ~1h of idling.

To reproduce
Steps to reproduce the behavior:

Add a simple cairomodule (see below for the code)
Monitor the memory usage of the ironbar process
Wait a little bit and observe the memory usage increase
Repeat without this test widget and observe the memory usage not increasing

Expected behavior

The memory usage of the ironbar process should not increase.

System information:

Distro: I use Arch Linux btw
Compositor: niri
Ironbar version: git

Configuration

Config

{
  $test = {
    orientation = "horizontal"
    direction = "vertical"
    name = "test"
    type = "cairo"
    path = "/home/fab/.config/ironbar/lua/test.lua"
    frequency = 500
    width = 20
    height = 200
  }
} in {
  monitors.DP-1 = {
    anchor_to_edges = true
    position = "left"
    start = [
      $test
    ]
    center = [
    ]
    end = [
    ]
  }
}

Widget lua code

function draw(cr)
  -- Draw widget
  local y = 20
  local x = 20

  cr:set_font_size(16)
  cr:move_to(x, y)
  cr:show_text("A")

  return 0
end

Additional context

I observed this behavior on two computers, both on Arch Linux.

Originally created by @kafeinnet on GitHub (May 12, 2025). Original GitHub issue: https://github.com/JakeStanger/ironbar/issues/972 **Describe the bug** If I add a cairo widget, the memory usage of the ironbar process increase quite quickly. The process is OOM-killed after ~1h of idling. **To reproduce** Steps to reproduce the behavior: 1. Add a simple `cairo`module (see below for the code) 2. Monitor the memory usage of the ironbar process 3. Wait a little bit and observe the memory usage increase 4. Repeat without this `test` widget and observe the memory usage not increasing **Expected behavior** The memory usage of the ironbar process should not increase. **System information:** - Distro: I use Arch Linux btw - Compositor: niri - Ironbar version: git **Configuration** <details><summary>Config</summary> ```corn { $test = { orientation = "horizontal" direction = "vertical" name = "test" type = "cairo" path = "/home/fab/.config/ironbar/lua/test.lua" frequency = 500 width = 20 height = 200 } } in { monitors.DP-1 = { anchor_to_edges = true position = "left" start = [ $test ] center = [ ] end = [ ] } } ``` </details> <details><summary>Widget lua code</summary> ```lua function draw(cr) -- Draw widget local y = 20 local x = 20 cr:set_font_size(16) cr:move_to(x, y) cr:show_text("A") return 0 end ``` </details> **Additional context** I observed this behavior on two computers, both on Arch Linux.

JakeStanger

2026-05-22 21:52:30 +01:00

closed this issue
added the
P:Critical

A:Client

T:Bug

M:Cairo
labels

JakeStanger commented

2026-05-22 21:52:31 +01:00

Author

Owner

@kafeinnet commented on GitHub (May 12, 2025):

To be sure, I graphed the memory usage with and without a cairo plugin (and my full config).

With a cairo widget:

Without a cairo widget:

@kafeinnet commented on GitHub (May 12, 2025): To be sure, I graphed the memory usage with and without a cairo plugin (and my full config). With a cairo widget: ![Image](https://github.com/user-attachments/assets/e43782ed-39fe-41d6-91df-d231882f42a3) Without a cairo widget: ![Image](https://github.com/user-attachments/assets/fa2f3d0d-12ef-4d9e-99c1-af32401cec38)

JakeStanger commented

2026-05-22 21:52:31 +01:00

Author

Owner

@JakeStanger commented on GitHub (May 12, 2025):

Thanks for the detailed report. What's the unit for the bottom axis?

@JakeStanger commented on GitHub (May 12, 2025): Thanks for the detailed report. What's the unit for the bottom axis?

JakeStanger commented

2026-05-22 21:52:31 +01:00

Author

Owner

@kafeinnet commented on GitHub (May 12, 2025):

That's the time in seconds since the measuring started (one measure per second).

@kafeinnet commented on GitHub (May 12, 2025): That's the time in seconds since the measuring started (one measure per second).

JakeStanger commented

2026-05-22 21:52:31 +01:00

Author

Owner

@JakeStanger commented on GitHub (May 14, 2025):

Got it. Turns out if you increase a reference count you gotta decrease it again. Who knew...

Not even Rust is safe.

I've made some slight changes elsewhere that should have a (very small) improvement on memory/performance in addition.

@JakeStanger commented on GitHub (May 14, 2025): Got it. Turns out if you increase a reference count you gotta decrease it again. Who knew... Not even Rust is safe. I've made some slight changes elsewhere that should have a (very small) improvement on memory/performance in addition.

JakeStanger referenced this issue

2026-05-22 21:54:05 +01:00

[PR #301] [MERGED] build(deps): bump clap from 4.4.1 to 4.4.2 #614

JakeStanger referenced this issue

2026-05-22 22:55:37 +01:00

[PR #301] [MERGED] build(deps): bump clap from 4.4.1 to 4.4.2 #2021

JakeStanger referenced this issue

2026-05-22 23:55:46 +01:00

[PR #301] [MERGED] build(deps): bump clap from 4.4.1 to 4.4.2 #3428

JakeStanger referenced this issue

2026-05-23 00:55:33 +01:00

[PR #301] [MERGED] build(deps): bump clap from 4.4.1 to 4.4.2 #4833