[PATCH v3 1/6] mm/memory-hotplug: fix lowmem count overflow when offline pages

2013-05-25 Thread Wanpeng Li
Changelog: v1 -> v2: * show number of HighTotal before hotremove * remove CONFIG_HIGHMEM * cc stable kernels * add Michal reviewed-by Logic memory-remove code fails to correctly account the Total High Memory when a memory block which contains High Memory is

Re: Stupid VFS name lookup interface..

2013-05-25 Thread Linus Torvalds
On Sat, May 25, 2013 at 10:04 PM, James Morris wrote: > On Sat, 25 May 2013, Linus Torvalds wrote: > >> But I haven't even looked at what non-selinux setups do to >> performance. Last time I tried Ubuntu (they still use apparmor, no?), >> "make modules_install ; make install" didn't work for the

Re: Stupid VFS name lookup interface..

2013-05-25 Thread James Morris
On Sat, 25 May 2013, Linus Torvalds wrote: > But I haven't even looked at what non-selinux setups do to > performance. Last time I tried Ubuntu (they still use apparmor, no?), > "make modules_install ; make install" didn't work for the kernel, and > if the Ubuntu people don't want to support

Re: Stupid VFS name lookup interface..

2013-05-25 Thread James Morris
On Sat, 25 May 2013, Al Viro wrote: > Well... The problem I see here is not even selinux per se - it's that > "LSM stacking" insanity. FWIW, I don't see a concrete rationale for merging this stacking work. That doesn't mean there won't be one in the future, but I wouldn't let the idea of a

[PATCH 2/2] perf script: add option to dump parent pid

2013-05-25 Thread David Ahern
Signed-off-by: David Ahern Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: Jiri Olsa Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Stephane Eranian --- tools/perf/Documentation/perf-script.txt |2 +- tools/perf/builtin-script.c |9 +++-- 2 files changed, 8 insertions(+), 3

[PATCH 1/2] perf: save parent pid in thread struct

2013-05-25 Thread David Ahern
Information is available, so why not save it in case some command wants to use it. Signed-off-by: David Ahern Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: Jiri Olsa Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Stephane Eranian --- tools/perf/util/thread.c |4 tools/perf/util/thread.h |

[PATCH 02/02] swapon: allow a more flexible swap discard policy

2013-05-25 Thread Rafael Aquini
Introduce the necessary changes to swapon(8) allowing a sysadmin to leverage the new changes introduced to sys_swapon by "swap: discard while swapping only if SWAP_FLAG_DISCARD_PAGES", therefore allowing a more flexible set of choices when selection the discard policy for mounted swap areas. This

[PATCH 01/02] swap: discard while swapping only if SWAP_FLAG_DISCARD_PAGES

2013-05-25 Thread Rafael Aquini
This patch introduces SWAP_FLAG_DISCARD_PAGES and SWAP_FLAG_DISCARD_ONCE new flags to allow more flexibe swap discard policies being flagged through swapon(8). The default behavior is to keep both single-time, or batched, area discards (SWAP_FLAG_DISCARD_ONCE) and fine-grained discards for

[PATCH 00/02] swap: allowing a more flexible DISCARD policy V2

2013-05-25 Thread Rafael Aquini
Considering the use cases where the swap device supports discard: a) and can do it quickly; b) but it's slow to do in small granularities (or concurrent with other I/O); c) but the implementation is so horrendous that you don't even want to send one down; And assuming that the sysadmin

Re: [PATCH 1/2] ARM: kirkwood: proper retain MAC address workaround on DT ethernet

2013-05-25 Thread David Miller
From: Sebastian Hesselbarth Date: Wed, 22 May 2013 22:04:01 +0200 > + memcpy((void *)p->value, reg, 6); This cast is completely unnecessary, non-void to void pointer casts are automatic. If it is necessary, because p->value is const, then you are trying to change something

Re: [PATCH v2] ipc/sem.c: fix lockup, restore FIFO behavior

2013-05-25 Thread Mike Galbraith
On Sun, 2013-05-26 at 05:19 +0200, Mike Galbraith wrote: > On Sat, 2013-05-25 at 22:00 +0200, Manfred Spraul wrote: > > > P.S.: > > Btw, I found some code that uses a semop with 2 ops: > > http://publib.boulder.ibm.com/infocenter/iseries/v5r3/index.jsp?topic=%2Fapis%2Fapiexusmem.htm > > I

Re: Stupid VFS name lookup interface..

2013-05-25 Thread Linus Torvalds
On Sat, May 25, 2013 at 11:33 AM, Casey Schaufler wrote: > > Now I'll put on my Smack maintainer hat. Performance improvement is > always welcome, but I would rather see attention to performance of > the LSM architecture than SELinux specific hacks. I haven't seen huge issues with performance at

