Possible memory leak in Skia.Gtk heap

[mcNets]

Hi.

I developed an app that runs on a Linux device controlling a machine and I have noticed that is increasingly consuming a lot of memory.

I am trying to detect the memory leak using the Performance profiler tool, I am sorry but I am not used to it. What I did is:

1. Run the Skia.Gtk heap in my computer and take some snapshots.
2. Run the Windows heap and take again the aprox same snapshots.

And these are the results:

Windows

Skia.Gtk

Any hint on how can I dig deeper and find the memory leak?

The app is very simple, just send gRPC messages to a service and receive broadcast notifications from it.

Report20230722-1430.zip

[jeromelaban]

Thanks for the report. The trace you've provided is done on the .Windows head, not on the skia.gtk head. If there's a leak in the .Windows head, it's not something Uno can adjust and you'll want to report it in the official winui repository.

[mcNets]

sysmacui.skia.gtk.zip

[jeromelaban]

Thanks for the update. What's leaking is unclear, but is having 24 SysmacUI.Controls.KPIControl instances something that should be happening? Also, there are 5 instances of MainMenuPage, which could indicate multiple forward navigation?

You may want to try to simplify your pages to see what is leaking.

[mcNets]

@jeromelaban thanks a lot for your time and help.

I don't understand why there are 5 instances of ``MainMenuPage`

This is how my navigation service looks like, it doesn't allow to navigate to the current page:

internal class NavService {

    public static void NavigateTo(string pageName, object? parameters) {
        var app = (App)Application.Current;
        var window = app.MainWindow;

        if (window == null || window.Content == null) {
            Debug.WriteLine("NavigateTo requires a Window.Content");
            return;
        }

        string aqn = $"SysmacUI.Pages.{pageName}";
        Type? type = Type.GetType(aqn);
        if (type == null) {
            Debug.WriteLine($"There is no {aqn}");
            return;
        }

        var frame = (Frame)window.Content;
        if (frame.Content.GetType() == type) {
            Debug.WriteLine($"Already in {aqn}");
            return;
        }

        frame?.Navigate(type, parameters);
    }

    public static void MainMenu() => NavigateTo("MainMenuPage", null);
    public static void Autocontrol() => NavigateTo("AutocontrolPage", null);
    public static void Controls() => NavigateTo("ControlsPage", null);

    public static async void GPIO() {
        if (await DlgService.ShowRfidDlgAsync() == false) {
            return;
        }
        NavigateTo("GPIOPage", null);
    }

And I use it in XAML in this way:

<controls:BigButtonControl Grid.Row="0"
                           Grid.Column="3"
                           Text="PRODUCCIO"
                           Background="{StaticResource SysmacGradient4}"
                           ButtonWidth="150"
                           HorizontalAlignment="Center"
                           Margin="0,0,0,5"
                           OnClick="{x:Bind serv:NavService.Production}" />

Let me start again with a simple project and see what happens.

I don't think my company will accept to pay for technical support, and I'm not sure I can afford it.

What's the cost per hour?

I am using some custom controls, and I've given them a star-trek console look just for fun. Maybe one of them is causing the memory leak. I can try using standard controls, except for the gauge.

I've been working with WinForms for a long time so I'm really a newbie in the XAML world.

This is a old version, but you can take a look here

[mcNets]

I have build a new UNO project (Windows + Skia) with just two pages and some standard buttons.

If you navigate from one page to another several times, memory is steadily increasing and even GC is called (that never happens in my app), memory is never released.

I'm not sure about that but it seems related to Microsoft.UI.Xaml.DependencyProperties libraries.

SysmacUI.zip

[jeromelaban]

#13958 may help for this issue.