[PATCH] kernel/events: using uninitialized variable: slots.flexible for arm cross-compiling

2013-02-05 Thread Chen Gang
for arm cross-compiling, in function __reserve_bp_slot: slots.flexible may be used without initializing. slots.flexible is not initialized when define it. it is as a parameter passed to fetch_bp_busy_slots. in function fetch_bp_busy_slots: if bp->cpu < 0

Re: [PATCH v7 01/10] ARM: davinci: move private EDMA API to arm/common

2013-02-05 Thread Felipe Balbi
On Tue, Feb 05, 2013 at 10:26:30PM +, Arnd Bergmann wrote: > On Tuesday 05 February 2013, Tony Lindgren wrote: > > * Felipe Balbi [130204 07:46]: > > > > > > Current DMA abstraction is quite poor, for example there's no way to > > > compile support for multiple DMA engines. Code also makes

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

2013-02-05 Thread Stephen Rothwell
Hi all, On Wed, 6 Feb 2013 18:30:40 +1100 Stephen Rothwell wrote: > > After merging the final tree, today's linux-next build (sparc defconfig) > failed like this: > > arch/sparc/kernel/kgdb_32.c: In function 'sleeping_thread_to_gdb_regs': > arch/sparc/kernel/kgdb_32.c:46:9: error: implicit

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

2013-02-05 Thread Stephen Rothwell
Hi all, After merging the final tree, today's linux-next build (sparc defconfig) failed like this: arch/sparc/kernel/kgdb_32.c: In function 'sleeping_thread_to_gdb_regs': arch/sparc/kernel/kgdb_32.c:46:9: error: implicit declaration of function 'task_thread_info'

RE: [PATCH] kdump, oldmem: support mmap on /dev/oldmem

2013-02-05 Thread Hatayama, Daisuke
From: Vivek Goyal Subject: Re: [PATCH] kdump, oldmem: support mmap on /dev/oldmem Date: Tue, 5 Feb 2013 10:12:56 -0500 > On Mon, Feb 04, 2013 at 04:59:35AM +, Hatayama, Daisuke wrote: > [..] >> For design decision, I didn't support mmap() on /proc/vmcore because >> it abstracts old memory

Re: [PATCH 12/15] USB: gadget/freescale: disable non-multiplatform drivers

2013-02-05 Thread Felipe Balbi
On Tue, Feb 05, 2013 at 05:55:11PM -0800, Greg Kroah-Hartman wrote: > On Tue, Feb 05, 2013 at 10:54:00PM +, Arnd Bergmann wrote: > > On Tuesday 05 February 2013, Felipe Balbi wrote: > > > [linus/master] Merge branch 'fix-max-write' of > > >

Re: 3.8-rc1 patch_cirrus 4.0 regression...

2013-02-05 Thread Takashi Iwai
At Wed, 6 Feb 2013 08:47:41 +0800, Daniel J Blueman wrote: > > On 6 February 2013 01:40, Takashi Iwai wrote: > > At Tue, 05 Feb 2013 17:34:15 +0100, > > Takashi Iwai wrote: > >> > >> At Wed, 6 Feb 2013 00:29:54 +0800, > >> Daniel J Blueman wrote: > >> > > >> > On 6 February 2013 00:16, Takashi

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

2013-02-05 Thread Stephen Rothwell
Hi Andrew, After merging the akpm tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: arch/powerpc/mm/mem.c: In function 'arch_add_memory': arch/powerpc/mm/mem.c:134:2: error: implicit declaration of function '__add_pages' [-Werror=implicit-function-declaration]

linux-next: manual merge of the akpm tree with the nfsd tree

2013-02-05 Thread Stephen Rothwell
Hi Andrew, Today's linux-next merge of the akpm tree got a conflict in fs/nfsd/nfscache.c between commit 2c6b691c05bf ("nfsd: when updating an entry with RC_NOCACHE, just free it") from the nfsd tree and commit "hlist: drop the node parameter from iterators" from the akpm tree. I fixed it up

[LSF/MM TOPIC] f2fs: Dynamic Control of GC Thread at User Level and Stabilizing sudden eject behavior for SD Cards.

2013-02-05 Thread Namjae Jeon
Hello. I am considering the following subjects on f2fs. I'm not sure if it is proper TOPIC. Let me know opinions :) 1. Dynamic Control of GC Thread at User Level. To cope with rapid increase in number of smart phone users and increasing demands for different media application with Quality Of

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Ben Greear
On 02/05/2013 08:36 PM, Steven Rostedt wrote: On Tue, 2013-02-05 at 19:30 -0800, Ben Greear wrote: It's huge, so here's a link: http://www.candelatech.com/~greearb/debug.tgz The trace shows that __netif_receive_skb() is grabbing an rcu_read_lock() but never releasing it. But I don't see

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Ben Greear
On 02/05/2013 08:26 PM, Steven Rostedt wrote: On Tue, Feb 05, 2013 at 05:10:37PM -0800, Ben Greear wrote: === [ INFO: suspicious RCU usage. ] 3.7.6+ #4 Tainted: G C O --- /home/greearb/git/linux-3.7.dev.y/kernel/rcutree.c:360

Re: [PATCH v5 3/3] ARM: mm: use static_vm for managing static mapped areas

