Re: [PATCH v3 2/9] drivers: irqchip: Add STM32 external interrupts support

2016-09-02 Thread Thomas Gleixner
Alexandre, On Fri, 2 Sep 2016, Alexandre TORGUE wrote: This all looks very reasonable. The only complaint I have is your variable declaration ordering or the lack thereof. 1) > + struct irq_domain *domain = irq_desc_get_handler_data(desc); > + struct irq_chip_generic *gc =

Re: [PATCH v3 2/9] drivers: irqchip: Add STM32 external interrupts support

2016-09-02 Thread Thomas Gleixner
Alexandre, On Fri, 2 Sep 2016, Alexandre TORGUE wrote: This all looks very reasonable. The only complaint I have is your variable declaration ordering or the lack thereof. 1) > + struct irq_domain *domain = irq_desc_get_handler_data(desc); > + struct irq_chip_generic *gc =

Re: [PATCH 3/3] nvme: Enable autonomous power state transitions

2016-09-02 Thread Andy Lutomirski
On Fri, Sep 2, 2016 at 11:11 AM, J Freyensee wrote: > >> > > >> > > >> > > > >> > > > + /* >> > > > + * By default, allow up to 25ms of APST-induced >> > > > latency. This will >> > > > + * have no effect on non-APST supporting controllers >> > >

Re: [PATCH 3/3] nvme: Enable autonomous power state transitions

2016-09-02 Thread Andy Lutomirski
On Fri, Sep 2, 2016 at 11:11 AM, J Freyensee wrote: > >> > > >> > > >> > > > >> > > > + /* >> > > > + * By default, allow up to 25ms of APST-induced >> > > > latency. This will >> > > > + * have no effect on non-APST supporting controllers >> > > > (i.e. >> > > > any >> > > > +

Re: [PATCH] Fix chance of sign extension to nsec after its msb is set during calculation.

2016-09-02 Thread Thomas Gleixner
On Fri, 2 Sep 2016, Thomas Gleixner wrote: > > As a side note. John, why is that stuff unsigned at all? Shouldn't we use s/unsigned/signed/ > u64 for all of this? > > Thanks, > > tglx >

Re: [PATCH] Fix chance of sign extension to nsec after its msb is set during calculation.

2016-09-02 Thread Thomas Gleixner
On Fri, 2 Sep 2016, Thomas Gleixner wrote: > > As a side note. John, why is that stuff unsigned at all? Shouldn't we use s/unsigned/signed/ > u64 for all of this? > > Thanks, > > tglx >

[PATCH net-next v2 1/3] net: dsa: mv88e6xxx: fix module naming

2016-09-02 Thread Vivien Didelot
Since the mv88e6xxx.c file has been renamed, the driver compiled as a module is called chip.ko instead of mv88e6xxx.ko. Fix this. Fixes: fad09c73c270 ("net: dsa: mv88e6xxx: rename single-chip support") Signed-off-by: Vivien Didelot ---

[PATCH net-next v2 1/3] net: dsa: mv88e6xxx: fix module naming

2016-09-02 Thread Vivien Didelot
Since the mv88e6xxx.c file has been renamed, the driver compiled as a module is called chip.ko instead of mv88e6xxx.ko. Fix this. Fixes: fad09c73c270 ("net: dsa: mv88e6xxx: rename single-chip support") Signed-off-by: Vivien Didelot --- drivers/net/dsa/mv88e6xxx/Makefile | 3 ++- 1 file changed,

Re: Memory barrier needed with wake_up_process()?

2016-09-02 Thread Paul E. McKenney
On Fri, Sep 02, 2016 at 02:10:13PM -0400, Alan Stern wrote: > Paul, Peter, and Ingo: > > This must have come up before, but I don't know what was decided. > > Isn't it often true that a memory barrier is needed before a call to > wake_up_process()? A typical scenario might look like this: > >

[PATCH net-next v2 0/3] net: dsa: mv88e6xxx: isolate Global2 support

2016-09-02 Thread Vivien Didelot
Registers of Marvell chips are organized in internal SMI devices. One of them at address 0x1C is called Global2. It provides an extended set of registers, used for interrupt control, EEPROM access, indirect PHY access (to bypass the PHY Polling Unit) and cross-chip setup. Most chips have it, but

[PATCH net-next v2 0/3] net: dsa: mv88e6xxx: isolate Global2 support

2016-09-02 Thread Vivien Didelot
Registers of Marvell chips are organized in internal SMI devices. One of them at address 0x1C is called Global2. It provides an extended set of registers, used for interrupt control, EEPROM access, indirect PHY access (to bypass the PHY Polling Unit) and cross-chip setup. Most chips have it, but

Re: Memory barrier needed with wake_up_process()?

2016-09-02 Thread Paul E. McKenney
On Fri, Sep 02, 2016 at 02:10:13PM -0400, Alan Stern wrote: > Paul, Peter, and Ingo: > > This must have come up before, but I don't know what was decided. > > Isn't it often true that a memory barrier is needed before a call to > wake_up_process()? A typical scenario might look like this: > >

[PATCH net-next v2 3/3] net: dsa: mv88e6xxx: make global2 code optional

2016-09-02 Thread Vivien Didelot
Since not every chip has a Global2 set of registers, make its support optional, in which case the related functions will return -EOPNOTSUPP. This also allows to reduce the size of the mv88e6xxx driver for devices such as home routers embedding Ethernet chips without Global2 support. It is

[PATCH net-next v2 2/3] net: dsa: mv88e6xxx: move Global2 code

2016-09-02 Thread Vivien Didelot
Marvell chips are composed of multiple SMI devices. One of them at address 0x1C is called Global2. It provides an extended set of registers, used for interrupt control, EEPROM access, indirect PHY access (to bypass the PHY Polling Unit) and cross-chip related setup. Most chips have it, but some

[PATCH net-next v2 3/3] net: dsa: mv88e6xxx: make global2 code optional

2016-09-02 Thread Vivien Didelot
Since not every chip has a Global2 set of registers, make its support optional, in which case the related functions will return -EOPNOTSUPP. This also allows to reduce the size of the mv88e6xxx driver for devices such as home routers embedding Ethernet chips without Global2 support. It is

[PATCH net-next v2 2/3] net: dsa: mv88e6xxx: move Global2 code

2016-09-02 Thread Vivien Didelot
Marvell chips are composed of multiple SMI devices. One of them at address 0x1C is called Global2. It provides an extended set of registers, used for interrupt control, EEPROM access, indirect PHY access (to bypass the PHY Polling Unit) and cross-chip related setup. Most chips have it, but some

[tip:smp/hotplug] cpu/hotplug: Handle unbalanced hotplug enable/disable

2016-09-02 Thread tip-bot for Lianwei Wang
Commit-ID: 01b41159066531cc8d664362ff0cd89dd137bbfa Gitweb: http://git.kernel.org/tip/01b41159066531cc8d664362ff0cd89dd137bbfa Author: Lianwei Wang AuthorDate: Thu, 9 Jun 2016 23:43:28 -0700 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep

[tip:smp/hotplug] cpu/hotplug: Handle unbalanced hotplug enable/disable

2016-09-02 Thread tip-bot for Lianwei Wang
Commit-ID: 01b41159066531cc8d664362ff0cd89dd137bbfa Gitweb: http://git.kernel.org/tip/01b41159066531cc8d664362ff0cd89dd137bbfa Author: Lianwei Wang AuthorDate: Thu, 9 Jun 2016 23:43:28 -0700 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:37:17 +0200 cpu/hotplug: Handle

Re: [PATCH] f2fs: merge WRITE bio into previous WRITE_SYNC

2016-09-02 Thread Jaegeuk Kim
On Fri, Sep 02, 2016 at 03:33:33PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On 2016/8/27 8:53, Jaegeuk Kim wrote: > > This can avoid bio splits due to different op_flags. > > I thought about this, but I think this is not a good idea to increase merging > ratio of pages in bio. It breaks the rule

Re: [PATCH] f2fs: merge WRITE bio into previous WRITE_SYNC

2016-09-02 Thread Jaegeuk Kim
On Fri, Sep 02, 2016 at 03:33:33PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On 2016/8/27 8:53, Jaegeuk Kim wrote: > > This can avoid bio splits due to different op_flags. > > I thought about this, but I think this is not a good idea to increase merging > ratio of pages in bio. It breaks the rule

Re: [PATCH] dmaengine: tegra210-adma: Add memcpy support

2016-09-02 Thread Nicolin Chen
Hi Vinod, On Fri, Sep 02, 2016 at 04:55:32PM +0530, Vinod Koul wrote: > On Thu, Sep 01, 2016 at 03:43:16PM -0700, Nicolin Chen wrote: > > > +#define ADMA_CH_CTRL_MODE_ONCE (1 << 8) > > BIT(8)? You should change the existing ones too :) Ah..right...Will change them

Re: [PATCH] dmaengine: tegra210-adma: Add memcpy support

2016-09-02 Thread Nicolin Chen
Hi Vinod, On Fri, Sep 02, 2016 at 04:55:32PM +0530, Vinod Koul wrote: > On Thu, Sep 01, 2016 at 03:43:16PM -0700, Nicolin Chen wrote: > > > +#define ADMA_CH_CTRL_MODE_ONCE (1 << 8) > > BIT(8)? You should change the existing ones too :) Ah..right...Will change them

Re: [PATCH] Fix chance of sign extension to nsec after its msb is set during calculation.

2016-09-02 Thread Thomas Gleixner
On Fri, 2 Sep 2016, Thomas Gleixner wrote: > On Thu, 1 Sep 2016, Liav Rehana wrote: > > From: Liav Rehana > > > > During the calculation of the nsec variable, "delta * tkr->mult" may cause > > overflow to the msb, if the suspended time is too long. > > In that case, we need

Re: [PATCH] Fix chance of sign extension to nsec after its msb is set during calculation.

2016-09-02 Thread Thomas Gleixner
On Fri, 2 Sep 2016, Thomas Gleixner wrote: > On Thu, 1 Sep 2016, Liav Rehana wrote: > > From: Liav Rehana > > > > During the calculation of the nsec variable, "delta * tkr->mult" may cause > > overflow to the msb, if the suspended time is too long. > > In that case, we need to guarantee that the

Re: [Patch v9] driver/clk/clk-si5338: Add common clock framework driver for si5338

2016-09-02 Thread york sun
On 09/02/2016 07:04 AM, Rob Herring wrote: > On Fri, Aug 26, 2016 at 02:45:49PM -0700, York Sun wrote: >> From: York Sun >> >> SI5338 is a programmable clock generator. It has 4 sets of inputs, >> PLL, multisynth and dividers to make 4 outputs. This driver splits >> them

Re: [Patch v9] driver/clk/clk-si5338: Add common clock framework driver for si5338

2016-09-02 Thread york sun
On 09/02/2016 07:04 AM, Rob Herring wrote: > On Fri, Aug 26, 2016 at 02:45:49PM -0700, York Sun wrote: >> From: York Sun >> >> SI5338 is a programmable clock generator. It has 4 sets of inputs, >> PLL, multisynth and dividers to make 4 outputs. This driver splits >> them into multiple clocks to

Re: [PATCH v3 2/5] firmware: annotate thou shalt not request fw on init or probe

2016-09-02 Thread Luis R. Rodriguez
On Wed, Aug 24, 2016 at 10:17:38AM +0200, Gabriel Paubert wrote: > On Tue, Aug 23, 2016 at 05:45:04PM -0700, mcg...@kernel.org wrote: > > [snip] > > --- > > Documentation/firmware_class/README| 20 > > drivers/base/Kconfig | 2 +- > >

Re: [PATCH v3 2/5] firmware: annotate thou shalt not request fw on init or probe

2016-09-02 Thread Luis R. Rodriguez
On Wed, Aug 24, 2016 at 10:17:38AM +0200, Gabriel Paubert wrote: > On Tue, Aug 23, 2016 at 05:45:04PM -0700, mcg...@kernel.org wrote: > > [snip] > > --- > > Documentation/firmware_class/README| 20 > > drivers/base/Kconfig | 2 +- > >

[tip:irq/core] genirq/generic_chip: Verify irqs_per_chip <= 32

2016-09-02 Thread tip-bot for Sebastian Frias
Commit-ID: f88eecfe2f22b2790e7527c0aaec14ea175919de Gitweb: http://git.kernel.org/tip/f88eecfe2f22b2790e7527c0aaec14ea175919de Author: Sebastian Frias AuthorDate: Tue, 16 Aug 2016 16:05:08 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep

[tip:irq/core] genirq/generic_chip: Verify irqs_per_chip <= 32

2016-09-02 Thread tip-bot for Sebastian Frias
Commit-ID: f88eecfe2f22b2790e7527c0aaec14ea175919de Gitweb: http://git.kernel.org/tip/f88eecfe2f22b2790e7527c0aaec14ea175919de Author: Sebastian Frias AuthorDate: Tue, 16 Aug 2016 16:05:08 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:20:59 +0200 genirq/generic_chip:

Re: [PATCH v3 1/2] wcn36xx: Transition driver to SMD client

2016-09-02 Thread Kalle Valo
Bjorn Andersson writes: > On Fri 02 Sep 09:24 PDT 2016, Kalle Valo wrote: > >> Bjorn Andersson writes: >> >> > --- a/drivers/net/wireless/ath/wcn36xx/Kconfig >> > +++ b/drivers/net/wireless/ath/wcn36xx/Kconfig >> > @@ -1,6 +1,6 @@ >> >

Re: [PATCH v3 1/2] wcn36xx: Transition driver to SMD client

2016-09-02 Thread Kalle Valo
Bjorn Andersson writes: > On Fri 02 Sep 09:24 PDT 2016, Kalle Valo wrote: > >> Bjorn Andersson writes: >> >> > --- a/drivers/net/wireless/ath/wcn36xx/Kconfig >> > +++ b/drivers/net/wireless/ath/wcn36xx/Kconfig >> > @@ -1,6 +1,6 @@ >> > config WCN36XX >> >tristate "Qualcomm Atheros

[tip:smp/hotplug] net/virtio-net: Convert to hotplug state machine

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: 8017c279196ab29174bafc104ac4ebbd42c7ca7f Gitweb: http://git.kernel.org/tip/8017c279196ab29174bafc104ac4ebbd42c7ca7f Author: Sebastian Andrzej Siewior AuthorDate: Fri, 12 Aug 2016 19:49:43 +0200 Committer: Thomas Gleixner CommitDate:

[tip:smp/hotplug] net/virtio-net: Convert to hotplug state machine

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: 8017c279196ab29174bafc104ac4ebbd42c7ca7f Gitweb: http://git.kernel.org/tip/8017c279196ab29174bafc104ac4ebbd42c7ca7f Author: Sebastian Andrzej Siewior AuthorDate: Fri, 12 Aug 2016 19:49:43 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:05:06 +0200

[tip:smp/hotplug] bus/arm-cci: Use cpu-hp's multi instance support instead custom list

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: b230f0db913136f465a951806f2978b179df95d5 Gitweb: http://git.kernel.org/tip/b230f0db913136f465a951806f2978b179df95d5 Author: Sebastian Andrzej Siewior AuthorDate: Fri, 12 Aug 2016 19:49:41 +0200 Committer: Thomas Gleixner CommitDate:

[tip:smp/hotplug] bus/arm-cci: Use cpu-hp's multi instance support instead custom list

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: b230f0db913136f465a951806f2978b179df95d5 Gitweb: http://git.kernel.org/tip/b230f0db913136f465a951806f2978b179df95d5 Author: Sebastian Andrzej Siewior AuthorDate: Fri, 12 Aug 2016 19:49:41 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:05:06 +0200

[tip:smp/hotplug] bus/arm-ccn: Use cpu-hp's multi instance support instead custom list

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: 8df038725ad5351a9730759e0a24a5c5d96be661 Gitweb: http://git.kernel.org/tip/8df038725ad5351a9730759e0a24a5c5d96be661 Author: Sebastian Andrzej Siewior AuthorDate: Fri, 12 Aug 2016 19:49:42 +0200 Committer: Thomas Gleixner CommitDate:

[tip:smp/hotplug] bus/arm-ccn: Use cpu-hp's multi instance support instead custom list

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: 8df038725ad5351a9730759e0a24a5c5d96be661 Gitweb: http://git.kernel.org/tip/8df038725ad5351a9730759e0a24a5c5d96be661 Author: Sebastian Andrzej Siewior AuthorDate: Fri, 12 Aug 2016 19:49:42 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:05:06 +0200

[tip:smp/hotplug] cpu/hotplug: Add multi instance support

2016-09-02 Thread tip-bot for Thomas Gleixner
Commit-ID: cf392d10b69e6e6c57ceea48b347a2ab1a4b75b2 Gitweb: http://git.kernel.org/tip/cf392d10b69e6e6c57ceea48b347a2ab1a4b75b2 Author: Thomas Gleixner AuthorDate: Fri, 12 Aug 2016 19:49:39 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep

[tip:smp/hotplug] cpu/hotplug: Add multi instance support

2016-09-02 Thread tip-bot for Thomas Gleixner
Commit-ID: cf392d10b69e6e6c57ceea48b347a2ab1a4b75b2 Gitweb: http://git.kernel.org/tip/cf392d10b69e6e6c57ceea48b347a2ab1a4b75b2 Author: Thomas Gleixner AuthorDate: Fri, 12 Aug 2016 19:49:39 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:05:05 +0200 cpu/hotplug: Add

[tip:irq/core 8/8] drivers/irqchip/irq-sirfsoc.c:46:12: error: passing argument 4 of '__irq_alloc_domain_generic_chips' from incompatible pointer type

2016-09-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core head: 895d3b95ed05f72a94f69ab52cb313915a6b889f commit: 895d3b95ed05f72a94f69ab52cb313915a6b889f [8/8] genirq/generic_chip: Verify irqs_per_chip <= 32 config: arm-multi_v7_defconfig (attached as .config) compiler:

[tip:irq/core 8/8] drivers/irqchip/irq-sirfsoc.c:46:12: error: passing argument 4 of '__irq_alloc_domain_generic_chips' from incompatible pointer type

2016-09-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core head: 895d3b95ed05f72a94f69ab52cb313915a6b889f commit: 895d3b95ed05f72a94f69ab52cb313915a6b889f [8/8] genirq/generic_chip: Verify irqs_per_chip <= 32 config: arm-multi_v7_defconfig (attached as .config) compiler:

[tip:smp/hotplug] arm/perf: Use multi instance instead of custom list

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: 6e103c0cfeb9ab8d40822a015da9769595096411 Gitweb: http://git.kernel.org/tip/6e103c0cfeb9ab8d40822a015da9769595096411 Author: Sebastian Andrzej Siewior AuthorDate: Wed, 17 Aug 2016 19:14:20 +0200 Committer: Thomas Gleixner CommitDate:

[tip:smp/hotplug] arm/perf: Use multi instance instead of custom list

2016-09-02 Thread tip-bot for Sebastian Andrzej Siewior
Commit-ID: 6e103c0cfeb9ab8d40822a015da9769595096411 Gitweb: http://git.kernel.org/tip/6e103c0cfeb9ab8d40822a015da9769595096411 Author: Sebastian Andrzej Siewior AuthorDate: Wed, 17 Aug 2016 19:14:20 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:05:06 +0200 arm/perf:

[tip:smp/hotplug] cpu/hotplug: Rework callback invocation logic

2016-09-02 Thread tip-bot for Thomas Gleixner
Commit-ID: a724632ca0c84b494875e9367e07e29472c139ba Gitweb: http://git.kernel.org/tip/a724632ca0c84b494875e9367e07e29472c139ba Author: Thomas Gleixner AuthorDate: Fri, 12 Aug 2016 19:49:38 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep

[tip:smp/hotplug] cpu/hotplug: Rework callback invocation logic

2016-09-02 Thread tip-bot for Thomas Gleixner
Commit-ID: a724632ca0c84b494875e9367e07e29472c139ba Gitweb: http://git.kernel.org/tip/a724632ca0c84b494875e9367e07e29472c139ba Author: Thomas Gleixner AuthorDate: Fri, 12 Aug 2016 19:49:38 +0200 Committer: Thomas Gleixner CommitDate: Fri, 2 Sep 2016 20:05:05 +0200 cpu/hotplug: Rework

Re: [RFC PATCH v2 07/20] x86: Provide general kernel support for memory encryption

2016-09-02 Thread Borislav Petkov
On Mon, Aug 22, 2016 at 05:36:46PM -0500, Tom Lendacky wrote: > Adding general kernel support for memory encryption includes: > - Modify and create some page table macros to include the Secure Memory > Encryption (SME) memory encryption mask > - Update kernel boot support to call an SME routine

Re: [RFC PATCH v2 07/20] x86: Provide general kernel support for memory encryption

2016-09-02 Thread Borislav Petkov
On Mon, Aug 22, 2016 at 05:36:46PM -0500, Tom Lendacky wrote: > Adding general kernel support for memory encryption includes: > - Modify and create some page table macros to include the Secure Memory > Encryption (SME) memory encryption mask > - Update kernel boot support to call an SME routine

RE: [PATCH RFC 1/4] lib/radix: add universal radix_tree_fill_range

2016-09-02 Thread Matthew Wilcox
I have a rewrite of the iterators; would you like to take a look? http://git.infradead.org/users/willy/linux-dax.git/shortlog/refs/heads/idr-2016-09-02 There's five distinct sets of changes in that tree: 1. Test suite enhancements (first 8 patches) 2. Split/Join (patches 9-11) 3. Misc cleanups

RE: [PATCH RFC 1/4] lib/radix: add universal radix_tree_fill_range

2016-09-02 Thread Matthew Wilcox
I have a rewrite of the iterators; would you like to take a look? http://git.infradead.org/users/willy/linux-dax.git/shortlog/refs/heads/idr-2016-09-02 There's five distinct sets of changes in that tree: 1. Test suite enhancements (first 8 patches) 2. Split/Join (patches 9-11) 3. Misc cleanups

Re: [PATCH v3 5/5] firmware: fix fw cache to avoid usermode helper on suspend

2016-09-02 Thread Luis R. Rodriguez
On Wed, Aug 31, 2016 at 09:03:53AM +0200, Daniel Wagner wrote: > Hi Luis, > > On 08/24/2016 02:45 AM, mcg...@kernel.org wrote: > >diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c > >index 22d1760a4278..dca4f9cbf4db 100644 > >--- a/drivers/base/firmware_class.c > >+++

Re: [PATCH v3 5/5] firmware: fix fw cache to avoid usermode helper on suspend

2016-09-02 Thread Luis R. Rodriguez
On Wed, Aug 31, 2016 at 09:03:53AM +0200, Daniel Wagner wrote: > Hi Luis, > > On 08/24/2016 02:45 AM, mcg...@kernel.org wrote: > >diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c > >index 22d1760a4278..dca4f9cbf4db 100644 > >--- a/drivers/base/firmware_class.c > >+++

Re: [PATCH 3/3] nvme: Enable autonomous power state transitions

2016-09-02 Thread J Freyensee
> > > > > > > > > > > > > > + /* > > > > +  * By default, allow up to 25ms of APST-induced > > > > latency.  This will > > > > +  * have no effect on non-APST supporting controllers > > > > (i.e. > > > > any > > > > +  * controller with APSTA == 0). > > > > +  */ > > > > +  

