Re: Cannot hot remove a memory device

2013-08-05 Thread Yasuaki Ishimatsu
(2013/08/05 13:00), Yasuaki Ishimatsu wrote: (2013/08/04 9:37), Toshi Kani wrote: On Sat, 2013-08-03 at 03:01 +0200, Rafael J. Wysocki wrote: On Friday, August 02, 2013 06:04:40 PM Toshi Kani wrote: On Sat, 2013-08-03 at 01:43 +0200, Rafael J. Wysocki wrote: On Friday, August 02, 2013

Re: [PATCH] zram: bug fix: delay lock holding in zram_slot_free_noity

2013-08-05 Thread Greg Kroah-Hartman
On Mon, Aug 05, 2013 at 04:18:34PM +0900, Minchan Kim wrote: I was preparing to promote zram and it was almost done. Before sending patch, I tried to test and eyebrows went up. [1] introduced down_write in zram_slot_free_notify to prevent race between zram_slot_free_notify and

About TRB_TO_EP_INDEX() macro using

2013-08-05 Thread Du, ChangbinX
Recently when I check xHCI code, find that some functions try to get EP index from a Command Completion Event TRB via TRB_TO_EP_INDEX() macro. This is totally wrong. The macro definition is: #define TRB_TO_EP_INDEX(p) p) (0x1f 16)) 16) - 1) TRB_TO_EP_INDEX only

Re: [PATCH 1/4] mm, page_alloc: add likely macro to help compiler optimization

2013-08-05 Thread Joonsoo Kim
Hello, Michal. On Fri, Aug 02, 2013 at 11:36:07PM +0200, Michal Hocko wrote: On Fri 02-08-13 16:47:10, Johannes Weiner wrote: On Fri, Aug 02, 2013 at 06:27:22PM +0200, Michal Hocko wrote: On Fri 02-08-13 11:07:56, Joonsoo Kim wrote: We rarely allocate a page with ALLOC_NO_WATERMARKS and

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Ingo Molnar
* Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com By default perf currently links with the GTK2 gui. This pulls in a lot of external libraries. It also causes dependency problems for distribution packages: simply installing perf requires pulling in GTK2 with

stable: please queue commit 0231bb5 (perf: Fix event group context move)

2013-08-05 Thread Li Zefan
Plese consider adding commit 0231bb5336758426b44ccd798ccd3c5419c95d58 to stable tree. It fixes a real bug, which was explained here: http://thread.gmane.org/gmane.linux.kernel.perf.user/1144 and I've managed to reproduce the bug in 3.4 kernel with the test program attached in the above

Re: [PATCH 02/13] tracing/kprobes: Add fetch{,_size} member into symbol and deref fetch method

2013-08-05 Thread Masami Hiramatsu
(2013/07/31 18:03), Namhyung Kim wrote: From: Hyeoncheol Lee cheol@lge.com Those symbol and deref fetch methods access a memory region but they assume it's a kernel memory since uprobes does not support them. Add -fetch and -fetch_size member in order to provide a proper access method

[PATCH] sound/sgtl5000: fix codec clock source transition to avoid clockless moment

2013-08-05 Thread Oskar Schirmer
Powering down PLL before switching to a mode that does not use it is a bad idea. It would cause the SGTL5000 be without internal clock supply, especially on the I2C interface, which would make subsequent access to it fail. Thus, in case of not using PLL any longer, first set the mode control,

Re: stable: please queue commit 0231bb5 (perf: Fix event group context move)

2013-08-05 Thread Ingo Molnar
* Li Zefan lize...@huawei.com wrote: Plese consider adding commit 0231bb5336758426b44ccd798ccd3c5419c95d58 to stable tree. It fixes a real bug, which was explained here: http://thread.gmane.org/gmane.linux.kernel.perf.user/1144 and I've managed to reproduce the bug in 3.4 kernel

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Christoph Hellwig
On Mon, Aug 05, 2013 at 10:16:41AM +0200, Ingo Molnar wrote: If you want fewer dependencies then build with 'make NO_GTK=1'. Doesn't help the distros. Installing perf and pulling all the graphics libraries in is highly annoying, especially in size constrained VM or Cloud images. Having a

Re: [PATCH] perf: sample after exit loses thread correlation - v3

2013-08-05 Thread Adrian Hunter
On 02/08/13 06:52, David Ahern wrote: Occassionally events (e.g., context-switch, sched tracepoints) 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 selected events shown ls 30482 [000]

Re: [PATCH] zram: bug fix: delay lock holding in zram_slot_free_noity

2013-08-05 Thread Minchan Kim
Hello Greg, On Mon, Aug 05, 2013 at 04:04:22PM +0800, Greg Kroah-Hartman wrote: On Mon, Aug 05, 2013 at 04:18:34PM +0900, Minchan Kim wrote: I was preparing to promote zram and it was almost done. Before sending patch, I tried to test and eyebrows went up. [1] introduced down_write in

Re: O_TMPFILE fs corruption (Re: Linux 3.11-rc4)

2013-08-05 Thread Christoph Hellwig
On Sun, Aug 04, 2013 at 08:45:16PM -0700, Linus Torvalds wrote: The patch looks right to me - we should pass in similar flags for the create case as for tmpfile to the filesystem. But let's make sure we're all on the same page. Al? Given all the problems and very limited fs support I'd much