2013-02-05 Thread Nicolas Pitre
On Wed, 6 Feb 2013, Joonsoo Kim wrote: > On Wed, Feb 06, 2013 at 11:07:07AM +0900, Joonsoo Kim wrote: > > Hello, Rob. > > > > On Tue, Feb 05, 2013 at 01:12:51PM -0600, Rob Herring wrote: > > > On 02/05/2013 12:13 PM, Nicolas Pitre wrote: > > > > On Tue, 5 Feb 2013, Rob Herring wrote: > > > > >

Re: [PATCH V2] clk: Add composite clock type

2013-02-05 Thread Hiroshi Doyu
Prashant Gaikwad wrote @ Wed, 6 Feb 2013 03:55:00 +0100: > >> No, clk_ops depends on the clocks you are using. There could be a clock > >> with mux and gate while another one with mux and div. > > You are right. What about the following? We don't have to have similar > > copy of

[PATCH] smp: give WARN in case of in_interrupt() when calling smp_call_function_many/single

2013-02-05 Thread Chuansheng Liu
Currently, in function smp_call_function_many/single, it will give WARN just in case of irqs_disabled(), but it is not enough. In many other cases such as softirq handling/interrupt handling, the two APIs still can not be called, just as the smp_call_function_many() comments said: * You must

RE: [PATCH V2 1/6] pinctrl: pinctrl-single: use arch_initcall and module_exit

