Re: [PATCH 00/17] Backport rt/deadline crash and the ardous story of FUTEX_UNLOCK_PI to 4.4

2018-12-13 Thread Henrik Austad
On Fri, Dec 14, 2018 at 08:18:26AM +0100, Greg Kroah-Hartman wrote: > On Mon, Nov 19, 2018 at 12:27:21PM +0100, Henrik Austad wrote: > > On Fri, Nov 09, 2018 at 11:35:31AM +0100, Henrik Austad wrote: > > > On Fri, Nov 09, 2018 at 11:07:28AM +0100, Henrik Austad wrote: > >

Re: [PATCH 00/17] Backport rt/deadline crash and the ardous story of FUTEX_UNLOCK_PI to 4.4

2018-11-19 Thread Henrik Austad
On Fri, Nov 09, 2018 at 11:35:31AM +0100, Henrik Austad wrote: > On Fri, Nov 09, 2018 at 11:07:28AM +0100, Henrik Austad wrote: > > From: Henrik Austad > > > > Short story: > > Sorry for the spam, it looks like I was not very specific in /which/ > versi

Re: [PATCH 00/17] Backport rt/deadline crash and the ardous story of FUTEX_UNLOCK_PI to 4.4

2018-11-19 Thread Henrik Austad
On Fri, Nov 09, 2018 at 11:35:31AM +0100, Henrik Austad wrote: > On Fri, Nov 09, 2018 at 11:07:28AM +0100, Henrik Austad wrote: > > From: Henrik Austad > > > > Short story: > > Sorry for the spam, it looks like I was not very specific in /which/ > versi

Re: [PATCH 00/17] Backport rt/deadline crash and the ardous story of FUTEX_UNLOCK_PI to 4.4

2018-11-09 Thread Henrik Austad
On Fri, Nov 09, 2018 at 11:07:28AM +0100, Henrik Austad wrote: > From: Henrik Austad > > Short story: Sorry for the spam, it looks like I was not very specific in /which/ version I targeted this to, as well as not providing a full Cc-list for the cover-letter. The series is

Re: [PATCH 00/17] Backport rt/deadline crash and the ardous story of FUTEX_UNLOCK_PI to 4.4

2018-11-09 Thread Henrik Austad
On Fri, Nov 09, 2018 at 11:07:28AM +0100, Henrik Austad wrote: > From: Henrik Austad > > Short story: Sorry for the spam, it looks like I was not very specific in /which/ version I targeted this to, as well as not providing a full Cc-list for the cover-letter. The series is

[PATCH 06/17] futex: Change locking rules

2018-11-09 Thread Henrik Austad
icios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.751993...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 165 + 1 file changed, 132 inser

[PATCH 07/17] futex: Cleanup refcounting

2018-11-09 Thread Henrik Austad
From: Peter Zijlstra commit bf92cf3a5100f5a0d5f9834787b130159397cb22 upstream. Add a put_pit_state() as counterpart for get_pi_state() so the refcounting becomes consistent. Signed-off-by: Peter Zijlstra (Intel) Cc: juri.le...@arm.com Cc: bige...@linutronix.de Cc: xlp...@redhat.com Cc:

[PATCH 04/17] rtmutex: Make wait_lock irq safe

2018-11-09 Thread Henrik Austad
From: Thomas Gleixner commit b4abf91047cf054f203dcfac97e1038388826937 upstream. Sasha reported a lockdep splat about a potential deadlock between RCU boosting rtmutex and the posix timer it_lock. CPU0CPU1 rtmutex_lock(>rt_mutex)

[PATCH 02/17] futex: Use smp_store_release() in mark_wake_futex()

