linux-next: build warning after merge of the final tree (net-next tree related)

2013-05-21 Thread Stephen Rothwell
Hi all, After merging the final tree, today's linux-next build (sparc32 defconfig) produced this warning: drivers/net/ethernet/sun/sunbmac.c: In function 'bigmac_set_multicast': drivers/net/ethernet/sun/sunbmac.c:998:6: warning: unused variable 'i' [-Wunused-variable] Introduced by commit

linux-next: build warning after merge of the final tree (drm-intel tree related)

2013-05-21 Thread Stephen Rothwell
Hi , After merging the final tree, today's linux-next build (i386 defconfig) produced this warning: drivers/gpu/drm/i915/i915_gem.c:2978:3: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'size_t' [-Wformat] Introduced by commit e4341d0356bc ("drm/i915: Be

Re: [PATCH 0/4] media: remove duplicate check for EPERM

2013-05-21 Thread Prabhakar Lad
Hi All, On Tue, May 14, 2013 at 11:15 AM, Lad Prabhakar wrote: > From: Lad, Prabhakar > > This patch series cleanups the check for EPERM in dbg_g/s_register > and vidioc_g/s_register. > > Lad, Prabhakar (4): > media: i2c: remove duplicate checks for EPERM in dbg_g/s_register > media:

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Chinner
On Wed, May 22, 2013 at 01:29:38AM -0400, Dave Jones wrote: > On Wed, May 22, 2013 at 03:12:43PM +1000, Dave Chinner wrote: > > > > [ 36.339105] XFS (sda2): xfs_setattr_size: mask 0xa068 mismatch on > file 0\xffb8\xffd3-\xff88\x\x > > > > So, still the same

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

2013-05-21 Thread Prabhakar Lad
Hi Laurent/Sylwester, On Thu, May 16, 2013 at 6:48 PM, Lad Prabhakar wrote: > From: Lad, Prabhakar > > This patch adds "sync-on-green" property as part of > endpoint properties and also support to parse them in the parser. > If you are ok with the patch can I send non RFC version of it ?

[PATCH ipvs-next v3 1/2] sched: add cond_resched_rcu() helper

2013-05-21 Thread Simon Horman
This is intended for use in loops which read data protected by RCU and may have a large number of iterations. Such an example is dumping the list of connections known to IPVS: ip_vs_conn_array() and ip_vs_conn_seq_next(). The benefits are for CONFIG_PREEMPT_RCU=y where we save CPU cycles by

[PATCH ipvs-next v3 0/2] sched: Add cond_resched_rcu_lock() helper

2013-05-21 Thread Simon Horman
Add a helper that for use in loops which read data protected by RCU and may have a large number of iterations. Such an example is dumping the list of connections known to IPVS: ip_vs_conn_array() and ip_vs_conn_seq_next(). This series also updates IPVS to use the helper. Simon Horman (2):

[PATCH ipvs-next v3 2/2] ipvs: use cond_resched_rcu() helper when walking connections

