Re: Mistake in include IS_ENABLED(CONFIG_LIVEPATCH)

2017-02-10 Thread Jiri Kosina
On Sat, 11 Feb 2017, Denys Fedoryshchenko wrote: > I noticed that sample of livepatch is not working in 4.9.9, because in > include, > linux/livepatch.h > it is: > #if IS_ENABLED(CONFIG_LIVEPATCH) > > while config option is: > CONFIG_HAVE_LIVEPATCH=y > > After editing livepatch.h sample module

Re: Mistake in include IS_ENABLED(CONFIG_LIVEPATCH)

2017-02-10 Thread Jiri Kosina
On Sat, 11 Feb 2017, Denys Fedoryshchenko wrote: > I noticed that sample of livepatch is not working in 4.9.9, because in > include, > linux/livepatch.h > it is: > #if IS_ENABLED(CONFIG_LIVEPATCH) > > while config option is: > CONFIG_HAVE_LIVEPATCH=y > > After editing livepatch.h sample module

Re: [PATCH] ASoC: fsl_sai: support more than 2 channels

2017-02-10 Thread Nicolin Chen
On Fri, Feb 10, 2017 at 07:42:43PM +0100, Alexandre Belloni wrote: > The FSL SAI can support up to 32 channels using TDM. Report that value so > they can actually be used. > > Tested using 8 channels. > > Signed-off-by: Alexandre Belloni Acked-by: Nicolin

Re: [PATCH] ASoC: fsl_sai: support more than 2 channels

2017-02-10 Thread Nicolin Chen
On Fri, Feb 10, 2017 at 07:42:43PM +0100, Alexandre Belloni wrote: > The FSL SAI can support up to 32 channels using TDM. Report that value so > they can actually be used. > > Tested using 8 channels. > > Signed-off-by: Alexandre Belloni Acked-by: Nicolin Chen > --- >

Re: [PULL] IIO fixes for 4.10 set 3 - a couple of regression fixes.

2017-02-10 Thread Greg Kroah-Hartman
On Fri, Feb 10, 2017 at 11:35:35PM +0100, Peter Rosin wrote: > > On Sun, Feb 05, 2017 at 10:35:02AM +, Jonathan Cameron wrote: > >> The following changes since commit > >> 5c113b5e0082e90d2e1c7b12e96a7b8cf0623e27: > >> > >> iio: dht11: Use usleep_range instead of msleep for start signal >

Re: [PULL] IIO fixes for 4.10 set 3 - a couple of regression fixes.

2017-02-10 Thread Greg Kroah-Hartman
On Fri, Feb 10, 2017 at 11:35:35PM +0100, Peter Rosin wrote: > > On Sun, Feb 05, 2017 at 10:35:02AM +, Jonathan Cameron wrote: > >> The following changes since commit > >> 5c113b5e0082e90d2e1c7b12e96a7b8cf0623e27: > >> > >> iio: dht11: Use usleep_range instead of msleep for start signal >

Re: [PATCH] device-dax: don't set kobj parent during cdev init

2017-02-10 Thread Greg Kroah-Hartman
On Fri, Feb 10, 2017 at 02:25:35PM -0800, Dan Williams wrote: > On Fri, Feb 10, 2017 at 12:17 PM, Greg Kroah-Hartman > wrote: > > On Fri, Feb 10, 2017 at 11:41:20AM -0800, Dan Williams wrote: > >> On Fri, Feb 10, 2017 at 11:19 AM, Logan Gunthorpe

Re: [PATCH] device-dax: don't set kobj parent during cdev init

2017-02-10 Thread Greg Kroah-Hartman
On Fri, Feb 10, 2017 at 02:25:35PM -0800, Dan Williams wrote: > On Fri, Feb 10, 2017 at 12:17 PM, Greg Kroah-Hartman > wrote: > > On Fri, Feb 10, 2017 at 11:41:20AM -0800, Dan Williams wrote: > >> On Fri, Feb 10, 2017 at 11:19 AM, Logan Gunthorpe > >> wrote: > >> > I copied this code and per

Re: [PATCH 2/2] sched/deadline: Throttle a constrained deadline task activated after the deadline

2017-02-10 Thread luca abeni
Hi Daniel, On Fri, 10 Feb 2017 20:48:11 +0100 Daniel Bristot de Oliveira wrote: > During the activation, CBS checks if it can reuse the current task's > runtime and period. If the deadline of the task is in the past, CBS > cannot use the runtime, and so it replenishes the

Re: [PATCH 2/2] sched/deadline: Throttle a constrained deadline task activated after the deadline

2017-02-10 Thread luca abeni
Hi Daniel, On Fri, 10 Feb 2017 20:48:11 +0100 Daniel Bristot de Oliveira wrote: > During the activation, CBS checks if it can reuse the current task's > runtime and period. If the deadline of the task is in the past, CBS > cannot use the runtime, and so it replenishes the task. This rule >

Re: [PATCH 1/2] sched/deadline: Replenishment timer should fire in the next period