Re: [PATCH v2] ipc/sem.c: fix lockup, restore FIFO behavior

2013-05-25 Thread Mike Galbraith
On Sat, 2013-05-25 at 22:00 +0200, Manfred Spraul wrote: > P.S.: > Btw, I found some code that uses a semop with 2 ops: > http://publib.boulder.ibm.com/infocenter/iseries/v5r3/index.jsp?topic=%2Fapis%2Fapiexusmem.htm I recently analyzed rt traces containing 9 ops. -Mike -- To unsubscribe from

[PATCH] MAINTAINERS: add linux-gpio mailing list

2013-05-25 Thread Alexandre Courbot
Signed-off-by: Alexandre Courbot --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index b645a3d..1541796 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3566,6 +3566,7 @@ GPIO SUBSYSTEM M: Grant Likely M: Linus Walleij S: Maintained

proc: add nsfd mount option to allow mounting for pid namespaces other than the current

2013-05-25 Thread Stephen Mell
From: Stephen Mell Currently, there is no userspace method to mount proc for a pid namespace other than the current one. In light of the new namespace filedescriptors, this patch adds a mount option to use the namespace represented by the specified filedescriptor instead of the current pid

Re: [PATCH] ARM: tegra114: correctly output clk_32k

2013-05-25 Thread Alexandre Courbot
On Sat, May 25, 2013 at 12:30 AM, Stephen Warren wrote: > On 05/24/2013 01:10 AM, Alexandre Courbot wrote: >> Tegra has a blink timer register that allows to modulate the >> clk_32k clock before outputting it. Since clk_32k is presented to the >> kernel as a fixed clock, make sure this register

[PATCH RESEND] ARM: tegra114: correctly output clk_32k

2013-05-25 Thread Alexandre Courbot
Tegra has a blink timer register that allows to modulate the clk_32k clock before outputting it. Since clk_32k is presented to the kernel as a fixed clock, make sure this register does not tamper with the clock frequency and that clk_32k is outputted as-is, similarly to what is done on t20 and

[PATCH] perf: sample after exit loses thread correlation

2013-05-25 Thread David Ahern
Some events (context-switch, sched:sched_switch) are losing the conversion of sample data associated with a thread. For example: $ perf record -e sched:sched_switch -c 1 -a -- sleep 5 $ perf script ls 30482 [000] 1379727.583037: sched:sched_switch: prev_comm=ls prev_pid=30482 ... ls

[PATCH 2/2] f2fs: add sysfs support for controlling the gc_thread

2013-05-25 Thread Namjae Jeon
From: Namjae Jeon Add sysfs entries to control the timing parameters for f2fs gc thread and also provide a control flag, which will allow the user to forcefully start a FG garbage collection if required. Signed-off-by: Namjae Jeon Signed-off-by: Pankaj Kumar --- fs/f2fs/f2fs.h |6 +++

[PATCH v2 1/2] f2fs: return proper error from start_gc_thread

2013-05-25 Thread Namjae Jeon
From: Namjae Jeon when there is an error from kthread_run, then return proper error rather than returning -ENOMEM. Signed-off-by: Namjae Jeon Signed-off-by: Amit Sahrawat --- fs/f2fs/gc.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/fs/f2fs/gc.c

Re: [PATCH v2 1/4] mm/memory-hotplug: fix lowmem count overflow when offline pages

2013-05-25 Thread KOSAKI Motohiro
(5/25/13 9:23 PM), Wanpeng Li wrote: > Hi KOSAKI, > On Sat, May 25, 2013 at 09:16:24PM -0400, KOSAKI Motohiro wrote: >>> --- >>> mm/page_alloc.c | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/mm/page_alloc.c b/mm/page_alloc.c >>> index 98cbdf6..23b921f 100644 >>> ---

Re: [PATCH v2 0/4] media: i2c: ths7303 cleanup

2013-05-25 Thread Laurent Pinchart
On Saturday 25 May 2013 23:09:32 Prabhakar Lad wrote: > From: Lad, Prabhakar > > Trivial cleanup of the driver. > > Changes for v2: > 1: Dropped the asynchronous probing and, OF >support patches will be handling them independently because of > dependencies. 2: Arranged the patches logically

Re: [PATCH v2 1/4] mm/memory-hotplug: fix lowmem count overflow when offline pages

2013-05-25 Thread KOSAKI Motohiro
> --- > mm/page_alloc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index 98cbdf6..23b921f 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -6140,6 +6140,8 @@ __offline_isolated_pages(unsigned long start_pfn, > unsigned long end_pfn)