2018-11-09 Thread Henrik Austad
-by: Henrik Austad --- kernel/futex.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/kernel/futex.c b/kernel/futex.c index bb87324..9e92f12 100644 --- a/kernel/futex.c +++ b/kernel/futex.c @@ -1284,8 +1284,7 @@ static void mark_wake_futex(struct wake_q_head *wake_q, struct

[PATCH 06/17] futex: Change locking rules

2018-11-09 Thread Henrik Austad
icios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.751993...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 165 + 1 file changed, 132 inser

[PATCH 07/17] futex: Cleanup refcounting

2018-11-09 Thread Henrik Austad
From: Peter Zijlstra commit bf92cf3a5100f5a0d5f9834787b130159397cb22 upstream. Add a put_pit_state() as counterpart for get_pi_state() so the refcounting becomes consistent. Signed-off-by: Peter Zijlstra (Intel) Cc: juri.le...@arm.com Cc: bige...@linutronix.de Cc: xlp...@redhat.com Cc:

[PATCH 04/17] rtmutex: Make wait_lock irq safe

2018-11-09 Thread Henrik Austad
From: Thomas Gleixner commit b4abf91047cf054f203dcfac97e1038388826937 upstream. Sasha reported a lockdep splat about a potential deadlock between RCU boosting rtmutex and the posix timer it_lock. CPU0CPU1 rtmutex_lock(>rt_mutex)

[PATCH 02/17] futex: Use smp_store_release() in mark_wake_futex()

2018-11-09 Thread Henrik Austad
-by: Henrik Austad --- kernel/futex.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/kernel/futex.c b/kernel/futex.c index bb87324..9e92f12 100644 --- a/kernel/futex.c +++ b/kernel/futex.c @@ -1284,8 +1284,7 @@ static void mark_wake_futex(struct wake_q_head *wake_q, struct

[PATCH 00/17] Backport rt/deadline crash and the ardous story of FUTEX_UNLOCK_PI to 4.4

2018-11-09 Thread Henrik Austad
From: Henrik Austad Short story: The following patches are needed on a 4.4 kernel to avoid Oops in the scheduler when a sched_rr and sched_deadline task contends on the same futex (with PI). Longer story: On one of our arm64 systems, we occasionally crash with an Oops in the scheduler

[PATCH 00/17] Backport rt/deadline crash and the ardous story of FUTEX_UNLOCK_PI to 4.4

2018-11-09 Thread Henrik Austad
From: Henrik Austad Short story: The following patches are needed on a 4.4 kernel to avoid Oops in the scheduler when a sched_rr and sched_deadline task contends on the same futex (with PI). Longer story: On one of our arm64 systems, we occasionally crash with an Oops in the scheduler

[PATCH 09/17] futex: Rename free_pi_state() to put_pi_state()

2018-11-09 Thread Henrik Austad
Cc: Peter Zijlstra Cc: Darren Hart Cc: Davidlohr Bueso Cc: bhuvanesh_surach...@mentor.com Cc: Andy Lowe Link: http://lkml.kernel.org/r/20151219200607.259636...@linutronix.de Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 17 ++--- 1 file changed, 10

[PATCH 09/17] futex: Rename free_pi_state() to put_pi_state()

2018-11-09 Thread Henrik Austad
Cc: Peter Zijlstra Cc: Darren Hart Cc: Davidlohr Bueso Cc: bhuvanesh_surach...@mentor.com Cc: Andy Lowe Link: http://lkml.kernel.org/r/20151219200607.259636...@linutronix.de Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 17 ++--- 1 file changed, 10

[PATCH 11/17] futex,rt_mutex: Introduce rt_mutex_init_waiter()

2018-11-09 Thread Henrik Austad
...@goodmis.org Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.950039...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 5

[PATCH 11/17] futex,rt_mutex: Introduce rt_mutex_init_waiter()

2018-11-09 Thread Henrik Austad
...@goodmis.org Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.950039...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 5

[PATCH 16/17] rtmutex: Deboost before waking up the top waiter

2018-11-09 Thread Henrik Austad
c: juri.le...@arm.com Cc: bige...@linutronix.de Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170323150216.110065...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 5 +-

[PATCH 08/17] futex: Rework inconsistent rt_mutex/futex_q state

2018-11-09 Thread Henrik Austad
nel.org/r/20170322104151.850383...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 50 ++ 1 file changed, 14 insertions(+), 36 deletions(-) diff --git a/kernel/futex.c b/kernel/futex.c index 9d7d462..91acb65 100644 ---

[PATCH 05/17] futex,rt_mutex: Provide futex specific rt_mutex API

2018-11-09 Thread Henrik Austad
From: Peter Zijlstra commit 5293c2efda37775346885c7e924d4ef7018ea60b upstream. Part of what makes futex_unlock_pi() intricate is that rt_mutex_futex_unlock() -> rt_mutex_slowunlock() can drop rt_mutex::wait_lock. This means it cannot rely on the atomicy of wait_lock, which would be preferred

[PATCH 10/17] futex: Pull rt_mutex_futex_unlock() out from under hb->lock

2018-11-09 Thread Henrik Austad
From: Peter Zijlstra commit 16ffa12d742534d4ff73e8b3a4e81c1de39196f0 upstream. There's a number of 'interesting' problems, all caused by holding hb->lock while doing the rt_mutex_unlock() equivalient. Notably: - a PI inversion on hb->lock; and, - a SCHED_DEADLINE crash because of pointer

[PATCH 16/17] rtmutex: Deboost before waking up the top waiter

2018-11-09 Thread Henrik Austad
c: juri.le...@arm.com Cc: bige...@linutronix.de Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170323150216.110065...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 5 +-

[PATCH 08/17] futex: Rework inconsistent rt_mutex/futex_q state

2018-11-09 Thread Henrik Austad
nel.org/r/20170322104151.850383...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 50 ++ 1 file changed, 14 insertions(+), 36 deletions(-) diff --git a/kernel/futex.c b/kernel/futex.c index 9d7d462..91acb65 100644 ---

[PATCH 05/17] futex,rt_mutex: Provide futex specific rt_mutex API

2018-11-09 Thread Henrik Austad
From: Peter Zijlstra commit 5293c2efda37775346885c7e924d4ef7018ea60b upstream. Part of what makes futex_unlock_pi() intricate is that rt_mutex_futex_unlock() -> rt_mutex_slowunlock() can drop rt_mutex::wait_lock. This means it cannot rely on the atomicy of wait_lock, which would be preferred

[PATCH 10/17] futex: Pull rt_mutex_futex_unlock() out from under hb->lock

2018-11-09 Thread Henrik Austad
From: Peter Zijlstra commit 16ffa12d742534d4ff73e8b3a4e81c1de39196f0 upstream. There's a number of 'interesting' problems, all caused by holding hb->lock while doing the rt_mutex_unlock() equivalient. Notably: - a PI inversion on hb->lock; and, - a SCHED_DEADLINE crash because of pointer

[PATCH 15/17] futex: Drop hb->lock before enqueueing on the rtmutex

2018-11-09 Thread Henrik Austad
oy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104152.161341...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 30 + ker

[PATCH 14/17] futex: Futex_unlock_pi() determinism

2018-11-09 Thread Henrik Austad
Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104152.112378...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/kernel/futex.c b/kernel/fute

[PATCH 15/17] futex: Drop hb->lock before enqueueing on the rtmutex

2018-11-09 Thread Henrik Austad
oy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104152.161341...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 30 + ker

[PATCH 14/17] futex: Futex_unlock_pi() determinism

2018-11-09 Thread Henrik Austad
Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104152.112378...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/kernel/futex.c b/kernel/fute

[PATCH 17/17] sched/rtmutex/deadline: Fix a PI crash for deadline tasks

2018-11-09 Thread Henrik Austad
From: Xunlei Pang commit e96a7705e7d3fef96aec9b590c63b2f6f7d2ba22 upstream. A crash happened while I was playing with deadline PI rtmutex. BUG: unable to handle kernel NULL pointer dereference at 0018 IP: [] rt_mutex_get_top_task+0x1f/0x30 PGD 232a75067 PUD 230947067

[PATCH 12/17] futex,rt_mutex: Restructure rt_mutex_finish_proxy_lock()

2018-11-09 Thread Henrik Austad
at.com Cc: rost...@goodmis.org Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104152.001659...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/f

[PATCH 13/17] futex: Rework futex_lock_pi() to use rt_mutex_*_proxy_lock()

2018-11-09 Thread Henrik Austad
redhat.com Link: http://lkml.kernel.org/r/20170322104152.062785...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 77 + kernel/locking/rtmutex.c| 26 -- kernel/locking/rtmutex_c

[PATCH 12/17] futex,rt_mutex: Restructure rt_mutex_finish_proxy_lock()

2018-11-09 Thread Henrik Austad
at.com Cc: rost...@goodmis.org Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104152.001659...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/f

[PATCH 13/17] futex: Rework futex_lock_pi() to use rt_mutex_*_proxy_lock()

2018-11-09 Thread Henrik Austad
redhat.com Link: http://lkml.kernel.org/r/20170322104152.062785...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c | 77 + kernel/locking/rtmutex.c| 26 -- kernel/locking/rtmutex_c

[PATCH 17/17] sched/rtmutex/deadline: Fix a PI crash for deadline tasks

2018-11-09 Thread Henrik Austad
From: Xunlei Pang commit e96a7705e7d3fef96aec9b590c63b2f6f7d2ba22 upstream. A crash happened while I was playing with deadline PI rtmutex. BUG: unable to handle kernel NULL pointer dereference at 0018 IP: [] rt_mutex_get_top_task+0x1f/0x30 PGD 232a75067 PUD 230947067

[PATCH 01/17] futex: Cleanup variable names for futex_top_waiter()

2018-11-09 Thread Henrik Austad
: xlp...@redhat.com Cc: rost...@goodmis.org Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.554710...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c

[PATCH 03/17] futex: Remove rt_mutex_deadlock_account_*()

2018-11-09 Thread Henrik Austad
: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.652692...@infradead.org Signed-off-by: Thomas Gleixner Conflicts: kernel/locking/rtmutex.c (WAKE_Q) Tested-by: Henrik Austad --- kernel/locking/rtmutex-debug.c | 9

[PATCH 01/17] futex: Cleanup variable names for futex_top_waiter()

2018-11-09 Thread Henrik Austad
: xlp...@redhat.com Cc: rost...@goodmis.org Cc: mathieu.desnoy...@efficios.com Cc: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.554710...@infradead.org Signed-off-by: Thomas Gleixner Tested-by: Henrik Austad --- kernel/futex.c

[PATCH 03/17] futex: Remove rt_mutex_deadlock_account_*()

2018-11-09 Thread Henrik Austad
: jdesfos...@efficios.com Cc: dvh...@infradead.org Cc: bris...@redhat.com Link: http://lkml.kernel.org/r/20170322104151.652692...@infradead.org Signed-off-by: Thomas Gleixner Conflicts: kernel/locking/rtmutex.c (WAKE_Q) Tested-by: Henrik Austad --- kernel/locking/rtmutex-debug.c | 9

Re: [PATCH] backport: sched/rtmutex/deadline: Fix a PI crash for deadline tasks

2018-11-06 Thread Henrik Austad
On Tue, Nov 06, 2018 at 02:22:10PM +0100, Peter Zijlstra wrote: > On Tue, Nov 06, 2018 at 01:47:21PM +0100, Henrik Austad wrote: > > From: Xunlei Pang > > > > On some of our systems, we notice this error popping up on occasion, > > completely hanging the system. >

Re: [PATCH] backport: sched/rtmutex/deadline: Fix a PI crash for deadline tasks

2018-11-06 Thread Henrik Austad
On Tue, Nov 06, 2018 at 02:22:10PM +0100, Peter Zijlstra wrote: > On Tue, Nov 06, 2018 at 01:47:21PM +0100, Henrik Austad wrote: > > From: Xunlei Pang > > > > On some of our systems, we notice this error popping up on occasion, > > completely hanging the system. >

[PATCH] backport: sched/rtmutex/deadline: Fix a PI crash for deadline tasks

2018-11-06 Thread Henrik Austad
(cherry picked from commit e96a7705e7d3fef96aec9b590c63b2f6f7d2ba22) Conflicts: include/linux/sched.h Backported-and-tested-by: Henrik Austad Cc: Greg Kroah-Hartman --- include/linux/init_task.h | 1 + include/linux/sched.h | 2 ++ include/linux/sched/rt.h | 1 + kernel/fork.c

[PATCH] backport: sched/rtmutex/deadline: Fix a PI crash for deadline tasks

2018-11-06 Thread Henrik Austad
(cherry picked from commit e96a7705e7d3fef96aec9b590c63b2f6f7d2ba22) Conflicts: include/linux/sched.h Backported-and-tested-by: Henrik Austad Cc: Greg Kroah-Hartman --- include/linux/init_task.h | 1 + include/linux/sched.h | 2 ++ include/linux/sched/rt.h | 1 + kernel/fork.c

Re: [RFD/RFC PATCH 0/8] Towards implementing proxy execution

2018-10-10 Thread Henrik Austad
On Tue, Oct 09, 2018 at 11:24:26AM +0200, Juri Lelli wrote: > Hi all, Hi, nice series, I have a lot of details to grok, but I like the idea of PE > Proxy Execution (also goes under several other names) isn't a new > concept, it has been mentioned already in the past to this community > (both in

Re: [RFD/RFC PATCH 0/8] Towards implementing proxy execution

2018-10-10 Thread Henrik Austad
On Tue, Oct 09, 2018 at 11:24:26AM +0200, Juri Lelli wrote: > Hi all, Hi, nice series, I have a lot of details to grok, but I like the idea of PE > Proxy Execution (also goes under several other names) isn't a new > concept, it has been mentioned already in the past to this community > (both in

[PATCH] net: export netdev_txq_to_tc to allow sch_mqprio to compile as module

2017-10-17 Thread Henrik Austad
t; Cc: Jesus Sanchez-Palencia <jesus.sanchez-palen...@intel.com> Cc: David S. Miller <da...@davemloft.net> Signed-off-by: Henrik Austad <haus...@cisco.com> --- net/core/dev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/core/dev.c b/net/core/dev.c index fcddccb..d2b20e7 1

[PATCH] net: export netdev_txq_to_tc to allow sch_mqprio to compile as module

2017-10-17 Thread Henrik Austad
d S. Miller Signed-off-by: Henrik Austad --- net/core/dev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/core/dev.c b/net/core/dev.c index fcddccb..d2b20e7 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -2040,6 +2040,7 @@ int netdev_txq_to_tc(struct net_device *dev, unsigned int

Re: [TSN RFC v2 0/9] TSN driver for the kernel

2016-12-17 Thread Henrik Austad
e much in the way of AVB-support *In* kernel. Sorry about that! Since then, the iMX7 from NXP has arrived, and this also has HW-support for TSN, but not in the kernel AFAICT. So, the next issue I plan to tackle, is how I do buffers, the current approach where tsn_core allocates memory is on its way out and I'll let the shim (which means alsa/v4l2) will provide a buffer. Then I'll start looking at qdisc. Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [TSN RFC v2 0/9] TSN driver for the kernel

2016-12-17 Thread Henrik Austad
e much in the way of AVB-support *In* kernel. Sorry about that! Since then, the iMX7 from NXP has arrived, and this also has HW-support for TSN, but not in the kernel AFAICT. So, the next issue I plan to tackle, is how I do buffers, the current approach where tsn_core allocates memory is on its way out and I'll let the shim (which means alsa/v4l2) will provide a buffer. Then I'll start looking at qdisc. Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [TSN RFC v2 5/9] Add TSN header for the driver

2016-12-17 Thread Henrik Austad
mpressed video format" and then the shim/tsn_core will ship out the frames over the network - and then you need to set TSN_CVF as subtype in each header. That does not that mean you should do H.264 encode/decode *in* the kernel Perhaps this is better placed in include/uapi/tsn.h so that userspace and kernel share the same header? -- Henrik Austad signature.asc Description: PGP signature

Re: [TSN RFC v2 5/9] Add TSN header for the driver

2016-12-17 Thread Henrik Austad
mpressed video format" and then the shim/tsn_core will ship out the frames over the network - and then you need to set TSN_CVF as subtype in each header. That does not that mean you should do H.264 encode/decode *in* the kernel Perhaps this is better placed in include/uapi/tsn.h so that userspace and kernel share the same header? -- Henrik Austad signature.asc Description: PGP signature

Re: [TSN RFC v2 0/9] TSN driver for the kernel

2016-12-16 Thread Henrik Austad
On Fri, Dec 16, 2016 at 01:20:57PM -0500, David Miller wrote: > From: Greg <gvrose8...@gmail.com> > Date: Fri, 16 Dec 2016 10:12:44 -0800 > > > On Fri, 2016-12-16 at 18:59 +0100, hen...@austad.us wrote: > >> From: Henrik Austad <haus...@cisco.com> > &g

Re: [TSN RFC v2 0/9] TSN driver for the kernel

2016-12-16 Thread Henrik Austad
On Fri, Dec 16, 2016 at 01:20:57PM -0500, David Miller wrote: > From: Greg > Date: Fri, 16 Dec 2016 10:12:44 -0800 > > > On Fri, 2016-12-16 at 18:59 +0100, hen...@austad.us wrote: > >> From: Henrik Austad > >> > >> > >> The driver is d

Re: [PATCH] tracing: (backport) Replace kmap with copy_from_user() in trace_marker

2016-12-09 Thread Henrik Austad
On Fri, Dec 09, 2016 at 08:22:05AM +0100, Greg KH wrote: > On Fri, Dec 09, 2016 at 07:34:04AM +0100, Henrik Austad wrote: > > Instead of using get_user_pages_fast() and kmap_atomic() when writing > > to the trace_marker file, just allocate enough space on the ring buffer > >

Re: [PATCH] tracing: (backport) Replace kmap with copy_from_user() in trace_marker

2016-12-09 Thread Henrik Austad
On Fri, Dec 09, 2016 at 08:22:05AM +0100, Greg KH wrote: > On Fri, Dec 09, 2016 at 07:34:04AM +0100, Henrik Austad wrote: > > Instead of using get_user_pages_fast() and kmap_atomic() when writing > > to the trace_marker file, just allocate enough space on the ring buffer > >

[PATCH] tracing: (backport) Replace kmap with copy_from_user() in trace_marker

2016-12-08 Thread Henrik Austad
t immediately explode on impact. By definition [2] it must therefore be perfect 2) https://www.spinics.net/lists/kernel/msg2400769.html 2) http://lkml.iu.edu/hypermail/linux/kernel/9804.1/0149.html Cc: Ingo Molnar <mi...@kernel.org> Cc: Henrik Austad <hen...@austad.us> Cc: Peter Zijls

[PATCH] tracing: (backport) Replace kmap with copy_from_user() in trace_marker

2016-12-08 Thread Henrik Austad
t immediately explode on impact. By definition [2] it must therefore be perfect 2) https://www.spinics.net/lists/kernel/msg2400769.html 2) http://lkml.iu.edu/hypermail/linux/kernel/9804.1/0149.html Cc: Ingo Molnar Cc: Henrik Austad Cc: Peter Zijlstra Cc: Steven Rostedt Cc: sta...@vger.kernel.o

Re: [RFD] sched/deadline: Support single CPU affinity

2016-11-10 Thread Henrik Austad
On Thu, Nov 10, 2016 at 01:38:40PM +0100, luca abeni wrote: > Hi Henrik, Hi Luca, > On Thu, 10 Nov 2016 13:21:00 +0100 > Henrik Austad <hen...@austad.us> wrote: > > On Thu, Nov 10, 2016 at 09:08:07AM +0100, Peter Zijlstra wrote: > [...] > > > We define the time

Re: [RFD] sched/deadline: Support single CPU affinity

2016-11-10 Thread Henrik Austad
On Thu, Nov 10, 2016 at 01:38:40PM +0100, luca abeni wrote: > Hi Henrik, Hi Luca, > On Thu, 10 Nov 2016 13:21:00 +0100 > Henrik Austad wrote: > > On Thu, Nov 10, 2016 at 09:08:07AM +0100, Peter Zijlstra wrote: > [...] > > > We define the time to fail as: > &g

Re: [RFD] sched/deadline: Support single CPU affinity

2016-11-10 Thread Henrik Austad
On Thu, Nov 10, 2016 at 09:08:07AM +0100, Peter Zijlstra wrote: > > > Add support for single CPU affinity to SCHED_DEADLINE; the supposed reason for > wanting single CPU affinity is better QoS than provided by G-EDF. > > Therefore the aim is to provide harder guarantees, similar to UP, for

Re: [RFD] sched/deadline: Support single CPU affinity

2016-11-10 Thread Henrik Austad
On Thu, Nov 10, 2016 at 09:08:07AM +0100, Peter Zijlstra wrote: > > > Add support for single CPU affinity to SCHED_DEADLINE; the supposed reason for > wanting single CPU affinity is better QoS than provided by G-EDF. > > Therefore the aim is to provide harder guarantees, similar to UP, for

Re: [Intel-wired-lan] [PATCH] igb: add missing fields to TXDCTL-register

2016-10-19 Thread Henrik Austad
On Wed, Oct 19, 2016 at 07:25:10AM -0700, Jesse Brandeburg wrote: > On Wed, 19 Oct 2016 14:37:59 +0200 > Henrik Austad <hen...@austad.us> wrote: > > > The current list of E1000_TXDCTL-registers is incomplete. This adds > > the missing parts for the Transmit

Re: [Intel-wired-lan] [PATCH] igb: add missing fields to TXDCTL-register

2016-10-19 Thread Henrik Austad
On Wed, Oct 19, 2016 at 07:25:10AM -0700, Jesse Brandeburg wrote: > On Wed, 19 Oct 2016 14:37:59 +0200 > Henrik Austad wrote: > > > The current list of E1000_TXDCTL-registers is incomplete. This adds > > the missing parts for the Transmit Descriptor Control

[PATCH] igb: add missing fields to TXDCTL-register

2016-10-19 Thread Henrik Austad
that this was left out in the commit that added support for 82575 Gigabit Ethernet driver 9d5c8243 (igb: PCI-Express 82575 Gigabit Ethernet driver). Signed-off-by: Henrik Austad <hen...@austad.us> Cc: linux-kernel@vger.kernel.org Cc: Jeff Kirsher <jeffrey.t.kirs...@intel.com> Cc:

[PATCH] igb: add missing fields to TXDCTL-register

2016-10-19 Thread Henrik Austad
that this was left out in the commit that added support for 82575 Gigabit Ethernet driver 9d5c8243 (igb: PCI-Express 82575 Gigabit Ethernet driver). Signed-off-by: Henrik Austad Cc: linux-kernel@vger.kernel.org Cc: Jeff Kirsher Cc: intel-wired-...@lists.osuosl.org Signed-off-by: Henrik Austad

Re: [alsa-devel] [very-RFC 0/8] TSN driver for the kernel

2016-06-23 Thread Henrik Austad
perhaps use the timestamp in skb. Hooking into ktime_get() instead of directly to the PTP-subsystem (if that is even possible) makes it a lot easier to debug when running this in a VM as it doesn't *have* to use PTP-time when I'm crashing a new kernel :) Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [alsa-devel] [very-RFC 0/8] TSN driver for the kernel