2017-02-10 Thread luca abeni
Hi Daniel, On Fri, 10 Feb 2017 20:48:10 +0100 Daniel Bristot de Oliveira wrote: [...] > diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c > index 70ef2b1..3c94d85 100644 > --- a/kernel/sched/deadline.c > +++ b/kernel/sched/deadline.c > @@ -505,10 +505,15 @@

Re: [GIT PULL] PCI fixes for v4.10

2017-02-10 Thread Yinghai Lu
On Fri, Feb 10, 2017 at 6:39 PM, Yinghai Lu wrote: > Ashok, > > Can ask your QA guys check only attached patch and commit 68db9bc ? more clean patches: split that into two small patches. Thanks Yinghai From 68db9bc814362e7f24371c27d12a4f34477d9356 Mon Sep 17 00:00:00 2001

Re: [PATCH 1/2] sched/deadline: Replenishment timer should fire in the next period

2017-02-10 Thread luca abeni
Hi Daniel, On Fri, 10 Feb 2017 20:48:10 +0100 Daniel Bristot de Oliveira wrote: [...] > diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c > index 70ef2b1..3c94d85 100644 > --- a/kernel/sched/deadline.c > +++ b/kernel/sched/deadline.c > @@ -505,10 +505,15 @@ static void

Re: [GIT PULL] PCI fixes for v4.10

2017-02-10 Thread Yinghai Lu
On Fri, Feb 10, 2017 at 6:39 PM, Yinghai Lu wrote: > Ashok, > > Can ask your QA guys check only attached patch and commit 68db9bc ? more clean patches: split that into two small patches. Thanks Yinghai From 68db9bc814362e7f24371c27d12a4f34477d9356 Mon Sep 17 00:00:00 2001 From: Lukas Wunner

[PATCH] HID: intel-ish-hid: constify device_type structure

2017-02-10 Thread Bhumika Goyal
Declare device_type structure as const as it is only stored in the type field of a device structure. This field is of type const, so add const to the declaration of device_type structure. File size before: drivers/hid/intel-ish-hid/ishtp/bus.o textdata bss dec hex filename

[PATCH] HID: intel-ish-hid: constify device_type structure

2017-02-10 Thread Bhumika Goyal
Declare device_type structure as const as it is only stored in the type field of a device structure. This field is of type const, so add const to the declaration of device_type structure. File size before: drivers/hid/intel-ish-hid/ishtp/bus.o textdata bss dec hex filename

Re: [PATCH] staging: rtl8192u: Fix brace placement