Re: [PATCH v2 4/5] media: davinci: vpif_capture: Convert to devm_* api

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:35 Prabhakar Lad wrote: > From: Lad, Prabhakar > > use devm_request_irq() instead of request_irq(). > This ensures more consistent error values and simplifies error paths. > > use module_platform_driver to simplify the code.

Re: [RFC][PATCH] mm: Fix RLIMIT_MEMLOCK

2013-05-25 Thread KOSAKI Motohiro
On Fri, May 24, 2013 at 11:40 AM, Christoph Lameter wrote: > On Fri, 24 May 2013, Peter Zijlstra wrote: > >> Patch bc3e53f682 ("mm: distinguish between mlocked and pinned pages") >> broke RLIMIT_MEMLOCK. > > Nope the patch fixed a problem with double accounting. > > The problem that we seem to

Re: [PATCH 2/7] Documentation/devices.txt: Remove /dev/oldmem description

2013-05-25 Thread Zhang Yanfei
于 2013年05月26日 07:20, Eric W. Biederman 写道: > Zhang Yanfei writes: > >> From: Zhang Yanfei > > Won't we want to keep this reservation around to so that this number > doesn't get reused, and cause people confusion when > upgrading/downgrading kernels? Ah, yes. I will just keep this and add a

Re: [PATCH v2 3/5] media: davinci: vpif: remove unnecessary braces around defines

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:34 Prabhakar Lad wrote: > From: Lad, Prabhakar > > Signed-off-by: Lad, Prabhakar > --- > drivers/media/platform/davinci/vpif.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git

Re: [PATCH v2 1/5] media: davinci: vpif: remove unwanted header mach/hardware.h and sort the includes alphabetically

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:32 Prabhakar Lad wrote: > From: Lad, Prabhakar > > This patch removes unwanted header include of mach/hardware.h > and along side sorts the header inclusion alphabetically. > > Signed-off-by: Lad, Prabhakar Acked-by:

Re: [PATCH v2 2/5] media: davinci: vpif: Convert to devm_* api

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:33 Prabhakar Lad wrote: > From: Lad, Prabhakar > > Use devm_ioremap_resource instead of reques_mem_region()/ioremap(). > This ensures more consistent error values and simplifies error paths. > > Signed-off-by: Lad, Prabhakar

[PATCH 03/16 v2] perf evlist: add initialzation function for tracepoints

2013-05-25 Thread David Ahern
Handles initializations typically done as part of processing the file header and HEADER_TRACING_DATA event. v2: close fd as noticed by Namhyung. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim

[PATCH 05/16] perf top: move CONSOLE_CLEAR to header file

2013-05-25 Thread David Ahern
For use with kvm-live mode. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang --- tools/perf/builtin-top.c |2 -- tools/perf/util/top.h|2 ++ 2

[PATCH 08/16] perf stats: fix divide by 0 in variance

2013-05-25 Thread David Ahern
Number of samples needs to be greater 1 to have a variance. Fixes nan% in perf-kvm-live output. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang ---

[PATCH 01/16] perf evlist: restore methods removed in earlier cleanup

2013-05-25 Thread David Ahern
e60fc847 removed the perf_evlist__add_tracepoints and helpers, but they are useful for kvm's upcoming live mode. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen

[PATCH 06/16] perf kvm: handle realloc failures

2013-05-25 Thread David Ahern
Save previous pointer and free on failure. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang --- tools/perf/builtin-kvm.c |3 +++ 1 file changed, 3

[PATCH 10/16 v2] perf kvm: add live mode

2013-05-25 Thread David Ahern
perf kvm stat currently requires back to back record and report commands to see stats. e.g,. perf kvm stat record -p $pid -- sleep 1 perf kvm stat report This is inconvenvient for on box monitoring of a VM. This patch introduces a 'live' mode that in effect combines the record plus report

[PATCH 12/16] perf kvm: option to print events that exceed a threshold

2013-05-25 Thread David Ahern
This is useful to spot high latency blips. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang --- tools/perf/builtin-kvm.c | 24

[PATCH 16/16] perf kvm: set live mode refresh time dynamically

2013-05-25 Thread David Ahern
David's patch (https://lkml.org/lkml/2013/5/9/125 ) add a live mode for perf kvm stat, but his patch can't set the refresh frequency dynamically. This patch accepts user's input and changes the value of kvm->display_time variable. For example, when the live mode is running, you press any key, it

[PATCH 15/16] perf kvm: move the prompt_integer() to /util/top.c

