Re: [RFC PATCH] set_mb: Use smp_store_release() instead of set_mb()

2014-11-30 Thread Pranith Kumar
On Wed, Nov 26, 2014 at 11:57 AM, Pranith Kumar bobby.pr...@gmail.com wrote: set_mb() and smp_store_release() perform the same function. Also there are only a few users of set_mb(). We can convert these users to use smp_store_release() and delete the set_mb() definition. The following patch

[RFC PATCH] set_mb: Use smp_store_release() instead of set_mb()

2014-11-26 Thread Pranith Kumar
() definition. Comments and suggestions welcome. Thanks! Pranith Signed-off-by: Pranith Kumar --- fs/select.c | 6 +++--- include/linux/sched.h | 14 +++--- kernel/futex.c| 4 ++-- kernel/sched/wait.c | 4 ++-- 4 files changed, 14 insertions(+), 14 deletions(-) diff --git

[RFC PATCH] set_mb: Use smp_store_release() instead of set_mb()

2014-11-26 Thread Pranith Kumar
() definition. Comments and suggestions welcome. Thanks! Pranith Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- fs/select.c | 6 +++--- include/linux/sched.h | 14 +++--- kernel/futex.c| 4 ++-- kernel/sched/wait.c | 4 ++-- 4 files changed, 14 insertions(+), 14

Re: [PATCH tip/core/rcu 3/9] drivers/md: Use rcu_dereference() for accessing rcu pointer

2014-11-23 Thread Pranith Kumar
On Fri, Nov 21, 2014 at 9:58 AM, Kirill A. Shutemov wrote: > On Fri, Nov 21, 2014 at 09:30:36AM -0500, Pranith Kumar wrote: >> On 11/21/2014 08:31 AM, Kirill A. Shutemov wrote: >> > On Tue, Oct 28, 2014 at 03:09:56PM -0700, Paul E. McKenney wrote: >> >> From: Pranith

Re: [PATCH tip/core/rcu 3/9] drivers/md: Use rcu_dereference() for accessing rcu pointer

2014-11-23 Thread Pranith Kumar
On Fri, Nov 21, 2014 at 9:58 AM, Kirill A. Shutemov kir...@shutemov.name wrote: On Fri, Nov 21, 2014 at 09:30:36AM -0500, Pranith Kumar wrote: On 11/21/2014 08:31 AM, Kirill A. Shutemov wrote: On Tue, Oct 28, 2014 at 03:09:56PM -0700, Paul E. McKenney wrote: From: Pranith Kumar bobby.pr

Re: [PATCH v2 9/9] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
On Fri, Nov 21, 2014 at 7:05 PM, Eric Dumazet wrote: > > On Fri, 2014-11-21 at 16:57 -0500, Pranith Kumar wrote: > >> Hi Eric, >> >> Thanks for looking at this patch. >> >> I've been scratching my head since morning trying to find out what was >> so o

Re: [PATCH v2 9/9] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
On Fri, Nov 21, 2014 at 11:12 AM, Eric Dumazet wrote: > On Fri, 2014-11-21 at 10:06 -0500, Pranith Kumar wrote: >> Recently lockless_dereference() was added which can be used in place of >> hard-coding smp_read_barrier_depends(). The following PATCH makes the change. >> >

Re: [PATCH v2 7/9] seccomp: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
On 11/21/2014 11:33 AM, Kees Cook wrote: > On Fri, Nov 21, 2014 at 7:06 AM, Pranith Kumar wrote: >> Recently lockless_dereference() was added which can be used in place of >> hard-coding smp_read_barrier_depends(). The following PATCH makes the change. >> >> S

[PATCH v2 0/9] Replace smp_read_barrier_depends() with lockless_derefrence()

2014-11-21 Thread Pranith Kumar
with. I will send in separate patches/questions regarding them. Pranith Kumar (9): doc: memory-barriers.txt: Document use of lockless_dereference() drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference() dcache: Replace smp_read_barrier_depends() with lockless_dereference

[PATCH v2 6/9] perf: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- kernel/events/core.c| 3 +-- kernel/events/uprobes.c | 8 2 files changed, 5 insertions(+), 6

[PATCH v2 5/9] percpu: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- include/linux/percpu-refcount.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include

[PATCH v2 4/9] hyperv: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- include/linux/hyperv.h | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/include

[PATCH v2 8/9] task_work: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- kernel/task_work.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/kernel/task_work.c b

