Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Michael Kerrisk (man-pages)
Hi Jason, On 03/15/2016 08:32 AM, Jason Baron wrote: > > > On 03/14/2016 01:47 PM, Michael Kerrisk (man-pages) wrote: >> [Restoring CC, which I see I accidentally dropped, one iteration back.] [...] values in events yield an error. EPOLLEXCLUSIVE may be

Re: [patch v2] scsi_dh_alua: uninitialized variable in alua_check_vpd()

2016-03-14 Thread Martin K. Petersen
> "Dan" == Dan Carpenter writes: Dan> The pg_updated variable is support to be set to false at the start Dan> but it is uninitialized. Applied to 4.6/scsi-queue. -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Joe Perches
On Mon, 2016-03-14 at 16:59 -0300, Javier Martinez Canillas wrote: > I don't think they make little sense now since even a non-native english > speaker like me can understand it :) That's a non sequitur if ever I read one. > But yes, it's cryptic at the very least. That's the problem with long te

Re: [PATCH net-next 0/3] net: dsa: finer bridging control

2016-03-14 Thread David Miller
From: Vivien Didelot Date: Sun, 13 Mar 2016 16:21:31 -0400 > This patchset renames the bridging routines of the DSA layer, make the > unbridging routine return void, and rework the DSA netdev notifier handler, > similar to what the Mellanox Spectrum driver does. > > Changes RFC -> v1: > - drop

Re: [PATCH V7 03/12] thermal: tegra: get rid of PDIV/HOTSPOT hack

2016-03-14 Thread Eduardo Valentin
On Fri, Mar 11, 2016 at 11:09:14AM +0800, Wei Ni wrote: > Get rid of T124-specific PDIV/HOTSPOT hack. > tegra-soctherm.c contained a hack to set the SENSOR_PDIV and > SENSOR_HOTSPOT_OFFSET registers - it just did two writes of > T124-specific opaque values. Convert these into a form that can be >

Re: [PATCHv2] vsock: Fix blocking ops call in prepare_to_wait

2016-03-14 Thread Laura Abbott
On 03/14/2016 12:24 PM, David Miller wrote: From: Claudio Imbrenda Date: Fri, 11 Mar 2016 13:39:23 +0100 I think I found a problem with the patch submitted by Laura Abbott ( https://lkml.org/lkml/2016/2/4/711 ): we might miss wakeups. Since the condition is not checked between the prepare_to_w

Re: [PATCH v2 0/3] net/phy: Improvements to Cavium Thunder MDIO code.

2016-03-14 Thread David Daney
On 03/14/2016 12:27 PM, David Miller wrote: From: David Daney Date: Fri, 11 Mar 2016 09:53:08 -0800 Changes from v1: - In 1/3 Add back check for non-OF objects in bgx_init_of_phy(). It is probably not necessary, but better safe than sorry... The firmware on many Cavium Thunder systems

Re: [PATCH v5 1/5] Shared library support

2016-03-14 Thread Emese Revfy
On Fri, 11 Mar 2016 15:19:33 +0900 Masahiro Yamada wrote: > As an alternative, you can add needed build rules > into tools/gcc/Makefile, not scripts/Makefile.host > > I guess these rule won't be used in other places. I think it is better if the rules stay under scripts/ because I expect that t

Re: [PATCH v2 0/5] Add Korean translation of memory-barriers.txt

2016-03-14 Thread Paul E. McKenney
On Fri, Mar 11, 2016 at 12:06:55AM +0900, SeongJae Park wrote: > This patchset aims to add Korean translation of memory-barriers document. > > The patchset starts from fixing minor and trivial problems in the original > document that found during translation. After that, the final patch adds the

Re: [PATCH v4 2/5] x86/msr: Carry on after a non-"safe" MSR access fails without !panic_on_oops

2016-03-14 Thread Peter Zijlstra
On Mon, Mar 14, 2016 at 11:10:16AM -0700, Andy Lutomirski wrote: > A couple of the wrmsr users actually care about performance. These > are the ones involved in context switching and, to a lesser extent, in > switching in and out of guest mode. Right, this very much includes a number of perf MSRs

Re: [PATCH v2 0/3] net/phy: Improvements to Cavium Thunder MDIO code.

2016-03-14 Thread David Miller
From: David Daney Date: Mon, 14 Mar 2016 13:12:53 -0700 > At this point, I think the best path forward is for me to rebase > against net-next and send you a small fixup set to what was merged. > > What do you think? That is in fact the one and only option.

Re: [PATCH] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Alexandre Belloni
On 14/03/2016 at 16:59:33 -0300, Javier Martinez Canillas wrote : > Hello Joe, > > On 03/14/2016 04:38 PM, Joe Perches wrote: > > On Mon, 2016-03-14 at 16:31 -0300, Javier Martinez Canillas wrote: > >> On 03/14/2016 04:11 PM, Joe Perches wrote:> > On Mon, 2016-03-14 at 16:05 > >> -0300, Javier Ma

Re: [PATCH v4 2/2] power: reset: at91-shdwc: add new shutdown controller driver

2016-03-14 Thread Alexandre Belloni
On 10/03/2016 at 16:37:40 +0100, Nicolas Ferre wrote : > +ATMEL AT91 Alternative Shutdown Controller Can't that be sama5d2 instead of alternative? > +M: Nicolas Ferre > +S: Supported > +F: drivers/power/reset/at91-shdwc.c I would also use that in the filename. Because once there is a thir

Re: [PATCH] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Javier Martinez Canillas
Hello Joe, On 03/14/2016 05:03 PM, Joe Perches wrote: > On Mon, 2016-03-14 at 16:59 -0300, Javier Martinez Canillas wrote: >> I don't think they make little sense now since even a non-native english >> speaker like me can understand it :) > > That's a non sequitur if ever I read one. > I was try

[PATCH v2] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Javier Martinez Canillas
The clock and source clock looked up by the driver may not be available just because the clock controller driver was not probed yet so printing an error in this case is not correct and only adds confusion to users. However, knowing that a driver's probe was deferred may be useful so it can be prin

Re: [v3] pstore-ram: add Device Tree bindings

2016-03-14 Thread Brian Norris
Hi Greg, On Thu, Jan 07, 2016 at 03:40:56PM -0800, Greg Hackmann wrote: > ramoops is one of the remaining places where ARM vendors still rely on > board-specific shims. Device Tree lets us replace those shims with > generic code. > > These bindings mirror the ramoops module parameters, with two

Re: [RFC 1/9] mm/hugetlb: Make GENERAL_HUGETLB functions PGD implementation aware

2016-03-14 Thread Andrew Morton
On Fri, 11 Mar 2016 08:31:55 +0530 Anshuman Khandual wrote: > On 03/09/2016 05:40 PM, Anshuman Khandual wrote: > > Currently both the ARCH_WANT_GENERAL_HUGETLB functions 'huge_pte_alloc' > > and 'huge_pte_offset' dont take into account huge page implementation > > at the PGD level. With addition

Re: [PATCH v2] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Joe Perches
On Mon, 2016-03-14 at 17:25 -0300, Javier Martinez Canillas wrote: > The clock and source clock looked up by the driver may not be available > just because the clock controller driver was not probed yet so printing > an error in this case is not correct and only adds confusion to users. > > Howeve

Re: [PATCH] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Joe Perches
On Mon, 2016-03-14 at 21:19 +0100, Alexandre Belloni wrote: > checkpatch will not complain for messages but it will definitively > complain if they are not properly aligned:) checkpatch wouldn't actually complain unless the --strict option was used or these files were in drivers/net or drivers/sta

