[PATCH] leds: do not overflow sysfs buffer in led_trigger_show

2016-08-15 Thread Zach Brown
From: Nathan Sullivan Per the documentation, use scnprintf instead of sprintf to ensure there is never more than PAGE_SIZE bytes of trigger names put into the buffer. Signed-off-by: Nathan Sullivan Signed-off-by: Zach Brown

[PATCH] leds: do not overflow sysfs buffer in led_trigger_show

2016-08-15 Thread Zach Brown
From: Nathan Sullivan Per the documentation, use scnprintf instead of sprintf to ensure there is never more than PAGE_SIZE bytes of trigger names put into the buffer. Signed-off-by: Nathan Sullivan Signed-off-by: Zach Brown --- drivers/leds/led-triggers.c | 12 +++- 1 file changed, 7

Re: [PATCHv2 3/4] pci: Determine actual VPD size on first access

2016-08-15 Thread Benjamin Herrenschmidt
On Tue, 2016-08-16 at 08:23 +1000, Benjamin Herrenschmidt wrote: > > I don't think desktop users appreciate hangs any more than anyone else, and  > >   > > that is one of the symptoms that can arise here without the vfio   > > coordination. > > And can happen with it as well  Oh and your

Re: [PATCHv2 3/4] pci: Determine actual VPD size on first access

2016-08-15 Thread Benjamin Herrenschmidt
On Tue, 2016-08-16 at 08:23 +1000, Benjamin Herrenschmidt wrote: > > I don't think desktop users appreciate hangs any more than anyone else, and  > >   > > that is one of the symptoms that can arise here without the vfio   > > coordination. > > And can happen with it as well  Oh and your

Re: [PATCH 2/2] power: reset: syscon-reboot-mode: Use managed resource API

2016-08-15 Thread Sebastian Reichel
Hi, On Wed, Aug 03, 2016 at 10:04:06PM -0700, Bjorn Andersson wrote: > Use the managed resource version of reboot_mode_register(). > > [...] > > drivers/power/reset/syscon-reboot-mode.c | 12 +--- > 1 file changed, 1 insertion(+), 11 deletions(-) Thanks, queued. -- Sebastian

Re: [PATCH 2/2] power: reset: syscon-reboot-mode: Use managed resource API

2016-08-15 Thread Sebastian Reichel
Hi, On Wed, Aug 03, 2016 at 10:04:06PM -0700, Bjorn Andersson wrote: > Use the managed resource version of reboot_mode_register(). > > [...] > > drivers/power/reset/syscon-reboot-mode.c | 12 +--- > 1 file changed, 1 insertion(+), 11 deletions(-) Thanks, queued. -- Sebastian

Re: [PATCH 1/2] power: reset: reboot-mode: Add managed resource API

2016-08-15 Thread Sebastian Reichel
Hi, On Wed, Aug 03, 2016 at 10:04:05PM -0700, Bjorn Andersson wrote: > Provide managed resource version of reboot_mode_register() and > reboot_mode_unregister() to simplify implementations. Thanks, queued. -- Sebastian signature.asc Description: PGP signature

Re: [PATCH 1/2] power: reset: reboot-mode: Add managed resource API

2016-08-15 Thread Sebastian Reichel
Hi, On Wed, Aug 03, 2016 at 10:04:05PM -0700, Bjorn Andersson wrote: > Provide managed resource version of reboot_mode_register() and > reboot_mode_unregister() to simplify implementations. Thanks, queued. -- Sebastian signature.asc Description: PGP signature

RE: [PATCH v2 RESEND 2/4] Drivers: hv: balloon: account for gaps in hot add regions