2013-05-25 Thread David Ahern
As David suggested, this patch has no functional changes just a code move. move prompt_integer() from tools/perf/builtin-top.c into tools/perf/util/top.c Signed-off-by: Runzhen Wang Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter

[PATCH 14/16] perf kvm: reuse some code of perf_kvm__timerfd_create()

2013-05-25 Thread David Ahern
From: Runzhen Wang Create a new function perf_kvm__timerfd_set() to reuse some code in perf_kvm__timerfd_create() Signed-off-by: Runzhen Wang Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim

[PATCH 13/16] perf kvm: debug for missing vmexit/vmentry event

2013-05-25 Thread David Ahern
Expected to have missing events for each vcpu when perf is started. After that should not have missing events. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen

[PATCH 09/16] perf stats: add max and min stats

2013-05-25 Thread David Ahern
Need an initialization function to set min to -1 to differentiate from an actual min of 0. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang ---

[PATCH 11/16] perf kvm: add min and max stats to display

2013-05-25 Thread David Ahern
Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang --- tools/perf/builtin-kvm.c | 21 ++--- 1 file changed, 18 insertions(+), 3

[PATCH 07/16] perf kvm: split out tracepoints from record args

2013-05-25 Thread David Ahern
Needed by kvm live command. Make record_args a local while we are messing with the args. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang ---

[PATCH 04/16 v2] perf session: export a few functions for event processing

2013-05-25 Thread David Ahern
Allows kvm live mode to reuse the event processing and ordered samples processing used by the perf-report path. v2: removed flush_sample_queue as noticed by Jiri Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa

[PATCH 02/16] perf evlist: move tracepoint processing code to evlist.c

2013-05-25 Thread David Ahern
Per function names they are more aligned with the evlist code than the header code. Export perf_evlist__prepare_tracepoint_events in the process. Code move only; no functional changes. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter

[PATCH 00/16] perf: kvm live mode

2013-05-25 Thread David Ahern
This version addresses all of the comments from the last review and includes an enhancement from Runzhen Wang. I still need to add Ingo's enhancements -- aliasing 'perf stat kvm' to 'perf kvm stat live', and adding options to parallel top (--repeat ). patchset can also be obtained from

Re: [PATCH 07/13] perf kvm: split out tracepoints from record args

2013-05-25 Thread David Ahern
On 5/10/13 6:39 AM, Jiri Olsa wrote: + for (j = 0; j < ARRAY_SIZE(kvm_events_tp); j++) { + rec_argv[i++] = "-e"; + rec_argv[i++] = STRDUP_FAIL_EXIT(kvm_events_tp[j]); + } + just curious.. I dont understand why it needs to be dynamic? jirka passing a

Re: [GIT PULL] battery-2.6.git

2013-05-25 Thread David Woodhouse
On Sat, 2013-05-25 at 15:23 -0700, Anton Vorontsov wrote: > Please pull battery-2.6 git tree to get a few one-liners: wrong kfree > usage fix, module alias fixup and kconfig adjustments. The patches really > small, so for convenience I inline them here. Please note git.infradead.org has just been

[PATCH 1/2] perf: fix printing of perf_event_paranoid message

2013-05-25 Thread David Ahern
message is currently shown as: Error: You may not have permission to collect %sstats. Consider tweaking /proc/sys/kernel/perf_event_paranoid: Note the %sstats. With patch this becomes: Error: You may not have permission to collect stats. Consider tweaking

[PATCH] perf: reset SIGTERM handler in workload child process

2013-05-25 Thread David Ahern
Jiri reported hanging perf tests on latest acme's perf/core and bisected it to 87f303a9f: [jolsa@krava2 perf]$ cat /proc/sys/kernel/perf_event_paranoid 1 [jolsa@krava2 perf]$ ./perf record -C 0 kill Error: You may not have permission to collect %sstats. Consider tweaking

Re: [GIT PULL] battery-2.6.git

2013-05-25 Thread Anton Vorontsov
On Sun, May 26, 2013 at 12:23:32AM +0100, David Woodhouse wrote: > On Sat, 2013-05-25 at 15:23 -0700, Anton Vorontsov wrote: > > Please pull battery-2.6 git tree to get a few one-liners: wrong kfree > > usage fix, module alias fixup and kconfig adjustments. The patches really > > small, so for

Re: [PATCH 0/7] Remove unused /dev/oldmem interface

2013-05-25 Thread Eric W. Biederman
Zhang Yanfei writes: > /dev/oldmem provides the interface for us to access the "old memory" in > the dump-capture kernel. Unfortunately, no one actually uses this interface. > > And this interface could actually cause some real problems if used on ia64 > where the cached/uncached accesses are