Re: [PATCH 3/3] nvme: Enable autonomous power state transitions

2016-09-02 Thread J Freyensee
> > > > > > > > > > > > > > + /* > > > > +  * By default, allow up to 25ms of APST-induced > > > > latency.  This will > > > > +  * have no effect on non-APST supporting controllers > > > > (i.e. > > > > any > > > > +  * controller with APSTA == 0). > > > > +  */ > > > > +  

Memory barrier needed with wake_up_process()?

2016-09-02 Thread Alan Stern
Paul, Peter, and Ingo: This must have come up before, but I don't know what was decided. Isn't it often true that a memory barrier is needed before a call to wake_up_process()? A typical scenario might look like this: CPU 0 - for (;;) {

Memory barrier needed with wake_up_process()?

2016-09-02 Thread Alan Stern
Paul, Peter, and Ingo: This must have come up before, but I don't know what was decided. Isn't it often true that a memory barrier is needed before a call to wake_up_process()? A typical scenario might look like this: CPU 0 - for (;;) {

Re: [PATCH 1/2] wd719x: Remove last declaration using DEFINE_PCI_DEVICE_TABLE

2016-09-02 Thread Martin K. Petersen
> "Andrew" == Andrew Morton writes: Andrew> That creates an ordering dependency between the scsi tree and Andrew> -mm's "treewide: remove references to the now unnecessary Andrew> DEFINE_PCI_DEVICE_TABLE". Andrew> So an ack would be preferred, please. OK. I was

Re: [PATCH 1/2] wd719x: Remove last declaration using DEFINE_PCI_DEVICE_TABLE

2016-09-02 Thread Martin K. Petersen
> "Andrew" == Andrew Morton writes: Andrew> That creates an ordering dependency between the scsi tree and Andrew> -mm's "treewide: remove references to the now unnecessary Andrew> DEFINE_PCI_DEVICE_TABLE". Andrew> So an ack would be preferred, please. OK. I was assuming that a subsequent

Re: A potential race in drivers/iio/adc/vf610_adc.ko

2016-09-02 Thread Alison Schofield
On Fri, Sep 02, 2016 at 11:05:09AM +0300, Pavel Andrianov wrote: > > Hi! > > There is a potential race in drivers/iio/adc/vf610_adc.ko. > Handlers vf610_set_conversion_mode and vf610_write_raw are called via > device_attibute interface, but they are related to different attributes, so > may be

RE: [PATCH] x86/AMD: Fix Socket ID for LLC topology for AMD Fam17h systems

2016-09-02 Thread Ghannam, Yazen
> I asked the obvious question whether Fam17h will only ever have two > sockets, > because your patch limits it hard to that. > I don't know if Fam17h will only ever have two sockets, so I'll send a V2 removing the hard limit. Thanks, Yazen

Re: A potential race in drivers/iio/adc/vf610_adc.ko

2016-09-02 Thread Alison Schofield
On Fri, Sep 02, 2016 at 11:05:09AM +0300, Pavel Andrianov wrote: > > Hi! > > There is a potential race in drivers/iio/adc/vf610_adc.ko. > Handlers vf610_set_conversion_mode and vf610_write_raw are called via > device_attibute interface, but they are related to different attributes, so > may be

RE: [PATCH] x86/AMD: Fix Socket ID for LLC topology for AMD Fam17h systems

2016-09-02 Thread Ghannam, Yazen
> I asked the obvious question whether Fam17h will only ever have two > sockets, > because your patch limits it hard to that. > I don't know if Fam17h will only ever have two sockets, so I'll send a V2 removing the hard limit. Thanks, Yazen

Re: [writeback] 8bc4ad9498: INFO: suspicious RCU usage. ]

2016-09-02 Thread Tejun Heo
Hello, Paul. On Fri, Sep 02, 2016 at 10:00:56AM -0700, Paul E. McKenney wrote: > Note that lockdep believes that the rcu_read_lock() has also been > dropped, otherwise it would have listed it. There isn't any actual code between rcu_read_lock() and the following rcu deref. That's the first

Re: [writeback] 8bc4ad9498: INFO: suspicious RCU usage. ]

