Re: [PATCH V2 4/5] cpufreq: create cpu/cpufreq/policyX directories

2015-10-28 Thread Rafael J. Wysocki
On Friday, October 16, 2015 11:21:18 AM Viresh Kumar wrote: > On 15-10-15, 12:25, Saravana Kannan wrote: > > Btw, does a Review-by have an implicit Acked-by? > > I have attended a session at Linaro Connect where this was discussed > and the answer was: > > Acked-by: is more of a general agreement

Re: [PATCH v3 0/3] virtio DMA API core stuff

2015-10-28 Thread Christian Borntraeger
Am 28.10.2015 um 16:17 schrieb Michael S. Tsirkin: > On Tue, Oct 27, 2015 at 11:38:57PM -0700, Andy Lutomirski wrote: >> This switches virtio to use the DMA API unconditionally. I'm sure >> it breaks things, but it seems to work on x86 using virtio-pci, with >> and without Xen, and using both the

Re: [PATCH 3.19.y-ckt 13/86] drm/vmwgfx: Fix up user_dmabuf refcounting

2015-10-28 Thread Thomas Hellstrom
Kamal, On 10/27/2015 10:29 PM, Kamal Mostafa wrote: > 3.19.8-ckt9 -stable review patch. If anyone has any objections, please let > me know. > > -- > > From: Thomas Hellstrom > > commit 54c12bc374408faddbff75dbf1a6167c19af39c4 upstream. > Unfortunately there was a regression int

Re: [PATCH 10/10] merge_config.sh: add tests for cmdline configs

2015-10-28 Thread Darren Hart
On Wed, Oct 28, 2015 at 09:42:11AM +0900, Olof Johansson wrote: > Again, these tests could be more complicated but at least it gets > the very basics covered. > > Signed-off-by: Olof Johansson Good thing you added =n as valid, this would be much harder otherwise :-) Signed-off-by: Darren Hart

Re: [PATCHv2] i2c: cadence: Enable power management

2015-10-28 Thread Shubhrajyoti Datta
On Wed, Oct 28, 2015 at 11:34 AM, Shubhrajyoti Datta wrote: > Currently the clocks are enabled at probe and disabled at remove. > This patch enables the clocks at the start of transfer and disables > after it. > > Also adapts to runtime pm. > Remove xi2c->suspended and use pm runtime status instea

Re: [PATCH 08/10] merge_config.sh: use trap for cleanup

2015-10-28 Thread Darren Hart
On Wed, Oct 28, 2015 at 09:42:09AM +0900, Olof Johansson wrote: > Use the trap to cleanup even on regular exit. > > Signed-off-by: Olof Johansson Looks like this should trade spots in the series with 9 where CLEAN_FILES is defined. Otherwise: Reviewed-by: Darren Hart > --- > scripts/kconfig/

[PATCH v2] i2c: cadence: Move to sensible power management

2015-10-28 Thread Shubhrajyoti Datta
Currently the clocks are enabled at probe and disabled at remove. Which keeps the clocks enabled even if no transaction is going on. This patch enables the clocks at the start of transfer and disables after it. Also adapts to runtime pm. Remove xi2c->suspended and use pm runtime status instead. c

Re: character driver - poll() timeout

2015-10-28 Thread Clemens Ladisch
Muni Sekhar wrote: > On Tue, Oct 27, 2015 at 8:48 PM, Clemens Ladisch wrote: >> Muni Sekhar wrote: >>> I need to find out when exactly driver's poll callback returned timeout. >> >> Your poll callback _cannot_ return a timeout. >> >> Why do you think you need this information for? > > During stres

Re: [PATCH 09/10] merge_config.sh: allow single configs to be passed in on cmdline

