Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-30 Thread Tejun Heo
On Mon, Jul 30, 2018 at 07:59:36PM +0200, Pavel Machek wrote: > Its true I have no interest in psi. But I'm trying to use same kernel > you are trying to "improve" and I was confused enough by seing > "CONFIG_PSI". And yes, my association was "pounds per square inch" and > "what is it doing here".

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-30 Thread Tejun Heo
Hello, On Mon, Jul 30, 2018 at 10:54:05AM -0700, Randy Dunlap wrote: > I'd say he's trying to make something that is readable and easier to > understand for users. Sure, it's perfectly fine to make those suggestions and discuss but the counter points have already been discussed (e.g. PSI is a kno

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-30 Thread Pavel Machek
On Mon 2018-07-30 10:51:20, Tejun Heo wrote: > Hello, > > On Mon, Jul 30, 2018 at 07:39:40PM +0200, Pavel Machek wrote: > > > I'd rather have the internal config symbol match the naming scheme in > > > the code, where psi is a shorter, unique token as copmared to e.g. > > > pressure, press, prsr,

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-30 Thread Randy Dunlap
On 07/30/2018 10:51 AM, Tejun Heo wrote: > Hello, > > On Mon, Jul 30, 2018 at 07:39:40PM +0200, Pavel Machek wrote: >>> I'd rather have the internal config symbol match the naming scheme in >>> the code, where psi is a shorter, unique token as copmared to e.g. >>> pressure, press, prsr, etc. >> >>

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-30 Thread Tejun Heo
Hello, On Mon, Jul 30, 2018 at 07:39:40PM +0200, Pavel Machek wrote: > > I'd rather have the internal config symbol match the naming scheme in > > the code, where psi is a shorter, unique token as copmared to e.g. > > pressure, press, prsr, etc. > > I'd do "pressure", really. Yes, psi is shorter,

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-30 Thread Pavel Machek
On Mon 2018-07-30 11:40:35, Johannes Weiner wrote: > On Sat, Jul 28, 2018 at 12:01:23AM +0200, Pavel Machek wrote: > > > How do you use this feature? > > > > > > A kernel with CONFIG_PSI=y will create a /proc/pressure directory with > > > 3 files: cpu, memory, and io. If using cgroup2, c

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-30 Thread Johannes Weiner
On Sat, Jul 28, 2018 at 12:01:23AM +0200, Pavel Machek wrote: > > How do you use this feature? > > > > A kernel with CONFIG_PSI=y will create a /proc/pressure directory with > > 3 files: cpu, memory, and io. If using cgroup2, cgroups will also > > Could we get the config named CONFIG_

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-27 Thread Suren Baghdasaryan
On Thu, Jul 26, 2018 at 1:07 PM, Johannes Weiner wrote: > On Thu, Jul 26, 2018 at 11:07:32AM +1000, Singh, Balbir wrote: >> On 7/25/18 1:15 AM, Johannes Weiner wrote: >> > On Tue, Jul 24, 2018 at 07:14:02AM +1000, Balbir Singh wrote: >> >> Does the mechanism scale? I am a little concerned about ho

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-27 Thread Pavel Machek
Hi! > The idea is to eventually incorporate this back into the kernel, so > that Linux can avoid OOM livelocks (which TECHNICALLY aren't memory > deadlocks, but for the user indistinguishable) out of the box. > > We also use psi memory pressure for loadshedding. Our batch job psi->PSI? >

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-26 Thread Johannes Weiner
On Thu, Jul 26, 2018 at 11:07:32AM +1000, Singh, Balbir wrote: > On 7/25/18 1:15 AM, Johannes Weiner wrote: > > On Tue, Jul 24, 2018 at 07:14:02AM +1000, Balbir Singh wrote: > >> Does the mechanism scale? I am a little concerned about how frequently > >> this infrastructure is monitored/read/acted

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-25 Thread Singh, Balbir
On 7/25/18 1:15 AM, Johannes Weiner wrote: > Hi Balbir, > > On Tue, Jul 24, 2018 at 07:14:02AM +1000, Balbir Singh wrote: >> Does the mechanism scale? I am a little concerned about how frequently >> this infrastructure is monitored/read/acted upon. > > I expect most users to poll in the freque

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-24 Thread Johannes Weiner
Hi Balbir, On Tue, Jul 24, 2018 at 07:14:02AM +1000, Balbir Singh wrote: > Does the mechanism scale? I am a little concerned about how frequently > this infrastructure is monitored/read/acted upon. I expect most users to poll in the frequency ballpark of the running averages (10s, 1m, 5m). Our OO

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-23 Thread Balbir Singh
On Fri, Jul 13, 2018 at 3:27 AM Johannes Weiner wrote: > > PSI aggregates and reports the overall wallclock time in which the > tasks in a system (or cgroup) wait for contended hardware resources. > > This helps users understand the resource pressure their workloads are > under, which allows them

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-17 Thread Michal Hocko
On Mon 16-07-18 10:57:45, Daniel Drake wrote: > Hi Johannes, > > Thanks for your work on psi! > > We have also been investigating the "thrashing problem" on our Endless > desktop OS. We have seen that systems can easily get into a state where the > UI becomes unresponsive to input, and the mouse

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-13 Thread Johannes Weiner
On Thu, Jul 12, 2018 at 04:44:22PM -0700, Andrew Morton wrote: > On Thu, 12 Jul 2018 13:29:32 -0400 Johannes Weiner wrote: > > > > > ... > > > > The io file is similar to memory. Because the block layer doesn't have > > a concept of hardware contention right now (how much longer is my IO > > requ

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-12 Thread Andrew Morton
On Thu, 12 Jul 2018 13:29:32 -0400 Johannes Weiner wrote: > > ... > > The io file is similar to memory. Because the block layer doesn't have > a concept of hardware contention right now (how much longer is my IO > request taking due to other tasks?), it reports CPU potential lost on > all IO dela

Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-12 Thread Linus Torvalds
On Thu, Jul 12, 2018 at 10:27 AM Johannes Weiner wrote: > > PSI aggregates and reports the overall wallclock time in which the > tasks in a system (or cgroup) wait for contended hardware resources. No comments on the patches themselves (the concept looks sane, and I'm finding it more intriguing f

[PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

2018-07-12 Thread Johannes Weiner
PSI aggregates and reports the overall wallclock time in which the tasks in a system (or cgroup) wait for contended hardware resources. This helps users understand the resource pressure their workloads are under, which allows them to rootcause and fix throughput and latency problems caused by over