2016-09-02 Thread Tejun Heo
Hello, Paul. On Fri, Sep 02, 2016 at 10:00:56AM -0700, Paul E. McKenney wrote: > Note that lockdep believes that the rcu_read_lock() has also been > dropped, otherwise it would have listed it. There isn't any actual code between rcu_read_lock() and the following rcu deref. That's the first

[tip:irq/core 8/8] drivers/gpio/gpio-dwapb.c:308:15: sparse: not enough arguments for function __irq_alloc_domain_generic_chips

2016-09-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core head: 895d3b95ed05f72a94f69ab52cb313915a6b889f commit: 895d3b95ed05f72a94f69ab52cb313915a6b889f [8/8] genirq/generic_chip: Verify irqs_per_chip <= 32 reproduce: # apt-get install sparse git checkout

[tip:irq/core 8/8] drivers/gpio/gpio-dwapb.c:308:15: sparse: not enough arguments for function __irq_alloc_domain_generic_chips

2016-09-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core head: 895d3b95ed05f72a94f69ab52cb313915a6b889f commit: 895d3b95ed05f72a94f69ab52cb313915a6b889f [8/8] genirq/generic_chip: Verify irqs_per_chip <= 32 reproduce: # apt-get install sparse git checkout

Re: [PATCH v3 net-next 0/6] perf, bpf: add support for bpf in sw/hw perf_events