2013-05-21 Thread Simon Horman
This avoids the situation where walking of a large number of connections may prevent scheduling for a long time while also avoiding excessive calls to rcu_read_unlock() and rcu_read_lock(). Note that in the case of !CONFIG_PREEMPT_RCU this will add a call to cond_resched(). [j...@ssi.bg: add

Re: [PATCH v2] rcu: fix a race in hlist_nulls_for_each_entry_rcu macro

2013-05-21 Thread Eric Dumazet
On Tue, 2013-05-21 at 19:01 -0700, Eric Dumazet wrote: > Please use ACCESS_ONCE(), which is the standard way to deal with this, > and remove the rcu_dereference_raw() in > hlist_nulls_for_each_entry_rcu() > > something like : (for the nulls part only) Thinking a bit more about this, I am a bit

[PATCH 0/2] Improvement and fixes for BHRB

2013-05-21 Thread Anshuman Khandual
(1) The first patch fixes a situation like this Before patch:- ./perf record -j any -e branch-misses:k ls Error: The sys_perf_event_open() syscall returned with 95 (Operation not supported) for event (branch-misses:k). /bin/dmesg may provide additional information. No

[PATCH 2/2] powerpc, perf: BHRB filter configuration should follow the task

2013-05-21 Thread Anshuman Khandual
When the task moves around the system, the corresponding cpuhw per cpu strcuture should be popullated with the BHRB filter request value so that PMU could be configured appropriately with that during the next call into power_pmu_enable(). Signed-off-by: Anshuman Khandual ---

[PATCH 1/2] powerpc, perf: Ignore separate BHRB privilege state filter request

2013-05-21 Thread Anshuman Khandual
Completely ignore BHRB privilege state filter request as we are already configuring MMCRA register with privilege state filtering attribute for the accompanying PMU event. This would help achieve cleaner user space interaction for BHRB. Signed-off-by: Anshuman Khandual ---

Re: [PATCH] virtio_net: enable napi after enabling new queues

2013-05-21 Thread Jason Wang
On 05/22/2013 01:22 PM, Jason Wang wrote: > Commit 55257d72bd1c51f25106350f4983ec19f62ed1fa (virtio-net: fill only rx > queues which are being used) forgets to enable napi for new enabled queues > which > breaks the multiqueue receiving. This patch fixes it. > > Cc: Sasha Levin > Signed-off-by:

[RFC PATCHv2] power: Add option to log time spent in suspend

2013-05-21 Thread Zoran Markovic
From: Colin Cross Below is a patch from android kernel that maintains a histogram of suspend times. Please review and provide feedback. Statistices on the time spent in suspend are kept in /sys/kernel/debug/sleep_time. Cc: Android Kernel Team Cc: Colin Cross Cc: Todd Poynor Cc: San Mehat

[PATCH] virtio_net: enable napi after enabling new queues

2013-05-21 Thread Jason Wang
Commit 55257d72bd1c51f25106350f4983ec19f62ed1fa (virtio-net: fill only rx queues which are being used) forgets to enable napi for new enabled queues which breaks the multiqueue receiving. This patch fixes it. Cc: Sasha Levin Signed-off-by: Jason Wang --- drivers/net/virtio_net.c |4 +++- 1

Re: [PATCH RESEND] mtd: fix kfree bcm47xxsflash

2013-05-21 Thread Rafał Miłecki
2013/5/22 Libo Chen : > mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its > member. Thanks! -- Rafał -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Jones
On Wed, May 22, 2013 at 03:12:43PM +1000, Dave Chinner wrote: > > [ 36.339105] XFS (sda2): xfs_setattr_size: mask 0xa068 mismatch on file > > 0\xffb8\xffd3-\xff88\x\x > > So, still the same strange mask. That just doesn't seem right. any idea what I screwed

Re: [PATCH v4 00/22] x86, ACPI, numa: Parse numa info early

2013-05-21 Thread H. Peter Anvin
Sorry, just have been swamped since -rc1 came out. -hpa -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Chinner
On Wed, May 22, 2013 at 12:15:21AM -0400, Dave Jones wrote: > On Wed, May 22, 2013 at 02:03:18PM +1000, Dave Chinner wrote: > > > That doesn't make a whole lot of sense to me. What am I missing? > > Are you seeing this fire at all from notify_change()? > > > >

Re: [PATCH v4 00/22] x86, ACPI, numa: Parse numa info early

2013-05-21 Thread Tang Chen
On 05/10/2013 02:24 AM, Yinghai Lu wrote: .. If you have any thinking of this patch-set, please let me know. Talked to HPA, and he will put my patchset into tip/x86/mm after v3.10-rc1. after that we can work on put pagetable on local node for hotadd path. Hi, It is Linux v3.10-rc2

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

2013-05-21 Thread Michael S. Tsirkin
does not imply > might_sleep"). > > I have used the version of the vhost tree from next-20130521 for today. > > It was unexpected that such patches would turn up in the "vhost" tree ... > > -- > Cheers, > Stephen Rothwells...@canb.auug.org.a

Re: [PATCH] bestcomm: no need to free when kzalloc fail

2013-05-21 Thread Li Zefan
On 2013/5/22 12:49, Libo Chen wrote: > > ping... > > On 2013/5/5 16:38, chenlib...@gmail.com wrote: >> From: Libo Chen >> >> There is no need to free bcom_eng if kzalloc fail >> kfree(NULL) is fine. We gain nothing from this patch, and it even adds one more line to the code, so just drop thi

[PATCH 3.9-stable] regulator: mc13892: Fix MC13892_SWITCHERS0_SWxHI bit in set_voltage_sel

2013-05-21 Thread Jonghwan Choi
This patch looks like it should be in the 3.9-stable tree, should we apply it? -- From: "Axel Lin " commit 78040b630f28298e0ecb56bf1bc53c7976314eab upstream It is necessary to clear MC13892_SWITCHERS0_SWxHI bit when set voltage to the voltage range from 110 to 1375000.

Re: [PATCH] bestcomm: no need to free when kzalloc fail

2013-05-21 Thread Libo Chen
ping... On 2013/5/5 16:38, chenlib...@gmail.com wrote: > From: Libo Chen > > There is no need to free bcom_eng if kzalloc fail > > Signed-off-by: Libo Chen > --- > drivers/dma/bestcomm/bestcomm.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git

Re: [PATCH 2/2 v2, RFC] Driver core: Introduce offline/online callbacks for memory blocks

2013-05-21 Thread Tang Chen
Hi Rafael, On 05/21/2013 07:15 PM, Rafael J. Wysocki wrote: .. + mem->state = to_state; + if (to_state == MEM_ONLINE) + mem->last_online = online_type; Why do we need to remember last online type ? And as far as I know, we can obtain

Re: [PATCH v2 12/13] x86, numa, acpi, memory-hotplug: Make movablecore=acpi have higher priority.

2013-05-21 Thread Tang Chen
Hi Vasilis, Maybe the following two problems are the cause of the reboot panic problem in qemu you mentioned. On 04/30/2013 05:21 PM, Tang Chen wrote: .. diff --git a/mm/page_alloc.c b/mm/page_alloc.c index b9ea143..2fe9ebf 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -4793,9

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

2013-05-21 Thread Stephen Rothwell
ht_fault' if (_might_fault()) ^ and many, many more. Caused by commit d198a8acf543 ("kernel: might_fault does not imply might_sleep"). I have used the version of the vhost tree from next-20130521 for today. It was unexpected that such patches would turn up in the "vhost" tree ...

Re: [ANNOUNCE] ktap 0.1 released

2013-05-21 Thread Ming Lei
On Wed, May 22, 2013 at 12:19 PM, Ming Lei wrote: > On Wed, May 22, 2013 at 12:15 PM, Ming Lei wrote: >> On Tue, May 21, 2013 at 11:56 AM, zhangwei(Jovi) >> >> Nice job, I have run it on ARM already with only one line change. >> >> But looks 'Control-C' can't stop the tracing or need some time

linux-next: manual merge of the vhost tree with the rr-fixes tree

2013-05-21 Thread Stephen Rothwell
Hi Michael, Today's linux-next merge of the vhost tree got conflicts in include/linux/uio.h and lib/iovec.c between commit d2f83e9078b8 ("Hoist memcpy_fromiovec/memcpy_toiovec into lib/") from the rr-fixes tree and commit 48f0d22c9a46 ("Hoist memcpy_fromiovec into lib/") from the vhost tree. The

Re: [ANNOUNCE] ktap 0.1 released

2013-05-21 Thread Ming Lei
On Wed, May 22, 2013 at 12:15 PM, Ming Lei wrote: > On Tue, May 21, 2013 at 11:56 AM, zhangwei(Jovi) > > Nice job, I have run it on ARM already with only one line change. > > But looks 'Control-C' can't stop the tracing or need some time to complete it, > see below: Sometimes, it doesn't work:

Re: [ANNOUNCE] ktap 0.1 released

2013-05-21 Thread Ming Lei
On Tue, May 21, 2013 at 11:56 AM, zhangwei(Jovi) wrote: > > Dear, > > I'm pleased to announce that ktap release v0.1, this is the first official > release of ktap project, it is expected that this release is not fully > functional or very stable and we welcome bug reports and fixes for the

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Jones
On Wed, May 22, 2013 at 02:03:18PM +1000, Dave Chinner wrote: > That doesn't make a whole lot of sense to me. What am I missing? > Are you seeing this fire at all from notify_change()? > > WARN_ON_ONCE(!mutex_is_locked(>i_mutex)); No. > > > What's wrong with this code in

Re: [RFC PATCH v2, part 2 14/18] PCI, EDAC: use hotplug-safe PCI bus iterators to walk PCI buses

2013-05-21 Thread Gu Zheng
On 05/15/2013 12:51 AM, Jiang Liu wrote: > Enhance EDAC drviers to use hotplug-safe iterators to walk PCI buses. > > Signed-off-by: Jiang Liu > Cc: Mauro Carvalho Chehab > Cc: Doug Thompson > Cc: linux-e...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > drivers/edac/i7core_edac.c

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Chinner
On Tue, May 21, 2013 at 10:56:05PM -0400, Dave Jones wrote: > On Wed, May 22, 2013 at 10:16:03AM +1000, Dave Chinner wrote: > > Seems like I can trigger this from paths other than truncate too.. (eg, > sys_open) O_TRUNC? > The mask is always 0xa068 though if that helps. A bit - it confirms

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

2013-05-21 Thread Stephen Rothwell
Hi all, On Wed, 22 May 2013 13:47:41 +1000 Stephen Rothwell wrote: > > Today's linux-next merge of the driver-core tree got a conflict in > drivers/base/cpu.c between commit 0902a9044fa5 ("Driver core: Use generic > offline/online for CPU offline/online") from the pm tree and commit >

[PATCH RESEND] mtd: fix kfree bcm47xxsflash

2013-05-21 Thread Libo Chen
mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member. Signed-off-by: Libo Chen Acked-by: Rafał Miłecki --- drivers/mtd/devices/bcm47xxsflash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) * add Acked-by: Rafał Miłecki * change send list diff --git

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

