[GIT PULL] perf fixes

2015-02-20 Thread Ingo Molnar
Linus, Please pull the latest perf-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git perf-urgent-for-linus # HEAD: b7e37567d080301d38a302bb93ba79d1ca446dca kprobes/x86: Mark 2 bytes NOP as boostable This tree contains two uprobes fixes, an

[GIT PULL] locking fixes

2015-02-20 Thread Ingo Molnar
Linus, Please pull the latest locking-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git locking-urgent-for-linus # HEAD: d6abfdb2022368d8c6c4be3f11a06656601a6cc2 x86/spinlocks/paravirt: Fix memory corruption on unlock Two fixes: the paravirt

[GIT PULL] IRQ fixes

2015-02-20 Thread Ingo Molnar
Linus, Please pull the latest irq-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq-urgent-for-linus # HEAD: d97eb8966c91f2c9d05f0a22eb89ed5b76d966d1 x86/irq: Check for valid irq descriptor in check_irq_vectors_for_cpu_disable() Two x86 irq

Re: [PATCH 1/2] kprobes/x86: Use 5-byte NOP when the code might be modified by ftrace

2015-02-20 Thread Petr Mladek
On Fri 2015-02-20 13:52:23, Ingo Molnar wrote: > > * Masami Hiramatsu wrote: > > > (2015/02/20 19:26), Ingo Molnar wrote: > > > > > > * Petr Mladek wrote: > > > > > >> can_probe() checks if the given address points to the > > >> beginning of an instruction. It analyzes all the > > >>

[RFT RESEND 3/4] power_supply: ipaq_micro_battery: Check return values in probe

2015-02-20 Thread Krzysztof Kozlowski
The return values of create_singlethread_workqueue() and power_supply_register() calls were not checked and even on error probe() function returned 0. 1. If allocation of workqueue failed (returning NULL) then further accesses could lead to NULL pointer dereference. The queue_delayed_work()

[RFT RESEND 4/4] power_supply: lp8788-charger: Fix leaked power supply on probe fail

2015-02-20 Thread Krzysztof Kozlowski
Driver forgot to unregister charger power supply if registering of battery supply failed in probe(). In such case the memory associated with power supply leaked. Signed-off-by: Krzysztof Kozlowski Fixes: 98a276649358 ("power_supply: Add new lp8788 charger driver") Cc: ---

[RFT RESEND 1/4] power_supply: twl4030_madc: Check return value of power_supply_register

2015-02-20 Thread Krzysztof Kozlowski
The return value of power_supply_register() call was not checked and even on error probe() function returned 0. If registering failed then during unbind the driver tried to unregister power supply which was not actually registered. This could lead to memory corruption because

Re: time / gtod seconds value out of sync?

2015-02-20 Thread Jan Stancek
- Original Message - > From: "Nishanth Aravamudan" > To: "John Stultz" > Cc: "Thomas Gleixner" , "lkml" > , jstan...@redhat.com, "Ingo Molnar" > > Sent: Thursday, 19 February, 2015 8:28:40 PM > Subject: Re: time / gtod seconds value out of sync? > > Hi John! > > On 19.02.2015

Re: [PATCH 0/3] x86, fpu: more eagerfpu cleanups

2015-02-20 Thread Oleg Nesterov
On 02/20, Borislav Petkov wrote: > > On Mon, Jan 19, 2015 at 07:51:09PM +0100, Oleg Nesterov wrote: > > math_state_restore() and its usage doesn't look nice. init_fpu() too, > > Yeah, about that: > > I see: > > math_state_restore > ... >if (!tsk_used_math(tsk)) > init_fpu() >

[RFT RESEND 2/4] power_supply: ipaq_micro_battery: Fix leaking workqueue

2015-02-20 Thread Krzysztof Kozlowski
Driver allocates singlethread workqueue in probe but it is not destroyed during removal. Signed-off-by: Krzysztof Kozlowski Fixes: 00a588f9d27f ("power: add driver for battery reading on iPaq h3xxx") Cc: --- drivers/power/ipaq_micro_battery.c | 1 + 1 file changed, 1 insertion(+) diff --git

[GIT PULL] RCU fix

2015-02-20 Thread Ingo Molnar
Linus, Please pull the latest core-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git core-urgent-for-linus # HEAD: 3b3336d4fed58ea5a019a8a1d00fa741be492716 Merge branch 'rcu/next' of git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu

Re: [PATCH RESEND v9 08/10] sched: replace capacity_factor by usage

2015-02-20 Thread Vincent Guittot
On 20 February 2015 at 12:14, Peter Zijlstra wrote: > On Thu, Jan 15, 2015 at 11:09:28AM +0100, Vincent Guittot wrote: > >> Finally, the sched_group->sched_group_capacity->capacity_orig has been >> removed >> because it's no more used during load balance. > > Maybe do that in a separate patch to

[PATCH] cxl: Remove useless precision specifiers

2015-02-20 Thread Rasmus Villemoes
C99 says that a precision given as simply '.' with no following digits or * should be interpreted as 0. The kernel's printf implementation, however, treats this case as if the precision was omitted. C99 also says that if both the precision and value are 0, no digits should be printed. Even if the

Re: [PATCH 1/2] kprobes/x86: Use 5-byte NOP when the code might be modified by ftrace