Re: [PATCH 2/5] tools, perf: Add support to evsel for enabling counters

2013-08-05 Thread Namhyung Kim
Hi Andi, On Fri, 2 Aug 2013 17:41:10 -0700, Andi Kleen wrote: From: Andi Kleen a...@linux.intel.com Add support for enabling already set up counters by using an ioctl. I share some code with the filter setup. Signed-off-by: Andi Kleen a...@linux.intel.com --- tools/perf/util/evsel.c |

Re: [PATCH] perf x86: Fix compile of util/tsc.c

2013-08-05 Thread Adrian Hunter
On 26/07/13 17:27, David Ahern wrote: On Fedora 18, with gcc 4.6.4 compile fails with: arch/x86/util/tsc.c: In function ‘perf_time_to_tsc’: arch/x86/util/tsc.c:13:6: error: declaration of ‘time’ shadows a global declaration [-Werror=shadow] cc1: all warnings being treated as errors make:

[PATCH 0/2] cma: do some clean up

2013-08-05 Thread Xishi Qiu
1. use PFN_DOWN(r-size) instead of r-size PAGE_SHIFT. 2. adjust the function structure, one for the success path, the other for the failure path. Xishi Qiu (2): cma: use macro PFN_DOWN when converting size to pages cma: adjust goto branch in function cma_create_area()

[PATCH 2/2] cma: adjust goto branch in function cma_create_area()

2013-08-05 Thread Xishi Qiu
Adjust the function structure, one for the success path, the other for the failure path. Signed-off-by: Xishi Qiu qiuxi...@huawei.com --- drivers/base/dma-contiguous.c | 16 +--- 1 files changed, 9 insertions(+), 7 deletions(-) diff --git a/drivers/base/dma-contiguous.c

[PATCH 1/2] cma: use macro PFN_DOWN when converting size to pages

2013-08-05 Thread Xishi Qiu
Use PFN_DOWN(r-size) instead of r-size PAGE_SHIFT. Signed-off-by: Xishi Qiu qiuxi...@huawei.com --- drivers/base/dma-contiguous.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/base/dma-contiguous.c b/drivers/base/dma-contiguous.c index 0ca5442..1bcfaed

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Ingo Molnar
* Christoph Hellwig h...@infradead.org wrote: On Mon, Aug 05, 2013 at 10:16:41AM +0200, Ingo Molnar wrote: If you want fewer dependencies then build with 'make NO_GTK=1'. Doesn't help the distros. Installing perf and pulling all the graphics libraries in is highly annoying, especially

Re: About TRB_TO_EP_INDEX() macro using

2013-08-05 Thread gre...@linuxfoundation.org
On Mon, Aug 05, 2013 at 08:07:25AM +, Du, ChangbinX wrote: Recently when I check xHCI code, find that some functions try to get EP index from a Command Completion Event TRB via TRB_TO_EP_INDEX() macro. This is totally wrong. The macro definition is: #define TRB_TO_EP_INDEX(p)

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Christoph Hellwig
On Mon, Aug 05, 2013 at 10:31:32AM +0200, Ingo Molnar wrote: Nonsense, a distro, if it truly worried about this, could create two packages already, there's no need to expose configuration options in the binary name itself and burden users with the separation. I sometimes switch the UI

Re: [PATCH] i2c: Fix Kontron PLD prescaler calculation

2013-08-05 Thread Wolfram Sang
On Fri, Jul 26, 2013 at 02:04:55PM +0200, Michael Brunner wrote: Add some necessary braces that have been removed during driver cleanup. This fixes the I2C prescaler calculation. Signed-off-by: Michael Brunner michael.brun...@kontron.com Applied to for-current, thanks! signature.asc

Re: [PATCH] perf x86: Fix compile of util/tsc.c

2013-08-05 Thread Jiri Olsa
On Mon, Aug 05, 2013 at 11:34:57AM +0300, Adrian Hunter wrote: On 26/07/13 17:27, David Ahern wrote: On Fedora 18, with gcc 4.6.4 compile fails with: arch/x86/util/tsc.c: In function ‘perf_time_to_tsc’: arch/x86/util/tsc.c:13:6: error: declaration of ‘time’ shadows a global

Warnings from silentoldconfig (Re: [GIT PULL] PCI updates for v3.11)

2013-08-05 Thread Michal Marek
On Fri, Aug 02, 2013 at 01:28:25PM -0700, Linus Torvalds wrote: On Fri, Aug 2, 2013 at 11:17 AM, Bjorn Helgaas bhelg...@google.com wrote: Yinghai is still working on another SR-IOV-related fix or two, which will be simpler if pciehp is non-modular, so I included the Kconfig changes now to

Re: [PATCHv3] bridge: disable snooping if there is no querier

2013-08-05 Thread Paul Bolle
On Wed, 2013-07-31 at 17:40 -0700, David Miller wrote: If there is no querier on a link then we won't get periodic reports and therefore won't be able to learn about multicast listeners behind ports, potentially leading to lost multicast packets, especially for multicast listeners that

Re: [PATCH 01/13] tracing/kprobes: Move fetch functions to trace_kprobe.c