2013-05-21 Thread Stephen Rothwell
Hi Greg, Today's linux-next merge of the driver-core tree got a conflict in drivers/base/cpu.c between commit 0902a9044fa5 ("Driver core: Use generic offline/online for CPU offline/online") from the pm tree and commit 1c4e2d70afb1 ("cpu: make sure that cpu/online file created before KOBJ_ADD is

linux-next: build warning after merge of the tty.current tree

2013-05-21 Thread Stephen Rothwell
Hi Greg, After merging the tty.current tree, today's linux-next build (x86_64 allmodconfig) produced this warning: drivers/tty/mxser.c: In function 'mxser_ioctl': drivers/tty/mxser.c:1680:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] int p; ^

[PATCH 1/2] f2fs: should not make_bad_inode on f2fs_link failure

2013-05-21 Thread Jaegeuk Kim
If -ENOSPC is met during f2fs_link, we should not make the inode as bad. The inode is still alive. Signed-off-by: Jaegeuk Kim --- fs/f2fs/namei.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c index 729b285..71aa305 100644 --- a/fs/f2fs/namei.c +++

[PATCH 2/2] f2fs: use ihold

2013-05-21 Thread Jaegeuk Kim
Use the following helper function committed by Al. commit 7de9c6ee3ecffd99e1628e81a5ea5468f7581a1f Author: Al Viro Date: Sat Oct 23 11:11:40 2010 -0400 new helper: ihold() ... Signed-off-by: Jaegeuk Kim --- fs/f2fs/namei.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v2] usb: omap2430: fix memleak in err case