2016-09-02 Thread David Miller
From: Alexei Starovoitov Date: Thu, 1 Sep 2016 18:37:20 -0700 > Hi Peter, Dave, > > this patch set is a follow up to the discussion: > https://lkml.kernel.org/r/20160804142853.GO6862%20()%20twins%20!%20programming%20!%20kicks-ass%20!%20net > It turned out to be simpler than what we

Re: [PATCH v3 net-next 0/6] perf, bpf: add support for bpf in sw/hw perf_events

2016-09-02 Thread David Miller
From: Alexei Starovoitov Date: Thu, 1 Sep 2016 18:37:20 -0700 > Hi Peter, Dave, > > this patch set is a follow up to the discussion: > https://lkml.kernel.org/r/20160804142853.GO6862%20()%20twins%20!%20programming%20!%20kicks-ass%20!%20net > It turned out to be simpler than what we discussed. >

Re: [PATCH v2] mfd: arizona: Add DT options for max_channels_clocked and PDM speaker config

2016-09-02 Thread Rob Herring
On Fri, Sep 2, 2016 at 10:56 AM, Richard Fitzgerald wrote: > On Fri, 2016-09-02 at 10:34 -0500, Rob Herring wrote: >> On Wed, Aug 31, 2016 at 10:25:43AM +0100, Richard Fitzgerald wrote: >> > This patch adds DT settings for the max_channels_clocked, spk_fmt and >>