[PATCH v2 7/9] seccomp: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- kernel/seccomp.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/kernel/seccomp.c b

[PATCH v2 9/9] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- net/ipv4/netfilter/arp_tables.c | 3 +-- net/ipv4/netfilter/ip_tables.c | 3 +-- net/ipv6/netfilter

[PATCH v2 3/9] dcache: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- fs/dcache.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/fs/dcache.c b/fs/dcache.c

[PATCH v2 2/9] drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- drivers/dma/ioat/dma_v2.c | 3 +-- drivers/dma/ioat/dma_v3.c | 3 +-- 2 files changed, 2 insertions(+), 4

[PATCH v2 1/9] doc: memory-barriers.txt: Document use of lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- Documentation/memory-barriers.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH tip/core/rcu 3/9] drivers/md: Use rcu_dereference() for accessing rcu pointer

2014-11-21 Thread Pranith Kumar
On 11/21/2014 08:31 AM, Kirill A. Shutemov wrote: > On Tue, Oct 28, 2014 at 03:09:56PM -0700, Paul E. McKenney wrote: >> From: Pranith Kumar >> >> Got Paul's email wrong the first time. >> >> The map field in 'struct mapped_device' is an rcu pointer. Use >&g

Re: [PATCH tip/core/rcu 3/9] drivers/md: Use rcu_dereference() for accessing rcu pointer

2014-11-21 Thread Pranith Kumar
On 11/21/2014 08:31 AM, Kirill A. Shutemov wrote: On Tue, Oct 28, 2014 at 03:09:56PM -0700, Paul E. McKenney wrote: From: Pranith Kumar bobby.pr...@gmail.com Got Paul's email wrong the first time. The map field in 'struct mapped_device' is an rcu pointer. Use rcu_dereference() while

[PATCH v2 1/9] doc: memory-barriers.txt: Document use of lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- Documentation/memory-barriers.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH v2 2/9] drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- drivers/dma/ioat/dma_v2.c | 3 +-- drivers/dma/ioat/dma_v3.c | 3 +-- 2 files changed, 2

[PATCH v2 9/9] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- net/ipv4/netfilter/arp_tables.c | 3 +-- net/ipv4/netfilter/ip_tables.c | 3 +-- net/ipv6

[PATCH v2 3/9] dcache: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- fs/dcache.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/fs

[PATCH v2 8/9] task_work: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/task_work.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH v2 4/9] hyperv: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- include/linux/hyperv.h | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff

[PATCH v2 7/9] seccomp: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/seccomp.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git

[PATCH v2 6/9] perf: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/events/core.c| 3 +-- kernel/events/uprobes.c | 8 2 files changed, 5

[PATCH v2 5/9] percpu: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- include/linux/percpu-refcount.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions

[PATCH v2 0/9] Replace smp_read_barrier_depends() with lockless_derefrence()

2014-11-21 Thread Pranith Kumar
with. I will send in separate patches/questions regarding them. Pranith Kumar (9): doc: memory-barriers.txt: Document use of lockless_dereference() drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference() dcache: Replace smp_read_barrier_depends() with lockless_dereference

Re: [PATCH v2 7/9] seccomp: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
On 11/21/2014 11:33 AM, Kees Cook wrote: On Fri, Nov 21, 2014 at 7:06 AM, Pranith Kumar bobby.pr...@gmail.com wrote: Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith

Re: [PATCH v2 9/9] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
On Fri, Nov 21, 2014 at 11:12 AM, Eric Dumazet eric.duma...@gmail.com wrote: On Fri, 2014-11-21 at 10:06 -0500, Pranith Kumar wrote: Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off

Re: [PATCH v2 9/9] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-21 Thread Pranith Kumar
On Fri, Nov 21, 2014 at 7:05 PM, Eric Dumazet eric.duma...@gmail.com wrote: On Fri, 2014-11-21 at 16:57 -0500, Pranith Kumar wrote: Hi Eric, Thanks for looking at this patch. I've been scratching my head since morning trying to find out what was so obviously wrong with this patch. Alas, I

Re: [PATCH 08/16] rcupdate: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-18 Thread Pranith Kumar
On 11/18/2014 12:16 PM, Paul E. McKenney wrote: > On Thu, Nov 13, 2014 at 02:24:14PM -0500, Pranith Kumar wrote: >> Recently lockless_dereference() was added which can be used in place of >> hard-coding smp_read_barrier_depends(). The following PATCH makes the change. >> >