Re: [PATCH 1/4] acpi,pci,irq: reduce resource requirements

2016-03-14 Thread Sinan Kaya
Hi Bjorn, On 3/14/2016 2:52 PM, Bjorn Helgaas wrote: >> bool acpi_isa_irq_available(int irq) >> > @@ -840,13 +881,6 @@ bool acpi_isa_irq_available(int irq) >> > */ >> > void acpi_penalize_sci_irq(int irq, int trigger, int polarity) >> > { >> > - if (irq >= 0 && irq < ARRAY_SIZE(acpi_irq_penal

Re: linux-next: build failure after merge of the aio tree

2016-03-14 Thread Stephen Rothwell
Hi Ben, On Mon, 14 Mar 2016 13:08:07 -0400 Benjamin LaHaise wrote: > > I put in a patch to use get_user() for now since the 32 bit architectures > don't seem to have any plans for fixing this issue in a predictable > timeframe. There shouldn't be any build failures now -- I've checked ia64, >

Re: coccinelle: generalized removal of unnecessary pointer casts?

2016-03-14 Thread Julia Lawall
On Mon, 14 Mar 2016, Joe Perches wrote: > I wrote a little cocci script to remove unnecessary > casts for memset and memcpy (below) and tested it on > linux kernel's drivers/staging/ directory. > > For instance, when dst and src are already pointers: > > - memcpy((u8 *)dst, (u8 *)src, r871

Re: [PATCH 1/2] x86/mm/pat: Change pat_disable() to emulate PAT table

2016-03-14 Thread Toshi Kani
On Sat, 2016-03-12 at 12:55 +0100, Borislav Petkov wrote: > Ok, let's start again. > > So what's wrong with this simpler, cleaner version below? > > * It makes sure to run the rendezvous handler on the BP on init. > > * It disables PAT otherwise. > > * Oh, and it boots fine with Paul's reproduc

[PATCH] Staging: wlan-ng: defined oui_rfc1042[] and oui_8021h[] arrays as const arrays

2016-03-14 Thread Claudiu Beznea
This patch defines oui_rfc1042[] and oui_8021h[] arrays from p80211conv.c as const arrays since these are not changed anywhere in code. Signed-off-by: Claudiu Beznea --- drivers/staging/wlan-ng/p80211conv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/w

Re: [PATCH 1/5] net: macb: Fix coding style error message

2016-03-14 Thread Michal Simek
On 13.3.2016 20:10, Moritz Fischer wrote: > checkpatch.pl gave the following error: > > ERROR: space required before the open parenthesis '(' > + for(; p < end; p++, offset += 4) > > Signed-off-by: Moritz Fischer > --- > drivers/net/ethernet/cadence/macb.c | 2 +- > 1 file changed, 1 insert

Re: [PATCH 5/5] net: macb: Fix simple typo.

2016-03-14 Thread Michal Simek
On 13.3.2016 20:10, Moritz Fischer wrote: > Signed-off-by: Moritz Fischer > --- > drivers/net/ethernet/cadence/macb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/cadence/macb.c > b/drivers/net/ethernet/cadence/macb.c > index a0c01e5..681e5bf 100

Re: [PATCH 4/5] net: macb: Use ether_addr_copy over memcpy

2016-03-14 Thread Michal Simek
On 13.3.2016 20:10, Moritz Fischer wrote: > Checkpatch suggests using ether_addr_copy over memcpy > to copy the mac address. > > Signed-off-by: Moritz Fischer > --- > drivers/net/ethernet/cadence/macb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/etherne

Re: [PATCH 3/5] net: macb: Address checkpatch 'check' suggestions

2016-03-14 Thread Michal Simek
On 13.3.2016 20:10, Moritz Fischer wrote: > This commit deals with a bunch of checkpatch suggestions > that without changing behavior make checkpatch happier. > > Signed-off-by: Moritz Fischer > --- > drivers/net/ethernet/cadence/macb.c | 46 > +++-- > 1 file cha

Re: [PATCH] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Alexandre Belloni
On 14/03/2016 at 13:33:08 -0700, Joe Perches wrote : > On Mon, 2016-03-14 at 21:19 +0100, Alexandre Belloni wrote: > > checkpatch will not complain for messages but it will definitively > > complain if they are not properly aligned:) > > checkpatch wouldn't actually complain unless the --strict op

Re: [PATCH 2/5] net: macb: Fix coding style warnings

2016-03-14 Thread Michal Simek
On 13.3.2016 20:10, Moritz Fischer wrote: > This commit takes care of the coding style warnings > that are mostly due to a different comment style and > lines over 80 chars, as well as a dangling else. > > Signed-off-by: Moritz Fischer > --- > drivers/net/ethernet/cadence/macb.c | 101 > +++

Re: [PATCH v5 2/5] GCC plugin infrastructure

2016-03-14 Thread Emese Revfy
On Fri, 11 Mar 2016 15:25:19 +0900 Masahiro Yamada wrote: > Maybe scripts/gcc-plugins/ is better than tools/gcc ? > > In the directory "scripts/", we have several tools used during > building the kernel image. > We have some optional programs in the directory "tools/", which are not used > for b

Re: [PATCH RESEND v2 16/18] fuse: Support fuse filesystems outside of init_user_ns

2016-03-14 Thread Miklos Szeredi
On Wed, Mar 9, 2016 at 6:07 PM, Seth Forshee wrote: > On Wed, Mar 09, 2016 at 04:51:42PM +0100, Miklos Szeredi wrote: >> On Wed, Mar 9, 2016 at 4:25 PM, Seth Forshee >> wrote: >> > On Wed, Mar 09, 2016 at 03:48:22PM +0100, Miklos Szeredi wrote: >> >> >> Can't we use current_cred()->uid/gid? Or f

[3.13.y-ckt stable] Linux 3.13.11-ckt36

2016-03-14 Thread Kamal Mostafa
I am announcing the release of the Linux 3.13.11-ckt36 kernel. The updated 3.13.y-ckt tree can be found at: git://kernel.ubuntu.com/ubuntu/linux.git linux-3.13.y and can be browsed at: http://kernel.ubuntu.com/git/ubuntu/linux.git/log/?h=linux-3.13.y The diff from v3.13.11-ckt35 is poste

Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm

2016-03-14 Thread Eduardo Valentin
On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote: > The "critical" type trip in thermal zone can be > set to SOC_THERM hardware, it can trigger shut down > or reset event from hardware. > > Signed-off-by: Wei Ni > Acked-by: Rob Herring > --- > Documentation/devicetree/bindings/thermal/teg

Re: [PATCH 1/4] acpi,pci,irq: reduce resource requirements

2016-03-14 Thread Bjorn Helgaas
On Mon, Mar 14, 2016 at 04:37:51PM -0400, Sinan Kaya wrote: > Hi Bjorn, > > On 3/14/2016 2:52 PM, Bjorn Helgaas wrote: > >> bool acpi_isa_irq_available(int irq) > >> > @@ -840,13 +881,6 @@ bool acpi_isa_irq_available(int irq) > >> > */ > >> > void acpi_penalize_sci_irq(int irq, int trigger, int

Re: [PATCH V7 00/12] Add T210 support in Tegra soctherm

2016-03-14 Thread Eduardo Valentin
On Fri, Mar 11, 2016 at 11:09:11AM +0800, Wei Ni wrote: > This patchset adds following functions for tegra_soctherm driver: > 1. add T210 support. It would be good to update the compatible string in the binding documentation. > 2. export debugfs to show some registers. > 3. add thermtrip funciton

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Michael Kerrisk (man-pages)
Hi Jason, On 03/15/2016 09:01 AM, Michael Kerrisk (man-pages) wrote: > Hi Jason, > > On 03/15/2016 08:32 AM, Jason Baron wrote: >> >> >> On 03/14/2016 01:47 PM, Michael Kerrisk (man-pages) wrote: >>> [Restoring CC, which I see I accidentally dropped, one iteration back.] [...] >>> Returning to

Re: [PATCH v5 3/5] Add Cyclomatic complexity GCC plugin

2016-03-14 Thread Emese Revfy
On Fri, 11 Mar 2016 15:26:39 +0900 Masahiro Yamada wrote: > > diff --git a/scripts/Makefile.gcc-plugins b/scripts/Makefile.gcc-plugins > > index 7c85bf2..dd7b56d 100644 > > --- a/scripts/Makefile.gcc-plugins > > +++ b/scripts/Makefile.gcc-plugins > > @@ -5,7 +5,11 @@ else > > PLUGINCC := $(shell

Re: [PATCH v2] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Javier Martinez Canillas
Hello Joe, On 03/14/2016 05:30 PM, Joe Perches wrote: > On Mon, 2016-03-14 at 17:25 -0300, Javier Martinez Canillas wrote: >> The clock and source clock looked up by the driver may not be available >> just because the clock controller driver was not probed yet so printing >> an error in this case

Re: [PATCH 05/13] input: convert sun4i-ts to use devm_thermal_zone_of_sensor_register

2016-03-14 Thread Eduardo Valentin
On Wed, Mar 09, 2016 at 01:45:14PM -0800, Dmitry Torokhov wrote: > On Wed, Mar 09, 2016 at 01:35:27PM -0800, Eduardo Valentin wrote: > > This changes the driver to use the devm_ version > > of thermal_zone_of_sensor_register and cleans > > up the local points and unregister calls. > > > > Cc: Dmi

[PATCH v8 1/2] staging/android: remove redundant comments on sync_merge_data

2016-03-14 Thread Gustavo Padovan
From: Gustavo Padovan struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- drivers/staging/android/uapi/sync.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

[PATCH v8 2/2] staging/android: refactor SYNC IOCTLs

2016-03-14 Thread Gustavo Padovan
From: Gustavo Padovan Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid future API breaks and optimize buffer allocation. Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the kernel. If the num_fences passed is greater than ze

[patch] direct-io: propagate -ENOSPC errors

2016-03-14 Thread Jeff Moyer
dio_bio_complete turns all errors into -EIO. This is historical, since you used to only get 1 bit precision for errors (BIO_UPTODATE). Now that we get actual error codes, we can return the appropriate code to userspace. File systems seem to only propagate either EIO or ENOSPC, so I've followed su

Re: [PATCH 12/13] thermal: convert tegra_thermal to use devm_thermal_zone_of_sensor_register

2016-03-14 Thread Eduardo Valentin
On Thu, Mar 10, 2016 at 04:46:55PM +0800, Wei Ni wrote: > > > On 2016年03月10日 05:35, Eduardo Valentin wrote: > > This changes the driver to use the devm_ version > > of thermal_zone_of_sensor_register and cleans > > up the local points and unregister calls. > > > > Cc: Zhang Rui > > Cc: Stephen

Re: [PATCH v5 2/5] GCC plugin infrastructure

2016-03-14 Thread PaX Team
On 11 Mar 2016 at 15:25, Masahiro Yamada wrote: > > diff --git a/scripts/gcc-plugin.sh b/scripts/gcc-plugin.sh > > new file mode 100644 > > index 000..eaa4fce > > --- /dev/null > > +++ b/scripts/gcc-plugin.sh > > @@ -0,0 +1,51 @@ > > +#!/bin/sh > > +srctree=$(dirname "$0") > > +gccplugins_dir=

[PATCH v3 0/2] mm, thp: Fix unnecessarry resource consuming in swapin

2016-03-14 Thread Ebru Akagunduz
This patch series fixes unnecessarry resource consuming in khugepaged swapin and introduces a new function to calculate value of specific vm event. Ebru Akagunduz (2): mm, vmstat: calculate particular vm event mm, thp: avoid unnecessary swapin in khugepaged include/linux/vmstat.h | 6 ++

[PATCH v3 1/2] mm, vmstat: calculate particular vm event

2016-03-14 Thread Ebru Akagunduz
Currently, vmstat can calculate specific vm event with all_vm_events() however it allocates all vm events to stack. This patch introduces a helper to sum value of a specific vm event over all cpu, without loading all the events. Signed-off-by: Ebru Akagunduz Acked-by: Kirill A. Shutemov --- Chan

[PATCH v3 2/2] mm, thp: avoid unnecessary swapin in khugepaged

2016-03-14 Thread Ebru Akagunduz
Currently khugepaged makes swapin readahead to improve THP collapse rate. This patch checks vm statistics to avoid workload of swapin, if unnecessary. So that when system under pressure, khugepaged won't consume resources to swapin. The patch was tested with a test program that allocates 800MB of

[PATCH] cred/userns: define current_user_ns() as a function

2016-03-14 Thread Arnd Bergmann
The current_user_ns() macro currently returns &init_user_ns when user namespaces are disabled, and that causes several warnings when building with gcc-6.0 in code that compares the result of the macro to &init_user_ns itself: fs/xfs/xfs_ioctl.c: In function 'xfs_ioctl_setattr_check_projid': fs/xfs

Re: [PATCH 1/4] acpi,pci,irq: reduce resource requirements

2016-03-14 Thread Sinan Kaya
On 3/14/2016 5:01 PM, Bjorn Helgaas wrote: > On Mon, Mar 14, 2016 at 04:37:51PM -0400, Sinan Kaya wrote: >> Hi Bjorn, >> >> On 3/14/2016 2:52 PM, Bjorn Helgaas wrote: bool acpi_isa_irq_available(int irq) > @@ -840,13 +881,6 @@ bool acpi_isa_irq_available(int irq) > */ > void acp

Re: [PATCH v2 0/5] Add Korean translation of memory-barriers.txt

2016-03-14 Thread SeongJae Park
On Tue, Mar 15, 2016 at 5:17 AM, Paul E. McKenney wrote: > On Fri, Mar 11, 2016 at 12:06:55AM +0900, SeongJae Park wrote: >> This patchset aims to add Korean translation of memory-barriers document. >> >> The patchset starts from fixing minor and trivial problems in the original >> document that f

[PATCH 1/2] misc: add nirtfeatures driver

2016-03-14 Thread Kyle Roeschley
From: Jeff Westfahl This driver introduces support for hardware features of National Instruments real-time controllers. This is an ACPI device that exposes LEDs, switches, and watchdogs. Signed-off-by: Jeff Westfahl Signed-off-by: Kyle Roeschley --- drivers/misc/Kconfig| 9 + driver

[PATCH 2/2] misc: nirtfeatures: physical interface elements

2016-03-14 Thread Kyle Roeschley
From: Gratian Crisan These changes add support for PIEs (physical interface elements), which are defined as physical elements fixed to a controller/chassis with which a user can interact (e.g. LEDs and switches) and whose meaning is user-defined and implementation-specific. The support for these

Re: [PATCH 1/2] misc: add nirtfeatures driver

2016-03-14 Thread Greg KH
On Mon, Mar 14, 2016 at 04:54:32PM -0500, Kyle Roeschley wrote: > From: Jeff Westfahl > > This driver introduces support for hardware features of National > Instruments real-time controllers. This is an ACPI device that exposes > LEDs, switches, and watchdogs. If it's an acpi driver, why not put

Re: [GIT PULL] RCU changes for v4.6

2016-03-14 Thread Linus Torvalds
On Mon, Mar 14, 2016 at 3:55 AM, Ingo Molnar wrote: > > (Note the somewhat larger .html/.htmlx files about RCU data structures - let > us > know if you think they are over the top.) I don't think they are over the top because they are *large*. I don't want to pull them because they seem to be t

Re: [PATCH v3 1/2] mm, vmstat: calculate particular vm event

2016-03-14 Thread Rik van Riel
On Mon, 2016-03-14 at 23:40 +0200, Ebru Akagunduz wrote: > Currently, vmstat can calculate specific vm event with > all_vm_events() > however it allocates all vm events to stack. This patch introduces > a helper to sum value of a specific vm event over all cpu, without > loading all the events. >

[PATCH] HID: i2c-hid: fix OOB write in i2c_hid_set_or_send_report()

2016-03-14 Thread Dmitry Torokhov
From: Dmitry Torokhov Even though hid_hw_* checks that passed in data_len is less than HID_MAX_BUFFER_SIZE it is not enough, as i2c-hid does not necessarily allocate buffers of HID_MAX_BUFFER_SIZE but rather checks all device reports and select largest size. In-kernel users normally just send as

Re: [PATCH 1/2] i2c: rk3x: add support for rk3228

2016-03-14 Thread Heiko Stübner
Am Montag, 14. März 2016, 11:09:15 schrieb Yakir Yang: > Enable the I2C core for this SoC. > > Signed-off-by: Yakir Yang Reviewed-by: Heiko Stuebner Heiko > --- > Documentation/devicetree/bindings/i2c/i2c-rk3x.txt | 4 ++-- > drivers/i2c/busses/i2c-rk3x.c | 1 + > 2 fil

Re: coccinelle: generalized removal of unnecessary pointer casts?

2016-03-14 Thread Joe Perches
On Mon, 2016-03-14 at 21:43 +0100, Julia Lawall wrote: > On Mon, 14 Mar 2016, Joe Perches wrote: > > I wrote a little cocci script to remove unnecessary > > casts for memset and memcpy (below) and tested it on > > linux kernel's drivers/staging/ directory. > >  > > For instance, when dst and src ar

[PATCH] media: Update documentation for media_entity_notify

2016-03-14 Thread Shuah Khan
Update documentation for media_entity_notify to clearly state the usage restrictions. This handler is intended for creating links between exiting entities and should not used to create and register entities. Signed-off-by: Shuah Khan --- include/media/media-device.h | 6 -- 1 file changed, 4

Re: [PATCH 2/3] lpfc: fix misleading indentation

2016-03-14 Thread Sebastian Herbszt
Arnd Bergmann wrote: > gcc-6 complains about the indentation of the lpfc_destroy_vport_work_array() > call in lpfc_online(), which clearly doesn't look right: > > drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_online': > drivers/scsi/lpfc/lpfc_init.c:2880:3: warning: statement is indented as if

Re: [GIT PULL] RCU changes for v4.6

2016-03-14 Thread Paul E. McKenney
On Mon, Mar 14, 2016 at 03:12:19PM -0700, Linus Torvalds wrote: > On Mon, Mar 14, 2016 at 3:55 AM, Ingo Molnar wrote: > > > > (Note the somewhat larger .html/.htmlx files about RCU data structures - > > let us > > know if you think they are over the top.) > > I don't think they are over the top

Re: [PATCH 1/2] misc: add nirtfeatures driver

2016-03-14 Thread Josh Cartwright
On Mon, Mar 14, 2016 at 03:05:59PM -0700, Greg KH wrote: > On Mon, Mar 14, 2016 at 04:54:32PM -0500, Kyle Roeschley wrote: > > From: Jeff Westfahl > > > > This driver introduces support for hardware features of National > > Instruments real-time controllers. This is an ACPI device that exposes >

[PATCH] x86/mm, x86/mce: Fix return type/value for memcpy_mcsafe()

2016-03-14 Thread Tony Luck
Returning a 'bool' was very unpopular. Doubly so because the code was just wrong (returning zero for true, one for false; great for shell programming, not so good for C). Change return type to "int". Keep zero as the success indicator because it matches other similar code and people may be more co

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Jason Baron
Hi Michael, On 03/14/2016 05:03 PM, Michael Kerrisk (man-pages) wrote: > Hi Jason, > > On 03/15/2016 09:01 AM, Michael Kerrisk (man-pages) wrote: >> Hi Jason, >> >> On 03/15/2016 08:32 AM, Jason Baron wrote: >>> >>> >>> On 03/14/2016 01:47 PM, Michael Kerrisk (man-pages) wrote: [Restoring CC

Re: [PATCH] crypto: s5p-sss - Enable COMPILE_TEST

2016-03-14 Thread Vladimir Zapolskiy
On 14.03.2016 06:20, Krzysztof Kozlowski wrote: > Get some build coverage of S5P/Exynos AES H/W acceleration driver. > Driver uses DMA and devm_ioremap_resource() so add DMA and IOMEM > dependencies for the compile testing. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Compile-tested on x8

Re: [PATCH v2 3/3] crypto: s5p-sss - Sort the headers to improve readability

2016-03-14 Thread Vladimir Zapolskiy
On 10.03.2016 03:31, Krzysztof Kozlowski wrote: > Sort the headers alphabetically to improve readability and to spot > duplications easier. > > Suggested-by: Vladimir Zapolskiy > Signed-off-by: Krzysztof Kozlowski > > --- > > Changes since v1: > 1. New patch. > > My email differs from one use

Re: [RESEND PATCH] pinctrl: rockchip: add support the get_direction

2016-03-14 Thread Heiko Stübner
Hi Caesar, Am Montag, 14. März 2016, 14:01:19 schrieb Caesar Wang: > This patch adds the get_direction to support the gpio > interface. > > The gpio direction is not used on rockchip platform when use the gpio > debugfs. > > Tested on kylin board. (RK3036 SoCs) > The repro steps: > $/sys/class/g

[PATCH] cgroup: avoid false positive gcc-6 warning

2016-03-14 Thread Arnd Bergmann
When all subsystems are disabled, gcc notices that cgroup_subsys_enabled_key is a zero-length array and that any access to it must be out of bounds: In file included from ../include/linux/cgroup.h:19:0, from ../kernel/cgroup.c:31: ../kernel/cgroup.c: In function 'cgroup_add_cftype

[PATCH 2/3] [media] am437x-vfpe: fix typo in vpfe_get_app_input_index

2016-03-14 Thread Arnd Bergmann
gcc-6 points out an obviously silly comparison in vpfe_get_app_input_index(): drivers/media/platform/am437x/am437x-vpfe.c: In function 'vpfe_get_app_input_index': drivers/media/platform/am437x/am437x-vpfe.c:1709:27: warning: self-comparison always evaluats to true [-Wtautological-compare]

[PATCH 1/3] [media] cobalt: add MTD dependency

2016-03-14 Thread Arnd Bergmann
The cobalt driver fails to link when it is built-in and MTD is disabled or a loadable module: drivers/media/built-in.o: In function `cobalt_flash_probe': :(.text+0xb8b46): undefined reference to `mtd_device_parse_register' :(.text+0xb8b88): undefined reference to `do_map_probe' drivers/media/built

[PATCH 3/3] [media] v4l2-mc: remove unused dtv_demod variable

2016-03-14 Thread Arnd Bergmann
A recent patch removed the only user of the 'dtv_demod' variable in v4l2_mc_create_media_graph, but did not remove the declaration, possibly as a result of an incorrect rebase: drivers/media/v4l2-core/v4l2-mc.c: In function 'v4l2_mc_create_media_graph': drivers/media/v4l2-core/v4l2-mc.c:37:55: err

[PATCH] smc91x: avoid self-comparison warning

2016-03-14 Thread Arnd Bergmann
The smc91x driver defines a macro that compares its argument to itself, apparently to get a true result while using its argument to avoid a warning about unused local variables. Unfortunately, this triggers a warning with gcc-6, as the comparison is obviously useless: drivers/net/ethernet/smsc/sm

[PATCH] mailbox: rockchip: avoid 64-bit division

2016-03-14 Thread Arnd Bergmann
The newly added rockchip mailbox driver causes a bug in the ARM allyesconfig build because of a division of a resource_size_t variable that may be 64 bit wide: drivers/mailbox/built-in.o: In function `rockchip_mbox_probe': :(.text+0x6614): undefined reference to `__aeabi_uldivmod' This adds a cas

[PATCH] staging/comedi/dt282x: avoid integer overflow warning

2016-03-14 Thread Arnd Bergmann
gcc-6 warns about passing negative signed integer into swab16() in the dt282x driver: drivers/staging/comedi/drivers/dt282x.c: In function 'dt282x_load_changain': include/uapi/linux/swab.h:14:33: warning: integer overflow in expression [-Woverflow] (((__u16)(x) & (__u16)0xff00U) >> 8))) ~

Re: ext4_file_open: Inconsistent encryption contexts (commit ff978b09f973) breaking Docker

2016-03-14 Thread Daniel Axtens
Hi Miklos, > Could you please try the below patch? > > It's actually three patches rolled into one: > > 1) ext4: use dget_parent() in ext4_file_open() > >Even with dget_parent() we don't hold i_mutex on directory and so the file >could be moved in the mean time. Not sure if it's a problem

[PATCH] paride: make 'verbose' parameter an 'int' again

2016-03-14 Thread Arnd Bergmann
gcc-6.0 found an ancient bug in the paride driver, which had a "module_param(verbose, bool, 0);" since before 2.6.12, but actually uses it to accept '0', '1' or '2' as arguments: drivers/block/paride/pd.c: In function 'pd_init_dev_parms': drivers/block/paride/pd.c:298:29: warning: comparison of co

Re: [PATCH 0/4] Remove un-needed 'major' registration when alloc_disk(0) is used.

2016-03-14 Thread Ross Zwisler
On Thu, Mar 10, 2016 at 08:59:28AM +1100, NeilBrown wrote: > When alloc_disk(0) is used, the ->major number is ignored and > irrelevant. Yet several drivers register a major number anyway. > > This series of patches removes the pointless registrations. The pmem > driver also does this, but a pat

Re: [PATCH 2/2] x86/mtrr: Refactor PAT initialization code

2016-03-14 Thread Luis R. Rodriguez
On Fri, Mar 11, 2016 at 11:34:26AM -0700, Toshi Kani wrote: > On Fri, 2016-03-11 at 09:13 +, Ingo Molnar wrote: > > * Ingo Molnar wrote: > > > > > > > > * Toshi Kani wrote: > > > > > > > MTRR manages PAT initialization as it implements a rendezvous > > > > handler that initializes PAT as p

[PATCH] staging: lustre: really make lustre dependent on LNet

2016-03-14 Thread Arnd Bergmann
A patch intended to add a dependency on LNET for lustre didn't actually do that and instead allowed configurations that contain lustre with lnet but without IPv4 support that subsequently fail to link: warning: (LUSTRE_FS) selects LNET which has unmet direct dependencies (STAGING && INET && m &&

[PATCH] MAINTAINERS: add myself as i2c mux maintainer

2016-03-14 Thread Peter Rosin
From: Peter Rosin Signed-off-by: Peter Rosin --- MAINTAINERS | 10 ++ 1 file changed, 10 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index f5e6a535bc34..929e91b011cb 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5202,6 +5202,16 @@ F: include/linux/hyperv.h F: tools

[PATCH v2 5/5] arm: davinci: remove mmc dma resources

2016-03-14 Thread David Lechner
The davinci_mmc driver no longer uses platform resources for getting dma channels. Instead lookup is now done using dma_slave_map. Signed-off-by: David Lechner --- v2 changes: This is a new patch in v2. arch/arm/mach-davinci/devices-da8xx.c | 20 arch/arm/mach-davinci/dev

[PATCH v2 4/5] mmc: davinci: don't use dma platform resources

2016-03-14 Thread David Lechner
The davinci arch now has dma_slave_map tables for dma resources, so it is no longer necessary to pass dma resources through the platform device. Signed-off-by: David Lechner --- v2 changes: Remove platform_get_resource completly instead of just ignoring it. drivers/mmc/host/davinci_mmc.c | 19

[PATCH v2 1/5] mmc: davinci: remove matching string

2016-03-14 Thread David Lechner
The string "MMCSDCLK" is not actually used for clock lookup, so can be removed. Signed-off-by: David Lechner --- v2 changes: none drivers/mmc/host/davinci_mmc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/host/davinci_mmc.c b/drivers/mmc/host/davinci_mmc.c

[PATCH v2 0/5] davinci_mmc fixes

2016-03-14 Thread David Lechner
I've updated the patche set based on feedback. This is tested on linux-next with LEGO MINDSTORMS EV3 (da850), both with and without device tree enabled. David Lechner (5): mmc: davinci: remove matching string mmc: davinci: fix unwinding in probe mmc: davinci: prepare clock mmc: davinci: do

[PATCH v2 2/5] mmc: davinci: fix unwinding in probe

2016-03-14 Thread David Lechner
Unwiding from an error in davinci_mmcsd_probe was a mess. Some errors were not handled and not all paths unwound correctly. Also using devm_ where possible to simplify things. Signed-off-by: David Lechner --- v2 changes: use devm_ where possible drivers/mmc/host/davinci_mmc.c | 93

Re: [PATCH v2] rtc: s3c: Don't print an error on probe deferral

2016-03-14 Thread Alexandre Belloni
On 14/03/2016 at 18:07:36 -0300, Javier Martinez Canillas wrote : > Hello Joe, > > On 03/14/2016 05:30 PM, Joe Perches wrote: > > On Mon, 2016-03-14 at 17:25 -0300, Javier Martinez Canillas wrote: > >> The clock and source clock looked up by the driver may not be available > >> just because the cl

[PATCH v2 3/5] mmc: davinci: prepare clock

2016-03-14 Thread David Lechner
When trying to use this driver with the common clock framework, enabling the clock fails because it was not prepared. This fixes the problem by calling clk_prepare and clk_enable in a single function. Ditto for clk_disable_unprepare. Signed-off-by: David Lechner --- v2 changes: use clk_disable_u

[PATCH] f2fs: add missing argument to f2fs_setxattr stub

2016-03-14 Thread Arnd Bergmann
The f2fs_setxattr() prototype for CONFIG_F2FS_FS_XATTR=n has been wrong for a long time, since 8ae8f1627f39 ("f2fs: support xattr security labels"), but there have never been any callers, so it did not matter. Now, the function gets called from f2fs_ioc_keyctl(), which causes a build failure: fs/

Re: [PATCH 1/2] misc: add nirtfeatures driver

2016-03-14 Thread Kyle Roeschley
On Mon, Mar 14, 2016 at 05:30:22PM -0500, Josh Cartwright wrote: > On Mon, Mar 14, 2016 at 03:05:59PM -0700, Greg KH wrote: > > On Mon, Mar 14, 2016 at 04:54:32PM -0500, Kyle Roeschley wrote: > > > From: Jeff Westfahl > > > > > > This driver introduces support for hardware features of National >

[ANNOUNCE] iproute2 4.5

2016-03-14 Thread Stephen Hemminger
Update to iproute2 utility to support new features in Linux 4.5. The usual array small documentation changes and bug fixes. Areas of large changes were BPF and more integration of management of bridges. Source: http://www.kernel.org/pub/linux/utils/net/iproute2/iproute2-4.5.0.tar.gz Repository

Re: [PATCH 4.4 00/50] 4.4.6-stable review

2016-03-14 Thread Shuah Khan
On 03/14/2016 11:50 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.6 release. > There are 50 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses sho

Re: [PATCH 3.14 00/19] 3.14.65-stable review

2016-03-14 Thread Shuah Khan
On 03/14/2016 11:52 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.14.65 release. > There are 19 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses s

Re: [PATCH 3.10 00/18] 3.10.101-stable review

2016-03-14 Thread Shuah Khan
On 03/14/2016 11:52 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.10.101 release. > There are 18 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses

Re: [PATCH] cgroup: avoid false positive gcc-6 warning

2016-03-14 Thread kbuild test robot
Hi Arnd, [auto build test WARNING on cgroup/for-next] [also build test WARNING on v4.5 next-20160314] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/Arnd-Bergmann/cgroup-avoid-false-positive

[PATCH] cgroup: fix boolreturn.cocci warnings

2016-03-14 Thread kbuild test robot
kernel/cgroup.c:262:9-10: WARNING: return of 0/1 in function 'cgroup_ssid_enabled' with return type bool Return statements in functions returning bool should use true/false instead of 1/0. Generated by: scripts/coccinelle/misc/boolreturn.cocci CC: Arnd Bergmann Signed-off-by: Fengguang Wu --

Re: [PATCH] Driver: Vmxnet3: Fix regression caused by cec0556

2016-03-14 Thread Shrikrishna Khare
On Mon, 14 Mar 2016, David Miller wrote: > From: Shrikrishna Khare > Date: Fri, 11 Mar 2016 13:29:44 -0800 > > > don't pass uninitialized flags to spin_unlock_irqrestore. > > > > Reported-by: Tetsuo Handa > > Signed-off-by: Shrikrishna Khare > > Signed-off-by: Guolin Yang > > Sorry, I not

[PATCH v2] cgroup: avoid false positive gcc-6 warning

2016-03-14 Thread Arnd Bergmann
When all subsystems are disabled, gcc notices that cgroup_subsys_enabled_key is a zero-length array and that any access to it must be out of bounds: In file included from ../include/linux/cgroup.h:19:0, from ../kernel/cgroup.c:31: ../kernel/cgroup.c: In function 'cgroup_add_cftype

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Madars Vitolins
Hi Jason and Michael, Hmm... I tried to play with those pipe samples bellow, but even with sleep I got that all process wakeups (maybe I miss something too), also tried with EPOLLIN. On same bases I created sample with Posix Queues with EPOLLIN | EPOLLEXCLUSIVE and the goods news are that it

Re: [PATCH] i2c: immediately mark ourselves as registered

2016-03-14 Thread Nicolai Stange
he original issue was, but this patch fixes a boot failure in linux-next-20160314 for me (see below). Feel free to add a Tested-by: Nicolai Stange [0.881976] [ cut here ] [0.886594] WARNING: CPU: 0 PID: 1 at /mnt/scratch/nic/linux-next/drivers/i2c/i

<    3   4   5   6   7   8   9   >