2013-08-05 Thread Namhyung Kim
Hi Masami, On Mon, 05 Aug 2013 14:30:35 +0900, Masami Hiramatsu wrote: (2013/07/31 18:03), Namhyung Kim wrote: From: Hyeoncheol Lee cheol@lge.com Move kprobes-specific fetch functions to the trace_kprobe.c file. Also define kprobes_fetch_type_table in the .c file. This table is shared

[PATCH] doc: fix a typo about irq affinity

2013-08-05 Thread Hu Tao
smp_affinity holds bitmask and smp_affinity_list holds list. So we should write a list to smp_affinity_list, instead of smp_affinity. Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- Documentation/IRQ-affinity.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 09/13] tracing/uprobes: Fix a comment for uprobe registration syntax

2013-08-05 Thread Namhyung Kim
On Mon, 05 Aug 2013 14:59:59 +0900, Masami Hiramatsu wrote: (2013/07/31 18:03), Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com The uprobe syntax requires an offset after a file path not a symbol. Is this a bugfix? If so, IMHO, it should be applied earlier in the series.

Re: [PATCH 04/13] tracing/kprobes: Factor out struct trace_probe

2013-08-05 Thread Namhyung Kim
On Mon, 05 Aug 2013 15:00:25 +0900, Masami Hiramatsu wrote: (2013/07/31 18:03), Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com There are functions that can be shared to both of kprobes and uprobes. Separate common data structure to struct trace_probe and use it from the shared

Re: [PATCH 06/13] tracing/kprobes: Move common functions to trace_probe.c

2013-08-05 Thread Namhyung Kim
On Mon, 05 Aug 2013 15:03:48 +0900, Masami Hiramatsu wrote: (2013/07/31 18:03), Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com The __get_data_size() and store_trace_args() will be used by uprobes too. Move them to a common location. Hmm, could you move this into

Re: [PATCH 1/4] mm, page_alloc: add likely macro to help compiler optimization

2013-08-05 Thread Joonsoo Kim
On Mon, Aug 05, 2013 at 05:10:08PM +0900, Joonsoo Kim wrote: Hello, Michal. On Fri, Aug 02, 2013 at 11:36:07PM +0200, Michal Hocko wrote: On Fri 02-08-13 16:47:10, Johannes Weiner wrote: On Fri, Aug 02, 2013 at 06:27:22PM +0200, Michal Hocko wrote: On Fri 02-08-13 11:07:56, Joonsoo

Re: [PATCH 13/13] tracing/probes: Fix basic print type functions

2013-08-05 Thread Namhyung Kim
On Mon, 05 Aug 2013 15:22:10 +0900, Masami Hiramatsu wrote: (2013/07/31 18:03), Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com The print format of s32 type was ld and it's casted to long. So it turned out to print 4294967295 for -1 on 64-bit systems. Not sure whether it

Re: [PATCH 1/2] cma: use macro PFN_DOWN when converting size to pages

2013-08-05 Thread Greg Kroah-Hartman
On Mon, Aug 05, 2013 at 04:31:00PM +0800, Xishi Qiu wrote: Use PFN_DOWN(r-size) instead of r-size PAGE_SHIFT. Signed-off-by: Xishi Qiu qiuxi...@huawei.com --- drivers/base/dma-contiguous.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git

Re: [PATCH 2/2] cma: adjust goto branch in function cma_create_area()

2013-08-05 Thread Greg Kroah-Hartman
On Mon, Aug 05, 2013 at 04:31:07PM +0800, Xishi Qiu wrote: Adjust the function structure, one for the success path, the other for the failure path. Signed-off-by: Xishi Qiu qiuxi...@huawei.com --- drivers/base/dma-contiguous.c | 16 +--- 1 files changed, 9 insertions(+), 7

Re: [PATCH 13/13] tracing/probes: Fix basic print type functions

2013-08-05 Thread Namhyung Kim
Hi Joe, On Sun, 04 Aug 2013 23:31:57 -0700, Joe Perches wrote: On Mon, 2013-08-05 at 15:22 +0900, Masami Hiramatsu wrote: (2013/07/31 18:03), Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com The print format of s32 type was ld and it's casted to long. So it turned out to

Re: [PATCH 02/13] tracing/kprobes: Add fetch{,_size} member into symbol and deref fetch method

2013-08-05 Thread Namhyung Kim
On Mon, 05 Aug 2013 17:19:50 +0900, Masami Hiramatsu wrote: (2013/07/31 18:03), Namhyung Kim wrote: From: Hyeoncheol Lee cheol@lge.com Those symbol and deref fetch methods access a memory region but they assume it's a kernel memory since uprobes does not support them. Add -fetch and

Re: [PATCH 1/4] mm, page_alloc: add likely macro to help compiler optimization

2013-08-05 Thread Michal Hocko
On Mon 05-08-13 17:50:41, Joonsoo Kim wrote: [...] IMHO, although there is no effect, it is better to add likely macro, because arrangement can be changed from time to time without any consideration of assembly code generation. How about your opinion, Johannes and Michal? This is a matter of

Re: [PATCH] i2c: add sanity check to i2c_put_adapter

2013-08-05 Thread Wolfram Sang
On Thu, Aug 01, 2013 at 02:10:46PM +0200, Sebastian Hesselbarth wrote: i2c_put_adapter dereferences i2c_adapter pointer passed without check for NULL. This adds a check for non-NULL pointer to allow i2c_put_adapter called with NULL and behave the same way i2c_release_client does already. What

