Re: [PATCH 1/4] staging: omap-thermal: Correct checkpatch.pl warnings

2012-09-12 Thread Dan Carpenter
On Wed, Sep 12, 2012 at 12:19:00PM +0300, Valentin, Eduardo wrote: > Hello Dan, > > On Wed, Sep 12, 2012 at 11:26 AM, Dan Carpenter > wrote: > > On Wed, Sep 12, 2012 at 11:11:27AM +0300, Dan Carpenter wrote: > >> On Tue, Sep 11, 2012 at 07:06:52PM +0300, Eduardo Valentin wrote: > >> > From: J

Re: [PATCH tip/core/rcu 11/26] rcu: Exit RCU extended QS on user preemption

2012-09-12 Thread Peter Zijlstra
On Wed, 2012-09-12 at 15:54 +0200, Frederic Weisbecker wrote: > On Wed, Sep 12, 2012 at 02:52:40PM +0200, Peter Zijlstra wrote: > > On Wed, 2012-09-12 at 14:41 +0200, Peter Zijlstra wrote: > > > > > We could of course mandate that all remote wakeups to special nohz cpus > > > get queued. That

[PATCH v2 1/3] hrtimer: add hrtimer_init_cpu()

2012-09-12 Thread Stephane Eranian
hrtimer_init() assumes it is called for the current CPU as it accesses per-cpu variables (hrtimer_bases). However, there can be cases where a hrtimer is initialized from a different CPU, so add an entry point to make this more explicit. Signed-off-by: Stephane Eranian ---

[PATCH v2 3/3] perf: add sysfs entry to adjust multiplexing interval per PMU

2012-09-12 Thread Stephane Eranian
This patch adds /sys/device/xxx/mux_interval_ms to ajust the multiplexing interval per PMU. The unit is milliseconds. Value has to be >= 1. Signed-off-by: Stephane Eranian --- include/linux/perf_event.h |1 + kernel/events/core.c | 54 --- 2

[PATCH v2 2/3] perf: use hrtimer for event multiplexing

2012-09-12 Thread Stephane Eranian
The current scheme of using the timer tick was fine for per-thread events. However, it was causing bias issues in system-wide mode (including for uncore PMUs). Event groups would not get their fair share of runtime on the PMU. With tickless kernels, if a core is idle there is no timer tick, and

[PATCH v2 0/3] perf: use hrtimer for event multiplexing

2012-09-12 Thread Stephane Eranian
The current scheme of using the timer tick was fine for per-thread events. However, it was causing bias issues in system-wide mode (including for uncore PMUs). Event groups would not get their fair share of runtime on the PMU. With tickless kernels, if a core is idle there is no timer tick, and

Re: [PATCH tip/core/rcu 04/15] rcu: Permit RCU_NONIDLE() to be used from interrupt context

2012-09-12 Thread Steven Rostedt
On Tue, 2012-09-11 at 18:07 -0700, Paul E. McKenney wrote: > > > Paul, you just push your changes through tip, right? Then we can just > > let Ingo know. I could even make the patch a separate branch, that Ingo > > can pull into the RCU branch too. > > Yep! But we also need to worry about

Re: [PATCH v2 1/5] fat: allocate persistent inode numbers

2012-09-12 Thread Namjae Jeon
2012/9/12 OGAWA Hirofumi : > Namjae Jeon writes: > I think that it is unfixable because we can not know i_pos of inode changed by rename. And even though we know it, there is no rebuild inode routine in -mm. And It even can not fix in our patches. >>> > And are you tried

Re: [alsa-devel] [PATCH 11/11] ASoC: omap-pcm: Convert to use dmaengine

2012-09-12 Thread Takashi Iwai
At Wed, 12 Sep 2012 13:00:28 +0100, Russell King - ARM Linux wrote: > > On Wed, Sep 12, 2012 at 02:47:07PM +0300, Peter Ujfalusi wrote: > > 2. Pause/Resume > >OMAP DMA engine backend does not support pausing and resuming > >an in-progress transfer. It is unclear from the specs what > >

Re: [PATCH 1/2] staging/vme: Use pr_ printks in vme_user.c