2013-05-21 Thread Libo Chen
when omap_get_control_dev faild, we should release related platform_device * Changelog from v1: * fix spell: s/fail/fails/, s/relational/related/ , thank Sergei Signed-off-by: Libo Chen --- drivers/usb/musb/omap2430.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)

Re: [ANNOUNCE] ktap 0.1 released

2013-05-21 Thread zhangwei(Jovi)
On 2013/5/22 6:19, Jonathan Corbet wrote: > On Tue, 21 May 2013 11:56:14 +0800 > "zhangwei(Jovi)" wrote: > >> we welcome bug reports and fixes for the issues. > > I'm messing with it...first impression: > > unable create tracepoint event sys_enter_mmap on cpu 4, err: -19 > unable

Re: [PATCH 2/2] Staging: bcm: Change comment style to C89 in vendorspecificextn.c

2013-05-21 Thread Greg Kroah-Hartman
On Wed, May 22, 2013 at 02:13:58AM +0200, Severin Gsponer wrote: > Changed the comment style to C89 in vendorspecificextn.c. Found with > checkpatch.pl > > Signed-off-by: Severin Gsponer Where is patch 1/2? I see another patch you sent me, but that didn't have a number on it. confused, greg

[PATCH v7 1/8] vmcore: clean up read_vmcore()

2013-05-21 Thread HATAYAMA Daisuke
Rewrite part of read_vmcore() that reads objects in vmcore_list in the same way as part reading ELF headers, by which some duplicated and redundant codes are removed. Signed-off-by: HATAYAMA Daisuke Acked-by: Vivek Goyal --- fs/proc/vmcore.c | 68

[PATCH v7 7/8] vmcore: calculate vmcore file size from buffer size and total size of vmcore objects

2013-05-21 Thread HATAYAMA Daisuke
The previous patches newly added holes before each chunk of memory and the holes need to be count in vmcore file size. There are two ways to count file size in such a way: 1) supporse m as a poitner to the last vmcore object in vmcore_list. , then file size is (m->offset + m->size), or 2)

[PATCH v7 8/8] vmcore: support mmap() on /proc/vmcore

2013-05-21 Thread HATAYAMA Daisuke
This patch introduces mmap_vmcore(). Don't permit writable nor executable mapping even with mprotect() because this mmap() is aimed at reading crash dump memory. Non-writable mapping is also requirement of remap_pfn_range() when mapping linear pages on non-consecutive physical pages; see

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Jones
On Wed, May 22, 2013 at 10:16:03AM +1000, Dave Chinner wrote: Seems like I can trigger this from paths other than truncate too.. (eg, sys_open) Here's what I ended up with for debug diff --git a/fs/dcache.c b/fs/dcache.c index 2b39d16..b579dfe 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@

[PATCH v7 2/8] vmcore: allocate buffer for ELF headers on page-size alignment

2013-05-21 Thread HATAYAMA Daisuke
Allocate ELF headers on page-size boundary using __get_free_pages() instead of kmalloc(). Later patch will merge PT_NOTE entries into a single unique one and decrease the buffer size actually used. Keep original buffer size in variable elfcorebuf_sz_orig to kfree the buffer later and actually

[PATCH v7 4/8] vmalloc: make find_vm_area check in range

2013-05-21 Thread HATAYAMA Daisuke
Currently, __find_vmap_area searches for the kernel VM area starting at a given address. This patch changes this behavior so that it searches for the kernel VM area to which the address belongs. This change is needed by remap_vmalloc_range_partial to be introduced in later patch that receives any

[PATCH v7 5/8] vmalloc: introduce remap_vmalloc_range_partial

2013-05-21 Thread HATAYAMA Daisuke
We want to allocate ELF note segment buffer on the 2nd kernel in vmalloc space and remap it to user-space in order to reduce the risk that memory allocation fails on system with huge number of CPUs and so with huge ELF note segment that exceeds 11-order block size. Although there's already

[PATCH v7 6/8] vmcore: allocate ELF note segment in the 2nd kernel vmalloc memory

2013-05-21 Thread HATAYAMA Daisuke
The reasons why we don't allocate ELF note segment in the 1st kernel (old memory) on page boundary is to keep backward compatibility for old kernels, and that if doing so, we waste not a little memory due to round-up operation to fit the memory to page boundary since most of the buffers are in

[PATCH v7 3/8] vmcore: treat memory chunks referenced by PT_LOAD program header entries in page-size boundary in vmcore_list

2013-05-21 Thread HATAYAMA Daisuke
Treat memory chunks referenced by PT_LOAD program header entries in page-size boundary in vmcore_list. Formally, for each range [start, end], we set up the corresponding vmcore object in vmcore_list to [rounddown(start, PAGE_SIZE), roundup(end, PAGE_SIZE)]. This change affects layout of

[PATCH v7 0/8] kdump, vmcore: support mmap() on /proc/vmcore

