<div dir="ltr">That file is, without doubt, the best part of the project.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 16 Mar 2021 at 15:45, Aleix Pol <<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Mar 16, 2021 at 3:40 PM Konstantin Kharlamov <<a href="mailto:hi-angel@yandex.ru" target="_blank">hi-angel@yandex.ru</a>> wrote:<br>
><br>
> I think the most interesting thing here is to know what plasmashell developers<br>
> think about it. Is it perhaps possible they don't have a reproducer for memory<br>
> leak, and you have it? In which case it seems more productive to fix the bug.<br>
><br>
> Please, get in touch with plasmashell devs, I'm sure they will be glad to get<br>
> any help in addressing that sort of issues.<br>
><br>
> On Tue, 2021-03-16 at 15:34 +0100, Alberto Salvia Novella wrote:<br>
> > The thing is that plasmashell isn't always started via its service, at least<br>
> > this is the case on my system. I shall look into that.<br>
> ><br>
> > On Tue, 16 Mar 2021 at 07:41, Konstantin Kharlamov <<a href="mailto:hi-angel@yandex.ru" target="_blank">hi-angel@yandex.ru</a>> wrote:<br>
> > > On Tue, 2021-03-16 at 01:29 +0100, Alberto Salvia Novella wrote:<br>
> > > > I have created this tool for the Plasma desktop:<br>
> > > > <a href="https://gitlab.com/es20490446e/plasma-leakguard" rel="noreferrer" target="_blank">https://gitlab.com/es20490446e/plasma-leakguard</a><br>
> > > ><br>
> > > > Shall I announce it somewhere?<br>
> > > ><br>
> > > > (When replying please include my email address on the "to" field, as I<br>
> > > > have<br>
> > > > mail delivery disabled for this list)<br>
> > ><br>
> > > (note: I'm just a random passer-by contributor)<br>
> > ><br>
> > > I think you better discuss this with plasmashell developers. Clearly, memory<br>
> > > leaks need to be fixed instead of working them around. Although if they deem<br>
> > > such tool as you suggest necessary, to me it seems easier to implement by<br>
> > > creating a plasmashell user-level service with `MemoryMax=` variable set and<br>
> > > being restartable (or, in case they wouldn't want to depend on systemd, I<br>
> > > imagine it should be available with bare cgroups too).<br>
> > ><br>
<br>
I don't think this will ever be part of the solution. I've never seen<br>
this problem myself, but in the end we just need to remove leaks<br>
rather than just restarting plasma. I guess this is probably useful<br>
for the original developer, so good for him.<br>
<br>
Note this app will make you lose whatever you're doing with the shell<br>
at the time of the restart.<br>
<br>
The fact that there's this file in the project, makes me question the<br>
intent of the project altogether:<br>
<a href="https://gitlab.com/es20490446e/plasma-leakguard/-/blob/master/info/jokes.md" rel="noreferrer" target="_blank">https://gitlab.com/es20490446e/plasma-leakguard/-/blob/master/info/jokes.md</a><br>
<br>
Aleix<br>
</blockquote></div>