Re: [PATCH v2 6/6] selftests: exec: simpification the Makefile

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 22:04 +0800, Bamvor Jian Zhang wrote: > selftests: exec: simpification the Makefile Please give us a better subject, ie something like: selftests/exec: Revert to default emit rule > With the previous patch, the installation method change from install > to rsync. There is

Re: Use (two) different compilers at build-time?

2015-09-07 Thread Sedat Dilek
On Tue, Sep 8, 2015 at 7:45 AM, Sedat Dilek wrote: > On Tue, Sep 8, 2015 at 12:30 AM, Sedat Dilek wrote: >> On Mon, Sep 7, 2015 at 11:20 PM, Sedat Dilek wrote: >>> On Mon, Sep 7, 2015 at 10:59 PM, Linus Torvalds >>> wrote: On Mon, Sep 7, 2015 at 1:53 PM, Sedat Dilek wrote: > >

Re: [PATCH v2 5/6] selftests: change install command to rsync

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 22:04 +0800, Bamvor Jian Zhang wrote: > The command of install could not handle the special files in exec > testcases, change the default rule to rsync to fix this. > The result of installation is unchanged after this commit. > > Suggested-by: Michael Ellerman >

[PATCH v2 2/4] perf: Add function to enable perf events in kernel with ring buffer

2015-09-07 Thread Takao Indoh
perf_event_create_kernel_counter is used to enable perf events in kernel without buffer for logging its events. This patch add new fucntion which enable perf events with ring buffer. Intel PT logger uses this to enable Intel PT and some associated events with its log buffer. Signed-off-by: Takao

[PATCH v2 4/4] x86: Stop Intel PT and save its registers when panic occurs

2015-09-07 Thread Takao Indoh
When panic occurs, Intel PT logging is stopped to prevent it from overwrite its log buffer. The registers of Intel PT are saved in the memory on panic, they are needed for debugger to find the last position where Intel PT wrote data. Signed-off-by: Takao Indoh --- arch/x86/kernel/crash.c |9

[PATCH v2 0/4] x86: Intel Processor Trace Logger

2015-09-07 Thread Takao Indoh
Hi all, These patch series provide logging feature for Intel Processor Trace (Intel PT). Intel PT is a new feature of Intel CPU "Broadwell", it captures information about program execution flow. Here is a article about Intel PT. https://software.intel.com/en-us/blogs/2013/09/18/processor-tracing

[PATCH v2 3/4] perf/x86/intel/pt: Add Intel PT logger

2015-09-07 Thread Takao Indoh
This patch provides Intel PT logging feature. When system boots with a parameter "intel_pt_log", log buffers for Intel PT are allocated and logging starts, then processor flow information is written in the log buffer by hardware like flight recorder. This is very helpful to investigate a cause of

[PATCH v2 1/4] perf/trace: Add function to find event type by name

2015-09-07 Thread Takao Indoh
This patch adds function to find struct trace_event by event name like "sched_switch" , and return its type so that Intel PT logger can enable the trace event in kernel. Intel PT logger needs this because it needs sched_switch tracing to collect side-band data. Signed-off-by: Takao Indoh ---

Re: [PATCH v2 4/6] selftests: mqueue: simpification the Makefile

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 22:04 +0800, Bamvor Jian Zhang wrote: > selftests: mqueue: simpification the Makefile This should be: selftests/mqueue: Simplify the Makefile And please write a change log, even for simple commits like this. You can say something like: Use make's implict rule for

Re: Use (two) different compilers at build-time?

2015-09-07 Thread Sedat Dilek
On Tue, Sep 8, 2015 at 12:30 AM, Sedat Dilek wrote: > On Mon, Sep 7, 2015 at 11:20 PM, Sedat Dilek wrote: >> On Mon, Sep 7, 2015 at 10:59 PM, Linus Torvalds >> wrote: >>> On Mon, Sep 7, 2015 at 1:53 PM, Sedat Dilek wrote: That does not work. >>> >>> .. because you didn't do what I

Re: [PATCH v2 3/6] selftests: mqueue: allow extra cflags

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 22:04 +0800, Bamvor Jian Zhang wrote: > change from = to += in order to pass the proper headers and librareis Please start your changelogs with a Capital letter, they are supposed to be proper English sentences. > (popt.h and libpopt.so) in order to build successful in

Re: [PATCH v2 2/6] selftests: only compile userfaultfd for x86 and powperpc

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 22:04 +0800, Bamvor Jian Zhang wrote: > Check it before compiling to avoid the failure of building and > installation. I asked you not to do this. http://lkml.kernel.org/r/1440991580.5735.4.ca...@ellerman.id.au Can you tell me why you ignored my comments? cheers -- To

Re: [PATCH v2 1/6] selftests: rename jump label to static_keys

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 22:04 +0800, Bamvor Jian Zhang wrote: > commit "2bf9e0a locking/static_keys: Provide a selftest" rename ^ d > jump_label directory to static_keys.

Re: [PATCH] sched: fix lose fair sleeper bonus in switch_to_fair()

2015-09-07 Thread Wanpeng Li
On 9/8/15 1:28 PM, Byungchul Park wrote: On Tue, Sep 08, 2015 at 11:46:01AM +0800, Wanpeng Li wrote: On 9/7/15 10:02 PM, Peter Zijlstra wrote: Please always Cc at least the person who wrote the lines you modify. On Mon, Sep 07, 2015 at 05:45:20PM +0800, Wanpeng Li wrote: The sleeper task