2013-05-21 Thread HATAYAMA Daisuke
Currently, read to /proc/vmcore is done by read_oldmem() that uses ioremap/iounmap per a single page. For example, if memory is 1GB, ioremap/iounmap is called (1GB / 4KB)-times, that is, 262144 times. This causes big performance degradation due to repeated page table changes, TLB flush and

[PATCH v6 05/10 fix2] mtd: get the ECC info from the Extended Parameter Page

2013-05-21 Thread Huang Shijie
Since the ONFI 2.1, the onfi spec adds the Extended Parameter Page to store the ECC info. The onfi spec tells us that if the nand chip's recommended ECC codeword size is not 512 bytes, then the @ecc_bits is 0xff. The host _SHOULD_ then read the Extended ECC information that is part of the

Re: [PATCH v2 0/4] dma-mapping: Define dma_{alloc,free}_attrs() for all archs

2013-05-21 Thread Damian Hobson-Garcia
Hello, On 2013/04/30 12:01, Damian Hobson-Garcia wrote: > Most architectures that define CONFIG_HAVE_DMA=y, have implementations for > both dma_alloc_attrs() and dma_free_attrs(). All achitectures that do > not define CONFIG_HAVE_DMA also have both of these definitions provided by >

[PATCH 2/2] spi: convert drivers to use bits_per_word_mask

2013-05-21 Thread Stephen Warren
Fill in the recently added spi_master.bits_per_word_mask field in as many drivers as possible. Make related cleanups, such as removing any redundant error-checking, or empty setup callbacks. Cc: device-drivers-de...@blackfin.uclinux.org Cc: Mike Frysinger Signed-off-by: Stephen Warren --- The

[PATCH 1/2] spi: introduce macros to set bits_per_word_mask

2013-05-21 Thread Stephen Warren
Introduce two macros to make setting up spi_master.bits_per_word_mask easier, and avoid mistakes like writing BIT(n) instead of BIT(n - 1). SPI_BPW_MASK is for a single supported value of bits_per_word_mask. SPI_BPW_RANGE_MASK represents a contiguous set of bit lengths. Signed-off-by: Stephen

[RESEND][PATCH 2/2] clk: wm831x: Fix wm831x_clkout_get_parent

2013-05-21 Thread Axel Lin
Current code looks strange because calling wm831x_clkout_set_parent() with 0 as parent parameter, wm831x_clkout_get_parent() will return 1. According to the datasheet: R16528 (4090h) Clock Control1 BIT 0: CLKOUT output source select 0 = FLL output 1 =

[RESEND][PATCH 1/2] clk: wm831x: Fix update wrong register for enable/disable FLL

2013-05-21 Thread Axel Lin
According to the datasheet, FLL Enable is controlled by R16530 (4092h) FLL Control1 BIT 0: FLL Enable 0 = Disable 1 = Enable Thus the code should update WM831X_FLL_CONTROL_1 register rather than WM831X_FLL_CONTROL_2 register. Also fixes a trivial typo in

[PATCH v2 2/5] perf: Swap the parameters of strglobmatch

2013-05-21 Thread Masami Hiramatsu
Swap the parameters of strglobmatch() so that the first parameter is the glob pattern as like as regexec(), because the subjective parameter of strglobmatch() must be the glob pattern, but not a sample string. The new interface is: bool strglobmatch(const char *glob, const char *str); Actually

[PATCH v2 5/5] tracing: Support enable/disable multiple events trigger by wild cards

2013-05-21 Thread Masami Hiramatsu
Support enable/disable multiple events trigger on ftrace by using wild cards. This makes enabling multiple events at once easy. e.g.) # echo vfs_symlink:enable_event:\*:\*rq\* > set_ftrace_filter # cat set_ftrace_filter all functions enabled vfs_symlink:enable_event:*:*rq*:unlimited

[PATCH v2 1/5] [BUGFIX] tracing: Returns -EBUSY when event_enable_func fails to get module

2013-05-21 Thread Masami Hiramatsu
Since try_module_get returns false( = 0) when it fails to pindown a module, event_enable_func() returns 0 which means "succeed". This can cause a kernel panic when the entry is removed, because the event is already released. This fixes the bug by returning -EBUSY, because the reason why it fails

[PATCH v2 3/5] lib/string: Add a generic wildcard string matching function

2013-05-21 Thread Masami Hiramatsu
Add strglobmatch() for generic wildcard string matching. This code is originally from perf-tools. For porting in the kernel, the limitation of the number of wildcards is introduced, because of the limitation of the stack. Signed-off-by: Masami Hiramatsu Cc: Andrew Morton Cc: Akinobu Mita Cc:

[PATCH v2 4/5] tracing/kprobes: Allow user to delete kprobe events by wild cards

2013-05-21 Thread Masami Hiramatsu
Allow user to delete multiple kprobe events by using wild cards. This makes removing events on a specific function easy. e.g.) # echo p vfs_symlink >> kprobe_events # echo p vfs_symlink+5 >> kprobe_events # echo p vfs_read >> kprobe_events # cat kprobe_events p:kprobes/p_vfs_symlink_0

[PATCH v2 0/5] Add glob pattern matching support on trigger and kprobe-event