2012-09-12 Thread Greg Kroah-Hartman
On Wed, Sep 12, 2012 at 04:36:19PM +0900, Toshiaki Yamane wrote: > On Wed, Sep 12, 2012 at 4:08 AM, Greg Kroah-Hartman > wrote: > > On Tue, Sep 11, 2012 at 04:16:14PM +0900, Toshiaki Yamane wrote: > >> The below checkpatch warnings was fixed, > >> > >> -WARNING: Prefer pr_info(... to

Re: [PATCH -v2] x86, 32-bit: Fix invalid stack address while in softirq

2012-09-12 Thread Steven Rostedt
Added hpa, as he's one of the main active maintainers of the x86 arch. On Wed, 2012-09-12 at 15:50 +0200, Robert Richter wrote: > Updated version below. Changes are: > > * add comments to kernel_stack_pointer() > * always return a valid stack address by falling back to the address >

Re: [PATCH] usb: host: tegra: code clean up

2012-09-12 Thread Felipe Balbi
Hi, On Wed, Sep 12, 2012 at 12:32:42PM +0530, Venu Byravarasu wrote: > As part of code clean up, used devm counterparts for the APIs > possible. > > Signed-off-by: Venu Byravarasu EHCI is Alan's domain. > --- > drivers/usb/host/ehci-tegra.c | 46 ++-- >

Re: [PATCH tip/core/rcu 11/26] rcu: Exit RCU extended QS on user preemption

2012-09-12 Thread Frederic Weisbecker
On Wed, Sep 12, 2012 at 02:52:40PM +0200, Peter Zijlstra wrote: > On Wed, 2012-09-12 at 14:41 +0200, Peter Zijlstra wrote: > > > We could of course mandate that all remote wakeups to special nohz cpus > > get queued. That would just leave us with RCU and it would simply not > > send resched IPIs

Re: [PATCH] usb: move phy driver from mach-tegra to drivers/usb

2012-09-12 Thread Felipe Balbi
On Tue, Sep 11, 2012 at 10:34:38AM -0600, Stephen Warren wrote: > On 09/05/2012 07:20 AM, Venu Byravarasu wrote: > > As part of this patch: > > 1. Moved existing tegra phy driver to drivers/USB directory. > > 2. Added standard USB phy driver APIs to tegra phy driver. > > > >

Re: [PATCH] spi: omap2-mcspi: Cleanup the omap2_mcspi_txrx_dma function

2012-09-12 Thread Mark Brown
On Tue, Sep 11, 2012 at 12:13:20PM +0530, Shubhrajyoti D wrote: > Currently in omap2_mcspi_txrx_dma the tx and the rx support is > interleaved. Make the rx related code in omap2_mcspi_rx_dma > and the tx related code omap2_mcspi_tx_dma and call the functions. Applied, thanks. -- To unsubscribe

Re: [PATCH tip/core/rcu 11/26] rcu: Exit RCU extended QS on user preemption

2012-09-12 Thread Frederic Weisbecker
On Wed, Sep 12, 2012 at 02:41:36PM +0200, Peter Zijlstra wrote: > On Wed, 2012-09-12 at 14:06 +0200, Frederic Weisbecker wrote: > > > > 1) This can happen if something calls set_need_resched() while no other > > task is > > on the runqueue. > > People really shouldn't be doing that... I think I

Re: [PATCH v3] trace: Move trace event enable from fs_initcall to core_initcall