Re: [PATCH 2/7] Documentation/devices.txt: Remove /dev/oldmem description

2013-05-25 Thread Eric W. Biederman
Zhang Yanfei writes: > From: Zhang Yanfei Won't we want to keep this reservation around to so that this number doesn't get reused, and cause people confusion when upgrading/downgrading kernels? > Signed-off-by: Zhang Yanfei > Cc: Dave Jones > --- > Documentation/devices.txt |2 -- > 1

[GIT PULL] battery-2.6.git

2013-05-25 Thread Anton Vorontsov
Hello Linus, Please pull battery-2.6 git tree to get a few one-liners: wrong kfree usage fix, module alias fixup and kconfig adjustments. The patches really small, so for convenience I inline them here. Thanks! Anton The following changes since commit 6b170807cd5cac8dc6353f47a88ccb14bbf76c4f:

[PATCH 1/1] regulator: lp397x: use devm_kzalloc() to make cleanup paths simpler

2013-05-25 Thread Nikolay Balandin
From: Nikolay Balandin Signed-off-by: Nikolay Balandin --- drivers/regulator/lp3971.c | 11 +++ drivers/regulator/lp3972.c | 11 +++ 2 files changed, 6 insertions(+), 16 deletions(-) diff --git a/drivers/regulator/lp3971.c b/drivers/regulator/lp3971.c index

Re: [PATCH RESEND 5/5] watchdog: sp805_wdt: use devm_clk_get()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_clk_get() to make cleanup paths more simple. > > Signed-off-by: Jingoo Han > --- > drivers/watchdog/sp805_wdt.c |7 ++- > 1 files changed, 2 insertions(+), 5 deletions(-) > Patch added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this

Re: [PATCH 0/2] kdump/mmap: Fix mmap of /proc/vmcore for s390

2013-05-25 Thread Eric W. Biederman
Zhang Yanfei writes: > 于 2013年05月25日 11:01, Eric W. Biederman 写道: >> Zhang Yanfei writes: >> >>> Hello Eric, >>> The function copy_oldmem_page also concerns me. I don't have a clue why we duplicate that function on every architecutre in a slightly different form. There should

Re: [PATCH RESEND 4/5] watchdog: shwdt: use devm_clk_get()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_clk_get() to make cleanup paths more simple. > > Signed-off-by: Jingoo Han > --- > drivers/watchdog/shwdt.c | 16 > 1 files changed, 4 insertions(+), 12 deletions(-) > Patch added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from

Re: [PATCH RESEND 3/5] watchdog: pnx4008_wdt: use devm_clk_get()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_clk_get() to make cleanup paths more simple. > > Signed-off-by: Jingoo Han > --- > drivers/watchdog/pnx4008_wdt.c |7 ++- > 1 files changed, 2 insertions(+), 5 deletions(-) > Patch added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this

Re: [PATCH v2] ARM: at91: Fix link breakage when !CONFIG_PHYLIB

2013-05-25 Thread Jean-Christophe PLAGNIOL-VILLARD
On 09:39 Sat 25 May , Alexandre Belloni wrote: > Fixes: > arch/arm/mach-at91/built-in.o: In function `ksz9021rn_phy_fixup': > :(.text+0x1174): undefined reference to `mdiobus_write' > :(.text+0x1188): undefined reference to `mdiobus_write' > :(.text+0x119c): undefined reference to

Re: [PATCH RESEND 2/5] watchdog: imx2_wdt: use devm_clk_get()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_clk_get() to make cleanup paths more simple. > > Signed-off-by: Jingoo Han > --- > drivers/watchdog/imx2_wdt.c |6 ++ > 1 files changed, 2 insertions(+), 4 deletions(-) > Patch added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list:

Re: [PATCH v2] ipc/sem.c: fix lockup, restore FIFO behavior

2013-05-25 Thread Manfred Spraul
On 05/25/2013 08:32 PM, Davidlohr Bueso wrote: Yep, could you please explain what benefits you see in keeping FIFO order? a) It's user space visible. b) It's a well-defined behavior that might even make sense for some applications. Right now, a 2 semop operation with "+1, then -2" is

[GIT PULL] Power management and ACPI fixes for v3.10-rc3

2013-05-25 Thread Rafael J. Wysocki
Hi Linus, Please pull from the git repository at git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git pm+acpi-3.10-rc3 to receive power management and ACPI fixes for v3.10-rc3 with top-most commit 5a2bff8d7c7f63e2e8d467f66f6492fbb4d9509c Merge branch 'pm-cpufreq' on top of

