[RFC PATCH 0/6] Reduce cache miss for snmp_fold_field

2016-08-29 Thread Jia He
In a PowerPc server with large cpu number(160), besides commit a3a773726c9f ("net: Optimize snmp stat aggregation by walking all the percpu data at once"), I watched several other snmp_fold_field callsites which will cause high cache miss rate. #My simple test case, which read from the procfs

[RFC PATCH 1/6] proc: Reduce cache miss in {snmp,netstat}_seq_show

2016-08-29 Thread Jia He
This patch exchanges the two loop for collecting the percpu statistics data. This can aggregate the data by going through all the items of each cpu sequentially. In snmp_seq_show, just use one buff copy to dislay the Udp and UdpLite data because they are the same. Signed-off-by: Jia He ---

[RFC PATCH 3/6] proc: Reduce cache miss in sctp_snmp_seq_show

2016-08-29 Thread Jia He
This patch exchanges the two loop for collecting the percpu statistics data. This can reduce cache misses by going through all the items of each cpu sequentially. Signed-off-by: Jia He --- net/sctp/proc.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git

[RFC PATCH 2/6] proc: Reduce cache miss in snmp6_seq_show

2016-08-29 Thread Jia He
This patch exchanges the two loop for collecting the percpu statistics data. This can reduce cache misses by going through all the items of each cpu sequentially. Signed-off-by: Jia He --- net/ipv6/proc.c | 47 --- 1 file changed, 36 insertions(+), 11

[RFC PATCH 4/6] proc: Reduce cache miss in xfrm_statistics_seq_show

2016-08-29 Thread Jia He
This patch exchanges the two loop for collecting the percpu statistics data. This can reduce cache misses by going through all the items of each cpu sequentially. Signed-off-by: Jia He --- net/xfrm/xfrm_proc.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git

[RFC PATCH 0/6] Reduce cache miss for snmp_fold_field

2016-08-29 Thread Jia He
In a PowerPc server with large cpu number(160), besides commit a3a773726c9f ("net: Optimize snmp stat aggregation by walking all the percpu data at once"), I watched several other snmp_fold_field callsites which will cause high cache miss rate. #My simple test case, which read from the procfs

Re: [PATCH 1/2] f2fs: set encryption name flag in add inline entry path

2016-08-29 Thread Chao Yu
On 2016/8/29 11:27, Shuoran Liu wrote: > This patch sets encryption name flag in the add inline entry path > if filename is encrypted. > > Signed-off-by: Shuoran Liu Acked-by: Chao Yu

Re: [PATCH 1/2] f2fs: set encryption name flag in add inline entry path

2016-08-29 Thread Chao Yu
On 2016/8/29 11:27, Shuoran Liu wrote: > This patch sets encryption name flag in the add inline entry path > if filename is encrypted. > > Signed-off-by: Shuoran Liu Acked-by: Chao Yu

Re: [PATCH v2] drm/nouveau/gr: mark gm107_grctx_generate_tpcid() static

2016-08-29 Thread Sean Paul
On Mon, Aug 29, 2016 at 9:02 AM, Baoyou Xie wrote: > We get 1 warning when build kernel with W=1: > drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c:937:1: warning: no previous > prototype for 'gm107_grctx_generate_tpcid' [-Wmissing-prototypes] > > In fact, this function

Re: [PATCH v2] drm/nouveau/gr: mark gm107_grctx_generate_tpcid() static

2016-08-29 Thread Sean Paul
On Mon, Aug 29, 2016 at 9:02 AM, Baoyou Xie wrote: > We get 1 warning when build kernel with W=1: > drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c:937:1: warning: no previous > prototype for 'gm107_grctx_generate_tpcid' [-Wmissing-prototypes] > > In fact, this function is only used in the

Re: OOM detection regressions since 4.7

2016-08-29 Thread Olaf Hering
On Mon, Aug 29, Michal Hocko wrote: > On Mon 29-08-16 16:52:03, Olaf Hering wrote: > > I ran rc3 for a few hours on Friday amd FireFox was not killed. > > Now rc3 is running for a day with the usual workload and FireFox is > > still running. > Is the patch >

Re: Kernel Freeze with American Megatrends BIOS

2016-08-29 Thread Bjorn Helgaas
[+cc linux-acpi, linux-kernel, dri-devel] Hi Roland, I have no idea how to debug this problem. Are you seeing something that suggests it may be a PCI problem? On Tue, Aug 23, 2016 at 11:23:45AM +0200, Roland Singer wrote: > Hi, > > hope somebody can help me fix this kernel problem which

Re: OOM detection regressions since 4.7

2016-08-29 Thread Olaf Hering
On Mon, Aug 29, Michal Hocko wrote: > On Mon 29-08-16 16:52:03, Olaf Hering wrote: > > I ran rc3 for a few hours on Friday amd FireFox was not killed. > > Now rc3 is running for a day with the usual workload and FireFox is > > still running. > Is the patch >

Re: Kernel Freeze with American Megatrends BIOS