2016-06-23 Thread Henrik Austad
perhaps use the timestamp in skb. Hooking into ktime_get() instead of directly to the PTP-subsystem (if that is even possible) makes it a lot easier to debug when running this in a VM as it doesn't *have* to use PTP-time when I'm crashing a new kernel :) Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [alsa-devel] [very-RFC 0/8] TSN driver for the kernel

2016-06-20 Thread Henrik Austad
I will be looking into what to put in the .trigger-handler in the ALSA shim and experimenting with this to see how it make sense to connect it from the TSN-stream. Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [alsa-devel] [very-RFC 0/8] TSN driver for the kernel

2016-06-20 Thread Henrik Austad
I will be looking into what to put in the .trigger-handler in the ALSA shim and experimenting with this to see how it make sense to connect it from the TSN-stream. Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-20 Thread Henrik Austad
On Sun, Jun 19, 2016 at 11:46:29AM +0200, Richard Cochran wrote: > On Sun, Jun 19, 2016 at 12:45:50AM +0200, Henrik Austad wrote: > > edit: this turned out to be a somewhat lengthy answer. I have tried to > > shorten it down somewhere. it is getting late and I'm getti

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-20 Thread Henrik Austad
On Sun, Jun 19, 2016 at 11:46:29AM +0200, Richard Cochran wrote: > On Sun, Jun 19, 2016 at 12:45:50AM +0200, Henrik Austad wrote: > > edit: this turned out to be a somewhat lengthy answer. I have tried to > > shorten it down somewhere. it is getting late and I'm getti

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-18 Thread Henrik Austad
k, but more importantly, no dropped frames. gPTP gives you a central reference to time. > [1] [alsa-lib][PATCH 0/9 v3] ctl: add APIs for control element set > http://mailman.alsa-project.org/pipermail/alsa-devel/2016-June/109274.html > [2] IEEE 1722-2011 > http://ieeexplore.ieee.org/servlet/opac?punumber=5764873 > [3] 5.5 Timing and Synchronization > op. cit. > [4] 1394 Open Host Controller Interface Specification > http://download.microsoft.com/download/1/6/1/161ba512-40e2-4cc9-843a-923143f3456c/ohci_11.pdf I hope this cleared some of the questions -- Henrik Austad signature.asc Description: Digital signature

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-18 Thread Henrik Austad
k, but more importantly, no dropped frames. gPTP gives you a central reference to time. > [1] [alsa-lib][PATCH 0/9 v3] ctl: add APIs for control element set > http://mailman.alsa-project.org/pipermail/alsa-devel/2016-June/109274.html > [2] IEEE 1722-2011 > http://ieeexplore.ieee.org/servlet/opac?punumber=5764873 > [3] 5.5 Timing and Synchronization > op. cit. > [4] 1394 Open Host Controller Interface Specification > http://download.microsoft.com/download/1/6/1/161ba512-40e2-4cc9-843a-923143f3456c/ohci_11.pdf I hope this cleared some of the questions -- Henrik Austad signature.asc Description: Digital signature