2015-10-28 Thread Darren Hart
On Wed, Oct 28, 2015 at 09:42:10AM +0900, Olof Johansson wrote: > Treat CONFIG_FOO=.. on the command line the same way as a single-entry file > would. > > Signed-off-by: Olof Johansson > --- > scripts/kconfig/merge_config.sh | 29 - > 1 file changed, 24 insertions(+

[PATCH 1/2] mcb: Do not return zero on error path in mcb_pci_probe()

2015-10-28 Thread Johannes Thumshirn
From: Alexey Khoroshilov There is an error path in mcb_pci_probe() where it returns zero instead of error code. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov Signed-off-by: Johannes Thumshirn --- drivers/mcb/mcb-pci.c | 1 + 1 file changed,

[PATCH 2/2] mcb: Destroy IDA on module unload

2015-10-28 Thread Johannes Thumshirn
Destroy mcb_ida on module_unload Signed-off-by: Johannes Thumshirn --- drivers/mcb/mcb-core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mcb/mcb-core.c b/drivers/mcb/mcb-core.c index 9018ab8..a4be451 100644 --- a/drivers/mcb/mcb-core.c +++ b/drivers/mcb/mcb-core.c @@ -409,6 +409

[PATCH 0/2] MCB Changes for 4.4

2015-10-28 Thread Johannes Thumshirn
Hi Greg, These are the MCB Changes for 4.4. Alexey Khoroshilov (1): mcb: Do not return zero on error path in mcb_pci_probe() Johannes Thumshirn (1): mcb: Destroy IDA on module unload drivers/mcb/mcb-core.c | 1 + drivers/mcb/mcb-pci.c | 1 + 2 files changed, 2 insertions(+) -- 2.6.2 --

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2015-10-28 Thread Minchan Kim
Hello Chen, n Fri, Dec 05, 2014 at 06:58:29AM +0800, Chen Gang wrote: > For uapi, need try to let all macros have same value, and MADV_FREE is > added into main branch recently, so need redefine MADV_FREE for it. > > At present, '8' can be shared with all architectures, so redefine it to > '8'.

Re: [PATCH v3 0/3] virtio DMA API core stuff

2015-10-28 Thread Michael S. Tsirkin
On Tue, Oct 27, 2015 at 11:38:57PM -0700, Andy Lutomirski wrote: > This switches virtio to use the DMA API unconditionally. I'm sure > it breaks things, but it seems to work on x86 using virtio-pci, with > and without Xen, and using both the modern 1.0 variant and the > legacy variant. I'm very g

Re: [PATCH V3 1/5] cpufreq: ondemand: Drop unnecessary locks from update_sampling_rate()

2015-10-28 Thread Rafael J. Wysocki
On Wednesday, October 28, 2015 12:13:17 PM Viresh Kumar wrote: > On 28-10-15, 06:54, Rafael J. Wysocki wrote: > > On Wednesday, October 28, 2015 10:14:51 AM Viresh Kumar wrote: > > > In cases where a single policy controls multiple CPUs, a timer is > > > queued for every cpu present in policy->cpus

[PATCH] usb : replace dma_pool_alloc and memset with dma_pool_zalloc

2015-10-28 Thread Saurabh Sengar
replace dma_pool_alloc and memset with a single call to dma_pool_zalloc Signed-off-by: Saurabh Sengar --- drivers/usb/chipidea/udc.c | 3 +-- drivers/usb/gadget/udc/gr_udc.c | 3 +-- drivers/usb/host/uhci-q.c | 3 +-- drivers/usb/host/whci/qset.c| 3 +-- drivers/usb/host/xhci-mem.

[PATCH] Document that IRQ_NONE should be returned when IRQ not actually *handled*

2015-10-28 Thread David Woodhouse
Our IRQ storm detection works when an interrupt handler returns IRQ_NONE for thousands of consecutive interrupts in a second. It doesn't hurt to occasionally return IRQ_NONE when the interrupt is actually genuine. Drivers should only be returning IRQ_HANDLED if they have actually *done* something

Re: [PATCH 08/10] merge_config.sh: use trap for cleanup

2015-10-28 Thread Darren Hart
On Wed, Oct 28, 2015 at 09:42:09AM +0900, Olof Johansson wrote: > Use the trap to cleanup even on regular exit. > > Signed-off-by: Olof Johansson > --- > scripts/kconfig/merge_config.sh | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/scripts/kconfig/merge_config

Re: [PATCH v3 0/3] virtio DMA API core stuff

2015-10-28 Thread Andy Lutomirski
On Tue, Oct 27, 2015 at 11:53 PM, David Woodhouse wrote: > On Tue, 2015-10-27 at 23:38 -0700, Andy Lutomirski wrote: >> >> Changes from v2: >> - Fix really embarrassing bug. This version actually works. > > So embarrassing you didn't want to tell us what it was? ... Shhh, it's a secret! I some

Re: [PATCH v3 1/3] virtio_net: Stop doing DMA from the stack

2015-10-28 Thread Michael S. Tsirkin
On Tue, Oct 27, 2015 at 11:38:58PM -0700, Andy Lutomirski wrote: > From: Andy Lutomirski > > Once virtio starts using the DMA API, we won't be able to safely DMA > from the stack. virtio-net does a couple of config DMA requests > from small stack buffers -- switch to using dynamically-allocated

Re: [PATCH v2 1/3] virtio_net: Stop doing DMA from the stack

2015-10-28 Thread Michael S. Tsirkin
On Tue, Oct 27, 2015 at 10:30:19PM -0700, Andy Lutomirski wrote: > From: Andy Lutomirski > > Once virtio starts using the DMA API, we won't be able to safely DMA > from the stack. virtio-net does a couple of config DMA requests > from small stack buffers -- switch to using dynamically-allocated

Re: [PATCH 07/10] merge_config.sh: add tests

2015-10-28 Thread Olof Johansson
On Wed, Oct 28, 2015 at 4:00 PM, Darren Hart wrote: > On Wed, Oct 28, 2015 at 09:42:08AM +0900, Olof Johansson wrote: >> For being a small script, merge_config.sh is fairly scary to change since >> there's no >> real way to know if you did something wrong. So it seems appropriate to add >> a sim

Re: [PATCH 00/10] merge_config misc reworks and testcases

2015-10-28 Thread Darren Hart
On Wed, Oct 28, 2015 at 01:30:59AM -0400, Bruce Ashfield wrote: > On 10/28/2015 01:02 AM, Darren Hart wrote: > >On Wed, Oct 28, 2015 at 09:42:01AM +0900, Olof Johansson wrote: > >>- The script now prints the warnings on stderr, and returns non-0 when > >> something is encountered > > > >This one

Re: [PATCH] drm/vmwgfx: switch from ioremap_cache to memremap

2015-10-28 Thread Thomas Hellstrom
Dan, On 10/19/2015 11:34 PM, Williams, Dan J wrote: > On Tue, 2015-10-13 at 20:52 +0200, Thomas Hellstrom wrote: >>> Ok, I'll make local read_fifo() and write_fifo() macros to make this >>> explicit. Are these names ok with you? >> Sure. >> > So I ended up just leaving the __iomem annotation on m

Re: [PATCH V3 3/5] cpufreq: ondemand: queue work for policy->cpus together

2015-10-28 Thread Rafael J. Wysocki
On Wednesday, October 28, 2015 12:16:35 PM Viresh Kumar wrote: > On 28-10-15, 07:38, Rafael J. Wysocki wrote: > > On Tuesday, October 13, 2015 01:39:03 PM Viresh Kumar wrote: > > > Currently update_sampling_rate() runs over each online CPU and > > > cancels/queues work on it. Its very inefficient f

Re: perf: fuzzer triggered trouble on AMD, maybe ibs related

2015-10-28 Thread Stephane Eranian
On Thu, Oct 22, 2015 at 6:46 PM, Vince Weaver wrote: > Hello > > I've been busy but finally had a chance to run perf_fuzzer on current git. > I am running on an AMD A10 system (my traditional Haswell system is > otherwise occupied). > > I got the following WARNING which was followed by an NMI stor

Re: [PATCH 1/6] dmaengine: tegra-apb: Correct runtime-pm usage

2015-10-28 Thread Vinod Koul
On Fri, Oct 16, 2015 at 09:25:52AM +0100, Jon Hunter wrote: > @@ -1182,14 +1182,11 @@ static int tegra_dma_alloc_chan_resources(struct > dma_chan *dc) > { > struct tegra_dma_channel *tdc = to_tegra_dma_chan(dc); > struct tegra_dma *tdma = tdc->tdma; > - int ret; > > dma_co

[PATCH v2] cputime: fix invalid gtime

2015-10-28 Thread Hiroshi Shimamoto
From: Hiroshi Shimamoto /proc/stats shows invalid gtime when the thread is running in guest. When vtime accounting is not enabled, we cannot get a valid delta. The delta is calculated now - tsk->vtime_snap, but tsk->vtime_snap is only updated when vtime accounting is enabled. This patch makes ta

Re: [PATCH 07/10] merge_config.sh: add tests

2015-10-28 Thread Darren Hart
On Wed, Oct 28, 2015 at 09:42:08AM +0900, Olof Johansson wrote: > For being a small script, merge_config.sh is fairly scary to change since > there's no > real way to know if you did something wrong. So it seems appropriate to add a > simple > test suite. > > I've started with testcases in the a

<    5   6   7   8   9   10