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