Re: [very-RFC 7/8] AVB ALSA - Add ALSA shim for TSN

2016-06-15 Thread Henrik Austad
On Wed, Jun 15, 2016 at 01:49:08PM +0200, Richard Cochran wrote: > Now that I understand better... > > On Sun, Jun 12, 2016 at 01:01:35AM +0200, Henrik Austad wrote: > > Userspace is supposed to reserve bandwidth, find StreamID etc. > > > > To use as a Talker: >

Re: [very-RFC 7/8] AVB ALSA - Add ALSA shim for TSN

2016-06-15 Thread Henrik Austad
On Wed, Jun 15, 2016 at 01:49:08PM +0200, Richard Cochran wrote: > Now that I understand better... > > On Sun, Jun 12, 2016 at 01:01:35AM +0200, Henrik Austad wrote: > > Userspace is supposed to reserve bandwidth, find StreamID etc. > > > > To use as a Talker: >

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-15 Thread Henrik Austad
On Wed, Jun 15, 2016 at 09:04:41AM +0200, Richard Cochran wrote: > On Tue, Jun 14, 2016 at 10:38:10PM +0200, Henrik Austad wrote: > > Whereas I want to do > > > > aplay some_song.wav > > Can you please explain how your patches accomplish this? In short: modprobe

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-15 Thread Henrik Austad
On Wed, Jun 15, 2016 at 09:04:41AM +0200, Richard Cochran wrote: > On Tue, Jun 14, 2016 at 10:38:10PM +0200, Henrik Austad wrote: > > Whereas I want to do > > > > aplay some_song.wav > > Can you please explain how your patches accomplish this? In short: modprobe

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-14 Thread Henrik Austad
On Tue, Jun 14, 2016 at 08:26:15PM +0200, Richard Cochran wrote: > On Tue, Jun 14, 2016 at 11:30:00AM +0200, Henrik Austad wrote: > > So loop data from kernel -> userspace -> kernelspace and finally back to > > userspace and the media application? > > Huh? I wonder wh

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-14 Thread Henrik Austad
On Tue, Jun 14, 2016 at 08:26:15PM +0200, Richard Cochran wrote: > On Tue, Jun 14, 2016 at 11:30:00AM +0200, Henrik Austad wrote: > > So loop data from kernel -> userspace -> kernelspace and finally back to > > userspace and the media application? > > Huh? I wonder wh

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-14 Thread Henrik Austad
On Mon, Jun 13, 2016 at 09:32:10PM +0200, Richard Cochran wrote: > On Mon, Jun 13, 2016 at 03:00:59PM +0200, Henrik Austad wrote: > > On Mon, Jun 13, 2016 at 01:47:13PM +0200, Richard Cochran wrote: > > > Which driver is that? > > > > drivers/net/ethernet/renesas/ &g

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-14 Thread Henrik Austad
On Mon, Jun 13, 2016 at 09:32:10PM +0200, Richard Cochran wrote: > On Mon, Jun 13, 2016 at 03:00:59PM +0200, Henrik Austad wrote: > > On Mon, Jun 13, 2016 at 01:47:13PM +0200, Richard Cochran wrote: > > > Which driver is that? > > > > drivers/net/ethernet/renesas/ &g

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-14 Thread Henrik Austad
ably don't want > another mechanism to map hw queues/tcs/etc if the existing interfaces > work or can be extended to support this. Sure, I get that, as long as the complexity for setting up a link doesn't go through the roof :) Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-14 Thread Henrik Austad
ably don't want > another mechanism to map hw queues/tcs/etc if the existing interfaces > work or can be extended to support this. Sure, I get that, as long as the complexity for setting up a link doesn't go through the roof :) Thanks! -- Henrik Austad signature.asc Description: Digital signature

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-13 Thread Henrik Austad
On Mon, Jun 13, 2016 at 01:47:13PM +0200, Richard Cochran wrote: > Henrik, Hi Richard, > On Sun, Jun 12, 2016 at 01:01:28AM +0200, Henrik Austad wrote: > > There are at least one AVB-driver (the AV-part of TSN) in the kernel > > already, > > Which driver is that? driv

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-13 Thread Henrik Austad
On Mon, Jun 13, 2016 at 01:47:13PM +0200, Richard Cochran wrote: > Henrik, Hi Richard, > On Sun, Jun 12, 2016 at 01:01:28AM +0200, Henrik Austad wrote: > > There are at least one AVB-driver (the AV-part of TSN) in the kernel > > already, > > Which driver is that? driv