2013-05-21 Thread Masami Hiramatsu
Hi, Here is a series of ftrace/perf updates to support multiple event select operation by glob-based wild cards. I've ported strglobmatch from perf-tools (with recursive call limitation) for this use. It is easier to use (just replacing strcmp) but slower than current parser-based matching. I

Re: [PATCH v6 05/10 fix] mtd: get the ECC info from the Extended Parameter Page

2013-05-21 Thread Huang Shijie
于 2013年05月22日 00:35, Vikram Narayanan 写道: > What if we do like this? Let's do a strict check for now. ("EPPS") > If some chip manufacturer had _strictly_ followed the spec, i.e., any > two bytes are valid. > Then make the code comply with the spec. > Any comments? Agreed. I will send a new patch

Re: [PATCH beta 1] 0/3] Fix race conditions in mrf24j40 interrupts

2013-05-21 Thread Alan Ott
On 05/21/2013 10:01 PM, Alan Ott wrote: > David Hauweele noticed that the mrf24j40 would hang arbitrarily after some > period of heavy traffic. Two race conditions were discovered, and the > driver was changed to use threaded interrupts, since the enable/disable of > interrupts in the driver has

[PATCH beta 1] 0/3] Fix race conditions in mrf24j40 interrupts

2013-05-21 Thread Alan Ott
David Hauweele noticed that the mrf24j40 would hang arbitrarily after some period of heavy traffic. Two race conditions were discovered, and the driver was changed to use threaded interrupts, since the enable/disable of interrupts in the driver has recently been a lighning rod whenever issues

[PATCH beta 1] 1/3] mrf24j40: Move INIT_COMPLETION() to before packet transmission

2013-05-21 Thread Alan Ott
This avoids a race condition where complete(tx_complete) could be called before tx_complete is initialized. Signed-off-by: Alan Ott --- drivers/net/ieee802154/mrf24j40.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ieee802154/mrf24j40.c

[PATCH beta 1] 2/3] mrf24j40: Use threaded IRQ handler

2013-05-21 Thread Alan Ott
Eliminate all the workqueue and interrupt enable/disable. Signed-off-by: Alan Ott --- drivers/net/ieee802154/mrf24j40.c | 27 +++ 1 file changed, 7 insertions(+), 20 deletions(-) diff --git a/drivers/net/ieee802154/mrf24j40.c b/drivers/net/ieee802154/mrf24j40.c index

[PATCH beta 1] 3/3] mrf24j40: Use level-triggered interrupts

2013-05-21 Thread Alan Ott
The mrf24j40 generates level interrupts. There are rare cases where it appears that the interrupt line never gets de-asserted between interrupts, causing interrupts to be lost, and causing a hung device from the driver's perspective. Switching the driver to interpret these interrupts as

Re: [PATCH v2] rcu: fix a race in hlist_nulls_for_each_entry_rcu macro

2013-05-21 Thread Eric Dumazet
On Tue, 2013-05-21 at 22:12 +0400, Roman Gushchin wrote: > > If other rcu accessors have the same problem, a more complete patch is > > needed. > > [PATCH] rcu: fix a race in rcu lists traverse macros > > Some network functions (udp4_lib_lookup2(), for instance) use the > rcu lists traverse

RE: [PATCH] ARM: bcm2835: override the HW UART periphid

2013-05-21 Thread Jongsung Kim
Stephen Warren : > Looking at the TRM, it seems this is really the only change, according > to the changelog in the documentation (although it's a little difficult > to tell since the document seems to have a bunch of changes that presumably > don't affect behaviour). So, faking the periphid

[PATCH 3.9-stable] leds: leds-gpio: reserve gpio before using it.