Re: [PATCH RESEND 1/5] watchdog: dw_wdt: use devm_clk_get()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_clk_get() to make cleanup paths more simple. > > Signed-off-by: Jingoo Han > --- > drivers/watchdog/dw_wdt.c |7 ++- > 1 files changed, 2 insertions(+), 5 deletions(-) > Patch added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list:

Re: [PATCH V3 7/7] watchdog: ts72xx_wdt: use devm_*() functions

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_*() functions to make cleanup paths simpler. > > Signed-off-by: Jingoo Han > --- > Changes since v2: > - removed unnecessary platform_set_drvdata(pdev, NULL) > > drivers/watchdog/ts72xx_wdt.c | 67 ++-- > 1 files changed, 10

Re: [PATCH 4/5] watchdog: riowd: use platform_{get,set}_drvdata()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use the wrapper functions for getting and setting the driver data using > platform_device instead of using dev_{get,set}_drvdata() with >dev, > so we can directly pass a struct platform_device. > > Signed-off-by: Jingoo Han > --- > drivers/watchdog/riowd.c |4 ++-- > 1 file

Re: [PATCH V2 6/7] watchdog: riowd: use devm_kzalloc()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_kzalloc() to make cleanup paths simpler. > > Reviewed-by: Guenter Roeck > Signed-off-by: Jingoo Han > --- > No changes since v1: > > drivers/watchdog/riowd.c |8 ++-- > 1 files changed, 2 insertions(+), 6 deletions(-) > Patch added to linux-watchdog-next.

Re: [PATCH 1/2] Initial support for Allwinner's Security ID fuses