2017-02-10 Thread Greg KH
On Sat, Feb 11, 2017 at 10:46:27AM +0530, simran singhal wrote: > Fix brace placement errors caught by checkpatch.pl ERROR: that open > brace { should be on the previous line > > Signed-off-by: simran singhal > --- > .../staging/rtl8192u/ieee80211/rtl819x_BAProc.c|

Re: [PATCH] staging: rtl8192u: Fix brace placement

2017-02-10 Thread Greg KH
On Sat, Feb 11, 2017 at 10:46:27AM +0530, simran singhal wrote: > Fix brace placement errors caught by checkpatch.pl ERROR: that open > brace { should be on the previous line > > Signed-off-by: simran singhal > --- > .../staging/rtl8192u/ieee80211/rtl819x_BAProc.c| 90 >

Re: [PATCH] staging: rtl8192u: Removing multiple blank lines

2017-02-10 Thread Greg KH
On Sat, Feb 11, 2017 at 09:34:12AM +0530, SIMRAN SINGHAL wrote: > Multiple patches ...? You sent me lots of patches, how am I supposed to know which one to apply in what order? > Can you please clarify what all patches you are including in "Multiple > Patches". Everything you have sent me. >

Re: [PATCH] staging: rtl8192u: Removing multiple blank lines

2017-02-10 Thread Greg KH
On Sat, Feb 11, 2017 at 09:34:12AM +0530, SIMRAN SINGHAL wrote: > Multiple patches ...? You sent me lots of patches, how am I supposed to know which one to apply in what order? > Can you please clarify what all patches you are including in "Multiple > Patches". Everything you have sent me. >

Re: [PATCH] staging:vt6656:channel.h: fix function definition argument without identifier name issue

2017-02-10 Thread Greg KH
On Sat, Feb 11, 2017 at 07:48:35AM +0530, Arushi Singhal wrote: > Hi > Sorry Greg but how this not applying to your mailing list. Your patch did not apply to my staging-next git tree. Probably because soemone else already did this same work before you did. Try rebasing your patch on my

Re: [PATCH] staging:vt6656:channel.h: fix function definition argument without identifier name issue

2017-02-10 Thread Greg KH
On Sat, Feb 11, 2017 at 07:48:35AM +0530, Arushi Singhal wrote: > Hi > Sorry Greg but how this not applying to your mailing list. Your patch did not apply to my staging-next git tree. Probably because soemone else already did this same work before you did. Try rebasing your patch on my

[PATCH] rbd: constify device_type structure

2017-02-10 Thread Bhumika Goyal
Declare device_type structure as const as it is only stored in the type field of a device structure. This field is of type const, so add const to the declaration of device_type structure. File size before: textdata bss dec hex filename 61546 11610 208 73364 11e94

[PATCH] rbd: constify device_type structure

2017-02-10 Thread Bhumika Goyal
Declare device_type structure as const as it is only stored in the type field of a device structure. This field is of type const, so add const to the declaration of device_type structure. File size before: textdata bss dec hex filename 61546 11610 208 73364 11e94

Re: [PATCH v4] fork: free vmapped stacks in cache when cpus are offline

2017-02-10 Thread Michal Hocko
On Sat 11-02-17 08:40:38, Hoeun Ryu wrote: > Using virtually mapped stack, kernel stacks are allocated via vmalloc. > In the current implementation, two stacks per cpu can be cached when > tasks are freed and the cached stacks are used again in task duplications. > but the cached stacks may

Re: [PATCH v4] fork: free vmapped stacks in cache when cpus are offline

2017-02-10 Thread Michal Hocko
On Sat 11-02-17 08:40:38, Hoeun Ryu wrote: > Using virtually mapped stack, kernel stacks are allocated via vmalloc. > In the current implementation, two stacks per cpu can be cached when > tasks are freed and the cached stacks are used again in task duplications. > but the cached stacks may

[GIT PULL] SCSI fixes for 4.10-rc7

2017-02-10 Thread James Bottomley
Six fairly small fixes. None is a real show stopper, two automation detected problems: one memory leak, one use after free and four others each of which fixes something that has been a significant source of annoyance to someone. The patch is available here:

[GIT PULL] SCSI fixes for 4.10-rc7

2017-02-10 Thread James Bottomley
Six fairly small fixes. None is a real show stopper, two automation detected problems: one memory leak, one use after free and four others each of which fixes something that has been a significant source of annoyance to someone. The patch is available here:

Re: [PATCH] f2fs: introduce nid cache

2017-02-10 Thread Chao Yu
On 2017/2/9 9:28, Jaegeuk Kim wrote: > On 02/08, Chao Yu wrote: >> On 2017/2/7 15:24, Chao Yu wrote: >>> Hi Jaegeuk, >>> >>> Happy Chinese New Year! :) >>> >>> On 2017/1/24 12:35, Jaegeuk Kim wrote: Hi Chao, On 01/22, Chao Yu wrote: > In scenario of intensively node allocation,

Re: [PATCH] f2fs: introduce nid cache

2017-02-10 Thread Chao Yu
On 2017/2/9 9:28, Jaegeuk Kim wrote: > On 02/08, Chao Yu wrote: >> On 2017/2/7 15:24, Chao Yu wrote: >>> Hi Jaegeuk, >>> >>> Happy Chinese New Year! :) >>> >>> On 2017/1/24 12:35, Jaegeuk Kim wrote: Hi Chao, On 01/22, Chao Yu wrote: > In scenario of intensively node allocation,

[PATCH] staging: rtl8192u: Fix RETURN_VOID warnings

2017-02-10 Thread simran singhal
Fix 'void function return statements are not generally useful' checkpatch.pl warnings. Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH] staging: rtl8192u: Fix RETURN_VOID warnings

2017-02-10 Thread simran singhal
Fix 'void function return statements are not generally useful' checkpatch.pl warnings. Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c

[PATCH] staging: rtl8192u: Fix brace placement