Re: [PATCH 1/2] crypto: testmgr: test IV value after a cipher operation

2015-09-07 Thread Herbert Xu
On Tue, Sep 08, 2015 at 05:23:59AM +0200, Stephan Mueller wrote: > Am Dienstag, 16. Juni 2015, 11:46:46 schrieb Boris Brezillon: > > Hi Herbert, > > >The crypto drivers are supposed to update the IV passed to the crypto > >request before calling the completion callback. > >Test for the IV value

Re: [PATCH] sched: fix lose fair sleeper bonus in switch_to_fair()

2015-09-07 Thread Byungchul Park
On Tue, Sep 08, 2015 at 11:46:01AM +0800, Wanpeng Li wrote: > On 9/7/15 10:02 PM, Peter Zijlstra wrote: > >Please always Cc at least the person who wrote the lines you modify. > > > >On Mon, Sep 07, 2015 at 05:45:20PM +0800, Wanpeng Li wrote: > >>The sleeper task will be normalized when moved from

linux-next: Tree for Sep 8

2015-09-07 Thread Stephen Rothwell
Hi all, Please do not add material for v4.4 until after v4.3-rc1 is out. Changes since 20150903: I used the h8300 tree from next-20150828 since the current tree has been rebased onto something very old :-( The parisc-hd tree gained a conflict against Linus' tree. The v4l-dvb tree gained a

Re: eata fails to load on post 4.2 kernels

2015-09-07 Thread Jiang Liu
Hi Auhur, Could you please help to apply the test patch against the latest mainstream linux kernel? Thanks! Gerry On 2015/9/7 14:26, Arthur Marsh wrote: > > > Jiang Liu wrote on 07/09/15 12:36: >> On 2015/9/7 4:31, Arthur Marsh wrote: >>> Arthur Marsh wrote on 06/09/15 21:07:

Re: [PATCH] zram: don't copy invalid compression algorithms

2015-09-07 Thread Sergey Senozhatsky
On (09/08/15 13:50), Minchan Kim wrote: > For exmaple, disksize, max_comp_streams are changed only if > it is successful. > If your logic were right approach, we should change > max_comp_streams for *stupid* script although it doesn't check define stupid. is echo 210 >

Re: [PATCH v4 0/4] cpufreq-dt: add suspend frequency support

2015-09-07 Thread Viresh Kumar
On 08-09-15, 13:48, Krzysztof Kozlowski wrote: > Somehow my mind stuck on solving Exynos4x12 cpufreq issues. > > Right, it should go through Rafael's, probably except DTS patch (4/4) > because it depends on previous DTS changes. These changes are still in > arm-soc, not in Linus' tree [0].

Re: [PATCH] leds:lp55xx: Correct Kconfig dependency for f/w user helper

2015-09-07 Thread Takashi Iwai
On Tue, 08 Sep 2015 02:30:07 +0200, Kim, Milo wrote: > > Hi Takashi, > > On 9/7/2015 11:19 PM, Jacek Anaszewski wrote: > > Hi Takashi, > > > > Thanks for chasing this. > > Milo, could you express your opinion? > > > > On 09/07/2015 02:25 PM, Takashi Iwai wrote: > >> The commit [b67893206fc0:

Re: [PATCH] zram: don't copy invalid compression algorithms

2015-09-07 Thread Sergey Senozhatsky
On (09/08/15 13:50), Minchan Kim wrote: [..] > And it's straightforward/consistent to change the thing's state > only if is successful. > what for? I provided several good reasons not to do this, because it makes life easier for users. we added this check in Jun 25, 2015 while this functionality

Re: [PATCH v6 1/2] perf,kvm/powerpc: Add kvm_perf.h for powerpc

2015-09-07 Thread Hemant Kumar
On 09/07/2015 10:40 AM, Michael Ellerman wrote: On Fri, 2015-09-04 at 17:51 -0300, Arnaldo Carvalho de Melo wrote: Em Tue, Sep 01, 2015 at 12:18:47PM +0530, Hemant Kumar escreveu: Should I try to process the 5 together, applying thest two first? Yes, this patchset needs to be applied

Re: [git pull] drm for 4.3

2015-09-07 Thread Dave Airlie
On 8 September 2015 at 14:04, Stephen Rothwell wrote: > Hi Dave, > > On Tue, 8 Sep 2015 13:01:21 +1000 Dave Airlie wrote: >> >> Is this to be a regular thing? because I know I'd prefer to merge >> fixes than wait for -rc1 to be an accurate copy of linux-next. > > It happens when I can (almost)

linux-next: build failure after merge of the akpm tree

2015-09-07 Thread Stephen Rothwell
Hi Andrew, After merging the akpm tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/net/ieee802154/at86rf230.c: In function 'at86rf230_stats_show': drivers/net/ieee802154/at86rf230.c:1650:6: error: void value not ignored as it ought to be ret = seq_printf(file,

Re: [PATCH] zram: don't copy invalid compression algorithms

2015-09-07 Thread Minchan Kim
On Tue, Sep 08, 2015 at 10:58:31AM +0900, Sergey Senozhatsky wrote: > On (09/08/15 10:33), Sergey Senozhatsky wrote: > > > I don't understand your concern. To me, this patch makes sense to me. > > > Could you explain your point clearly, again? > > > > OK. suppose someone landed a typo in a 'zram

Re: [PATCH v4 0/4] cpufreq-dt: add suspend frequency support

2015-09-07 Thread Krzysztof Kozlowski
On 08.09.2015 13:20, Viresh Kumar wrote: > On 08-09-15, 12:56, Krzysztof Kozlowski wrote: >> There were no comments from your side for this patchset nor for previous >> Marek's fix. After mentioned change and Bart's re-spin, do you plan to >> grab this patchset and send it for current v4.3 cycle?

Re: linux-next: build failure after merge of the drm-misc tree

2015-09-07 Thread Maarten Lankhorst
Op 08-09-15 om 01:42 schreef Stephen Rothwell: > Hi all, > > On Thu, 3 Sep 2015 10:49:19 +1000 Stephen Rothwell > wrote: >> After merging the drm-misc tree, today's linux-next build (arm >> multi_v7_defconfig) failed like this: >> >> drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c: In function >>

Re: [PATCH v4 0/4] cpufreq-dt: add suspend frequency support

2015-09-07 Thread Viresh Kumar
On 08-09-15, 12:56, Krzysztof Kozlowski wrote: > There were no comments from your side for this patchset nor for previous > Marek's fix. After mentioned change and Bart's re-spin, do you plan to > grab this patchset and send it for current v4.3 cycle? Why do you think it should go via Kukjin's

Re: [PATCH 2/3] selftests: add membarrier syscall test

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 16:01 +, Mathieu Desnoyers wrote: > - On Sep 3, 2015, at 11:36 PM, Michael Ellerman m...@ellerman.id.au wrote: > > On Thu, 2015-09-03 at 15:47 +, Mathieu Desnoyers wrote: > >> > >> My personal experience is that make headers_install does not necessarily > >>

Re: [git pull] drm for 4.3

2015-09-07 Thread Stephen Rothwell
Hi Dave, On Tue, 8 Sep 2015 13:01:21 +1000 Dave Airlie wrote: > > Is this to be a regular thing? because I know I'd prefer to merge > fixes than wait for -rc1 to be an accurate copy of linux-next. It happens when I can (almost) keep up with Linus' merge rate (often I can't). It is not an issue

Re: [PATCH] powerpc32: memcpy: only use dcbz once cache is enabled

2015-09-07 Thread Michael Ellerman
On Mon, 2015-09-07 at 16:24 +0200, Christophe Leroy wrote: > memcpy() uses instruction dcbz to speed up copy by not wasting time > loading cache line with data that will be overwritten. > Some platform like mpc52xx do no have cache active at startup and > can therefore not use memcpy(). Allthough

Re: [PATCH v4 0/4] cpufreq-dt: add suspend frequency support

2015-09-07 Thread Krzysztof Kozlowski
On 08.09.2015 11:46, Viresh Kumar wrote: > On 07-09-15, 17:41, Bartlomiej Zolnierkiewicz wrote: >> Hi, >> >> This patch series adds suspend frequency support (using opp-v2 >> bindings and suspend-opp functionality) to cpufreq-dt driver and >> then adds suspend opp for Exynos4412 based boards. >>

Re: [PATCH] sched: fix lose fair sleeper bonus in switch_to_fair()

2015-09-07 Thread Wanpeng Li
On 9/7/15 10:02 PM, Peter Zijlstra wrote: Please always Cc at least the person who wrote the lines you modify. On Mon, Sep 07, 2015 at 05:45:20PM +0800, Wanpeng Li wrote: The sleeper task will be normalized when moved from fair_sched_class, in order that vruntime will be adjusted either the

Re: [PATCH 1/2] crypto: testmgr: test IV value after a cipher operation

2015-09-07 Thread Stephan Mueller
Am Dienstag, 16. Juni 2015, 11:46:46 schrieb Boris Brezillon: Hi Herbert, >The crypto drivers are supposed to update the IV passed to the crypto >request before calling the completion callback. >Test for the IV value before considering the test as successful. May I ask whether there is anything

Re: [PATCH] kernel/sysctl.c: If "count" including the terminating byte '\0' the write system call should retrun success.

2015-09-07 Thread Sean Fu
On Fri, Aug 28, 2015 at 11:31 AM, Sean Fu wrote: > On Thu, Aug 27, 2015 at 4:32 PM, Sean Fu wrote: >> On Thu, Aug 27, 2015 at 10:32 AM, Steven Rostedt wrote: >>> On Thu, 27 Aug 2015 08:17:29 +0800 >>> Sean Fu wrote: strace execute result: write(3, "1\2\0", 3)= -1

Re: [git pull] drm for 4.3

2015-09-07 Thread Dave Airlie
On 8 September 2015 at 12:03, Stephen Rothwell wrote: > Hi Linus, > > On Fri, 4 Sep 2015 23:40:53 +0100 (IST) Dave Airlie wrote: >> >> This is the main pull request for the drm for 4.3. Nouveau is probably the >> biggest >> amount of changes in here, since it missed 4.2. Highlights below, along

Re: [v2,3/6] powerpc: kill mfvtb()

2015-09-07 Thread Kevin Hao
On Mon, Sep 07, 2015 at 07:29:42PM +1000, Michael Ellerman wrote: > On Mon, 2015-24-08 at 11:20:25 UTC, Kevin Hao wrote: > > This function is only used by get_vtb(). They are almost the same > > except the reading from the real register. Move the mfspr() to > > get_vtb() and kill the function

[PATCH 0/2] perf: we can now read separate debug-info files based on a build ID

2015-09-07 Thread Dima Kogan
Date: Mon, 7 Sep 2015 19:27:01 -0700 (Please Cc me when replying; I'm not subscribed) Hi. Perf currently has trouble reading separate debug-info files when trying to look up symbols in a 'perf report'. According to the gdb documentation:

[PATCH] ARM:mm: define arch-specific IOREMAP_MAX_ORDER only in !SMP && !LPAE case

2015-09-07 Thread Zhang Zhen
The arch-specific IOREMAP_MAX_ORDER is introduced in commit: ff0daca([ARM] Add section support to ioremap) and commit: a069c89 ([ARM] 3705/1: add supersection support to ioremap()). But supersections and sections mappings are only used in !SMP && !LPAE case. Otherwise, mapping is created using the

Re: [PATCH v4 0/4] cpufreq-dt: add suspend frequency support

2015-09-07 Thread Viresh Kumar
On 07-09-15, 17:41, Bartlomiej Zolnierkiewicz wrote: > Hi, > > This patch series adds suspend frequency support (using opp-v2 > bindings and suspend-opp functionality) to cpufreq-dt driver and > then adds suspend opp for Exynos4412 based boards. > > This patch series fixes suspend/resume support

Re: [PATCH v4 3/4] cpufreq-dt: add suspend frequency support

2015-09-07 Thread Viresh Kumar
On 07-09-15, 17:41, Bartlomiej Zolnierkiewicz wrote: > +#ifdef CONFIG_PM > + .suspend = cpufreq_generic_suspend, > +#endif I don't think there is any need of the #ifdef here. -- viresh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH V2 6/9] cpufreq: ondemand: queue work for policy->cpus together

2015-09-07 Thread Viresh Kumar
On 08-09-15, 07:41, Viresh Kumar wrote: > > > next_sampling = jiffies + usecs_to_jiffies(new_rate); > > > appointed_at = dbs_info->cdbs.dwork.timer.expires; > > > > For that to work we always need to do stuff for policy->cpus in sync. > > Do we? > > Hmm, we are not in 100%

Re: [PATCH] sched: fix lose fair sleeper bonus in switch_to_fair()

2015-09-07 Thread Byungchul Park
hello wanpeng, On Mon, Sep 07, 2015 at 05:45:20PM +0800, Wanpeng Li wrote: > The sleeper task will be normalized when moved from fair_sched_class, in > order that vruntime will be adjusted either the task is running or sleeping > when moved back. The nomalization in switch_to_fair for sleep

Re: [PATCH V2 6/9] cpufreq: ondemand: queue work for policy->cpus together

2015-09-07 Thread Viresh Kumar
On 08-09-15, 03:33, Rafael J. Wysocki wrote: > > + /* Make sure the work is not canceled on policy->cpus */ > > I'm not sure what scenario can lead to that. Care to explain? CPUFREQ_GOV_STOP event called for the policy and so all its works are in canceled state. > > + if

Re: linux-next: build failure after merge of the kvm tree

2015-09-07 Thread Wanpeng Li
Hi Paolo, Stephen, On 9/8/15 9:35 AM, Stephen Rothwell wrote: Hi all, After merging the kvm tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c: In function 'grow_halt_poll_ns':

Re: [git pull] drm for 4.3

2015-09-07 Thread Stephen Rothwell
Hi Linus, On Fri, 4 Sep 2015 23:40:53 +0100 (IST) Dave Airlie wrote: > > This is the main pull request for the drm for 4.3. Nouveau is probably the > biggest > amount of changes in here, since it missed 4.2. Highlights below, along with > the usual > bunch of fixes. There are a few minor

Re: [PATCH] ARM: fix bug which lowmem size is limited to 760MB

2015-09-07 Thread Nicolas Pitre
On Tue, 8 Sep 2015, Russell King - ARM Linux wrote: > On Mon, Sep 07, 2015 at 03:40:36PM -0400, Nicolas Pitre wrote: > > On Mon, 7 Sep 2015, Arnd Bergmann wrote: > > > > > On Thursday 03 September 2015 21:24:00 Nicolas Pitre wrote: > > > > If 768MB targets were common place then it could be

Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-07 Thread Viresh Kumar
On 08-09-15, 03:15, Rafael J. Wysocki wrote: > On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > > __gov_queue_work() isn't required anymore and can be merged with > > gov_queue_work(). Do it. > > > > Signed-off-by: Viresh Kumar > > Quite frankly I don't see the point. But isn't that

Re: [PATCH V2 3/9] cpufreq: ondemand: only queue canceled works from update_sampling_rate()

2015-09-07 Thread Viresh Kumar
On 08-09-15, 03:11, Rafael J. Wysocki wrote: > There really are two cases, either you pass a CPU or gov_queue_work() has to > walk policy->cpus. Right (At least for now, we are doing just that.) > Doing it the way you did hides that IMO. Maybe. But I see it otherwise. Adding special meaning to

Re: [GIT PULL] Please pull NFS client changes

2015-09-07 Thread Stephen Rothwell
Hi Linus, On Mon, 07 Sep 2015 16:59:04 -0400 Trond Myklebust wrote: > > The following changes since commit 74d33293e467df61de1b1d8b2fbe29e550dec33b: > > Linux 4.2-rc5 (2015-08-02 18:34:55 -0700) > > are available in the git repository at: > >

Re: [PATCH] zram: don't copy invalid compression algorithms

2015-09-07 Thread Sergey Senozhatsky
On (09/08/15 10:33), Sergey Senozhatsky wrote: > > I don't understand your concern. To me, this patch makes sense to me. > > Could you explain your point clearly, again? > > OK. suppose someone landed a typo in a 'zram device management' script > > echo llzo > /sys/block/zram0/comp_algorithm >

Re: [PATCH v2 1/5] perf probe: Split add_perf_probe_events()

2015-09-07 Thread Namhyung Kim
On Sun, Sep 06, 2015 at 03:47:37PM +0800, Wangnan (F) wrote: > Hi Namhyung, Hi, I'm off until Wednesday. I'll be able to take a look at it on Thursday. Thanks, Namhyung > > Thanks for this patchset. > > Could you plase have a look at patch 5/27 and 6/27 in my newest pull > request? > These

Re: [PATCH v5 0/5] Let the power allocator thermal governor run on any thermal zone

2015-09-07 Thread Daniel Kurtz
Hi Javi, On Mon, Sep 7, 2015 at 9:19 PM, Javi Merino wrote: > Relax the thermal governor requirements of sustainable_power and at > least two trip points so that it can be bound to any thermal zone. > Its behavior won't be optimal, it would be the best possible with the > data provided. I

Re: [PATCH] arm64: kernel: Use a separate stack for irq interrupts.

2015-09-07 Thread AKASHI Takahiro
Jungseok, On 09/08/2015 01:34 AM, Jungseok Lee wrote: On Sep 8, 2015, at 1:06 AM, James Morse wrote: On 07/09/15 16:48, Jungseok Lee wrote: On Sep 7, 2015, at 11:36 PM, James Morse wrote: Hi James, Having to handle interrupts on top of an existing kernel stack means the kernel stack must

[PATCH] kasan: fix last shadow judgement in memory_is_poisoned_16()

2015-09-07 Thread Xishi Qiu
The shadow which correspond 16 bytes may span 2 or 3 bytes. If shadow only take 2 bytes, we can return in "if (likely(!last_byte)) ...", but it calculates wrong, so fix it. Signed-off-by: Xishi Qiu --- mm/kasan/kasan.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git

[PATCH V3 2/9] cpufreq: conservative: remove 'enable' field

2015-09-07 Thread Viresh Kumar
Conservative governor has its own 'enable' field to check if conservative governor is used for a CPU or not This can be checked by policy->governor with 'cpufreq_gov_conservative' and so this field can be dropped. Because its not guaranteed that dbs_info->cdbs.shared will a valid pointer for all

linux-next: build failure after merge of the kvm tree

2015-09-07 Thread Stephen Rothwell
Hi all, After merging the kvm tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: arch/powerpc/kvm/../../../virt/kvm/kvm_main.c: In function 'grow_halt_poll_ns': arch/powerpc/kvm/../../../virt/kvm/kvm_main.c:1931:2: error: implicit declaration of function

Re: [PATCH V2 2/9] cpufreq: conservative: remove 'enable' field

2015-09-07 Thread Viresh Kumar
On 08-09-15, 02:17, Rafael J. Wysocki wrote: > > static inline unsigned int get_freq_target(struct cs_dbs_tuners *cs_tuners, > >struct cpufreq_policy *policy) > > { > > @@ -119,12 +132,14 @@ static int dbs_cpufreq_notifier(struct notifier_block > > *nb,

Re: [PATCH v2] zram: fix possible use after free in zcomp_create()

2015-09-07 Thread Minchan Kim
On Tue, Sep 08, 2015 at 10:20:38AM +0900, Sergey Senozhatsky wrote: > On (09/08/15 10:07), Minchan Kim wrote: > [..] > > > + int ret; > > > > For the clarification, I want to call it as 'error' instead of ret. > > > > > > > > backend = find_backend(compress); > > > if (!backend) > > > @@

Re: [PATCH] zram: don't copy invalid compression algorithms

2015-09-07 Thread Sergey Senozhatsky
On (09/08/15 10:14), Minchan Kim wrote: [..] > > NACK. > > > > This is intentional. We haven't returned 'invalid compression algorithm' > > error from comp_algorithm_store() historically, so someone's script can > > simply ignore it. However, the script will fail to init the device and > > user

Re: [PATCH v2] zram: fix possible use after free in zcomp_create()

2015-09-07 Thread Sergey Senozhatsky
On (09/08/15 10:07), Minchan Kim wrote: [..] > > + int ret; > > For the clarification, I want to call it as 'error' instead of ret. > > > > > backend = find_backend(compress); > > if (!backend) > > @@ -347,10 +348,10 @@ struct zcomp *zcomp_create(const char *compress, int > >

[PATCH] Documentation: Remove misleading examples of the barriers in wake_*()

2015-09-07 Thread Boqun Feng
Two examples for barriers in wake_up() and co. in memory-barriers.txt are misleading, along with their explanations: 1. The example which wanted to explain the write barrier in wake_up() and co. [spotted by Oleg Nesterov ] 2. The example which wanted to explain that the write

Re: [PATCH] zram: don't copy invalid compression algorithms

2015-09-07 Thread Minchan Kim
Hey Sergey, On Tue, Sep 08, 2015 at 08:56:35AM +0900, Sergey Senozhatsky wrote: > On (09/07/15 21:48), Luis Henriques wrote: > > Validate the new compression algorithm before copying it into the zram > > 'compressor' field, keeping the old one if it's invalid. > > > > NACK. > > This is

RE: [RFC PATCH] PM / Runtime: runtime: Add sysfs option for forcing runtime suspend

2015-09-07 Thread Tirdea, Irina
> -Original Message- > From: linux-input-ow...@vger.kernel.org > [mailto:linux-input-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki > Sent: 08 September, 2015 0:20 > To: Tirdea, Irina > Cc: Alan Stern; linux...@vger.kernel.org; linux-in...@vger.kernel.org; >

RE: [PATCH v2 1/4] iio: bmc150: Use i2c regmap

2015-09-07 Thread Tirdea, Irina
> -Original Message- > From: Srinivas Pandruvada [mailto:srinivas.pandruv...@linux.intel.com] > Sent: 31 August, 2015 22:39 > To: Jonathan Cameron; Markus Pargmann; Tirdea, Irina > Cc: Lars-Peter Clausen; linux-...@vger.kernel.org; > linux-kernel@vger.kernel.org; ker...@pengutronix.de >

Re: [PATCH V2 6/9] cpufreq: ondemand: queue work for policy->cpus together

2015-09-07 Thread Rafael J. Wysocki
On Monday, July 27, 2015 05:58:11 PM Viresh Kumar wrote: > Currently update_sampling_rate() runs over each online CPU and > cancels/queues work on it. Its very inefficient for the case where a > single policy manages multiple CPUs, as they can be processed together. > > Also drop the unnecessary

Re: [PATCH v2] zram: fix possible use after free in zcomp_create()

2015-09-07 Thread Minchan Kim
Hello, First of all, Thanks for catching a bug and review, Guys. Below there are just some cleanup. If you guys think it's better, please respin. On Mon, Sep 07, 2015 at 03:13:10PM +0100, Luis Henriques wrote: > zcomp_create() verifies the success of zcomp_strm_{multi,siggle}_create() > through

Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-07 Thread Rafael J. Wysocki
On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > __gov_queue_work() isn't required anymore and can be merged with > gov_queue_work(). Do it. > > Signed-off-by: Viresh Kumar Quite frankly I don't see the point. I'd even remove the inline from its definition and let the compiler decide

Re: [PATCH V2 3/9] cpufreq: ondemand: only queue canceled works from update_sampling_rate()

2015-09-07 Thread Rafael J. Wysocki
On Monday, July 27, 2015 05:58:08 PM Viresh Kumar wrote: > The sampling rate is updated with a call to update_sampling_rate(), and > we process CPUs one by one here. While the work is canceled on per-cpu > basis, it is getting queued (by mistake) for all policy->cpus. > > This would result in

Re: [PATCH v4 4/4] ARM: dts: add suspend opp to exynos4412

2015-09-07 Thread Krzysztof Kozlowski
On 08.09.2015 00:41, Bartlomiej Zolnierkiewicz wrote: > Mark 800MHz OPP as a suspend opp for Exynos4412 based > boards so effectively cpufreq-dt driver behavior w.r.t. > suspend frequency matches what the old exynos-cpufreq > driver has been doing. > > This patch fixes suspend/resume support on

Re: [PATCH] leds:lp55xx: Correct Kconfig dependency for f/w user helper

2015-09-07 Thread Kim, Milo
Hi Takashi, On 9/7/2015 11:19 PM, Jacek Anaszewski wrote: Hi Takashi, Thanks for chasing this. Milo, could you express your opinion? On 09/07/2015 02:25 PM, Takashi Iwai wrote: The commit [b67893206fc0: leds:lp55xx: fix firmware loading error] tries to address the firmware file handling with

Re: wake_up_process implied memory barrier clarification

2015-09-07 Thread Boqun Feng
Hi Oleg, On Mon, Sep 07, 2015 at 07:06:52PM +0200, Oleg Nesterov wrote: > Sorry for delay, > > On 09/02, Boqun Feng wrote: > > > > On Tue, Sep 01, 2015 at 06:39:23PM +0200, Oleg Nesterov wrote: > > > On 09/01, Boqun Feng wrote: > > > > > > > > On Tue, Sep 01, 2015 at 11:59:23AM +0200, Oleg

Re: [PATCH] ARM: fix bug which lowmem size is limited to 760MB

2015-09-07 Thread Russell King - ARM Linux
On Mon, Sep 07, 2015 at 03:40:36PM -0400, Nicolas Pitre wrote: > On Mon, 7 Sep 2015, Arnd Bergmann wrote: > > > On Thursday 03 September 2015 21:24:00 Nicolas Pitre wrote: > > > If 768MB targets were common place then it could be worth changing the > > > default vmalloc size to accommodate this

Re: [PATCH v2] zram: fix possible use after free in zcomp_create()

2015-09-07 Thread Sergey Senozhatsky
On (09/07/15 15:13), Luis Henriques wrote: > zcomp_create() verifies the success of zcomp_strm_{multi,siggle}_create() > through comp->stream, which can potentially be pointing to memory that was > freed if these functions returned an error. > > Fixes: beca3ec71fe5 ("zram: add multi stream

Re: [PATCH] zram: don't copy invalid compression algorithms

2015-09-07 Thread Sergey Senozhatsky
On (09/07/15 21:48), Luis Henriques wrote: > Validate the new compression algorithm before copying it into the zram > 'compressor' field, keeping the old one if it's invalid. > NACK. This is intentional. We haven't returned 'invalid compression algorithm' error from comp_algorithm_store()

Re: [PATCH V2 2/9] cpufreq: conservative: remove 'enable' field

2015-09-07 Thread Rafael J. Wysocki
On Monday, July 27, 2015 05:58:07 PM Viresh Kumar wrote: > Conservative governor has its own 'enable' field to check if > conservative governor is used for a CPU or not > > This can be checked by policy->governor with 'cpufreq_gov_conservative' > and so this field can be dropped. > > Because its

Re: [PATCH] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-07 Thread Frank Rowand
On 9/6/2015 8:50 PM, Frank Rowand wrote: > On 9/6/2015 7:16 PM, Frank Rowand wrote: >> On 9/6/2015 1:46 PM, Rob Herring wrote: >>> On Fri, Sep 4, 2015 at 8:14 PM, Frank Rowand wrote: On 9/4/2015 12:12 PM, David Daney wrote: > From: David Daney > > It is perfectly legitimate for

Re: linux-next: build failure after merge of the drm-misc tree

2015-09-07 Thread Stephen Rothwell
Hi all, On Thu, 3 Sep 2015 10:49:19 +1000 Stephen Rothwell wrote: > > After merging the drm-misc tree, today's linux-next build (arm > multi_v7_defconfig) failed like this: > > drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c: In function > 'mdp5_plane_cleanup_fb': >

Re: linux-next: manual merge of the akpm-current tree with the tip tree

2015-09-07 Thread Stephen Rothwell
Hi Linus, On Wed, 29 Jul 2015 19:12:56 +0200 Andrea Arcangeli wrote: > > On Tue, Jul 28, 2015 at 04:00:15PM +1000, Stephen Rothwell wrote: > > -359 i386userfaultfd sys_userfaultfd > > ++374 i386userfaultfd sys_userfaultfd > > Do I understand

linux-next: manual merge of the v4l-dvb tree with Linus' tree

2015-09-07 Thread Stephen Rothwell
Hi Mauro, Today's linux-next merge of the v4l-dvb tree got a conflict in: mm/Makefile between commit: c1a4de99fada ("userfaultfd: mcopy_atomic|mfill_zeropage: UFFDIO_COPY|UFFDIO_ZEROPAGE preparation") from Linus' tree and commit: 8025e5ddf9c1 ("[media] mm: Provide new

Re: [PATCH V2 1/9] cpufreq: Use __func__ to print function's name

2015-09-07 Thread Rafael J. Wysocki
On Monday, July 27, 2015 05:58:06 PM Viresh Kumar wrote: > Its better to use __func__ to print functions name instead of writing > the name in the print statement. This also has the advantage that a > change in function's name doesn't force us to change the print message > as well. > >

linux-next: manual merge of the parisc-hd tree with Linus' tree

2015-09-07 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the parisc-hd tree got a conflict in: arch/parisc/kernel/irq.c between commit: d2109a12198e ("parisc/irq: Use access helper irq_data_get_affinity_mask()") from Linus' tree and commit: b15e2f939727 ("parisc: Filter out spurious interrupts in PA-RISC

[RFC PATCH v2] Fix: clocksource watchdog marks TSC unstable on guest VM

2015-09-07 Thread Mathieu Desnoyers
I have been getting those warnings across a range of guest kernels in my development virtual machines. The host is a 3.13 Ubuntu kernel. The latest guest on which I reproduced this is a 4.2 kernel (akpm's tree). [ 126.902240] clocksource: timekeeping watchdog: Marking clocksource 'tsc' as

[PATCH] staging: lustre: ptlrpc: include ptlrpc_internal.h

2015-09-07 Thread Anders Fridlund
Remove sparse warning "symbol 'sptlrpc_plain_init' was not declared" by including ptlrpc_internal.h, which includes sptlrpc_plain_init. Signed-off-by: Anders Fridlund --- drivers/staging/lustre/lustre/ptlrpc/sec_plain.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v3] Documentation: add Device tree bindings for hwmon/nct7802

2015-09-07 Thread Constantine Shulyupin
Add add Device tree bindings for registers which are not covered by hwmon ABI and are required to configure specific HW. --- Changed in v3: - Fixed vendor prefix - Added short registers description, full registers description is available at

Re: Use (two) different compilers at build-time?

2015-09-07 Thread Sedat Dilek
On Mon, Sep 7, 2015 at 11:20 PM, Sedat Dilek wrote: > On Mon, Sep 7, 2015 at 10:59 PM, Linus Torvalds > wrote: >> On Mon, Sep 7, 2015 at 1:53 PM, Sedat Dilek wrote: >>> >>> That does not work. >> >> .. because you didn't do what I told you to do. >> >>> I copied a gcc-compiled percpu.o OR

[RFC PATCH] Fix: clocksource watchdog marks TSC unstable on guest VM

2015-09-07 Thread Mathieu Desnoyers
I have been getting those warnings across a range of guest kernels in my development virtual machines. The host is a 3.13 Ubuntu kernel. The latest guest on which I reproduced this is a 4.2 kernel (akpm's tree). [ 126.902240] clocksource: timekeeping watchdog: Marking clocksource 'tsc' as

Re: Dealing with the NMI mess

2015-09-07 Thread Andy Lutomirski
On Mon, Sep 7, 2015 at 12:30 PM, Maciej W. Rozycki wrote: > On Mon, 7 Sep 2015, Andy Lutomirski wrote: > >> > These are all implementation-specific details, including the INT1 >> > instruction, which is why I am not at all surprised that they are omitted >> > from architecture manuals. >> >>

[PATCH] ARM: exynos_defconfig: Enable USB Video Class support

2015-09-07 Thread Javier Martinez Canillas
The Exynos5420 Peach Pit and Exynos5800 Peach Pi boards have a built-in Silicon Motion USB UVC WebCam. Enable support for the USB Video Class driver and its needed media Kconfig symbols so the camera is supported. Signed-off-by: Javier Martinez Canillas --- arch/arm/configs/exynos_defconfig |

Energy Aware Scheduling and Power Management LPC microconference summary

2015-09-07 Thread Rafael J. Wysocki
Hi All, After the Energy Aware Scheduling and Power Management microconference at the LPC I and Morten prepared a summary of it for the LPC readout session. There was not enough time to present the whole of it, though, so I promised to send it out, but then got distracted by urgent stuff and

Re: [PATCH v2] stmmac: fix check for phydev being open

2015-09-07 Thread Alexey Brodkin
Hi Sergei, On Tue, 2015-09-08 at 00:24 +0300, Sergei Shtylyov wrote: > On 09/07/2015 11:50 PM, Alexey Brodkin wrote: > > > Current implementation via IS_ERR(phydev) may make no sense because > > of_phy_attach() returns NULL on failure instead of error value. > > Not of_phy_connect()? I

Re: [PATCH v2] stmmac: fix check for phydev being open

2015-09-07 Thread Sergei Shtylyov
On 09/07/2015 11:50 PM, Alexey Brodkin wrote: Current implementation via IS_ERR(phydev) may make no sense because of_phy_attach() returns NULL on failure instead of error value. Not of_phy_connect()? Still for checking result of phy_connect() IS_ERR() is useful. To address both

Re: Use (two) different compilers at build-time?

2015-09-07 Thread Sedat Dilek
On Mon, Sep 7, 2015 at 10:59 PM, Linus Torvalds wrote: > On Mon, Sep 7, 2015 at 1:53 PM, Sedat Dilek wrote: >> >> That does not work. > > .. because you didn't do what I told you to do. > >> I copied a gcc-compiled percpu.o OR deleted/renamed percpu.o and >> re-invoked make - this starts a

Re: [PATCH][rfc] intel_pstate: Fix user input of min/max to legal policy region

2015-09-07 Thread Rafael J. Wysocki
On Wednesday, August 12, 2015 11:49:19 AM Chen Yu wrote: > In current code, if system is using performance policy, user can > modify the max_perf_pct to any values lower than 100: > > $ grep . /sys/devices/system/cpu/intel_pstate/m*_perf_pct > /sys/devices/system/cpu/intel_pstate/max_perf_pct:100

Re: [RFC v0 2/9] suspend: Add getter function to report if freezing is active

2015-09-07 Thread Rafael J. Wysocki
On Monday, September 07, 2015 10:55:43 AM Daniel Wagner wrote: > On 09/05/2015 04:11 AM, Rafael J. Wysocki wrote: > > On Friday, September 04, 2015 03:34:55 PM Daniel Wagner wrote: > >> Instead encode the FREEZE state via the CPU state we allow the > >> interesting subsystems (MCE, microcode) to

Re: [PATCH v2] stmmac: fix check for phydev being open

2015-09-07 Thread Alexey Brodkin
Hi Sergei, On Mon, 2015-09-07 at 23:53 +0300, Sergei Shtylyov wrote: > On 09/07/2015 11:50 PM, Alexey Brodkin wrote: > > > Current implementation via IS_ERR(phydev) may make no sense because > > of_phy_attach() returns NULL on failure instead of error value. > > > > Still for checking result of

Re: Possible deadlock related to CPU hotplug and kernfs

2015-09-07 Thread Rafael J. Wysocki
On Monday, September 07, 2015 11:11:19 AM Jiang Liu wrote: > On 2015/9/4 22:16, Rafael J. Wysocki wrote: > > Hi, > > > > On Fri, Sep 4, 2015 at 9:20 AM, Jiang Liu wrote: > >> On 2015/9/4 4:08, Rafael J. Wysocki wrote: > >>> Hi Tejun, > >>> > >>> On Thu, Sep 3, 2015 at 6:19 PM, Tejun Heo wrote:

Re: [PATCH 1/5] acpi: Add basic device probing infrastructure

2015-09-07 Thread Rafael J. Wysocki
On Friday, September 04, 2015 06:06:48 PM Marc Zyngier wrote: > IRQ controllers and timers are the two types of device the kernel > requires before being able to use the device driver model. > > ACPI so far lacks a proper probing infrastructure similar to the one > we have with DT, where we're

Re: Use (two) different compilers at build-time?

2015-09-07 Thread Linus Torvalds
On Mon, Sep 7, 2015 at 1:53 PM, Sedat Dilek wrote: > > That does not work. .. because you didn't do what I told you to do. > I copied a gcc-compiled percpu.o OR deleted/renamed percpu.o and > re-invoked make - this starts a complete new build from scratch. Right. Because you changed the

  1   2   3   4   5   6   7   8   9   10   >