Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-23 Thread Dave Martin
On Thu, Nov 22, 2018 at 09:40:24AM +0800, Li, Aubrey wrote: > On 2018/11/21 17:53, Peter Zijlstra wrote: > > On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: > >> On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > Also; you were going to shop around with the other

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-23 Thread Dave Martin
On Thu, Nov 22, 2018 at 09:40:24AM +0800, Li, Aubrey wrote: > On 2018/11/21 17:53, Peter Zijlstra wrote: > > On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: > >> On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > Also; you were going to shop around with the other

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Li, Aubrey
On 2018/11/21 17:53, Peter Zijlstra wrote: > On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: >> On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: Also; you were going to shop around with the other architectures to see what they want/need for this interface. I

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Li, Aubrey
On 2018/11/21 17:53, Peter Zijlstra wrote: > On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: >> On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: Also; you were going to shop around with the other architectures to see what they want/need for this interface. I

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Palmer Dabbelt
On Wed, 21 Nov 2018 01:53:50 PST (-0800), pet...@infradead.org wrote: On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > > Also; you were going to shop around with the other architectures to see > > what they want/need

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Palmer Dabbelt
On Wed, 21 Nov 2018 01:53:50 PST (-0800), pet...@infradead.org wrote: On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > > Also; you were going to shop around with the other architectures to see > > what they want/need

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Peter Zijlstra
On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: > On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > > > Also; you were going to shop around with the other architectures to see > > > what they want/need for this interface. I see nothing on that. > > > > > I'm open for

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Peter Zijlstra
On Wed, Nov 21, 2018 at 09:19:36AM +0100, Peter Zijlstra wrote: > On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > > > Also; you were going to shop around with the other architectures to see > > > what they want/need for this interface. I see nothing on that. > > > > > I'm open for

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Peter Zijlstra
On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > > Also; you were going to shop around with the other architectures to see > > what they want/need for this interface. I see nothing on that. > > > I'm open for your suggestion, :) Well, we have linux-arch and the various maintainers

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-21 Thread Peter Zijlstra
On Wed, Nov 21, 2018 at 09:39:00AM +0800, Li, Aubrey wrote: > > Also; you were going to shop around with the other architectures to see > > what they want/need for this interface. I see nothing on that. > > > I'm open for your suggestion, :) Well, we have linux-arch and the various maintainers

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-20 Thread Li, Aubrey
On 2018/11/20 1:39, Peter Zijlstra wrote: > On Thu, Nov 15, 2018 at 07:00:07AM +0800, Aubrey Li wrote: >> Add a /proc//arch_state interface to expose per-task cpu specific >> state values. >> >> Exposing AVX-512 Hi16_ZMM registers usage is for the user space job >> scheduler to cluster AVX-512

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-20 Thread Li, Aubrey
On 2018/11/20 1:39, Peter Zijlstra wrote: > On Thu, Nov 15, 2018 at 07:00:07AM +0800, Aubrey Li wrote: >> Add a /proc//arch_state interface to expose per-task cpu specific >> state values. >> >> Exposing AVX-512 Hi16_ZMM registers usage is for the user space job >> scheduler to cluster AVX-512

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-19 Thread Peter Zijlstra
On Thu, Nov 15, 2018 at 07:00:07AM +0800, Aubrey Li wrote: > Add a /proc//arch_state interface to expose per-task cpu specific > state values. > > Exposing AVX-512 Hi16_ZMM registers usage is for the user space job > scheduler to cluster AVX-512 using tasks together, because these tasks > could

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-19 Thread Peter Zijlstra
On Thu, Nov 15, 2018 at 07:00:07AM +0800, Aubrey Li wrote: > Add a /proc//arch_state interface to expose per-task cpu specific > state values. > > Exposing AVX-512 Hi16_ZMM registers usage is for the user space job > scheduler to cluster AVX-512 using tasks together, because these tasks > could

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-15 Thread Li, Aubrey
On 2018/11/15 23:18, Dave Hansen wrote: > On 11/14/18 3:00 PM, Aubrey Li wrote: >> +void arch_thread_state(struct seq_file *m, struct task_struct *task) >> +{ >> +/* >> + * Report AVX-512 Hi16_ZMM registers usage >> + */ >> +if (task->thread.fpu.hi16zmm_usage) >> +

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-15 Thread Li, Aubrey
On 2018/11/15 23:18, Dave Hansen wrote: > On 11/14/18 3:00 PM, Aubrey Li wrote: >> +void arch_thread_state(struct seq_file *m, struct task_struct *task) >> +{ >> +/* >> + * Report AVX-512 Hi16_ZMM registers usage >> + */ >> +if (task->thread.fpu.hi16zmm_usage) >> +

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-15 Thread Dave Hansen
On 11/14/18 3:00 PM, Aubrey Li wrote: > +void arch_thread_state(struct seq_file *m, struct task_struct *task) > +{ > + /* > + * Report AVX-512 Hi16_ZMM registers usage > + */ > + if (task->thread.fpu.hi16zmm_usage) > + seq_putc(m, '1'); > + else > +

Re: [PATCH v3 2/2] proc: add /proc//arch_state

2018-11-15 Thread Dave Hansen
On 11/14/18 3:00 PM, Aubrey Li wrote: > +void arch_thread_state(struct seq_file *m, struct task_struct *task) > +{ > + /* > + * Report AVX-512 Hi16_ZMM registers usage > + */ > + if (task->thread.fpu.hi16zmm_usage) > + seq_putc(m, '1'); > + else > +

[PATCH v3 2/2] proc: add /proc//arch_state

2018-11-14 Thread Aubrey Li
Add a /proc//arch_state interface to expose per-task cpu specific state values. Exposing AVX-512 Hi16_ZMM registers usage is for the user space job scheduler to cluster AVX-512 using tasks together, because these tasks could cause core turbo frequency drop. Signed-off-by: Aubrey Li Cc: Peter

[PATCH v3 2/2] proc: add /proc//arch_state

2018-11-14 Thread Aubrey Li
Add a /proc//arch_state interface to expose per-task cpu specific state values. Exposing AVX-512 Hi16_ZMM registers usage is for the user space job scheduler to cluster AVX-512 using tasks together, because these tasks could cause core turbo frequency drop. Signed-off-by: Aubrey Li Cc: Peter