Re: [very-RFC 6/8] Add TSN event-tracing

2016-06-13 Thread Henrik Austad
On Sun, Jun 12, 2016 at 10:22:01PM -0400, Steven Rostedt wrote: > On Sun, 12 Jun 2016 23:25:10 +0200 > Henrik Austad <hen...@austad.us> wrote: > > > > > +#include > > > > +#include > > > > +/* #include */ > > > >

Re: [very-RFC 6/8] Add TSN event-tracing

2016-06-13 Thread Henrik Austad
On Sun, Jun 12, 2016 at 10:22:01PM -0400, Steven Rostedt wrote: > On Sun, 12 Jun 2016 23:25:10 +0200 > Henrik Austad wrote: > > > > > +#include > > > > +#include > > > > +/* #include */ > > > > + > > > > +/* FIXME: update t

Re: [very-RFC 6/8] Add TSN event-tracing

2016-06-12 Thread Henrik Austad
On Sun, Jun 12, 2016 at 12:58:03PM -0400, Steven Rostedt wrote: > On Sun, 12 Jun 2016 01:01:34 +0200 > Henrik Austad <hen...@austad.us> wrote: > > > From: Henrik Austad <haus...@cisco.com> > > > > This needs refactoring and should be updated to use TRACE_CL

Re: [very-RFC 6/8] Add TSN event-tracing