Re: [PATCH trivial] include/linux/coda.h: remove useless '#else'

2013-08-05 Thread Chen Gang
On 08/05/2013 10:12 AM, Chen Gang wrote: On 08/05/2013 09:53 AM, Chen Gang wrote: On 08/05/2013 09:21 AM, Joe Perches wrote: On Mon, 2013-08-05 at 08:24 +0800, Chen Gang wrote: On 07/31/2013 11:16 AM, Chen Gang F T wrote: On 07/31/2013 09:53 AM, Chen Gang wrote: On 07/31/2013 09:44 AM, Chen

Re: [PATCH 3/4] dts: dra7-evm: add USB support

2013-08-05 Thread George Cherian
On 8/5/2013 1:25 PM, Roger Quadros wrote: On 08/02/2013 06:04 PM, George Cherian wrote: On 8/1/2013 8:28 PM, Roger Quadros wrote: Add USB drv_vbus pinctrl information and USB mode for the USB controller. Signed-off-by: Roger Quadros rog...@ti.com --- arch/arm/boot/dts/dra7-evm.dts | 22

Re: [PATCH 03/13] tracing/kprobes: Make stack and memory fetch functions static

2013-08-05 Thread Masami Hiramatsu
(2013/07/31 18:03), Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com Those fetch functions need to be implemented differently for kprobes and uprobes. Since the symbol and deref fetch functions don't call those directly anymore, we can make them static and implement them

Re: [PATCH 1/2] cma: use macro PFN_DOWN when converting size to pages

2013-08-05 Thread Xishi Qiu
On 2013/8/5 16:54, Greg Kroah-Hartman wrote: On Mon, Aug 05, 2013 at 04:31:00PM +0800, Xishi Qiu wrote: Use PFN_DOWN(r-size) instead of r-size PAGE_SHIFT. Signed-off-by: Xishi Qiu qiuxi...@huawei.com --- drivers/base/dma-contiguous.c |5 ++--- 1 files changed, 2 insertions(+), 3

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Namhyung Kim
Hi, On Mon, 5 Aug 2013 01:23:20 -0700, Christoph Hellwig wrote: On Mon, Aug 05, 2013 at 10:16:41AM +0200, Ingo Molnar wrote: If you want fewer dependencies then build with 'make NO_GTK=1'. Doesn't help the distros. Installing perf and pulling all the graphics libraries in is highly

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Ingo Molnar
* Christoph Hellwig h...@infradead.org wrote: On Mon, Aug 05, 2013 at 10:31:32AM +0200, Ingo Molnar wrote: Nonsense, a distro, if it truly worried about this, could create two packages already, there's no need to expose configuration options in the binary name itself and burden users

[PATCH V2] cma: use macro PFN_DOWN when converting size to pages

2013-08-05 Thread Xishi Qiu
Use PFN_DOWN(r-size) instead of r-size PAGE_SHIFT. Signed-off-by: Xishi Qiu qiuxi...@huawei.com --- drivers/base/dma-contiguous.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/base/dma-contiguous.c b/drivers/base/dma-contiguous.c index 0ca5442..b3d711d

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Ingo Molnar
* Namhyung Kim namhy...@kernel.org wrote: Hi, On Mon, 5 Aug 2013 01:23:20 -0700, Christoph Hellwig wrote: On Mon, Aug 05, 2013 at 10:16:41AM +0200, Ingo Molnar wrote: If you want fewer dependencies then build with 'make NO_GTK=1'. Doesn't help the distros. Installing perf and

[PATCH 1/1] extcon: arizona: Fix a typo in a comment

2013-08-05 Thread Sachin Kamat
Fixes an incomplete comment introduced by commit 9c2ba270 (extcon: arizona: Simplify HPDET based identification). Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/extcon/extcon-arizona.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-05 Thread Pekka Enberg
On Mon, Aug 5, 2013 at 12:12 PM, Ingo Molnar mi...@kernel.org wrote: Assuming that in the normal case it can be made to work like a full build of perf today (i.e. without forcing LD_PRELOAD) then splitting the libraries away looks like a much better solution than splitting the binary. Yup, if

[PATCH 1/2] extcon: max8997: Fix checkpatch warning

2013-08-05 Thread Sachin Kamat
Fixes the following warning: WARNING: space prohibited before semicolon Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/extcon/extcon-max8997.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c

Re: [patch 5/7] mm: memcg: enable memcg OOM killer only for user faults

2013-08-05 Thread Michal Hocko
On Sat 03-08-13 12:59:58, Johannes Weiner wrote: System calls and kernel faults (uaccess, gup) can handle an out of memory situation gracefully and just return -ENOMEM. Enable the memcg OOM killer only for user faults, where it's really the only option available. Signed-off-by: Johannes

[PATCH 2/2] extcon: max77693: Fix checkpatch warning

2013-08-05 Thread Sachin Kamat
Fixes the following warning: WARNING: space prohibited before semicolon Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/extcon/extcon-max77693.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/extcon/extcon-max77693.c

Re: i915 INFO: trying to register non-static key.