Re: [PATCH v2] mfd: arizona: Add DT options for max_channels_clocked and PDM speaker config

2016-09-02 Thread Rob Herring
On Fri, Sep 2, 2016 at 10:56 AM, Richard Fitzgerald wrote: > On Fri, 2016-09-02 at 10:34 -0500, Rob Herring wrote: >> On Wed, Aug 31, 2016 at 10:25:43AM +0100, Richard Fitzgerald wrote: >> > This patch adds DT settings for the max_channels_clocked, spk_fmt and >> > spk_mute pdata. >> > >> >

[PATCH] qed: Remove OOM messages

2016-09-02 Thread Joe Perches
These messages are unnecessary as OOM allocation failures already do a dump_stack() giving more or less the same information. $ size drivers/net/ethernet/qlogic/qed/built-in.o* (defconfig x86-64) textdata bss dec hex filename 126849 27968 32800 187617 2dce1

[PATCH] qed: Remove OOM messages

2016-09-02 Thread Joe Perches
These messages are unnecessary as OOM allocation failures already do a dump_stack() giving more or less the same information. $ size drivers/net/ethernet/qlogic/qed/built-in.o* (defconfig x86-64) textdata bss dec hex filename 126849 27968 32800 187617 2dce1

Re: [writeback] 8bc4ad9498: INFO: suspicious RCU usage. ]