2016-08-29 Thread Bjorn Helgaas
[+cc linux-acpi, linux-kernel, dri-devel] Hi Roland, I have no idea how to debug this problem. Are you seeing something that suggests it may be a PCI problem? On Tue, Aug 23, 2016 at 11:23:45AM +0200, Roland Singer wrote: > Hi, > > hope somebody can help me fix this kernel problem which

Re: [PATCH v2] iio: adc: ltc2485: add support for Linear Technology LTC2485 ADC

2016-08-29 Thread Jonathan Cameron
On 25/08/16 06:48, Alison Schofield wrote: > Adds basic support for the LTC2485 ADC - a delta-sigma analog-to-digital > converter with an I2C interface that operates in single shot conversion > mode. > > The driver supports an on board 5V reference and the power-on default > configuration which

[PATCH] f2fs: support async discard

2016-08-29 Thread Chao Yu
From: Chao Yu Like most filesystems, f2fs will issue discard command synchronously, so when user trigger fstrim through ioctl, multiple discard commands will be issued serially with sync mode, which makes poor performance. In this patch we try to support async discard, so

Re: [PATCH v2] iio: adc: ltc2485: add support for Linear Technology LTC2485 ADC

2016-08-29 Thread Jonathan Cameron
On 25/08/16 06:48, Alison Schofield wrote: > Adds basic support for the LTC2485 ADC - a delta-sigma analog-to-digital > converter with an I2C interface that operates in single shot conversion > mode. > > The driver supports an on board 5V reference and the power-on default > configuration which

[PATCH] f2fs: support async discard

2016-08-29 Thread Chao Yu
From: Chao Yu Like most filesystems, f2fs will issue discard command synchronously, so when user trigger fstrim through ioctl, multiple discard commands will be issued serially with sync mode, which makes poor performance. In this patch we try to support async discard, so that all discard

Re: [PATCH V6 0/5] perf: Driver specific configuration for PMU

2016-08-29 Thread Mathieu Poirier
On 28 August 2016 at 08:18, Jiri Olsa wrote: > On Thu, Aug 25, 2016 at 02:20:40PM -0600, Mathieu Poirier wrote: > > SNIP > >> perf record -e some_event/@cfg1,@cfg2=config/ ... >> >> The above are all valid configuration and will see the strings 'cfg1' >> and 'cfg2=config' sent

Re: [PATCH V6 0/5] perf: Driver specific configuration for PMU

2016-08-29 Thread Mathieu Poirier
On 28 August 2016 at 08:18, Jiri Olsa wrote: > On Thu, Aug 25, 2016 at 02:20:40PM -0600, Mathieu Poirier wrote: > > SNIP > >> perf record -e some_event/@cfg1,@cfg2=config/ ... >> >> The above are all valid configuration and will see the strings 'cfg1' >> and 'cfg2=config' sent to the PMU driver

Re: [PATCH V6 5/5] perf tools: adding sink configuration for cs_etm PMU