2013-08-05 Thread Borislav Petkov
On Mon, Aug 05, 2013 at 01:06:35AM +0200, Daniel Vetter wrote: On Wed, Jul 31, 2013 at 6:22 PM, Borislav Petkov b...@alien8.de wrote: Dudes, has anyone already reported this (happens on Linus of today + tip/master): I think this should be fixed with commit

Re: Warnings from silentoldconfig (Re: [GIT PULL] PCI updates for v3.11)

2013-08-05 Thread Yann E. MORIN
Michal, All, On Monday 05 August 2013 10:39:43 Michal Marek wrote: On Fri, Aug 02, 2013 at 01:28:25PM -0700, Linus Torvalds wrote: On Fri, Aug 2, 2013 at 11:17 AM, Bjorn Helgaas bhelg...@google.com wrote: Yinghai is still working on another SR-IOV-related fix or two, which will be

[PATCH] regmap: cache: Make sure to sync the last register in a block

2013-08-05 Thread Lars-Peter Clausen
regcache_sync_block_raw_flush() expects the address of the register after last register that needs to be synced as its parameter. But the last call to regcache_sync_block_raw_flush() in regcache_sync_block_raw() passes the address of the last register in the block. This effectively always skips

Re: [PATCH] i2c: add sanity check to i2c_put_adapter

2013-08-05 Thread Sebastian Hesselbarth
On 08/05/13 11:00, Wolfram Sang wrote: On Thu, Aug 01, 2013 at 02:10:46PM +0200, Sebastian Hesselbarth wrote: i2c_put_adapter dereferences i2c_adapter pointer passed without check for NULL. This adds a check for non-NULL pointer to allow i2c_put_adapter called with NULL and behave the same way

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-08-05 Thread Christian Ruppert
On Wed, Jul 24, 2013 at 11:31:44PM +0900, Shinya Kuribayashi wrote: On 7/22/13 10:17 PM, Christian Ruppert wrote: On Wed, Jul 17, 2013 at 11:39:58PM +0900, Shinya Kuribayashi wrote: On 7/16/13 8:16 PM, Christian Ruppert wrote: On Sat, Jul 13, 2013 at 02:36:43PM +0900, Shinya Kuribayashi

RE: [PATCH v2 3/3] dma: Add Freescale eDMA engine driver support

2013-08-05 Thread Lu Jingchang-B35083
-Original Message- From: Lothar Waßmann [mailto:l...@karo-electronics.de] Sent: Monday, August 05, 2013 3:54 PM To: Lu Jingchang-B35083 Cc: vinod.k...@intel.com; Li Xiaochun-B41219; Wang Huan-B18965; linux- ker...@vger.kernel.org; d...@fb.com; shawn@linaro.org; linux-arm-

Re: Build breakage due to latest ARM fixes

2013-08-05 Thread Jonathan Austin
On 04/08/13 20:36, Uwe Kleine-König wrote: Hello, On Sun, Aug 04, 2013 at 11:47:04AM -0700, Olof Johansson wrote: On Sun, Aug 4, 2013 at 11:37 AM, Russell King - ARM Linux The only real testing I'm aware of is when I recreated the OKI67001 support a while back and got my board to boot. Uwe

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Ingo Molnar
* Gleb Natapov g...@redhat.com wrote: On Fri, Aug 02, 2013 at 11:25:39AM +0200, Ingo Molnar wrote: Ingo, Do you have any concerns reg this series? please let me know if this looks good now to you. I'm inclined to NAK it for excessive quotation - who knows how many people

Re: Misc perf stat improvements

2013-08-05 Thread Jiri Olsa
On Fri, Aug 02, 2013 at 05:41:08PM -0700, Andi Kleen wrote: Here are a couple of perf stat improvements/cleanups: - output more information (ratios) in CSV mode - add --initial-delay to skip startup phase of program - handle pipes better in interval mode - some cleanup for the patchset:

Re: [PATCH 5/5] perf, tools: Output running time and run/enabled ratio in CSV mode

2013-08-05 Thread Jiri Olsa
On Fri, Aug 02, 2013 at 05:41:13PM -0700, Andi Kleen wrote: From: Andi Kleen a...@linux.intel.com The information how much a counter ran in perf stat can be quite interesting for other tools to judge how trustworthy a measurement is. Currently it is only output in non CSV mode. This

Re: [patch 7/7] mm: memcg: do not trap chargers with full callstack on OOM

2013-08-05 Thread Michal Hocko
On Sat 03-08-13 13:00:00, Johannes Weiner wrote: The memcg OOM handling is incredibly fragile and can deadlock. When a task fails to charge memory, it invokes the OOM killer and loops right there in the charge code until it succeeds. Comparably, any other task that enters the charge path at

Re: [PATCH 1/2] ARM: remove dmacap,memset from Device tree binding

2013-08-05 Thread Mark Rutland
On Sun, Aug 04, 2013 at 02:09:04AM +0100, Jason Cooper wrote: DT Maintainers, It's been a week with no comment. Shall I assume it's ok to apply this? Apologies for the delay, still catching up on binding review. I'm not entirely sure on this. The property described is still true for the

Re: [RFC v2 0/5] Transparent on-demand struct page initialization embedded in the buddy allocator