2016-06-12 Thread Henrik Austad
On Sun, Jun 12, 2016 at 12:58:03PM -0400, Steven Rostedt wrote: > On Sun, 12 Jun 2016 01:01:34 +0200 > Henrik Austad wrote: > > > From: Henrik Austad > > > > This needs refactoring and should be updated to use TRACE_CLASS, but for > > now it provides a fair d

Re: [very-RFC 5/8] Add TSN machinery to drive the traffic from a shim over the network

2016-06-12 Thread Henrik Austad
On Sun, Jun 12, 2016 at 12:35:10AM -0700, Joe Perches wrote: > On Sun, 2016-06-12 at 00:22 +0200, Henrik Austad wrote: > > From: Henrik Austad <haus...@cisco.com> > > > > In short summary: > > > > * tsn_core.c is the main driver of tsn, all new links go

Re: [very-RFC 5/8] Add TSN machinery to drive the traffic from a shim over the network

2016-06-12 Thread Henrik Austad
On Sun, Jun 12, 2016 at 12:35:10AM -0700, Joe Perches wrote: > On Sun, 2016-06-12 at 00:22 +0200, Henrik Austad wrote: > > From: Henrik Austad > > > > In short summary: > > > > * tsn_core.c is the main driver of tsn, all new links go through > >   here and

