Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

On (03/16/17 14:51), Minchan Kim wrote: [..] > > > @@ -1414,7 +1414,7 @@ static bool try_to_unmap_one(struct page *page, > > > struct vm_area_struct *vma, > > >*/ > > > if (unlikely(PageSwapBacked(page) != > > > PageSwapCache(page))) { > > >

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

On (03/16/17 14:51), Minchan Kim wrote: [..] > > > @@ -1414,7 +1414,7 @@ static bool try_to_unmap_one(struct page *page, > > > struct vm_area_struct *vma, > > >*/ > > > if (unlikely(PageSwapBacked(page) != > > > PageSwapCache(page))) { > > >

linux-next: tty: BUG: spinlock bad magic on CPU#0, init/1

Hello, We run CRIU tests for linux-next and here is a new bug in the kernel log [2.431229] Freeing unused kernel memory: 1356K [2.436371] Freeing unused kernel memory: 168K [2.522236] BUG: spinlock bad magic on CPU#0, init/1 [2.527487] lock: 0x94915477fd88, .magic: ,

linux-next: tty: BUG: spinlock bad magic on CPU#0, init/1

Hello, We run CRIU tests for linux-next and here is a new bug in the kernel log [2.431229] Freeing unused kernel memory: 1356K [2.436371] Freeing unused kernel memory: 168K [2.522236] BUG: spinlock bad magic on CPU#0, init/1 [2.527487] lock: 0x94915477fd88, .magic: ,

Re: [PATCH v3 4/7] xen/9pfs: connect to the backend

On 15/03/17 19:44, Stefano Stabellini wrote: > On Wed, 15 Mar 2017, Juergen Gross wrote: >> On 14/03/17 22:22, Stefano Stabellini wrote: >>> Hi Juergen, >>> >>> thank you for the review! >>> >>> On Tue, 14 Mar 2017, Juergen Gross wrote: On 14/03/17 00:50, Stefano Stabellini wrote: >

Re: [PATCH v2 1/6] powerpc/perf: Define big-endian version of perf_mem_data_src

On Wednesday 15 March 2017 05:53 PM, Peter Zijlstra wrote: On Wed, Mar 15, 2017 at 05:20:15PM +1100, Michael Ellerman wrote: I see no implementation; so why are you poking at it. Maddy has posted an implementation of the kernel part for powerpc in patch 2 of this series, but maybe you're

Re: [PATCH v2 1/6] powerpc/perf: Define big-endian version of perf_mem_data_src

On Wednesday 15 March 2017 05:53 PM, Peter Zijlstra wrote: On Wed, Mar 15, 2017 at 05:20:15PM +1100, Michael Ellerman wrote: I see no implementation; so why are you poking at it. Maddy has posted an implementation of the kernel part for powerpc in patch 2 of this series, but maybe you're

Re: [PATCH v3 4/7] xen/9pfs: connect to the backend

On 15/03/17 19:44, Stefano Stabellini wrote: > On Wed, 15 Mar 2017, Juergen Gross wrote: >> On 14/03/17 22:22, Stefano Stabellini wrote: >>> Hi Juergen, >>> >>> thank you for the review! >>> >>> On Tue, 14 Mar 2017, Juergen Gross wrote: On 14/03/17 00:50, Stefano Stabellini wrote: >

Re: change uprobe_events default ? Was: [PATCH] perf: Rename CONFIG_[UK]PROBE_EVENT to CONFIG_[UK]PROBE_EVENTS

On 2017/3/16 9:06, Arnaldo Carvalho de Melo wrote: Added more people to the CC list. Em Wed, Mar 15, 2017 at 05:58:19PM -0700, Alexei Starovoitov escreveu: On Thu, Feb 16, 2017 at 05:00:50PM +1100, Anton Blanchard wrote: We have uses of CONFIG_UPROBE_EVENT and CONFIG_KPROBE_EVENT as well as

Re: change uprobe_events default ? Was: [PATCH] perf: Rename CONFIG_[UK]PROBE_EVENT to CONFIG_[UK]PROBE_EVENTS

On 2017/3/16 9:06, Arnaldo Carvalho de Melo wrote: Added more people to the CC list. Em Wed, Mar 15, 2017 at 05:58:19PM -0700, Alexei Starovoitov escreveu: On Thu, Feb 16, 2017 at 05:00:50PM +1100, Anton Blanchard wrote: We have uses of CONFIG_UPROBE_EVENT and CONFIG_KPROBE_EVENT as well as

Re: [PATCH] ARM: dts: mvebu: linksys: enable buffer manager support

On Wed, 15 Mar 2017 20:17:35 +0100 Gregory CLEMENT wrote: > Hi Ralph, > > On mer., mars 08 2017, Ralph Sennhauser > wrote: > > > > > @@ -88,6 +89,9 @@ > > ethernet@7 { > >

Re: [PATCH v2 1/6] powerpc/perf: Define big-endian version of perf_mem_data_src

On Wednesday 15 March 2017 11:50 AM, Michael Ellerman wrote: Hi Peter, Peter Zijlstra writes: On Tue, Mar 14, 2017 at 02:31:51PM +0530, Madhavan Srinivasan wrote: Huh? PPC hasn't yet implemented this? Then why are you fixing it? yes, PPC hasn't implemented this

Re: [PATCH] ARM: dts: mvebu: linksys: enable buffer manager support

On Wed, 15 Mar 2017 20:17:35 +0100 Gregory CLEMENT wrote: > Hi Ralph, > > On mer., mars 08 2017, Ralph Sennhauser > wrote: > > > > > @@ -88,6 +89,9 @@ > > ethernet@7 { > > status = "okay"; > > phy-mode =

Re: [PATCH v2 1/6] powerpc/perf: Define big-endian version of perf_mem_data_src

On Wednesday 15 March 2017 11:50 AM, Michael Ellerman wrote: Hi Peter, Peter Zijlstra writes: On Tue, Mar 14, 2017 at 02:31:51PM +0530, Madhavan Srinivasan wrote: Huh? PPC hasn't yet implemented this? Then why are you fixing it? yes, PPC hasn't implemented this (until now). until now

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

On (03/16/17 14:33), Minchan Kim wrote: [..] > "There is no user for it" > > I was liar so need to be a honest guy. ha-ha-ha. I didn't say that :) [..] > @@ -1414,7 +1414,7 @@ static bool try_to_unmap_one(struct page *page, struct > vm_area_struct *vma, >*/ >

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

On (03/16/17 14:33), Minchan Kim wrote: [..] > "There is no user for it" > > I was liar so need to be a honest guy. ha-ha-ha. I didn't say that :) [..] > @@ -1414,7 +1414,7 @@ static bool try_to_unmap_one(struct page *page, struct > vm_area_struct *vma, >*/ >

Re: perf: race with automatic rdpmc() disabling

On Wed, 15 Mar 2017, Andy Lutomirski wrote: > Can you give this a try: > > https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git/commit/?h=x86/fixes=9edb8154863ba1a7f6f1f15ffe6aecf3cf32bf21 > > (The link doesn't work yet but it should in a minute or two.) I've tested it and I am

Re: perf: race with automatic rdpmc() disabling

On Wed, 15 Mar 2017, Andy Lutomirski wrote: > Can you give this a try: > > https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git/commit/?h=x86/fixes=9edb8154863ba1a7f6f1f15ffe6aecf3cf32bf21 > > (The link doesn't work yet but it should in a minute or two.) I've tested it and I am

Re: [PATCH 2/2] usb; dwc3: of-simple: Add support to get resets for the device

Hi, On 03/15/2017 04:15 PM, Philipp Zabel wrote: On Wed, 2017-02-22 at 10:54 +0530, Vivek Gautam wrote: Add support to get a list of resets available for the device. These resets must be kept de-asserted until the device is in use. Cc: Felipe Balbi Signed-off-by: Vivek

Re: [PATCH 2/2] usb; dwc3: of-simple: Add support to get resets for the device

Hi, On 03/15/2017 04:15 PM, Philipp Zabel wrote: On Wed, 2017-02-22 at 10:54 +0530, Vivek Gautam wrote: Add support to get a list of resets available for the device. These resets must be kept de-asserted until the device is in use. Cc: Felipe Balbi Signed-off-by: Vivek Gautam --- Based on

Re: change uprobe_events default ? Was: [PATCH] perf: Rename CONFIG_[UK]PROBE_EVENT to CONFIG_[UK]PROBE_EVENTS

> Added more people to the CC list. > > Em Wed, Mar 15, 2017 at 05:58:19PM -0700, Alexei Starovoitov escreveu: > > On Thu, Feb 16, 2017 at 05:00:50PM +1100, Anton Blanchard wrote: > > > We have uses of CONFIG_UPROBE_EVENT and CONFIG_KPROBE_EVENT as > > > well as CONFIG_UPROBE_EVENTS and

Re: change uprobe_events default ? Was: [PATCH] perf: Rename CONFIG_[UK]PROBE_EVENT to CONFIG_[UK]PROBE_EVENTS

> Added more people to the CC list. > > Em Wed, Mar 15, 2017 at 05:58:19PM -0700, Alexei Starovoitov escreveu: > > On Thu, Feb 16, 2017 at 05:00:50PM +1100, Anton Blanchard wrote: > > > We have uses of CONFIG_UPROBE_EVENT and CONFIG_KPROBE_EVENT as > > > well as CONFIG_UPROBE_EVENTS and

[PATCH 03/17] thermal: cpu_cooling: Replace cpufreq_device with cpufreq_dev

Objects of "struct cpufreq_cooling_device" are named a bit inconsistently. Lets use cpufreq_dev everywhere. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 133 +- 1 file changed, 66 insertions(+), 67 deletions(-)

[PATCH 03/17] thermal: cpu_cooling: Replace cpufreq_device with cpufreq_dev

Objects of "struct cpufreq_cooling_device" are named a bit inconsistently. Lets use cpufreq_dev everywhere. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 133 +- 1 file changed, 66 insertions(+), 67 deletions(-) diff --git

[PATCH 10/17] thermal: cpu_cooling: OPPs are registered for all CPUs

The OPPs are registered for all CPUs of a cpufreq policy now and we don't need to run the loop in build_dyn_power_table(). Just check for the policy->cpu and we should be fine. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 26 +++---

[PATCH 07/17] thermal: cpu_cooling: use cpufreq_policy to register cooling device

The CPU cooling driver uses the cpufreq policy, to get clip_cpus, the frequency table, etc. Most of the callers of CPU cooling driver's registration routines have the cpufreq policy with them, but they only pass the policy->related_cpus cpumask. The __cpufreq_cooling_register() routine then gets

[PATCH 07/17] thermal: cpu_cooling: use cpufreq_policy to register cooling device

The CPU cooling driver uses the cpufreq policy, to get clip_cpus, the frequency table, etc. Most of the callers of CPU cooling driver's registration routines have the cpufreq policy with them, but they only pass the policy->related_cpus cpumask. The __cpufreq_cooling_register() routine then gets

[PATCH 10/17] thermal: cpu_cooling: OPPs are registered for all CPUs

The OPPs are registered for all CPUs of a cpufreq policy now and we don't need to run the loop in build_dyn_power_table(). Just check for the policy->cpu and we should be fine. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 26 +++--- 1 file changed, 11

Re: ZONE_NORMAL vs. ZONE_MOVABLE

On Wed, Mar 15, 2017 at 05:37:29PM +0100, Andrea Arcangeli wrote: > On Wed, Mar 15, 2017 at 02:11:40PM +0100, Michal Hocko wrote: > > OK, I see now. I am afraid there is quite a lot of code which expects > > that zones do not overlap. We can have holes in zones but not different > > zones

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

Hey, Sergey, On Thu, Mar 16, 2017 at 01:40:23PM +0900, Sergey Senozhatsky wrote: > Hello, > > > On (03/15/17 14:24), Minchan Kim wrote: > > There is no user for it. Remove it. > > > > there is one. > > mm/rmap.c > > try_to_unmap_one() > ... > if (unlikely(PageSwapBacked(page) !=

Re: ZONE_NORMAL vs. ZONE_MOVABLE

On Wed, Mar 15, 2017 at 05:37:29PM +0100, Andrea Arcangeli wrote: > On Wed, Mar 15, 2017 at 02:11:40PM +0100, Michal Hocko wrote: > > OK, I see now. I am afraid there is quite a lot of code which expects > > that zones do not overlap. We can have holes in zones but not different > > zones

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

Hey, Sergey, On Thu, Mar 16, 2017 at 01:40:23PM +0900, Sergey Senozhatsky wrote: > Hello, > > > On (03/15/17 14:24), Minchan Kim wrote: > > There is no user for it. Remove it. > > > > there is one. > > mm/rmap.c > > try_to_unmap_one() > ... > if (unlikely(PageSwapBacked(page) !=

[PATCH 08/17] cpufreq: create cpufreq_table_count_valid_entries()

We need such a routine at two places already, lets create one. Signed-off-by: Viresh Kumar --- drivers/cpufreq/cpufreq_stats.c | 13 - drivers/thermal/cpu_cooling.c | 22 +- include/linux/cpufreq.h | 14 ++ 3 files

[PATCH 08/17] cpufreq: create cpufreq_table_count_valid_entries()

We need such a routine at two places already, lets create one. Signed-off-by: Viresh Kumar --- drivers/cpufreq/cpufreq_stats.c | 13 - drivers/thermal/cpu_cooling.c | 22 +- include/linux/cpufreq.h | 14 ++ 3 files changed, 27 insertions(+),

[PATCH 14/17] thermal: cpu_cooling: get_level() can't fail

The frequency passed to get_level() is returned by cpu_power_to_freq() and it is guaranteed that get_level() can't fail. Get rid of error code. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 20 +--- 1 file changed, 5 insertions(+), 15

Re: [PATCH] vc04_services: Fixing coding guideline error

Any comment? On Wed, Mar 15, 2017 at 12:01 PM, Pushkar Jambhlekar wrote: > Fixing 'if' block coding style. '{' should follow 'if' for multiline block > > Signed-off-by: Pushkar Jambhlekar > --- >

[PATCH 14/17] thermal: cpu_cooling: get_level() can't fail

The frequency passed to get_level() is returned by cpu_power_to_freq() and it is guaranteed that get_level() can't fail. Get rid of error code. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 20 +--- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git

Re: [PATCH] vc04_services: Fixing coding guideline error

Any comment? On Wed, Mar 15, 2017 at 12:01 PM, Pushkar Jambhlekar wrote: > Fixing 'if' block coding style. '{' should follow 'if' for multiline block > > Signed-off-by: Pushkar Jambhlekar > --- > drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +-- > 1 file changed, 1

[PATCH 15/17] thermal: cpu_cooling: don't store cpu_dev in cpufreq_dev

'cpu_dev' is used by only one function, get_static_power(), and it wouldn't be time consuming to get the cpu device structure within it. This would help removing cpu_dev from struct cpufreq_cooling_device. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c |

[PATCH 15/17] thermal: cpu_cooling: don't store cpu_dev in cpufreq_dev

'cpu_dev' is used by only one function, get_static_power(), and it wouldn't be time consuming to get the cpu device structure within it. This would help removing cpu_dev from struct cpufreq_cooling_device. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 22 ++

[PATCH 17/17] thermal: cpu_cooling: Rearrange struct cpufreq_cooling_device

This shrinks the size of the structure on arm64 by 8 bytes by avoiding padding of 4 bytes at two places. Also add missing doc comment for freq_table Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 15 --- 1 file changed, 8 insertions(+), 7

[PATCH 13/17] thermal: cpu_cooling: create structure for idle time stats

We keep two arrays for idle time stats and allocate memory for them separately. It would be much easier to follow if we create an array of idle stats structure instead and allocate it once. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 53

[PATCH 16/17] thermal: cpu_cooling: 'freq' can't be zero in cpufreq_state2power()

The frequency table shouldn't have any zero frequency entries and so such a check isn't required. Though it would be better to make sure 'state' is within limits. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 7 --- 1 file changed, 4 insertions(+),

[PATCH 13/17] thermal: cpu_cooling: create structure for idle time stats

We keep two arrays for idle time stats and allocate memory for them separately. It would be much easier to follow if we create an array of idle stats structure instead and allocate it once. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 53

[PATCH 16/17] thermal: cpu_cooling: 'freq' can't be zero in cpufreq_state2power()

The frequency table shouldn't have any zero frequency entries and so such a check isn't required. Though it would be better to make sure 'state' is within limits. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff

[PATCH 17/17] thermal: cpu_cooling: Rearrange struct cpufreq_cooling_device

This shrinks the size of the structure on arm64 by 8 bytes by avoiding padding of 4 bytes at two places. Also add missing doc comment for freq_table Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git

[PATCH 06/17] thermal: cpu_cooling: get rid of a variable in cpufreq_set_cur_state()

'cpu' is used at only one place and there is no need to keep a separate variable for it. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c

[PATCH 12/17] thermal: cpu_cooling: merge frequency and power tables

The cpu_cooling driver keeps two tables: - freq_table: table of frequencies in descending order, built from policy->freq_table. - power_table: table of frequencies and power in ascending order, built from OPP table. If the OPPs are used for the CPU device then both these tables are actually

[PATCH 05/17] thermal: cpu_cooling: remove cpufreq_cooling_get_level()

There is only one user of cpufreq_cooling_get_level() and that already has pointer to the cpufreq_dev structure. It can directly call get_level() instead and we can get rid of cpufreq_cooling_get_level(). Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c |

[PATCH 09/17] thermal: cpu_cooling: store cpufreq policy

The cpufreq policy can be used by the cpu_cooling driver, lets store it in the cpufreq_cooling_device structure. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/thermal/cpu_cooling.c

[PATCH 06/17] thermal: cpu_cooling: get rid of a variable in cpufreq_set_cur_state()

'cpu' is used at only one place and there is no need to keep a separate variable for it. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index

[PATCH 12/17] thermal: cpu_cooling: merge frequency and power tables

The cpu_cooling driver keeps two tables: - freq_table: table of frequencies in descending order, built from policy->freq_table. - power_table: table of frequencies and power in ascending order, built from OPP table. If the OPPs are used for the CPU device then both these tables are actually

[PATCH 05/17] thermal: cpu_cooling: remove cpufreq_cooling_get_level()

There is only one user of cpufreq_cooling_get_level() and that already has pointer to the cpufreq_dev structure. It can directly call get_level() instead and we can get rid of cpufreq_cooling_get_level(). Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 33

[PATCH 09/17] thermal: cpu_cooling: store cpufreq policy

The cpufreq policy can be used by the cpu_cooling driver, lets store it in the cpufreq_cooling_device structure. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c

[PATCH 02/17] thermal: cpu_cooling: rearrange globals

Just to make it look better. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 6fd258d62e47..7ce73eee866f 100644 ---

[PATCH 11/17] thermal: cpu_cooling: get rid of 'allowed_cpus'

'allowed_cpus' is a copy of policy->related_cpus and can be replaced by it directly. At some places we are only concerned about online CPUs and policy->cpus can be used there. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 77

[PATCH 04/17] thermal: cpu_cooling: replace cool_dev with cdev

Objects of "struct thermal_cooling_device" are named a bit inconsistently. Lets use cdev everywhere. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git

[PATCH 01/17] thermal: cpu_cooling: Avoid accessing potentially freed structures

After the lock is dropped, it is possible that the cpufreq_dev gets freed before we call get_level() and that can cause kernel to crash. Drop the lock after we are done using the structure. Cc: 4.2+ Fixes: 02373d7c69b4 ("thermal: cpu_cooling: fix lockdep problems in

[PATCH 02/17] thermal: cpu_cooling: rearrange globals

Just to make it look better. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 6fd258d62e47..7ce73eee866f 100644 ---

[PATCH 11/17] thermal: cpu_cooling: get rid of 'allowed_cpus'

'allowed_cpus' is a copy of policy->related_cpus and can be replaced by it directly. At some places we are only concerned about online CPUs and policy->cpus can be used there. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 77 --- 1 file

[PATCH 04/17] thermal: cpu_cooling: replace cool_dev with cdev

Objects of "struct thermal_cooling_device" are named a bit inconsistently. Lets use cdev everywhere. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git

[PATCH 01/17] thermal: cpu_cooling: Avoid accessing potentially freed structures

After the lock is dropped, it is possible that the cpufreq_dev gets freed before we call get_level() and that can cause kernel to crash. Drop the lock after we are done using the structure. Cc: 4.2+ Fixes: 02373d7c69b4 ("thermal: cpu_cooling: fix lockdep problems in cpu_cooling")

[PATCH 00/17] thermal: cpu_cooling: improve interaction with cpufreq core

Hi Guys, The cpu_cooling driver is designed to use CPU frequency scaling to avoid high thermal states for a platform. But it wasn't glued really well with cpufreq core. This series tries to improve interactions between cpufreq core and cpu_cooling driver and does some fixes/cleanups to the

[PATCH 00/17] thermal: cpu_cooling: improve interaction with cpufreq core

Hi Guys, The cpu_cooling driver is designed to use CPU frequency scaling to avoid high thermal states for a platform. But it wasn't glued really well with cpufreq core. This series tries to improve interactions between cpufreq core and cpu_cooling driver and does some fixes/cleanups to the

[PATCH 2/2] rpmsg: Introduce Qualcomm RPM glink driver

This introduces a basic driver for communicating over "native glink" with the RPM found in Qualcomm platforms. Signed-off-by: Bjorn Andersson --- drivers/rpmsg/Kconfig |8 + drivers/rpmsg/Makefile |1 + drivers/rpmsg/qcom_glink_rpm.c | 1249

[PATCH 2/2] rpmsg: Introduce Qualcomm RPM glink driver

This introduces a basic driver for communicating over "native glink" with the RPM found in Qualcomm platforms. Signed-off-by: Bjorn Andersson --- drivers/rpmsg/Kconfig |8 + drivers/rpmsg/Makefile |1 + drivers/rpmsg/qcom_glink_rpm.c | 1249

[PATCH 1/2] soc: qcom: Add device tree binding for GLINK RPM

Add device tree binding documentation for the Qualcomm GLINK RPM, used for communication with the Resource Power Management subsystem in various Qualcomm SoCs. Signed-off-by: Bjorn Andersson --- .../devicetree/bindings/soc/qcom/qcom,glink.txt| 73

[PATCH 1/2] soc: qcom: Add device tree binding for GLINK RPM

Add device tree binding documentation for the Qualcomm GLINK RPM, used for communication with the Resource Power Management subsystem in various Qualcomm SoCs. Signed-off-by: Bjorn Andersson --- .../devicetree/bindings/soc/qcom/qcom,glink.txt| 73 ++ 1 file changed, 73

Re: [PATCH] scsi: sr: fix oob access in get_capabilities

On 2017/3/16 8:07, Martin K. Petersen wrote: > Kefeng Wang writes: > > Kefeng, > >> 'n = header_length + block_descriptor_length' could be greater than 512, >> and will lead to oob access, so enlarge transfer buffer to fix it. > > Can you share the output of

Re: [PATCH] scsi: sr: fix oob access in get_capabilities

On 2017/3/16 8:07, Martin K. Petersen wrote: > Kefeng Wang writes: > > Kefeng, > >> 'n = header_length + block_descriptor_length' could be greater than 512, >> and will lead to oob access, so enlarge transfer buffer to fix it. > > Can you share the output of sg_modes -p 0x2a /dev/srN for the

[PATCH] rpmsg: Release rpmsg devices in backends

The rpmsg devices are allocated in the backends and as such must be freed there as well. Signed-off-by: Bjorn Andersson --- drivers/rpmsg/qcom_smd.c | 11 +++ drivers/rpmsg/virtio_rpmsg_bus.c | 9 + 2 files changed, 20 insertions(+) diff

[PATCH] rpmsg: Release rpmsg devices in backends

The rpmsg devices are allocated in the backends and as such must be freed there as well. Signed-off-by: Bjorn Andersson --- drivers/rpmsg/qcom_smd.c | 11 +++ drivers/rpmsg/virtio_rpmsg_bus.c | 9 + 2 files changed, 20 insertions(+) diff --git

[mmotm] "x86/atomic: move __arch_atomic_add_unless out of line" build error

Hello, commit 4f86a82ff7df ("x86/atomic: move __arch_atomic_add_unless out of line") moved __arch_atomic_add_unless() out atomic.h and new KASAN atomic instrumentation [1] can't see it anymore In file included from ./arch/x86/include/asm/atomic.h:257:0, from

[mmotm] "x86/atomic: move __arch_atomic_add_unless out of line" build error

Hello, commit 4f86a82ff7df ("x86/atomic: move __arch_atomic_add_unless out of line") moved __arch_atomic_add_unless() out atomic.h and new KASAN atomic instrumentation [1] can't see it anymore In file included from ./arch/x86/include/asm/atomic.h:257:0, from

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

Hello, On (03/15/17 14:24), Minchan Kim wrote: > There is no user for it. Remove it. > there is one. mm/rmap.c try_to_unmap_one() ... if (unlikely(PageSwapBacked(page) != PageSwapCache(page))) { WARN_ON_ONCE(1); ret = SWAP_FAIL;

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

Hello, On (03/15/17 14:24), Minchan Kim wrote: > There is no user for it. Remove it. > there is one. mm/rmap.c try_to_unmap_one() ... if (unlikely(PageSwapBacked(page) != PageSwapCache(page))) { WARN_ON_ONCE(1); ret = SWAP_FAIL;

Re: [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS is set

"Andrew F. Davis" writes: > When CONFIG_MACINTOSH_DRIVERS is not set make will still descend into the > macintosh directory but nothing will be built. This produces unneeded > build artifacts and messages in addition to slowing the build. > Fix this here. > > Signed-off-by: Andrew

Re: [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS is set

"Andrew F. Davis" writes: > When CONFIG_MACINTOSH_DRIVERS is not set make will still descend into the > macintosh directory but nothing will be built. This produces unneeded > build artifacts and messages in addition to slowing the build. > Fix this here. > > Signed-off-by: Andrew F. Davis >

[PATCH v2] net: mvneta: support suspend and resume

Add basic support for handling suspend and resume. Signed-off-by: Jane Li --- Since v1: - add mvneta_conf_mbus_windows() and mvneta_bm_port_init() in mvneta_resume() drivers/net/ethernet/marvell/mvneta.c | 62 --- 1 file changed, 58

[PATCH v2] net: mvneta: support suspend and resume

Add basic support for handling suspend and resume. Signed-off-by: Jane Li --- Since v1: - add mvneta_conf_mbus_windows() and mvneta_bm_port_init() in mvneta_resume() drivers/net/ethernet/marvell/mvneta.c | 62 --- 1 file changed, 58 insertions(+), 4 deletions(-)

[PATCH] Staging: goldfish: use __func__ instead of embedded function names

Embedded function names are less appropriate to use when refactoring, can cause function renaming. Prefer the use of "%s", __func__ to embedded function names Signed-off-by: Mohsin Shan --- drivers/staging/goldfish/goldfish_nand.c | 16 1 file changed,

[PATCH] Staging: goldfish: use __func__ instead of embedded function names

Embedded function names are less appropriate to use when refactoring, can cause function renaming. Prefer the use of "%s", __func__ to embedded function names Signed-off-by: Mohsin Shan --- drivers/staging/goldfish/goldfish_nand.c | 16 1 file changed, 8 insertions(+), 8

Re: [PATCH 1/2] reset: Add API to count number of reset available with device

Hi Philipp, On Wed, Mar 15, 2017 at 4:10 PM, Philipp Zabel wrote: > Hi Vivek, > > On Fri, 2017-03-10 at 20:10 +0530, Vivek Gautam wrote: >> Hi Philipp, >> >> >> On Wed, Feb 22, 2017 at 10:54 AM, Vivek Gautam >> wrote: >> > Count number of

Re: [PATCH 1/2] reset: Add API to count number of reset available with device

Hi Philipp, On Wed, Mar 15, 2017 at 4:10 PM, Philipp Zabel wrote: > Hi Vivek, > > On Fri, 2017-03-10 at 20:10 +0530, Vivek Gautam wrote: >> Hi Philipp, >> >> >> On Wed, Feb 22, 2017 at 10:54 AM, Vivek Gautam >> wrote: >> > Count number of reset phandles available with the device node >> > to

Re: [PATCH v2] net: mvneta: support suspend and resume

On Thu, 16 Mar 2017 11:19:10 +0800 Jane Li wrote: > Add basic support for handling suspend and resume. > > Signed-off-by: Jane Li > --- > Since v1: > - add mvneta_conf_mbus_windows() and mvneta_bm_port_init() in mvneta_resume() > > drivers/net/ethernet/marvell/mvneta.c | 62

Re: [PATCH v2] net: mvneta: support suspend and resume

On Thu, 16 Mar 2017 11:19:10 +0800 Jane Li wrote: > Add basic support for handling suspend and resume. > > Signed-off-by: Jane Li > --- > Since v1: > - add mvneta_conf_mbus_windows() and mvneta_bm_port_init() in mvneta_resume() > > drivers/net/ethernet/marvell/mvneta.c | 62 >

Re: [PATCH v5 2/3] soc/imx: Add GPCv2 power gating driver

On Tue, Mar 14, 2017 at 08:05:39AM -0700, Andrey Smirnov wrote: > Add code allowing for control of various power domains managed by GPCv2 > IP block found in i.MX7 series of SoCs. Power domains covered by this > patch are: > > - PCIE PHY > - MIPI PHY > - USB HSIC PHY > - USB

Re: [PATCH 1/7] trace: Move trace_seq_overflowed out of line

On Wed, 15 Mar 2017 23:20:30 -0400 Steven Rostedt wrote: > > It is used in lots of places outside trace_handle_return, so that would > > give far less savings. Actually, I think you'll probably have *more* savings inlining trace_handle_return() than

Re: [PATCH v5 2/3] soc/imx: Add GPCv2 power gating driver

On Tue, Mar 14, 2017 at 08:05:39AM -0700, Andrey Smirnov wrote: > Add code allowing for control of various power domains managed by GPCv2 > IP block found in i.MX7 series of SoCs. Power domains covered by this > patch are: > > - PCIE PHY > - MIPI PHY > - USB HSIC PHY > - USB

Re: [PATCH 1/7] trace: Move trace_seq_overflowed out of line

On Wed, 15 Mar 2017 23:20:30 -0400 Steven Rostedt wrote: > > It is used in lots of places outside trace_handle_return, so that would > > give far less savings. Actually, I think you'll probably have *more* savings inlining trace_handle_return() than trace_seq_has_overflowed(). Why? Think

[PATCH v1 1/1] platform/x86: intel_pmc_ipc: fix io mem mapping size

Mapping entire GCR mem region in this driver creates mem region request conflict in sub devices that depend on PMC. This creates driver probe failure in devices like iTC0_wdt and telemetry device. Currently this driver only need memory mapping for s0ix counter registers. So this patch fixes this

[PATCH v1 1/1] platform/x86: intel_pmc_ipc: fix io mem mapping size

Mapping entire GCR mem region in this driver creates mem region request conflict in sub devices that depend on PMC. This creates driver probe failure in devices like iTC0_wdt and telemetry device. Currently this driver only need memory mapping for s0ix counter registers. So this patch fixes this

[PATCH v2 3/4] drm/rockchip/dsi: enable the grf clk before writing grf registers

For RK3399, the grf clk should be enabled before writing grf registers, otherwise the register value can not be changed. Signed-off-by: Chris Zhong --- Changes in v2: - check the grf_clk only for RK3399 drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 21 + 1

[PATCH v2 3/4] drm/rockchip/dsi: enable the grf clk before writing grf registers

For RK3399, the grf clk should be enabled before writing grf registers, otherwise the register value can not be changed. Signed-off-by: Chris Zhong --- Changes in v2: - check the grf_clk only for RK3399 drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 21 + 1 file changed, 21

[PATCH v2 4/4] drm/rockchip/dsi: correct the grf_switch_reg name

For the RK3399, the grf_switch_reg name should be RK3399_GRF_SOC_CON20, not RK3399_GRF_SOC_CON19. Signed-off-by: Chris Zhong --- Changes in v2: None drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 4/4] drm/rockchip/dsi: correct the grf_switch_reg name

For the RK3399, the grf_switch_reg name should be RK3399_GRF_SOC_CON20, not RK3399_GRF_SOC_CON19. Signed-off-by: Chris Zhong --- Changes in v2: None drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 0/4] RK3399 dw-mipi-dsi patches

Hi all This series set the phy_cfg_clk to be a required clock for RK3399, and add a grf clock control in dw-mipi-dsi driver. And then correct a register name. Changes in v2: - check the grf_clk only for RK3399 Chris Zhong (4): drm/rockchip/dsi: check phy_cfg_clk only for RK3399

[PATCH v2 1/4] drm/rockchip/dsi: check phy_cfg_clk only for RK3399

For RK3399, the phy_cfg_clk is a required clock, if phy_cfg_clk is disabled, MIPI phy can not work. Let's return a error if there is no phy_cfg_clk in dts property, when the pdata match RK3399. Signed-off-by: Chris Zhong --- Changes in v2: None

[PATCH v2 2/4] dt-bindings: add the grf clock for dw-mipi-dsi

For RK3399, the grf clock should be controlled by dw-mipi-dsi driver, add the description for this clock. Signed-off-by: Chris Zhong --- Changes in v2: None .../devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip.txt | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v2 0/4] RK3399 dw-mipi-dsi patches

Hi all This series set the phy_cfg_clk to be a required clock for RK3399, and add a grf clock control in dw-mipi-dsi driver. And then correct a register name. Changes in v2: - check the grf_clk only for RK3399 Chris Zhong (4): drm/rockchip/dsi: check phy_cfg_clk only for RK3399

  1   2   3   4   5   6   7   8   9   10   >