D17217: Implement free memory notifier

McPain created this revision. McPain added a reviewer: broulik. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. McPain requested review of this revision. REVISION SUMMARY FreeMemoryNotifier will warn when free RAM is running out. By default, it is 25% without swap spac

D17217: Implement free memory notifier

ngraham added a reviewer: VDG. ngraham added a comment. Thanks for all the work! ...But is this actually useful or actionable for the majority of users? Free space is a fairly understandable concept: if you run out, you need to delete things to make more room before you can add more stuf

D17217: Implement free memory notifier

McPain added a comment. In D17217#367525 , @ngraham wrote: > Thanks for all the work! > > ...But is this actually useful or actionable for the majority of users? Free space is a fairly understandable concept: if you run out, you need to dele

D17217: Implement free memory notifier

abetts added a comment. Is having this feature an optional feature? Can it be turned off? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D17217 To: McPain, broulik, #vdg Cc: abetts, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, je

D17217: Implement free memory notifier

McPain added a comment. In D17217#367545 , @abetts wrote: > Is having this feature an optional feature? Can it be turned off? I'll make the notification "disableable" tomorrow REPOSITORY R120 Plasma Workspace REVISION DETAIL https://

D17217: Implement free memory notifier

ngraham added a comment. In D17217#367544 , @McPain wrote: > Sometimes I see what happens if you're running out of memory and oom can't do anything - the whole system halts and only solution for this is hard reset with all unsaved data lost. >

D17217: Implement free memory notifier

graesslin added a comment. I don't think that this is useful - sorry. I run out of memory about once every five years, so polling every five seconds is way too often and a waste of resources. On the other hand the time when it happens polling 5 sec is way too low as then we are already dead.

D17217: Implement free memory notifier

ngraham added a comment. In D17217#367643 , @graesslin wrote: > I don't think that this is useful - sorry. I run out of memory about once every five years, so polling every five seconds is way too often and a waste of resources. On the other ha

D17217: Implement free memory notifier

davidedmundson added a comment. Given there is some negativity in the feedback I do want to point out that it's cool to still upload whatever to the KDE store/wider ecosystem. Not everything run in plasma needs to be from plasma; we go to great lengths to make sure that's the case. Ther

D17217: Implement free memory notifier

McPain added a comment. In D17217#367842 , @davidedmundson wrote: > There's also an alternative that I'd like to suggest: > According to https://www.kernel.org/doc/Documentation/cgroup-v1/memory.txt there's a system called "memory pressure" y

D17217: Implement free memory notifier

McPain updated this revision to Diff 46892. McPain added a comment. Default limit: 25% -> 10% Fix typos Auto detect swap, removed "includeSwap" setting KSysGuard: show kill button (implemented in D17366 ) CHANGES SINCE LAST UPDATE https://phabricato

D17217: Implement free memory notifier

McPain added a comment. Anybody? REVISION DETAIL https://phabricator.kde.org/D17217 To: McPain, broulik, #vdg, ngraham Cc: davidedmundson, graesslin, abetts, ngraham, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart

D17217: Implement free memory notifier

davidedmundson added a comment. > Thanks. Any examples in KDE? I can't implement this from scratch right now. There is not, it would requires writing new C code. It's something I can help with. REVISION DETAIL https://phabricator.kde.org/D17217 To: McPain, broulik, #vdg, ngraham Cc:

D17217: Implement free memory notifier

McPain added a comment. In D17217#378768 , @davidedmundson wrote: > > Thanks. Any examples in KDE? I can't implement this from scratch right now. > > There is not, it would requires writing new C code. > It's something I can help with.

D17217: Implement free memory notifier

rooty added a comment. My two cents - I really like this idea and this would be a **very** useful feature for those of us that really dislike swap space and have OOM problems (one too many browser tabs). But I'd set the intervals at 5-10 or even 20 minutes (not seconds). Also, I sugges

D17217: Implement free memory notifier

ngraham added a comment. In D17217#412463 , @rooty wrote: > have OOM problems In D17217#412463 , @rooty wrote: > dislike swap space Erm, I think I may have found your problem. :

D17217: Implement free memory notifier

rooty added a comment. Let me just say that I realize that this is pretty niche, but I would find it extremely useful and a lot of thought and care seem to have gone into this, hence +1. In D17217#412888 , @ngraham wrote: > In D17217#412463

D17217: Implement free memory notifier

ngraham resigned from this revision. ngraham added a comment. If it's not enabled by default like the free space notifier is, I'm okay with shipping this. REVISION DETAIL https://phabricator.kde.org/D17217 To: McPain, broulik, #vdg Cc: rooty, davidedmundson, graesslin, abetts, ngraham, pla

D17217: Implement free memory notifier

rooty added a comment. In D17217#412918 , @ngraham wrote: > If it's not enabled by default like the free space notifier is, I'm okay with shipping this. I totally agree, it shouldn't be enabled by default REVISION DETAIL https://phabri

D17217: Implement free memory notifier

davidedmundson added a comment. > It would just be nice not to have to think about this stuff Well that's the point. You're only going to run out of memory on a working system if you have a runaway process. There is an OOM killer. That OOM killer is faster than a human, has access to

D17217: Implement free memory notifier

McPain added a comment. In D17217#412961 , @davidedmundson wrote: > > It would just be nice not to have to think about this stuff > > There is an OOM killer. That OOM killer is faster than a human, has access to more info about processes and