2015-02-20 Thread Masami Hiramatsu
(2015/02/20 21:52), Ingo Molnar wrote: > > * Masami Hiramatsu wrote: > >> (2015/02/20 19:26), Ingo Molnar wrote: >>> >>> * Petr Mladek wrote: >>> can_probe() checks if the given address points to the beginning of an instruction. It analyzes all the instructions from the

Re: [PATCH 2/4] of: DT quirks infrastructure

2015-02-20 Thread Peter Hurley
On 02/19/2015 09:41 AM, Pantelis Antoniou wrote: > Hi Frank, > >> On Feb 19, 2015, at 04:08 , Frank Rowand wrote: >> >> On 2/18/2015 6:59 AM, Pantelis Antoniou wrote: >>> Implement a method of applying DT quirks early in the boot sequence. >>> >>> A DT quirk is a subtree of the boot DT that can

[PATCH 2/2] [PATCH] fix checkpatch.pl errors in kvm/irqchip.c

2015-02-20 Thread Kevin Mulvey
Fix whitespace around while Signed-off-by: Kevin Mulvey --- virt/kvm/irqchip.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/virt/kvm/irqchip.c b/virt/kvm/irqchip.c index 7f256f3..1d56a90 100644 --- a/virt/kvm/irqchip.c +++ b/virt/kvm/irqchip.c @@ -105,7 +105,7 @@ int

[PATCH 1/2] white space formatting in kvm_main.c

2015-02-20 Thread Kevin Mulvey
Better alignment of loop using tabs rather than spaces, this makes checkpatch.pl happier. Signed-off-by: Kevin Mulvey --- virt/kvm/kvm_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index a109370..36ab89d 100644 ---

Re: [PATCH] clockevents: Add (missing) default case for switch blocks

2015-02-20 Thread Ingo Molnar
* Viresh Kumar wrote: > On 20 February 2015 at 17:11, Ingo Molnar wrote: > > > > * Peter Zijlstra wrote: > > >> Maybe we should break that enum into two; one for devices > >> and one for the core interface and avoid the problem that > >> way. > > > > Yeah, that would do the trick. > >

[RFT PATCH v3 1/2] compal-laptop: Fix leaking hwmon device

2015-02-20 Thread Krzysztof Kozlowski
The commit c2be45f09bb0 ("compal-laptop: Use devm_hwmon_device_register_with_groups") wanted to change the registering of hwmon device to resource-managed version. It mostly did it except the main thing - it forgot to use devm-like function so the hwmon device leaked after device removal or probe

[PATCH] staging: emxx_udc: Replace GFP_KERNEL with GFP_ATOMIC

2015-02-20 Thread Tapasweni Pathak
To avoid deadlock, do not call blocking functions with spinlocks held. Replace GFP_KERNEL with GFP_ATOMIC, as the latter will fail if the pile doesn't have enough free pages but will not sleep and hence deadlock can be avoided. Found by Coccinelle. Signed-off-by: Tapasweni Pathak --- Is there

[RFT PATCH v3 2/2] compal-laptop: Check return value of power_supply_register

2015-02-20 Thread Krzysztof Kozlowski
The return value of power_supply_register() call was not checked and even on error probe() function returned 0. If registering failed then during unbind the driver tried to unregister power supply which was not actually registered. This could lead to memory corruption because

[PATCH] staging: android: ion: fix some format strings

2015-02-20 Thread Rasmus Villemoes
C99 says that a precision which is simply '.' with no following digits or * should be interpreted as 0, which means that these format strings actually mean 'print 16 spaces'. However, the kernel's printf implementation treats this case as if the precision was omitted. Don't rely on that quirk.

Re: [RFT PATCH v2 2/2] compal-laptop: Check return value of power_supply_register

2015-02-20 Thread Krzysztof Kozlowski
On pią, 2015-02-20 at 14:04 +0100, Krzysztof Kozlowski wrote: > The return value of power_supply_register() call was not checked and > even on error probe() function returned 0. If registering failed then > during unbind the driver tried to unregister power supply which was not > actually

Re: usb serial: pl2303 driver TxD "break" stays after close() bug

2015-02-20 Thread Jari Ruusu
On 2/20/15, Johan Hovold wrote: > On Thu, Feb 19, 2015 at 03:38:39PM +0200, Jari Ruusu wrote: >> To clear it, you need to poke it with ioctl(fd, TIOCCBRK, 0) >> or disconnect the device. > > That's definitely a bug. > > Care to test the patch below? Your patch fixes the bug. Thanks. -- Jari

[RFT PATCH v2 2/2] compal-laptop: Check return value of power_supply_register

2015-02-20 Thread Krzysztof Kozlowski
The return value of power_supply_register() call was not checked and even on error probe() function returned 0. If registering failed then during unbind the driver tried to unregister power supply which was not actually registered. This could lead to memory corruption because

[PATCH] usb: gadget: function: Replace GFP_KERNEL with GFP_ATOMIC

2015-02-20 Thread Tapasweni Pathak
To avoid deadlock, do not call blocking functions with spinlocks held. Replace GFP_KERNEL with GFP_ATOMIC, as the latter will fail if the heap doesn't have enough free pages but will not sleep and hence deadlock can be avoided. Found by Coccinelle. Signed-off-by: Tapasweni Pathak --- Is there

[RFT PATCH v2 1/2] compal-laptop: Fix leaking hwmon device

2015-02-20 Thread Krzysztof Kozlowski
The commit c2be45f09bb0 ("compal-laptop: Use devm_hwmon_device_register_with_groups") wanted to change the registering of hwmon device to resource-managed version. It mostly did it except the main thing - it forgot to use devm-like function so the hwmon device leaked after device removal or probe