2013-08-05 Thread Ingo Molnar
* Nathan Zimmer nzim...@sgi.com wrote: We are still restricting ourselves ourselves to 2MiB initialization to keep the patch set a little smaller and more clear. We are still struggling with the expand(). Nearly always the first reference to a struct page which is in the middle of the

[PATCH 4/6] Staging: winbond: mds: fixed some lines over 80 characters

2013-08-05 Thread Iker Pedrosa
Fixed some coding style issues Signed-off-by: Iker Pedrosa ikerpedro...@gmail.com --- drivers/staging/winbond/mds.c | 56 +-- 1 file changed, 33 insertions(+), 23 deletions(-) diff --git a/drivers/staging/winbond/mds.c b/drivers/staging/winbond/mds.c

[PATCH 5/6] Staging: winbond: mds_f: fixed lines over 80 characters

2013-08-05 Thread Iker Pedrosa
Fixed some coding style issues. Signed-off-by: Iker Pedrosa ikerpedro...@gmail.com --- drivers/staging/winbond/mds_f.h | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/drivers/staging/winbond/mds_f.h b/drivers/staging/winbond/mds_f.h index ce8be07..ce5a78c 100644

[PATCH 6/6] Staging: winbond: mds_f: deleted extern function

2013-08-05 Thread Iker Pedrosa
Deleted an extern function that wasn't used in this driver. Signed-off-by: Iker Pedrosa ikerpedro...@gmail.com --- drivers/staging/winbond/mds_f.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/winbond/mds_f.h b/drivers/staging/winbond/mds_f.h index ce5a78c..159b2eb 100644

[PATCH 2/6] Staging: winbond: wb35reg: white space deleted

2013-08-05 Thread Iker Pedrosa
White space deleted before a semicolon. Signed-off-by: Iker Pedrosa ikerpedro...@gmail.com --- drivers/staging/winbond/wb35reg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/winbond/wb35reg.c b/drivers/staging/winbond/wb35reg.c index 34a2618..1314f93

[PATCH 3/6] Staging: winbond: wb35reg: avoided use of extern function

2013-08-05 Thread Iker Pedrosa
Prototype of a function added to the header to avoid the use of extern. Signed-off-by: Iker Pedrosa ikerpedro...@gmail.com --- drivers/staging/winbond/phy_calibration.h | 1 + drivers/staging/winbond/wb35reg.c | 3 +-- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 0/6] Staging: winbond: Fixed some coding style issues

2013-08-05 Thread Iker Pedrosa
Fixes some coding style issues from drivers/staging/winbond Iker Pedrosa (6): Staging: winbond: wb35reg: changed sleep function from msleep to usleep Staging: winbond: wb35reg: white space deleted Staging: winbond: wb35reg: avoided use of extern function Staging: winbond: mds: fixed

[PATCH 1/6] Staging: winbond: wb35reg: changed sleep function from msleep to usleep

2013-08-05 Thread Iker Pedrosa
Changed sleep function from msleep to usleep according to the documentation. Signed-off-by: Iker Pedrosa ikerpedro...@gmail.com --- drivers/staging/winbond/wb35reg.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/winbond/wb35reg.c

[tip:perf/urgent] perf/x86: Fix intel QPI uncore event definitions

2013-08-05 Thread tip-bot for Vince Weaver
Commit-ID: c9601247f8f3fdc18aed7ed7e490e8dfcd07f122 Gitweb: http://git.kernel.org/tip/c9601247f8f3fdc18aed7ed7e490e8dfcd07f122 Author: Vince Weaver vincent.wea...@maine.edu AuthorDate: Fri, 2 Aug 2013 10:47:34 -0400 Committer: Ingo Molnar mi...@kernel.org CommitDate: Mon, 5 Aug 2013

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-08-05 Thread Wolfram Sang
Would it make sense to add generic I2C device tree properties for those parameters? These parameters are independent of the actual bus driver, rather a PCB property... And as such the correct place would be device tree or ACPI or similar. If there are other bus drivers that make use

Re: [PATCH] staging: ozwpan: Fix build warning.

2013-08-05 Thread Rupesh Gujare
On 04/08/13 19:17, Anca Emanuel wrote: Why do not use div_u64() ? [quote]unsigned 64bit divide with 32bit divisor This is the most common 64bit divide and should be used if possible, as many 32bit archs can optimize this variant better than a full 64bit divide. [/quote] -

Re: [PATCH] mtd: jedec_probe: fix LH28F640BF definition

2013-08-05 Thread Artem Bityutskiy
On Sun, 2013-07-21 at 01:07 +0200, Andrea Adami wrote: Zaurus 5500 contains 2 LH28F640BFHE-PTTL90 (64M 4Mx16) and the LH28F640BFHE-PTTL90.pdf datasheet available on the net shows the exact erasesize and the OTP support. At the moment only jedec_probe can discover the chip and the NOR is

Re: [PATCH v7] watchdog: Add MOXA ART watchdog driver

2013-08-05 Thread Mark Rutland
On Fri, Aug 02, 2013 at 05:39:49PM +0100, Guenter Roeck wrote: On 08/02/2013 04:41 AM, Mark Rutland wrote: On Mon, Jul 29, 2013 at 01:33:48PM +0100, Jonas Jensen wrote: Add watchdog driver for MOXA ART SoCs. [ ... ] + + err = watchdog_register_device(moxart_wdt-dev); + if

