Re: [PATCH] net: rtnl: avoid uninitialized data in IFLA_VF_VLAN_LIST handling

2016-09-30 Thread Arnd Bergmann
On Friday 30 September 2016, Eric Dumazet wrote: > > @@ -1753,6 +1753,9 @@ static int do_setvfinfo(struct net_device *dev, > > struct nlattr **tb) > > > > len++; > > } > > + if (len == 0) > > + return -EINVAL; > > + > >

[PATCH] staging: rtl8188eu: fix double unlock error in rtw_resume_process()

2016-09-30 Thread Wei Yongjun
Fix following static checker warning: drivers/staging/rtl8188eu/os_dep/usb_intf.c:311 rtw_resume_process() error: double unlock 'mutex:&pwrpriv->mutex_lock' Fixes: eaf47b713b60 ("staging: rtl8188eu: fix missing unlock on error in rtw_resume_process()") Reported-By: Dan Carpenter Signed-of

Re: [PATCH 3/3] arm64: dump: Add checking for writable and exectuable pages

2016-09-30 Thread Mark Rutland
On Fri, Sep 30, 2016 at 09:25:45AM -0700, Kees Cook wrote: > On Fri, Sep 30, 2016 at 8:58 AM, Mark Rutland wrote: > > Would it be worth verifying that all kernel mappings are UXN, too? > > > > ARMv8 allows execute-only mappings, and a !UXN mapping could result in an > > info > > leak (e.g. point

Re: [PATCH] mtd: mtk: avoid warning in mtk_ecc_encode

2016-09-30 Thread Boris Brezillon
Hi Arnd, On Fri, 30 Sep 2016 18:33:02 +0200 Arnd Bergmann wrote: > When building with -Wmaybe-uninitialized, gcc produces a silly false positive > warning for the mtk_ecc_encode function: > > drivers/mtd/nand/mtk_ecc.c: In function 'mtk_ecc_encode': > drivers/mtd/nand/mtk_ecc.c:402:15: error: '

Re: [PATCH] Adding Support for Coresight Components on Zynq 7000.

2016-09-30 Thread Sören Brinkmann
Hi Muhammad, On Fri, 2016-09-30 at 09:39:51 +0200, Muhammad Abdul WAHAB wrote: > Hi Sören, > > Thank you for your remarks. I corrected a few things as you suggested. > [...] > >> +tpiu@F8803000 { > >>> > >>> +compatible = "arm,coresight-tpiu", "arm,primecell"; > >>> +

RE: [PATCH] drm/amdgpu: enable powerplay unconditionally

2016-09-30 Thread Deucher, Alexander
> -Original Message- > From: Arnd Bergmann [mailto:a...@arndb.de] > Sent: Friday, September 30, 2016 12:21 PM > To: Deucher, Alexander; Koenig, Christian > Cc: Arnd Bergmann; David Airlie; Zhou, Jammy; Zhu, Rex; dri- > de...@lists.freedesktop.org; linux-kernel@vger.kernel.org > Subject: [PA

Re: [PATCH 2/3] netfilter: hide reference to nf_hooks_ingress

2016-09-30 Thread Aaron Conole
Arnd Bergmann writes: > A recent cleanup added an unconditional reference to the nf_hooks_ingress > pointer, > but that fails when CONFIG_NETFILTER_INGRESS is disabled and that member is > not present in net_device: > > net/netfilter/core.c: In function 'nf_set_hooks_head': > net/netfilter/core.

[PATCH v2] clk: bcm2835: Clamp the PLL's requested rate to the hardware limits.

2016-09-30 Thread Eric Anholt
Fixes setting low-resolution video modes on HDMI. Now the PLLH_PIX divider adjusts itself until the PLLH is within bounds. Signed-off-by: Eric Anholt --- drivers/clk/bcm/clk-bcm2835.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/clk/bcm/clk-bcm2835.c b

Re: [PATCH 1/2 v2] f2fs: use crc and cp version to determine roll-forward recovery

2016-09-30 Thread Jaegeuk Kim
Previously, we used cp_version only to detect recoverable dnodes. In order to avoid same garbage cp_version, we needed to truncate the next dnode during checkpoint, resulting in additional discard or data write. If we can distinguish this by using crc in addition to cp_version, we can remove this o

[PATCH] usb: fix a typo in usb_class_driver documentation

2016-09-30 Thread Amitesh Singh
replace usb_unregister_dev by usb_deregister_dev Signed-off-by: Amitesh Singh --- include/linux/usb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/usb.h b/include/linux/usb.h index eba1f10..7e68259 100644 --- a/include/linux/usb.h +++ b/include/linux/usb.h @@

[PATCH] usb: fix a typo in usb_class_driver documentation

2016-09-30 Thread Amitesh Singh
replace usb_unregister_dev by usb_deregister_dev Signed-off-by: Amitesh Singh --- include/linux/usb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/usb.h b/include/linux/usb.h index eba1f10..7e68259 100644 --- a/include/linux/usb.h +++ b/include/linux/usb.h @@

Re: [PATCH V2 2/2] fs/super.c: don't fool lockdep in freeze_super() and thaw_super() paths

2016-09-30 Thread Oleg Nesterov
On 09/27, Oleg Nesterov wrote: > > On 09/27, Jan Kara wrote: > > > > You can run either: > > > > ./check -g freeze > > passed all 6 tests. > > > to check just the freezing tests or > > > > ./check > > > > to run all sensible tests which is what I'd do (but it will take couple of > > hours t

Re: [PATCH 3/3] arm64: dump: Add checking for writable and exectuable pages

2016-09-30 Thread Kees Cook
On Fri, Sep 30, 2016 at 9:41 AM, Mark Rutland wrote: > On Fri, Sep 30, 2016 at 09:25:45AM -0700, Kees Cook wrote: >> On Fri, Sep 30, 2016 at 8:58 AM, Mark Rutland wrote: > >> > Would it be worth verifying that all kernel mappings are UXN, too? >> > >> > ARMv8 allows execute-only mappings, and a !

Re: [PATCH 3/3] netfilter: xt_hashlimit: uses div_u64 for division

2016-09-30 Thread Vishwanath Pai
On 09/30/2016 12:38 PM, Eric Dumazet wrote: > On Fri, 2016-09-30 at 18:05 +0200, Arnd Bergmann wrote: >> The newly added support for high-resolution pps rates introduced multiple >> 64-bit >> division operations in one function, which fails on all 32-bit architectures: >> >> net/netfilter/xt_hashl

Re: [PATCH 1/3] drm/radeon: add missing header dependencies

2016-09-30 Thread Alex Deucher
On Fri, Sep 30, 2016 at 7:52 AM, Christian König wrote: > This one and patch #3 are Reviewed-by: Christian König > . > Applied 1 and 3, thanks! Alex > Where is patch #2? That never made it into my inbox. > > Regards, > Christian. > > > Am 30.09.2016 um 10:13 schrieb Baoyou Xie: >> >> We get a f

Re: [PATCH] mtd: mtk: avoid warning in mtk_ecc_encode

2016-09-30 Thread Arnd Bergmann
On Friday 30 September 2016, Boris Brezillon wrote: > > + /* copy into possibly unaligned OOB region with actual length */ > > + memcpy(data + bytes, eccdata, len); > > Is it better than > > for (i = 0; i < len; i += 4) { > u32 val = __raw_readl(ecc->regs + ECC_ENC

Re: [PATCH 1/2] drm/amd/powerplay: mark symbols static where possible

2016-09-30 Thread Alex Deucher
On Fri, Sep 30, 2016 at 8:16 AM, Christian König wrote: > Both patches are Acked-by: Christian König . Applied patch 1. I'd like to get Rex's ack on patch 2 before applying it. Alex > > Regards, > Christian. > > > Am 30.09.2016 um 11:58 schrieb Baoyou Xie: >> >> We get a few warnings when buil

Re: [PATCH 1/1] Add devices to spidev device tree compatibility list

2016-09-30 Thread Geert Uytterhoeven
On Fri, Sep 30, 2016 at 12:15 PM, Fabien Lahoudere wrote: > Entries are needed in the spidev ID list to configure configure it from a > device tree. Add entries for the following devices: > - "exar,xra1403" : a GPIO expander Looks like a suitable target for a drivers/gpio/ driver. Datasheet avai

Re: [PATCH] edma: avoid uninitialized variable use

2016-09-30 Thread Vinod Koul
On Fri, Sep 30, 2016 at 06:19:01PM +0200, Arnd Bergmann wrote: > If edma_read_slot() gets an invalid argument, it does not set a result, > as found by "gcc -Wmaybe-uninitialized" > > drivers/dma/edma.c: In function 'dma_ccerr_handler': > drivers/dma/edma.c:1499:21: error: 'p.a_b_cnt' may be used u

Re: [PATCH 3/3] netfilter: xt_hashlimit: uses div_u64 for division

2016-09-30 Thread Arnd Bergmann
On Friday 30 September 2016, Eric Dumazet wrote: > On Fri, 2016-09-30 at 18:05 +0200, Arnd Bergmann wrote: > > net/netfilter/xt_hashlimit.c | 17 ++--- > > 1 file changed, 10 insertions(+), 7 deletions(-) > > > > diff --git a/net/netfilter/xt_hashlimit.c b/net/netfilter/xt_hashlimit.c

Re: [PATCH][V2] dmaengine: coh901318: fix integer overflow when shifting more than 32 places

2016-09-30 Thread Vinod Koul
On Thu, Sep 29, 2016 at 07:14:38PM +0100, Colin King wrote: > From: Colin Ian King > > Currently U300_DMA_CHANNELS is set to 40, meaning that the shift of 1 can > be more than 32 places, which leads to a 32 bit integer overflow. Fix this > by using 1ULL instead of 1 before shifting it. Also add

[RFC] arm64: Enforce observed order for spinlock and data

2016-09-30 Thread Brent DeGraaf
Prior spinlock code solely used load-acquire and store-release semantics to ensure ordering of the spinlock lock and the area it protects. However, store-release semantics and ordinary stores do not protect against accesses to the protected area being observed prior to the access that locks the loc

Re: [PATCH] dmaengine: jz4780: fix resource leaks on error exit return

2016-09-30 Thread Vinod Koul
On Thu, Sep 29, 2016 at 06:45:05PM +0100, Colin King wrote: > From: Colin Ian King > > In two cases when jz4780_dma_setup_hwdesc fails, there is a memory > leak on the allocated desc and associated DMA pools on the error > exit return path. Fix this by free'ing the resources before > returning.

Re: [PATCH] dma/fsldma : Unmap region obtained by of_iomap

2016-09-30 Thread Vinod Koul
On Wed, Sep 28, 2016 at 04:15:11PM +0530, Arvind Yadav wrote: > Free memory mapping, if probe is not successful. Please use proper subsystem tags for patches. Hint: use git log to find that out Applied after fixing the tag -- ~Vinod

Re: [PATCH 1/3] netfilter: nf_tables: avoid uninitialized variable warning

2016-09-30 Thread Pablo Neira Ayuso
On Fri, Sep 30, 2016 at 06:05:34PM +0200, Arnd Bergmann wrote: > The newly added nft_range_eval() function handles the two possible > nft range operations, but as the compiler warning points out, > any unexpected value would lead to the 'mismatch' variable being > used without being initialized: >

[PATCH 2/3] proc: Stop trying to report thread stacks

2016-09-30 Thread Andy Lutomirski
This reverts more of b76437579d13 ("procfs: mark thread stack correctly in proc//maps"), which was partially reverted by 65376df58217 ("proc: revert /proc//maps [stack:TID] annotation"). Originally, /proc/PID/task/TID/maps was the same as /proc/TID/maps. In current kernels, /proc/PID/maps (or /pr

[PATCH 0/3] ABI CHANGE!!! Remove questionable remote SP reads

2016-09-30 Thread Andy Lutomirski
Jann Horn noticed that KSTK_ESP + eager task stack freeing was a bad combination and could crash. I could very easily fix it to not crash, but I think that using KSTK_ESP on a remote task is questionable in general. Therefore, I propose to get rid of the major users for 4.9. This series makes tw

[PATCH 1/3] proc: Stop reporting eip and esp in /proc/PID/stat

2016-09-30 Thread Andy Lutomirski
Reporting these fields on a non-current task is dangerous. If the task is in any state other than normal kernel code, they may contain garbage or even kernel addresses on some architectures. (x86_64 used to do this. I bet lots of architectures still do.) With CONFIG_THREAD_INFO_IN_TASK, it can

[PATCH 3/3] mm: Change vm_is_stack_for_task() to vm_is_stack_for_current()

2016-09-30 Thread Andy Lutomirski
Asking for a non-current task's stack can't be done without races unless the task is frozen in kernel mode. As far as I know, vm_is_stack_for_task() never had a safe non-current use case. The __unused annotation is because some KSTK_ESP implementations ignore their parameter, which IMO is further

Re: [PATCH 2/2] mlxsw: spectrum_router: avoid potential uninitialized data usage

2016-09-30 Thread Ido Schimmel
On Fri, Sep 30, 2016 at 06:17:10PM +0200, Arnd Bergmann wrote: > If fi->fib_nhs is zero, the router interface pointer is uninitialized, as > shown by > this warning: > > drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c: In function > 'mlxsw_sp_router_fib_event': > drivers/net/ethernet/mella

Re: [PATCH] dmaengine: k3dma: fix off by one

2016-09-30 Thread Vinod Koul
On Wed, Sep 21, 2016 at 10:16:55PM +0200, Vincent Stehlé wrote: > In k3_of_dma_simple_xlate(), the d->chans[] array has d->dma_requests > elements so > should be >=. > > Fixes: 8e6152bc660e69f5 ("dmaengine: Add hisilicon k3 DMA engine driver") > Signed-off-by: Vincent Stehlé > Cc: Zhangfei Gao >

[PATCH] lib/raid6: Add AVX2 optimized xor_syndrome functions

2016-09-30 Thread Gayatri Kammela
Implement the AVX2 optimization of RAID6 xor_syndrome functions which is simply based on sse2.c written by hpa. Cc: H. Peter Anvin Cc: Yuanhan Liu Cc: Fenghua Yu Signed-off-by: Gayatri Kammela --- lib/raid6/avx2.c | 232 ++- 1 file changed,

Re: [PATCH v2 01/14] irqchip: i8259: Add domain before mapping parent irq

2016-09-30 Thread Thomas Gleixner
On Fri, 30 Sep 2016, Paul Burton wrote: > On Monday, 19 September 2016 22:21:18 BST Paul Burton wrote: > > Mapping the parent IRQ will use a virq number which may conflict with > > the hardcoded I8259A_IRQ_BASE..I8259A_IRQ_BASE+15 range that the i8259 > > driver expects to be free. If this occurs

Re: [RFC] arm64: Enforce observed order for spinlock and data

2016-09-30 Thread Robin Murphy
Hi Brent, On 30/09/16 18:40, Brent DeGraaf wrote: > Prior spinlock code solely used load-acquire and store-release > semantics to ensure ordering of the spinlock lock and the area it > protects. However, store-release semantics and ordinary stores do > not protect against accesses to the protected

Re: [RFC PATCH v3 2/2] usb: typec: Type-C Port Controller Interface driver (tcpci)

2016-09-30 Thread Guenter Roeck
On Thu, Sep 29, 2016 at 11:24 PM, Jun Li wrote: > Hi Guenter, > >> -Original Message- >> From: linux-usb-ow...@vger.kernel.org [mailto:linux-usb- >> ow...@vger.kernel.org] On Behalf Of Guenter Roeck >> Sent: Wednesday, August 24, 2016 5:11 AM >> To: Felipe Balbi >> Cc: Chandra Sekhar Anag

Re: [PATCH 1/3] netfilter: nf_tables: avoid uninitialized variable warning

2016-09-30 Thread Pablo Neira Ayuso
On Fri, Sep 30, 2016 at 07:47:49PM +0200, Pablo Neira Ayuso wrote: > On Fri, Sep 30, 2016 at 06:05:34PM +0200, Arnd Bergmann wrote: > > The newly added nft_range_eval() function handles the two possible > > nft range operations, but as the compiler warning points out, > > any unexpected value would

Re: [PATCH RT] kernel/futex: don't deboost too early

2016-09-30 Thread Thomas Gleixner
On Fri, 30 Sep 2016, Steven Rostedt wrote: > This looks awfully complex. Would something as simple as this work? > > What harm can happen by moving the holding of the lock after the > wakeups for RT? That's exactly bringing us back to the state before we added the delayed wakeup so that the woken

Re: [RFC] arm64: Enforce observed order for spinlock and data

2016-09-30 Thread Peter Zijlstra
On Fri, Sep 30, 2016 at 01:40:57PM -0400, Brent DeGraaf wrote: > Prior spinlock code solely used load-acquire and store-release > semantics to ensure ordering of the spinlock lock and the area it > protects. However, store-release semantics and ordinary stores do > not protect against accesses to t

Re: [PATCH 2/2] staging: vt6656: Make 'rx_rate' u8 instead of u8 *

2016-09-30 Thread Malcolm Priestley
On 28/09/16 08:40, Dan Carpenter wrote: On Tue, Sep 27, 2016 at 02:52:49PM -0300, Martin Alonso wrote: Change the type and uses of rx_rate. Signed-off-by: Martin Alonso --- drivers/staging/vt6656/dpc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/stagin

Re: [PATCH 1/3] proc: Stop reporting eip and esp in /proc/PID/stat

2016-09-30 Thread Jann Horn
On Fri, Sep 30, 2016 at 10:58:56AM -0700, Andy Lutomirski wrote: > Reporting these fields on a non-current task is dangerous. If the > task is in any state other than normal kernel code, they may contain > garbage or even kernel addresses on some architectures. Stupid question: Doesn't something

Re: [PATCH 1/2] ARM: BCM5301X: Add DT for Luxul XAP-1510

2016-09-30 Thread Florian Fainelli
On 09/27/2016 10:27 AM, Dan Haab wrote: > Luxul XAP-1510 is an AP device based on BCM4708 SoC with 2 x BCM4360 > chipsets on PCB connected using PCIe. > > Signed-off-by: Dan Haab Series applied, thanks! -- Florian

Re: [PATCH 1/3] ARM: BCM5301X: Add separated DTS include file for BCM47094

2016-09-30 Thread Florian Fainelli
On 09/21/2016 01:58 PM, Rafał Miłecki wrote: > From: Rafał Miłecki > > Use it to store BCM47094 specific properties/values and avoid repeating > them in device DTS files. Series applied, thanks! -- Florian

Re: [RFC] arm64: Enforce observed order for spinlock and data

2016-09-30 Thread Peter Zijlstra
On Fri, Sep 30, 2016 at 01:40:57PM -0400, Brent DeGraaf wrote: > Prior spinlock code solely used load-acquire and store-release > semantics to ensure ordering of the spinlock lock and the area it > protects. However, store-release semantics and ordinary stores do > not protect against accesses to t

Re: [RFC PATCH v3 1/2] usb: typec: USB Type-C Port Manager (tcpm)

2016-09-30 Thread Guenter Roeck
On Thu, Sep 29, 2016 at 11:37 PM, Jun Li wrote: > Hi Guenter, >> -Original Message- >> From: linux-usb-ow...@vger.kernel.org [mailto:linux-usb- >> ow...@vger.kernel.org] On Behalf Of Guenter Roeck >> Sent: Wednesday, August 24, 2016 5:11 AM >> To: Felipe Balbi >> Cc: Chandra Sekhar Anagan

[PATCH] Staging: rts5208: Fix indentation warnings

2016-09-30 Thread Wayne Porter
Fix code indentation warnings detected by checkpatch.pl Signed-off-by: Wayne Porter --- drivers/staging/rts5208/ms.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/staging/rts5208/ms.c b/drivers/staging/rts5208/ms.c index 9fc64da..f27df0b

[PATCH v6 0/7] ZBC / Zoned block device support

2016-09-30 Thread Shaun Tancheff
This series introduces support for zoned block devices. It integrates earlier submissions by Hannes Reinecke, Damien Le Moal and Shaun Tancheff. Compared to the previous series version, the code was significantly simplified by limiting support to zoned devices satisfying the following conditions: 1

[PATCH v6 1/7] block: Add 'zoned' queue limit

2016-09-30 Thread Shaun Tancheff
From: Damien Le Moal Add the zoned queue limit to indicate the zoning model of a block device. Defined values are 0 (BLK_ZONED_NONE) for regular block devices, 1 (BLK_ZONED_HA) for host-aware zone block devices and 2 (BLK_ZONED_HM) for host-managed zone block devices. The standards defined drive

[PATCH v6 2/7] blk-sysfs: Add 'chunk_sectors' to sysfs attributes

2016-09-30 Thread Shaun Tancheff
From: Hannes Reinecke The queue limits already have a 'chunk_sectors' setting, so we should be presenting it via sysfs. Signed-off-by: Hannes Reinecke [Damien: Updated Documentation/ABI/testing/sysfs-block] Signed-off-by: Damien Le Moal Reviewed-by: Christoph Hellwig Reviewed-by: Martin K.

[PATCH v6 3/7] block: update chunk_sectors in blk_stack_limits()

2016-09-30 Thread Shaun Tancheff
From: Hannes Reinecke Signed-off-by: Hannes Reinecke Signed-off-by: Damien Le Moal Reviewed-by: Christoph Hellwig Reviewed-by: Martin K. Petersen Reviewed-by: Shaun Tancheff Tested-by: Shaun Tancheff --- block/blk-settings.c | 4 1 file changed, 4 insertions(+) diff --git a/block/blk

[PATCH v6 5/7] block: Implement support for zoned block devices

2016-09-30 Thread Shaun Tancheff
From: Hannes Reinecke Implement zoned block device zone information reporting and reset. Zone information are reported as struct blk_zone. This implementation does not differentiate between host-aware and host-managed device models and is valid for both. Two functions are provided: blkdev_report_

[PATCH v6 7/7] blk-zoned: implement ioctls

2016-09-30 Thread Shaun Tancheff
Adds the new BLKREPORTZONE and BLKRESETZONE ioctls for respectively obtaining the zone configuration of a zoned block device and resetting the write pointer of sequential zones of a zoned block device. The BLKREPORTZONE ioctl maps directly to a single call of the function blkdev_report_zones. The

[PATCH v6 4/7] block: Define zoned block device operations

2016-09-30 Thread Shaun Tancheff
From: Shaun Tancheff Define REQ_OP_ZONE_REPORT and REQ_OP_ZONE_RESET for handling zones of host-managed and host-aware zoned block devices. With with these two new operations, the total number of operations defined reaches 8 and still fits with the 3 bits definition of REQ_OP_BITS. Signed-off-by

[PATCH v6 6/7] sd: Implement support for ZBC devices

2016-09-30 Thread Shaun Tancheff
From: Hannes Reinecke Implement ZBC support functions to setup zoned disks, both host-managed and host-aware models. Only zoned disks that satisfy the following conditions are supported: 1) All zones are the same size, with the exception of an eventual last smaller runt zone. 2) For host-manag

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-09-30 Thread Peter Ujfalusi
On 09/30/2016 06:06 PM, Bartosz Golaszewski wrote: > 2016-09-30 14:59 GMT+02:00 Peter Ujfalusi : >> On 09/29/16 19:31, Bartosz Golaszewski wrote: >>> Default memory settings of da850 do not meet the throughput/latency >>> requirements of tilcdc. This results in the image displayed being >>> incorre

[PATCH] leds: leds-pca963x: add power management support

2016-09-30 Thread Matt Ranostay
Turn off LEDS on suspend and put device in low power state, and restore settings on resume. Cc: Tony Lindgren Cc: Richard Purdie Cc: Jacek Anaszewski Signed-off-by: Matt Ranostay --- drivers/leds/leds-pca963x.c | 31 +++ 1 file changed, 31 insertions(+) diff --git

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-30 Thread Sinan Kaya
Rafael, Bjorn; On 9/30/2016 11:56 AM, Ondrej Zary wrote: >>> It seems to work, at least on one machine. >> > >> > Ok, that comfirms my suspicion. We are having trouble detecting sci >> > interrupt type and we end up penalizing the wrong value. >> > >> > Can you try your other machines too? > Wor

Re: [RFC] arm64: Enforce observed order for spinlock and data

2016-09-30 Thread Mark Rutland
On Fri, Sep 30, 2016 at 01:40:57PM -0400, Brent DeGraaf wrote: > Prior spinlock code solely used load-acquire and store-release > semantics to ensure ordering of the spinlock lock and the area it > protects. However, store-release semantics and ordinary stores do > not protect against accesses to t

Re: [PATCH v15 04/13] task_isolation: add initial support

2016-09-30 Thread Chris Metcalf
On 8/30/2016 2:43 PM, Andy Lutomirski wrote: On Aug 30, 2016 10:02 AM, "Chris Metcalf" wrote: We really want to run task isolation last, so we can guarantee that all the isolation prerequisites are met (dynticks stopped, per-cpu lru cache empty, etc). But achieving that state can require enabl

Re: [PATCH 2/2] staging: vt6656: Make 'rx_rate' u8 instead of u8 *

2016-09-30 Thread Dan Carpenter
On Fri, Sep 30, 2016 at 07:53:00PM +0100, Malcolm Priestley wrote: > On 28/09/16 08:40, Dan Carpenter wrote: > >On Tue, Sep 27, 2016 at 02:52:49PM -0300, Martin Alonso wrote: > >>Change the type and uses of rx_rate. > >> > >>Signed-off-by: Martin Alonso > >>--- > >> drivers/staging/vt6656/dpc.c |

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-30 Thread Rafael J. Wysocki
On Fri, Sep 30, 2016 at 9:30 PM, Sinan Kaya wrote: > Rafael, Bjorn; > > On 9/30/2016 11:56 AM, Ondrej Zary wrote: It seems to work, at least on one machine. >>> > >>> > Ok, that comfirms my suspicion. We are having trouble detecting sci >>> > interrupt type and we end up penalizing the wron

Re: [RFC PATCH v3 1/2] usb: typec: USB Type-C Port Manager (tcpm)

2016-09-30 Thread Joe Perches
On Fri, 2016-09-30 at 12:06 -0700, Guenter Roeck wrote: > On Thu, Sep 29, 2016 at 11:37 PM, Jun Li wrote: [] > > diff --git a/include/linux/usb/pd.h b/include/linux/usb/pd.h [] > > +#define PDO_VAR(min_mv, max_mv, max_ma) > > \ > > + ((PDO_TYPE_VAR << PDO_

Validate Your USAA Bank Account

2016-09-30 Thread USAA
USAA NEW MESSAGE.pdf Description: Adobe PDF document

Re: [PATCH] net: rtnl: avoid uninitialized data in IFLA_VF_VLAN_LIST handling

2016-09-30 Thread Or Gerlitz
On Fri, Sep 30, 2016 at 7:13 PM, Arnd Bergmann wrote: > With the newly added support for IFLA_VF_VLAN_LIST netlink messages, > we get a warning about potential uninitialized variable use in > the parsing of the user input when enabling the -Wmaybe-uninitialized > warning: > > net/core/rtnetlink.c:

Re: Question on ext4 directory hashes in combination with file name encryption

2016-09-30 Thread Andreas Dilger
On Sep 30, 2016, at 8:09 AM, Richard Weinberger wrote: > > Hi, > > if I read the ext4 code correctly, you pass encrypted filenames to > ext4fs_dirhash(). > These filenames are not encoded and therefore binary gibberish. > Isn't this a problem for the ext4 hash functions? My fear is that these

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-30 Thread Sinan Kaya
On 9/30/2016 3:39 PM, Rafael J. Wysocki wrote: >> how do we feel about increasing the ISA IRQ range to 256 so that >> > we are safe for all SCI interrupts? > I'm not sure how this is related to the problem at hand. Care to elaborate? > Sure, let me explain. The code maintains a static array pe

Re: [PATCH] sched/fair: Do not decay new task load on first enqueue

2016-09-30 Thread Matt Fleming
On Wed, 28 Sep, at 08:37:31PM, Matt Fleming wrote: > > I'm away on FTO right now. I can test this when I return on Friday. I haven't had chance to review your patch or the other emails in this thread yet, but I ran the patch on my test machine and it also restores performance. I'll run it through

Re: [RFC PATCH v3 1/2] usb: typec: USB Type-C Port Manager (tcpm)

2016-09-30 Thread Guenter Roeck
On Fri, Sep 30, 2016 at 12:41 PM, Joe Perches wrote: > On Fri, 2016-09-30 at 12:06 -0700, Guenter Roeck wrote: >> On Thu, Sep 29, 2016 at 11:37 PM, Jun Li wrote: > [] >> > diff --git a/include/linux/usb/pd.h b/include/linux/usb/pd.h > [] >> > +#define PDO_VAR(min_mv, max_mv, max_ma)

Re: [PATCH v3 2/2] leds: Add driver for NCP5623 3-channel I2C LED driver

2016-09-30 Thread Jacek Anaszewski
Hi Florian, On 09/29/2016 06:18 PM, Florian Vaussard wrote: Hi Jacek, Thank you for your comments! Le 18. 09. 16 à 20:20, Jacek Anaszewski a écrit : Hi Florian, Thanks for the updated patch set. I have few comments below. On 09/16/2016 01:34 PM, Florian Vaussard wrote: The NCP5623 is a 3-c

Re: [RFC PATCH v3 1/2] usb: typec: USB Type-C Port Manager (tcpm)

2016-09-30 Thread Joe Perches
On Fri, 2016-09-30 at 13:57 -0700, Guenter Roeck wrote: > Code now looks as follows. > > #define PDO_VAR_MIN_VOLT(mv) mv) / 50) & PDO_VAR_MIN_VOLT_MASK) << \ >   PDO_VAR_MIN_VOLT_SHIFT) > #define PDO_VAR_MAX_VOLT(mv) mv) / 50) & PDO_VAR_MAX_VOLT_MASK) << \ >  

RE: [PATCH] hv: do not lose pending heartbeat vmbus packets

2016-09-30 Thread Long Li
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Thursday, September 29, 2016 2:22 AM > To: KY Srinivasan ; Long Li > Cc: Haiyang Zhang ; > de...@linuxdriverproject.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] hv: do not lose pending heartbea

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-30 Thread Rafael J. Wysocki
On Fri, Sep 30, 2016 at 10:24 PM, Sinan Kaya wrote: > On 9/30/2016 3:39 PM, Rafael J. Wysocki wrote: >>> how do we feel about increasing the ISA IRQ range to 256 so that >>> > we are safe for all SCI interrupts? >> I'm not sure how this is related to the problem at hand. Care to elaborate? >> > >

[PATCH] ARCv2: fix local_save_flags

2016-09-30 Thread Vineet Gupta
Commit d9676fa152c83b ("ARCv2: Enable LOCKDEP"), changed local_save_flags() to not return raw STATUS32 but encoded in the form such that it could be fed directly to CLRI/SETI instructions. However the STATUS32.E[] was not captured correctly as it corresponds to bits [4:1] in the register and not [3

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-30 Thread Sinan Kaya
On 9/30/2016 5:04 PM, Rafael J. Wysocki wrote: >> > >> > I hope it makes sense now. I tend to skip details sometimes. Feel free to >> > send more questions. > Thanks for the information! > > IIUC, basically, what you are proposing would be to restore the old > penalizing method for IRQs in the 0-2

[PATCH 0/4] [media] exynos-gsc: Cleanup and fixes

2016-09-30 Thread Javier Martinez Canillas
Hello, This patch series contains a cleanup and some fixes for the exynos-gsc driver. I found these issues when trying to use the driver with the GStreamer v4l2videoconvert element. I wasn't able to display to Exynos DRM/KMS driver and only a small set of formats were working correctly. After the

[PATCH 2/4] [media] exynos-gsc: don't clear format when freeing buffers with REQBUFS(0)

2016-09-30 Thread Javier Martinez Canillas
User-space applications can use the VIDIOC_REQBUFS ioctl to determine if a memory mapped, user pointer or DMABUF based I/O is supported by a driver. For example, GStreamer attempts to determine the I/O methods supported by the driver by doing many VIDIOC_REQBUFS ioctl calls with different memory t

[PATCH 1/4] [media] exynos-gsc: change spamming try_fmt log message to debug

2016-09-30 Thread Javier Martinez Canillas
The driver try_fmt handler prints a message each time that the image size has been changed due the maximum and minimum width and height. Since user-space can try different format and sizes, this logs a lot of unnecessary messages. Change the message log level to debug and while being there, also a

Re: [PATCH] ARCv2: intc: Use kflag if STATUS32.IE must be reset

2016-09-30 Thread Vineet Gupta
On 09/12/2016 08:55 AM, Yuriy Kolerov wrote: > In the end of "arc_init_IRQ" STATUS32.IE flag is going to be affected by > "flag" instruction but "flag" never touches IE flag on ARCv2. So "kflag" > instruction must be used instead of "flag". > > Signed-off-by: Yuriy Kolerov > --- > arch/arc/kerne

[PATCH 4/4] [media] exynos-gsc: do proper bytesperline and sizeimage calculation

2016-09-30 Thread Javier Martinez Canillas
The driver don't take into account the differences between packed, semi planar and multi planar formats when calculating the pixel format bytes per lines and image size values. This makes GStreamer to fail when the following formats are used NV12, NV21, NV16, NV61, YV12, I420 and Y42B: "gst_video_

[PATCH 3/4] [media] exynos-gsc: fix supported RGB pixel format

2016-09-30 Thread Javier Martinez Canillas
The driver exposes 32-bit A/XRGB 8-8-8-8 as supported format but testing shows that using this format produces frames with wrong colors. The test was done with the following GStreamer pipeline: $ gst-launch-1.0 videotestsrc num-buffers=20 ! video/x-raw,format=UYVY \ ! v4l2video3convert ! video/x-r

Re: [PATCH] leds: leds-pca963x: add power management support

2016-09-30 Thread Jacek Anaszewski
Hi Matt, Thanks for the patch. On 09/30/2016 09:27 PM, Matt Ranostay wrote: Turn off LEDS on suspend and put device in low power state, and restore settings on resume. Cc: Tony Lindgren Cc: Richard Purdie Cc: Jacek Anaszewski Signed-off-by: Matt Ranostay --- drivers/leds/leds-pca963x.c |

[PATCH v3 02/13] futex: Rename futex_pi_state to futex_state

2016-09-30 Thread Waiman Long
The futex_pi_state structure will be overloaded in later patches to store state information about non-PI futexes. So the structure name itself is no longer a good description of its purpose. So its name is changed to futex_state, a more generic name. Some of the functions that process the futex st

[PATCH v3 01/13] futex: Consolidate duplicated timer setup code

2016-09-30 Thread Waiman Long
A new futex_setup_timer() helper function is added to consolidate all the hrtimer_sleeper setup code. Signed-off-by: Waiman Long --- kernel/futex.c | 67 +++- 1 files changed, 37 insertions(+), 30 deletions(-) diff --git a/kernel/futex.c b/k

[PATCH v3 09/13] futex: Implement lock handoff for TP futexes to prevent lock starvation

2016-09-30 Thread Waiman Long
The current TP futexes has no guarantee that the top waiter (serialization mutex owner) can get the lock within a finite time. As a result, lock starvation can happen. A lock handoff mechanism is added to the TP futexes to prevent lock starvation from happening. The idea is that the top waiter can

[PATCH 1/2] futex.2: Add description about the FUTEX_LOCK/FUTEX_UNLOCK operations

2016-09-30 Thread Waiman Long
The FUTEX_LOCK and FUTEX_UNLOCK operation pair is for supporting the new throughput-optimized futexes. The futex(2) manpage is extended to include description about the operations. Signed-off-by: Waiman Long --- man2/futex.2 | 49 + 1 files chang

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-30 Thread Rafael J. Wysocki
On Fri, Sep 30, 2016 at 11:14 PM, Sinan Kaya wrote: > On 9/30/2016 5:04 PM, Rafael J. Wysocki wrote: >>> > >>> > I hope it makes sense now. I tend to skip details sometimes. Feel free to >>> > send more questions. >> Thanks for the information! >> >> IIUC, basically, what you are proposing would b

[PATCH 2/2] futex.7: Add description about the FUTEX_LOCK/FUTEX_UNLOCK operations

2016-09-30 Thread Waiman Long
The FUTEX_LOCK and FUTEX_UNLOCK operation pair is for supporting the new throughput-optimized futexes. The futex(7) manpage is extended to include description about the operations. Signed-off-by: Waiman Long --- man7/futex.7 | 22 ++ 1 files changed, 22 insertions(+), 0 del

[PATCH v3 00/13] futex: Introducing throughput-optimized futexes

2016-09-30 Thread Waiman Long
v2->v3: - Use the abbreviation TP for the new futexes instead of TO. - Make a number of changes accordingly to review comments from ThomasG, PeterZ and MikeG. - Breaks the main futex patch into smaller pieces to make them easier to review. - Integrate the microbenchmark into the "p

[PATCH 0/2] futex: extend the futex(2) and futex(7) manpages for TP futexes

2016-09-30 Thread Waiman Long
This patchset includes changes to the futex(2) and futex(7) manpages to add descriptions for the new throughput-optimized (TP) futexes that can be used to implement userspace mutexes. This patchset includes a place holder (4.X) for the exact version that the new TP futexes code is merged. That wil

[PATCH v3 03/13] futex: Add helpers to get & cmpxchg futex value without lock

2016-09-30 Thread Waiman Long
Two new helper functions cmpxchg_futex_value() and get_futex_value() are added to access and change the futex value without the hash bucket lock. As a result, page fault is enabled and the page will be faulted in if not present yet. Signed-off-by: Waiman Long --- kernel/futex.c | 15 +

[PATCH v3 13/13] perf bench: New microbenchmark for userspace mutex performance

2016-09-30 Thread Waiman Long
This microbenchmark simulates how the use of different futex types can affect the actual performanace of userspace mutex locks. The usage is: perf bench futex mutex Three sets of simple mutex lock and unlock functions are implemented using the wait-wake, PI and TP futexes respectively. T

[PATCH v3 05/13] futex: Add a new futex type field into futex_state

2016-09-30 Thread Waiman Long
As the futex_state structure will be overloaded in later patches to be used by non-PI futexes, it is necessary to add a type field to distinguish among different types of futexes. Signed-off-by: Waiman Long --- kernel/futex.c | 14 ++ 1 files changed, 10 insertions(+), 4 deletions(

[PATCH v3 12/13] futex, doc: TP futexes document

2016-09-30 Thread Waiman Long
This patch adds a new document file on how to use the TP futexes. Signed-off-by: Waiman Long --- Documentation/00-INDEX |2 + Documentation/tp-futex.txt | 147 2 files changed, 149 insertions(+), 0 deletions(-) create mode 100644 Documentati

[PATCH v3 08/13] futex: Enable robust handling of TP futexes

2016-09-30 Thread Waiman Long
The TP futexes don't have code to handle the death of futex owners. There are 2 different cases that need to be considered. As top waiter gets a reference to the task structure of the futex owner, the task structure will never go away even if the owner dies. When the futex owner died while the top

[PATCH v3 10/13] futex: Inform FUTEX_LOCK callers on how the lock is acquired

2016-09-30 Thread Waiman Long
As there are three different ways for a TP futex waiter in the kernel to acquire the lock. It may be useful to pass this information out so that user space has a better view of what is happening in the kernel. With this change, different non-negative values will now be returned depending on how the

[PATCH v3 04/13] futex: Consolidate pure pi_state_list add & delete codes to helpers

2016-09-30 Thread Waiman Long
Two new helper functions (task_pi_list_add & task_pi_list_del) are created to consolidate all the pure pi_state_list addition and insertion codes. The set_owner argument in task_pi_list_add() will be needed in a later patch. Signed-off-by: Waiman Long --- kernel/futex.c | 67 ++

[PATCH v3 11/13] futex: Add timeout support to TP futexes

2016-09-30 Thread Waiman Long
Unlike other futexes, TP futexes do not accept the specification of a timeout value. To align with the other futexes, timeout support is now added. However, the timeout isn't as precise as the other futex types due to the fact that timer expiration can't be detected when the thread is waiting in t

[PATCH v3 07/13] futex: Throughput-optimized (TP) futexes

2016-09-30 Thread Waiman Long
A new futex implementation called throughput-optimized (TP) futexes is introduced. The goal of this new futex type is to maximize locking throughput at the expense of fairness and deterministic latency. Its throughput is higher than that of the wait-wake futexes especially on systems with a large n

[PATCH v3 06/13] futex: Allow direct attachment of futex_state objects to hash bucket

2016-09-30 Thread Waiman Long
Currently, the futex state objects can only be located indirectly as hash bucket => futex_q => futex state Actually it can be beneficial in some cases to locate the futex state object directly from the hash bucket without the futex_q middleman. Therefore, a new list head to link the futex

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-30 Thread Sinan Kaya
On 9/30/2016 5:27 PM, Rafael J. Wysocki wrote: > On Fri, Sep 30, 2016 at 11:14 PM, Sinan Kaya wrote: >> On 9/30/2016 5:04 PM, Rafael J. Wysocki wrote: > > I hope it makes sense now. I tend to skip details sometimes. Feel free to > send more questions. >>> Thanks for the information! >>

[PATCH net-next 0/7] rxrpc: More fixes and adjustments

2016-09-30 Thread David Howells
s.git/log/?h=rxrpc-rewrite Tagged thusly: git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git rxrpc-rewrite-20160930 David --- David Howells (7): rxrpc: Actually display the tx_data trace retransmission note rxrpc: Switch to Congestion Avoidance mode at

<    1   2   3   4   5   >