2013-05-21 Thread Jonghwan Choi
This patch looks like it should be in the 3.9-stable tree, should we apply it? -- From: "Timo Teras " commit 803d19d57a042e86e9e9b685bbc3f4a0a751040f upstream This reverts commit a99d76f (leds: leds-gpio: use gpio_request_one) and commit 2d7c22f (leds: leds-gpio: set

Re: [PATCH] ARM: bcm2835: override the HW UART periphid

2013-05-21 Thread Stephen Warren
On 05/21/2013 10:34 AM, Stephen Warren wrote: > On 05/21/2013 12:02 AM, Jongsung Kim wrote: >> Stephen Warren reported the recent commit 78506f2 (add support for >> extended FIFO-size of PL011-r1p5) breaks the serial port on the >> BCM2835 ARM SoC. >> >> A UART compatible with the ARM PL011-r1p5

Re: [PATCH v6 3/7] KVM: MMU: fast invalidate all pages

2013-05-21 Thread Marcelo Tosatti
On Tue, May 21, 2013 at 11:39:03AM +0300, Gleb Natapov wrote: > > Any pages with stale information will be zapped by kvm_mmu_zap_all(). > > When that happens, page faults will take place which will automatically > > use the new generation number. > > > > So still not clear why is this necessary.

Re: [PATCH v6 3/7] KVM: MMU: fast invalidate all pages

2013-05-21 Thread Marcelo Tosatti
On Tue, May 21, 2013 at 11:36:57AM +0800, Xiao Guangrong wrote: > On 05/21/2013 04:40 AM, Marcelo Tosatti wrote: > > On Mon, May 20, 2013 at 11:15:45PM +0300, Gleb Natapov wrote: > >> On Mon, May 20, 2013 at 04:46:24PM -0300, Marcelo Tosatti wrote: > >>> On Fri, May 17, 2013 at 05:12:58AM +0800,

Re: [PATCH 00/19] driver:usb: add missing platform_driver owner

2013-05-21 Thread Gu Zheng
On 05/21/2013 05:42 PM, Libo Chen wrote: > On 2013/5/21 17:06, Thomas Petazzoni wrote: >> Dear Gu Zheng, >> >> On Tue, 21 May 2013 16:00:19 +0800, Gu Zheng wrote: >> Or, maybe make the existing module_platform_driver() macro do this? >>> >>> But not all the modules use

Re: [PATCH 0/3] firmware: Fix usermodehelper deadlock at shutdown

2013-05-21 Thread Ming Lei
On Wed, May 22, 2013 at 1:02 AM, Greg Kroah-Hartman wrote: > On Wed, May 08, 2013 at 08:56:34AM +0200, Takashi Iwai wrote: >> Hi, >> >> this is a series of patches for the issue we faced in the firmware >> loader code during debugging the problem with dell_rbu driver with >> 3.9 kernel. >> >> The

Re: [PATCH] usermodehelper: kill the sub_info->path[0] check

2013-05-21 Thread Lucas De Marchi
Hi Oleg, On Mon, May 20, 2013 at 1:55 PM, Oleg Nesterov wrote: > call_usermodehelper_exec() does nothing but returns success if > path[0] == 0. The only user which needs this strange feature is > request_module(), it can check modprobe_path[0] itself like other > users do if they want to detect

Re: [PATCH 18/19] drivers/net/ethernet/marvell: add missing platform_driver owne

2013-05-21 Thread Libo Chen
On 2013/5/21 21:11, Sergei Shtylyov wrote: > Hello. > > On 21-05-2013 6:42, Libo Chen wrote: > >> set the owner of platform_driver, to ensure that the >> caller of driver holds a module refernece > >> Signed-off-by: Libo Chen >> --- >> drivers/net/ethernet/marvell/mvneta.c |1 + >> 1

[PULL] virtio fixes

2013-05-21 Thread Rusty Russell
The following changes since commit d5fe85af85367d5892e4308f596de4e2a5fb9017: Merge tag 'pm+acpi-3.10-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm (2013-05-16 15:12:34 -0700) are available in the git repository at:

Re: [PATCH] usermodehelper: kill the sub_info->path[0] check

2013-05-21 Thread Rusty Russell
Oleg Nesterov writes: > call_usermodehelper_exec() does nothing but returns success if > path[0] == 0. The only user which needs this strange feature is > request_module(), it can check modprobe_path[0] itself like other > users do if they want to detect the "disabled by admin" case. > > Kill it.

Re: [PATCH] arm64: kernel: need extern variable 'screen_info' for related driver using.

2013-05-21 Thread Chen Gang
On 05/21/2013 09:15 PM, Catalin Marinas wrote: > On Tue, May 21, 2013 at 11:13:08AM +0100, Chen Gang wrote: >> > On 05/21/2013 05:03 PM, Will Deacon wrote: >>> > > On Tue, May 21, 2013 at 08:51:39AM +0100, Chen Gang wrote: > >> For me, I still recommend add 'ARM64' in the long list of >

Re: [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c"

2013-05-21 Thread Chen Gang
On 05/21/2013 09:17 PM, Catalin Marinas wrote: > On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: >> >> Need add the default file in 'asm/Kbuild' just like arm has done. >> >> The build command is: >> make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig >> make EXTRA_CFLAGS=-W ARCH=arm64

Re: [PATCH v2 1/2] Make the batch size of the percpu_counter configurable

2013-05-21 Thread Tim Chen
On Tue, 2013-05-21 at 16:41 -0700, Andrew Morton wrote: > I think we could use __percpu_counter_add() unconditionally here and > just do > > #ifdef CONFIG_SMP > #define vm_committed_as_batch 0 > #else > int vm_committed_as_batch; > #endif > > The EXPORT_SYMBOL(vm_committed_as_batch) is

Re: [PATCH 06/19] net/ethernet/dec/tulip/xircom_cb: Use module_pci_driver to register driver

2013-05-21 Thread Grant Grundler
[new gmail editor changed how to send plaint/text email - sorry for multiple copies] On Tue, May 21, 2013 at 3:42 PM, Peter Huewe wrote: > Removing some boilerplate by using module_pci_driver instead of calling > register and unregister in the otherwise empty init/exit functions. > >

Re: [PATCH - sort of] x86: Livelock in handle_pte_fault

2013-05-21 Thread Steven Rostedt
On Fri, 2013-05-17 at 10:42 +0200, Stanislav Meduna wrote: > Hi all, > > I don't know whether this is linux-rt specific or applies to > the mainline too, so I'll repeat some things the linux-rt > readers already know. > > Environment: > > - Geode LX or Celeron M > - _not_ CONFIG_SMP > - linux

Re: [PATCH v5 6/7] mm: Support address range reclaim

2013-05-21 Thread Andrew Morton
On Thu, 9 May 2013 16:21:28 +0900 Minchan Kim wrote: > This patch adds address range reclaim of a process. > The requirement is following as, > > Like webkit1, it uses a address space for handling multi tabs. > IOW, it uses *one* process model so all tabs shares address space > of the process.

Re: [PATCH 11/19] net/ethernet/broadcom/tg3: Use module_pci_driver to register driver

2013-05-21 Thread Nithin Nayak Sujir
On 05/21/2013 03:58 PM, Peter Huewe wrote: Removing some boilerplate by using module_pci_driver instead of calling register and unregister in the otherwise empty init/exit functions. Signed-off-by: Peter Huewe --- drivers/net/ethernet/broadcom/tg3.c | 13 + 1 file changed, 1

Re: [PATCH 03/19] net/ethernet/atheros/atl1e/atl1e_main: Use module_pci_driver to register driver

2013-05-21 Thread Hannes Frederic Sowa
On Wed, May 22, 2013 at 12:42:08AM +0200, Peter Huewe wrote: > Removing some boilerplate by using module_pci_driver instead of calling > register and unregister in the otherwise empty init/exit functions. > > Signed-off-by: Peter Huewe Acked-by: Hannes Frederic Sowa -- To unsubscribe from

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Chinner
On Tue, May 21, 2013 at 08:08:03PM -0400, Dave Jones wrote: > On Wed, May 22, 2013 at 09:54:10AM +1000, Dave Chinner wrote: > > On Tue, May 21, 2013 at 07:40:16PM -0400, Dave Jones wrote: > > > On Wed, May 22, 2013 at 09:34:29AM +1000, Dave Chinner wrote: > > > > On Tue, May 21, 2013 at

Re: [PATCH 0/3] Enable multiple MSI feature in pSeries

2013-05-21 Thread Benjamin Herrenschmidt
On Tue, 2013-05-21 at 16:45 +0200, Alexander Gordeev wrote: > On Tue, Jan 15, 2013 at 03:38:53PM +0800, Mike Qiu wrote: > > The test results is shown by 'cat /proc/interrups': > > CPU0 CPU1 CPU2 CPU3 > > 16: 240458 261601 226310 200425 XICS

Re: [PATCH] ARM: at91: Fix: Change internal SRAM memory type to "MT_MEMORY_SO"

2013-05-21 Thread Russell King - ARM Linux
On Mon, May 20, 2013 at 09:06:19AM +0800, Wenyou Yang wrote: > Signed-off-by: Wenyou Yang This needs more of a description. Also, for a single patch, it's silly to send two mails, the first being a cover which has a little more information in it about the patch than the patch itself. You need

[PATCH 2/2] f2fs: fix to handle do_recover_data errors

2013-05-21 Thread Jaegeuk Kim
This patch adds error handling codes of check_index_in_prev_nodes and its caller, do_recover_data. Signed-off-by: Jaegeuk Kim --- fs/f2fs/recovery.c | 24 +++- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c index

[PATCH 1/2] f2fs: reuse the locked dnode page and its inode

2013-05-21 Thread Jaegeuk Kim
This patch fixes the following deadlock bug during the recovery. INFO: task mount:1322 blocked for more than 120 seconds. "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message. mount D 81125870 0 1322 1266 0x 8801207e39d8 0046

[PATCH 2/2] Staging: bcm: Change comment style to C89 in vendorspecificextn.c

2013-05-21 Thread Severin Gsponer
Changed the comment style to C89 in vendorspecificextn.c. Found with checkpatch.pl Signed-off-by: Severin Gsponer --- drivers/staging/bcm/vendorspecificextn.c | 198 +++--- 1 file changed, 97 insertions(+), 101 deletions(-) diff --git

Re: [PATCH][3.10] nohz: Fix lockup on restart from wrong error code

2013-05-21 Thread Steven Rostedt
On Tue, 2013-05-21 at 22:14 +0200, Frederic Weisbecker wrote: > 2013/5/21 Steven Rostedt : > > commit a382bf934449 "nohz: Assign timekeeping duty to a CPU outside the > > full dynticks range" added a cpu notifier callback that would prevent > > the time keeping CPU from going offline if the

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-21 Thread Dave Jones
On Wed, May 22, 2013 at 09:54:10AM +1000, Dave Chinner wrote: > On Tue, May 21, 2013 at 07:40:16PM -0400, Dave Jones wrote: > > On Wed, May 22, 2013 at 09:34:29AM +1000, Dave Chinner wrote: > > > On Tue, May 21, 2013 at 06:52:57PM -0400, Dave Jones wrote: > > > > [ 464.210598] XFS:

Re: [PATCH] Finally eradicate CONFIG_HOTPLUG

2013-05-21 Thread Stephen Rothwell
Hi Peter, On Tue, 21 May 2013 12:13:49 +0200 Peter Stuge wrote: > > Are you changing the code to have HOTPLUG always -on- or -off- ? > > From the commit message I had expected always -on-. Yes, it has been that way for some time. > Stephen Rothwell wrote: > > ---

  1   2   3   4   5   6   7   8   9   10   >