aio fixes for use after free and freeze protection

2016-10-30 Thread Christoph Hellwig
Hi Linus, hi Al, below is the new version of the aio fix(es). Patch one just holds an additional file reference over AIO ops. This one is minimally invasive and a clear 4.9 and stable candidate. The next one drops the never implemented aio_fsync methods because it makes my life easier later on.

[PATCH 4/4] aio: fix freeze protection of aio writes

2016-10-30 Thread Christoph Hellwig
Currently we dropped freeze protection of aio writes just after IO was submitted. Thus aio write could be in flight while the filesystem was frozen and that could result in unexpected situation like aio completion wanting to convert extent type on frozen filesystem. Testcase from Dmitry triggering

[PATCH 3/4] fs: remove aio_run_iocb

2016-10-30 Thread Christoph Hellwig
Pass the ABI iocb structure to aio_setup_rw and let it handle the non-vectored I/O case as well. With that and a new helper for the AIO return value handling we can now define new aio_read and aio_write helpers that implement reads and writes in a self-contained way without duplicating too much co

[PATCH 2/4] fs: remove the never implemented aio_fsync file operation

2016-10-30 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- Documentation/filesystems/Locking | 1 - Documentation/filesystems/vfs.txt | 1 - fs/aio.c | 14 -- fs/ntfs/dir.c | 2 -- include/linux/fs.h| 1 - 5 files changed, 19 deletions(-) di

Re: [PATCH v5 4/8] drivers:input:tsc2007: add iio interface to read external ADC input and temperature

2016-10-30 Thread Jonathan Cameron
On 25/10/16 20:26, H. Nikolaus Schaller wrote: > The tsc2007 chip not only has a resistive touch screen controller but > also an external AUX adc imput which can be used for an ambient > light sensor, battery voltage monitoring or any general purpose. > > Additionally it can measure the chip tempe

[PATCH RT 2/6] ftrace: Fix trace header alignment