2016-08-15 Thread Alex Ng (LIS)
> @@ -676,35 +686,63 @@ static void hv_mem_hot_add(unsigned long start, > unsigned long size, > > static void hv_online_page(struct page *pg) { > - struct list_head *cur; > struct hv_hotadd_state *has; > + struct hv_hotadd_gap *gap; > unsigned long cur_start_pgp; >

RE: [PATCH v2 RESEND 2/4] Drivers: hv: balloon: account for gaps in hot add regions

2016-08-15 Thread Alex Ng (LIS)
> @@ -676,35 +686,63 @@ static void hv_mem_hot_add(unsigned long start, > unsigned long size, > > static void hv_online_page(struct page *pg) { > - struct list_head *cur; > struct hv_hotadd_state *has; > + struct hv_hotadd_gap *gap; > unsigned long cur_start_pgp; >

Re: [PATCH v2 2/5] clk: qcom: ipq4019: Added the apss cpu pll divider clock node

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > diff --git a/drivers/clk/qcom/gcc-ipq4019.c b/drivers/clk/qcom/gcc-ipq4019.c > index 17ca6d3..4c8a97f 100644 > --- a/drivers/clk/qcom/gcc-ipq4019.c > +++ b/drivers/clk/qcom/gcc-ipq4019.c > @@ -154,7 +154,7 @@ static const char * const gcc_xo_ddr_500_200[] = { >

Re: [PATCH v2 2/5] clk: qcom: ipq4019: Added the apss cpu pll divider clock node

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > diff --git a/drivers/clk/qcom/gcc-ipq4019.c b/drivers/clk/qcom/gcc-ipq4019.c > index 17ca6d3..4c8a97f 100644 > --- a/drivers/clk/qcom/gcc-ipq4019.c > +++ b/drivers/clk/qcom/gcc-ipq4019.c > @@ -154,7 +154,7 @@ static const char * const gcc_xo_ddr_500_200[] = { >

[PATCH v2] Added perf functionality to mmdc driver

2016-08-15 Thread Zhengyu Shen
MMDC is a multi-mode DDR controller that supports DDR3/DDR3L x16/x32/x64 and LPDDR2 two channel x16/x32 memory types. MMDC is configurable, high performance, and optimized. MMDC is present on i.MX6 Quad and i.MX6 QuadPlus devices, but this driver only supports i.MX6 Quad at the moment. MMDC

[PATCH v2] Added perf functionality to mmdc driver

2016-08-15 Thread Zhengyu Shen
MMDC is a multi-mode DDR controller that supports DDR3/DDR3L x16/x32/x64 and LPDDR2 two channel x16/x32 memory types. MMDC is configurable, high performance, and optimized. MMDC is present on i.MX6 Quad and i.MX6 QuadPlus devices, but this driver only supports i.MX6 Quad at the moment. MMDC

Re: [PATCH v2 4/6] kexec_file: Add mechanism to update kexec segments.

2016-08-15 Thread Andrew Morton
On Sat, 13 Aug 2016 00:18:23 -0300 Thiago Jung Bauermann wrote: > kexec_update_segment allows a given segment in kexec_image to have > its contents updated. This is useful if the current kernel wants to > send information to the next kernel that is up-to-date at the

Re: [PATCH v2 4/6] kexec_file: Add mechanism to update kexec segments.

2016-08-15 Thread Andrew Morton
On Sat, 13 Aug 2016 00:18:23 -0300 Thiago Jung Bauermann wrote: > kexec_update_segment allows a given segment in kexec_image to have > its contents updated. This is useful if the current kernel wants to > send information to the next kernel that is up-to-date at the time of > reboot. > > ... >

Re: [PATCH v2 3/5] clk: qcom: ipq4019: Added the nodes for pcnoc

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > The current ipq4019 clock driver does not have the node for > PCNOC so this patch adds and registers the PCNOC clock nodes. > > Signed-off-by: Abhishek Sahu > --- This one looks good. -- Qualcomm Innovation Center, Inc. is a member of

Re: [PATCH v2 3/5] clk: qcom: ipq4019: Added the nodes for pcnoc

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > The current ipq4019 clock driver does not have the node for > PCNOC so this patch adds and registers the PCNOC clock nodes. > > Signed-off-by: Abhishek Sahu > --- This one looks good. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a

Re: [RFC PATCH 0/3] Documentation: switch to pdflatex and fix pdf build

2016-08-15 Thread Mauro Carvalho Chehab
Hi Jon, Em Mon, 15 Aug 2016 09:17:52 -0300 Mauro Carvalho Chehab escreveu: > Em Mon, 15 Aug 2016 12:40:21 +0300 > Jani Nikula escreveu: > > > On Sat, 13 Aug 2016, Jonathan Corbet wrote: > > > On Wed, 10 Aug 2016 18:54:06 +0300 >

Re: [RFC PATCH 0/3] Documentation: switch to pdflatex and fix pdf build

2016-08-15 Thread Mauro Carvalho Chehab
Hi Jon, Em Mon, 15 Aug 2016 09:17:52 -0300 Mauro Carvalho Chehab escreveu: > Em Mon, 15 Aug 2016 12:40:21 +0300 > Jani Nikula escreveu: > > > On Sat, 13 Aug 2016, Jonathan Corbet wrote: > > > On Wed, 10 Aug 2016 18:54:06 +0300 > > > Jani Nikula wrote: > > > > > >> With these you

Re: [PATCH v2 5/5] clk: qcom: ipq4019: Added the cpu clock frequency change notifier

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > The current driver code gives the crash or gets hang while switching > the CPU frequency some time. The APSS CPU Clock divider is not glitch > free so it the APPS clock need to be switched for stable clock during > the change. > > This patch adds the frequency

Re: [PATCH v2 5/5] clk: qcom: ipq4019: Added the cpu clock frequency change notifier

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > The current driver code gives the crash or gets hang while switching > the CPU frequency some time. The APSS CPU Clock divider is not glitch > free so it the APPS clock need to be switched for stable clock during > the change. > > This patch adds the frequency

[PATCH] platform: olpc: make ec explicitly non-modular

2016-08-15 Thread Paul Gortmaker
The Kconfig entry controlling compilation of this code is: arch/x86/Kconfig:config OLPC arch/x86/Kconfig: bool "One Laptop Per Child support" ...meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular infrastructure use, so that when

[PATCH] platform: intel_pmic: make gpio explicitly non-modular

2016-08-15 Thread Paul Gortmaker
The Kconfig entry controlling compilation of this code is: drivers/platform/x86/Kconfig:config GPIO_INTEL_PMIC drivers/platform/x86/Kconfig: bool "Intel PMIC GPIO support" ...meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular

[PATCH] platform: olpc: make ec explicitly non-modular

2016-08-15 Thread Paul Gortmaker
The Kconfig entry controlling compilation of this code is: arch/x86/Kconfig:config OLPC arch/x86/Kconfig: bool "One Laptop Per Child support" ...meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular infrastructure use, so that when

[PATCH] platform: intel_pmic: make gpio explicitly non-modular

2016-08-15 Thread Paul Gortmaker
The Kconfig entry controlling compilation of this code is: drivers/platform/x86/Kconfig:config GPIO_INTEL_PMIC drivers/platform/x86/Kconfig: bool "Intel PMIC GPIO support" ...meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular

Re: [PATCH v2 4/5] clk: qcom: ipq4019: Added the all frequencies for apps cpu

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > @@ -554,11 +554,21 @@ static struct clk_rcg2 sdcc1_apps_clk_src = { > }; > > static const struct freq_tbl ftbl_gcc_apps_clk[] = { > - F(4800, P_XO, 1, 0, 0), > - F(2, P_FEPLL200, 1, 0, 0), > - F(5, P_FEPLL500, 1, 0,

Re: [PATCH v2 4/5] clk: qcom: ipq4019: Added the all frequencies for apps cpu

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > @@ -554,11 +554,21 @@ static struct clk_rcg2 sdcc1_apps_clk_src = { > }; > > static const struct freq_tbl ftbl_gcc_apps_clk[] = { > - F(4800, P_XO, 1, 0, 0), > - F(2, P_FEPLL200, 1, 0, 0), > - F(5, P_FEPLL500, 1, 0,

Re: [PATCHv2 3/4] pci: Determine actual VPD size on first access

2016-08-15 Thread Benjamin Herrenschmidt
On Mon, 2016-08-15 at 17:59 +, Rustad, Mark D wrote: > > Benjamin Herrenschmidt wrote: > > > > > We may want some kind of "strict" vs. "relaxed" model here to > > differenciate the desktop user wanting to give a function to his/her > > windows partition and doesn't

Re: [PATCHv2 3/4] pci: Determine actual VPD size on first access

2016-08-15 Thread Benjamin Herrenschmidt
On Mon, 2016-08-15 at 17:59 +, Rustad, Mark D wrote: > > Benjamin Herrenschmidt wrote: > > > > > We may want some kind of "strict" vs. "relaxed" model here to > > differenciate the desktop user wanting to give a function to his/her > > windows partition and doesn't care about strict

Re: [LKP] [lkp] [xfs] 68a9f5e700: aim7.jobs-per-min -13.6% regression

2016-08-15 Thread Dave Chinner
On Sun, Aug 14, 2016 at 10:12:20PM -0700, Linus Torvalds wrote: > On Aug 14, 2016 10:00 PM, "Dave Chinner" wrote: > > > > > What does it say if you annotate that _raw_spin_unlock_irqrestore() > function? > > > >¿ > >¿Disassembly of section load0: > >

Re: [LKP] [lkp] [xfs] 68a9f5e700: aim7.jobs-per-min -13.6% regression

2016-08-15 Thread Dave Chinner
On Sun, Aug 14, 2016 at 10:12:20PM -0700, Linus Torvalds wrote: > On Aug 14, 2016 10:00 PM, "Dave Chinner" wrote: > > > > > What does it say if you annotate that _raw_spin_unlock_irqrestore() > function? > > > >¿ > >¿Disassembly of section load0: > >¿ > >¿

Re: [PATCH v2 1/5] clk: qcom: ipq4019: Added the clock nodes and operations for pll

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > The current ipq4019 clock driver registered the PLL clocks and > dividers as fixed clock. These fixed clock needs to be removed > from driver probe function and same need to be registered with > clock framework. These PLL clocks should be programmed only > once and

Re: [PATCH v2 1/5] clk: qcom: ipq4019: Added the clock nodes and operations for pll

2016-08-15 Thread Stephen Boyd
On 06/21, Abhishek Sahu wrote: > The current ipq4019 clock driver registered the PLL clocks and > dividers as fixed clock. These fixed clock needs to be removed > from driver probe function and same need to be registered with > clock framework. These PLL clocks should be programmed only > once and

Re: [PATCH] time,virt: resync steal time when guest & host lose sync

2016-08-15 Thread Wanpeng Li
2016-08-15 23:00 GMT+08:00 Rik van Riel : > On Mon, 2016-08-15 at 16:53 +0800, Wanpeng Li wrote: >> 2016-08-12 23:58 GMT+08:00 Rik van Riel : >> [...] >> > Wanpeng, does the patch below work for you? >> >> It will break steal time for full dynticks guest, and

Re: [PATCH] time,virt: resync steal time when guest & host lose sync

2016-08-15 Thread Wanpeng Li
2016-08-15 23:00 GMT+08:00 Rik van Riel : > On Mon, 2016-08-15 at 16:53 +0800, Wanpeng Li wrote: >> 2016-08-12 23:58 GMT+08:00 Rik van Riel : >> [...] >> > Wanpeng, does the patch below work for you? >> >> It will break steal time for full dynticks guest, and there is a >> calltrace of

Re: [PATCH v2 2/2] cpufreq / sched: Pass runqueue pointer to cpufreq_update_util()

2016-08-15 Thread Steve Muckle
LGTM On Fri, Aug 12, 2016 at 02:06:44AM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > All of the callers of cpufreq_update_util() pass rq_clock(rq) to it > as the time argument and some of them check whether or not cpu_of(rq) > is equal to

Re: [PATCH] iio: magnetometer: mag3110: claim direct mode during raw reads

2016-08-15 Thread Alison Schofield
On Mon, Aug 15, 2016 at 05:04:24PM +0100, Jonathan Cameron wrote: > On 01/08/16 16:48, Alison Schofield wrote: > > Driver was checking for direct mode but not locking it. Use > > claim/release helper functions to guarantee the device stays > > in direct mode during raw reads. > > > >

Re: [PATCH v2 2/2] cpufreq / sched: Pass runqueue pointer to cpufreq_update_util()

2016-08-15 Thread Steve Muckle
LGTM On Fri, Aug 12, 2016 at 02:06:44AM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > All of the callers of cpufreq_update_util() pass rq_clock(rq) to it > as the time argument and some of them check whether or not cpu_of(rq) > is equal to smp_processor_id() before calling it,

Re: [PATCH] iio: magnetometer: mag3110: claim direct mode during raw reads

2016-08-15 Thread Alison Schofield
On Mon, Aug 15, 2016 at 05:04:24PM +0100, Jonathan Cameron wrote: > On 01/08/16 16:48, Alison Schofield wrote: > > Driver was checking for direct mode but not locking it. Use > > claim/release helper functions to guarantee the device stays > > in direct mode during raw reads. > > > >

Re: [PATCH v2 1/2] cpufreq / sched: Pass flags to cpufreq_update_util()

2016-08-15 Thread Steve Muckle
LGTM On Fri, Aug 12, 2016 at 02:04:42AM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > It is useful to know the reason why cpufreq_update_util() has just > been called and that can be passed as flags to cpufreq_update_util() > and to the ->func()

Re: [PATCH v2 1/2] cpufreq / sched: Pass flags to cpufreq_update_util()

2016-08-15 Thread Steve Muckle
LGTM On Fri, Aug 12, 2016 at 02:04:42AM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > It is useful to know the reason why cpufreq_update_util() has just > been called and that can be passed as flags to cpufreq_update_util() > and to the ->func() callback in struct

Re: [PATCH 1/2] clk: fixed-factor: Remove export symbol on setup function

2016-08-15 Thread Stephen Boyd
On 08/12, Stephen Boyd wrote: > This function is marked __init, so it can't possibly need to be > exported to modules. Remove the marking. > > Cc: Gregory CLEMENT > Signed-off-by: Stephen Boyd > --- Applied to clk-next -- Qualcomm

Re: [PATCH 1/2] clk: fixed-factor: Remove export symbol on setup function

2016-08-15 Thread Stephen Boyd
On 08/12, Stephen Boyd wrote: > This function is marked __init, so it can't possibly need to be > exported to modules. Remove the marking. > > Cc: Gregory CLEMENT > Signed-off-by: Stephen Boyd > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a

Re: [PATCH 2/2] clk: fixed-rate: Remove export symbol on setup function

2016-08-15 Thread Stephen Boyd
On 08/12, Stephen Boyd wrote: > This function is only called by builtin code, but we always > exported it and had marked it as __init before commit > e4eda8e0654c (clk: remove exported function from __init section, > 2013-01-06) removed that marking. Given that it isn't used by > modules, lets

Re: [PATCH 2/2] clk: fixed-rate: Remove export symbol on setup function

2016-08-15 Thread Stephen Boyd
On 08/12, Stephen Boyd wrote: > This function is only called by builtin code, but we always > exported it and had marked it as __init before commit > e4eda8e0654c (clk: remove exported function from __init section, > 2013-01-06) removed that marking. Given that it isn't used by > modules, lets

Re: Building a subdirectory ignores parent subdir-ccflags

2016-08-15 Thread Michal Marek
Dne 15.8.2016 v 23:29 Joe Perches napsal(a): > On Mon, 2016-08-15 at 14:14 -0700, Joe Perches wrote: >> On Mon, 2016-08-15 at 23:04 +0200, Greg Kroah-Hartman wrote: >>> On Mon, Aug 15, 2016 at 12:33:23PM -0700, Joe Perches wrote: Start to rationalize include paths in source code files. >> []

Re: Building a subdirectory ignores parent subdir-ccflags

2016-08-15 Thread Michal Marek
Dne 15.8.2016 v 23:29 Joe Perches napsal(a): > On Mon, 2016-08-15 at 14:14 -0700, Joe Perches wrote: >> On Mon, 2016-08-15 at 23:04 +0200, Greg Kroah-Hartman wrote: >>> On Mon, Aug 15, 2016 at 12:33:23PM -0700, Joe Perches wrote: Start to rationalize include paths in source code files. >> []

Re: [PATCH 3.2 44/94] wext: Fix 32 bit iwpriv compatibility issue with 64 bit Kernel

2016-08-15 Thread Ben Hutchings
On Mon, 2016-08-15 at 07:51 +0200, Johannes Berg wrote: > On Sat, 2016-08-13 at 17:42 +, Ben Hutchings wrote: > > > > 3.2.82-rc1 review patch.  If anyone has any objections, please let me > > know. > > > > -- > > > > > > From: Prasun Maiti > > > >

Re: [PATCH 3.2 44/94] wext: Fix 32 bit iwpriv compatibility issue with 64 bit Kernel

2016-08-15 Thread Ben Hutchings
On Mon, 2016-08-15 at 07:51 +0200, Johannes Berg wrote: > On Sat, 2016-08-13 at 17:42 +, Ben Hutchings wrote: > > > > 3.2.82-rc1 review patch.  If anyone has any objections, please let me > > know. > > > > -- > > > > > > From: Prasun Maiti > > > > commit

[PATCH 07/11] perf symbols: Fix annotation of objects with debuginfo files

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Anton Blanchard Commit 73cdf0c6ea9c ("perf symbols: Record text offset in dso to calculate objdump address") started storing the offset of the text section for all DSOs: if (elf_section_by_name(elf, , , ".text", NULL)) dso->text_offset =

[PATCH 07/11] perf symbols: Fix annotation of objects with debuginfo files

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Anton Blanchard Commit 73cdf0c6ea9c ("perf symbols: Record text offset in dso to calculate objdump address") started storing the offset of the text section for all DSOs: if (elf_section_by_name(elf, , , ".text", NULL)) dso->text_offset = tshdr.sh_addr -

[PATCH 04/11] perf jitdump: Add the right header to get the major()/minor() definitions

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Noticed on Fedora Rawhide: $ gcc --version gcc (GCC) 6.1.1 20160721 (Red Hat 6.1.1-4) $ rpm -q glibc glibc-2.24.90-1.fc26.x86_64 $ CC /tmp/build/perf/util/jitdump.o util/jitdump.c: In function 'jit_repipe_code_load':

[PATCH 04/11] perf jitdump: Add the right header to get the major()/minor() definitions

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Noticed on Fedora Rawhide: $ gcc --version gcc (GCC) 6.1.1 20160721 (Red Hat 6.1.1-4) $ rpm -q glibc glibc-2.24.90-1.fc26.x86_64 $ CC /tmp/build/perf/util/jitdump.o util/jitdump.c: In function 'jit_repipe_code_load': util/jitdump.c:428:2:

[PATCH 02/11] perf tools mem: Fix -t store option for record command

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Michael reported 'perf mem -t store record' being broken. The reason is latest rework of this area: commit acbe613e0c03 ("perf tools: Add monitored events array") We don't mark perf_mem_events store record when -t store option is specified. Committer

[PATCH 02/11] perf tools mem: Fix -t store option for record command

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Michael reported 'perf mem -t store record' being broken. The reason is latest rework of this area: commit acbe613e0c03 ("perf tools: Add monitored events array") We don't mark perf_mem_events store record when -t store option is specified. Committer notes: Before: #

[PATCH 11/11] perf intel-pt: Fix occasional decoding errors when tracing system-wide

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter In order to successfully decode Intel PT traces, context switch events are needed from the moment the trace starts. Currently that is ensured by using the 'immediate' flag which enables the switch event when it is opened. However, since commit

[PATCH 11/11] perf intel-pt: Fix occasional decoding errors when tracing system-wide

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter In order to successfully decode Intel PT traces, context switch events are needed from the moment the trace starts. Currently that is ensured by using the 'immediate' flag which enables the switch event when it is opened. However, since commit 86c2786994bd ("perf intel-pt:

[PATCH 03/11] perf ppc64le: Fix build failure when libelf is not present

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Ravi Bangoria arch__post_process_probe_trace_events() calls get_target_map() to prepare symbol table. get_target_map() is defined inside util/probe-event.c. probe-event.c will only get included in perf binary if CONFIG_LIBELF is set. Hence

[PATCH 10/11] tools: Sync kvm related header files for arm64 and s390

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo >From a quick look nothing stands out as requiring changes to kvm tools such as tools/perf/arch/s390/util/kvm-stat.c. Silences these header checking warnings: $ make -C tools/perf make: Entering directory '/home/acme/git/linux/tools/perf'

[PATCH 08/11] perf probe: Check for dup and fdopen failures

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Colin Ian King dup and fdopen can potentially fail, so add some extra error handling checks rather than assuming they always work. Signed-off-by: Colin King Acked-by: Masami Hiramatsu Cc: Alexander Shishkin

[PATCH 03/11] perf ppc64le: Fix build failure when libelf is not present

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Ravi Bangoria arch__post_process_probe_trace_events() calls get_target_map() to prepare symbol table. get_target_map() is defined inside util/probe-event.c. probe-event.c will only get included in perf binary if CONFIG_LIBELF is set. Hence arch__post_process_probe_trace_events() needs to

[PATCH 10/11] tools: Sync kvm related header files for arm64 and s390

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo >From a quick look nothing stands out as requiring changes to kvm tools such as tools/perf/arch/s390/util/kvm-stat.c. Silences these header checking warnings: $ make -C tools/perf make: Entering directory '/home/acme/git/linux/tools/perf' BUILD: Doing

[PATCH 08/11] perf probe: Check for dup and fdopen failures

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Colin Ian King dup and fdopen can potentially fail, so add some extra error handling checks rather than assuming they always work. Signed-off-by: Colin King Acked-by: Masami Hiramatsu Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Wang Nan Link:

[PATCH 05/11] perf script: Show proper message when failed list scripts

2016-08-15 Thread Arnaldo Carvalho de Melo
From: He Kuang Perf shows the usage message when perf scripts folder failed to open, which misleads users to let them think the command is being mistyped. This patch shows a proper message and guides users to check the PERF_EXEC_PATH environment variable in that case.

[PATCH 06/11] perf script: Don't disable use_callchain if input is pipe

2016-08-15 Thread Arnaldo Carvalho de Melo
From: He Kuang Because perf data from pipe do not have a header with evsel attr, we should not check that and disable symbol_conf.use_callchain. Otherwise, perf script won't show callchains even if the data stream contains callchain. Before: $ perf record -g -o - uname

[PATCH 05/11] perf script: Show proper message when failed list scripts

2016-08-15 Thread Arnaldo Carvalho de Melo
From: He Kuang Perf shows the usage message when perf scripts folder failed to open, which misleads users to let them think the command is being mistyped. This patch shows a proper message and guides users to check the PERF_EXEC_PATH environment variable in that case. Before: $ perf script

[PATCH 06/11] perf script: Don't disable use_callchain if input is pipe

2016-08-15 Thread Arnaldo Carvalho de Melo
From: He Kuang Because perf data from pipe do not have a header with evsel attr, we should not check that and disable symbol_conf.use_callchain. Otherwise, perf script won't show callchains even if the data stream contains callchain. Before: $ perf record -g -o - uname |perf script Linux

[PATCH 01/11] perf intel-pt: Fix ip compression

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter The June 2015 Intel SDM introduced IP Compression types 4 and 6. Refer to section 36.4.2.2 Target IP (TIP) Packet - IP Compression. Existing Intel PT packet decoder did not support type 4, and got type 6 wrong. Because type 3 and type 4 have the

[GIT PULL 00/11] perf/urgent fixes

2016-08-15 Thread Arnaldo Carvalho de Melo
Hi Ingo, Please consider pulling, - Arnaldo Build stats: # time dm 1 70.159253018 alpine:3.4: Ok 2 27.099391445 android-ndk:r12b-arm: Ok 3 75.359247352 archlinux:latest: Ok 4 24.340381467 centos:5: Ok 5 35.444981358 centos:6: Ok 6 40.638249015 centos:7: Ok 7 39.903273551

[PATCH 01/11] perf intel-pt: Fix ip compression

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter The June 2015 Intel SDM introduced IP Compression types 4 and 6. Refer to section 36.4.2.2 Target IP (TIP) Packet - IP Compression. Existing Intel PT packet decoder did not support type 4, and got type 6 wrong. Because type 3 and type 4 have the same number of bytes, the

[GIT PULL 00/11] perf/urgent fixes

2016-08-15 Thread Arnaldo Carvalho de Melo
Hi Ingo, Please consider pulling, - Arnaldo Build stats: # time dm 1 70.159253018 alpine:3.4: Ok 2 27.099391445 android-ndk:r12b-arm: Ok 3 75.359247352 archlinux:latest: Ok 4 24.340381467 centos:5: Ok 5 35.444981358 centos:6: Ok 6 40.638249015 centos:7: Ok 7 39.903273551

[PATCH 09/11] perf probe: Release resources on error when handling exit paths

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Cc: Adrian Hunter Cc: Colin King Cc: David Ahern Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Namhyung Kim Cc: Wang

[PATCH 09/11] perf probe: Release resources on error when handling exit paths

2016-08-15 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Cc: Adrian Hunter Cc: Colin King Cc: David Ahern Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Namhyung Kim Cc: Wang Nan Link: http://lkml.kernel.org/n/tip-zh2j4iqimralugke5qq7d...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo ---

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-15 Thread Sebastian Reichel
Hi, On Fri, Jul 29, 2016 at 05:49:26PM +0200, Benjamin Tissoires wrote: > MSHW0011 replaces the battery firmware by using ACPI operation regions. > The values have been obtained by reverse engineering, and are subject to > errors. Looks like it works on overall pretty well. > > I couldn't manage

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-15 Thread Sebastian Reichel
Hi, On Fri, Jul 29, 2016 at 05:49:26PM +0200, Benjamin Tissoires wrote: > MSHW0011 replaces the battery firmware by using ACPI operation regions. > The values have been obtained by reverse engineering, and are subject to > errors. Looks like it works on overall pretty well. > > I couldn't manage

Re: [PATCH] Map in physical addresses in efi_map_region_fixed

2016-08-15 Thread H. Peter Anvin
On August 15, 2016 11:47:31 AM PDT, Alex Thorlton wrote: >On Mon, Aug 15, 2016 at 05:07:09PM +0200, Borislav Petkov wrote: >> On Mon, Aug 15, 2016 at 01:42:58PM +0100, Matt Fleming wrote: >> > (Cc'ing Boris and Dave) >> > >> > On Fri, 05 Aug, at 06:59:35PM, Alex Thorlton

Re: [PATCH] Map in physical addresses in efi_map_region_fixed

2016-08-15 Thread H. Peter Anvin
On August 15, 2016 11:47:31 AM PDT, Alex Thorlton wrote: >On Mon, Aug 15, 2016 at 05:07:09PM +0200, Borislav Petkov wrote: >> On Mon, Aug 15, 2016 at 01:42:58PM +0100, Matt Fleming wrote: >> > (Cc'ing Boris and Dave) >> > >> > On Fri, 05 Aug, at 06:59:35PM, Alex Thorlton wrote: >> > > This is a

Re: [BUGFIX PATCH 2/2] brcmfmac: Change vif_event_lock to spinlock

2016-08-15 Thread Arend Van Spriel
On 15-8-2016 11:41, Masami Hiramatsu wrote: > Change vif_event_lock to spinlock from mutex, since this lock is > used in wait_event_timeout() via vif_event_equals(). This caused > a warning report as below. > > As far as I can see, this lock protects regions where updating > structure members,

Re: [BUGFIX PATCH 2/2] brcmfmac: Change vif_event_lock to spinlock

2016-08-15 Thread Arend Van Spriel
On 15-8-2016 11:41, Masami Hiramatsu wrote: > Change vif_event_lock to spinlock from mutex, since this lock is > used in wait_event_timeout() via vif_event_equals(). This caused > a warning report as below. > > As far as I can see, this lock protects regions where updating > structure members,

[PATCH] clk: Return errors from clk providers in __of_clk_get_from_provider()

2016-08-15 Thread Stephen Boyd
Before commit 0861e5b8cf80 (clk: Add clk_hw OF clk providers, 2016-02-05) __of_clk_get_from_provider() would return an error pointer of the provider's choosing if there was a provider registered and EPROBE_DEFER otherwise. After that commit, it would return EPROBE_DEFER regardless of whether or

[PATCH] clk: Return errors from clk providers in __of_clk_get_from_provider()

2016-08-15 Thread Stephen Boyd
Before commit 0861e5b8cf80 (clk: Add clk_hw OF clk providers, 2016-02-05) __of_clk_get_from_provider() would return an error pointer of the provider's choosing if there was a provider registered and EPROBE_DEFER otherwise. After that commit, it would return EPROBE_DEFER regardless of whether or

Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

2016-08-15 Thread Arend Van Spriel
On 15-8-2016 13:52, Rafał Miłecki wrote: > On 15 August 2016 at 12:57, Kalle Valo wrote: >> Rafał Miłecki writes: >> Signed-off-by: Masami Hiramatsu >>> >>> Fixes: a63b09872c1d ("brcmfmac: delete interface directly in code that

Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

2016-08-15 Thread Arend Van Spriel
On 15-8-2016 13:52, Rafał Miłecki wrote: > On 15 August 2016 at 12:57, Kalle Valo wrote: >> Rafał Miłecki writes: >> Signed-off-by: Masami Hiramatsu >>> >>> Fixes: a63b09872c1d ("brcmfmac: delete interface directly in code that sent >>> fw request") >>> Acked-by: Rafał Miłecki >>> >>>

Re: [PATCH 19/58] staging: lustre: llite: add md_op_data parameter to ll_get_dir_page

2016-08-15 Thread James Simmons
> > From: wang di > > > > Pass in struct md_op_data for ll_get_dir_page function. > > > > Signed-off-by: wang di > > Reviewed-on: http://review.whamcloud.com/7043 > > Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3531 > > Reviewed-by: John L.

Re: [PATCH 19/58] staging: lustre: llite: add md_op_data parameter to ll_get_dir_page

2016-08-15 Thread James Simmons
> > From: wang di > > > > Pass in struct md_op_data for ll_get_dir_page function. > > > > Signed-off-by: wang di > > Reviewed-on: http://review.whamcloud.com/7043 > > Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3531 > > Reviewed-by: John L. Hammond > > Reviewed-by: Jinshan Xiong > >

Re: [PATCH v3 1/4] hwmon: iio_hwmon: delay probing with late_initcall

2016-08-15 Thread Jonathan Cameron
On 15 August 2016 18:07:30 BST, Guenter Roeck wrote: >On Mon, Aug 15, 2016 at 04:40:21PM +0100, Jonathan Cameron wrote: >> On 26/07/16 17:04, Guenter Roeck wrote: >> > On Tue, Jul 26, 2016 at 12:00:33PM +0200, Alexander Stein wrote: >> >> On Tuesday 26 July 2016 11:33:59,

Re: [PATCH v3 1/4] hwmon: iio_hwmon: delay probing with late_initcall

2016-08-15 Thread Jonathan Cameron
On 15 August 2016 18:07:30 BST, Guenter Roeck wrote: >On Mon, Aug 15, 2016 at 04:40:21PM +0100, Jonathan Cameron wrote: >> On 26/07/16 17:04, Guenter Roeck wrote: >> > On Tue, Jul 26, 2016 at 12:00:33PM +0200, Alexander Stein wrote: >> >> On Tuesday 26 July 2016 11:33:59, Quentin Schulz wrote:

Re: [PATCH 4.7 00/41] 4.7.1-stable review

2016-08-15 Thread Kevin Hilman
Greg Kroah-Hartman writes: > On Sun, Aug 14, 2016 at 06:44:02PM -0700, kernelci.org bot wrote: >> stable-rc boot: 468 boots: 1 failed, 432 passed with 33 offline, 2 conflicts >> (v4.7-41-g57464803070d) >> >> Full Boot Summary: >>

Re: [PATCH 4.7 00/41] 4.7.1-stable review

2016-08-15 Thread Kevin Hilman
Greg Kroah-Hartman writes: > On Sun, Aug 14, 2016 at 06:44:02PM -0700, kernelci.org bot wrote: >> stable-rc boot: 468 boots: 1 failed, 432 passed with 33 offline, 2 conflicts >> (v4.7-41-g57464803070d) >> >> Full Boot Summary: >>

Re: [RFC PATCH v5 3/3] net: phy: Add gmiitorgmii converter support

2016-08-15 Thread Andrew Lunn
> Signed-off-by: Kedareswara rao Appana > --- > Thanks a lot Andrew for your inputs. > Changes for v5: > --> Fixed return values in the probe as suggested by punnaiah. > --> Added a mask for the converter speed as suggested by punnaiah. > +/* Xilinx GMII2RGMII Converter driver

Re: [RFC PATCH v5 3/3] net: phy: Add gmiitorgmii converter support

2016-08-15 Thread Andrew Lunn
> Signed-off-by: Kedareswara rao Appana > --- > Thanks a lot Andrew for your inputs. > Changes for v5: > --> Fixed return values in the probe as suggested by punnaiah. > --> Added a mask for the converter speed as suggested by punnaiah. > +/* Xilinx GMII2RGMII Converter driver > + * > + *

Building a subdirectory ignores parent subdir-ccflags (was: Re: [PATCH 1/2] staging: lustre: Add include path to Makefile)

2016-08-15 Thread Joe Perches
On Mon, 2016-08-15 at 14:14 -0700, Joe Perches wrote: > On Mon, 2016-08-15 at 23:04 +0200, Greg Kroah-Hartman wrote: > > On Mon, Aug 15, 2016 at 12:33:23PM -0700, Joe Perches wrote: > > > Start to rationalize include paths in source code files. > [] > > > diff --git

Building a subdirectory ignores parent subdir-ccflags (was: Re: [PATCH 1/2] staging: lustre: Add include path to Makefile)

2016-08-15 Thread Joe Perches
On Mon, 2016-08-15 at 14:14 -0700, Joe Perches wrote: > On Mon, 2016-08-15 at 23:04 +0200, Greg Kroah-Hartman wrote: > > On Mon, Aug 15, 2016 at 12:33:23PM -0700, Joe Perches wrote: > > > Start to rationalize include paths in source code files. > [] > > > diff --git

Re: [PATCH v3 6/6] clk: mvebu: Add the peripheral clock driver for Armada 3700

2016-08-15 Thread Stephen Boyd
On 07/19, Gregory CLEMENT wrote: > + > +static const struct clk_ops clk_double_div_ops = { > + .recalc_rate = clk_double_div_recalc_rate, > +}; > + > +static const struct of_device_id armada_3700_periph_clock_of_match[] = { > + { .compatible = "marvell,armada-3700-periph-clock-nb", > +

Re: [PATCH v3 5/6] dt-bindings: clock: add DT binding for the peripheral clocks on Armada 3700

2016-08-15 Thread Stephen Boyd
On 07/19, Gregory CLEMENT wrote: > This commit adds the DT binding documentation for the peripheral clocks > used in the Marvell Armada 3700 SoCs. > > Signed-off-by: Gregory CLEMENT > Acked-by: Rob Herring > --- Applied to clk-next --

Re: [PATCH v3 5/6] dt-bindings: clock: add DT binding for the peripheral clocks on Armada 3700

2016-08-15 Thread Stephen Boyd
On 07/19, Gregory CLEMENT wrote: > This commit adds the DT binding documentation for the peripheral clocks > used in the Marvell Armada 3700 SoCs. > > Signed-off-by: Gregory CLEMENT > Acked-by: Rob Herring > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code

Re: [PATCH v3 6/6] clk: mvebu: Add the peripheral clock driver for Armada 3700

2016-08-15 Thread Stephen Boyd
On 07/19, Gregory CLEMENT wrote: > + > +static const struct clk_ops clk_double_div_ops = { > + .recalc_rate = clk_double_div_recalc_rate, > +}; > + > +static const struct of_device_id armada_3700_periph_clock_of_match[] = { > + { .compatible = "marvell,armada-3700-periph-clock-nb", > +

Re: [LKP] [lkp] [xfs] 68a9f5e700: aim7.jobs-per-min -13.6% regression

2016-08-15 Thread Dave Chinner
On Mon, Aug 15, 2016 at 10:14:55PM +0800, Fengguang Wu wrote: > Hi Christoph, > > On Sun, Aug 14, 2016 at 06:17:24PM +0200, Christoph Hellwig wrote: > >Snipping the long contest: > > > >I think there are three observations here: > > > >(1) removing the mark_page_accessed (which is the only

Re: [LKP] [lkp] [xfs] 68a9f5e700: aim7.jobs-per-min -13.6% regression

2016-08-15 Thread Dave Chinner
On Mon, Aug 15, 2016 at 10:14:55PM +0800, Fengguang Wu wrote: > Hi Christoph, > > On Sun, Aug 14, 2016 at 06:17:24PM +0200, Christoph Hellwig wrote: > >Snipping the long contest: > > > >I think there are three observations here: > > > >(1) removing the mark_page_accessed (which is the only

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