2017-02-10 Thread simran singhal
Fix brace placement errors caught by checkpatch.pl ERROR: that open brace { should be on the previous line Signed-off-by: simran singhal --- .../staging/rtl8192u/ieee80211/rtl819x_BAProc.c| 90 -- 1 file changed, 30 insertions(+), 60

[PATCH] staging: rtl8192u: Fix brace placement

2017-02-10 Thread simran singhal
Fix brace placement errors caught by checkpatch.pl ERROR: that open brace { should be on the previous line Signed-off-by: simran singhal --- .../staging/rtl8192u/ieee80211/rtl819x_BAProc.c| 90 -- 1 file changed, 30 insertions(+), 60 deletions(-) diff --git

Mistake in include IS_ENABLED(CONFIG_LIVEPATCH)

2017-02-10 Thread Denys Fedoryshchenko
Hello, I noticed that sample of livepatch is not working in 4.9.9, because in include, linux/livepatch.h it is: #if IS_ENABLED(CONFIG_LIVEPATCH) while config option is: CONFIG_HAVE_LIVEPATCH=y After editing livepatch.h sample module compiles fine Probably that's just a typo?

Mistake in include IS_ENABLED(CONFIG_LIVEPATCH)

2017-02-10 Thread Denys Fedoryshchenko
Hello, I noticed that sample of livepatch is not working in 4.9.9, because in include, linux/livepatch.h it is: #if IS_ENABLED(CONFIG_LIVEPATCH) while config option is: CONFIG_HAVE_LIVEPATCH=y After editing livepatch.h sample module compiles fine Probably that's just a typo?

[PATCH] staging: dgnc: dgnc_tty.c: fix argument list alignment issue.

2017-02-10 Thread Nathan Howard
Fix checkpatch.pl issue of the form: "CHECK: Alignment should match open parenthesis". Signed-off-by: Nathan Howard --- drivers/staging/dgnc/dgnc_tty.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c

[PATCH] staging: dgnc: dgnc_tty.c: fix argument list alignment issue.

2017-02-10 Thread Nathan Howard
Fix checkpatch.pl issue of the form: "CHECK: Alignment should match open parenthesis". Signed-off-by: Nathan Howard --- drivers/staging/dgnc/dgnc_tty.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c

Re: [PATCH v2] arm64: dts: Enable ir-spi in the tm2 and tm2e boards

2017-02-10 Thread Andi Shyti
Hi Javier, On Fri, Feb 10, 2017 at 11:04:50AM -0300, Javier Martinez Canillas wrote: > On 02/09/2017 11:22 PM, Andi Shyti wrote: ... > > + irda_regulator: irda-regulator { > > + compatible = "regulator-fixed"; > > + enable-active-high; > > + gpio = < 3

Re: [PATCH v2] arm64: dts: Enable ir-spi in the tm2 and tm2e boards

2017-02-10 Thread Andi Shyti
Hi Javier, On Fri, Feb 10, 2017 at 11:04:50AM -0300, Javier Martinez Canillas wrote: > On 02/09/2017 11:22 PM, Andi Shyti wrote: ... > > + irda_regulator: irda-regulator { > > + compatible = "regulator-fixed"; > > + enable-active-high; > > + gpio = < 3

Re: module: Optimize search_module_extables()

2017-02-10 Thread Jessica Yu
+++ Peter Zijlstra [08/02/17 15:48 +0100]: While looking through the __ex_table stuff I found that we do a linear lookup of the module. Also fix up a comment. Signed-off-by: Peter Zijlstra (Intel) Applied, thanks. Hm. A quick scan through module.c still shows a couple

Re: module: Optimize search_module_extables()

2017-02-10 Thread Jessica Yu
+++ Peter Zijlstra [08/02/17 15:48 +0100]: While looking through the __ex_table stuff I found that we do a linear lookup of the module. Also fix up a comment. Signed-off-by: Peter Zijlstra (Intel) Applied, thanks. Hm. A quick scan through module.c still shows a couple of places that use

[PATCH] staging: greybus: arpc.h: remove duplicate line.

2017-02-10 Thread Nathan Howard
Fix checkpatch.pl issue of the form: "CHECK: Please don't use multiple blank lines". Signed-off-by: Nathan Howard --- drivers/staging/greybus/arpc.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/greybus/arpc.h b/drivers/staging/greybus/arpc.h index

[PATCH] staging: greybus: arpc.h: remove duplicate line.

2017-02-10 Thread Nathan Howard
Fix checkpatch.pl issue of the form: "CHECK: Please don't use multiple blank lines". Signed-off-by: Nathan Howard --- drivers/staging/greybus/arpc.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/greybus/arpc.h b/drivers/staging/greybus/arpc.h index 7fbddfc..c0b63c0 100644

Re: [PATCH] staging: rtl8192u: Removing multiple blank lines

2017-02-10 Thread SIMRAN SINGHAL
Multiple patches ...? Can you please clarify what all patches you are including in "Multiple Patches". And the Order you should go for is the order in which I submitted them. On Sat, Feb 11, 2017 at 7:48 AM, SIMRAN SINGHAL wrote: > Multiple patches ...? > Can you

Re: [PATCH] staging: rtl8192u: Removing multiple blank lines

2017-02-10 Thread SIMRAN SINGHAL
Multiple patches ...? Can you please clarify what all patches you are including in "Multiple Patches". And the Order you should go for is the order in which I submitted them. On Sat, Feb 11, 2017 at 7:48 AM, SIMRAN SINGHAL wrote: > Multiple patches ...? > Can you please clarify what all patches

[PATCH] block/loop: fix race between I/O and set_status

2017-02-10 Thread Ming Lei
Inside set_status, transfer need to setup again, so we have to drain IO before the transition, otherwise oops may be triggered like the following: divide error: [#1] SMP KASAN CPU: 0 PID: 2935 Comm: loop7 Not tainted 4.10.0-rc7+ #213 Hardware name: QEMU Standard PC

[PATCH] block/loop: fix race between I/O and set_status

2017-02-10 Thread Ming Lei
Inside set_status, transfer need to setup again, so we have to drain IO before the transition, otherwise oops may be triggered like the following: divide error: [#1] SMP KASAN CPU: 0 PID: 2935 Comm: loop7 Not tainted 4.10.0-rc7+ #213 Hardware name: QEMU Standard PC

Re: [PATCH] checkpatch: add warning on %pk instead of %pK usage

2017-02-10 Thread Joe Perches
On Sat, 2017-02-11 at 01:32 +, Roberts, William C wrote: > > > > By "normal" I'm referring to things that call into pointer(), just > > > casually looking I see bstr_printf vsnprintf kvasprintf, which would > > > be easy enough to add > > > > > > > What do you think is missing? sn?printf ?

Re: [PATCH] checkpatch: add warning on %pk instead of %pK usage

2017-02-10 Thread Joe Perches
On Sat, 2017-02-11 at 01:32 +, Roberts, William C wrote: > > > > By "normal" I'm referring to things that call into pointer(), just > > > casually looking I see bstr_printf vsnprintf kvasprintf, which would > > > be easy enough to add > > > > > > > What do you think is missing? sn?printf ?

Re: [PATCH 3/3] DT: add Faraday Tec. as vendor

2017-02-10 Thread Joel Stanley
On Fri, Feb 10, 2017 at 11:46 PM, Linus Walleij wrote: > On Wed, Feb 8, 2017 at 9:00 PM, Hans Ulli Kroll > wrote: > >> add Faraday Technology Corporation as vendor faraday for DT >> >> Signed-off-by: Hans Ulli Kroll

Re: [PATCH 3/3] DT: add Faraday Tec. as vendor

2017-02-10 Thread Joel Stanley
On Fri, Feb 10, 2017 at 11:46 PM, Linus Walleij wrote: > On Wed, Feb 8, 2017 at 9:00 PM, Hans Ulli Kroll > wrote: > >> add Faraday Technology Corporation as vendor faraday for DT >> >> Signed-off-by: Hans Ulli Kroll > > Reviewed-by: Linus Walleij > > I think I should use this for the PCI block

Re: [PATCH v4] drivers/misc: Add Aspeed LPC control driver

2017-02-10 Thread Joel Stanley
Hey Greg, On Sat, Feb 11, 2017 at 1:00 AM, Greg KH wrote: > On Wed, Feb 08, 2017 at 10:42:47AM +1100, Cyril Bur wrote: >> In order to manage server systems, there is typically another processor >> known as a BMC (Baseboard Management Controller) which is responsible

Re: [PATCH v4] drivers/misc: Add Aspeed LPC control driver

2017-02-10 Thread Joel Stanley
Hey Greg, On Sat, Feb 11, 2017 at 1:00 AM, Greg KH wrote: > On Wed, Feb 08, 2017 at 10:42:47AM +1100, Cyril Bur wrote: >> In order to manage server systems, there is typically another processor >> known as a BMC (Baseboard Management Controller) which is responsible >> for powering the server

Re: [GIT PULL] PCI fixes for v4.10

2017-02-10 Thread Yinghai Lu
On Thu, Feb 9, 2017 at 12:11 PM, Bjorn Helgaas wrote: > On Thu, Feb 09, 2017 at 09:09:50AM -0600, Bjorn Helgaas wrote: >> [+cc Ashok, Keith] >> >> On Thu, Feb 09, 2017 at 05:06:48AM +0100, Lukas Wunner wrote: >> > On Wed, Feb 08, 2017 at 01:22:56PM -0600, Bjorn Helgaas wrote:

Re: [GIT PULL] PCI fixes for v4.10

2017-02-10 Thread Yinghai Lu
On Thu, Feb 9, 2017 at 12:11 PM, Bjorn Helgaas wrote: > On Thu, Feb 09, 2017 at 09:09:50AM -0600, Bjorn Helgaas wrote: >> [+cc Ashok, Keith] >> >> On Thu, Feb 09, 2017 at 05:06:48AM +0100, Lukas Wunner wrote: >> > On Wed, Feb 08, 2017 at 01:22:56PM -0600, Bjorn Helgaas wrote: >> > > Bjorn Helgaas

[PATCH v6 2/8] devicetree: property-units: Add uWh and uAh units

2017-02-10 Thread Liam Breck
From: Matt Ranostay Add entries for microwatt-hours and microamp-hours. Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Matt Ranostay

[PATCH v6 2/8] devicetree: property-units: Add uWh and uAh units

2017-02-10 Thread Liam Breck
From: Matt Ranostay Add entries for microwatt-hours and microamp-hours. Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Matt Ranostay Signed-off-by: Liam Breck Acked-by: Sebastian Reichel Acked-by: Rob Herring ---

Re: Is it really safe to use workqueues to drive expedited grace periods?

2017-02-10 Thread Tejun Heo
Hello, Paul. On Fri, Feb 10, 2017 at 01:21:58PM -0800, Paul E. McKenney wrote: > So RCU's expedited grace periods have been using workqueues for a > little while, and things seem to be working. But as usual, I worry... > Is this use subject to some sort of deadlock where RCU's workqueue cannot >

Re: Is it really safe to use workqueues to drive expedited grace periods?

2017-02-10 Thread Tejun Heo
Hello, Paul. On Fri, Feb 10, 2017 at 01:21:58PM -0800, Paul E. McKenney wrote: > So RCU's expedited grace periods have been using workqueues for a > little while, and things seem to be working. But as usual, I worry... > Is this use subject to some sort of deadlock where RCU's workqueue cannot >

Re: [RFC PATCH 2/2] mm/sparse: add last_section_nr in sparse_init() to reduce some iteration cycle

2017-02-10 Thread Tejun Heo
Hello, On Sat, Feb 11, 2017 at 10:18:29AM +0800, Wei Yang wrote: > During the sparse_init(), it iterate on each possible section. On x86_64, > it would always be (2^19) even there is not much memory. For example, on a > typical 4G machine, it has only (2^5) to (2^6) present sections. This >

Re: [RFC PATCH 2/2] mm/sparse: add last_section_nr in sparse_init() to reduce some iteration cycle

2017-02-10 Thread Tejun Heo
Hello, On Sat, Feb 11, 2017 at 10:18:29AM +0800, Wei Yang wrote: > During the sparse_init(), it iterate on each possible section. On x86_64, > it would always be (2^19) even there is not much memory. For example, on a > typical 4G machine, it has only (2^5) to (2^6) present sections. This >

[RFC PATCH 2/2] mm/sparse: add last_section_nr in sparse_init() to reduce some iteration cycle

2017-02-10 Thread Wei Yang
During the sparse_init(), it iterate on each possible section. On x86_64, it would always be (2^19) even there is not much memory. For example, on a typical 4G machine, it has only (2^5) to (2^6) present sections. This benefits more on a system with smaller memory. This patch calculates the last

[RFC PATCH 2/2] mm/sparse: add last_section_nr in sparse_init() to reduce some iteration cycle

2017-02-10 Thread Wei Yang
During the sparse_init(), it iterate on each possible section. On x86_64, it would always be (2^19) even there is not much memory. For example, on a typical 4G machine, it has only (2^5) to (2^6) present sections. This benefits more on a system with smaller memory. This patch calculates the last

[RFC PATCH 1/2] mm/memblock: introduce for_each_mem_pfn_range_rev()

2017-02-10 Thread Wei Yang
This patch introduces the helper function for_each_mem_pfn_range_rev() for later use. Signed-off-by: Wei Yang --- include/linux/memblock.h | 18 ++ mm/memblock.c| 39 ++- 2 files changed, 56

[RFC PATCH 1/2] mm/memblock: introduce for_each_mem_pfn_range_rev()

2017-02-10 Thread Wei Yang
This patch introduces the helper function for_each_mem_pfn_range_rev() for later use. Signed-off-by: Wei Yang --- include/linux/memblock.h | 18 ++ mm/memblock.c| 39 ++- 2 files changed, 56 insertions(+), 1 deletion(-) diff --git

Re: [PATCH v2 2/5] time: mark syscore_ops as __ro_after_init

2017-02-10 Thread John Stultz
On Fri, Feb 10, 2017 at 5:37 PM, Jess Frazelle wrote: > Marked syscore_ops structs as __ro_after_init when register_syscore_ops was > called only during init. Most of the caller functions were already annotated > as > __init. > unregister_syscore_ops() was never called on

Re: [PATCH v2 2/5] time: mark syscore_ops as __ro_after_init

2017-02-10 Thread John Stultz
On Fri, Feb 10, 2017 at 5:37 PM, Jess Frazelle wrote: > Marked syscore_ops structs as __ro_after_init when register_syscore_ops was > called only during init. Most of the caller functions were already annotated > as > __init. > unregister_syscore_ops() was never called on these ops. > This

[PATCH v13 07/12] usb: ehci: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
Set the dma for ehci from sysdev. The sysdev is pointing to device that is known to the system firmware or hardware. Cc: Arnd Bergmann Cc: Sriram Dash Signed-off-by: Peter Chen Acked-by: Alan Stern ---

[PATCH v13 07/12] usb: ehci: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
Set the dma for ehci from sysdev. The sysdev is pointing to device that is known to the system firmware or hardware. Cc: Arnd Bergmann Cc: Sriram Dash Signed-off-by: Peter Chen Acked-by: Alan Stern --- drivers/usb/host/ehci-mem.c | 12 ++-- 1 file changed, 6 insertions(+), 6

[PATCH v13 06/12] usb: xhci: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
From: Arnd Bergmann For xhci-hcd platform device, all the DMA parameters are not configured properly, notably dma ops for dwc3 devices. So, set the dma for xhci from sysdev. sysdev is pointing to device that is known to the system firmware or hardware. Cc: Baolin Wang

[PATCH v13 06/12] usb: xhci: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
From: Arnd Bergmann For xhci-hcd platform device, all the DMA parameters are not configured properly, notably dma ops for dwc3 devices. So, set the dma for xhci from sysdev. sysdev is pointing to device that is known to the system firmware or hardware. Cc: Baolin Wang Cc: Vivek Gautam Cc:

[PATCH v13 10/12] ARM: dts: imx6qdl: Enable usb node children with

2017-02-10 Thread Peter Chen
From: Joshua Clayton Give usb nodes #address and #size attributes, so that a child node representing a permanently connected device such as an onboard hub may be addressed with a attribute Signed-off-by: Joshua Clayton Signed-off-by: Peter

[PATCH v13 10/12] ARM: dts: imx6qdl: Enable usb node children with

2017-02-10 Thread Peter Chen
From: Joshua Clayton Give usb nodes #address and #size attributes, so that a child node representing a permanently connected device such as an onboard hub may be addressed with a attribute Signed-off-by: Joshua Clayton Signed-off-by: Peter Chen --- arch/arm/boot/dts/imx6qdl.dtsi | 6 ++

[PATCH v13 02/12] power: add power sequence library

2017-02-10 Thread Peter Chen
We have an well-known problem that the device needs to do some power sequence before it can be recognized by related host, the typical example like hard-wired mmc devices and usb devices. This power sequence is hard to be described at device tree and handled by related host driver, so we have

[PATCH v13 02/12] power: add power sequence library

2017-02-10 Thread Peter Chen
We have an well-known problem that the device needs to do some power sequence before it can be recognized by related host, the typical example like hard-wired mmc devices and usb devices. This power sequence is hard to be described at device tree and handled by related host driver, so we have

[PATCH v13 00/12] power: add power sequence library

2017-02-10 Thread Peter Chen
Hi all, This is a follow-up for my last power sequence framework patch set [1]. According to Rob Herring and Ulf Hansson's comments[2]. The kinds of power sequence instances will be added at postcore_initcall, the match criteria is compatible string first, if the compatible string is not matched

[PATCH v13 00/12] power: add power sequence library

2017-02-10 Thread Peter Chen
Hi all, This is a follow-up for my last power sequence framework patch set [1]. According to Rob Herring and Ulf Hansson's comments[2]. The kinds of power sequence instances will be added at postcore_initcall, the match criteria is compatible string first, if the compatible string is not matched

[PATCH v13 12/12] ARM: dts: imx6q-evi: Fix onboard hub reset line

2017-02-10 Thread Peter Chen
From: Joshua Clayton Previously the onboard hub was made to work by treating its reset gpio as a regulator enable. Get rid of that kludge now that pwseq has added reset gpio support Move pin muxing the hub reset pin into the usbh1 group Signed-off-by: Joshua Clayton

[PATCH v13 12/12] ARM: dts: imx6q-evi: Fix onboard hub reset line

2017-02-10 Thread Peter Chen
From: Joshua Clayton Previously the onboard hub was made to work by treating its reset gpio as a regulator enable. Get rid of that kludge now that pwseq has added reset gpio support Move pin muxing the hub reset pin into the usbh1 group Signed-off-by: Joshua Clayton Signed-off-by: Peter Chen

[PATCH v13 03/12] usb: separate out sysdev pointer from usb_bus

2017-02-10 Thread Peter Chen
From: Arnd Bergmann For xhci-hcd platform device, all the DMA parameters are not configured properly, notably dma ops for dwc3 devices. The idea here is that you pass in the parent of_node along with the child device pointer, so it would behave exactly like the parent already

[PATCH v13 03/12] usb: separate out sysdev pointer from usb_bus

2017-02-10 Thread Peter Chen
From: Arnd Bergmann For xhci-hcd platform device, all the DMA parameters are not configured properly, notably dma ops for dwc3 devices. The idea here is that you pass in the parent of_node along with the child device pointer, so it would behave exactly like the parent already does. The

[PATCH v13 04/12] usb: chipidea: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
From: Arnd Bergmann Set the dma for chipidea from sysdev. This is inherited from its parent node. Also, do not set dma mask for child as it is not required now. Signed-off-by: Arnd Bergmann Signed-off-by: Sriram Dash Acked-by: Peter Chen

[PATCH v13 11/12] ARM: dts: imx6qdl-udoo.dtsi: fix onboard USB HUB property

2017-02-10 Thread Peter Chen
The current dts describes USB HUB's property at USB controller's entry, it is improper. The USB HUB should be the child node under USB controller, and power sequence properties are under it. Besides, using gpio pinctrl setting for USB2415's reset pin. Signed-off-by: Peter Chen

[PATCH v13 09/12] usb: core: add power sequence handling for USB devices

2017-02-10 Thread Peter Chen
Some hard-wired USB devices need to do power sequence to let the device work normally, the typical power sequence like: enable USB PHY clock, toggle reset pin, etc. But current Linux USB driver lacks of such code to do it, it may cause some hard-wired USB devices works abnormal or can't be

[PATCH v13 04/12] usb: chipidea: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
From: Arnd Bergmann Set the dma for chipidea from sysdev. This is inherited from its parent node. Also, do not set dma mask for child as it is not required now. Signed-off-by: Arnd Bergmann Signed-off-by: Sriram Dash Acked-by: Peter Chen Signed-off-by: Mathias Nyman ---

[PATCH v13 11/12] ARM: dts: imx6qdl-udoo.dtsi: fix onboard USB HUB property

2017-02-10 Thread Peter Chen
The current dts describes USB HUB's property at USB controller's entry, it is improper. The USB HUB should be the child node under USB controller, and power sequence properties are under it. Besides, using gpio pinctrl setting for USB2415's reset pin. Signed-off-by: Peter Chen Signed-off-by:

[PATCH v13 09/12] usb: core: add power sequence handling for USB devices

2017-02-10 Thread Peter Chen
Some hard-wired USB devices need to do power sequence to let the device work normally, the typical power sequence like: enable USB PHY clock, toggle reset pin, etc. But current Linux USB driver lacks of such code to do it, it may cause some hard-wired USB devices works abnormal or can't be

[PATCH] net: ethernet: ti: cpsw: return NET_XMIT_DROP if skb_padto failed

2017-02-10 Thread Ivan Khoronzhuk
If skb_padto failed the skb has been dropped already, so it was consumed, but it doesn't mean it was sent, thus no need to update queue tx time, etc. So, return NET_XMIT_DROP as more appropriate. Signed-off-by: Ivan Khoronzhuk --- Based on net-next/master

[PATCH v13 05/12] usb: ehci: fsl: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
From: Arnd Bergmann For the dual role ehci fsl driver, sysdev will handle the dma config. Signed-off-by: Arnd Bergmann Signed-off-by: Sriram Dash Signed-off-by: Mathias Nyman --- drivers/usb/host/ehci-fsl.c |

[PATCH v13 01/12] binding-doc: power: pwrseq-generic: add binding doc for generic power sequence library

2017-02-10 Thread Peter Chen
Add binding doc for generic power sequence library. Signed-off-by: Peter Chen Acked-by: Philipp Zabel Acked-by: Rob Herring --- .../bindings/power/pwrseq/pwrseq-generic.txt | 48 ++ 1 file changed, 48

[PATCH] net: ethernet: ti: cpsw: return NET_XMIT_DROP if skb_padto failed

2017-02-10 Thread Ivan Khoronzhuk
If skb_padto failed the skb has been dropped already, so it was consumed, but it doesn't mean it was sent, thus no need to update queue tx time, etc. So, return NET_XMIT_DROP as more appropriate. Signed-off-by: Ivan Khoronzhuk --- Based on net-next/master drivers/net/ethernet/ti/cpsw.c | 2 +-

[PATCH v13 05/12] usb: ehci: fsl: use bus->sysdev for DMA configuration

2017-02-10 Thread Peter Chen
From: Arnd Bergmann For the dual role ehci fsl driver, sysdev will handle the dma config. Signed-off-by: Arnd Bergmann Signed-off-by: Sriram Dash Signed-off-by: Mathias Nyman --- drivers/usb/host/ehci-fsl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v13 01/12] binding-doc: power: pwrseq-generic: add binding doc for generic power sequence library

2017-02-10 Thread Peter Chen
Add binding doc for generic power sequence library. Signed-off-by: Peter Chen Acked-by: Philipp Zabel Acked-by: Rob Herring --- .../bindings/power/pwrseq/pwrseq-generic.txt | 48 ++ 1 file changed, 48 insertions(+) create mode 100644

[PATCH v13 08/12] binding-doc: usb: usb-device: add optional properties for power sequence

2017-02-10 Thread Peter Chen
Add optional properties for power sequence. Signed-off-by: Peter Chen Acked-by: Rob Herring --- Documentation/devicetree/bindings/usb/usb-device.txt | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git

[PATCH v13 08/12] binding-doc: usb: usb-device: add optional properties for power sequence

2017-02-10 Thread Peter Chen
Add optional properties for power sequence. Signed-off-by: Peter Chen Acked-by: Rob Herring --- Documentation/devicetree/bindings/usb/usb-device.txt | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/usb/usb-device.txt

[PATCH] usb: musb: add code comment for clarification

2017-02-10 Thread Gustavo A. R. Silva
Add code comment to make it clear that the fall-through is intentional. Read the link for more details: https://lkml.org/lkml/2017/2/9/292 Signed-off-by: Gustavo A. R. Silva --- drivers/usb/musb/musb_core.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH 2/3] Bluetooth: cmtp: fix possible might sleep error in cmtp_session

2017-02-10 Thread Brian Norris
Hi, On Tue, Jan 24, 2017 at 12:07:50PM +0800, Jeffy Chen wrote: > It looks like cmtp_session has same pattern as the issue reported in > old rfcomm: > > while (1) { > set_current_state(TASK_INTERRUPTIBLE); > if (condition) > break; >

[PATCH] usb: musb: add code comment for clarification

2017-02-10 Thread Gustavo A. R. Silva
Add code comment to make it clear that the fall-through is intentional. Read the link for more details: https://lkml.org/lkml/2017/2/9/292 Signed-off-by: Gustavo A. R. Silva --- drivers/usb/musb/musb_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/musb/musb_core.c

Re: [PATCH 2/3] Bluetooth: cmtp: fix possible might sleep error in cmtp_session

2017-02-10 Thread Brian Norris
Hi, On Tue, Jan 24, 2017 at 12:07:50PM +0800, Jeffy Chen wrote: > It looks like cmtp_session has same pattern as the issue reported in > old rfcomm: > > while (1) { > set_current_state(TASK_INTERRUPTIBLE); > if (condition) > break; >

  1   2   3   4   5   6   7   8   9   10   >