Re: [PATCH 1/2] kprobes/x86: Use 5-byte NOP when the code might be modified by ftrace

2015-02-20 Thread Ingo Molnar
* Masami Hiramatsu wrote: > (2015/02/20 19:26), Ingo Molnar wrote: > > > > * Petr Mladek wrote: > > > >> can_probe() checks if the given address points to the > >> beginning of an instruction. It analyzes all the > >> instructions from the beginning of the function until the > >> given

[PATCH V2 2/2] perf/x86: Provide TSC for PERF_SAMPLE_CLOCK_PT

2015-02-20 Thread Adrian Hunter
Provide TSC for PERF_SAMPLE_CLOCK_PT. This is needed to match perf events against hardware traces like Intel Processor Trace (Intel PT) which is the purpose for which PERF_SAMPLE_CLOCK_PT was invented. Signed-off-by: Adrian Hunter --- arch/x86/include/asm/perf_event.h | 6 ++

[PATCH V2 0/2] perf/x86: Add ability to sample TSC

2015-02-20 Thread Adrian Hunter
Hi TSC is needed to synchronize Intel Processor Trace (Intel PT) with perf event samples. Refer to patch 1 for more details. There is a description of Intel PT in the Intel Architecture manuals: http://www.intel.com/content/www/us/en/processors/architectures-software-developer-manuals.html

[PATCH V2 1/2] perf: Sample additional clock value

2015-02-20 Thread Adrian Hunter
This is needed to allow perf event samples to be synchronized with data from other sources, and in particular, sources like Intel Processor Trace (Intel PT) where the hardware produces a trace with hardware defined timestamps (i.e. TSC). For example, to decode an Intel PT trace, the decoder must

Re: [PATCH 1/2] kprobes/x86: Use 5-byte NOP when the code might be modified by ftrace

2015-02-20 Thread Masami Hiramatsu
(2015/02/20 19:26), Ingo Molnar wrote: > > * Petr Mladek wrote: > >> can_probe() checks if the given address points to the >> beginning of an instruction. It analyzes all the >> instructions from the beginning of the function until the >> given address. The code might be modified by another

[PATCH] Drivers: hv: hv_balloon: keep locks balanced on add_memory() failure

2015-02-20 Thread Vitaly Kuznetsov
When add_memory() fails the following BUG is observed: [ 743.646107] hv_balloon: hot_add memory failed error is -17 [ 743.679973] [ 743.680930] = [ 743.680930] [ BUG: bad unlock balance detected! ] [ 743.680930] 3.19.0-rc5_bug1131426+ #552 Not tainted [

[PATCH] drivers: dma: Remove extra check

2015-02-20 Thread Tapasweni Pathak
Remove double check on chan->desc. Found by Coccinelle. Signed-off-by: Tapasweni Pathak Acked-by: Julia Lawall --- drivers/dma/dma-jz4740.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/dma-jz4740.c b/drivers/dma/dma-jz4740.c index 4527a3e..f6a4c42 100644

Re: [PATCH] fbdev: via/via_clock: fix sparse warning