2012-09-12 Thread Steven Rostedt
On Wed, 2012-09-12 at 09:29 -0300, Ezequiel Garcia wrote: > Steven, > > On Tue, Sep 11, 2012 at 9:34 PM, Steven Rostedt wrote: > ble(void) > [...] > >> + /* > >> + * Early initialization already enabled ftrace event. > >> + * Now it's only necessary to create the event directory. >

[PATCH -v2] x86, 32-bit: Fix invalid stack address while in softirq

2012-09-12 Thread Robert Richter
Updated version below. Changes are: * add comments to kernel_stack_pointer() * always return a valid stack address by falling back to the address of regs -Robert >From 0114d0e2ff6ce3f6015fd991541a45261f14eab1 Mon Sep 17 00:00:00 2001 From: Robert Richter Date: Mon, 3 Sep 2012

[tip:x86/platform] x86: Document x86_init.paging.pagetable_init()

2012-09-12 Thread tip-bot for Attilio Rao
Commit-ID: 64282278989d5b0398dcb3ba7904cb00c621dc35 Gitweb: http://git.kernel.org/tip/64282278989d5b0398dcb3ba7904cb00c621dc35 Author: Attilio Rao AuthorDate: Tue, 21 Aug 2012 21:22:41 +0100 Committer: Thomas Gleixner CommitDate: Wed, 12 Sep 2012 15:33:07 +0200 x86: Document

Re: [PATCH v3 02/31] arm64: Kernel booting and initialisation

2012-09-12 Thread Catalin Marinas
On Wed, Sep 12, 2012 at 01:08:58PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 17:11 Mon 10 Sep , Catalin Marinas wrote: > > On Sun, Sep 09, 2012 at 06:20:46PM +0100, Jean-Christophe PLAGNIOL-VILLARD > > wrote: > > > On 17:26 Fri 07 Sep , Catalin Marinas wrote: > > > > +Before

Re: [PATCH 16/18] perf evsel: Introduce perf_evsel__{str,int}val methods

2012-09-12 Thread Arnaldo Carvalho de Melo
Em Wed, Sep 12, 2012 at 11:21:07AM +0900, Namhyung Kim escreveu: > On Tue, 11 Sep 2012 20:53:08 -0300, Arnaldo Carvalho de Melo wrote: > > From: Arnaldo Carvalho de Melo > > > > Wrappers to the libtraceevent routines, so that we can further reduce > > the surface contact perf builtins have with

[tip:x86/platform] x86: xen: Cleanup and remove x86_init.paging. pagetable_setup_done()

2012-09-12 Thread tip-bot for Attilio Rao
Commit-ID: c711288727a62f74d48032e56e51333dd104bf58 Gitweb: http://git.kernel.org/tip/c711288727a62f74d48032e56e51333dd104bf58 Author: Attilio Rao AuthorDate: Tue, 21 Aug 2012 21:22:40 +0100 Committer: Thomas Gleixner CommitDate: Wed, 12 Sep 2012 15:33:06 +0200 x86: xen: Cleanup and

Re: [PATCH 16/18] perf evsel: Introduce perf_evsel__{str,int}val methods

2012-09-12 Thread Arnaldo Carvalho de Melo
Em Wed, Sep 12, 2012 at 10:40:00AM +0900, Namhyung Kim escreveu: > On Tue, 11 Sep 2012 20:53:08 -0300, Arnaldo Carvalho de Melo wrote: > > From: Arnaldo Carvalho de Melo > > > > Wrappers to the libtraceevent routines, so that we can further reduce > > the surface contact perf builtins have with

[tip:x86/platform] x86: Move paging_init() call to x86_init.paging .pagetable_init()

2012-09-12 Thread tip-bot for Attilio Rao
Commit-ID: 843b8ed2ec598aae5e3516b21957ede62a070e36 Gitweb: http://git.kernel.org/tip/843b8ed2ec598aae5e3516b21957ede62a070e36 Author: Attilio Rao AuthorDate: Tue, 21 Aug 2012 21:22:39 +0100 Committer: Thomas Gleixner CommitDate: Wed, 12 Sep 2012 15:33:06 +0200 x86: Move paging_init()

Re: [PATCH] spi: omap2-mcspi: Cleanup the omap2_mcspi_txrx_dma function

2012-09-12 Thread Shubhrajyoti
On Wednesday 12 September 2012 07:09 PM, Felipe Balbi wrote: > seems to be working fine (at least from my quick tests) > > Tested-by: Felipe Balbi Thanks Felipe. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

[tip:x86/platform] x86: Rename pagetable_setup_start() to pagetable_init()

2012-09-12 Thread tip-bot for Attilio Rao
Commit-ID: 7737b215ad0f94d20a87d98315da9f6cadaf35c9 Gitweb: http://git.kernel.org/tip/7737b215ad0f94d20a87d98315da9f6cadaf35c9 Author: Attilio Rao AuthorDate: Tue, 21 Aug 2012 21:22:38 +0100 Committer: Thomas Gleixner CommitDate: Wed, 12 Sep 2012 15:33:06 +0200 x86: Rename

[tip:x86/platform] x86: Remove base argument from x86_init.paging. pagetable_setup_start

2012-09-12 Thread tip-bot for Attilio Rao
Commit-ID: 73090f8993a40a2f67fed1ab866a928c68cd3765 Gitweb: http://git.kernel.org/tip/73090f8993a40a2f67fed1ab866a928c68cd3765 Author: Attilio Rao AuthorDate: Tue, 21 Aug 2012 21:22:37 +0100 Committer: Thomas Gleixner CommitDate: Wed, 12 Sep 2012 15:33:06 +0200 x86: Remove base

Re: [PATCH] spi: omap2-mcspi: Cleanup the omap2_mcspi_txrx_dma function

2012-09-12 Thread Felipe Balbi
On Tue, Sep 11, 2012 at 12:13:20PM +0530, Shubhrajyoti D wrote: > Currently in omap2_mcspi_txrx_dma the tx and the rx support is > interleaved. Make the rx related code in omap2_mcspi_rx_dma > and the tx related code omap2_mcspi_tx_dma and call the functions. > > While at it remove the braces in

RE: [alsa-devel] [PATCH] ASoC: codecs: Add DA9055 codec driver

2012-09-12 Thread Ashish P. Chavan
FYI. -Original Message- From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] Sent: Wednesday, September 12, 2012 6:20 PM To: Ashish P. Chavan Cc: lrg; alsa-devel; linux-kernel; David Dajun Chen Subject: Re: [alsa-devel] [PATCH] ASoC: codecs: Add DA9055 codec driver On Wed, Sep

RE: [Resend][PATCH V3] trace,x86: add x86 irq vector tracepoints

2012-09-12 Thread Seiji Aguchi
Hi, I will submit some performance testing result. Thanks! Seiji > -Original Message- > From: H. Peter Anvin [mailto:h...@zytor.com] > Sent: Wednesday, September 12, 2012 12:18 AM > To: Seiji Aguchi > Cc: Thomas Gleixner (t...@linutronix.de); linux-kernel@vger.kernel.org; >

Re: [PATCH v2 0/5] X86/XEN: Merge x86_init.paging.pagetable_setup_start and x86_init.paging.pagetable_setup_done setup functions and document its semantic

2012-09-12 Thread Thomas Gleixner
On Tue, 11 Sep 2012, Konrad Rzeszutek Wilk wrote: > > > > The overall result is basically the same, but it's way simpler to look > > > > at obvious and well done patches than checking whether a subtle copy > > > > and paste bug happened in 3/5 of the first version. Copy and paste is > > > > the #1

Re: [PATCH v2 7/8] drivers/media/platform/davinci/vpbe.c: Removes useless kfree()

2012-09-12 Thread Marcos Souza
2012/9/12 Peter Senna Tschudin : > From: Peter Senna Tschudin > > Remove useless kfree() and clean up code related to the removal. > > The semantic patch that finds this problem is as follows: > (http://coccinelle.lip6.fr/) > > // > @r exists@ > position p1,p2; > expression x; > @@ > > if (x@p1

Re: [PATCH 1/4] Input: cyttsp4 - bus driver for Cypress TMA4XX touchscreen devices

2012-09-12 Thread Henrik Rydberg
Hi Ferruh, > Before finalizing the work, we want to learn if introducing new files > for new driver is acceptable and our effort is inline with your request, > can you please comment on it? New files are certainly alright when they are warranted, but of course it depends on the details. We will

Re: [PATCH 1/4] Input: cyttsp4 - bus driver for Cypress TMA4XX touchscreen devices

2012-09-12 Thread Ferruh Yigit
On 08/27/2012 08:46 AM, Ferruh Yigit wrote: On 08/24/2012 08:34 PM, Henrik Rydberg wrote: Hi Ferruh, As Javier mentioned, there is already a framework for the Cypress devices in the kernel. Please make sure your patches build on what is already there. Until then, I have nothing further to add.

Re: [PATCH] plist: Make plist test announcements KERN_DEBUG

2012-09-12 Thread Steven Rostedt
On Wed, 2012-09-12 at 12:38 +0200, Borislav Petkov wrote: > From: Borislav Petkov > > They show up in dmesg > > [4.041094] start plist test > [4.045804] end plist test > > without a lot of meaning so hide them behind debug loglevel. > Acked-by: Steven Rostedt -- Steve -- To

[PATCH 01/24] Staging: ipack/bridges/tpci200: add helpers for writing control regs.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Convert tpci200_set_clockrate and tpci200_interrupt. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/bridges/tpci200.c | 49 --- 1 file changed, 32 insertions(+), 17 deletions(-) diff --git

[PATCH 02/24] Staging: ipack/bridges/tpci200: Remove side effects of tpci200_{request,free}_irq.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Use the __tpci200_{set,clear}_mask routines to access control register. Do not overwrite flags unrelated to interrupt handling. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/bridges/tpci200.c | 31

[PATCH 04/24] Staging: ipack/devices/ipoctal: split ipoctal_channel from ipoctal.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge By moving everything channel related into a separate struct we will be able to clean up a lot of code. In the end tty->driver_data will no longer need to point to ipoctal but instead can point to the respective channel. Signed-off-by: Jens Taprogge Signed-off-by: Samuel

[PATCH 05/24] Staging: ipack/devices/ipoctal: Directly use ioread/iowrite function.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c | 151 +-- 1 file changed, 41 insertions(+), 110 deletions(-) diff --git a/drivers/staging/ipack/devices/ipoctal.c

Re: [PATCH] perf: Introduce function to report unsupported syscall attribute flags

2012-09-12 Thread Robert Richter
On 12.09.12 13:20:43, Peter Zijlstra wrote: > On Wed, 2012-09-12 at 13:01 +0200, Robert Richter wrote: > > + if (notsup) > > + pr_warn("perf: unsupported attribute flags: %016llx\n", > > notsup); > > This is a dmesg DoS.. This could be avoided by introducing a cpuinfo like

[PATCH 07/24] Staging: ipack/devices/ipoctal: Store isr masks in ipoctal_channel

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge This way interrupt handling becomes independent of the channel number. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff

Re: [PATCH 18/19] gpio: Provide the tc3589x GPIO expander driver with an IRQ domain

2012-09-12 Thread Linus Walleij
On Mon, Sep 10, 2012 at 3:10 PM, Linus Walleij wrote: > On Fri, Sep 7, 2012 at 1:14 PM, Lee Jones wrote: > >> In preparation for Device Tree enablement all IRQ controllers >> should control their own IRQ domain. This patch provides just >> that for the tc3589x GPIO expander. >> >> CC: Grant

[PATCH 09/24] Staging: ipack/devices/ipoctal: remove superfluous function.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge ipoctal_write_tty and ipoctal_write are merged. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git

[PATCH 08/24] Staging: ipack/devices/ipoctal: Split interrupt service routine.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Split the IRQ service routing in TX part and RX part. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c | 188 --- 1 file changed, 97 insertions(+), 91 deletions(-) diff --git

[PATCH 10/24] Staging: ipack/bridges/tpci200: RCU protect slot_irq pointers.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge In tpci200_request_irq as well as tpci200_free_irq we set and unset the pointer to struct slot_irq. This pointer is accessed in tpci200_interrupt. To ensure that the pointer is not freed after it has been fetched in tpci200_interrupt() it is now protected through RCU.

[PATCH 11/24] Staging: ipack/bridges/tpci200: Protect device registers with spinlock.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Some of the device registers are accessed from both interrupt and non-interrupt context. To ensure proper read-modify-write modification of these registers we can not use the "global" tpci200 mutex. Instead a spin-lock is used. Signed-off-by: Jens Taprogge Signed-off-by:

[PATCH 12/24] Staging: ipack/bridges/tpci200: Clean up interrupt handling.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Previously the return value from tpci200_interrupt was not quite correct if a slot had caused an interrupt but no handler was instellalled: IRQ_NONE was returned. However in this case we react to the interrupt by disabling the IPack device interrupt. Basically there are two

[PATCH 13/24] Staging: ipack/bridges/tpci200: Cleanup in tpci200_slot_irq() and tpci200_interrupt()

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Minor cleanup. No functional changes. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/bridges/tpci200.c | 58 +-- 1 file changed, 31 insertions(+), 27 deletions(-) diff --git

[PATCH 14/24] Staging: ipack/bridges/tpci200: More cleanups.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Rename __tpci_request_irq() to tpci_enable_irq() and __tpci_free_irq() to tpci_disable_irq(). Change their second argument, move them above tpci200_interrupt(), and use tpci_disable_irq() in the latter Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez

[PATCH 17/24] Staging: ipack/devices/ipoctal: Clean up device removal.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Make use of dev_set_drvdata() and dev_get_drvdata() to store and obtain a pointer to the ipoctal struct corresponding to a struct dev. Previously we relied on a private list of devices. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez ---

[PATCH 18/24] Staging: ipack/devices/ipoctal: Check tty_register_device return value.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/staging/ipack/devices/ipoctal.c

[PATCH 19/24] Staging: ipack/devices/ipoctal: Use KBUILD_MODNAME instead of hardcoded string.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ipack/devices/ipoctal.c b/drivers/staging/ipack/devices/ipoctal.c

[PATCH 20/24] Staging: ipack/devices/ipoctal: Get rid of ipoctal_list.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Use tty_dev->dev's drdata to associate struct ipocal_channel to the respective tty_struct. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c | 39 +++ 1 file changed, 3

[PATCH 03/24] Staging: ipack/bridges/tpci200: Clean up interrupt handler.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge This also removes a possible bug in the unhandles_ints part when slots[i] is not set. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/bridges/tpci200.c | 42 +-- 1 file changed, 17

[PATCH 06/24] Staging: ipack/devices/ipoctal: put ipoctal_channel into tty->driver_data.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Each tty's driver_data is now pointing to the channel it is talking to. struct ipoctal_channel contains all the information the callbacks require to do their work. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez ---

[PATCH 22/24] Staging: ipack: update TODO file

2012-09-12 Thread Samuel Iglesias Gonsalvez
With the latest patches, the TODO file was outdated. Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/TODO | 25 ++--- 1 file changed, 2 insertions(+), 23 deletions(-) diff --git a/drivers/staging/ipack/TODO b/drivers/staging/ipack/TODO index

[PATCH 21/24] Staging: ipack/devices/ipoctal: read more than one character from RX FIFO.

2012-09-12 Thread Samuel Iglesias Gonsalvez
The RX FIFO has a size of 3 characters. Check if when we are picking the oldest one, we have more to read. Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c | 60 ++- 1 file changed, 36 insertions(+), 24 deletions(-) diff --git

[PATCH 15/24] Staging: ipack/bridges/tpci200: move tpci200_free_irq() and tpci200_request_irq()

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Now, all the interrupt related functions are next to each other. Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/bridges/tpci200.c | 154 +++ 1 file changed, 77 insertions(+), 77 deletions(-)

[PATCH 24/24] staging: ipack/bridges/tpci200: Use endianess-aware types where applicable.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/bridges/tpci200.c | 10 +- drivers/staging/ipack/bridges/tpci200.h |8 2 files changed, 9 insertions(+), 9 deletions(-) diff --git

[PATCH v2 7/8] drivers/media/platform/davinci/vpbe.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

[PATCH v2 6/8] drivers/media/dvb-frontends/tda665x.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

[PATCH v2 5/8] drivers/media/dvb-frontends/stb6100.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

[PATCH v2 4/8] drivers/media/dvb-frontends/s921.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

Re: [PATCH] coredump: prevent double-free on an error path in core dumper

2012-09-12 Thread Oleg Nesterov
On 09/12, Venu Byravarasu wrote: > > > --- a/fs/binfmt_elf.c > > +++ b/fs/binfmt_elf.c > > @@ -1695,30 +1695,19 @@ static int elf_note_info_init(struct elf_note_info > > *info) > > return 0; > > info->psinfo = kmalloc(sizeof(*info->psinfo), GFP_KERNEL); > > Why don't you change

[PATCH 3/3] thp: introduce khugepaged_cleanup_page

2012-09-12 Thread Xiao Guangrong
It is used to release the page on the fail path, then the page need not be cleaned up in khugepaged_prealloc_page anymore Signed-off-by: Xiao Guangrong --- mm/huge_memory.c | 19 +++ 1 files changed, 15 insertions(+), 4 deletions(-) diff --git a/mm/huge_memory.c

[PATCH v2 1/8] drivers/media/dvb-frontends/dvb_dummy_fe.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

[PATCH 1/3] thp: fix forgetting to reset the page alloc indicator

2012-09-12 Thread Xiao Guangrong
If NUMA is enabled, the indicator is not reset if the previous page request is failed, then it will trigger the VM_BUG_ON in khugepaged_alloc_page Signed-off-by: Xiao Guangrong --- mm/huge_memory.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/mm/huge_memory.c

[PATCH v2 3/8] drivers/media/dvb-frontends/s5h1432.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

[PATCH 2/3] thp: move release mmap_sem lock out of khugepaged_alloc_page

2012-09-12 Thread Xiao Guangrong
To make the code more clear, move release the lock out of khugepaged_alloc_page Signed-off-by: Xiao Guangrong --- mm/huge_memory.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/mm/huge_memory.c b/mm/huge_memory.c index 66d2bc6..5622347 100644 ---

Re: [PATCH 3/3] block: add back command filter modification via sysfs

2012-09-12 Thread Paolo Bonzini
Il 12/09/2012 14:41, Alan Cox ha scritto: >> > + if (!q->cmd_filter) { >> > + q->cmd_filter = kmalloc(sizeof(struct blk_cmd_filter), >> > + GFP_KERNEL); >> > + blk_set_cmd_filter_defaults(q->cmd_filter); >> > + } >> > + > This also needs

[PATCH v2 2/8] drivers/media/dvb-frontends/lg2160.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

[PATCH v2 8/8] drivers/media/tuners/mt2063.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged

[PATCH 23/24] Staging: ipack/devices/ipoctal: Unmap memory on device removal.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/devices/ipoctal.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/ipack/devices/ipoctal.c b/drivers/staging/ipack/devices/ipoctal.c index

[PATCH 16/24] Staging: ipack: Let interrupts return irqreturn_t.

2012-09-12 Thread Samuel Iglesias Gonsalvez
From: Jens Taprogge Signed-off-by: Jens Taprogge Signed-off-by: Samuel Iglesias Gonsalvez --- drivers/staging/ipack/bridges/tpci200.c |2 +- drivers/staging/ipack/bridges/tpci200.h |3 +-- drivers/staging/ipack/devices/ipoctal.c |2 +- drivers/staging/ipack/ipack.h |

[PATCH 0/3] thp: tidy and fix khugepaged_prealloc_page

2012-09-12 Thread Xiao Guangrong
There has a bug in khugepaged_prealloc_page, the page-alloc indicator is not reset if the previous page request is failed, then it will trigger the VM_BUG_ON in khugepaged_alloc_page. It is fixed by the first patch which need not be back port for it was introduced by recent commit. (sorry for

Re: [PATCH tip/core/rcu 11/26] rcu: Exit RCU extended QS on user preemption

2012-09-12 Thread Peter Zijlstra
On Wed, 2012-09-12 at 14:41 +0200, Peter Zijlstra wrote: > We could of course mandate that all remote wakeups to special nohz cpus > get queued. That would just leave us with RCU and it would simply not > send resched IPIs to extended quiescent CPUs anyway, right? > > So at that point all return

Re: [PATCH 11/11] ASoC: omap-pcm: Convert to use dmaengine

2012-09-12 Thread Peter Ujfalusi
On 09/12/2012 03:00 PM, Russell King - ARM Linux wrote: > On Wed, Sep 12, 2012 at 02:47:07PM +0300, Peter Ujfalusi wrote: >> 2. Pause/Resume >>OMAP DMA engine backend does not support pausing and resuming >>an in-progress transfer. It is unclear from the specs what >>effect clearing

Re: [PATCH] amd64_edac: Memory size reported double on processor family 0Fh

2012-09-12 Thread Josh Hunt
On 09/12/2012 07:38 AM, Josh Hunt wrote: > On 09/12/2012 03:51 AM, Borislav Petkov wrote: >> On Tue, Sep 11, 2012 at 06:02:01PM -0500, Josh Hunt wrote: >>> On 09/11/2012 05:52 PM, Josh Hunt wrote: With recent kernels we noticed that edac was reporting double the memory size on

Re: [alsa-devel] [PATCH] ASoC: codecs: Add DA9055 codec driver

2012-09-12 Thread Mark Brown
On Wed, Sep 12, 2012 at 06:26:18PM +0530, Ashish Chavan wrote: > On Wed, 2012-09-12 at 10:57 +0800, Mark Brown wrote: > > Why is any of this being exposed to userspace? If this should be > > configured I'd expect it to be static platform data, not something that > > gets changed at runtime. >

Re: [PATCH tip/core/rcu 11/26] rcu: Exit RCU extended QS on user preemption

2012-09-12 Thread Peter Zijlstra
On Wed, 2012-09-12 at 14:06 +0200, Frederic Weisbecker wrote: > > 1) This can happen if something calls set_need_resched() while no other task > is > on the runqueue. People really shouldn't be doing that... I think I know why RCU does this, but yuck. I also think RCU can avoid doing this, but

Re: [PATCH] uprobes: don't put NULL pointer in uprobe_register()

2012-09-12 Thread Srikar Dronamraju
* Sebastian Andrzej Siewior [2012-08-30 19:26:22]: > alloc_uprobe() might return a NULL pointer, put_uprobe() can't deal with > this. > > Signed-off-by: Sebastian Andrzej Siewior Acked-by: Srikar Dronamraju > --- > kernel/events/uprobes.c |3 ++- > 1 file changed, 2 insertions(+), 1

Re: [PATCH 03/20] Staging: ipack/bridges/tpci200: provide new callbacks to tpci200

2012-09-12 Thread Samuel Iglesias Gonsálvez
On Wed, 2012-09-12 at 14:28 +0200, Jens Taprogge wrote: [...] > Sam, > > can you please add the endianess-aware versions back? Patch attached. > > One of the earlier uses was not correct. Done! The patch will be submitted with the next batch. Thanks, Sam signature.asc Description: This

Re: [alsa-devel] [PATCH] ASoC: codecs: Add DA9055 codec driver

2012-09-12 Thread Ashish Chavan
On Wed, 2012-09-12 at 10:57 +0800, Mark Brown wrote: > > +/* Digital MIC clock rate select */ > > +static const char * const da9055_dmic_clk_rate_txt[] = { > > + "3MHz", "1.5MHz" > > +}; > > + > > +static const struct soc_enum da9055_dmic_clk_rate = > > + SOC_ENUM_SINGLE(DA9055_MIC_CONFIG, 2,

Re: [PATCH] drivers/media: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
I'll split this patch in one patch for file and resend. On Wed, Sep 12, 2012 at 10:06 AM, Dan Carpenter wrote: > On Tue, Sep 11, 2012 at 08:00:32PM +0200, Peter Senna Tschudin wrote: >> diff --git a/drivers/media/dvb-frontends/lg2160.c >> b/drivers/media/dvb-frontends/lg2160.c >> index

Re: [PATCH] amd64_edac: Memory size reported double on processor family 0Fh

2012-09-12 Thread Josh Hunt
On 09/12/2012 03:51 AM, Borislav Petkov wrote: > On Tue, Sep 11, 2012 at 06:02:01PM -0500, Josh Hunt wrote: >> On 09/11/2012 05:52 PM, Josh Hunt wrote: >>> With recent kernels we noticed that edac was reporting double the memory >>> size on >>> systems running with AMD family 0Fh processors. I'm

Re: [PATCH 8/7] uprobes: Fix arch_uprobe_disable_step() && UTASK_SSTEP_TRAPPED interaction

2012-09-12 Thread Srikar Dronamraju
* Oleg Nesterov [2012-09-08 19:06:23]: > arch_uprobe_disable_step() should also take UTASK_SSTEP_TRAPPED into > account. In this case the probed insn was not executed, we need to > clear X86_EFLAGS_TF if it was set by us and that is all. > > Again, this code will look more clean when we move it

Re: [PATCH 0/7] uprobes: single-step fixes

2012-09-12 Thread Srikar Dronamraju
* Oleg Nesterov [2012-09-08 19:06:01]: > On 09/03, Oleg Nesterov wrote: > > > > Sebastian, I changed your patches a bit: > > > > 1/7: > > > > - Change the subject and update the changelog. In particular, > > s/utrace/uprobes/. I am wondering where this typo came

Re: [PATCH 3/3] block: add back command filter modification via sysfs

2012-09-12 Thread Alan Cox
> +ssize_t blk_filter_store(struct request_queue *q, > + const char *page, size_t count, int rw) > +{ > + unsigned long okbits[BLK_SCSI_CMD_PER_LONG], *target_okbits; > + bool set; > + const char *p = page; > + char *endp; > + int start = -1, cmd; > + > +

WARNING: at kernel/rcutree.c:1558 rcu_do_batch+0x386/0x3a0(), during CPU hotplug

2012-09-12 Thread Srivatsa S. Bhat
On 07/19/2012 10:45 PM, Paul E. McKenney wrote: > On Thu, Jul 19, 2012 at 05:39:30PM +0530, Srivatsa S. Bhat wrote: >> Hi Paul, >> >> While running a CPU hotplug stress test on v3.5-rc7+ >> (mainline commit 8a7298b7805ab) I hit this warning. >> I haven't tried to debug this yet... >> >> Line

Re: [PATCH 3/3] block: add back command filter modification via sysfs

2012-09-12 Thread Alan Cox
O > + if (!q->cmd_filter) { > + q->cmd_filter = kmalloc(sizeof(struct blk_cmd_filter), > + GFP_KERNEL); > + blk_set_cmd_filter_defaults(q->cmd_filter); Out of memory - memset - oops -- To unsubscribe from this list: send the line

Re: [PATCH 2/3] scsi: create an all-zero filter for scanners

2012-09-12 Thread Paolo Bonzini
Il 12/09/2012 14:36, Alan Cox ha scritto: >> >switch (sdev->type) { >> > + case TYPE_SCANNER: >> > + sdev->request_queue->cmd_filter = >> > + kzalloc(sizeof(struct blk_cmd_filter), GFP_KERNEL); >> > + /* fallthrough */ > Is that really desireable in the

Re: [PATCH 7/7] uprobes: Make arch_uprobe_task->saved_trap_nr "unsigned int"

2012-09-12 Thread Srikar Dronamraju
* Oleg Nesterov [2012-09-03 17:26:20]: > Make arch_uprobe_task->saved_trap_nr "unsigned int" and move it down > after ->saved_scratch_register, this changes sizeof() from 24 to 16. > > Signed-off-by: Oleg Nesterov Acked-by: Srikar Dronamraju > --- > arch/x86/include/asm/uprobes.h |2

Re: [PATCH 2/3] scsi: create an all-zero filter for scanners

2012-09-12 Thread Alan Cox
On Wed, 12 Sep 2012 13:23:43 +0200 Paolo Bonzini wrote: > Using /dev/sg for scanners is blocked from unprivileged users. Reimplement > this using customizable command filters, so that the sysfs knobs will work > in this case too. > > Cc: linux-is...@vger.kernel.org > Signed-off-by: Paolo

Re: [PATCH v3] trace: Move trace event enable from fs_initcall to core_initcall

2012-09-12 Thread Ezequiel Garcia
Steven, On Tue, Sep 11, 2012 at 9:34 PM, Steven Rostedt wrote: ble(void) [...] >> + /* >> + * Early initialization already enabled ftrace event. >> + * Now it's only necessary to create the event directory. >> + */ >> for_each_event(call, __start_ftrace_events,

Re: [PATCH 03/20] Staging: ipack/bridges/tpci200: provide new callbacks to tpci200

2012-09-12 Thread Jens Taprogge
On Wed, Sep 12, 2012 at 02:59:26PM +0300, Dan Carpenter wrote: > On Wed, Sep 12, 2012 at 01:21:17PM +0200, Samuel Iglesias Gonsálvez wrote: > > On Wed, 2012-09-12 at 14:13 +0300, Dan Carpenter wrote: > > > On Wed, Sep 12, 2012 at 11:28:33AM +0200, Jens Taprogge wrote: > > > > On Tue, Sep 11, 2012

[PATCH v3] x86/mce: Honour bios-set CMCI threshold

2012-09-12 Thread Naveen N. Rao
The ACPI spec doesn't provide for a way for the bios to pass down recommended thresholds to the OS on a _per-bank_ basis. This patch adds a new boot option, which if passed, allows bios to initialize the CMCI threshold. In such a case, we simply skip programming any threshold value. As fail-safe,

[PATCH 1/2] leds-lp5523: turn off the LED engines on unloading the driver

2012-09-12 Thread Kim, Milo
The LP5523 has 3 engines which are used for running LED patterns. These engines should be off while unloading the driver. Obviously, LP5523 platform data are used for releasing the resource such like enable()/release_resource(), but these are not mandatory. Therefore this patch is required

[PATCH 2/2] leds-lp5523: Fix riskiness of the page fault

2012-09-12 Thread Kim, Milo
The last attribute should be terminated as NULL because any member of attribute structure is accessed while adding the sysfs file. If not, invalid address may cause the page fault problem. Signed-off-by: Milo(Woogyom) Kim --- drivers/leds/leds-lp5523.c |1 + 1 file changed, 1

Re: [PATCH 6/7] uprobes: Xol should send SIGTRAP if X86_EFLAGS_TF was set

2012-09-12 Thread Srikar Dronamraju
* Oleg Nesterov [2012-09-03 17:26:16]: > arch_uprobe_disable_step() correctly preserves X86_EFLAGS_TF and > returns to user-mode. But this means the application gets SIGTRAP > only after the next insn. > Agree. > This means that UPROBE_CLEAR_TF logic is not really right. _enable > should only

Re: [PATCH v3 02/31] arm64: Kernel booting and initialisation

2012-09-12 Thread Jean-Christophe PLAGNIOL-VILLARD
On 17:11 Mon 10 Sep , Catalin Marinas wrote: > On Sun, Sep 09, 2012 at 06:20:46PM +0100, Jean-Christophe PLAGNIOL-VILLARD > wrote: > > On 17:26 Fri 07 Sep , Catalin Marinas wrote: > > > +The device tree blob (dtb) must be no bigger than 2 megabytes in size > > > +and placed at a

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