2016-10-30 Thread Steven Rostedt
4.4.27-rt38-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Mike Galbraith Line up helper arrows to the right column. Cc: stable...@vger.kernel.org Signed-off-by: Mike Galbraith Signed-off-by: Steven Rostedt [bigeasy: fixup function tracer

[PATCH RT 6/6] Linux 4.4.27-rt38-rc1

2016-10-30 Thread Steven Rostedt
4.4.27-rt38-rc1 stable review patch. If anyone has any objections, please let me know. -- From: "Steven Rostedt (Red Hat)" --- localversion-rt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index a3b2408c1da6..625367387621

[PATCH RT 4/6] x86/apic: get rid of "warning: acpi_ioapic_lock defined but not used"

2016-10-30 Thread Steven Rostedt
4.4.27-rt38-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior kbuild test robot reported this against the -RT tree: | In file included from include/linux/mutex.h:30:0, |from include/linux/notifier

[PATCH RT 5/5] Linux 4.1.35-rt41-rc1

2016-10-30 Thread Steven Rostedt
4.1.35-rt41-rc1 stable review patch. If anyone has any objections, please let me know. -- From: "Steven Rostedt (Red Hat)" --- localversion-rt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index 2af6c89aee6d..ad263cec032a

Re: [PATCH] ppdev: fix double-free of pp->pdev->name

2016-10-30 Thread Jann Horn
On Sun, Oct 30, 2016 at 09:29:10AM -0700, Andy Lutomirski wrote: > On Sun, Oct 30, 2016 at 8:44 AM, Jann Horn wrote: > > free_pardevice() is called by parport_unregister_device() and already frees > > pp->pdev->name, don't try to do it again. > > > > This bug causes kernel crashes. > > > > I found

[PATCH RT 2/5] zsmalloc: turn that get_cpu_light() into a local_lock()

2016-10-30 Thread Steven Rostedt
4.1.35-rt41-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior We might get preempted, grab the same ressource again and then corrupt the memory. Cc: stable...@vger.kernel.org Signed-off-by: Sebastian Andrzej Siewior S

[PATCH RT 5/6] kbuild: add -fno-PIE

2016-10-30 Thread Steven Rostedt
4.4.27-rt38-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior Debian started to build the gcc with -fPIE by default so the kernel build ends before it starts properly with: |kernel/bounds.c:1:0: error: code model kernel

[PATCH RT 0/5] Linux 4.1.35-rt41-rc1

2016-10-30 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 4.1.35-rt41-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release cand

[PATCH RT 0/6] Linux 4.4.27-rt38-rc1

2016-10-30 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 4.4.27-rt38-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release cand

[PATCH RT 3/5] x86/apic: get rid of "warning: acpi_ioapic_lock defined but not used"

2016-10-30 Thread Steven Rostedt
4.1.35-rt41-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior kbuild test robot reported this against the -RT tree: | In file included from include/linux/mutex.h:30:0, |from include/linux/notifier

[PATCH RT 1/6] kernel/futex: dont deboost too early

2016-10-30 Thread Steven Rostedt
4.4.27-rt38-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior The sequence: T1 holds futex T2 blocks on futex and boosts T1 T1 unlocks futex and holds hb->lock T1 unlocks rt mutex, so T1 has no more pi waiters T3 b

[PATCH RT 3/6] zsmalloc: turn that get_cpu_light() into a local_lock()

2016-10-30 Thread Steven Rostedt
4.4.27-rt38-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior We might get preempted, grab the same ressource again and then corrupt the memory. Cc: stable...@vger.kernel.org Signed-off-by: Sebastian Andrzej Siewior S

[PATCH RT 1/5] ftrace: Fix trace header alignment

2016-10-30 Thread Steven Rostedt
4.1.35-rt41-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Mike Galbraith Line up helper arrows to the right column. Cc: stable...@vger.kernel.org Signed-off-by: Mike Galbraith Signed-off-by: Steven Rostedt [bigeasy: fixup function tracer

[PATCH RT 4/5] kbuild: add -fno-PIE

2016-10-30 Thread Steven Rostedt
4.1.35-rt41-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior Debian started to build the gcc with -fPIE by default so the kernel build ends before it starts properly with: |kernel/bounds.c:1:0: error: code model kernel

[PATCH RT 2/3] kbuild: add -fno-PIE

2016-10-30 Thread Steven Rostedt
3.12.66-rt89-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior Debian started to build the gcc with -fPIE by default so the kernel build ends before it starts properly with: |kernel/bounds.c:1:0: error: code model kerne

[PATCH RT 3/3] Linux 3.12.66-rt89-rc1

2016-10-30 Thread Steven Rostedt
3.12.66-rt89-rc1 stable review patch. If anyone has any objections, please let me know. -- From: "Steven Rostedt (Red Hat)" --- localversion-rt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index 666227d921a3..d1173dc87024

[PATCH RT 3/3] Linux 3.10.104-rt117-rc1

2016-10-30 Thread Steven Rostedt
3.10.104-rt117-rc1 stable review patch. If anyone has any objections, please let me know. -- From: "Steven Rostedt (Red Hat)" --- localversion-rt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index 34eca4e89203..d5f654342a

[PATCH RT 2/3] kbuild: add -fno-PIE

2016-10-30 Thread Steven Rostedt
3.10.104-rt117-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Sebastian Andrzej Siewior Debian started to build the gcc with -fPIE by default so the kernel build ends before it starts properly with: |kernel/bounds.c:1:0: error: code model ker

[PATCH RT 0/3] Linux 3.10.104-rt117-rc1

2016-10-30 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 3.10.104-rt117-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release c

[PATCH RT 0/3] Linux 3.12.66-rt89-rc1

2016-10-30 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 3.12.66-rt89-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release can

[PATCH RT 1/3] ftrace: Fix trace header alignment

2016-10-30 Thread Steven Rostedt
3.10.104-rt117-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Mike Galbraith Line up helper arrows to the right column. Cc: stable...@vger.kernel.org Signed-off-by: Mike Galbraith Signed-off-by: Steven Rostedt [bigeasy: fixup function trac

[PATCH RT 1/3] ftrace: Fix trace header alignment

2016-10-30 Thread Steven Rostedt
3.12.66-rt89-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Mike Galbraith Line up helper arrows to the right column. Cc: stable...@vger.kernel.org Signed-off-by: Mike Galbraith Signed-off-by: Steven Rostedt [bigeasy: fixup function tracer

[PATCH RT 1/3] genirq: Fix pointer cast warning for randomness on 32-bit architectures

2016-10-30 Thread Steven Rostedt
3.2.83-rt121-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Ben Hutchings gcc warns about casting directly from a 32-bit pointer to u64. In the current rt patch series we cast to unsigned long to avoid the warning, but this fix hasn't been b

[PATCH RT 0/3] Linux 3.2.83-rt121-rc1

2016-10-30 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 3.2.83-rt121-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release can

[PATCH RT 3/3] Linux 3.2.83-rt121-rc1

2016-10-30 Thread Steven Rostedt
3.2.83-rt121-rc1 stable review patch. If anyone has any objections, please let me know. -- From: "Steven Rostedt (Red Hat)" --- localversion-rt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index fff72aad59ab..6096ce5ed673

[PATCH RT 2/3] work-simple: Add missing #include

2016-10-30 Thread Steven Rostedt
3.2.83-rt121-rc1 stable review patch. If anyone has any objections, please let me know. -- From: Ben Hutchings work-simple.c uses EXPORT_SYMBOL_GPL(), defined in . Link: http://lkml.kernel.org/r/20160930090504.gx10...@decadent.org.uk Signed-off-by: Ben Hutchings Signed-off-by

Re: [PATCH 1/1] nvdimm: use the right length of "pmem"

2016-10-30 Thread Nicolas Iooss
On 30/10/16 16:50, Dan Williams wrote: > On Sat, Oct 29, 2016 at 4:28 AM, Nicolas Iooss > wrote: >> In order to test that the name of a resource begins with "pmem", call >> strncmp() with 4 as length instead of 3 to match the whole prefix. >> >> Fixes: 16660eaea0cc ("libnvdimm, namespace: update l

Re: aio fixes for use after free and freeze protection

2016-10-30 Thread Al Viro
On Sun, Oct 30, 2016 at 11:42:00AM -0500, Christoph Hellwig wrote: > Hi Linus, hi Al, > > below is the new version of the aio fix(es). > > Patch one just holds an additional file reference over AIO ops. This > one is minimally invasive and a clear 4.9 and stable candidate. > > The next one drop

Re: [PATCH v6 2/2] iio: adc: add support for Allwinner SoCs ADC

2016-10-30 Thread Jonathan Cameron
On 15/09/16 13:44, Quentin Schulz wrote: > The Allwinner SoCs all have an ADC that can also act as a touchscreen > controller and a thermal sensor. This patch adds the ADC driver which is > based on the MFD for the same SoCs ADC. > > This also registers the thermal adc channel in the iio map array

Re: aio fixes for use after free and freeze protection

2016-10-30 Thread Christoph Hellwig
Btw, patch 4 should be attributed to Jan - for some reason git keeps resetting the attribution whenever there is a conflict during a rebase.

Re: [PATCH 1/4] mfd: ti_am335x_tscadc: store physical address

2016-10-30 Thread Jonathan Cameron
On 26/10/16 13:17, Lee Jones wrote: > On Fri, 30 Sep 2016, Mugunthan V N wrote: > >> On Wednesday 28 September 2016 01:10 AM, Lee Jones wrote: >>> On Wed, 21 Sep 2016, Mugunthan V N wrote: >>> store the physical address of the device in its priv to use it for DMA addressing in the client

Re: [PATCH v6 4/8] drivers:input:tsc2007: add iio interface to read external ADC input and temperature

2016-10-30 Thread Jonathan Cameron
On 27/10/16 09:44, H. Nikolaus Schaller wrote: > The tsc2007 chip not only has a resistive touch screen controller but > also an external AUX adc imput which can be used for an ambient > light sensor, battery voltage monitoring or any general purpose. > > Additionally it can measure the chip tempe

Re: [PATCH] firmware: fix async/manual firmware loading

2016-10-30 Thread Yves-Alexis Perez
On Sun, 2016-10-30 at 15:50 +0100, Yves-Alexis Perez wrote: > wait_for_completion_interruptible_timeout() return value is either > -ERESTARTSYS (in case it was interrupted), 0 (in case the timeout expired) > or the number of jiffies left until timeout. The return value is stored in > a long, but in

Re: [PATCH v3] staging: iio: cdc: ad7746: add additional config defines

2016-10-30 Thread Jonathan Cameron
On 28/10/16 09:26, Eva Rachel Retuya wrote: > Introduce defines for shifting and mask under the config register for > better readability. Also, introduce helper variables for index > calculation. > > Signed-off-by: Eva Rachel Retuya Looks good to me. Lars could you sanity check this one as well?

Re: [PATCH 01/10] staging: iio: tsl2583: add of_match table for device tree support

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > Add device tree support for the tsl2583 IIO driver with no custom > properties. > > Signed-off-by: Brian Masney Trivial enough that I feel I can take this without an explicit devicetree ack. Of course one is always welcome if anyone gets a chance to look

Re: [PATCH 01/10] staging: iio: tsl2583: add of_match table for device tree support

2016-10-30 Thread Jonathan Cameron
On 30/10/16 17:43, Jonathan Cameron wrote: > On 28/10/16 11:00, Brian Masney wrote: >> Add device tree support for the tsl2583 IIO driver with no custom >> properties. >> >> Signed-off-by: Brian Masney > Trivial enough that I feel I can take this without an explicit devicetree > ack. Of course on

Re: [PATCH v3] staging: iio: cdc: ad7746: add additional config defines

2016-10-30 Thread Lars-Peter Clausen
On 10/30/2016 06:41 PM, Jonathan Cameron wrote: > On 28/10/16 09:26, Eva Rachel Retuya wrote: >> Introduce defines for shifting and mask under the config register for >> better readability. Also, introduce helper variables for index >> calculation. >> >> Signed-off-by: Eva Rachel Retuya > Looks go

Re: [PATCH 00/14] introduce the BFQ-v0 I/O scheduler as an extra scheduler

2016-10-30 Thread Manuel Krause
Dear blk-mq maintainers, Since years now I use the BFQ disk IO scheduler by default, always fetching the newest release. Now a reality story of mine: For a clean BUG hunt, I was forced to leave out BFQ for a week recently. Result was an unusable experience with CFQ. Long time pauses of deskt

Re: [PATCH 02/10] staging: iio: tsl2583: check for error code from i2c_smbus_read_byte()

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > taos_i2c_read() and taos_als_calibrate() does not check to see if the > value returned by i2c_smbus_read_byte() was an error code. This patch > adds the appropriate error handling. > > Signed-off-by: Brian Masney Applied to the togreg branch of iio.git and

Re: [PATCH 03/10] staging: iio: tsl2583: return proper error code instead of -1

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > taos_als_calibrate() has a code path where -1 is returned. This patch > changes the code so that a proper error code is returned. > > Signed-off-by: Brian Masney Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play

Re: [PATCH 04/10] staging: iio: tsl2583: remove redundant power_state sysfs attribute

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > IIO devices have a /sys/bus/iio/devices/iio:deviceX/power/ directory > that allows viewing and controling various power parameters. The tsl2583 > driver also has an additional custom sysfs attribute named power_state > that is not needed. This patch removes

Re: [PATCH] ppdev: fix double-free of pp->pdev->name

2016-10-30 Thread Sudip Mukherjee
On Sunday 30 October 2016 09:14 PM, Jann Horn wrote: free_pardevice() is called by parport_unregister_device() and already frees pp->pdev->name, don't try to do it again. This bug causes kernel crashes. I found and verified this with KASAN and some added pr_emerg()s: [ 60.316568] pp_release:

Re: [PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off}

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > The return value from taos_chip_on() and taos_chip_off() was not > checked in taos_luxtable_store() and taos_probe(). This patch adds > proper error checking to these function calls. > > Signed-off-by: Brian Masney This does raise the question of whether w

Re: [PATCH v2 02/16] scsi: don't use fc_bsg_job::request and fc_bsg_job::reply directly

2016-10-30 Thread Johannes Thumshirn
On Fri, Oct 28, 2016 at 11:53:46AM +0200, Steffen Maier wrote: [...] > > > > > > > @@ -3937,6 +3944,7 @@ fc_bsg_request_handler(struct request_queue *q, > > > > struct Scsi_Host *shost, > > > > struct request *req; > > > > struct fc_bsg_job *job; > > > > enum fc_dispatch

Re: [PATCH v3 6/8] iio: dpot-dac: DAC driver based on a digital potentiometer

2016-10-30 Thread Peter Rosin
On 2016-10-30 14:33, Jonathan Cameron wrote: > On 23/10/16 23:39, Peter Rosin wrote: >> It is assumed that the dpot is used as a voltage divider between the >> current dpot wiper setting and the maximum resistance of the dpot. The >> divided voltage is provided by a vref regulator. >> >>

Re: [PATCH] ppdev: fix double-free of pp->pdev->name

2016-10-30 Thread Jann Horn
On Sun, Oct 30, 2016 at 11:24:27PM +0530, Sudip Mukherjee wrote: > On Sunday 30 October 2016 09:14 PM, Jann Horn wrote: > >free_pardevice() is called by parport_unregister_device() and already frees > >pp->pdev->name, don't try to do it again. > > > >This bug causes kernel crashes. > > > >I found a

[linus:master] BUILD REGRESSION 2a26d99b251b8625d27aed14e97fc10707a3a81f

2016-10-30 Thread kbuild test robot
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master 2a26d99b251b8625d27aed14e97fc10707a3a81f Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net arch/ia64/kernel/entry.S:622: Error: Operand 2 of `adds' should be a 14-bit integer (-8192-8191) arch/ia64/kernel/fsy

Re: [PATCH 06/10] staging: iio: tsl2583: convert to use iio_chan_spec and {read,write}_raw

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > The tsl2583 driver directly creates sysfs attributes that should instead > be created by the IIO core on behalf of the driver. This patch adds the > iio_chan_spec array, the relevant info_mask elements and the read_raw() > and write_raw() functions to take a

[RFC] fpga: Pull checks for supported operations into framework

2016-10-30 Thread Moritz Fischer
Most of the drivers only support a subset of {PARTIAL, FULL} reconfiguration. Pull duplicate checks in each driver into the framework. Signed-off-by: Moritz Fischer Cc: Alan Tull Cc: Michal Simek Cc: Sören Brinkmann Cc: linux-kernel@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org ---

Re: mmotm 2016-10-27-18-27 uploaded

2016-10-30 Thread Paul Gortmaker
On Thu, Oct 27, 2016 at 9:28 PM, wrote: > The mm-of-the-moment snapshot 2016-10-27-18-27 has been uploaded to > >http://www.ozlabs.org/~akpm/mmotm/ Just a heads up: Somehow one of the akpm commits as it appears in linux-next has had spaces replaced with garbage chars: https://git.kernel.or

Re: [PATCH] m32r: add simple dma

2016-10-30 Thread Sudip Mukherjee
Hi Andrew, On Friday 21 October 2016 08:59 AM, Andrew Morton wrote: On Sat, 8 Oct 2016 23:23:18 +0530 Sudip Mukherjee wrote: Some builds of m32r were failing as it tried to build few drivers which needed dma but m32r is not having dma support. Objections were raised when it was tried to mak

Re: [4.1 backport trouble] Re: BUGreport: fix minor infoleak in get_user_ex()

2016-10-30 Thread Levin, Alexander
On Thu, Oct 27, 2016 at 10:02:10PM -0400, Al Viro wrote: > ... and frankly, backporting 548acf19234d would be my preference. It's a bit > more intrusive than needed (_ASM_EXTABLE_FAULT is used only in > memcpy_mcsafe(), > which is used only by pmem and it's the only reason for passing the trap >

Re: [PATCH] iommu/vt-d: Fix IOMMU lookup for VF's

2016-10-30 Thread David Woodhouse
On Fri, 2016-10-21 at 15:32 -0700, Ashok Raj wrote: > IOMMU driver must pick the same IOMMU as that of a Physical Function (PF) for > any of its Virtual Functions (VF). It is not practical to list all the VF's > in the DMAR scope, as this list could be quite large. Linux also ignores any > VF's lis

Re: [PATCH] iommu/vt-d: Fix the size calculation of pasid table

2016-10-30 Thread David Woodhouse
On Wed, 2016-10-12 at 13:17 +0100, David Woodhouse wrote: > Yes, that looks correct. I think we may also need to limit it, because > full 20-bit PASID support means we'll attempt an order 11 allocation. > But that's certainly correct for now Actually, not quite correct. You fixed the allocation bu

Re: [PATCH 07/10] staging: iio: tsl2583: convert illuminance0_calibscale sysfs attr to use iio_chan_spec

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > The illuminance0_calibscale sysfs attribute is not currently created by > the IIO core. This patch adds the appropriate mask to iio_chan_spec, > along with the appropriate data handling in the read_raw() and > write_raw() functions, so that the sysfs attribu

Re: [PATCH 08/10] staging: iio: tsl2583: use IIO_*_ATTR* macros to create sysfs entries

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > Use the IIO_CONST_ATTR, IIO_DEVICE_ATTR_RW, and IIO_DEVICE_ATTR_WO > macros for creating the in_illuminance_calibscale_available, > in_illuminance_integration_time_available, in_illuminance_input_target, > in_illuminance_calibrate, and in_illuminance_lux_tab

Re: [PATCH 09/10] staging: iio: tsl2583: add error code to sysfs store functions

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > in_illuminance_input_target_store() and in_illuminance_calibrate_store() > validated the data from userspace, however it would not return an > error code to userspace if an invalid value was passed in. This patch > changes these functions so that they return

Re: [PATCH 10/10] staging: iio: tsl2583: add locking to sysfs attributes

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > in_illuminance_input_target_show(), in_illuminance_input_target_store(), > in_illuminance_calibrate_store(), and in_illuminance_lux_table_store() > accesses data from the tsl2583_chip struct. Some of these fields can be > modified by other parts of the drive

Re: [PATCH v3] staging: iio: cdc: ad7746: add additional config defines

2016-10-30 Thread Jonathan Cameron
On 30/10/16 17:46, Lars-Peter Clausen wrote: > On 10/30/2016 06:41 PM, Jonathan Cameron wrote: >> On 28/10/16 09:26, Eva Rachel Retuya wrote: >>> Introduce defines for shifting and mask under the config register for >>> better readability. Also, introduce helper variables for index >>> calculation.

Re: [PATCH v3 6/8] iio: dpot-dac: DAC driver based on a digital potentiometer

2016-10-30 Thread Peter Rosin
On 2016-10-30 16:32, Peter Meerwald-Stadler wrote: > On Sun, 30 Oct 2016, Jonathan Cameron wrote: > >> On 23/10/16 23:39, Peter Rosin wrote: >>> It is assumed that the dpot is used as a voltage divider between the >>> current dpot wiper setting and the maximum resistance of the dpot. The >>> divid

Re: [PATCH v4 2/2] fpga: Add support for Lattice iCE40 FPGAs

2016-10-30 Thread Geert Uytterhoeven
On Sat, Oct 29, 2016 at 11:32 PM, Joel Holdsworth wrote: > --- /dev/null > +++ b/Documentation/devicetree/bindings/fpga/lattice-ice40-fpga-mgr.txt > @@ -0,0 +1,23 @@ > +- cdone-gpio: GPIO connected to CDONE pin > +- creset_b-gpio: GPIO connected to CRESET_B pin. Note that CRESET_B

core.c:undefined reference to `fpu_save'

2016-10-30 Thread kbuild test robot
Hi Andrew, It's probably a bug fix that unveils the link errors. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 2a26d99b251b8625d27aed14e97fc10707a3a81f commit: c60f169202c7643991a8b4bfeea60e06843d5b5a arch/mn10300/kernel/fpu-nofpu.c: needs asm/elf.h d

[PATCH] cxl: Fix memory allocation failure test

2016-10-30 Thread Christophe JAILLET
'cxl_context_alloc()' does not return an error pointer. It is just a shortcut for a call to 'kzalloc' with 'sizeof(struct cxl_context)' as the size parameter. So its return value should be compared with NULL. While fixing it, simplify a bit the code. Signed-off-by: Christophe JAILLET --- un-comp

Re: [PATCH 07/10] staging: iio: tsl2583: convert illuminance0_calibscale sysfs attr to use iio_chan_spec

2016-10-30 Thread Brian Masney
On Sun, Oct 30, 2016 at 06:37:40PM +, Jonathan Cameron wrote: > On 28/10/16 11:00, Brian Masney wrote: > > The illuminance0_calibscale sysfs attribute is not currently created by > > the IIO core. This patch adds the appropriate mask to iio_chan_spec, > > along with the appropriate data handlin

Re: [PATCH] cxl: Fix memory allocation failure test

2016-10-30 Thread walter harms
Am 30.10.2016 20:35, schrieb Christophe JAILLET: > 'cxl_context_alloc()' does not return an error pointer. It is just a > shortcut for a call to 'kzalloc' with 'sizeof(struct cxl_context)' as the > size parameter. > > So its return value should be compared with NULL. > While fixing it, simplify

Re: [linus:master] BUILD REGRESSION 2a26d99b251b8625d27aed14e97fc10707a3a81f

2016-10-30 Thread Linus Torvalds
On Sun, Oct 30, 2016 at 11:03 AM, kbuild test robot wrote: > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > 2a26d99b251b8625d27aed14e97fc10707a3a81f Merge > git://git.kernel.org/pub/scm/linux/kernel/git/davem/net Hmm. The build errors seem to be pretty independent

Re: [PATCH 07/10] staging: iio: tsl2583: convert illuminance0_calibscale sysfs attr to use iio_chan_spec

2016-10-30 Thread Jonathan Cameron
On 30 October 2016 20:04:09 GMT+00:00, Brian Masney wrote: >On Sun, Oct 30, 2016 at 06:37:40PM +, Jonathan Cameron wrote: >> On 28/10/16 11:00, Brian Masney wrote: >> > The illuminance0_calibscale sysfs attribute is not currently >created by >> > the IIO core. This patch adds the appropriat

[PATCH 5/9] s390: char: make slcp_quiesce explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Makefile currently controlling compilation of this code is obj-y, meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular infrastructure, so that when reading the driver there is no doubt it is builtin-only. Since module_init translates to

[PATCH 1/9] s390: cio: make it explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Makefile currently controlling compilation of this code is: obj-y += airq.o blacklist.o chsc.o cio.o css.o chp.o idset.o isc.o \ fcx.o itcw.o crw.o ccwreq.o trace.o ioasm.o ccw_device-objs += device.o device_fsm.o device_ops.o ccw_device-objs += device_id.o device_pgid.o device

[PATCH 3/9] s390: char: make con3215 explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: drivers/s390/char/Kconfig:config TN3215 drivers/s390/char/Kconfig: def_bool y ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is essentially orphaned, so that when reading

[PATCH 8/9] s390: kernel: make lgr explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Makefile currently controlling compilation of this code is obj-y meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular infrastructure use, so that when reading the driver there is no doubt it is builtin-only. Since module_init translates

[PATCH 9/9] s390: virtio: make ccw explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: arch/s390/Kconfig:config S390_GUEST arch/s390/Kconfig: def_bool y ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is essentially orphaned, so that when reading the driver

[PATCH 4/9] s390: char: make sclp_tty explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: drivers/s390/char/Kconfig:config SCLP_TTY drivers/s390/char/Kconfig: def_bool y ...meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular infrastructure use, so that when rea

Re: [PATCH] net: stmmac: Add OXNAS Glue Driver

2016-10-30 Thread Rob Herring
On Fri, Oct 21, 2016 at 10:44:45AM +0200, Neil Armstrong wrote: > Add Synopsys Designware MAC Glue layer for the Oxford Semiconductor OX820. > > Signed-off-by: Neil Armstrong > --- > .../devicetree/bindings/net/oxnas-dwmac.txt| 44 + It's preferred that bindings are a separate patch

Re: linux-next: build warning in Linus' tree

2016-10-30 Thread Andrey Ryabinin
On 10/30/2016 12:29 AM, Geert Uytterhoeven wrote: > Hi Linus, > > On Fri, Oct 28, 2016 at 1:01 AM, Linus Torvalds > wrote: >> On Thu, Oct 27, 2016 at 3:48 PM, Linus Torvalds >> wrote: >>> >>> I wonder if we should make KASAN depend on !COMPILE_TEST, because it >>> does seem to disable a lot of

[PATCH 0/9] s390: remove modular usage from non-modular code

2016-10-30 Thread Paul Gortmaker
My ongoing audit looking for non-modular code that needlessly uses modular macros (vs. built-in equivalents) and/or has dead code relating to module unloading that can never be executed led to the creation of these s390 related commits. For anyone new to the underlying goal of this cleanup, we are

[PATCH 2/9] s390: char: make zcore explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: arch/s390/Kconfig:config CRASH_DUMP arch/s390/Kconfig: bool "kernel crash dumps" ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is essentially orphaned, so that when read

[PATCH 6/9] s390: hotplug: make pci_hpc explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: drivers/pci/hotplug/Kconfig:config HOTPLUG_PCI_S390 drivers/pci/hotplug/Kconfig:bool "System z PCI Hotplug Support" ...meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular i

[PATCH 7/9] s390: hypfs: make inode explicitly non-modular

2016-10-30 Thread Paul Gortmaker
The Kconfig currently controlling compilation of this code is: arch/s390/Kconfig:config S390_HYPFS_FS arch/s390/Kconfig: def_bool y ...meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modular infrastructure use, so that when reading the dr

Re: [PATCH v3 7/8] dt-bindings: iio: document envelope-detector bindings

2016-10-30 Thread Rob Herring
On Mon, Oct 24, 2016 at 12:39:40AM +0200, Peter Rosin wrote: > Signed-off-by: Peter Rosin > --- > .../bindings/iio/adc/envelope-detector.txt | 54 > ++ > MAINTAINERS| 6 +++ > 2 files changed, 60 insertions(+) > create mode 10

Re: [PATCH 5/5] irqchip: st: Remove obsolete platforms from dt binding doc

2016-10-30 Thread Rob Herring
On Fri, Oct 21, 2016 at 05:57:23PM +0200, patrice.chot...@st.com wrote: > From: Patrice Chotard > > STiH415/6 SoC support is being removed from the kernel. > This patch updates the sti irchip and removes > references to these obsolete platforms. > > Signed-off-by: Patrice Chotard > Cc: > Cc:

Re: [PATCH v3 4/8] dt-bindings: add axentia to vendor-prefixes

2016-10-30 Thread Rob Herring
On Mon, Oct 24, 2016 at 12:39:37AM +0200, Peter Rosin wrote: > Signed-off-by: Peter Rosin > --- > Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + > 1 file changed, 1 insertion(+) Acked-by: Rob Herring

Re: [PATCH v3 5/8] dt-bindings: iio: document dpot-dac bindings

2016-10-30 Thread Rob Herring
On Mon, Oct 24, 2016 at 12:39:38AM +0200, Peter Rosin wrote: > Signed-off-by: Peter Rosin > --- > .../devicetree/bindings/iio/dac/dpot-dac.txt | 41 > ++ > MAINTAINERS| 6 > 2 files changed, 47 insertions(+) > create mode 1

Re: [PATCH v4] media: et8ek8: add device tree binding documentation

2016-10-30 Thread Rob Herring
On Sun, Oct 23, 2016 at 09:17:06PM +0200, Pavel Machek wrote: > > Add device tree binding documentation for toshiba et8ek8 sensor. > > Signed-off-by: Ivaylo Dimitrov > Signed-off-by: Pavel Machek > > --- > > diff from v3: explain what clock-frequency means > > > diff --git a/Documentation/d

Re: [PATCH/RFT v2 09/17] regulator: fixed: Add over current event

2016-10-30 Thread Rob Herring
On Mon, Oct 24, 2016 at 06:46:26PM +0200, ahas...@baylibre.com wrote: > From: Axel Haslam > > Some regulator supplies have an over-current pin that is > activated when the hw detects an over current condition. > When this happens, the hardware enters a current limited > mode. > > Extend the fixe

Re: [PATCH] dmaengine: nbpfaxi: add optional max-burst property for memory reads/writes

2016-10-30 Thread Rob Herring
On Mon, Oct 24, 2016 at 03:29:15PM +0200, Niklas Cassel wrote: > From: Niklas Cassel > > Due to a hardware bug, reading memory (from the Accelerator Coherency Port) > with a burst size equal to the maximum burst size allowed by the DMA > hardware's buffer size will cause a hardware hang on the AR

Re: [PATCH v2 3/3] clocksource: Add clockevent support to NPS400 driver

2016-10-30 Thread Rob Herring
On Sun, Oct 23, 2016 at 03:12:28PM +0300, Noam Camus wrote: > From: Noam Camus > > Till now we used clockevent from generic ARC driver. > This was enough as long as we worked with simple multicore SoC. > When we are working with multithread SoC each HW thread can be > scheduled to receive timer i

Re: [PATCH] dt-bindings: video: exynos7-decon: Remove obsolete samsung,power-domain property

2016-10-30 Thread Rob Herring
On Fri, Oct 21, 2016 at 05:05:54PM +0300, Krzysztof Kozlowski wrote: > The samsung,power-domain property is obsolete since commit 0da658704136 > ("ARM: dts: convert to generic power domain bindings for exynos DT"). > Replace it with generic one. > > Signed-off-by: Krzysztof Kozlowski > --- > Doc

Re: [PATCH v4 01/23] reset: Add renesas,rst DT bindings

2016-10-30 Thread Rob Herring
On Fri, Oct 21, 2016 at 03:17:15PM +0200, Geert Uytterhoeven wrote: > Add DT bindings for the Renesas R-Car Reset Controller (R-Car Gen1 > RESET/WDT and R-Car Gen2/Gen3 and RZ/G RST). > > As the features provided by the hardware module differ a lot across the > various SoC families and members, on

Re: [RFC PATCH 01/13] pinctrl: meson: Add GXL pinctrl definitions

2016-10-30 Thread Rob Herring
On Fri, Oct 21, 2016 at 04:40:26PM +0200, Neil Armstrong wrote: > Add support for the Amlogic Meson GXL SoC, this is a partially complete > definition only based on the Amlogic Vendor tree. > > This definition differs a lot from the GXBB and needs a separate entry. > > Signed-off-by: Neil Armstro

Re: [PATCH v7 REPOST 8/9] arm: add sysfs cpu_capacity attribute

2016-10-30 Thread Russell King - ARM Linux
On Mon, Oct 17, 2016 at 04:46:49PM +0100, Juri Lelli wrote: > +#ifdef CONFIG_PROC_SYSCTL > +#include > +#include Include files at the top of the file please. No need to ifdef them. They're sorted alphabetically, so new additions should be alphabetical. (That's a general rule - if something is a

Re: [PATCH] IBcore/CM: Issue DREQ when receiving REQ/REP for stale QP

2016-10-30 Thread Sagi Grimberg
from "InfiBand Architecture Specifications Volume 1": A QP is said to have a stale connection when only one side has connection information. A stale connection may result if the remote CM had dropped the connection and sent a DREQ but the DREQ was never received by the local CM. Alternati

Re: [PATCH v4 05/10] IB/isert: Replace semaphore sem with completion

2016-10-30 Thread Sagi Grimberg
The semaphore 'sem' in isert_device is used as completion, so convert it to struct completion. Semaphores are going away in the future. Umm, this is 100% *not* true. np->sem is designed as a counting to sync the iscsi login thread with the connect requests coming from the initiators. So this is

Re: [PATCH v7 REPOST 0/9] CPUs capacity information for heterogeneous systems

2016-10-30 Thread Catalin Marinas
On Mon, Oct 17, 2016 at 04:46:41PM +0100, Juri Lelli wrote: > I'm thus now assuming that everybody is OK with the patches and that they can > be queued for 4.10 (we certainly need this plumbing at this point). Please > speak if my assumption is wrong (and provide feedback! :). > Otherwise I'm going

Re: [PATCH v4 10/10] IB/mlx5: Simplify completion into a wait_event

2016-10-30 Thread Sagi Grimberg
On 27/10/16 09:59, Binoy Jayan wrote: Convert the completion 'mlx5_ib_umr_context:done' to a wait_event as it just waits for the return value to be filled. Signed-off-by: Binoy Jayan --- drivers/infiniband/hw/mlx5/mlx5_ib.h | 2 +- drivers/infiniband/hw/mlx5/mr.c | 9 + include/

<    1   2   3   4   >