Re: [PATCH 08/16] rcupdate: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-18 Thread Pranith Kumar
On 11/18/2014 12:16 PM, Paul E. McKenney wrote: On Thu, Nov 13, 2014 at 02:24:14PM -0500, Pranith Kumar wrote: Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar

[RFC PATCH] cpuidle: Use a lighter barrier in snooze_loop()

2014-11-15 Thread Pranith Kumar
Using smp_mb() here so that the bit clear is not reordered is an overkill. It is more appropriate to use smp_mb__after_atomic() which ensures that the bit clear is not reordered. Signed-off-by: Pranith Kumar --- drivers/cpuidle/cpuidle-powernv.c | 3 ++- 1 file changed, 2 insertions(+), 1

[RFC PATCH] cpuidle: Use a lighter barrier in snooze_loop()

2014-11-15 Thread Pranith Kumar
Using smp_mb() here so that the bit clear is not reordered is an overkill. It is more appropriate to use smp_mb__after_atomic() which ensures that the bit clear is not reordered. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- drivers/cpuidle/cpuidle-powernv.c | 3 ++- 1 file changed, 2

Re: [RFC PATCH] printk: Use ACCESS_ONCE() instead of a volatile type

2014-11-14 Thread Pranith Kumar
On 11/14/2014 11:39 AM, Alex Elder wrote: > On 11/13/2014 11:24 PM, Steven Rostedt wrote: >> On Thu, 13 Nov 2014 23:57:22 -0500 >> Steven Rostedt wrote: >> >>> That assignment is what it is initialized to at boot up. I can't see >>> any optimization that would cause gcc to modify that.

Re: [PATCH 09/16] percpu: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-14 Thread Pranith Kumar
On Fri, Nov 14, 2014 at 8:14 AM, Tejun Heo wrote: > > How should this be routed? There's a pending change on the same > region of code and if this patch is routed outside percpu tree it'd > cause a conflict and I can't route this as lockless_dereference() > isn't in the mainline yet. Maybe

Re: [PATCH 09/16] percpu: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-14 Thread Pranith Kumar
On Fri, Nov 14, 2014 at 8:14 AM, Tejun Heo t...@kernel.org wrote: How should this be routed? There's a pending change on the same region of code and if this patch is routed outside percpu tree it'd cause a conflict and I can't route this as lockless_dereference() isn't in the mainline yet.

Re: [RFC PATCH] printk: Use ACCESS_ONCE() instead of a volatile type

2014-11-14 Thread Pranith Kumar
On 11/14/2014 11:39 AM, Alex Elder wrote: On 11/13/2014 11:24 PM, Steven Rostedt wrote: On Thu, 13 Nov 2014 23:57:22 -0500 Steven Rostedt rost...@goodmis.org wrote: That assignment is what it is initialized to at boot up. I can't see any optimization that would cause gcc to modify that.

Re: [RFC PATCH] printk: Use ACCESS_ONCE() instead of a volatile type

2014-11-13 Thread Pranith Kumar
On Thu, Nov 13, 2014 at 10:47 PM, Steven Rostedt wrote: > On Thu, 13 Nov 2014 22:21:21 -0500 > Pranith Kumar wrote: > >> Remove volatile type qualifier and use ACCESS_ONCE() in its place for each >> access. Using volatile is not recommended as documented in >> Docume

[RFC PATCH] printk: Use ACCESS_ONCE() instead of a volatile type

2014-11-13 Thread Pranith Kumar
volatile accesses entirely here, but for now make a safer change of using ACCESS_ONCE(). Signed-off-by: Pranith Kumar --- kernel/printk/printk.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index e748971..4790191 100644

Re: [PATCH 16/16] rxrpc: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
On 11/13/2014 03:47 PM, David Howells wrote: > Pranith Kumar wrote: > >> Recently lockless_dereference() was added which can be used in place of >> hard-coding smp_read_barrier_depends(). The following PATCH makes the change. > > Actually, the use of smp_read_ba

[PATCH 01/16] crypto: caam - Remove unnecessary smp_read_barrier_depends()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- drivers/crypto/caam/jr.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/crypto/caam/jr.c b

[PATCH 05/16] overlayfs: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- fs/overlayfs/super.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/fs/overlayfs/super.c