2013-02-05 Thread Vishwanathrao Badarkhe, Manish
Hi Russell, On Tue, Feb 05, 2013 at 16:57:31, Russell King - ARM Linux wrote: > On Tue, Feb 05, 2013 at 06:36:34AM +, Vishwanathrao Badarkhe, Manish > wrote: > > I made following changes, in order to update "dip->p" pointer with > > correct value: > > > > - if (!dpi->p) { > > +

Re: [PATCH] X.509: Support parse long form of length octets in Authority Key Identifier

2013-02-05 Thread joeyli
於 三,2013-02-06 於 13:08 +0800,Lee, Chun-Yi 提到: > Per X.509 spec in 4.2.1.1 section, the structure of Authority Key > Identifier Exception is: ^^ Extension Sorry for my typo, I will send patch again. Thanks Joey Lee > >AuthorityKeyIdentifier ::= SEQUENCE { >

[PATCH] X.509: Support parse long form of length octets in Authority Key Identifier

2013-02-05 Thread Lee, Chun-Yi
Per X.509 spec in 4.2.1.1 section, the structure of Authority Key Identifier Extension is: AuthorityKeyIdentifier ::= SEQUENCE { keyIdentifier [0] KeyIdentifier OPTIONAL, authorityCertIssuer [1] GeneralNamesOPTIONAL,

Re: linux-next: manual merge of the driver-core tree with the spi tree

2013-02-05 Thread Greg KH
On Wed, Feb 06, 2013 at 04:21:51PM +1100, Stephen Rothwell wrote: > Hi Greg, > > Today's linux-next merge of the driver-core tree got a conflict in > drivers/spi/spi-s3c64xx.c between commit 4eb770067f3a ("spi/s3c64xx: Use > devm_clk_get() and devm_request_irq()") from the tree and commit >

Re: linux-next: manual merge of the driver-core tree with the iommu tree

2013-02-05 Thread Greg KH
On Wed, Feb 06, 2013 at 04:25:23PM +1100, Stephen Rothwell wrote: > Hi Greg, > > Today's linux-next merge of the driver-core tree got a conflict in > drivers/iommu/tegra-smmu.c between commit a6870e928d1b ("iommu/tegra: > smmu: Support variable MMIO ranges/blocks") from the iommu tree and >

[PATCH 3/5] ARM: OMAP2: MUSB: Specify omap4 has mailbox

2013-02-05 Thread Kishon Vijay Abraham I
Added has_mailbox to the musb platform data to specify that omap uses an external mailbox (in control module) to communicate with the musb core during device connect and disconnect. Signed-off-by: Kishon Vijay Abraham I Acked-by: Tony Lindgren --- arch/arm/mach-omap2/usb-musb.c |3 +++

[PATCH 2/5] ARM: OMAP: devices: create device for usb part of control module

2013-02-05 Thread Kishon Vijay Abraham I
A seperate driver has been added to handle the usb part of control module. A device for the above driver is created here, using the register address information to be used by the driver for powering on the PHY and for writing to the mailbox. Signed-off-by: Kishon Vijay Abraham I Acked-by: Tony

[PATCH 5/5] usb: omap_control_usb: fix compile warning

2013-02-05 Thread Kishon Vijay Abraham I
From: Felipe Balbi When CONFIG_OMAP_CONTROL_USB isn't enabled, there's a compile warning stating that a particular function isn't a prototype. Fix it. Signed-off-by: Felipe Balbi Signed-off-by: Kishon Vijay Abraham I --- include/linux/usb/omap_control_usb.h |2 +- 1 file changed, 1

[PATCH 1/5] ARM: OMAP4: remove control module address space from PHY and OTG

2013-02-05 Thread Kishon Vijay Abraham I
Now that we have a separate driver for the control module, stop populating the control module device data in other modules (PHY and OTG) device info. Signed-off-by: Kishon Vijay Abraham I Acked-by: Tony Lindgren --- arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 13 - 1 file

[PATCH 4/5] ARM: OMAP: USB: Add phy binding information

2013-02-05 Thread Kishon Vijay Abraham I
This is w.r.t the changes in PHY library to support adding and getting multiple PHYs of the same type. In the new design, the binding information between the PHY and the USB controller should be specified in the platform specific initialization code. So it's been done here for OMAP platforms.

[PATCH 0/5] ARM: OMAP: arm part of usb patches

2013-02-05 Thread Kishon Vijay Abraham I
Hi Greg, Here is the patch series that includes the arch/arm part to get MUSB working in OMAP platforms. As discussed in the other mail thread with Tony, this patch series should go into usb-next. This patch series also contains a patch by Felipe to fix compilation warning in usb-next when

[PATCH 2/3] perf sort: Make setup_sorting returns an error code

2013-02-05 Thread Namhyung Kim
From: Namhyung Kim Currently the setup_sorting() is called for parsing sort keys and exits if it failed to add the sort key. As it's included in libperf it'd be better returning an error code rather than exiting application inside of the library. Suggested-by: Arnaldo Carvalho de Melo Cc:

Re: [PATCH] unbreak automounter support on 64-bit kernel with 32-bit userspace (v2)

2013-02-05 Thread Ian Kent
On Mon, 2013-02-04 at 20:39 +0100, Helge Deller wrote: > automount-support is broken on the parisc architecture, because the existing > #if list does not include a check for defined(__hppa__). The HPPA (parisc) > architecture is similiar to other 64bit Linux targets where we have to define >

[PATCH 3/3] perf sort: Check return value of strdup()

2013-02-05 Thread Namhyung Kim
From: Namhyung Kim When setup_sorting() is called, 'str' is passed to strtok_r() but it's not checked to have a valid pointer. As strtok_r() accepts NULL pointer on a first argument and use the third argument in that case, it can cause a trouble since our third argument, tmp, is not

[PATCH RESEND 1/3] perf sort: Drop ip_[lr] arguments from _sort__sym_cmp()

2013-02-05 Thread Namhyung Kim
From: Namhyung Kim Current _sort__sym_cmp() function is used for comparing symbols between two hist entries on symbol, symbol_from and symbol_to sort keys. Those functions pass addresses of symbols but it's meaningless since it gets over-written inside of the _sort__sym_cmp function to a start

RE: [PATCH 2/3] omap: craneboard: support the TPS65910 PMU

2013-02-05 Thread Kim, Milo
> Thanks applying 1 and 2 into omap-for-v3.9/board. Did not > see patch 3 of this anywhere? Thank you. The third patch is for the TPS65910 PMU on the Craneboard. It was already applied. https://lkml.org/lkml/2012/11/30/192 Regards, Milo -- To unsubscribe from this list: send the line

Re: linux-next: manual merge of the crypto tree with the net-next tree

2013-02-05 Thread Herbert Xu
On Wed, Feb 06, 2013 at 01:20:11PM +1100, Stephen Rothwell wrote: > Hi Herbert, > > Today's linux-next merge of the crypto tree got a conflict in > crypto/ctr.c between commit 69d3150cfc20 ("crypto: ctr - make rfc3686 > asynchronous block cipher") from the net-next tree and commit > 3e8afe35c36f

Re: [PATCH] cpufreq: ondemand: Replace down_differential tuner with adj_up_threshold

2013-02-05 Thread Viresh Kumar
On Wed, Feb 6, 2013 at 2:34 AM, Stratos Karafotis wrote: > In order to avoid the calculation of up_threshold - down_differential > every time that the frequency must be decreased, we replace the > down_differential tuner with the adj_up_threshold which keeps the > difference across multiple

[PATCH 3/7] fs/buffer.c: change type of max_buffer_heads to unsigned long

2013-02-05 Thread Zhang Yanfei
max_buffer_heads is calculated from nr_free_buffer_pages(), so change its type to unsigned long in case of overflow. Signed-off-by: Zhang Yanfei --- fs/buffer.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/buffer.c b/fs/buffer.c index 7a75c3e..3eb675b 100644

[PATCH 2/7] ia64: use %ld to print pages calculated in nr_free_buffer_pages

2013-02-05 Thread Zhang Yanfei
Now the function nr_free_buffer_pages returns unsigned long, so use %ld to print its return value. Signed-off-by: Zhang Yanfei --- arch/ia64/mm/contig.c|2 +- arch/ia64/mm/discontig.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/ia64/mm/contig.c

[PATCH 4/7] fs/nfsd: change type of max_delegations, nfsd_drc_max_mem and nfsd_drc_mem_used

2013-02-05 Thread Zhang Yanfei
The three variables are calculated from nr_free_buffer_pages so change their types to unsigned long in case of overflow. Signed-off-by: Zhang Yanfei --- fs/nfsd/nfs4state.c |6 +++--- fs/nfsd/nfsd.h |6 +++--- fs/nfsd/nfssvc.c|6 +++--- 3 files changed, 9 insertions(+), 9

linux-next: manual merge of the driver-core tree with the iommu tree

2013-02-05 Thread Stephen Rothwell
Hi Greg, Today's linux-next merge of the driver-core tree got a conflict in drivers/iommu/tegra-smmu.c between commit a6870e928d1b ("iommu/tegra: smmu: Support variable MMIO ranges/blocks") from the iommu tree and commit bc5e6dea81b0 ("iommu: Convert to devm_ioremap_resource()") from the

[PATCH 7/7] net: change type of virtio_chan->p9_max_pages

2013-02-05 Thread Zhang Yanfei
This member of struct virtio_chan is calculated from nr_free_buffer_pages so change its type to unsigned long in case of overflow. Signed-off-by: Zhang Yanfei --- net/9p/trans_virtio.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/9p/trans_virtio.c

[PATCH 6/7] net: change type of netns_ipvs->sysctl_sync_qlen_max

2013-02-05 Thread Zhang Yanfei
This member of struct netns_ipvs is calculated from nr_free_buffer_pages so change its type to unsigned long in case of overflow. Signed-off-by: Zhang Yanfei --- include/net/ip_vs.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/net/ip_vs.h

linux-next: manual merge of the driver-core tree with the spi tree

2013-02-05 Thread Stephen Rothwell
Hi Greg, Today's linux-next merge of the driver-core tree got a conflict in drivers/spi/spi-s3c64xx.c between commit 4eb770067f3a ("spi/s3c64xx: Use devm_clk_get() and devm_request_irq()") from the tree and commit b0ee5605234a ("spi: Convert to devm_ioremap_resource()") from the driver-core

[PATCH RESEND] ARM: sched: correct update_sched_clock()

2013-02-05 Thread Joonsoo Kim
If we want load epoch_cyc and epoch_ns atomically, we should update epoch_cyc_copy first of all. This notify reader that updating is in progress. If we update epoch_cyc first like as current implementation, there is subtle error case. Look at the below example. cyc = 9 ns = 900 cyc_copy = 9 ==

[PATCH 5/7] vmscan: change type of vm_total_pages to unsigned long

2013-02-05 Thread Zhang Yanfei
This variable is calculated from nr_free_pagecache_pages so change its type to unsigned long. Signed-off-by: Zhang Yanfei --- include/linux/swap.h |2 +- mm/vmscan.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/swap.h

RE: [GIT PULL] ARM part of USB patches

2013-02-05 Thread Hiremath, Vaibhav
On Tue, Feb 05, 2013 at 23:49:47, Tony Lindgren wrote: > * gre...@linuxfoundation.org [130205 09:28]: > > On Tue, Feb 05, 2013 at 08:56:13PM +0530, kishon wrote: > > > Hi Tony, Greg, > > > > > > On Tuesday 05 February 2013 08:54 PM, kishon wrote: > > > >Hi Tony, > > > > > > > >As discussed, I'm

[PATCH 1/7] mm: fix return type for functions nr_free_*_pages

2013-02-05 Thread Zhang Yanfei
Currently, the amount of RAM that functions nr_free_*_pages return is held in unsigned int. But in machines with big memory (exceeding 16TB), the amount may be incorrect because of overflow, so fix it. Signed-off-by: Zhang Yanfei --- include/linux/swap.h |4 ++-- mm/page_alloc.c |8

[PATCH 0/7] mm: fix types for some functions and variables in case of overflow

2013-02-05 Thread Zhang Yanfei
Currently, the amount of RAM that functions nr_free_*_pages return is held in unsigned int. But in machines with big memory (exceeding 16TB), the amount may be incorrect because of overflow, so fix this problem. Also, fix the types of variables that is related to nr_free_*_pages. For these

[PATCH] X.509: Support parse long form of length octets in Authority Key Identifier

2013-02-05 Thread Lee, Chun-Yi
Per X.509 spec in 4.2.1.1 section, the structure of Authority Key Identifier Exception is: AuthorityKeyIdentifier ::= SEQUENCE { keyIdentifier [0] KeyIdentifier OPTIONAL, authorityCertIssuer [1] GeneralNamesOPTIONAL,

Re: [patch v4 0/18] sched: simplified fork, release load avg and power awareness scheduling

2013-02-05 Thread Alex Shi
BTW, Since numa balance scheduling is also a kind of cpu locality policy, it is natural compatible with power aware scheduling. The v2/v3 of this patch had developed on tip/master, testing show above 2 scheduling policy work together well. -- Thanks Alex -- To unsubscribe from this list: send

Re: [PATCH v5 3/3] ARM: mm: use static_vm for managing static mapped areas

2013-02-05 Thread Joonsoo Kim
On Wed, Feb 06, 2013 at 11:07:07AM +0900, Joonsoo Kim wrote: > Hello, Rob. > > On Tue, Feb 05, 2013 at 01:12:51PM -0600, Rob Herring wrote: > > On 02/05/2013 12:13 PM, Nicolas Pitre wrote: > > > On Tue, 5 Feb 2013, Rob Herring wrote: > > > > > >> On 02/04/2013 10:44 PM, Nicolas Pitre wrote: > >

Re: [RFC 2/2] initramfs with digital signature protection

2013-02-05 Thread H. Peter Anvin
On 02/05/2013 02:09 PM, Kasatkin, Dmitry wrote: It should not be like that. Actually when pre-init exits, cleanup code umount tmpfs, which in turn cleanups the RAM. It doesn't quite... the rootfs is permanent. This is also only one usage mode: there are quite a few Linux systems running

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2013-02-05 Thread Namhyung Kim
On Mon, 4 Feb 2013 13:32:54 +0100, Jiri Olsa wrote: > hi, > adding support to read sample values through the PERF_SAMPLE_READ > sample type. It's now possible to specify 'S' modifier for an event > and get its sample value by PERF_SAMPLE_READ. > > For group the 'S' modifier will enable sampling

Re: [PATCH] f2fs: fix to name f2fs GC task as per partition instead of per device

2013-02-05 Thread Namjae Jeon
2013/2/6, Jaegeuk Kim : > Hi, > > How about this? > > 2013-02-05 (화), 23:24 +0900, Namjae Jeon: >> From: Namjae Jeon >> >> After the commit 2184ad190a79ae2b40b5f5db1fbde5c22db6d310, it allowed >> for naming GC threads based on the device. >> i.e., if we have F2FS formatted partition in different

RE: [PATCH V2 1/6] pinctrl: pinctrl-single: use arch_initcall and module_exit

2013-02-05 Thread Vishwanathrao Badarkhe, Manish
Hi Linus On Tue, Feb 05, 2013 at 18:30:48, Linus Walleij wrote: > On Tue, Feb 5, 2013 at 7:36 AM, Vishwanathrao Badarkhe, Manish > wrote: > > > I made following changes, in order to update "dip->p" pointer with > > correct value: > > > > - if (!dpi->p) { > > + if

Re: [PATCH 5/9] gpiolib: use gpio_chips list in gpiochip_find_base

2013-02-05 Thread Alex Courbot
On 02/06/2013 02:21 AM, Linus Walleij wrote: This looks like it is preserving this userspace-sensitive semantic so that dynamically added chips will still get the same assigned numbers. It does (it should, at least), the assigned ranges should be strictly identical to the previous version.

Re: [ 105/128] efi: Make efi_enabled a function to query EFI facilities

2013-02-05 Thread Ben Hutchings
On Mon, 2013-02-04 at 10:20 -0800, Greg KH wrote: > On Mon, Feb 04, 2013 at 04:44:25PM +, Matt Fleming wrote: > > On Sun, 2013-02-03 at 16:15 +0100, Ben Hutchings wrote: > > > As you can see this needed quite a lot of work to backport, and I > > > haven't been able to test it yet. So I would

linux-next: manual merge of the kvm tree with the arm tree

2013-02-05 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the kvm tree got a conflict in include/uapi/linux/kvm.h between commit aa024c2f35a0 ("KVM: ARM: Power State Coordination Interface implementation") from the arm tree and commits fa6b7fe9928d ("KVM: s390: Add support for channel I/O instructions") and

Re: [PATCH] f2fs: fix to name f2fs GC task as per partition instead of per device

2013-02-05 Thread Jaegeuk Kim
Hi, How about this? 2013-02-05 (화), 23:24 +0900, Namjae Jeon: > From: Namjae Jeon > > After the commit 2184ad190a79ae2b40b5f5db1fbde5c22db6d310, it allowed > for naming GC threads based on the device. > i.e., if we have F2FS formatted partition in different devices - we will > have their GC

Re: [PATCH 01/11] perf ui/hist: Add support to display whole group data for raw columns

2013-02-05 Thread Namhyung Kim
Hi Jiri, On Mon, 4 Feb 2013 13:32:55 +0100, Jiri Olsa wrote: > Currently we don't display group members' values for raw > columns like 'Samples' and 'Period' when in group report > mode. Oops, it was lost during the rework. Thanks for fixing this. Thanks, Namhyung -- To unsubscribe from this

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Steven Rostedt
On Tue, 2013-02-05 at 19:30 -0800, Ben Greear wrote: > It's huge, so here's a link: > > http://www.candelatech.com/~greearb/debug.tgz > The trace shows that __netif_receive_skb() is grabbing an rcu_read_lock() but never releasing it. But I don't see any possible way that can be true in the

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Steven Rostedt
On Tue, Feb 05, 2013 at 05:10:37PM -0800, Ben Greear wrote: > > > === > [ INFO: suspicious RCU usage. ] > 3.7.6+ #4 Tainted: G C O > --- > /home/greearb/git/linux-3.7.dev.y/kernel/rcutree.c:360 Illegal idle entry in > RCU read-side

Re: [PATCH linux-next] x86/mm: Fix compile warning in remove_pagetable()

2013-02-05 Thread Yinghai Lu
On Tue, Feb 5, 2013 at 6:52 PM, Shuah Khan wrote: > Fix the following compile warning in remove_pagetable(): > arch/x86/mm/init_64.c: In function ‘remove_pagetable’: > arch/x86/mm/init_64.c:996:9: warning: ‘next’ may be used uninitialized in > this function [-Wuninitialized] > > > Signed-off-by:

Re: [ 014/128] mwifiex: wakeup and stop multiple tx queues in net_device

2013-02-05 Thread Ben Hutchings
On Mon, 2013-02-04 at 16:36 -0800, Bing Zhao wrote: > Hi Ben, > > > On Mon, 2013-02-04 at 11:43 -0800, Bing Zhao wrote: > > > Hi Ben, > > > > > > > 3.2-stable review patch. If anyone has any objections, please let me > > > > know. > > > > > > > > -- > > > > > > > > From: Avinash

Re: [ 000/128] 3.2.38-stable review

2013-02-05 Thread Ben Hutchings
On Mon, 2013-02-04 at 23:39 +0900, Satoru Takeuchi wrote: > At Sun, 03 Feb 2013 15:46:44 +0100, > Ben Hutchings wrote: > > > > This is the start of the stable review cycle for the 3.2.38 release. > > There are 128 patches in this series, which will be posted as responses > > to this one. If

linux-next: manual merge of the spi tree with the sound-asoc tree

2013-02-05 Thread Stephen Rothwell
Hi Grant, Today's linux-next merge of the spi tree got a conflict in drivers/spi/spi-s3c64xx.c between commit b5be04d35dbb ("spi: s3c64xx: Modify SPI driver to use generic DMA DT support") from the sound-asoc tree and commit 75bf33611041 ("spi/s3c64xx: fix checkpatch warnings and error") from the

Re: [PATCH] ASoC: fsl: fix allyesconfig build for imx-pcm

2013-02-05 Thread Shawn Guo
On Tue, Feb 05, 2013 at 04:42:25PM +, Arnd Bergmann wrote: > Patch 25b8d314 "ASoC: fsl: fix multiple definition of init_module changed > the Makefile for imx-pcm.ko to build two modules that both contain the > imx-pcm.c file, which nicely solves the problem of building both versions > of the

[PATCH V2 0/2] ARM: davinci: da850: add wdt DT node

2013-02-05 Thread Kumar, Anil
Add wdt DT node and OF_DEV_AUXDATA for wdt driver in da850 board dt file to use wdt clock. This is tested on da850 EVM. Test Procedure: $watchdog -F /dev/watchdog (Device should not reboot) After wdt heartbeat timeout(DEFAULT_HEARTBEAT is 60 sec) $ctr + c (Device should reboot after

[PATCH V2 2/2] ARM: davinci: restart: fix wdt to machine restart with DT

2013-02-05 Thread Kumar, Anil
In non DT case da8xx_register_watchdog() is called to register platform device "da8xx_wdt_device" by board file. But in DT case it is not called and wdt device get registered via wdt DT node. Currently code is passing platform device "da8xx_wdt_device" in "davinci_watchdog_reset" function in both

[PATCH V2 1/2] ARM: davinci: da850: add wdt DT node

2013-02-05 Thread Kumar, Anil
Add da850 wdt DT node. Add OF_DEV_AUXDATA for wdt driver to use wdt clock. Signed-off-by: Kumar, Anil --- Changes for V2: -Make single patch for wdt DT node and its OF_DEV_AUXDATA entry changes. :100644 100644 37dc5a3... 4666ce9... M arch/arm/boot/dts/da850-evm.dts :100644 100644 640ab75...

RE: [PATCH] i2c: davinci: update to devm_* API

2013-02-05 Thread Vishwanathrao Badarkhe, Manish
Hi Sekhar On Tue, Feb 05, 2013 at 17:17:25, Nori, Sekhar wrote: > > > On 2/5/2013 1:54 PM, Vishwanathrao Badarkhe, Manish wrote: > > Update the code to use devm_* API so that driver core will manage > > resources. > > > > Signed-off-by: Vishwanathrao Badarkhe, Manish > > You can use

linux-next: manual merge of the battery tree with the mfd tree

2013-02-05 Thread Stephen Rothwell
Hi Anton, Today's linux-next merge of the battery tree got a conflict in include/linux/mfd/abx500/ab8500.h between commit 1d843a6c8c20 ("mfd: ab8500-core: Allow the possibility to dump all AB8500 registers") from the mfd tree and commit 0ed5107fa860 ("ab8500-charger: Do not touch VBUSOVV bits")

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Ben Greear
On 02/05/2013 06:52 PM, Steven Rostedt wrote: On Tue, 2013-02-05 at 18:26 -0800, Ben Greear wrote: Well, here it is..something is calling rcu_read_lock lots and lots, Or a bug in the way lockdep handles rcu mappings. it seems. Any way to get a better idea of where those calls are made?

Re: [PATCH] clk: Fix build for Allwinner A1X SoCs

2013-02-05 Thread Prashant Gaikwad
On Tuesday 05 February 2013 08:35 PM, Thierry Reding wrote: Commit 85a1819 (clk: sunxi: Use common of_clk_init() function) removed the source file but didn't update the Makefile, therefore causing the build to break on Allwinner A1X SoCs. Signed-off-by: Thierry Reding Thanks Thierry!! ---

[PATCH 2/2] regulator: max8998: Let regulator core handle the case selector == old_selector

2013-02-05 Thread Axel Lin
Signed-off-by: Axel Lin --- drivers/regulator/max8998.c | 14 +- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/drivers/regulator/max8998.c b/drivers/regulator/max8998.c index 0a8dd1c..b588f07 100644 --- a/drivers/regulator/max8998.c +++ b/drivers/regulator/max8998.c

[PATCH 1/2] regulator: core: Optimize _regulator_do_set_voltage if voltage does not change

2013-02-05 Thread Axel Lin
Optimize _regulator_do_set_voltage() for the case selector is equal to old_selector. Since the voltage does not change, we don't need to call set_voltage_sel() and set_voltage_time_sel() in this case. Signed-off-by: Axel Lin --- drivers/regulator/core.c |9 ++--- 1 file changed, 6

Re: [PATCH v5 01/14] memory-hotplug: try to offline the memory twice to avoid dependence

2013-02-05 Thread Tang Chen
Hi Glauber, all, An old thing I want to discuss with you. :) On 01/09/2013 11:09 PM, Glauber Costa wrote: memory can't be offlined when CONFIG_MEMCG is selected. For example: there is a memory device on node 1. The address range is [1G, 1.5G). You will find 4 new directories memory8, memory9,

Re: [RFC] arm: use built-in byte swap function

2013-02-05 Thread Kim Phillips
On Fri, 1 Feb 2013 07:33:17 + "Woodhouse, David" wrote: > On Fri, 2013-02-01 at 01:17 +, Russell King - ARM Linux wrote: > > > > > I've tried both gcc 4.6.3 [1] and 4.6.4 [2]. If you can point me to > > > a 4.5.x, I'll try that, too, but as it stands now, if one moves the > > > code

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

2013-02-05 Thread Stephen Rothwell
Hi Jens, After merging the block tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/block/rsxx/core.c:322:22: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'rsxx_pci_probe' drivers/block/rsxx/core.c:513:23: error: expected '=', ',', ';', 'asm' or

Re: [PATCH V2] clk: Add composite clock type

2013-02-05 Thread Prashant Gaikwad
On Tuesday 05 February 2013 03:45 PM, Tomasz Figa wrote: Hi Prashant, Thank you for your patch. Please see some comments inline. On Monday 04 of February 2013 13:41:22 Prashant Gaikwad wrote: Not all clocks are required to be decomposed into basic clock types but at the same time want to use

[PATCH 3/3] regulator: s5m8767: Prevent possible NULL pointer dereference

2013-02-05 Thread Axel Lin
s5m8767_pmic_dt_parse_pdata dereferenes pdata, thus check pdata earlier to avoid NULL pointer dereference. Signed-off-by: Axel Lin --- drivers/regulator/s5m8767.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/regulator/s5m8767.c

[PATCH linux-next] x86/kvm: Fix compile warning in kvm_register_steal_time()

2013-02-05 Thread Shuah Khan
Fix the following compile warning in kvm_register_steal_time(): CC arch/x86/kernel/kvm.o arch/x86/kernel/kvm.c: In function ‘kvm_register_steal_time’: arch/x86/kernel/kvm.c:302:3: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘phys_addr_t’

[PATCH 2/3] regulator: s5m8767: Fix using wrong dev argument at various places

2013-02-05 Thread Axel Lin
Use >dev rather than iodev->dev for dev_err(), dev_warn() and dev_info(). Use >dev rather than iodev->dev for devm_kzalloc() and of_get_regulator_init_data(), this fixes memory leak. Signed-off-by: Axel Lin --- drivers/regulator/s5m8767.c | 47 ++- 1

Re: [PATCH v3 1/3] drm/exynos: Get HDMI version from device tree

2013-02-05 Thread 김승우
On 2013년 02월 06일 09:56, Sean Paul wrote: > On Tue, Feb 5, 2013 at 4:42 PM, Stephen Warren wrote: >> On 02/05/2013 05:37 PM, Sean Paul wrote: >>> On Tue, Feb 5, 2013 at 4:22 PM, Stephen Warren >>> wrote: n 02/05/2013 04:42 PM, Sean Paul wrote: > Use the compatible string in the device

[PATCH 1/3] regulator: s5m8767: Use of_get_child_count()

2013-02-05 Thread Axel Lin
Signed-off-by: Axel Lin --- drivers/regulator/s5m8767.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/regulator/s5m8767.c b/drivers/regulator/s5m8767.c index 1250cef..194b5dd 100644 --- a/drivers/regulator/s5m8767.c +++ b/drivers/regulator/s5m8767.c @@ -536,9

Re: [PATCH V2] clk: Add composite clock type

2013-02-05 Thread Prashant Gaikwad
On Tuesday 05 February 2013 03:52 PM, Hiroshi Doyu wrote: Prashant Gaikwad wrote @ Tue, 5 Feb 2013 09:33:41 +0100: The members of "clk_composite_ops" seems to be always assigned statically. Istead of dynamically allocating/assigning, can't we just have "clk_composite_ops" statically as below?

Re: [PATCH v2 linux-next 2/2] cpufreq: Convert the cpufreq_driver_lock to use the rcu

2013-02-05 Thread Viresh Kumar
On 6 February 2013 07:34, Nathan Zimmer wrote: > In general rwlocks are discourged so we are moving it to use the rcu instead. > > Cc: Viresh Kumar > Cc: "Rafael J. Wysocki" > Signed-off-by: Nathan Zimmer > --- > drivers/cpufreq/cpufreq.c | 173 >

[PATCH linux-next] x86/mm: Fix compile warning in remove_pagetable()

2013-02-05 Thread Shuah Khan
Fix the following compile warning in remove_pagetable(): arch/x86/mm/init_64.c: In function ‘remove_pagetable’: arch/x86/mm/init_64.c:996:9: warning: ‘next’ may be used uninitialized in this function [-Wuninitialized] Signed-off-by: Shuah Khan --- arch/x86/mm/init_64.c |2 +- 1 file

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Steven Rostedt
On Tue, 2013-02-05 at 18:26 -0800, Ben Greear wrote: > Well, here it is..something is calling rcu_read_lock lots and lots, Or a bug in the way lockdep handles rcu mappings. > it seems. Any way to get a better idea of where those calls are > made? Yeah, with ftrace. > 96 locks held by

Re: [PATCH v2] zsmalloc: Add Kconfig for enabling PTE method

2013-02-05 Thread Minchan Kim
r enabling PTE method Zsmalloc has two methods 1) copy-based and 2) pte-based to access allocations that span two pages. You can see history why we supported two approach from [1]. In summary, copy-based method is 3 times fater in x86 while pte-based is 6 times faster in ARM. But it was bad ch

Re: [PATCH v3 1/3] drm/exynos: Get HDMI version from device tree

2013-02-05 Thread Stephen Warren
On 02/05/2013 05:56 PM, Sean Paul wrote: > On Tue, Feb 5, 2013 at 4:42 PM, Stephen Warren wrote: >> On 02/05/2013 05:37 PM, Sean Paul wrote: >>> On Tue, Feb 5, 2013 at 4:22 PM, Stephen Warren >>> wrote: n 02/05/2013 04:42 PM, Sean Paul wrote: > Use the compatible string in the device

Re: [PATCH v2 linux-next 1/2] cpufreq: Convert the cpufreq_driver_lock to a rwlock

2013-02-05 Thread Viresh Kumar
On 6 February 2013 07:34, Nathan Zimmer wrote: > This eliminates the contention I am seeing in __cpufreq_cpu_get. > It also nicely stages the lock to be replaced by the rcu. > > Cc: Viresh Kumar > Cc: "Rafael J. Wysocki" > Signed-off-by: Nathan Zimmer > --- > drivers/cpufreq/cpufreq.c | 42

Re: [PATCH 2/7] cpufreq: Retrieve current frequency from scaling drivers with internal governors

2013-02-05 Thread Viresh Kumar
On 6 February 2013 08:01, Dirk Brandewie wrote: > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > index 2817c3c..7516b7d 100644 > --- a/drivers/cpufreq/cpufreq.c > +++ b/drivers/cpufreq/cpufreq.c > @@ -1178,9 +1178,14 @@ static void cpufreq_out_of_sync(unsigned int cpu, >

Re: [PATCH 5/7] cpufreq: balance out cpufreq_cpu_{get,put} for scaling drivers using setpolicy

2013-02-05 Thread Viresh Kumar
On 6 February 2013 07:38, Dirk Brandewie wrote: > On 02/05/2013 05:58 PM, Viresh Kumar wrote: >> >> On Tue, Feb 5, 2013 at 11:54 PM, wrote: >>> >>> From: Dirk Brandewie >>> >>> There is an additional reference added to the driver in >>> cpufreq_add_dev() that is removed in__cpufreq_governor()

Re: [PATCH 3/7] cpufreq: Only query drivers that implement cpufreq_driver.target()

2013-02-05 Thread Viresh Kumar
On 6 February 2013 07:36, Dirk Brandewie wrote: > On 02/05/2013 05:47 PM, Viresh Kumar wrote: >> >> On Tue, Feb 5, 2013 at 11:54 PM, wrote: >>> >>> From: Dirk Brandewie >>> >>> Scaling drivers that implement cpufreq_driver.setpolicy() have >>> internal governors and may/will change the current

Re: [PATCH 2/7] cpufreq: Retrieve current frequency from scaling drivers with internal governors

2013-02-05 Thread Dirk Brandewie
On 02/05/2013 06:25 PM, Viresh Kumar wrote: On Wed, Feb 6, 2013 at 7:45 AM, Dirk Brandewie wrote: How about this? diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 2817c3c..9c0eac4 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1182,7

Re: [PATCH 1/2] mm: hotplug: implement non-movable version of get_user_pages() called get_user_pages_non_movable()

2013-02-05 Thread Michel Lespinasse
Just nitpicking, but: On Tue, Feb 5, 2013 at 3:57 AM, Mel Gorman wrote: > +static inline bool zone_is_idx(struct zone *zone, enum zone_type idx) > +{ > + /* This mess avoids a potentially expensive pointer subtraction. */ > + int zone_off = (char *)zone - (char

Re: [PATCH v2] zsmalloc: Add Kconfig for enabling PTE method

2013-02-05 Thread Greg Kroah-Hartman
On Wed, Feb 06, 2013 at 11:17:08AM +0900, Minchan Kim wrote: > diff --git a/drivers/staging/zsmalloc/Kconfig > b/drivers/staging/zsmalloc/Kconfig > index 9084565..232b3b6 100644 > --- a/drivers/staging/zsmalloc/Kconfig > +++ b/drivers/staging/zsmalloc/Kconfig > @@ -8,3 +8,15 @@ config ZSMALLOC >

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Ben Greear
On 02/05/2013 05:54 PM, Steven Rostedt wrote: On Tue, Feb 05, 2013 at 05:10:37PM -0800, Ben Greear wrote: I'm debugging something that could be my own bug in my wanlink module (but then again, haven't seen this on 3.5 kernels, and my module is basically un-changed since then). I'm using

Re: [PATCH 2/7] cpufreq: Retrieve current frequency from scaling drivers with internal governors

2013-02-05 Thread Viresh Kumar
On Wed, Feb 6, 2013 at 7:45 AM, Dirk Brandewie wrote: > How about this? > > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > index 2817c3c..9c0eac4 100644 > --- a/drivers/cpufreq/cpufreq.c > +++ b/drivers/cpufreq/cpufreq.c > @@ -1182,7 +1182,12 @@ unsigned int

Re: [PATCH 3/3] acpi, memory-hotplug: Support getting hotplug info from SRAT.

2013-02-05 Thread Tang Chen
On 02/05/2013 07:26 AM, Andrew Morton wrote: On Fri, 25 Jan 2013 17:42:09 +0800 Tang Chen wrote: We now provide an option for users who don't want to specify physical memory address in kernel commandline. /* * For movablemem_map=acpi: * * SRAT:

linux-next: manual merge of the crypto tree with the net-next tree

2013-02-05 Thread Stephen Rothwell
Hi Herbert, Today's linux-next merge of the crypto tree got a conflict in crypto/ctr.c between commit 69d3150cfc20 ("crypto: ctr - make rfc3686 asynchronous block cipher") from the net-next tree and commit 3e8afe35c36f ("crypto: use ERR_CAST") from the crypto tree. I fixed it up (see below) and

Re: Question on lockdep and MAX_LOCK_DEPTH

2013-02-05 Thread Ben Greear
On 02/05/2013 05:54 PM, Steven Rostedt wrote: I'm not sure the swapper task is part of the 'do_each_thread()' loop. Perhaps what you want to do is add a: lockdep_print_held_locks(current); I'll add that and test... I'm curious. Does your code grab a read lock? If you grab the same

  1   2   3   4   5   6   7   8   9   10   >