2013-05-25 Thread Oliver Schinagl
On 05/25/13 14:22, Maxime Ripard wrote: Hi Oliver, On Fri, May 24, 2013 at 11:50:38PM +0200, Oliver Schinagl wrote: On 05/18/13 19:19, Oliver Schinagl wrote: + + +/* We read the entire key, using a look up table. Returned is only the + * requested byte. This is of course slower then it could

Re: [PATCH V2 5/7] watchdog: rc32434_wdt: use devm_ioremap_nocache() functions

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_ioremap_nocache() functions to make cleanup paths simpler. > > Reviewed-by: Guenter Roeck > Signed-off-by: Jingoo Han > --- > No changes since v1: > > drivers/watchdog/rc32434_wdt.c | 10 +++--- > 1 files changed, 3 insertions(+), 7 deletions(-) > Patch added to

Re: [PATCH V2 4/7] watchdog: nuc900_wdt: use devm_*() functions

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_*() functions to make cleanup paths simpler. > > Signed-off-by: Jingoo Han > --- > Changes since v1: > - used sizeof(*nuc900_wdt) instead of sizeof(struct nuc900_wdt) > - declared 'nuc900_wdt->res' variable as logical variable > > drivers/watchdog/nuc900_wdt.c | 50 >

Re: [PATCH V2 3/7] watchdog: mv64x60_wdt: use devm_ioremap()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_ioremap() to make cleanup paths simpler. > > Reviewed-by: Guenter Roeck > Signed-off-by: Jingoo Han > --- > No changes since v1: > > drivers/watchdog/mv64x60_wdt.c |4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) > Patch added to linux-watchdog-next.

Re: [PATCH V2 2/7] watchdog: bcm63xx_wdt: use devm_ioremap_nocache()

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_ioremap_nocache() to make cleanup paths simpler. > > Reviewed-by: Guenter Roeck > Signed-off-by: Jingoo Han > --- > No changes since v1: > > drivers/watchdog/bcm63xx_wdt.c |9 - > 1 files changed, 4 insertions(+), 5 deletions(-) Patch added to

Re: [PATCH V2 1/7] watchdog: at32ap700x_wdt: use devm_*() functions

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_*() functions to make cleanup paths simpler. > > Acked-by: Guenter Roeck > Signed-off-by: Jingoo Han Patch added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: Stupid VFS name lookup interface..

2013-05-25 Thread Casey Schaufler
On 5/25/2013 9:57 AM, Al Viro wrote: > On Fri, May 24, 2013 at 08:21:08PM -0700, Linus Torvalds wrote: >> On Tue, May 21, 2013 at 3:22 PM, Linus Torvalds >> wrote: >>> Untested patch attached. It compiles cleanly, looks sane, and most of >>> it is just making the function prototypes look much

Re: [PATCH v2] ipc/sem.c: fix lockup, restore FIFO behavior

2013-05-25 Thread Davidlohr Bueso
On Sat, 2013-05-25 at 13:55 -0400, Rik van Riel wrote: > On 05/25/2013 11:16 AM, Manfred Spraul wrote: > > The double coward solution: > > - wakeup stays FIFO > > - fast switch back to per-semaphore spinlock mode > > > > The patch > > a) fixes a lockup due to a missing restart. > > b) makes the

[GIT PULL] parisc fixes for 3.10-rc3

2013-05-25 Thread Helge Deller
Hi Linus, please pull the latest parisc architecture fixes. This time we made the kernel- and interruption stack allocation reentrant which fixed some strange kernel crashes (specifically protection ID traps). Furthemore this patchset fixes the interrupt stack in UP and SMP configurations by

Re: rcu_read_lock() used illegally while idle!

2013-05-25 Thread Steven Rostedt
On Sat, 2013-05-25 at 09:59 -0400, Dave Jones wrote: > On Fri, May 24, 2013 at 03:23:40PM -0400, Steven Rostedt wrote: > > On Fri, 2013-05-24 at 10:23 -0400, Dave Jones wrote: > > > > > .config: http://paste.fedoraproject.org/14281/94052971/raw/ > > > > > > trace shows the problem process

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Sylwester Nawrocki
On 05/25/2013 04:26 PM, Prabhakar Lad wrote: Thus it might make sense to have only following integer properties (added > as needed): > > composite-sync-active > sync-on-green-active > sync-on-comp-active > sync-on-luma-active > > This would allow to specify polarity of each signal and at

Re: [PATCH v2] ipc/sem.c: fix lockup, restore FIFO behavior

2013-05-25 Thread Rik van Riel
On 05/25/2013 11:16 AM, Manfred Spraul wrote: The double coward solution: - wakeup stays FIFO - fast switch back to per-semaphore spinlock mode The patch a) fixes a lockup due to a missing restart. b) makes the wakeups again FIFO (as linux <= 3.0.9) c) tries to limit the time while in global

Re: [PATCH v2 1/4] ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata

2013-05-25 Thread Prabhakar Lad
Hi Sekhar, On Sat, May 25, 2013 at 11:09 PM, Prabhakar Lad wrote: > From: Lad, Prabhakar > > remove init_enable from ths7303 pdata as it is being dropped > from ths7303_platform_data. > Can you please ack this patch as I intend to take this patch via media tree. Regards, --Prabhakar Lad -- To

[PATCH v2 4/4] media: i2c: ths7303: make the pdata as a constant pointer

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar generally the pdata needs to be a constant pointer in the device state structure. This patch makes the pdata as a constant pointer and alongside returns -EINVAL when pdata is NULL. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho

[PATCH v2 3/4] media: i2c: ths7303: remove unnecessary function ths7303_setup()

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar the ths7303_setup() was doing the same thing as ths7303_setval() except that ths7303_setval() sets it to some particular mode. This patch removes ths7303_setup() function and calls ths7303_setval() in the probe setting the device to 480I_576I filter mode in the probe.

[PATCH v2 2/4] media: i2c: ths7303: remove init_enable option from pdata

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar This patch removes init_enable option from pdata, the init_enable was intended that the device should start streaming video immediately but ideally the bridge drivers should call s_stream explicitly for such devices to start video. Signed-off-by: Lad, Prabhakar Cc: Hans

[PATCH v2 1/4] ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar remove init_enable from ths7303 pdata as it is being dropped from ths7303_platform_data. Signed-off-by: Lad, Prabhakar Cc: Sekhar Nori Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: linux-kernel@vger.kernel.org Cc:

[PATCH v2 0/4] media: i2c: ths7303 cleanup

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar Trivial cleanup of the driver. Changes for v2: 1: Dropped the asynchronous probing and, OF support patches will be handling them independently because of dependencies. 2: Arranged the patches logically so that git bisect succeeds. Lad, Prabhakar (4): ARM: davinci:

Re: Stupid VFS name lookup interface..

2013-05-25 Thread Linus Torvalds
On Sat, May 25, 2013 at 9:57 AM, Al Viro wrote: > > Well... The problem I see here is not even selinux per se - it's that > "LSM stacking" insanity. How would your anon union deal with that? Which > LSM gets to play with it when we have more than one of those turds around? We don't support

Re: [PATCH v5 2/3] extcon: Palmas Extcon Driver

2013-05-25 Thread Laxman Dewangan
Hi Graeme/Kishon, On Friday 24 May 2013 08:01 PM, Kishon Vijay Abraham I wrote: From: Graeme Gregory This is the driver for the USB comparator built into the palmas chip. It handles the various USB OTG events that can be generated by cable insertion/removal. I have following feedback on

Re: EnhanceIO(TM) caching driver features [1/3]

2013-05-25 Thread Amit Kale
On Saturday 25 May 2013, Jens Axboe wrote: > Please don't top post! Got to use a different email client for that. Note that I am writing this from my personal email address. This email and any future emails I write from this address are my personal views and sTec can't be held responsible for

Re: [PATCH 1/3] drivers: regulator: palmas: add an API to set/clear the switch bit on SMPS10

2013-05-25 Thread Laxman Dewangan
Hi Kishon/Graeme, On Friday 24 May 2013 08:01 PM, Kishon Vijay Abraham I wrote: From: Graeme Gregory Added an API to set/clear the switch bit on SMPS10 which can be used by palmas usb. The switch bit should be set in order for palmas to supply VBUS and is needed when OMAP is acting as USB

Re: [PATCH] usb: xhci: Disable runtime PM suspend for quirky controllers.

2013-05-25 Thread Shawn Nematbakhsh
Hi Sarah and Alan, Thanks for the comments. I will make the following revisions: 1. Call pm_runtime_get_noresume only when the first device is connected, and call pm_runtime_put when the last device is disconnected. 2. Wrap the calls in an ifdef CONFIG_USB_DEFAULT_PERSIST. 3. Make sure that all

Re: Stupid VFS name lookup interface..

2013-05-25 Thread Al Viro
On Fri, May 24, 2013 at 08:21:08PM -0700, Linus Torvalds wrote: > On Tue, May 21, 2013 at 3:22 PM, Linus Torvalds > wrote: > > > > Untested patch attached. It compiles cleanly, looks sane, and most of > > it is just making the function prototypes look much nicer. I think it > > works. > > Ok,

[PATCH] ioatdma: check dma_mapping_error in ioat_dma_self_test

2013-05-25 Thread Kyle McMartin
Not checking dma_mapping_error on these dma_map_single uses keeps showing up in our test runs with DMA-API debugging enabled. [ 19.744756] ioatdma: Intel(R) QuickData Technology Driver 4.00 [ 19.748979] [ cut here ] [ 19.748991] WARNING: at lib/dma-debug.c:933

[PATCH] score: remove redundant kcore_list entries

2013-05-25 Thread Kyle McMartin
kcore_vmalloc is in fs/proc/kcore.c and kcore_mem is unused across the tree. Noticed while grepping the tree for some other kcore stuff. (score looks pretty unmaintained to me.) Signed-off-by: Kyle McMartin --- a/arch/score/mm/init.c +++ b/arch/score/mm/init.c @@ -41,8 +41,6 @@ unsigned long

[PATCH v2 5/5] media: davinci: vpif_display: Convert to devm_* api

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar use devm_request_irq() instead of request_irq(). This ensures more consistent error values and simplifies error paths. use module_platform_driver to simplify the code. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif_display.c | 61

[PATCH v2 4/5] media: davinci: vpif_capture: Convert to devm_* api

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar use devm_request_irq() instead of request_irq(). This ensures more consistent error values and simplifies error paths. use module_platform_driver to simplify the code. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif_capture.c | 73

[PATCH v2 3/5] media: davinci: vpif: remove unnecessary braces around defines

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/davinci/vpif.c b/drivers/media/platform/davinci/vpif.c index 164c1b7..d9269c9 100644 ---

[PATCH v2 2/5] media: davinci: vpif: Convert to devm_* api

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar Use devm_ioremap_resource instead of reques_mem_region()/ioremap(). This ensures more consistent error values and simplifies error paths. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif.c | 27 --- 1 files changed, 4

[PATCH v2 1/5] media: davinci: vpif: remove unwanted header mach/hardware.h and sort the includes alphabetically

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar This patch removes unwanted header include of mach/hardware.h and along side sorts the header inclusion alphabetically. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif.c | 10 -- 1 files changed, 4 insertions(+), 6 deletions(-) diff --git

[PATCH v2 0/5] media: davinci: vpif trivial cleanup

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar This patch series cleans the VPIF driver, uses devm_* api wherever required and uses module_platform_driver() to simplify the code. This patch series applies on http://git.linuxtv.org/hverkuil/media_tree.git/ shortlog/refs/heads/for-v3.11 and is tested on OMAP-L138.

[PATCH 1/1] rtc: rtc-ds1307: use devm_*() functions

2013-05-25 Thread Nikolay Balandin
From: Nikolay Balandin Use devm_*() functions to make cleanup paths simpler. Signed-off-by: Nikolay Balandin --- drivers/rtc/rtc-ds1307.c | 38 +++--- 1 file changed, 15 insertions(+), 23 deletions(-) diff --git a/drivers/rtc/rtc-ds1307.c

  1   2   3   4   >