[PATCH 02/16] doc: memory-barriers.txt: Document use of lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- Documentation/memory-barriers.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 03/16] drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- drivers/dma/ioat/dma_v2.c | 3 +-- drivers/dma/ioat/dma_v3.c | 3 +-- 2 files changed, 2 insertions(+), 4

[PATCH 08/16] rcupdate: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- include/linux/rcupdate.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/include

[PATCH 06/16] assoc_array: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
with smp_read_barrier_depends(). Signed-off-by: Pranith Kumar --- include/linux/assoc_array_priv.h | 11 +++ lib/assoc_array.c| 7 --- security/keys/keyring.c | 6 -- 3 files changed, 7 insertions(+), 17 deletions(-) diff --git a/include/linux

[PATCH 12/16] task_work: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- kernel/task_work.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/kernel/task_work.c b

[PATCH 07/16] hyperv: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- include/linux/hyperv.h | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/include

[PATCH 09/16] percpu: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- include/linux/percpu-refcount.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include

[PATCH 11/16] seccomp: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- kernel/seccomp.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/kernel/seccomp.c b

[PATCH 13/16] ksm: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- mm/ksm.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/mm/ksm.c b/mm/ksm.c index

[PATCH 10/16] perf: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- kernel/events/core.c| 3 +-- kernel/events/uprobes.c | 8 2 files changed, 5 insertions(+), 6

[PATCH 15/16] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- net/ipv4/netfilter/arp_tables.c | 3 +-- net/ipv4/netfilter/ip_tables.c | 3 +-- net/ipv6/netfilter

[PATCH 16/16] rxrpc: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- net/rxrpc/ar-ack.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git

[PATCH 04/16] dcache: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- fs/dcache.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/fs/dcache.c b/fs/dcache.c

[PATCH 14/16] slab: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar --- mm/slab.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mm/slab.h b/mm/slab.h index

[RFC PATCH 00/16] Replace smp_read_barrier_depends() with lockless_derefrence()

2014-11-13 Thread Pranith Kumar
with. I will send in separate patches/questions regarding them. Pranith Kumar (16): crypto: caam - Remove unnecessary smp_read_barrier_depends() doc: memory-barriers.txt: Document use of lockless_dereference() drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference

[RFC PATCH 00/16] Replace smp_read_barrier_depends() with lockless_derefrence()

2014-11-13 Thread Pranith Kumar
with. I will send in separate patches/questions regarding them. Pranith Kumar (16): crypto: caam - Remove unnecessary smp_read_barrier_depends() doc: memory-barriers.txt: Document use of lockless_dereference() drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference

[PATCH 14/16] slab: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- mm/slab.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mm

[PATCH 04/16] dcache: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- fs/dcache.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/fs

[PATCH 16/16] rxrpc: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- net/rxrpc/ar-ack.c | 22 +- 1 file changed, 9 insertions(+), 13

[PATCH 15/16] netfilter: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- net/ipv4/netfilter/arp_tables.c | 3 +-- net/ipv4/netfilter/ip_tables.c | 3 +-- net/ipv6

[PATCH 10/16] perf: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/events/core.c| 3 +-- kernel/events/uprobes.c | 8 2 files changed, 5

[PATCH 11/16] seccomp: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/seccomp.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git

[PATCH 13/16] ksm: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- mm/ksm.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/mm

[PATCH 09/16] percpu: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- include/linux/percpu-refcount.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions

[PATCH 07/16] hyperv: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- include/linux/hyperv.h | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff

[PATCH 12/16] task_work: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/task_work.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 06/16] assoc_array: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
with smp_read_barrier_depends(). Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- include/linux/assoc_array_priv.h | 11 +++ lib/assoc_array.c| 7 --- security/keys/keyring.c | 6 -- 3 files changed, 7 insertions(+), 17 deletions(-) diff --git a/include

[PATCH 08/16] rcupdate: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- include/linux/rcupdate.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions

[PATCH 03/16] drivers: dma: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- drivers/dma/ioat/dma_v2.c | 3 +-- drivers/dma/ioat/dma_v3.c | 3 +-- 2 files changed, 2

[PATCH 02/16] doc: memory-barriers.txt: Document use of lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- Documentation/memory-barriers.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 05/16] overlayfs: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- fs/overlayfs/super.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH 01/16] crypto: caam - Remove unnecessary smp_read_barrier_depends()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- drivers/crypto/caam/jr.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers

Re: [PATCH 16/16] rxrpc: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
On 11/13/2014 03:47 PM, David Howells wrote: Pranith Kumar bobby.pr...@gmail.com wrote: Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. Actually, the use of smp_read_barrier_depends

[RFC PATCH] printk: Use ACCESS_ONCE() instead of a volatile type

2014-11-13 Thread Pranith Kumar
volatile accesses entirely here, but for now make a safer change of using ACCESS_ONCE(). Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/printk/printk.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index

Re: [RFC PATCH] printk: Use ACCESS_ONCE() instead of a volatile type

2014-11-13 Thread Pranith Kumar
On Thu, Nov 13, 2014 at 10:47 PM, Steven Rostedt rost...@goodmis.org wrote: On Thu, 13 Nov 2014 22:21:21 -0500 Pranith Kumar bobby.pr...@gmail.com wrote: Remove volatile type qualifier and use ACCESS_ONCE() in its place for each access. Using volatile is not recommended as documented

Re: [PATCH tip/core/rcu 0/6] Torture-test changes for 3.19

2014-10-29 Thread Pranith Kumar
-tests for RCU, courtesy > of Pranith Kumar. > > 3. Enable early-boot callback-posting self-tests in rcutorture, > courtesy of Pranith Kumar. > > 4. Remove old-version rcutorture configuration files, courtesy > of Pranith Kumar. > > 5. Remove the

Re: [PATCH tip/core/rcu 0/5] Documentation updates for 3.19

2014-10-29 Thread Pranith Kumar
eference instructions. Later Alpha CPUs are OK. > (The official Alpha maintainers have thus far been silent > on this patch.) > > 2. Document the new RCU self-test boot parameters, courtesy of > Pranith Kumar. > > 3. Records that short-circu

Re: [PATCH tip/core/rcu 10/10] cpu: Avoid puts_pending overflow

2014-10-29 Thread Pranith Kumar
off-by: Paul E. McKenney This patch seems to be missing in the cover letter. Reviewed-by: Pranith Kumar > --- > kernel/cpu.c | 19 +-- > 1 file changed, 13 insertions(+), 6 deletions(-) > > diff --git a/kernel/cpu.c b/kernel/cpu.c > index 90a3d017b90c..5d220234b3ca 10064

Re: [PATCH tip/core/rcu 0/9] Per-CPU-variable updates

2014-10-29 Thread Pranith Kumar
> > 3-9.Remove "cpu" arguments from a number of RCU functions that are > only ever invoked on that CPU, and use appropriate "this_"-style > accesssors for the per-CPU variables. > Reviewd

Re: [PATCH tip/core/rcu 0/9] Per-CPU-variable updates

2014-10-29 Thread Pranith Kumar
of RCU functions that are only ever invoked on that CPU, and use appropriate this_-style accesssors for the per-CPU variables. Reviewd-by: Pranith Kumar bobby.pr...@gmail.com Thanx, Paul

Re: [PATCH tip/core/rcu 10/10] cpu: Avoid puts_pending overflow

2014-10-29 Thread Pranith Kumar
. McKenney paul...@linux.vnet.ibm.com This patch seems to be missing in the cover letter. Reviewed-by: Pranith Kumar bobby.pr...@gmail.com --- kernel/cpu.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/kernel/cpu.c b/kernel/cpu.c index 90a3d017b90c

Re: [PATCH tip/core/rcu 0/5] Documentation updates for 3.19

2014-10-29 Thread Pranith Kumar
instructions. Later Alpha CPUs are OK. (The official Alpha maintainers have thus far been silent on this patch.) 2. Document the new RCU self-test boot parameters, courtesy of Pranith Kumar. 3. Records that short-circuit boolean evaluation does

Re: [PATCH tip/core/rcu 0/6] Torture-test changes for 3.19

2014-10-29 Thread Pranith Kumar
, courtesy of Pranith Kumar. 3. Enable early-boot callback-posting self-tests in rcutorture, courtesy of Pranith Kumar. 4. Remove old-version rcutorture configuration files, courtesy of Pranith Kumar. 5. Remove the --kversion parameter to kvm.sh, courtesy

Re: [PATCH tip/core/rcu 0/2] Signal-related changes for 3.19

2014-10-28 Thread Pranith Kumar
sly buggy" freeing of sighand by > __cleanup_sighand() without an RCU grace period really isn't > buggy, courtesy of Oleg Nesterov. Reviewed-by: Pranith Kumar > > Thanx, Paul > > --