Re: [PATCH 3/4] dts: dra7-evm: add USB support

2013-08-05 Thread Roger Quadros
On 08/05/2013 12:03 PM, George Cherian wrote: On 8/5/2013 1:25 PM, Roger Quadros wrote: On 08/02/2013 06:04 PM, George Cherian wrote: On 8/1/2013 8:28 PM, Roger Quadros wrote: Add USB drv_vbus pinctrl information and USB mode for the USB controller. Signed-off-by: Roger Quadros

Re: [PATCH v2 2/3] ARM: dts: vf610: Add eDMA node

2013-08-05 Thread Sascha Hauer
On Mon, Aug 05, 2013 at 02:07:03PM +0800, Jingchang Lu wrote: Signed-off-by: Jingchang Lu b35...@freescale.com --- changes in v2: using generic dma-channels property instead of fsl,dma-channels. diff --git a/include/dt-bindings/dma/vf610-edma.h b/include/dt-bindings/dma/vf610-edma.h

Re: [PATCH v2 1/1] mfd: palmas: Add power off control

2013-08-05 Thread Bill Huang
On Fri, 2013-08-02 at 22:39 +0800, Nishanth Menon wrote: On 08/02/2013 12:45 AM, Bill Huang wrote: On Thu, 2013-08-01 at 21:08 +0800, Nishanth Menon wrote: On 04:08-20130801, Bill Huang wrote: On Wed, 2013-07-31 at 19:57 +0800, Nishanth Menon wrote: If you notice the reference code I

Re: [alsa-devel] [PATCH v3] ASoC: core: init delayed_work for codec-codec links

2013-08-05 Thread Mark Brown
On Fri, Aug 02, 2013 at 11:51:56AM +0100, Richard Fitzgerald wrote: +static void codec2codec_close_delayed_work(struct work_struct *work) +{ +} + Having a commment here was a really good idea since this is the sort of thing I'd expect someone to come along and try to clean up otherwise. I'd

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Raghavendra K T
That result was only for patch 18 of the series, not pvspinlock in general. Okay - I've re-read the performance numbers and they are impressive, so no objections from me. The x86 impact seems to be a straightforward API change, with most of the changes on the virtualization side. So:

Re: [PATCH] i2c: add sanity check to i2c_put_adapter

2013-08-05 Thread Wolfram Sang
On Mon, Aug 05, 2013 at 11:31:42AM +0200, Sebastian Hesselbarth wrote: On 08/05/13 11:00, Wolfram Sang wrote: On Thu, Aug 01, 2013 at 02:10:46PM +0200, Sebastian Hesselbarth wrote: i2c_put_adapter dereferences i2c_adapter pointer passed without check for NULL. This adds a check for non-NULL

Re: [PATCH v2 1/5] ia64: add early_memremap() alias for early_ioremap()

2013-08-05 Thread Matt Fleming
On Fri, 02 Aug, at 05:56:18PM, Leif Lindholm wrote: early_ioremap() on IA64 chooses its mapping type based on the EFI memory map. This patch adds an alias early_memremap() to be used where the targeted location is memory rather than an i/o device. Signed-off-by: Leif Lindholm

Re: stable: please queue commit 0231bb5 (perf: Fix event group context move)

2013-08-05 Thread Luis Henriques
Li Zefan lize...@huawei.com writes: Plese consider adding commit 0231bb5336758426b44ccd798ccd3c5419c95d58 to stable tree. It fixes a real bug, which was explained here: http://thread.gmane.org/gmane.linux.kernel.perf.user/1144 and I've managed to reproduce the bug in 3.4 kernel with

Re: [PATCH][v3.2.y][v3.5.y][3.8.y] Input: elantech - fix for newer hardware versions (v7)

2013-08-05 Thread Luis Henriques
Joseph Salisbury joseph.salisb...@canonical.com writes: Hello, Please consider including upstream commit 9eebed7de660c0b5ab129a9de4f89d20b60de68c in the next v3.2.y, v3.5.y and v3.8.y releases. It was included upstream as of v3.11-rc1. It has been tested and confirmed to resolve

[PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-05 Thread Phil Carmody
Lines with incorrect spacing around an operator, such as: bystander, correct,incorrect would get fixed to bystander,correct, incorrect as the correct argument as well as the incorrectly-spaced operator were both being trimmed. The correct argument only needs to be right trimmed.

[PATCH v2] staging: ozwpan: Fix build warning.

2013-08-05 Thread Rupesh Gujare
This patch fixes following build warning. drivers/built-in.o: In function `oz_hcd_heartbeat': (.text+0x30aadd): undefined reference to `__divdi3' drivers/built-in.o: In function `oz_hcd_heartbeat': (.text+0x30ac85): undefined reference to `__divdi3' Signed-off-by: Rupesh Gujare

RE: [PATCH v8 06/12] ARM: dts: Add description of System MMU of Exynos SoCs

2013-08-05 Thread Cho KyongHo
-Original Message- From: Bartlomiej Zolnierkiewicz [mailto:b.zolnier...@samsung.com] Sent: Saturday, August 03, 2013 2:14 AM Hi, On Friday, July 26, 2013 08:28:19 PM Cho KyongHo wrote: Signed-off-by: Cho KyongHo pullip@samsung.com ---

Re: [PATCH 01/23] radix-tree: implement preload for multiple contiguous elements

2013-08-05 Thread Jan Kara
On Sun 04-08-13 05:17:03, Kirill A. Shutemov wrote: From: Kirill A. Shutemov kirill.shute...@linux.intel.com The radix tree is variable-height, so an insert operation not only has to build the branch to its corresponding item, it also has to build the branch to existing items if the size has

Re: [systemd-devel] Slow firmware timeouts again (Re: [3.11 regression?] iwlwifi firmware takes two minutes to load)

2013-08-05 Thread Kay Sievers
On Fri, Aug 2, 2013 at 6:28 PM, Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl wrote: On Fri, Aug 02, 2013 at 09:04:44AM -0700, Andy Lutomirski wrote: CONFIG_FW_LOADER_USER_HELPER=y Do you need this? Unsetting this should help. This option enables / disables the invocation of user-helper

Re: [PATCH v2 0/4] Radio device framework

2013-08-05 Thread Akhil Goyal
On 7/11/2013 9:52 AM, Greg KH wrote: On Thu, Jul 11, 2013 at 09:29:49AM +0530, Akhil Goyal wrote: On 7/8/2013 3:19 PM, akhil.go...@freescale.com wrote: From: Akhil Goyalakhil.go...@freescale.com RF signal path is integral part of any system that transmits/receives RF (radio frequency)

Re: [PATCH 08/23] block: implement add_bdi_stat()

2013-08-05 Thread Jan Kara
On Sun 04-08-13 05:17:10, Kirill A. Shutemov wrote: From: Kirill A. Shutemov kirill.shute...@linux.intel.com We're going to add/remove a number of page cache entries at once. This patch implements add_bdi_stat() which adjusts bdi stats by arbitrary amount. It's required for batched page

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Laszlo Ersek
On 08/01/13 18:49, Borislav Petkov wrote: On Wed, Jul 31, 2013 at 10:55:27PM +0100, David Woodhouse wrote: On Wed, 2013-07-31 at 22:54 +0200, Borislav Petkov wrote: so I'm seeing this funny thing where an EFI region changes when we enter efi_enter_virtual_mode when booting with edk2 on kvm.

[PATCH v2] staging: ozwpan: Fix farewell report.

2013-08-05 Thread Rupesh Gujare
This patch fix following issues reported by Dan:- 1) There is no check limiting the size to 32 and it could be up to 253 bytes. 2) Use defines instead of magic numbers. 3) The oz_farewell struct is supposed to be a variable length struct but the variable part is put in the middle. It

Re: [patch v2 3/3] mm: page_alloc: fair zone allocator policy

2013-08-05 Thread Andrea Arcangeli
On Mon, Aug 05, 2013 at 06:34:56PM +0800, Wanpeng Li wrote: Why round robin allocator don't consume ZONE_DMA? I guess lowmem reserve reserves it all, 4GB/256(ratio)=16MB. The only way to relax it would be 1) to account depending on memblock types and allow only the movable ones to bypass the

Re: [PATCH v4 01/11] tracing: Add support for SOFT_DISABLE to syscall events

2013-08-05 Thread Masami Hiramatsu
(2013/07/30 1:40), Tom Zanussi wrote: The original SOFT_DISABLE patches didn't add support for soft disable of syscall events; this adds it and paves the way for future patches allowing triggers to be added to syscall events, since triggers are built on top of SOFT_DISABLE. Add an array of

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-08-05 Thread Christian Ruppert
On Tue, Jul 30, 2013 at 12:35:03AM +0200, Linus Walleij wrote: Sorry for taking eternities to look into this. On Tue, Jun 18, 2013 at 11:29 AM, Christian Ruppert christian.rupp...@abilis.com wrote: The pinmux driver of the Abilis Systems TB10x platform based on ARC700 CPUs. Used to

Re: [PATCH] hid: add driver for IBM/Lenovo ScrollPoint mice

2013-08-05 Thread Jiri Kosina
[ Bernhard added to CC ] On Wed, 3 Jul 2013, Peter De Wachter wrote: IBM Scrollpoint have a trackpoint-like gizmo instead of a scroll wheel, which allows for two-dimensional scrolling. This driver provides an input mapping to make that work. The scrollpoint is also much more sensitive

Re: Re: regulator: s2mps11: Question about the code updating ramp_delay setting

2013-08-05 Thread Mark Brown
On Mon, Aug 05, 2013 at 05:58:36AM +, Yadwinder Singh Brar wrote: HTMLHEADTITLESamsung Enterprise Portal mySingle/TITLE META content=text/html; charset=windows-1252 http-equiv=Content-Type Please fix your mailer to send plain text mails, or at least provide a text/plain alternative.

[PATCH] f2fs: fix a deadlock in fsync

2013-08-05 Thread Jin Xu
From: Jin Xu jinuxst...@gmail.com This patch fixes a deadlock bug that occurs quite often when there are concurrent write and fsync on a same file. Following is the simplified call trace when tasks get hung. fsync thread: - f2fs_sync_file ... - f2fs_write_data_pages ... -

<    4   5   6   7   8   9   10   11   12   13   >