2016-09-02 Thread Paul E. McKenney
On Fri, Sep 02, 2016 at 09:49:25AM -0600, Jens Axboe wrote: > On 09/02/2016 09:36 AM, Paul E. McKenney wrote: > >On Fri, Sep 02, 2016 at 10:56:22AM -0400, Tejun Heo wrote: > >>(cc'ing Paul, hi!) > >> > >>Hello, > >> > >>On Thu, Sep 01, 2016 at 02:13:34PM -0600, Jens Axboe wrote: > >>>On 09/01/2016

Re: [writeback] 8bc4ad9498: INFO: suspicious RCU usage. ]

2016-09-02 Thread Paul E. McKenney
On Fri, Sep 02, 2016 at 09:49:25AM -0600, Jens Axboe wrote: > On 09/02/2016 09:36 AM, Paul E. McKenney wrote: > >On Fri, Sep 02, 2016 at 10:56:22AM -0400, Tejun Heo wrote: > >>(cc'ing Paul, hi!) > >> > >>Hello, > >> > >>On Thu, Sep 01, 2016 at 02:13:34PM -0600, Jens Axboe wrote: > >>>On 09/01/2016

Re: [PATCH] qed: fix kzalloc-simple.cocci warnings

2016-09-02 Thread Joe Perches
On Thu, 2016-09-01 at 07:37 +, Yuval Mintz wrote: > > drivers/net/ethernet/qlogic/qed/qed_dcbx.c:1230:13-20: WARNING: kzalloc > > should be used for dcbx_info, instead of kmalloc/memset > > drivers/net/ethernet/qlogic/qed/qed_dcbx.c:1192:13-20: WARNING: kzalloc > > should be used for

Re: [PATCH] qed: fix kzalloc-simple.cocci warnings

2016-09-02 Thread Joe Perches
On Thu, 2016-09-01 at 07:37 +, Yuval Mintz wrote: > > drivers/net/ethernet/qlogic/qed/qed_dcbx.c:1230:13-20: WARNING: kzalloc > > should be used for dcbx_info, instead of kmalloc/memset > > drivers/net/ethernet/qlogic/qed/qed_dcbx.c:1192:13-20: WARNING: kzalloc > > should be used for

Re: [PATCH net-next 3/3] net: dsa: mv88e6xxx: make global2 code optional

2016-09-02 Thread Vivien Didelot
Hi Andrew, Andrew Lunn writes: >> I agree. Does the following snippet looks OK? >> >> >> #ifndef CONFIG_NET_DSA_MV88E6XXX_GLOBAL2 >> if (mv88e6xxx_has(chip, MV88E6XXX_FLAG_GLOBAL2)) { >> dev_err(chip->dev, "Missing support for Global 2 >>

Re: [PATCH net-next 3/3] net: dsa: mv88e6xxx: make global2 code optional

2016-09-02 Thread Vivien Didelot
Hi Andrew, Andrew Lunn writes: >> I agree. Does the following snippet looks OK? >> >> >> #ifndef CONFIG_NET_DSA_MV88E6XXX_GLOBAL2 >> if (mv88e6xxx_has(chip, MV88E6XXX_FLAG_GLOBAL2)) { >> dev_err(chip->dev, "Missing support for Global 2 >> registers\n"); >

Can mei_timer() turn itself off when not needed?

2016-09-02 Thread Andy Lutomirski
On my laptop, the kernel has become good enough at idling that mei_timer() accounts for a decent fraction of wakeups. It looks like, unless something interesting is going on, it mostly just iterates over some empty lists and re-queues itself. Could it learn to stop re-queueing itself if there

Can mei_timer() turn itself off when not needed?

2016-09-02 Thread Andy Lutomirski
On my laptop, the kernel has become good enough at idling that mei_timer() accounts for a decent fraction of wakeups. It looks like, unless something interesting is going on, it mostly just iterates over some empty lists and re-queues itself. Could it learn to stop re-queueing itself if there

[tip:irq/core 8/8] drivers/gpio/gpio-dwapb.c:309:25: error: passing argument 4 of '__irq_alloc_domain_generic_chips' from incompatible pointer type

2016-09-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core head: 895d3b95ed05f72a94f69ab52cb313915a6b889f commit: 895d3b95ed05f72a94f69ab52cb313915a6b889f [8/8] genirq/generic_chip: Verify irqs_per_chip <= 32 config: i386-randconfig-s0-201635 (attached as .config) compiler:

[tip:irq/core 8/8] drivers/gpio/gpio-dwapb.c:309:25: error: passing argument 4 of '__irq_alloc_domain_generic_chips' from incompatible pointer type

2016-09-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core head: 895d3b95ed05f72a94f69ab52cb313915a6b889f commit: 895d3b95ed05f72a94f69ab52cb313915a6b889f [8/8] genirq/generic_chip: Verify irqs_per_chip <= 32 config: i386-randconfig-s0-201635 (attached as .config) compiler:

Re: [PATCH 4/4] kvm: x86: export TSC offset to user-space

2016-09-02 Thread Luiz Capitulino
On Fri, 2 Sep 2016 19:00:41 +0200 Paolo Bonzini wrote: > On 31/08/2016 19:05, Luiz Capitulino wrote: > > vcpu0: 18446742405270834952 > > vcpu1: 18446742405270834952 > > vcpu2: 18446742405270834952 > > vcpu3: 18446742405270834952 > > > > - We'll probably need to

Re: [PATCH 4/4] kvm: x86: export TSC offset to user-space

2016-09-02 Thread Luiz Capitulino
On Fri, 2 Sep 2016 19:00:41 +0200 Paolo Bonzini wrote: > On 31/08/2016 19:05, Luiz Capitulino wrote: > > vcpu0: 18446742405270834952 > > vcpu1: 18446742405270834952 > > vcpu2: 18446742405270834952 > > vcpu3: 18446742405270834952 > > > > - We'll probably need to export the TSC

Re: [PATCH v15 04/13] task_isolation: add initial support

2016-09-02 Thread Andy Lutomirski
On Sep 2, 2016 7:04 AM, "Chris Metcalf" wrote: > > On 8/30/2016 3:50 PM, Andy Lutomirski wrote: >> >> On Tue, Aug 30, 2016 at 12:37 PM, Chris Metcalf >> wrote: >>> >>> On 8/30/2016 2:43 PM, Andy Lutomirski wrote: What if we did it the

Re: [PATCH v15 04/13] task_isolation: add initial support

2016-09-02 Thread Andy Lutomirski
On Sep 2, 2016 7:04 AM, "Chris Metcalf" wrote: > > On 8/30/2016 3:50 PM, Andy Lutomirski wrote: >> >> On Tue, Aug 30, 2016 at 12:37 PM, Chris Metcalf >> wrote: >>> >>> On 8/30/2016 2:43 PM, Andy Lutomirski wrote: What if we did it the other way around: set a percpu flag saying

Re: [v10,2/2] PCI: Rockchip: Add Rockchip PCIe controller support

2016-09-02 Thread Guenter Roeck
On Thu, Sep 01, 2016 at 09:57:58AM -0700, Brian Norris wrote: > On Thu, Sep 01, 2016 at 11:34:55AM -0500, Bjorn Helgaas wrote: > > I can't conveniently build it, so I'm sure I've broken things. I > > Indeed, you have :) > > > pushed the current work-in-progress branch to pci/host-rockchip-wip.

Re: [v10,2/2] PCI: Rockchip: Add Rockchip PCIe controller support

2016-09-02 Thread Guenter Roeck
On Thu, Sep 01, 2016 at 09:57:58AM -0700, Brian Norris wrote: > On Thu, Sep 01, 2016 at 11:34:55AM -0500, Bjorn Helgaas wrote: > > I can't conveniently build it, so I'm sure I've broken things. I > > Indeed, you have :) > > > pushed the current work-in-progress branch to pci/host-rockchip-wip.

Re: [PATCH v3] IB/cxgb4: Mark symbols static for _free_qp

2016-09-02 Thread Doug Ledford
On 8/29/2016 10:26 AM, Steve Wise wrote: >> Subject: [PATCH v3] IB/cxgb4: Mark symbols static for _free_qp >> >> We get 1 warning when build kernel with W=1: >> drivers/infiniband/hw/cxgb4/qp.c:686:6: warning: no previous prototype for >> '_free_qp' [-Wmissing-prototypes] >> >> In fact, this

Re: [PATCH v3] IB/cxgb4: Mark symbols static for _free_qp

2016-09-02 Thread Doug Ledford
On 8/29/2016 10:26 AM, Steve Wise wrote: >> Subject: [PATCH v3] IB/cxgb4: Mark symbols static for _free_qp >> >> We get 1 warning when build kernel with W=1: >> drivers/infiniband/hw/cxgb4/qp.c:686:6: warning: no previous prototype for >> '_free_qp' [-Wmissing-prototypes] >> >> In fact, this

[PATCH v4] iio: humidity: hdc100x: add triggered buffer support for HDC100X

2016-09-02 Thread Alison Schofield
Triggered buffer support uses the HDC100X's dual acquisition mode to read both humidity and temperature in one shot. This patch depends on 447136effbf4 ("iio: humidity: hdc100x: fix sensor data reads of temp and humidity") Signed-off-by: Alison Schofield

[PATCH v4] iio: humidity: hdc100x: add triggered buffer support for HDC100X

2016-09-02 Thread Alison Schofield
Triggered buffer support uses the HDC100X's dual acquisition mode to read both humidity and temperature in one shot. This patch depends on 447136effbf4 ("iio: humidity: hdc100x: fix sensor data reads of temp and humidity") Signed-off-by: Alison Schofield Cc:

Re: [PATCH] ASoC: fsl_esai: fix spelling mistake "Transmition" -> "Transmission"

2016-09-02 Thread Nicolin Chen
On Fri, Sep 02, 2016 at 03:07:23PM +0100, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistakes in dev_dbg messages > > Signed-off-by: Colin Ian King Acked-by: Nicolin Chen Thanks > --- >

Re: [PATCH] ASoC: fsl_esai: fix spelling mistake "Transmition" -> "Transmission"

2016-09-02 Thread Nicolin Chen
On Fri, Sep 02, 2016 at 03:07:23PM +0100, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistakes in dev_dbg messages > > Signed-off-by: Colin Ian King Acked-by: Nicolin Chen Thanks > --- > sound/soc/fsl/fsl_esai.c | 6 +++--- > 1 file changed, 3 insertions(+), 3

Re: [PATCH net-next 3/3] net: dsa: mv88e6xxx: make global2 code optional

2016-09-02 Thread Andrew Lunn
> I agree. Does the following snippet looks OK? > > > #ifndef CONFIG_NET_DSA_MV88E6XXX_GLOBAL2 > if (mv88e6xxx_has(chip, MV88E6XXX_FLAG_GLOBAL2)) { > dev_err(chip->dev, "Missing support for Global 2 > registers\n"); I would include the name of the option

Re: [PATCH net-next 3/3] net: dsa: mv88e6xxx: make global2 code optional

2016-09-02 Thread Andrew Lunn
> I agree. Does the following snippet looks OK? > > > #ifndef CONFIG_NET_DSA_MV88E6XXX_GLOBAL2 > if (mv88e6xxx_has(chip, MV88E6XXX_FLAG_GLOBAL2)) { > dev_err(chip->dev, "Missing support for Global 2 > registers\n"); I would include the name of the option

Re: [PATCH 1/2] wd719x: Remove last declaration using DEFINE_PCI_DEVICE_TABLE

2016-09-02 Thread Andrew Morton
On Fri, 02 Sep 2016 06:36:05 -0400 "Martin K. Petersen" wrote: > > "Joe" == Joe Perches writes: > > Joe> Convert it to the preferred const struct pci_device_id instead. > > Applied to 4.9/scsi-queue. That creates an ordering dependency

Re: [PATCH 1/2] wd719x: Remove last declaration using DEFINE_PCI_DEVICE_TABLE

2016-09-02 Thread Andrew Morton
On Fri, 02 Sep 2016 06:36:05 -0400 "Martin K. Petersen" wrote: > > "Joe" == Joe Perches writes: > > Joe> Convert it to the preferred const struct pci_device_id instead. > > Applied to 4.9/scsi-queue. That creates an ordering dependency between the scsi tree and -mm's "treewide: remove

[PATCH V2 0/2] ARM: dts: Introduce support for rev BeagleBoard-X15 B1

2016-09-02 Thread Nishanth Menon
Hi, Please find the series to cleanup and support Production version of Beagleboard-X15 rev B1 support. More details can be found at https://github.com/beagleboard/beagleboard-x15 Bootlog on B1: http://pastebin.ubuntu.com/23124704/ (sd and hdmi hpd tested) Based on:

[PATCH V2 0/2] ARM: dts: Introduce support for rev BeagleBoard-X15 B1

2016-09-02 Thread Nishanth Menon
Hi, Please find the series to cleanup and support Production version of Beagleboard-X15 rev B1 support. More details can be found at https://github.com/beagleboard/beagleboard-x15 Bootlog on B1: http://pastebin.ubuntu.com/23124704/ (sd and hdmi hpd tested) Based on:

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