Re: [PATCH tip/core/rcu 0/2] Signal-related changes for 3.19

2014-10-28 Thread Pranith Kumar
of sighand by __cleanup_sighand() without an RCU grace period really isn't buggy, courtesy of Oleg Nesterov. Reviewed-by: Pranith Kumar bobby.pr...@gmail.com Thanx, Paul

Re: [PATCH] cmpxchg: Discard unnecessary cast to volatile

2014-10-21 Thread Pranith Kumar
On Tue, Oct 21, 2014 at 11:48 AM, H. Peter Anvin wrote: > On 10/21/2014 03:14 AM, Peter Zijlstra wrote: >> On Mon, Oct 20, 2014 at 04:22:27PM -0400, Pranith Kumar wrote: >>>> Generating a volatile pointer is really not necessary here. This is the >>>> only >&

Re: [PATCH] cmpxchg: Discard unnecessary cast to volatile

2014-10-21 Thread Pranith Kumar
On Tue, Oct 21, 2014 at 11:48 AM, H. Peter Anvin h...@zytor.com wrote: On 10/21/2014 03:14 AM, Peter Zijlstra wrote: On Mon, Oct 20, 2014 at 04:22:27PM -0400, Pranith Kumar wrote: Generating a volatile pointer is really not necessary here. This is the only location where a volatile pointer

Re: [PATCH] cmpxchg: Discard unnecessary cast to volatile

2014-10-20 Thread Pranith Kumar
ping. On Wed, Oct 1, 2014 at 1:57 PM, Pranith Kumar wrote: > Generating a volatile pointer is really not necessary here. This is the only > location where a volatile pointer is being generated for use in asm. > > This commit removes the unnecessary volatile pointer being created. &g

Re: [PATCH] cmpxchg: Discard unnecessary cast to volatile

2014-10-20 Thread Pranith Kumar
ping. On Wed, Oct 1, 2014 at 1:57 PM, Pranith Kumar bobby.pr...@gmail.com wrote: Generating a volatile pointer is really not necessary here. This is the only location where a volatile pointer is being generated for use in asm. This commit removes the unnecessary volatile pointer being created

Re: [PATCH 3.13, 3.14, 3.16, 3.17] rcu: Reduce rcu kthread wakeups

2014-10-13 Thread Pranith Kumar
On Mon, Oct 13, 2014 at 4:01 PM, Kamal Mostafa wrote: > On Mon, 2014-10-13 at 12:29 -0400, Pranith Kumar wrote: >> Backport 2aa792e and parts of 48a7639 to 3.13 which is Ubuntu LTS kernel. >> >> This commit reduces the number of rcu kthread wakeups. Tested for over a >

Re: [PATCH 3.13, 3.14, 3.16, 3.17] rcu: Reduce rcu kthread wakeups

2014-10-13 Thread Pranith Kumar
On Mon, Oct 13, 2014 at 4:01 PM, Kamal Mostafa ka...@canonical.com wrote: On Mon, 2014-10-13 at 12:29 -0400, Pranith Kumar wrote: Backport 2aa792e and parts of 48a7639 to 3.13 which is Ubuntu LTS kernel. This commit reduces the number of rcu kthread wakeups. Tested for over a week on my

[PATCH] powerpc: Wire up sys_bpf() syscall

2014-10-09 Thread Pranith Kumar
This patch wires up the new syscall sys_bpf() on powerpc. Signed-off-by: Pranith Kumar --- arch/powerpc/include/asm/systbl.h | 1 + arch/powerpc/include/asm/unistd.h | 2 +- arch/powerpc/include/uapi/asm/unistd.h | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH v2] networking: fm10k: Fix build failure

2014-10-09 Thread Pranith Kumar
`fm10k_ptp_unregister': (.text+0x12e7dc): undefined reference to `ptp_clock_unregister' Makefile:930: recipe for target 'vmlinux' failed Signed-off-by: Pranith Kumar --- drivers/net/ethernet/intel/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ethernet/intel/Kconfig b/drivers/net

[PATCH] networking: fm10k: Fix build failure

2014-10-09 Thread Pranith Kumar
`fm10k_ptp_unregister': (.text+0x12e7dc): undefined reference to `ptp_clock_unregister' Makefile:930: recipe for target 'vmlinux' failed Signed-off-by: Pranith Kumar --- drivers/net/ethernet/intel/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/intel/Kconfig b

<    1   2   3   4   5   6   7   8   9   10   >