Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-23 Thread Roger Quadros
On 20/05/16 12:53, Peter Chen wrote: > On Fri, May 20, 2016 at 12:19:07PM +0300, Roger Quadros wrote: >> On 20/05/16 11:31, Roger Quadros wrote: >>> On 18/05/16 15:59, Roger Quadros wrote: Hi Peter, On 18/05/16 10:45, Peter Chen wrote: > > > On Mon, May 16, 2016 at 5:00 P

Re: [PATCH 1/1 RFC] net/phy: Add Lantiq PHY driver

2016-05-23 Thread Mathias Kresin
2016-05-23 11:49 GMT+02:00 Alexander Stein : >> The registers are set to some reset values after the chip is coming out of >> reset, but we should set them all to the same value, Mathias said that all >> except for one board he knows are using only one LED per port, but they are >> often using di

Re: [PATCH v8 13/14] usb: gadget: udc: adapt to OTG core

2016-05-23 Thread Roger Quadros
On 23/05/16 06:21, Peter Chen wrote: > On Sat, May 21, 2016 at 10:29:40AM +0800, Peter Chen wrote: >> On Thu, May 19, 2016 at 10:32:44AM +0300, Roger Quadros wrote: >>> On 18/05/16 17:46, Jun Li wrote: >>> >>> I didn't want to have complex Kconfig so decided to have otg as >>>

Re: [patch] sched/fair: Move se->vruntime normalization state into struct sched_entity

2016-05-23 Thread Wanpeng Li
2016-05-23 17:40 GMT+08:00 Mike Galbraith : > On Mon, 2016-05-23 at 11:19 +0200, Peter Zijlstra wrote: >> On Sun, May 22, 2016 at 09:00:01AM +0200, Mike Galbraith wrote: >> > On Sat, 2016-05-21 at 21:00 +0200, Mike Galbraith wrote: >> > > On Sat, 2016-05-21 at 16:04 +0200, Mike Galbraith wrote: >>

Re: [PATCH] spi: spidev: fix possible arithmetic overflow for multi-transfer message

2016-05-23 Thread Ian Abbott
On 21/05/16 17:50, Dmitry Torokhov wrote: On Mon, Mar 23, 2015 at 10:50 AM, Ian Abbott wrote: `spidev_message()` sums the lengths of the individual SPI transfers to determine the overall SPI message length. It restricts the total length, returning an error if too long, but it does not check fo

Re: [patch] sched/fair: Move se->vruntime normalization state into struct sched_entity

2016-05-23 Thread Mike Galbraith
On Mon, 2016-05-23 at 18:13 +0800, Wanpeng Li wrote: > 2016-05-23 17:40 GMT+08:00 Mike Galbraith : > > On Mon, 2016-05-23 at 11:19 +0200, Peter Zijlstra wrote: > > > On Sun, May 22, 2016 at 09:00:01AM +0200, Mike Galbraith wrote: > > > > On Sat, 2016-05-21 at 21:00 +0200, Mike Galbraith wrote: > >

Re: [PATCH] drm/mm: avoid possible null pointer dereference

2016-05-23 Thread Eric Engestrom
On Wed, May 18, 2016 at 10:17:19PM +0200, Heinrich Schuchardt wrote: > Do not dereference node before the check if node is NULL. > > Signed-off-by: Heinrich Schuchardt > --- > drivers/gpu/drm/drm_mm.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/dr

Re: [tip:sched/core] sched/fair: Clean up scale confusion

2016-05-23 Thread Vincent Guittot
On 20 May 2016 at 12:12, Morten Rasmussen wrote: > On Fri, May 13, 2016 at 09:23:50AM +0200, Vincent Guittot wrote: >> On 12 May 2016 at 21:42, Yuyang Du wrote: >> > On Thu, May 12, 2016 at 03:31:27AM -0700, tip-bot for Peter Zijlstra wrote: >> >> Commit-ID: 1be0eb2a97d756fb7dd8c9baf372d81fa9699

Re: [GIT PULL] drm/arcpgu: use dedicated memory area for frame buffer

2016-05-23 Thread Alexey Brodkin
Hi Dave, On Mon, 2016-05-16 at 08:22 +, Alexey Brodkin wrote: > Hi Dave, > > On Tue, 2016-05-10 at 09:51 +, Alexey Brodkin wrote: > > > > Hi Dave, > > > > On Fri, 2016-04-29 at 11:36 +, Alexey Brodkin wrote: > > > > > > > > > Hi Dave, > > > > > > Please pull this mini-series that

Re: [PATCH v4 01/18] nbd: Fix might_sleep warning on xmit timeout

2016-05-23 Thread Pranay Srivastava
Hi Markus On Fri, May 20, 2016 at 1:52 PM, Markus Pargmann wrote: > On Friday 20 May 2016 02:05:36 Pranay Srivastava wrote: >> On Thu, May 19, 2016 at 11:52 AM, Markus Pargmann >> wrote: >> > Hi, >> > >> > On Wed, May 11, 2016 at 11:18:29AM +0300, Pranay Kr. Srivastava wrote: >> >> This patch f

RE: [PATCH v8 13/14] usb: gadget: udc: adapt to OTG core