2016-08-29 Thread Mathieu Poirier
On 28 August 2016 at 08:16, Jiri Olsa wrote: > On Thu, Aug 25, 2016 at 02:20:45PM -0600, Mathieu Poirier wrote: > > SNIP > >> return NULL; >> } >> + >> +static FILE *cs_device__open_file(const char *name) >> +{ >> + struct stat st; >> + char path[PATH_MAX]; >> +

Re: [PATCH V6 5/5] perf tools: adding sink configuration for cs_etm PMU

2016-08-29 Thread Mathieu Poirier
On 28 August 2016 at 08:16, Jiri Olsa wrote: > On Thu, Aug 25, 2016 at 02:20:45PM -0600, Mathieu Poirier wrote: > > SNIP > >> return NULL; >> } >> + >> +static FILE *cs_device__open_file(const char *name) >> +{ >> + struct stat st; >> + char path[PATH_MAX]; >> + const char

Re: [PATCH V6 1/5] perf tools: making coresight PMU listable

2016-08-29 Thread Mathieu Poirier
On 28 August 2016 at 08:16, Jiri Olsa wrote: > On Thu, Aug 25, 2016 at 02:20:41PM -0600, Mathieu Poirier wrote: > > SNIP > >> ARM/CORGI MACHINE SUPPORT >> M: Richard Purdie >> diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config >> index

Re: [PATCH V6 1/5] perf tools: making coresight PMU listable

2016-08-29 Thread Mathieu Poirier
On 28 August 2016 at 08:16, Jiri Olsa wrote: > On Thu, Aug 25, 2016 at 02:20:41PM -0600, Mathieu Poirier wrote: > > SNIP > >> ARM/CORGI MACHINE SUPPORT >> M: Richard Purdie >> diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config >> index 24803c58049a..72edf83d76b7 100644 >>

Re: [PATCH 2/2] phy: rcar-gen3-usb2: Add a compatible string for r8a7796

2016-08-29 Thread Rob Herring
On Wed, Aug 24, 2016 at 03:49:22PM +0900, Yoshihiro Shimoda wrote: > This driver can support for r8a7796 SoC. So, this patch adds it. > > Signed-off-by: Yoshihiro Shimoda > --- > Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb2.txt | 2 ++ >

Re: [PATCH 2/2] phy: rcar-gen3-usb2: Add a compatible string for r8a7796

2016-08-29 Thread Rob Herring
On Wed, Aug 24, 2016 at 03:49:22PM +0900, Yoshihiro Shimoda wrote: > This driver can support for r8a7796 SoC. So, this patch adds it. > > Signed-off-by: Yoshihiro Shimoda > --- > Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb2.txt | 2 ++ > drivers/phy/phy-rcar-gen3-usb2.c

Re: [PATCH 1/2] phy: rcar-gen3-usb2: revise the example of device tree doc

2016-08-29 Thread Rob Herring
On Wed, Aug 24, 2016 at 03:49:21PM +0900, Yoshihiro Shimoda wrote: > The clocks property should be set to , not _clks. > > Signed-off-by: Yoshihiro Shimoda > --- > Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb2.txt | 4 ++-- > 1 file changed, 2

Re: [PATCH 1/2] phy: rcar-gen3-usb2: revise the example of device tree doc

2016-08-29 Thread Rob Herring
On Wed, Aug 24, 2016 at 03:49:21PM +0900, Yoshihiro Shimoda wrote: > The clocks property should be set to , not _clks. > > Signed-off-by: Yoshihiro Shimoda > --- > Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb2.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Acked-by:

Re: [PATCH v8 4/4] nmi_backtrace: generate one-line reports for idle cpus

2016-08-29 Thread Chris Metcalf
On 8/18/2016 11:12 AM, Petr Mladek wrote: On Tue 2016-08-16 15:50:24, Chris Metcalf wrote: [...] -static inline void native_safe_halt(void) +static __always_inline void native_safe_halt(void) { asm volatile("sti; hlt": : :"memory"); } Ah, the __always_inline stuff did not helped

Re: [PATCH v8 4/4] nmi_backtrace: generate one-line reports for idle cpus

2016-08-29 Thread Chris Metcalf
On 8/18/2016 11:12 AM, Petr Mladek wrote: On Tue 2016-08-16 15:50:24, Chris Metcalf wrote: [...] -static inline void native_safe_halt(void) +static __always_inline void native_safe_halt(void) { asm volatile("sti; hlt": : :"memory"); } Ah, the __always_inline stuff did not helped

Re: [RFC][PATCH -v2 4/4] locking/mutex: Add lock handoff to avoid starvation

2016-08-29 Thread Peter Zijlstra
On Fri, Aug 26, 2016 at 07:40:34PM -0400, Waiman Long wrote: > On 08/26/2016 11:18 AM, Peter Zijlstra wrote: > >Still need to look at adding spinning to the handoff case. > >Also need to look at writing (much) better changelogs, they stink. > > > > I have looked at the handoff code and I didn't

Re: [RFC][PATCH -v2 4/4] locking/mutex: Add lock handoff to avoid starvation

2016-08-29 Thread Peter Zijlstra
On Fri, Aug 26, 2016 at 07:40:34PM -0400, Waiman Long wrote: > On 08/26/2016 11:18 AM, Peter Zijlstra wrote: > >Still need to look at adding spinning to the handoff case. > >Also need to look at writing (much) better changelogs, they stink. > > > > I have looked at the handoff code and I didn't

[PATCH v2 1/3] toshiba_acpi: Clean up variable declaration

2016-08-29 Thread Azael Avalos
This patch moves all the multiple line variable declaration to a single line declaration (except variables being initialized) following the reverse tree order, to conform to the practices of the kernel. Signed-off-by: Azael Avalos --- drivers/platform/x86/toshiba_acpi.c

[PATCH v2 1/3] toshiba_acpi: Clean up variable declaration

2016-08-29 Thread Azael Avalos
This patch moves all the multiple line variable declaration to a single line declaration (except variables being initialized) following the reverse tree order, to conform to the practices of the kernel. Signed-off-by: Azael Avalos --- drivers/platform/x86/toshiba_acpi.c | 33

Re: [PATCH v5] iio: stx104: Add IIO support for the ADC channels

2016-08-29 Thread Jonathan Cameron
On 23/08/16 18:46, William Breathitt Gray wrote: > The Apex Embedded Systems STX104 features 16 channels of single-ended (8 > channels of true differential) 16-bit analog input. Differential input > configuration may be selected via a physical jumper on the device. > Similarly, input polarity

Re: [PATCH v5] iio: stx104: Add IIO support for the ADC channels

2016-08-29 Thread Jonathan Cameron
On 23/08/16 18:46, William Breathitt Gray wrote: > The Apex Embedded Systems STX104 features 16 channels of single-ended (8 > channels of true differential) 16-bit analog input. Differential input > configuration may be selected via a physical jumper on the device. > Similarly, input polarity

Re: [PATCH 1/3] Documentation: dtb: xgene: Add PMD clock binding

2016-08-29 Thread Rob Herring
On Tue, Aug 23, 2016 at 08:52:39AM -0700, Hoan Tran wrote: > Add APM X-Gene clock binding documentation for PMD clock. > > Signed-off-by: Hoan Tran > --- > Documentation/devicetree/bindings/clock/xgene.txt | 22 ++ > 1 file changed, 22 insertions(+) > > diff

Re: [PATCH 1/3] Documentation: dtb: xgene: Add PMD clock binding

2016-08-29 Thread Rob Herring
On Tue, Aug 23, 2016 at 08:52:39AM -0700, Hoan Tran wrote: > Add APM X-Gene clock binding documentation for PMD clock. > > Signed-off-by: Hoan Tran > --- > Documentation/devicetree/bindings/clock/xgene.txt | 22 ++ > 1 file changed, 22 insertions(+) > > diff --git

Re: [PATCH 2/2] mm/usercopy: enable usercopy size checking for modern versions of gcc

2016-08-29 Thread Linus Torvalds
On Mon, Aug 29, 2016 at 7:48 AM, Josh Poimboeuf wrote: > > So I *think* your patch fixes the wrong problem. That's probably at > least somewhat my fault because I misunderstood the issue before and may > have described it wrong at some point. > > AFAICT, gcc isn't doing

Re: [PATCH 2/2] mm/usercopy: enable usercopy size checking for modern versions of gcc

2016-08-29 Thread Linus Torvalds
On Mon, Aug 29, 2016 at 7:48 AM, Josh Poimboeuf wrote: > > So I *think* your patch fixes the wrong problem. That's probably at > least somewhat my fault because I misunderstood the issue before and may > have described it wrong at some point. > > AFAICT, gcc isn't doing anything wrong, and the

[patch 2/2] i2c: mux: mellanox: add driver

2016-08-29 Thread vadimp
From: Vadim Pasternak This driver allows I2C routing controlled through CPLD select registers on wide range of Mellanox systems (CPLD Lattice device). MUX selection is provided by digital and analog HW. Analog part is not under SW control. Digital part is under CPLD control

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

2016-08-29 Thread Matthew Wilcox
Thanks, Ross. Konstantin, I think there are problems with the concept behind this series. You have multiple entries in the tree with the same value. That works out fine when the entry is a pointer (eg to a struct page), but not so well when it's an exceptional entry (eg a swap cache entry or

[PATCH v2 3/3] toshiba_acpi: Fix typo in *_cooling_method_set function

2016-08-29 Thread Azael Avalos
This patch simply fixes a typo in the error string printed in such function. Signed-off-by: Azael Avalos --- drivers/platform/x86/toshiba_acpi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/platform/x86/toshiba_acpi.c

[patch 2/2] i2c: mux: mellanox: add driver

2016-08-29 Thread vadimp
From: Vadim Pasternak This driver allows I2C routing controlled through CPLD select registers on wide range of Mellanox systems (CPLD Lattice device). MUX selection is provided by digital and analog HW. Analog part is not under SW control. Digital part is under CPLD control (channel

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

2016-08-29 Thread Matthew Wilcox
Thanks, Ross. Konstantin, I think there are problems with the concept behind this series. You have multiple entries in the tree with the same value. That works out fine when the entry is a pointer (eg to a struct page), but not so well when it's an exceptional entry (eg a swap cache entry or

[PATCH v2 3/3] toshiba_acpi: Fix typo in *_cooling_method_set function

2016-08-29 Thread Azael Avalos
This patch simply fixes a typo in the error string printed in such function. Signed-off-by: Azael Avalos --- drivers/platform/x86/toshiba_acpi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c index

[PATCH v2 0/3] toshiba_acpi: Functions cleanup

2016-08-29 Thread Azael Avalos
These set of patches make some cleanup in some of the driver's functions. The first patch changes the variables declaration, the second changes the error checking logic and the third simply fixes a typo found while making these changes. Changes since v1: - Small changes to patch 01 acording to

[PATCH v2 2/3] toshiba_acpi: Change error checking logic from TCI functions

2016-08-29 Thread Azael Avalos
Currently the success/error checking logic is intermixed, making the code a bit cumbersome to understand. This patch changes the affected functions to first check for errors and take appropriate actions, then check for the supported features. This patch also separates the error check from the

[PATCH v2 0/3] toshiba_acpi: Functions cleanup

2016-08-29 Thread Azael Avalos
These set of patches make some cleanup in some of the driver's functions. The first patch changes the variables declaration, the second changes the error checking logic and the third simply fixes a typo found while making these changes. Changes since v1: - Small changes to patch 01 acording to

[PATCH v2 2/3] toshiba_acpi: Change error checking logic from TCI functions

2016-08-29 Thread Azael Avalos
Currently the success/error checking logic is intermixed, making the code a bit cumbersome to understand. This patch changes the affected functions to first check for errors and take appropriate actions, then check for the supported features. This patch also separates the error check from the

Re: mm: use-after-free in collapse_huge_page

2016-08-29 Thread Andrea Arcangeli
Hello Kirill, On Mon, Aug 29, 2016 at 03:42:33PM +0300, Kirill A. Shutemov wrote: > @@ -898,13 +899,13 @@ static bool __collapse_huge_page_swapin(struct > mm_struct *mm, > /* do_swap_page returns VM_FAULT_RETRY with released mmap_sem */ > if (ret & VM_FAULT_RETRY) { >

Re: mm: use-after-free in collapse_huge_page

2016-08-29 Thread Andrea Arcangeli
Hello Kirill, On Mon, Aug 29, 2016 at 03:42:33PM +0300, Kirill A. Shutemov wrote: > @@ -898,13 +899,13 @@ static bool __collapse_huge_page_swapin(struct > mm_struct *mm, > /* do_swap_page returns VM_FAULT_RETRY with released mmap_sem */ > if (ret & VM_FAULT_RETRY) { >

Re: [PATCH] irqchip/gicv3-its: Enable cacheable attribute Read-allocate hints

2016-08-29 Thread Shanker Donthineni
Marc, Are you planning to push this change? I talked to Qualcomm ITS hw team and they told me nice to have this change even though we see a small gain. Shanker On 07/12/2016 08:32 AM, Shanker Donthineni wrote: Hi Marc, On 07/12/2016 03:09 AM, Marc Zyngier wrote: Hi Shanker, On 12/07/16

Re: [PATCH] irqchip/gicv3-its: Enable cacheable attribute Read-allocate hints

2016-08-29 Thread Shanker Donthineni
Marc, Are you planning to push this change? I talked to Qualcomm ITS hw team and they told me nice to have this change even though we see a small gain. Shanker On 07/12/2016 08:32 AM, Shanker Donthineni wrote: Hi Marc, On 07/12/2016 03:09 AM, Marc Zyngier wrote: Hi Shanker, On 12/07/16

Re: [PATCH 1/3] toshiba_acpi: Cleanup variable declaration

2016-08-29 Thread Azael Avalos
Hi Darren, 2016-08-28 10:49 GMT-06:00 Darren Hart : > On Tue, Aug 16, 2016 at 12:06:15PM -0600, Azael Avalos wrote: >> This patch moves all the multiple line variable declaration to a >> single line declaration (except variables being initialized) >> following the reverse

Re: [PATCH 1/3] toshiba_acpi: Cleanup variable declaration

2016-08-29 Thread Azael Avalos
Hi Darren, 2016-08-28 10:49 GMT-06:00 Darren Hart : > On Tue, Aug 16, 2016 at 12:06:15PM -0600, Azael Avalos wrote: >> This patch moves all the multiple line variable declaration to a >> single line declaration (except variables being initialized) >> following the reverse tree order, to conform

Re: [PATCH 2/3] toshiba_acpi: Change error checking logic from TCI functions

2016-08-29 Thread Azael Avalos
Hi Darren, 2016-08-28 10:56 GMT-06:00 Darren Hart : > On Tue, Aug 16, 2016 at 12:06:16PM -0600, Azael Avalos wrote: >> Currently the success/error checking logic is intermixed, making the >> code a bit cumbersome to understand. >> >> This patch changes the affected functions

Re: [PATCH v1] mm, sysctl: Add sysctl for controlling VM_MAYEXEC taint

2016-08-29 Thread Robert Foss
On 2016-08-29 11:25 AM, Will Drewry wrote: On Fri, Aug 26, 2016 at 4:32 PM, Kirill A. Shutemov > wrote: On Fri, Aug 26, 2016 at 12:30:04PM -0400, robert.f...@collabora.com wrote: > From: Will

Re: [PATCH v4] brcmfmac: add missing header dependencies

2016-08-29 Thread Rafał Miłecki
On 29 August 2016 at 14:39, Baoyou Xie wrote: > We get 1 warning when build kernel with W=1: > drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.c:23:6: warning: > no previous prototype for '__brcmf_err' [-Wmissing-prototypes] building? I'm not native English,

Re: [PATCH 2/3] toshiba_acpi: Change error checking logic from TCI functions

2016-08-29 Thread Azael Avalos
Hi Darren, 2016-08-28 10:56 GMT-06:00 Darren Hart : > On Tue, Aug 16, 2016 at 12:06:16PM -0600, Azael Avalos wrote: >> Currently the success/error checking logic is intermixed, making the >> code a bit cumbersome to understand. >> >> This patch changes the affected functions to first check for

Re: [PATCH v1] mm, sysctl: Add sysctl for controlling VM_MAYEXEC taint

2016-08-29 Thread Robert Foss
On 2016-08-29 11:25 AM, Will Drewry wrote: On Fri, Aug 26, 2016 at 4:32 PM, Kirill A. Shutemov mailto:kir...@shutemov.name>> wrote: On Fri, Aug 26, 2016 at 12:30:04PM -0400, robert.f...@collabora.com wrote: > From: Will Drewry

Re: [PATCH v4] brcmfmac: add missing header dependencies

2016-08-29 Thread Rafał Miłecki
On 29 August 2016 at 14:39, Baoyou Xie wrote: > We get 1 warning when build kernel with W=1: > drivers/net/wireless/broadcom/brcm80211/brcmfmac/tracepoint.c:23:6: warning: > no previous prototype for '__brcmf_err' [-Wmissing-prototypes] building? I'm not native English, but I think so. > In

Re: [PATCH] staging: iio: accel: sca3000: remove extra space

2016-08-29 Thread Jonathan Cameron
On 23/08/16 03:45, Clifton Barnes wrote: > fix checkpatch.pl warning about 'Statements should start on a > tabstop' > > Signed-off-by: Clifton Barnes Applied to the togreg branch of iio.git. Will be initially pushed out as testing for the autobuilders to play with

Re: [PATCH] staging: iio: accel: sca3000: remove extra space

2016-08-29 Thread Jonathan Cameron
On 23/08/16 03:45, Clifton Barnes wrote: > fix checkpatch.pl warning about 'Statements should start on a > tabstop' > > Signed-off-by: Clifton Barnes Applied to the togreg branch of iio.git. Will be initially pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- >

[AFC] cgroup: Fix block throttle bio more than once

2016-08-29 Thread aaronlee0817
Hi Tejun, Few months ago, we send an email to tell the problem of blkio throttle the split bio more than once, which leads to the actual bandwidth is smaller than we have set. And we try to fix this by adding a flag bit BIO_SPLIT in bio flag field. And these days we did some enhance test, and

[AFC] cgroup: Fix block throttle bio more than once

2016-08-29 Thread aaronlee0817
Hi Tejun, Few months ago, we send an email to tell the problem of blkio throttle the split bio more than once, which leads to the actual bandwidth is smaller than we have set. And we try to fix this by adding a flag bit BIO_SPLIT in bio flag field. And these days we did some enhance test, and

[PATCH] ARM: dts: rockchip: temporarily remove emmc hs200 speed from rk3288-veyron-speedy.

2016-08-29 Thread Vagrant Cascadian
This essentially mimics what was done with rk3288-veyron-minnie in commit 984926781122f034d5bc9962815d135b6c4a8e1d. The eMMC of the speedy Chromebook also appears to need the same tuning workaround, as it frequently fails to recognize the eMMC without it. ---

[PATCH] ARM: dts: rockchip: temporarily remove emmc hs200 speed from rk3288-veyron-speedy.

2016-08-29 Thread Vagrant Cascadian
This essentially mimics what was done with rk3288-veyron-minnie in commit 984926781122f034d5bc9962815d135b6c4a8e1d. The eMMC of the speedy Chromebook also appears to need the same tuning workaround, as it frequently fails to recognize the eMMC without it. ---

Re: [PATCH] tpm: fix a race condition in tpm2_unseal_trusted()

2016-08-29 Thread Jarkko Sakkinen
On Sun, Aug 28, 2016 at 12:51:49PM -0600, Jason Gunthorpe wrote: > On Sun, Aug 28, 2016 at 08:36:52AM +0200, Jarkko Sakkinen wrote: > > > > @@ -576,7 +576,8 @@ static int tpm2_load(struct tpm_chip *chip, > > goto out; > > } > > > > - rc = tpm_transmit_cmd(chip, buf.data,

Re: [PATCH] tpm: fix a race condition in tpm2_unseal_trusted()

2016-08-29 Thread Jarkko Sakkinen
On Sun, Aug 28, 2016 at 12:51:49PM -0600, Jason Gunthorpe wrote: > On Sun, Aug 28, 2016 at 08:36:52AM +0200, Jarkko Sakkinen wrote: > > > > @@ -576,7 +576,8 @@ static int tpm2_load(struct tpm_chip *chip, > > goto out; > > } > > > > - rc = tpm_transmit_cmd(chip, buf.data,

Re: [PATCH RFC v3] x86,mm,sched: make lazy TLB mode even lazier

2016-08-29 Thread Rik van Riel
On Sat, 2016-08-27 at 10:03 +0200, Ingo Molnar wrote: > * Rik van Riel wrote: > > > On Thu, 25 Aug 2016 12:42:15 -0700 > > "H. Peter Anvin" wrote: > > > > > Why grabbing a lock instead of cmpxchg? > > > > ... and some more cleanups later, this might actually

Re: [PATCH RFC v3] x86,mm,sched: make lazy TLB mode even lazier

2016-08-29 Thread Rik van Riel
On Sat, 2016-08-27 at 10:03 +0200, Ingo Molnar wrote: > * Rik van Riel wrote: > > > On Thu, 25 Aug 2016 12:42:15 -0700 > > "H. Peter Anvin" wrote: > > > > > Why grabbing a lock instead of cmpxchg? > > > > ... and some more cleanups later, this might actually be > > good to merge, assuming it

Re: [PATCH] livepatch: add load/unload hooks to objects

2016-08-29 Thread Petr Mladek
On Fri 2016-08-26 13:50:27, Chris J Arges wrote: > It can be useful to execute hook functions whenever a livepatch is applied > or unapplied to a particular object. Currently this is possible by writing > logic in the __init function of the livepatch kernel module. However to > handle executing

Re: [PATCH] livepatch: add load/unload hooks to objects

2016-08-29 Thread Petr Mladek
On Fri 2016-08-26 13:50:27, Chris J Arges wrote: > It can be useful to execute hook functions whenever a livepatch is applied > or unapplied to a particular object. Currently this is possible by writing > logic in the __init function of the livepatch kernel module. However to > handle executing

Re: [PATCH] arm64: KVM: Save two instructions in __guest_enter()

2016-08-29 Thread Shanker Donthineni
Hi Christoffer, This is change may not provide the measurable performance improvement, but still we can save a few cpu cycles on vCPU context switch and also improves the code readability. On 08/25/2016 08:31 AM, Christoffer Dall wrote: Hi Shanker, On Tue, Aug 09, 2016 at 08:15:36PM

Re: [PATCH] arm64: KVM: Save two instructions in __guest_enter()

2016-08-29 Thread Shanker Donthineni
Hi Christoffer, This is change may not provide the measurable performance improvement, but still we can save a few cpu cycles on vCPU context switch and also improves the code readability. On 08/25/2016 08:31 AM, Christoffer Dall wrote: Hi Shanker, On Tue, Aug 09, 2016 at 08:15:36PM

Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q

2016-08-29 Thread Shawn Guo
On Mon, Aug 29, 2016 at 02:44:47PM +, Yongcai Huang wrote: > This patch is to set the GINT bit for all low power mode transition. > > Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend > is > already supported with non-PCSI method, only i.MX7D uses PSCI, right? If

Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q

2016-08-29 Thread Shawn Guo
On Mon, Aug 29, 2016 at 02:44:47PM +, Yongcai Huang wrote: > This patch is to set the GINT bit for all low power mode transition. > > Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend > is > already supported with non-PCSI method, only i.MX7D uses PSCI, right? If

Re: 4.8-rc4 spews "BUG: sleeping function called from invalid context at fs/dcache.c:757"

2016-08-29 Thread Al Viro
On Mon, Aug 29, 2016 at 04:35:46PM +0200, Takashi Iwai wrote: > [] dput+0x46/0x400 ... which should not be called in atomic contexts > [] follow_down_one+0x27/0x60 ... and neither should this > [] autofs4_mount_busy+0x32/0x110 ... nor that (for fsck sake, there's

Re: 4.8-rc4 spews "BUG: sleeping function called from invalid context at fs/dcache.c:757"

2016-08-29 Thread Al Viro
On Mon, Aug 29, 2016 at 04:35:46PM +0200, Takashi Iwai wrote: > [] dput+0x46/0x400 ... which should not be called in atomic contexts > [] follow_down_one+0x27/0x60 ... and neither should this > [] autofs4_mount_busy+0x32/0x110 ... nor that (for fsck sake, there's

Re: [RFC PATCH v8 1/9] Restartable sequences system call

2016-08-29 Thread Mathieu Desnoyers
- On Aug 27, 2016, at 12:22 AM, Josh Triplett j...@joshtriplett.org wrote: > On Thu, Aug 25, 2016 at 05:56:25PM +, Ben Maurer wrote: >> rseq opens up a whole world of algorithms to userspace – algorithms >> that are O(num CPUs) and where one can have an extremely fast fastpath >> at the

Re: [RFC PATCH v8 1/9] Restartable sequences system call

2016-08-29 Thread Mathieu Desnoyers
- On Aug 27, 2016, at 12:22 AM, Josh Triplett j...@joshtriplett.org wrote: > On Thu, Aug 25, 2016 at 05:56:25PM +, Ben Maurer wrote: >> rseq opens up a whole world of algorithms to userspace – algorithms >> that are O(num CPUs) and where one can have an extremely fast fastpath >> at the

Re: [PATCH] iio: ad5755: fix off-by-one on devnr limit check

2016-08-29 Thread Jonathan Cameron
On 21/08/16 21:07, Lars-Peter Clausen wrote: > On 08/21/2016 09:30 PM, Jonathan Cameron wrote: >> On 25/07/16 23:40, Colin King wrote: >>> From: Colin Ian King >>> >>> The comparison for devnr limits is off-by-one, the current check >>> allows 0 to AD5755_NUM_CHANNELS

Re: [PATCH] iio: ad5755: fix off-by-one on devnr limit check

2016-08-29 Thread Jonathan Cameron
On 21/08/16 21:07, Lars-Peter Clausen wrote: > On 08/21/2016 09:30 PM, Jonathan Cameron wrote: >> On 25/07/16 23:40, Colin King wrote: >>> From: Colin Ian King >>> >>> The comparison for devnr limits is off-by-one, the current check >>> allows 0 to AD5755_NUM_CHANNELS and the limit should be in

Re: [PATCH] printk/nmi: avoid direct printk()-s from __printk_nmi_flush()

2016-08-29 Thread Petr Mladek
On Mon 2016-08-29 21:32:20, Sergey Senozhatsky wrote: > From: Sergey Senozhatsky > > __printk_nmi_flush() can be called from nmi_panic(), therefore it has to > test whether it's executed in NMI context and thus must route the messages > through deferred

Re: [PATCH] printk/nmi: avoid direct printk()-s from __printk_nmi_flush()

2016-08-29 Thread Petr Mladek
On Mon 2016-08-29 21:32:20, Sergey Senozhatsky wrote: > From: Sergey Senozhatsky > > __printk_nmi_flush() can be called from nmi_panic(), therefore it has to > test whether it's executed in NMI context and thus must route the messages > through deferred printk() or via direct printk(). Except

Re: [PATCH v4 0/4] Add support for cros-ec-sensors

2016-08-29 Thread Jonathan Cameron
On 21/08/16 20:57, Jonathan Cameron wrote: > On 16/08/16 13:49, Lee Jones wrote: >> On Mon, 15 Aug 2016, Jonathan Cameron wrote: >> >>> On 01/08/16 10:54, Enric Balletbo i Serra wrote: Hi, This is the four patchset to add support for the sensors behind the ChromeOS

Re: [PATCH v4 0/4] Add support for cros-ec-sensors

2016-08-29 Thread Jonathan Cameron
On 21/08/16 20:57, Jonathan Cameron wrote: > On 16/08/16 13:49, Lee Jones wrote: >> On Mon, 15 Aug 2016, Jonathan Cameron wrote: >> >>> On 01/08/16 10:54, Enric Balletbo i Serra wrote: Hi, This is the four patchset to add support for the sensors behind the ChromeOS

Re: [PATCH][v8] PM / hibernate: Verify the consistent of e820 memory map by md5 value

2016-08-29 Thread Pavel Machek
On Mon 2016-08-29 15:41:34, Rafael J. Wysocki wrote: > On Mon, Aug 29, 2016 at 6:59 AM, Borislav Petkov wrote: > > On Mon, Aug 29, 2016 at 12:35:40AM +0800, Chen Yu wrote: > >> On some platforms, there is occasional panic triggered when trying to > >> resume from hibernation, a

Re: [PATCH][v8] PM / hibernate: Verify the consistent of e820 memory map by md5 value

2016-08-29 Thread Pavel Machek
On Mon 2016-08-29 15:41:34, Rafael J. Wysocki wrote: > On Mon, Aug 29, 2016 at 6:59 AM, Borislav Petkov wrote: > > On Mon, Aug 29, 2016 at 12:35:40AM +0800, Chen Yu wrote: > >> On some platforms, there is occasional panic triggered when trying to > >> resume from hibernation, a typical panic

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

2016-08-29 Thread Shawn Guo
On Mon, Aug 29, 2016 at 03:51:30PM +0200, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 04:33:50PM +0800, Shawn Guo wrote: > > To avoid potential merge conflicts. > > Haven't heard of any so far. And I don't see how adding 1 or 2 DT > entries more per driver is a serious merge conflict.

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

2016-08-29 Thread Shawn Guo
On Mon, Aug 29, 2016 at 03:51:30PM +0200, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 04:33:50PM +0800, Shawn Guo wrote: > > To avoid potential merge conflicts. > > Haven't heard of any so far. And I don't see how adding 1 or 2 DT > entries more per driver is a serious merge conflict.

Re: [PATCH] drm/i915/skl: Don't try to update plane watermarks if they haven't changed

2016-08-29 Thread Jani Nikula
On Mon, 29 Aug 2016, Lyude wrote: > i915 sometimes needs to disable planes in the middle of an atomic > commit, and then reenable them later in the same commit. Because of > this, we can't make the assumption that the state of the plane actually > changed. Since the state of the

Re: [PATCH] drm/i915/skl: Don't try to update plane watermarks if they haven't changed

2016-08-29 Thread Jani Nikula
On Mon, 29 Aug 2016, Lyude wrote: > i915 sometimes needs to disable planes in the middle of an atomic > commit, and then reenable them later in the same commit. Because of > this, we can't make the assumption that the state of the plane actually > changed. Since the state of the plane hasn't

[PATCH] perf probe: Move dwarf specific functions to dwarf-aux.c

2016-08-29 Thread Ravi Bangoria
Move generic dwarf related functions from util/probe-finder.c to util/dwarf-aux.c. Function names and their prototype are also changed accordingly. No functionality changes. Suggested-by: Masami Hiramatsu Signed-off-by: Ravi Bangoria ---

[PATCH] perf probe: Move dwarf specific functions to dwarf-aux.c

2016-08-29 Thread Ravi Bangoria
Move generic dwarf related functions from util/probe-finder.c to util/dwarf-aux.c. Function names and their prototype are also changed accordingly. No functionality changes. Suggested-by: Masami Hiramatsu Signed-off-by: Ravi Bangoria --- tools/perf/util/dwarf-aux.c| 135

[PATCH] android: ashmem: handle the failures in ashmen driver

2016-08-29 Thread Arvind Yadav
Check return value of range_alloc, ashmem_shrink_scan and register_shrinker. Handle errors correctly. Signed-off-by: Arvind Yadav --- drivers/staging/android/ashmem.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git

[PATCH] android: ashmem: handle the failures in ashmen driver

2016-08-29 Thread Arvind Yadav
Check return value of range_alloc, ashmem_shrink_scan and register_shrinker. Handle errors correctly. Signed-off-by: Arvind Yadav --- drivers/staging/android/ashmem.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/drivers/staging/android/ashmem.c

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

2016-08-29 Thread J Freyensee
On Mon, 2016-08-29 at 02:25 -0700, Andy Lutomirski wrote: > NVME devices can advertise multiple power states.  These states can > be either "operational" (the device is fully functional but possibly > slow) or "non-operational" (the device is asleep until woken up). > Some devices can

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

2016-08-29 Thread J Freyensee
On Mon, 2016-08-29 at 02:25 -0700, Andy Lutomirski wrote: > NVME devices can advertise multiple power states.  These states can > be either "operational" (the device is fully functional but possibly > slow) or "non-operational" (the device is asleep until woken up). > Some devices can

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