[very-RFC 4/8] Add TSN header for the driver

2016-06-11 Thread Henrik Austad
From: Henrik Austad <haus...@cisco.com> This defines the general TSN headers for network packets, the shim-interface and the central 'tsn_list' structure. Cc: "David S. Miller" <da...@davemloft.net> Signed-off-by: Henrik Austad <haus...@cisco.com> --

[very-RFC 6/8] Add TSN event-tracing

2016-06-11 Thread Henrik Austad
From: Henrik Austad <haus...@cisco.com> This needs refactoring and should be updated to use TRACE_CLASS, but for now it provides a fair debug-window into TSN. Cc: "David S. Miller" <da...@davemloft.net> Cc: Steven Rostedt <rost...@goodmis.org> (maintainer:

[very-RFC 6/8] Add TSN event-tracing

2016-06-11 Thread Henrik Austad
From: Henrik Austad This needs refactoring and should be updated to use TRACE_CLASS, but for now it provides a fair debug-window into TSN. Cc: "David S. Miller" Cc: Steven Rostedt (maintainer:TRACING) Cc: Ingo Molnar (maintainer:TRACING) Signed-off-by: Henrik Austad --- inc

[very-RFC 4/8] Add TSN header for the driver

2016-06-11 Thread Henrik Austad
From: Henrik Austad This defines the general TSN headers for network packets, the shim-interface and the central 'tsn_list' structure. Cc: "David S. Miller" Signed-off-by: Henrik Austad --- include/linux/tsn.h | 806 1 file ch

[very-RFC 5/8] Add TSN machinery to drive the traffic from a shim over the network

2016-06-11 Thread Henrik Austad
From: Henrik Austad <haus...@cisco.com> In short summary: * tsn_core.c is the main driver of tsn, all new links go through here and all data to/form the shims are handled here core also manages the shim-interface. * tsn_configfs.c is the API to userspace. TSN is driven from use

[very-RFC 0/8] TSN driver for the kernel

2016-06-11 Thread Henrik Austad
remains - tie to (g)PTP properly, currently using ktime_get() for presentation time - get time from shim into TSN and vice versa - let shim create/manage buffer Henrik Austad (8): TSN: add documentation TSN: Add the standard formerly known as AVB to the kernel Adding TSN-driver to Intel

[very-RFC 5/8] Add TSN machinery to drive the traffic from a shim over the network

2016-06-11 Thread Henrik Austad
From: Henrik Austad In short summary: * tsn_core.c is the main driver of tsn, all new links go through here and all data to/form the shims are handled here core also manages the shim-interface. * tsn_configfs.c is the API to userspace. TSN is driven from userspace and a link is created

  1   2   3   >