2016-05-23 Thread Jun Li
Hi > -Original Message- > From: Roger Quadros [mailto:rog...@ti.com] > Sent: Monday, May 23, 2016 6:12 PM > To: Peter Chen > Cc: Jun Li ; peter.c...@freescale.com; ba...@kernel.org; > t...@atomide.com; gre...@linuxfoundation.org; dan.j.willi...@intel.com; > mathias.ny...@linux.intel.com;

Re: [PATCH 1/1] drm/mediatek: do not dereference NULL

2016-05-23 Thread Eric Engestrom
On Wed, May 18, 2016 at 10:59:50PM +0200, Heinrich Schuchardt wrote: > After having verified that dpi is NULL we should not > dereference it. > > Signed-off-by: Heinrich Schuchardt > --- > drivers/gpu/drm/mediatek/mtk_dpi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

[PATCH 2/8] mm: clean up non-standard page->_mapcount users

2016-05-23 Thread Vladimir Davydov
- Add a proper comment to page->_mapcount. - Introduce a macro for generating helper functions. - Place all special page->_mapcount values next to each other so that readers can see all possible values and so we don't get duplicates. Signed-off-by: Vladimir Davydov --- include/linux/mm_typ

[PATCH 6/8] arch: x86: charge page tables to kmemcg

2016-05-23 Thread Vladimir Davydov
Page tables can bite a relatively big chunk off system memory and their allocations are easy to trigger from userspace, so they should be accounted to kmemcg. This patch marks page table allocations as __GFP_ACCOUNT for x86. Note we must not charge allocations of kernel page tables, because they c

[PATCH 3/8] mm: memcontrol: cleanup kmem charge functions

2016-05-23 Thread Vladimir Davydov
- Handle memcg_kmem_enabled check out to the caller. This reduces the number of function definitions making the code easier to follow. At the same time it doesn't result in code bloat, because all of these functions are used only in one or two places. - Move __GFP_ACCOUNT check to the ca

Re: [PATCH v8 13/14] usb: gadget: udc: adapt to OTG core

2016-05-23 Thread Roger Quadros
On 23/05/16 13:34, Jun Li wrote: > Hi > >> -Original Message- >> From: Roger Quadros [mailto:rog...@ti.com] >> Sent: Monday, May 23, 2016 6:12 PM >> To: Peter Chen >> Cc: Jun Li ; peter.c...@freescale.com; ba...@kernel.org; >> t...@atomide.com; gre...@linuxfoundation.org; dan.j.willi...@i

Re: [PATCH] pinctrl: mediatek: fix dual-edge code defect

2016-05-23 Thread Matthias Brugger
On 21/05/16 09:23, hongkun@mediatek.com wrote: From: "hongkun.cao" When a dual-edge irq is triggered, an incorrect irq will be reported on condition that the external signal is not stable and this incorrect irq has been registered. Correct the register offset. I'm not sure what you wan

Re: [PATCH 7/8] pipe: account to kmemcg

2016-05-23 Thread kbuild test robot
-remove-pointless-struct-in-struct-page-definition/20160523-182939 config: i386-tinyconfig (attached as .config) compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430 reproduce: # save the attached .config to linux build tree make ARCH=i386 Note: the linux-review/Vladimir-Davydov/mm-remove

[PATCHv3] rcu: tree: correctly handle sparse possible cpus

2016-05-23 Thread Mark Rutland
In many cases in the RCU tree code, we iterate over the set of cpus for a leaf node described by rcu_node::grplo and rcu_node::grphi, checking per-cpu data for each cpu in this range. However, if the set of possible cpus is sparse, some cpus described in this range are not possible, and thus no per

[PATCH v5 1/2] skb_array: array based FIFO for skbs

2016-05-23 Thread Michael S. Tsirkin
A simple array based FIFO of pointers. Intended for net stack so uses skbs for type safety, but we can replace with with void * if others find it useful outside of net stack. Signed-off-by: Michael S. Tsirkin --- include/linux/skb_array.h | 127 ++ 1

[PATCH v5 0/2] skb_array: array based FIFO for skbs

2016-05-23 Thread Michael S. Tsirkin
This is in response to the proposal by Jason to make tun rx packet queue lockless using a circular buffer. My testing seems to show that at least for the common usecase in networking, which isn't lockless, circular buffer with indices does not perform that well, because each index access causes a c

[PATCH v5 2/2] skb_array: ring test

2016-05-23 Thread Michael S. Tsirkin
Add ringtest based unit test for skb array. Signed-off-by: Michael S. Tsirkin --- tools/virtio/ringtest/skb_array.c | 167 ++ tools/virtio/ringtest/Makefile| 4 +- 2 files changed, 170 insertions(+), 1 deletion(-) create mode 100644 tools/virtio/ringtes

Re: [PATCH 4/5] iommu/rockchip: add ARM64 cache flush operation for iommu

2016-05-23 Thread Robin Murphy
On 23/05/16 02:37, Shunqian Zheng wrote: From: Simon Signed-off-by: Simon --- drivers/iommu/rockchip-iommu.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/iommu/rockchip-iommu.c b/drivers/iommu/rockchip-iommu.c index 043d18c..1741b65 100644 --- a/drivers/iommu/rockchip-iom

Re: [PATCH v2 1/3] regulator: DT: Add DT property for operation mode configuration

2016-05-23 Thread Mark Rutland
On Mon, May 23, 2016 at 03:13:29PM +0800, Henry Chen wrote: > Some regulators support their operating mode to be changed by consumers for > module specific purpose. Add a DT property to support this. > > Signed-off-by: Henry Chen > --- > Documentation/devicetree/bindings/regulator/regulator.txt