2015-02-20 Thread Tomi Valkeinen
On 05/02/15 14:54, Lad Prabhakar wrote: > From: "Lad, Prabhakar" > > this patch fixes following sparse warning: > > via_clock.c:33:12: warning: symbol 'via_slap' was not declared. Should it be > static? > > Signed-off-by: Lad, Prabhakar > --- > Found this issue on linux-next (gcc version

Re: [PATCH] video: fbdev: use msecs_to_jiffies for time conversion

2015-02-20 Thread Tomi Valkeinen
On 06/02/15 11:14, Nicholas Mc Guire wrote: > This is only an API consolidation and should make things more readable > it replaces var * HZ / 1000 by msecs_to_jiffies(var). > > Signed-off-by: Nicholas Mc Guire > --- > > Patch was only compile tested with viper_defconfig (implies

Re: [PATCH] video: fbdev: make of_device_id array const

2015-02-20 Thread Tomi Valkeinen
On 09/02/15 13:02, Sanjeev Sharma wrote: > Make of_device_id array const. > > Signed-off-by: Sanjeev Sharma > --- > drivers/video/fbdev/imxfb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/fbdev/imxfb.c b/drivers/video/fbdev/imxfb.c > index

Re: [PATCH] fs: record task name which froze superblock

2015-02-20 Thread Jan Kara
On Fri 20-02-15 14:42:29, Alexey Dobriyan wrote: > On Wed, Feb 18, 2015 at 12:13 PM, Jan Kara wrote: > >> > > --- a/fs/ioctl.c > >> > > +++ b/fs/ioctl.c > >> > > @@ -518,6 +518,7 @@ static int ioctl_fioasync(unsigned int fd, struct > >> > > file *filp, > >> > > static int ioctl_fsfreeze(struct

Re: [PATCH] video: fbdev: fix possible null dereference

2015-02-20 Thread Tomi Valkeinen
On 12/02/15 17:47, Sudip Mukherjee wrote: > we were dereferencing edid first and the NULL check was after > accessing that. now we are using edid only if we know that > it is not NULL. > > Signed-off-by: Sudip Mukherjee > --- > drivers/video/fbdev/core/fbmon.c | 6 +++--- > 1 file changed, 3

Re: [PATCH] video: ARM CLCD: Add missing error check for devm_kzalloc

2015-02-20 Thread Tomi Valkeinen
On 11/02/15 11:36, Kiran Padwal wrote: > This patch add a missing check on the return value of devm_kzalloc, > which would cause a NULL pointer dereference in a OOM situation. > > Signed-off-by: Kiran Padwal > --- > drivers/video/fbdev/amba-clcd.c |3 +++ > 1 file changed, 3 insertions(+) >

Re: [PATCH 1/6] ASoC: max98088: Document DT bindings

2015-02-20 Thread Sylwester Nawrocki
On 20/02/15 01:36, Andreas Färber wrote: >>> >> So it seems the mclk is not always set up properly by the kernel, >>> >> relying on firmware. Who's in charge of setting that clock up? >> > >> > Right, it seems audio is only working due the firmware doing some previous >> > setup. Probably it

Re: [PATCH 0/3] x86, fpu: more eagerfpu cleanups

2015-02-20 Thread Borislav Petkov
On Mon, Jan 19, 2015 at 07:51:09PM +0100, Oleg Nesterov wrote: > math_state_restore() and its usage doesn't look nice. init_fpu() too, Yeah, about that: I see: math_state_restore ... if (!tsk_used_math(tsk)) init_fpu() and init_fpu() then does: if

Re: [PATCH] fbdev: sm501fb: use memset_io

2015-02-20 Thread Tomi Valkeinen
On 10/02/15 14:37, Sudip Mukherjee wrote: > we should really be using memset_io() instead of using memset() as > this is actually io space mapped into our memory. > > Signed-off-by: Sudip Mukherjee > --- > it has only been build tested. I only have the hardware of sm712 and sm750. > >

Re: [PATCH v6 2/2] fbcon: expose cursor blink interval via sysfs

2015-02-20 Thread Tomi Valkeinen
On 30/01/15 11:40, Scot Doyle wrote: > fbcon toggles cursor display state every 200 milliseconds when blinking. > Since users prefer different toggle intervals, expose the interval via > /sys/class/graphics/fbcon/cursor_blink_ms so that it may be customized. > > Values written to the interface

RE: [PATCH 0/2] bmc150-accel driver optimizations

2015-02-20 Thread Tirdea, Irina
> -Original Message- > From: linux-iio-ow...@vger.kernel.org > [mailto:linux-iio-ow...@vger.kernel.org] On Behalf Of Peter Meerwald > Sent: 16 February, 2015 21:27 > To: Tirdea, Irina > Cc: Jonathan Cameron; linux-...@vger.kernel.org; > linux-kernel@vger.kernel.org; Pandruvada,

RE: [PATCH 0/2] kxcjk-1013 driver optimizations

2015-02-20 Thread Tirdea, Irina
> -Original Message- > From: Peter Meerwald [mailto:pme...@pmeerw.net] > Sent: 16 February, 2015 21:14 > To: Tirdea, Irina > Cc: Jonathan Cameron; linux-...@vger.kernel.org; > linux-kernel@vger.kernel.org; Pandruvada, Srinivas; Reus, Adriana > Subject: Re: [PATCH 0/2] kxcjk-1013 driver

RE: [PATCH 2/2] iio: accel: kxcjk-1013: optimize i2c transfers in trigger handler

2015-02-20 Thread Tirdea, Irina
> -Original Message- > From: Peter Meerwald [mailto:pme...@pmeerw.net] > Sent: 16 February, 2015 21:26 > To: Tirdea, Irina > Cc: Jonathan Cameron; linux-...@vger.kernel.org; > linux-kernel@vger.kernel.org; Pandruvada, Srinivas; Reus, Adriana > Subject: Re: [PATCH 2/2] iio: accel:

Re: [PATCH] dmaengine: qcom_bam_dma: Add support for BAM v1.7.0

2015-02-20 Thread Stanimir Varbanov
On 01/22/2015 09:04 AM, Archit Taneja wrote: > Add register offset table entry for the newer (v1.7.0) version of the BAM IP > found on MSM8916. Update the DT bindings documentation. > > Signed-off-by: Archit Taneja > --- > .../devicetree/bindings/dma/qcom_bam_dma.txt | 1 + >

Re: [PATCH] clockevents: Add (missing) default case for switch blocks

2015-02-20 Thread Viresh Kumar
On 20 February 2015 at 17:11, Ingo Molnar wrote: > > * Peter Zijlstra wrote: >> Maybe we should break that enum into two; one for devices >> and one for the core interface and avoid the problem that >> way. > > Yeah, that would do the trick. Thanks for your suggestions. Just to confirm (before

Re: [B.A.T.M.A.N.] checkpatch: CHECK: No space is necessary after a cast

2015-02-20 Thread Marek Lindner
On Thursday, February 19, 2015 11:55:41 Joe Perches wrote: > > Can you make a suggestion / patch for checkpatch to better handle this > > case ? > The "sizeof" test in the current script doesn't work. > > I believe the patch below works with no false positives > but it's perl regexes against odd

Re: [Xen-devel] NUMA_BALANCING and Xen PV guest regression in 3.20-rc0

2015-02-20 Thread Andrew Cooper
On 20/02/15 11:29, Kirill A. Shutemov wrote: > On Fri, Feb 20, 2015 at 10:47:52AM +, Andrew Cooper wrote: >> On 20/02/15 01:49, Linus Torvalds wrote: >>> On Thu, Feb 19, 2015 at 5:05 PM, Kirill A. Shutemov >>> wrote: I'm feeling I miss very basic background on how Xen works, but why does

Re: [PATCH RESEND v9 00/10] sched: consolidation of CPU capacity and usage

2015-02-20 Thread Morten Rasmussen
On Fri, Feb 20, 2015 at 11:34:47AM +, Peter Zijlstra wrote: > On Thu, Feb 19, 2015 at 12:49:40PM +, Morten Rasmussen wrote: > > > Also, it still not clear why patch 10 uses relative capacity reduction > > instead of absolute capacity available to CFS tasks. > > As present in your

Re: [PATCH] net: macb: Add big endian CPU support

2015-02-20 Thread Arun Chandran
On Thu, Feb 19, 2015 at 5:52 PM, Michal Simek wrote: > > On 02/18/2015 12:29 PM, Arun Chandran wrote: > > This patch converts all __raw_readl and __raw_writel function calls > > to their corresponding readl_relaxed and writel_relaxed variants. > > > > It also tells the driver to set

Re: [PATCH] fs: record task name which froze superblock

2015-02-20 Thread Alexey Dobriyan
On Wed, Feb 18, 2015 at 12:13 PM, Jan Kara wrote: > On Wed 18-02-15 10:34:55, Alexey Dobriyan wrote: >> On Mon, Feb 16, 2015 at 10:38:52AM +0100, Jan Kara wrote: >> > On Sat 14-02-15 21:55:24, Alexey Dobriyan wrote: >> > > Freezing and thawing are separate system calls, task which is supposed >>

Re: [PATCH] clockevents: Add (missing) default case for switch blocks

2015-02-20 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Fri, Feb 20, 2015 at 10:36:59AM +0100, Ingo Molnar wrote: > > > But it does mean we need to be able to add values to > > > the enum. > > > > So I'm confused: if we are using proper callbacks (like > > my example outlined) , why is a 'mode enum' needed at > >

Re: [PATCH] clockevents: Add (missing) default case for switch blocks

2015-02-20 Thread Peter Zijlstra
On Fri, Feb 20, 2015 at 10:36:59AM +0100, Ingo Molnar wrote: > > But it does mean we need to be able to add values to the > > enum. > > So I'm confused: if we are using proper callbacks (like my > example outlined) , why is a 'mode enum' needed at all? Ah, its because the enum is shared

Re: [PATCH RESEND v9 00/10] sched: consolidation of CPU capacity and usage

2015-02-20 Thread Peter Zijlstra
On Thu, Feb 19, 2015 at 12:49:40PM +, Morten Rasmussen wrote: > Also, it still not clear why patch 10 uses relative capacity reduction > instead of absolute capacity available to CFS tasks. As present in your asymmetric big and small systems? Yes it would be unfortunate to migrate a task to

Re: [Xen-devel] NUMA_BALANCING and Xen PV guest regression in 3.20-rc0

2015-02-20 Thread Kirill A. Shutemov
On Fri, Feb 20, 2015 at 10:47:52AM +, Andrew Cooper wrote: > On 20/02/15 01:49, Linus Torvalds wrote: > > On Thu, Feb 19, 2015 at 5:05 PM, Kirill A. Shutemov > > wrote: > >> I'm feeling I miss very basic background on how Xen works, but why does it > >> set _PAGE_GLOBAL on userspace entries?

Re: [PATCH RESEND v9 10/10] sched: move cfs task on a CPU with higher capacity

2015-02-20 Thread Peter Zijlstra
On Thu, Jan 15, 2015 at 11:09:30AM +0100, Vincent Guittot wrote: > As a sidenote, this will note generate more spurious ilb because we already s/note/not/ > trig an ilb if there is more than 1 busy cpu. If this cpu is the only one that > has a task, we will trig the ilb once for migrating the

Re: [PATCH v2 1/2] dma: at_hdmac: Fix calculation of the residual bytes

2015-02-20 Thread Nicolas Ferre
Le 19/02/2015 18:07, Torsten Fleischer a écrit : > From: Torsten Fleischer > > This patch fixes the following issues regarding to the calculation of the > residue: > > 1. The residue is always calculated for the current transfer even if the > cookie is associated to a pending transfer. > > 2.

[PATCH v10 2/2] dt-bindings: add vendor-prefix for X-Powers

2015-02-20 Thread Chen-Yu Tsai
From: Carlo Caione Signed-off-by: Carlo Caione Signed-off-by: Chen-Yu Tsai Acked-by: Lee Jones --- Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt

[PATCH v10 0/2] mfd: axp20x: Add DT binding documentation

2015-02-20 Thread Chen-Yu Tsai
Hi everyone, This is v10 of the axp20x series. v10 reverts back to the DT bindings from v8. This is the binding currently supported by the kernel drivers since 3.18. Mark Brown raised concern on the needless changes to active regulator bindings in v9. Maxime, I dropped your Acked-by because of

[PATCH v10 1/2] mfd: AXP20x: Add bindings documentation

2015-02-20 Thread Chen-Yu Tsai
From: Carlo Caione Bindings documentation for the AXP20x driver. In this file also sub-nodes are documented. Signed-off-by: Carlo Caione [w...@csie.org: clarify interrupt source for the axp PMIC] [w...@csie.org: explain dcdc-workmode in detail and trim lines to 80 chars] [w...@csie.org: make

Re: [PATCH RESEND v9 08/10] sched: replace capacity_factor by usage

2015-02-20 Thread Peter Zijlstra
On Thu, Jan 15, 2015 at 11:09:28AM +0100, Vincent Guittot wrote: > Finally, the sched_group->sched_group_capacity->capacity_orig has been removed > because it's no more used during load balance. Maybe do that in a separate patch to avoid cluttering this one? > [1]

Re: [PATCH V2 RESEND] clk: mxs: Fix invalid 32-bit access to frac registers

2015-02-20 Thread Stefan Wahren
Hi Fabio, > Stefan Wahren hat am 11. Februar 2015 um 21:31 > geschrieben: > > > Hi Fabio, > > > Fabio Estevam hat am 11. Februar 2015 um 17:58 > > geschrieben: > > > > > > Hi Stefan, > > > > On Tue, Feb 10, 2015 at 7:24 PM, Stefan Wahren > > wrote: > > > Hi Fabio, > > > > > >> Fabio Estevam

Re: Fw: [PATCH] lib/kstrtox.c Stop parsing integer on overflow

2015-02-20 Thread Alexey Dobriyan
On Thu, Feb 19, 2015 at 8:53 PM, Anshul Garg wrote: > On Wed, Feb 18, 2015 at 1:14 PM, Alexey Dobriyan wrote: >> On Tue, Feb 17, 2015 at 04:17:24PM -0800, Andrew Morton wrote: >>> ? >>> >>> Begin forwarded message: >>> >>> Date: Mon, 16 Feb 2015 10:48:50 -0800 >>> From: Anshul Garg >>> To:

Re: live patching design (was: Re: [PATCH 1/3] sched: add sched_task_call())

2015-02-20 Thread Jiri Kosina
On Fri, 20 Feb 2015, Ingo Molnar wrote: > - the complicated method spread out over time: uses the > same essential mechanism plus the ftrace patching > machinery to detect whether all tasks have transitioned > through a version flip. [this is what kgraft does in > part.]

Call for Papers

2015-02-20 Thread MRI Net Journals
Microbiology Research International Dear Colleague, Microbiology Research International (MRI) is an international peer-reviewed open access journal, which publishes innovative work from all areas related to Microbiology. MRI covers a wide range of problems in the areas of fundamental and

[patch] w1: small cleanup in w1_family_notify()

2015-02-20 Thread Dan Carpenter
"sl->family->fops" and "fops" are the same thing, but it's nicer to use "fops" everywhere. Signed-off-by: Dan Carpenter diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c index 181f41c..59f932f 100644 --- a/drivers/w1/w1.c +++ b/drivers/w1/w1.c @@ -649,7 +649,7 @@ static int

Re: [PATCH] clockevents: Add (missing) default case for switch blocks

2015-02-20 Thread Ingo Molnar
* Viresh Kumar wrote: > > So why is a 'default' mode needed then? It makes the > > addition of new modes to the legacy handler easier, > > which looks backwards. > > The requirement was to add another mode ONESHOT_STOPPED > [1], to be supported only by the new per-mode callbacks.. Why

Re: [Xen-devel] NUMA_BALANCING and Xen PV guest regression in 3.20-rc0

2015-02-20 Thread Andrew Cooper
On 20/02/15 01:49, Linus Torvalds wrote: > On Thu, Feb 19, 2015 at 5:05 PM, Kirill A. Shutemov > wrote: >> I'm feeling I miss very basic background on how Xen works, but why does it >> set _PAGE_GLOBAL on userspace entries? It sounds strange to me. > It is definitely strange. I'm guessing that

live patching design (was: Re: [PATCH 1/3] sched: add sched_task_call())

2015-02-20 Thread Ingo Molnar
* Jiri Kosina wrote: > On Fri, 20 Feb 2015, Ingo Molnar wrote: > > > So if your design is based on being able to discover > > > 'live' functions in the kernel stack dump of all tasks > > in the system, I think you need a serious reboot of the > > whole approach and get rid of that fragility

Re: [PATCH v4] dma: Add Xilinx AXI Direct Memory Access Engine driver support

2015-02-20 Thread Michal Simek
Hi Vinod, On 02/20/2015 07:29 AM, Vinod Koul wrote: > On Sat, Feb 14, 2015 at 02:21:52PM +, Appana Durga Kedareswara Rao wrote: >> Hi Vinod, >> >> >>> -Original Message- >>> From: Vinod Koul [mailto:vinod.k...@intel.com] >>> Sent: Thursday, February 12, 2015 3:05 PM >>> To: Appana

Re: [PATCH v2] x86: fix output of show_stack_log_lvl()

2015-02-20 Thread Borislav Petkov
On Fri, Feb 20, 2015 at 09:10:03AM +0100, Ingo Molnar wrote: > This approach looks good to me, we want to print multi-line > messages with the same consistent loglevel. Right, I'll pick this one up for now as it is obviously correct and whatever we end up doing to pr_cont() won't influence it.

Re: [PATCH v4 3/5] irqchip: Add DT binding doc for the virtual irq demuxer chip

2015-02-20 Thread Mark Rutland
[...] > > > IRQF_NO_SUSPEND and wakeup fundamentally don't match due to the way > > > wakeup is implemented in the IRQ core now. > > > > > > Unless drivers with IRQF_NO_SUSPEND do the wakeup behind the core's back > > > which is just disgusting and should never happen. > > > > I completely

[tip:x86/urgent] x86/mm/ASLR: Avoid PAGE_SIZE redefinition for UML subarch

2015-02-20 Thread tip-bot for Jiri Kosina
Commit-ID: 570e1aa84c376ff39809442f09c7606ddf62cfd1 Gitweb: http://git.kernel.org/tip/570e1aa84c376ff39809442f09c7606ddf62cfd1 Author: Jiri Kosina AuthorDate: Fri, 20 Feb 2015 10:18:59 +0100 Committer: Ingo Molnar CommitDate: Fri, 20 Feb 2015 10:55:32 +0100 x86/mm/ASLR: Avoid

[tip:x86/asm] x86/asm: Add support for the pcommit instruction

2015-02-20 Thread tip-bot for Ross Zwisler
Commit-ID: 719d359dc7b6be3e43d6661f192ceb980b10ee26 Gitweb: http://git.kernel.org/tip/719d359dc7b6be3e43d6661f192ceb980b10ee26 Author: Ross Zwisler AuthorDate: Thu, 19 Feb 2015 10:37:28 -0700 Committer: Ingo Molnar CommitDate: Fri, 20 Feb 2015 09:43:36 +0100 x86/asm: Add support for

Re: [PATCH v5 2/3] mm: cma: allocation trigger

2015-02-20 Thread Sasha Levin
Stefan, Andrew, I ended up cherry-picking and older patch here by mistake. Joonsoo pointed it out but I didn't have time to address it yet since I'm travelling and they got pulled in to mmotm in the meanwhile. I'll send out patches to add documentation and fix the issues raised here early next

Re: [Xen-devel] NUMA_BALANCING and Xen PV guest regression in 3.20-rc0

2015-02-20 Thread David Vrabel
On 19/02/15 23:09, Linus Torvalds wrote: > On Thu, Feb 19, 2015 at 5:06 AM, David Vrabel wrote: >> >> The NUMA_BALANCING series beginning with 5d833062139d (mm: numa: do not >> dereference pmd outside of the lock during NUMA hinting fault) and >> specifically 8a0516ed8b90 (mm: convert

Re: [PATCH v4 4/4] phy: add phy-hi6220-usb

2015-02-20 Thread zhangfei
On 02/20/2015 12:38 PM, Felipe Balbi wrote: On Fri, Feb 20, 2015 at 11:07:21AM +0800, zhangfei wrote: Hi, Balbi On 02/18/2015 10:35 PM, Felipe Balbi wrote: On Wed, Feb 18, 2015 at 01:44:21PM +0800, zhangfei wrote: Hi, Kishon On 02/18/2015 01:35 PM, Kishon Vijay Abraham I wrote: Hi, On

Re: [PATCH 1/2] kprobes/x86: Use 5-byte NOP when the code might be modified by ftrace

2015-02-20 Thread Ingo Molnar
* Petr Mladek wrote: > can_probe() checks if the given address points to the > beginning of an instruction. It analyzes all the > instructions from the beginning of the function until the > given address. The code might be modified by another > Kprobe. In this case, the current code is read

Re: [RFC PATCH 1/3] eeprom: Add a simple EEPROM framework

2015-02-20 Thread Srinivas Kandagatla
On 20/02/15 08:14, Srinivas Kandagatla wrote: Doesn't this need some sort of select REGMAP somewhere? May be depends REGMAP would be good. You are right, just realized that it should be "select REGMAP" and for QFPROM it should be "select REGMAP_MMIO" --srini -- To unsubscribe from this

Re: [PATCH] [media] [kbuild] Add and use IS_REACHABLE macro

2015-02-20 Thread Michal Marek
On 2015-02-20 10:29, Arnd Bergmann wrote: > On Thursday 19 February 2015 16:06:18 Michal Marek wrote: >>> We have similar problems in other areas >>> of the kernel. In theory, we could enforce the VIDEO_TUNER driver to >>> be modular here by adding lots of dependencies to it: >>> >>> config

[bug] logger 2.26: regression, option -i not combinable with other options

2015-02-20 Thread Juergen Daubert
Hello, with logger from util-linux 2.25.x it's possible to combine the option -i with other options like: logger -is with 2.26 I get the following error message: $:~> logger -is logger: failed to parse id: 's' The changed behaviour breaks existing scripts like dhcpcd-run-hooks [1] from

Re: [PATCH 2/2] kprobes/x86: Check for invalid ftrace location in __recover_probed_insn()

2015-02-20 Thread Ingo Molnar
* Petr Mladek wrote: > __recover_probed_insn() should always be called from an address where > an instructions starts. The check for ftrace_location() might help to > discover a potential inconsistency. Something goes terribly wrong when > an address inside the ftrace location is checked. Let's

[PATCH 1/2] kprobes/x86: Use 5-byte NOP when the code might be modified by ftrace

2015-02-20 Thread Petr Mladek
can_probe() checks if the given address points to the beginning of an instruction. It analyzes all the instructions from the beginning of the function until the given address. The code might be modified by another Kprobe. In this case, the current code is read into a buffer, int3 breakpoint is

Re: [PATCH v2] kprobes/x86: Use 5-byte NOP when the code might be modified by ftrace

2015-02-20 Thread Petr Mladek
On Wed 2015-02-18 22:22:05, Ingo Molnar wrote: > > * Petr Mladek wrote: > > > On Tue 2015-02-03 16:41:39, Masami Hiramatsu wrote: > > > (2015/02/03 2:48), Petr Mladek wrote: > > > > can_probe() checks if the given address points to the beginning of > > > > an instruction. It analyzes all the

[PATCH 2/2] kprobes/x86: Check for invalid ftrace location in __recover_probed_insn()

2015-02-20 Thread Petr Mladek
__recover_probed_insn() should always be called from an address where an instructions starts. The check for ftrace_location() might help to discover a potential inconsistency. Something goes terribly wrong when an address inside the ftrace location is checked. Let's BUG() in this case.

Re: [GIT PULL] Btrfs

2015-02-20 Thread Markus Trippelsdorf
On 2015.02.19 at 15:36 -0500, Chris Mason wrote: > Hi Linus, > > Please pull my for-linus branch: > > git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus I get the following warnings during Firefox LTO build. lto1-wpa-stream outputs the final object files in parallel

fs: hfsplus: use after free in

2015-02-20 Thread Sasha Levin
Hi all, While fuzzing with trinity inside a KVM tools guest running the latest -next kernel, I've stumbled on the following spew: [ 2014.561050] BUG: KASan: use after free in memcpy+0x21/0x50 at addr 880ff138afee [ 2014.561050] Read of size 4 by task trinity-main/10201 [ 2014.561050]

[PATCH 2/3] vhost: add support for legacy virtio

2015-02-20 Thread Greg Kurz
Signed-off-by: Greg Kurz --- drivers/vhost/vhost.h | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) Michael, The vhost_is_little_endian() helper adds unconditionnal overhead to fixed endian architectures: that is all architectures except arm and ppc64. This was

[PATCH 3/3] vhost_net: fix virtio_net header endianness

2015-02-20 Thread Greg Kurz
Without this patch, packets are being silently dropped by the tap backend. Signed-off-by: Greg Kurz --- drivers/vhost/net.c | 32 ++-- 1 file changed, 26 insertions(+), 6 deletions(-) diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index afa06d2..2923eee

Re: [PATCH v3] pci: spear: Drop __initdata from spear13xx_pcie_driver

2015-02-20 Thread Matwey V. Kornilov
2015-02-20 13:06 GMT+03:00 Stanimir Varbanov : > On 02/19/2015 07:41 PM, Matwey V. Kornilov wrote: >> spear13xx_pcie_driver.driver is allocated in text.init section >> and then the pointer to it is passed futher. This patch is to avoid >> crashes like the following, when freed memory is used. >>

Re: [PATCH] clockevents: Add (missing) default case for switch blocks

2015-02-20 Thread Viresh Kumar
On 20 February 2015 at 15:06, Ingo Molnar wrote: > > * Peter Zijlstra wrote: > >> > So this whole approach looks fragile for several reasons: >> > >> >- 'mode setting' callbacks are just bad by design >> > because they mix several functions into the same entry >> > point,

[PATCH 1/3] vhost: add VHOST_VRING_F_LEGACY_BIG_ENDIAN flag

2015-02-20 Thread Greg Kurz
The VHOST_VRING_F_LEGACY_BIG_ENDIAN flag informs the kernel that the associated device is big endian. Of course, this only makes sense for legacy virtio devices since modern virtio devices are always little endian. It will be used by the vhost memory accessors to byteswap vring data when we have

[PATCH 0/3] vhost_net: support for cross endian guests

2015-02-20 Thread Greg Kurz
Hi, This patchset allows vhost_net to be used with legacy virtio when guest and host have a different endianness. It is based on previous work by Cédric Le Goater: https://www.mail-archive.com/kvm-ppc@vger.kernel.org/msg09848.html As suggested by MST: - the API now asks for a specific format

Re: [PATCH v3] pci: spear: Drop __initdata from spear13xx_pcie_driver

2015-02-20 Thread Stanimir Varbanov
On 02/19/2015 07:41 PM, Matwey V. Kornilov wrote: > spear13xx_pcie_driver.driver is allocated in text.init section > and then the pointer to it is passed futher. This patch is to avoid > crashes like the following, when freed memory is used. > > Also, __init has been dropped from the probe()

Re: [PATCH] pinctrl: mediatek: Fix build error in Mediatek pinctrl driver.

2015-02-20 Thread Linus Walleij
On Thu, Feb 12, 2015 at 8:56 AM, Hongzhou Yang wrote: > Since pinconf relate API changes in pinctrl devel branch, a build error > happened. > Send this small patch to fix it. > > --- > Due to pinconf_generic_parse_dt_config() API changes in pinctrl devel branch, > add one parameter to fix build

Re: [PATCH 1/3] sched: add sched_task_call()

2015-02-20 Thread Jiri Kosina
On Fri, 20 Feb 2015, Ingo Molnar wrote: > So if your design is based on being able to discover 'live' functions in > the kernel stack dump of all tasks in the system, I think you need a > serious reboot of the whole approach and get rid of that fragility > before any of that functionality gets

Re: [PATCH v2] extcon: otg_gpio: add driver for USB OTG port controlled by GPIO(s)

2015-02-20 Thread Linus Walleij
On Fri, Feb 20, 2015 at 7:41 AM, Robert Baldyga wrote: > Hi David, > > On 02/19/2015 08:59 PM, David Cohen wrote: >> Some Intel platforms have an USB OTG port fully (or partially) >> controlled by GPIOs: >> >> (1) USB ID is connected directly to a pulled up GPIO. >> >> Optionally: >> (2) VBUS is

<    1   2   3   4   5   6   7   8   9   10   >