Re: linux-next: Tree for May 23

2016-05-23 Thread Stephen Rothwell
so I used the version from > next-20160520. > > Non-merge commits (relative to Linus' tree): 2283 > 1939 files changed, 140199 insertions(+), 39283 deletions(-) I screwed the release up, but have now reuploaded next-20160523 to kernel.org. Sorry about that. -- Cheers, Stephen Rothwell

[PATCH 1/3] Add missing "goto out" after fbops->fb_open()

2016-05-23 Thread 'Max Staudt
From: Max Staudt It doesn't make sense to execute any further actions after a failed call to fbops->fb_open(). Signed-off-by: Max Staudt --- drivers/video/fbdev/core/fbmem.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/

[PATCH 0/3] Fix kernel WARNING in TTM/Bochs KMS driver

2016-05-23 Thread 'Max Staudt
This patch set addresses an instance of dangling/use-after-free pointers that can occur in the bochsdrm driver. The original report can be found at: https://lkml.kernel.org/g/s5hy49ue4y0.wl-ti...@suse.de It can be tested using 'qemu-system-x86_64 -vga std' with any kernel recent enough to ship th

[PATCH 2/3] Define fb_open_adj_file()

2016-05-23 Thread 'Max Staudt
From: Max Staudt This callback from fb_open() allows a fbdev driver to adjust things such as file->f_mapping to better represent the internal structures. This is needed to allow TTM drivers using ttm_fbdev_mmap() to properly set file->f_mapping to TTM's address_space from bo->bdev->dev_mapping,

[PATCH 3/3] Add fb_open_adj_file() to bochsdrmfb to avoid use-after-free

2016-05-23 Thread 'Max Staudt
From: Max Staudt Currently, when using bochsdrm(fb), opening /dev/drm/card0 will adjust file->f_mapping, but opening /dev/fb0 will not. This may result in dangling pointers from user space when memory is evicted, and therefore in use-after-free crashes when using the emulated framebuffer device.

Re: [PATCH V7 00/11] Support for generic ACPI based PCI host controller

2016-05-23 Thread Lorenzo Pieralisi
On Fri, May 20, 2016 at 11:14:03AM +0200, Ard Biesheuvel wrote: > On 20 May 2016 at 10:40, Gabriele Paoloni wrote: > > Hi Ard > > > >> -Original Message- > >> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > [...] > >> > >> Is the PCIe root complex so special that you cannot simpl

Re: [PATCH 7/8] pipe: account to kmemcg

2016-05-23 Thread Vladimir Davydov
On Mon, May 23, 2016 at 06:39:40PM +0800, kbuild test robot wrote: ... >fs/built-in.o: In function `anon_pipe_buf_steal': > >> pipe.c:(.text+0x5f8d): undefined reference to `memcg_kmem_uncharge' From: Vladimir Davydov Subject: [PATCH] pipe-account-to-kmemcg-fix diff --git a/fs/pipe.c b/fs/p

[PATCH 09/16] sched/fair: Let asymmetric cpu configurations balance at wake-up

2016-05-23 Thread Morten Rasmussen
Currently, SD_WAKE_AFFINE always takes priority over wakeup balancing if SD_BALANCE_WAKE is set on the sched_domains. For asymmetric configurations SD_WAKE_AFFINE is only desirable if the waking task's compute demand (utilization) is suitable for the cpu capacities available within the SD_WAKE_AFFI

[PATCH 06/16] sched: Disable WAKE_AFFINE for asymmetric configurations

2016-05-23 Thread Morten Rasmussen
If the system has cpu of different compute capacities (e.g. big.LITTLE) let affine wakeups be constrained to cpus of the same type. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by: Morten Rasmussen --- kernel/sched/core.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/kernel/sched/co

[PATCH 14/16] arm: Set SD_ASYM_CPUCAPACITY for big.LITTLE platforms

2016-05-23 Thread Morten Rasmussen
Set the SD_ASYM_CPUCAPACITY flag for DIE level sched_domain on big.LITTLE systems. cc: Russell King Signed-off-by: Morten Rasmussen --- arch/arm/kernel/topology.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology

[PATCH 16/16] arm: Update arch_scale_cpu_capacity() to reflect change to define

2016-05-23 Thread Morten Rasmussen
arch_scale_cpu_capacity() is no longer a weak function but a #define instead. Include the #define in topology.h. cc: Russell King Signed-off-by: Morten Rasmussen --- arch/arm/include/asm/topology.h | 5 + arch/arm/kernel/topology.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(

[PATCH 15/16] arm: Set SD_BALANCE_WAKE flag for asymmetric capacity systems

2016-05-23 Thread Morten Rasmussen
Asymmetric capacity systems (e.g. ARM big.LITTLE) can not rely exclusively on fast idle-based task placement at wake-up in all scenarios. Enable SD_BALANCE_WAKE to have the option to do load/utilization based wake-up task placement (existing tasks) if affine wake-up fails. cc: Russell King Signe

[PATCH 11/16] sched/fair: Consider spare capacity in find_idlest_group()

2016-05-23 Thread Morten Rasmussen
In low-utilization scenarios comparing relative loads in find_idlest_group() doesn't always lead to the most optimum choice. Systems with groups containing different numbers of cpus and/or cpus of different compute capacity are significantly better off when considering spare capacity rather than re

[PATCH 12/16] sched: Add per-cpu max capacity to sched_group_capacity

2016-05-23 Thread Morten Rasmussen
struct sched_group_capacity currently represents the compute capacity sum of all cpus in the sched_group. Unless it is divided by the group_weight to get the average capacity per cpu it hides differences in cpu capacity for mixed capacity systems (e.g. high RT/IRQ utilization or ARM big.LITTLE). Bu

[PATCH 10/16] sched/fair: Compute task/cpu utilization at wake-up more correctly

2016-05-23 Thread Morten Rasmussen
At task wake-up load-tracking isn't updated until the task is enqueued. The task's own view of its utilization contribution may therefore not be aligned with its contribution to the cfs_rq load-tracking which may have been updated in the meantime. Basically, the task's own utilization hasn't yet ac

[PATCH 13/16] sched/fair: Avoid pulling tasks from non-overloaded higher capacity groups

2016-05-23 Thread Morten Rasmussen
For asymmetric cpu capacity systems it is counter-productive for throughput if low capacity cpus are pulling tasks from non-overloaded cpus with higher capacity. The assumption is that higher cpu capacity is preferred over running alone in a group with lower cpu capacity. This patch rejects higher

[PATCH 05/16] sched: Introduce SD_ASYM_CPUCAPACITY sched_domain topology flag

2016-05-23 Thread Morten Rasmussen
Add a topology flag to the sched_domain hierarchy indicating which domains span cpus of different capacity (e.g. big.LITTLE). This information is currently only available through iterating through the capacities of all the cpus. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by: Morten Rasmussen

[PATCH 08/16] sched: Store maximum per-cpu capacity in root domain

2016-05-23 Thread Morten Rasmussen
From: Dietmar Eggemann To be able to compare the capacity of the target cpu with the highest available cpu capacity, store the maximum per-cpu capacity in the root domain. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by: Dietmar Eggemann Signed-off-by: Morten Rasmussen --- kernel/sched/co

[PATCH 07/16] sched: Make SD_BALANCE_WAKE a topology flag

2016-05-23 Thread Morten Rasmussen
For systems with the SD_ASYM_CPUCAPACITY flag set on higher level in the sched_domain hierarchy we need a way to enable wake-up balancing for the lower levels as well as we may want to balance tasks that don't fit the capacity of the previous cpu. We have the option of introducing a new topology f

[PATCH 03/16] sched/fair: Disregard idle task wakee_flips in wake_wide

2016-05-23 Thread Morten Rasmussen
wake_wide() is based on task wakee_flips of the waker and the wakee to decide whether an affine wakeup is desirable. On lightly loaded systems the waker is frequently the idle task (pid=0) which can accumulate a lot of wakee_flips in that scenario. It makes little sense to prevent affine wakeups on

Re: [PATCH] dma-buf: use vma_pages().

2016-05-23 Thread Eric Engestrom
On Sat, May 21, 2016 at 06:53:32PM +0530, Muhammad Falak R Wani wrote: > Replace explicit computation of vma page count by a call to > vma_pages() > > Signed-off-by: Muhammad Falak R Wani > --- Good change, but this function is defined in mm.h Please add: #include With that fixed: Reviewe

[PATCH 02/16] sched/fair: Consistent use of prev_cpu in wakeup path

2016-05-23 Thread Morten Rasmussen
In commit ac66f5477239 ("sched/numa: Introduce migrate_swap()") select_task_rq() got a 'cpu' argument to enable overriding of prev_cpu in special cases (NUMA task swapping). However, the select_task_rq_fair() helper functions: wake_affine() and select_idle_sibling(), still use task_cpu(p) directly

[PATCH 04/16] sched/fair: Optimize find_idlest_cpu() when there is no choice

2016-05-23 Thread Morten Rasmussen
In the current find_idlest_group()/find_idlest_cpu() search we end up calling find_idlest_cpu() in a sched_group containing only one cpu in the end. Checking idle-states becomes pointless when there is no alternative, so bail out instead. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by: Morten

[PATCH 01/16] sched: Fix power to capacity renaming in comment

2016-05-23 Thread Morten Rasmussen
It is seems that this one escaped Nico's renaming of cpu_power to cpu_capacity a while back. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by: Morten Rasmussen --- include/linux/sched.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/sched.h b/include/linux/s

[PATCH 00/16] sched: Clean-ups and asymmetric cpu capacity support

2016-05-23 Thread Morten Rasmussen
Hi, The scheduler is currently not doing much to help performance on systems with asymmetric compute capacities (read ARM big.LITTLE). This series improves the situation with a few tweaks mainly to the task wake-up path that considers compute capacity at wake-up and not just whether a cpu is idle

Re: [RFC PATCH 2/2] lightnvm: Append device name to target name

2016-05-23 Thread Wenwei Tao
2016-05-23 17:16 GMT+08:00 Matias Bjørling : > On 05/23/2016 11:13 AM, Wenwei Tao wrote: >> >> From: Wenwei Tao >> >> We may create targets with same name on different >> backend devices, this is not what we want, so append >> the device name to target name to make the new target >> name unique in

[PATCH] irqchip/mips-gic: Fix IRQs in gic_dev_domain

2016-05-23 Thread Harvey Hunt
When allocating a new device IRQ, gic_dev_domain_alloc() correctly calls irq_domain_set_hwirq_and_chip(), but gic_irq_domain_alloc() does not. This means that gic_irq_domain believes all IRQs from the dev domain have an hwirq of 0 and creates incorrect mappings in the linear_revmap. As gic_irq_doma

[PATCH] irqchip/mips-gic: Populate irq_domain names

2016-05-23 Thread Harvey Hunt
Set the irq_domain names so that they don't default to an unhelpful value. Signed-off-by: Harvey Hunt Reviewed-by: Matt Redfearn Cc: Cc: Cc: Qais Yousef --- drivers/irqchip/irq-mips-gic.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/irqchip/irq-mips-gic.c b/drivers/irqchip/

Re: [PATCH 40/54] MAINTAINERS: Add file patterns for regmap device tree bindings

2016-05-23 Thread Mark Brown
On Sun, May 22, 2016 at 11:06:17AM +0200, Geert Uytterhoeven wrote: > Please apply this patch directly if you want to be involved in device > tree binding documentation for your subsystem. Please use subject lines matching the style of the subsystem and/or include people in the cover letter so it

Re: [PATCH 43/54] MAINTAINERS: Add file patterns for sound device tree bindings

2016-05-23 Thread Mark Brown
On Sun, May 22, 2016 at 11:06:20AM +0200, Geert Uytterhoeven wrote: > +F: Documentation/devicetree/bindings/sound/ Note that this isn't specifically ASoC, though most things there are. signature.asc Description: PGP signature

drivers/net/ethernet/qlogic/qed/qed_dcbx.c:210: pointless test ?

2016-05-23 Thread David Binderman
Hello there, drivers/net/ethernet/qlogic/qed/qed_dcbx.c:210:16: warning: comparison is always false due to limited range of data type [-Wtype-limits] Source code is if (priority < 0) { but u8 tc, priority, priority_map; Regards David Binderman

Re: [PATCH 03/16] sched/fair: Disregard idle task wakee_flips in wake_wide

2016-05-23 Thread Mike Galbraith
On Mon, 2016-05-23 at 11:58 +0100, Morten Rasmussen wrote: > wake_wide() is based on task wakee_flips of the waker and the wakee to > decide whether an affine wakeup is desirable. On lightly loaded systems > the waker is frequently the idle task (pid=0) which can accumulate a lot > of wakee_flips i

Re: [PATCH 3/4] x86: Rewrite switch_to() code

2016-05-23 Thread Brian Gerst
On Sun, May 22, 2016 at 10:34 PM, Josh Poimboeuf wrote: > On Sun, May 22, 2016 at 10:59:38AM -0700, Andy Lutomirski wrote: >> cc: Josh Poimboeuf: do you care about the exact stack layout of the >> bottom of the stack of an inactive task? > > So there's one minor issue with this patch, relating to

Re: [PATCH] spi: spidev: fix possible arithmetic overflow for multi-transfer message

2016-05-23 Thread Mark Brown
On Sat, May 21, 2016 at 09:50:35AM -0700, Dmitry Torokhov wrote: > On Mon, Mar 23, 2015 at 10:50 AM, Ian Abbott wrote: > > `spidev_message()` sums the lengths of the individual SPI transfers to > > determine the overall SPI message length. It restricts the total > > length, returning an error if

siginfo memory leak?

2016-05-23 Thread Michal Hocko
Hi, Aleksa has reported that strace tells a bogus si_errno while debugging something on s390: [pid 20799] --- SIGSEGV {si_signo=SIGSEGV, si_code=SEGV_MAPERR, si_errno=2510266, si_addr=0x100} A quick look into do_sigsegv shows that siginfo is not completely initialized and it indeed mi

Re: [RFC PATCH 1/2] Input: rotary-encoder- Add support for absolute encoder

2016-05-23 Thread R, Vignesh
On 5/22/2016 3:56 PM, Uwe Kleine-König wrote: > Hello, > > On Thu, May 19, 2016 at 02:34:00PM +0530, Vignesh R wrote: >> There are rotary-encoders where GPIO lines reflect the actual position >> of the rotary encoder dial. For example, if dial points to 9, then four >> GPIO lines connected to th

Re: [PATCH 04/20] staging: comedi: drivers: re-do macros for PLX PCI 9080 LASxRR values

2016-05-23 Thread Ian Abbott
On 20/05/16 18:52, Hartley Sweeten wrote: On Friday, May 20, 2016 10:18 AM, Ian Abbott wrote: On 20/05/16 17:37, Hartley Sweeten wrote: On Friday, May 20, 2016 6:49 AM, Ian Abbott wrote: Rename the macros for the PLX PCI 9080 LAS0RR and LAS1RR registers in "plx9080.h", using the prefix `PLX_LA

Re: [PATCH] spi: spidev: fix possible arithmetic overflow for multi-transfer message

2016-05-23 Thread Ian Abbott
On 23/05/16 12:15, Mark Brown wrote: On Sat, May 21, 2016 at 09:50:35AM -0700, Dmitry Torokhov wrote: On Mon, Mar 23, 2015 at 10:50 AM, Ian Abbott wrote: `spidev_message()` sums the lengths of the individual SPI transfers to determine the overall SPI message length. It restricts the total len

[PATCH 1/3] dma-buf/fence: make fence context 64 bit v2

2016-05-23 Thread Christian König
From: Christian König Fence contexts are created on the fly (for example) by the GPU scheduler used in the amdgpu driver as a result of an userspace request. Because of this userspace could in theory force a wrap around of the 32bit context number if it doesn't behave well. Avoid this by increas

[PATCH 3/3] dma-buf/fence: add signal_on_any to the fence array

2016-05-23 Thread Christian König
From: Christian König If @signal_on_any is true the fence array signals if any fence in the array signals, otherwise it signals when all fences in the array signal. Signed-off-by: Christian König --- drivers/dma-buf/fence-array.c | 23 --- include/linux/fence-array.h | 3

Re: [PATCH 1/3] media: Media Device Allocator API

2016-05-23 Thread Hans Verkuil
Hi Shuah, Some comments below: On 05/13/2016 07:09 PM, Shuah Khan wrote: > Media Device Allocator API to allows multiple drivers share a media device. > Using this API, drivers can allocate a media device with the shared struct > device as the key. Once the media device is allocated by a driver,

Re: [PATCH 33/54] MAINTAINERS: Add file patterns for wireless device tree bindings

2016-05-23 Thread Kalle Valo
Geert Uytterhoeven writes: > Submitters of device tree binding documentation may forget to CC > the subsystem maintainer if this is missing. > > Signed-off-by: Geert Uytterhoeven > Cc: Kalle Valo > Cc: linux-wirel...@vger.kernel.org > --- > Please apply this patch directly if you want to be inv

[PATCH 2/3] dma-buf/fence: add fence_array fences v5

2016-05-23 Thread Christian König
From: Gustavo Padovan struct fence_collection inherits from struct fence and carries a collection of fences that needs to be waited together. It is useful to translate a sync_file to a fence to remove the complexity of dealing with sync_files on DRM drivers. So even if there are many fences in t

RE: drivers/net/ethernet/qlogic/qed/qed_dcbx.c:210: pointless test ?

2016-05-23 Thread Yuval Mintz
> Hello there, > > drivers/net/ethernet/qlogic/qed/qed_dcbx.c:210:16: warning: comparison is > always false due to limited range of data type [-Wtype-limits] > > Source code is > >if (priority < 0) { > > but > > u8 tc, priority, priority_map; > > > Regards > > David Binderman H

Re: [RFC PATCHv2] usb: USB Type-C Connector Class

2016-05-23 Thread Oliver Neukum
On Mon, 2016-05-23 at 12:57 +0300, Heikki Krogerus wrote: > Hi Oliver, > > On Fri, May 20, 2016 at 04:19:59PM +0200, Oliver Neukum wrote: > > On Thu, 2016-05-19 at 15:44 +0300, Heikki Krogerus wrote: > > > Like I've told some of you guys, I'm trying to implement a bus for > > > the Alternate Modes

Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4

2016-05-23 Thread Christian König
Am 23.05.2016 um 09:41 schrieb Daniel Vetter: On Fri, May 20, 2016 at 11:47:28AM -0300, Gustavo Padovan wrote: 2016-05-20 Christian König : From: Gustavo Padovan struct fence_collection inherits from struct fence and carries a collection of fences that needs to be waited together. It is use

Re: [PATCH v2 1/3] regulator: DT: Add DT property for operation mode configuration

2016-05-23 Thread Mark Brown
On Mon, May 23, 2016 at 11:47:04AM +0100, Mark Rutland wrote: > > +/* Regulator operating modes */ > > +#define REGULATOR_OPERATION_MODE_FAST 0x0 > > +#define REGULATOR_OPERATION_MODE_NORMAL0x1 > > +#define REGULATOR_OPERATION_MODE_IDLE

Re: [PATCH] regulator: mt6397: Constify struct regulator_ops

2016-05-23 Thread Mark Brown
On Mon, May 23, 2016 at 03:30:16PM +0800, Henry Chen wrote: > Consitify the structure of regulator operations. Oh, actually the mode setting patch does apply but it depends on this one (which was sent after...). It would have been better to send a patch series for the driver and separately a patc

Re: [PATCH v2 3/3] regulator: mt6397: Add buck change mode regulator interface for mt6397

2016-05-23 Thread Mark Brown
On Mon, May 23, 2016 at 03:13:31PM +0800, Henry Chen wrote: > BUCKs of mt6397 have auto mode and pwm mode. > User can use regulator interfaces to control modes This doesn't apply against current code, please check and resend. signature.asc Description: PGP signature

Re: [PATCH V7 08/11] pci, acpi: Support for ACPI based generic PCI host controller

2016-05-23 Thread Tomasz Nowicki
On 14.05.2016 11:07, Jayachandran C wrote: On Tue, May 10, 2016 at 8:49 PM, Tomasz Nowicki wrote: This patch is going to implement generic PCI host controller for ACPI world, similar to what pci-host-generic.c driver does for DT world. All such drivers, which we have seen so far, were implemen

Re: [PATCH V7 00/11] Support for generic ACPI based PCI host controller

2016-05-23 Thread Dongdong Liu
在 2016/5/10 23:19, Tomasz Nowicki 写道: From the functionality point of view this series may be split into the following logic parts: 1. New ECAM API and update for users of the pci-host-common API 2. Necessary fixes as the preparation for using driver on ARM64. 3. Use new MCFG interface and imp

Re: [PATCH 2/3] media: add media_device_unregister_put() interface

2016-05-23 Thread Hans Verkuil
On 05/13/2016 07:09 PM, Shuah Khan wrote: > Add media_device_unregister_put() interface to release reference to a media > device allocated using the Media Device Allocator API. The media device is > unregistered and freed when the last driver that holds the reference to the > media device releases

[PATCH v2] dma-buf: use vma_pages().

2016-05-23 Thread Muhammad Falak R Wani
Replace explicit computation of vma page count by a call to vma_pages(). Also, include Signed-off-by: Muhammad Falak R Wani --- drivers/dma-buf/dma-buf.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c index 4a2c07

[PATCH] MIPS: devicetree: fix cpu interrupt controller node-names

2016-05-23 Thread Antony Pavlov
Here is the quote from [1]: The unit-address must match the first address specified in the reg property of the node. If the node has no reg property, the @ and unit-address must be omitted and the node-name alone differentiates the node from other nodes at the same level This patc

[PATCH v3] input: tablet: add Pegasus Notetaker tablet driver

2016-05-23 Thread Martin Kepplinger
This adds a driver for the Pegasus Notetaker Pen. When connected, this uses the Pen as an input tablet. This device was sold in various different brandings, for example "Pegasus Mobile Notetaker M210", "Genie e-note The Notetaker", "Staedtler Digital ballpoint pen 990 01",

Re: [PATCH 3/4] x86: Rewrite switch_to() code

2016-05-23 Thread Josh Poimboeuf
On Sun, May 22, 2016 at 09:47:22PM -0700, Andy Lutomirski wrote: > On Sun, May 22, 2016 at 7:34 PM, Josh Poimboeuf wrote: > > On Sun, May 22, 2016 at 10:59:38AM -0700, Andy Lutomirski wrote: > >> cc: Josh Poimboeuf: do you care about the exact stack layout of the > >> bottom of the stack of an ina

[GIT PULL] Security subsystem updates

2016-05-23 Thread James Morris
Please pull these minor updates for the keys code. The following changes since commit 7639dad93a5564579987abded4ec05e3db13659d: Merge tag 'trace-v4.7-2' of git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace (2016-05-22 19:40:39 -0700) are available in the git repository at:

[PATCH 4/4] usb: gadget: mv_u3d: fix unused-but-set-variable warnings

2016-05-23 Thread Michal Nazarewicz
This patch fixes the following (W=1) warnings: drivers/usb/gadget/udc/mv_u3d_core.c: In function ‘mv_u3d_process_ep_req’: drivers/usb/gadget/udc/mv_u3d_core.c:124:6: warning: variable ‘trb_complete’ set but not used [-Wunused-but-set-variable] int trb_complete, actual, remaining_length = 0;

[PATCH 1/4] usb: gadget: fix unused-but-set-variale warnings

2016-05-23 Thread Michal Nazarewicz
Those are enabled with W=1 make option. type-limits warnings in uvc_configfs.o and f_hid.c are left because they are caused by generic macros used in a way where they produce always-false conditions, e.g.: CC [M] drivers/usb/gadget/function/uvc_configfs.o drivers/usb/gadget/function/uvc_config

[PATCH 0/4] usb: gadget: fix most of W=1 warinngs

2016-05-23 Thread Michal Nazarewicz
Fixes all of the unused-but-set-variable warnings enabled when building with W=1. As described in the first patch, some warnings are left off. See said patch for more description. Michal Nazarewicz (4): usb: gadget: fix unused-but-set-variale warnings usb: gadget: m66592: fix unused-but-set-

[PATCH 3/4] usb: gadget: r8a66597: fix unused-but-set-variable warnings

2016-05-23 Thread Michal Nazarewicz
This patch fixes the following (W=1) warnings: drivers/usb/gadget/udc/r8a66597-udc.c: In function ‘r8a66597_irq’: drivers/usb/gadget/udc/r8a66597-udc.c:1468:15: warning: variable ‘nrdyenb’ set but not used [-Wunused-but-set-variable] u16 brdyenb, nrdyenb, bempenb; ^ drivers/usb/g

Re: [PATCH 3/4] x86: Rewrite switch_to() code

2016-05-23 Thread Josh Poimboeuf
On Mon, May 23, 2016 at 07:14:14AM -0400, Brian Gerst wrote: > On Sun, May 22, 2016 at 10:34 PM, Josh Poimboeuf wrote: > > On Sun, May 22, 2016 at 10:59:38AM -0700, Andy Lutomirski wrote: > >> cc: Josh Poimboeuf: do you care about the exact stack layout of the > >> bottom of the stack of an inacti

[PATCH 2/4] usb: gadget: m66592: fix unused-but-set-variable warnings

2016-05-23 Thread Michal Nazarewicz
This patch fixes the following (W=1) warnings: drivers/usb/gadget/udc/m66592-udc.c: In function ‘m66592_irq’: drivers/usb/gadget/udc/m66592-udc.c:1203:15: warning: variable ‘nrdyenb’ set but not used [-Wunused-but-set-variable] u16 brdyenb, nrdyenb, bempenb; ^ drivers/usb/gadget/

Re: [PATCH 3/4] x86: Rewrite switch_to() code

2016-05-23 Thread Josh Poimboeuf
On Mon, May 23, 2016 at 06:47:22AM -0500, Josh Poimboeuf wrote: > On Mon, May 23, 2016 at 07:14:14AM -0400, Brian Gerst wrote: > > On Sun, May 22, 2016 at 10:34 PM, Josh Poimboeuf > > wrote: > > > On Sun, May 22, 2016 at 10:59:38AM -0700, Andy Lutomirski wrote: > > >> cc: Josh Poimboeuf: do you c

Re: [PATCH 3/4] x86: Rewrite switch_to() code

2016-05-23 Thread Brian Gerst
On Mon, May 23, 2016 at 7:40 AM, Josh Poimboeuf wrote: > On Sun, May 22, 2016 at 09:47:22PM -0700, Andy Lutomirski wrote: >> On Sun, May 22, 2016 at 7:34 PM, Josh Poimboeuf wrote: >> > On Sun, May 22, 2016 at 10:59:38AM -0700, Andy Lutomirski wrote: >> >> cc: Josh Poimboeuf: do you care about the

Re: [PATCH 3/3] dma-buf/fence: add signal_on_any to the fence array

2016-05-23 Thread Chris Wilson
On Mon, May 23, 2016 at 01:25:22PM +0200, Christian König wrote: > From: Christian König > > If @signal_on_any is true the fence array signals if any fence in the array > signals, otherwise it signals when all fences in the array signal. > > Signed-off-by: Christian König > --- > drivers/dma-b

Re: [PATCH 7/8] pipe: account to kmemcg

2016-05-23 Thread kbuild test robot
-remove-pointless-struct-in-struct-page-definition/20160523-182939 config: blackfin-TCM-BF537_defconfig (attached as .config) compiler: bfin-uclinux-gcc (GCC) 4.6.3 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross

Re: [PATCH 7/8] pipe: account to kmemcg

2016-05-23 Thread kbuild test robot
-remove-pointless-struct-in-struct-page-definition/20160523-182939 config: um-x86_64_defconfig (attached as .config) compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430 reproduce: # save the attached .config to linux build tree make ARCH=um SUBARCH=x86_64 Note: the linux-review/Vladimir

Re: [PATCH 03/16] sched/fair: Disregard idle task wakee_flips in wake_wide

2016-05-23 Thread Morten Rasmussen
On Mon, May 23, 2016 at 01:12:07PM +0200, Mike Galbraith wrote: > On Mon, 2016-05-23 at 11:58 +0100, Morten Rasmussen wrote: > > wake_wide() is based on task wakee_flips of the waker and the wakee to > > decide whether an affine wakeup is desirable. On lightly loaded systems > > the waker is freque

[PATCH] userns: Allow init_user_ns to be used from non-gpl modules

2016-05-23 Thread Nikolay Borisov
This patch changes the export attributes of the init_user_ns from GPL-only to any modules. This needed so that non-gpl modules, such as ZFS, utilize functions like i_(uid|gid)_(read|write). Signed-off-by: Nikolay Borisov --- kernel/user.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH V2 2/2] usb: musb: Stop bulk endpoint while queue is rotated

2016-05-23 Thread Andrew Goodbody
Ensure that the endpoint is stopped by clearing REQPKT before clearing DATAERR_NAKTIMEOUT before rotating the queue on the dedicated bulk endpoint. This addresses an issue where a race could result in the endpoint receiving data before it was reprogrammed resulting in a warning about such data from

[PATCH V2 0/2] usb: musb: fix dropped packets

2016-05-23 Thread Andrew Goodbody
The musb driver can drop rx packets when heavily loaded. These two patches address two issues that can cause this. Both issues arose when an endpoint was reprogrammed. The first patch is a logic bug that resulted in a shared_fifo in rx mode not having its state cleared out. The second patch fixes a

Re: bpf: use-after-free in array_map_alloc

2016-05-23 Thread Vlastimil Babka
[+CC Christoph, linux-mm] On 04/17/2016 07:29 PM, Alexei Starovoitov wrote: > On Sun, Apr 17, 2016 at 12:58:21PM -0400, Sasha Levin wrote: >> Hi all, >> >> I've hit the following while fuzzing with syzkaller inside a KVM tools guest >> running the latest -next kernel: > > thanks for the report. A

[PATCH V2 1/2] usb: musb: Ensure rx reinit occurs for shared_fifo endpoints

2016-05-23 Thread Andrew Goodbody
shared_fifo endpoints would only get a previous tx state cleared out, the rx state was only cleared for non shared_fifo endpoints Change this so that the rx state is cleared for all endpoints. This addresses an issue that resulted in rx packets being dropped silently. Signed-off-by: Andrew Goodbod

Re: [PATCH 3/4] x86: Rewrite switch_to() code

2016-05-23 Thread Josh Poimboeuf
On Mon, May 23, 2016 at 07:49:37AM -0400, Brian Gerst wrote: > On Mon, May 23, 2016 at 7:40 AM, Josh Poimboeuf wrote: > > On Sun, May 22, 2016 at 09:47:22PM -0700, Andy Lutomirski wrote: > >> On Sun, May 22, 2016 at 7:34 PM, Josh Poimboeuf > >> wrote: > >> > On Sun, May 22, 2016 at 10:59:38AM -0

Re: bpf: use-after-free in array_map_alloc

2016-05-23 Thread Vlastimil Babka
On 05/23/2016 02:01 PM, Vlastimil Babka wrote: >> if I read the report correctly it's not about bpf, but rather points to >> the issue inside percpu logic. >> First __alloc_percpu_gfp() is called, then the memory is freed with >> free_percpu() which triggers async pcpu_balance_work and then >> pcpu

<    1   2   3   4   5   6   7   8   9   >