Re: [RFC PATCH 2/2] bpf: Initialise mod[] in bpf_trace_printk

2017-08-08 Thread James Hogan
On 8 August 2017 17:48:57 BST, David Miller wrote: >From: Daniel Borkmann >Date: Tue, 08 Aug 2017 10:46:52 +0200 > >> On 08/08/2017 12:25 AM, James Hogan wrote: >>> In bpf_trace_printk(), the elements in mod[] are left uninitialised, >>> but >>> they

Re: [PATCH V8 1/2] PCI: handle CRS returned by device after FLR

2017-08-08 Thread Bjorn Helgaas
On Sun, Aug 06, 2017 at 10:09:51PM -0400, Sinan Kaya wrote: We should include some high-level description of the problem we're trying to solve here. I *think* the problem is that we do something like this: - perform an FLR - sleep up to 1000ms total - read ~0 from PCI_COMMAND - warn

Re: [PATCH] rcu: Skip additional checks if rcu_cpu_stall_suppress is set

2017-08-08 Thread Paul E. McKenney
On Wed, Aug 09, 2017 at 12:27:16AM +0530, Neeraj Upadhyay wrote: > > > On 08/08/2017 11:32 PM, Paul E. McKenney wrote: > >On Tue, Aug 08, 2017 at 10:50:26PM +0530, Neeraj Upadhyay wrote: > >>If rcu_kick_kthreads is set, and gp is in progress, check_cpu_stall() > >>does checks to figure out

Re: [PATCH 1/6] [media] v4l: mt9t001: constify video_subdev structures

2017-08-08 Thread Laurent Pinchart
Hi Julia, Thank you for the patch. On Tuesday 08 Aug 2017 12:58:27 Julia Lawall wrote: > The v4l2_subdev_ops structure is only passed as the third argument of > v4l2_i2c_subdev_init, which is const, so the v4l2_subdev_ops structure > can be const as well. The other structures are only stored in

Re: INITRAMFS_SOURCE broken by 6e19eded3684dc184181093af3bff2ff440f5b53?

2017-08-08 Thread Rob Landley
On 08/08/2017 07:04 AM, Willy Tarreau wrote: > Hi Thomas, > > On Tue, Aug 08, 2017 at 01:46:25PM +0200, Thomas Meyer wrote: >> Hi, >> >> did the commit 6e19eded3684dc184181093af3bff2ff440f5b53 break a linux kernel >> build with an included ramdisk? >> >> As fas as I understand you must expliclity

Re: [PATCH v06 05/36] uapi linux/sysctl.h: use __kernel_size_t instead of size_t

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:43:56PM +0200, Mikko Rapeli wrote: > Fixes userspace compilation error: > > error: unknown type name ‘size_t’ > > Signed-off-by: Mikko Rapeli > Cc: Arnd Bergmann > Cc: David S. Miller > --- >

[PATCH] usb: gadget: udc: renesas_usb3: fix error return code in renesas_usb3_probe()

2017-08-08 Thread Gustavo A. R. Silva
platform_get_irq() returns an error code, but the renesas_usb3 driver ignores it and always returns -ENODEV. This is not correct and, prevents -EPROBE_DEFER from being propagated properly. Also, notice that platform_get_irq() no longer returns 0 on error:

Re: [PATCH] tpm/tpm_crb: Access locality for only CRB_START method

2017-08-08 Thread Jason Gunthorpe
On Sun, Aug 06, 2017 at 09:54:15PM -0500, Jiandi An wrote: > static int __maybe_unused crb_go_idle(struct device *dev, struct crb_priv > *priv) > { > - if ((priv->flags & CRB_FL_ACPI_START) || > - (priv->flags & CRB_FL_CRB_SMC_START)) > + if (!(priv->flags & CRB_FL_CRB_START))

Re: [PATCH 0/6] In-kernel QMI handling

2017-08-08 Thread Bjorn Andersson
On Tue 08 Aug 04:02 PDT 2017, Bj?rn Mork wrote: > Bjorn Andersson writes: > > > This series starts by moving the common definitions of the QMUX protocol to > > the > > uapi header, as they are shared with clients - both in kernel and userspace. > > > > This series

[PATCH] tty: serial: sprd: fix error return code in sprd_probe()

2017-08-08 Thread Gustavo A. R. Silva
platform_get_irq() returns an error code, but the sprd_serial driver ignores it and always returns -ENODEV. This is not correct and, prevents -EPROBE_DEFER from being propagated properly. Also, notice that platform_get_irq() no longer returns 0 on error:

Re: [v4 2/4] mm, oom: cgroup-aware OOM killer

2017-08-08 Thread David Rientjes
On Tue, 1 Aug 2017, Roman Gushchin wrote: > > To the rest of the patch. I have to say I do not quite like how it is > > implemented. I was hoping for something much simpler which would hook > > into oom_evaluate_task. If a task belongs to a memcg with kill-all flag > > then we would update the

Re: [PATCH -mm] mm: Clear to access sub-page last when clearing huge page

2017-08-08 Thread Huang, Ying
Matthew Wilcox writes: > On Mon, Aug 07, 2017 at 03:21:31PM +0800, Huang, Ying wrote: >> @@ -2509,7 +2509,8 @@ enum mf_action_page_type { >> #if defined(CONFIG_TRANSPARENT_HUGEPAGE) || defined(CONFIG_HUGETLBFS) >> extern void clear_huge_page(struct page *page, >>

Re: [PATCH v06 22/36] uapi linux/reiserfs_xattr.h: use __kernel_size_t instead of size_t

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:13PM +0200, Mikko Rapeli wrote: > Fixes userspace compilation error: > > error: unknown type name ‘size_t’ > size_t length; > > Signed-off-by: Mikko Rapeli > Cc: reiserfs-de...@vger.kernel.org > --- > include/uapi/linux/reiserfs_xattr.h | 2

Re: [v4 4/4] mm, oom, docs: describe the cgroup-aware OOM killer

2017-08-08 Thread David Rientjes
On Wed, 26 Jul 2017, Roman Gushchin wrote: > +Cgroup-aware OOM Killer > +~~~ > + > +Cgroup v2 memory controller implements a cgroup-aware OOM killer. > +It means that it treats memory cgroups as first class OOM entities. > + > +Under OOM conditions the memory controller tries

[PATCH libdrm] drm: Remove create_handle() drm_framebuffer "virtual".

2017-08-08 Thread Joe Kniss
Because all drivers currently use gem objects for framebuffer planes, the virtual create_handle() is not required. This change adds a struct drm_gem_object *gems[4] field to drm_framebuffer and removes create_handle() function pointer from drm_framebuffer_funcs. The corresponding

[PATCH] cpufreq: x86: Disable interrupts during MSRs reading

2017-08-08 Thread Doug Smythies
According to Intel 64 and IA-32 Architectures SDM, Volume 3, Chapter 14.2, "Software needs to exercise care to avoid delays between the two RDMSRs (for example interrupts)". So, disable interrupts during reading MSRs IA32_APERF and IA32_MPERF. See also: commit

Re: [PATCH v3 01/13] mpt3sas: Update MPI Header

2017-08-08 Thread J Freyensee
Looks like your header has a white space error: Applying: mpt3sas: Update MPI Header .git/rebase-apply/patch:1452: new blank line at EOF. + Also, FYI, this project has a lot of sparse errors that look like existed before your patchset. As your patchset touches a few of the files that have

Re: [RFC PATCH 2/2] bpf: Initialise mod[] in bpf_trace_printk

2017-08-08 Thread David Miller
From: James Hogan Date: Tue, 08 Aug 2017 22:20:05 +0100 > cool, i hadn't realised unmentioned elements in an initialiser are > always zeroed, even when non-global/static, so had interpreted the > whole array as uninitialised. learn something new every day :-) > sorry for

Re: [PATCH] [RFC] tpm_tis: tpm_tcg_flush() after iowrite*()s

2017-08-08 Thread Jarkko Sakkinen
On Mon, Aug 07, 2017 at 09:59:35AM -0500, Julia Cartwright wrote: > On Fri, Aug 04, 2017 at 04:56:51PM -0500, Haris Okanovic wrote: > > I have a latency issue using a SPI-based TPM chip with tpm_tis driver > > from non-rt usermode application, which induces ~400 us latency spikes > > in cyclictest

Re: [PATCH 2/2] arm64: dts: move rpi into rpi directory

2017-08-08 Thread kbuild test robot
Hi Scott, [auto build test ERROR on v4.13-rc1] [cannot apply to robh/for-next arm64/for-next/core next-20170808] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Scott-Branden/arm64-dts-move-ns2

[PATCH v2 0/2] ACPI: Low power idle constraints check

2017-08-08 Thread Srinivas Pandruvada
This patchset enables debugging of low power idle. This is done by checking all the required constraints on wake. v2 - Changes as suggested by Lukas Wunner. - Using pm_debug_messages_on attribute to prevent constraints check to save some cycles on wake. Srinivas Pandruvada (2): PM / sleep:

[PATCH] binder: fix incorrect cmd to binder_stat_br

2017-08-08 Thread Todd Kjos
commit 26549d177410 ("binder: guarantee txn complete / errors delivered in-order") passed the locally declared and undefined cmd to binder_stat_br() which results in a bogus cmd field in a trace event and BR stats are incremented incorrectly. Change to use e->cmd which has been initialized.

Re: [PATCH v06 13/36] x86 uapi asm/sembuf.h: include linux/types.h and linux/ipc.h

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:04PM +0200, Mikko Rapeli wrote: > Fixes userspace compile errors like: > > error: field ‘sem_perm’ has incomplete type > struct ipc64_perm sem_perm; /* permissions .. see ipc.h */ > error: unknown type name ‘__kernel_time_t’ > __kernel_time_t sem_otime; /* last

[PATCH v6 17/17] powerpc/vas: Document FTW API/usage

2017-08-08 Thread Sukadev Bhattiprolu
Document the usage of the VAS Fast thread-wakeup API. Thanks for input/comments from Benjamin Herrenschmidt, Michael Neuling, Michael Ellerman, Robert Blackmore, Ian Munsie, Haren Myneni, Paul Mackerras. Cc:Ian Munsie Cc:Paul Mackerras Signed-off-by:

[PATCH v6 11/17] powerpc/vas: Define vas_win_close() interface

2017-08-08 Thread Sukadev Bhattiprolu
Define the vas_win_close() interface which should be used to close a send or receive windows. While the hardware configurations required to open send and receive windows differ, the configuration to close a window is the same for both. So we use a single interface to close the window.

[PATCH v6 00/17] powerpc/vas: Enable VAS

2017-08-08 Thread Sukadev Bhattiprolu
POWER9 introduces a hardware subsystem referred to as the Virtual Accelerator Switchboard (VAS). VAS allows kernel subsystems and user space processes to directly access the Nest Accelerator (NX) engines which implement compression and encryption algorithms in the hardware. NX has been in Power

[PATCH v6 08/17] powerpc/vas: Define vas_win_id()

2017-08-08 Thread Sukadev Bhattiprolu
Define an interface to return a system-wide unique id for a given VAS window. The vas_win_id() will be used in a follow-on patch to generate an unique handle for a user space receive window. Applications can use this handle to pair send and receive windows for fast thread-wakeup. The hardware

Re: [v4 3/4] mm, oom: introduce oom_priority for memory cgroups

2017-08-08 Thread David Rientjes
On Wed, 26 Jul 2017, Roman Gushchin wrote: > Introduce a per-memory-cgroup oom_priority setting: an integer number > within the [-1, 1] range, which defines the order in which > the OOM killer selects victim memory cgroups. > > OOM killer prefers memory cgroups with larger priority if

Re: [PATCH v9 1/4] PCI: Add new PCIe Fabric End Node flag, PCI_DEV_FLAGS_NO_RELAXED_ORDERING

2017-08-08 Thread Bjorn Helgaas
On Sat, Aug 05, 2017 at 03:15:10PM +0800, Ding Tianhong wrote: > From: Casey Leedom > > The patch adds a new flag PCI_DEV_FLAGS_NO_RELAXED_ORDERING to indicate that > Relaxed Ordering (RO) attribute should not be used for Transaction Layer > Packets (TLP) targetted towards

Re: [PATCH] netfilter: nf_nat_h323: fix logical-not-parentheses warning

2017-08-08 Thread Nick Desaulniers
bumping for review On Mon, Jul 31, 2017 at 11:39 AM, Nick Desaulniers wrote: > Clang produces the following warning: > > net/ipv4/netfilter/nf_nat_h323.c:553:6: error: > logical not is only applied to the left hand side of this comparison >

Re: [PATCH] sched/fair: Make PELT signal more accurate

2017-08-08 Thread Joel Fernandes
Hi Peter, On Mon, Aug 7, 2017 at 6:40 AM, Peter Zijlstra wrote: > On Fri, Aug 04, 2017 at 08:40:23AM -0700, Joel Fernandes wrote: >> The PELT signal (sa->load_avg and sa->util_avg) are not updated if the >> amount accumulated during a single update doesn't cross a period >>

Re: [RFC PATCH] exec: Avoid recursive modprobe for binary format handlers

2017-08-08 Thread Luis R. Rodriguez
On Wed, Aug 02, 2017 at 02:12:00AM +0200, Luis R. Rodriguez wrote: > On Fri, Jul 21, 2017 at 03:05:20PM +0100, Matt Redfearn wrote: > > diff --git a/fs/exec.c b/fs/exec.c > > index 62175cbcc801..004bb50a01fe 100644 > > --- a/fs/exec.c > > +++ b/fs/exec.c > > @@ -1644,6 +1644,9 @@ int

[PATCH v3] perf/core: Avoid context switch overheads

2017-08-08 Thread linxiulei
From: "leilei.lin" A performance issue caused by less strickly check in task sched when these tasks were once attached by per-task perf_event. A task will alloc task->perf_event_ctxp[ctxn] when it was called by perf_event_open, and task->perf_event_ctxp[ctxn] would not ever

Re: linux-next 2017-08-08 keyboard failure on Nokia N900

2017-08-08 Thread Pavel Machek
; > > > > > > > > > but if we will look at code - it can be seen that this variable is > > > > passed > > > > by reference to > > > > irq_set_chip_and_handler(). > > > > Ops. And what will happen when twl4030_init_irq() return

[PATCH] usb: ehci-omap: fix error return code in ehci_hcd_omap_probe()

2017-08-08 Thread Gustavo A. R. Silva
platform_get_irq() returns an error code, but the ehci-omap driver ignores it and always returns -ENODEV. This is not correct and, prevents -EPROBE_DEFER from being propagated properly. Also, notice that platform_get_irq() no longer returns 0 on error:

[PATCH v2 2/2] ACPI / Sleep: Check low power idle constraints for debug only

2017-08-08 Thread Srinivas Pandruvada
For SoC to achieve its lowest power platform idle state a set of hardware preconditions must be met. These preconditions or constraints can be obtained by issuing a device specific method (_DSM) with function "1". Refer to the document provided in the link below. Here during initialization (from

[PATCH v2 1/2] PM / sleep: Export the setting of pm_debug_messages_on

2017-08-08 Thread Srinivas Pandruvada
Added a function to export the value of pm_debug_messages_on, so that other parts of the system can use this flag to enable/disable executing PM debug code. Signed-off-by: Srinivas Pandruvada --- include/linux/suspend.h | 6 ++ kernel/power/main.c |

[PATCH v6 10/17] powerpc/vas: Define vas_rx_win_open() interface

2017-08-08 Thread Sukadev Bhattiprolu
Define the vas_rx_win_open() interface. This interface is intended to be used by the Nest Accelerator (NX) driver(s) to setup receive windows for one or more NX engines (which implement compression/encryption algorithms in the hardware). Follow-on patches will provide an interface to close the

[PATCH v6 06/17] powerpc/vas: Define helpers to alloc/free windows

2017-08-08 Thread Sukadev Bhattiprolu
Define helpers to allocate/free VAS window objects. These will be used in follow-on patches when opening/closing windows. Signed-off-by: Sukadev Bhattiprolu --- arch/powerpc/platforms/powernv/vas-window.c | 70 + 1 file changed, 70

[PATCH v6 03/17] powerpc/vas: Define vas_init() and vas_exit()

2017-08-08 Thread Sukadev Bhattiprolu
Implement vas_init() and vas_exit() functions for a new VAS module. This VAS module is essentially a library for other device drivers and kernel users of the NX coprocessors like NX-842 and NX-GZIP. In the future this will be extended to add support for user space to access the NX coprocessors.

[PATCH v6 09/17] powerpc/vas: Define vas_rx_win_open() interface

2017-08-08 Thread Sukadev Bhattiprolu
Define the vas_rx_win_open() interface. This interface is intended to be used by the Nest Accelerator (NX) driver(s) to setup receive windows for one or more NX engines (which implement compression/encryption algorithms in the hardware). Follow-on patches will provide an interface to close the

[PATCH v6 12/17] powerpc/vas: Define vas_tx_win_open()

2017-08-08 Thread Sukadev Bhattiprolu
Define an interface to open a VAS send window. This interface is intended to be used the Nest Accelerator (NX) driver(s) to open a send window and use it to submit compression/encryption requests to a VAS receive window. The receive window, identified by the [vasid, cop] parameters, must already

Re: [RFC] Tagging of vmalloc pages for supporting the pmalloc allocator

2017-08-08 Thread Jerome Glisse
On Tue, Aug 08, 2017 at 03:59:36PM +0300, Igor Stoppa wrote: > On 07/08/17 22:12, Jerome Glisse wrote: > > On Mon, Aug 07, 2017 at 05:13:00PM +0300, Igor Stoppa wrote: > > [...] > > >> I have an updated version of the old proposal: > >> > >> * put a magic number in the private field, during

Re: [PATCH v06 21/36] uapi linux/kexec.h: use __kernel_size_t instead of size_t

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:12PM +0200, Mikko Rapeli wrote: > Fixes userspace compilation error: > > error: unknown type name ‘size_t’ > size_t bufsz; > > Signed-off-by: Mikko Rapeli > Cc: ke...@lists.infradead.org > Cc: Eric Biederman > --- >

Re: [PATCH v06 35/36] uapi linux/tls.h: don't include in user space

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:26PM +0200, Mikko Rapeli wrote: > It is not needed and not part of uapi headers, but causes > user space compilation error: > > fatal error: net/tcp.h: No such file or directory > #include > ^ > > Signed-off-by: Mikko Rapeli

[PATCH] kmod: test_kmod: Correctly lock reg_dev_mutex

2017-08-08 Thread Daniel Mentz
It appears that, in the preamble of register_test_dev_kmod(), mutex_lock() has been confused with mutex_unlock(). Previously, register_test_dev_kmod() never called mutex_lock() but instead, it called mutex_unlock() twice. Signed-off-by: Daniel Mentz Cc: Luis R. Rodriguez

Re: [PATCH v2] perf/core: Avoid context switch overheads

2017-08-08 Thread 林守磊
2017-08-08 18:37 GMT+08:00 Peter Zijlstra : > On Tue, Aug 08, 2017 at 06:00:45PM +0800, 石祤 wrote: > >> diff --git a/kernel/events/core.c b/kernel/events/core.c >> index 426c2ff..3d86695 100644 >> --- a/kernel/events/core.c >> +++ b/kernel/events/core.c >> @@ -3180,6 +3180,13

[PATCH v3] perf/core: Avoid context switch overheads

2017-08-08 Thread 石祤
From: "leilei.lin" A performance issue caused by less strickly check in task sched when these tasks were once attached by per-task perf_event. A task will alloc task->perf_event_ctxp[ctxn] when it was called by perf_event_open, and task->perf_event_ctxp[ctxn] would

Re: [PATCH 0/6] In-kernel QMI handling

2017-08-08 Thread Dan Williams
On Tue, 2017-08-08 at 15:42 -0700, Bjorn Andersson wrote: > On Tue 08 Aug 04:02 PDT 2017, Bj?rn Mork wrote: > > > Bjorn Andersson writes: > > > > > This series starts by moving the common definitions of the QMUX > > > protocol to the > > > uapi header, as they are

Re: Possible null pointer dereference in rcar-dmac.ko

2017-08-08 Thread Kuninori Morimoto
Hi Anton # add Laurent > While searching for races in the Linux kernel I've come across > "drivers/dma/sh/rcar-dmac.ko" module. Here is a question that I came > up with while analyzing results. Lines are given using the info from > Linux v4.12. > > Consider the following case: > > Thread 1:

Re: [PATCH v4 1/2] x86/unwind: add ORC unwinder

2017-08-08 Thread Josh Poimboeuf
On Tue, Aug 08, 2017 at 01:09:08PM -0700, Andy Lutomirski wrote: > >> c) just add ORC data for the alternative statically and _unconditionally_. > >> > >> No runtime registration. Just an unconditional entry for the > >> particular IP that comes after the "pushfq". It cannot match the > >> "callq"

Re: [PATCH] loop: fix to a race condition due to the early registration of device

2017-08-08 Thread Omar Sandoval
On Mon, Aug 07, 2017 at 03:37:50PM +0300, Anton Volkov wrote: > The early device registration made possible a race leading to allocations > of disks with wrong minors. > > This patch moves the device registration further down the loop_init > function to make the race infeasible. > > Found by

[PATCH] video: fbdev: pxa3xx_gcu: fix error return code in pxa3xx_gcu_probe()

2017-08-08 Thread Gustavo A. R. Silva
platform_get_irq() returns an error code, but the pxa3xx_gcu driver ignores it and always returns -ENODEV. This is not correct and, prevents -EPROBE_DEFER from being propagated properly. Also, notice that platform_get_irq() no longer returns 0 on error:

Re: [PATCH v06 09/36] uapi asm-generic/sembuf.h: include asm/posix_types.h and asm/ipcbuf.h

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:00PM +0200, Mikko Rapeli wrote: > Fixes userspace compilation errors like: > > error: field ‘msg_perm’ has incomplete type > struct ipc64_perm msg_perm; > error: unknown type name ‘__kernel_ulong_t’ > > Signed-off-by: Mikko Rapeli > Acked-by:

Re: [PATCH v06 30/36] uapi rdma/rdma_user_rxe.h: include in.h and in6.h

2017-08-08 Thread Jason Gunthorpe
On Sun, Aug 06, 2017 at 06:44:21PM +0200, Mikko Rapeli wrote: > Fixes these userspace compilation errors when rdma_user_rxe.h is compiled > alone: > > rdma/rdma_user_rxe.h:59:20: error: field ???_sockaddr??? has incomplete type >struct sockaddr _sockaddr; > ^ >

Re: [PATCH v06 19/36] uapi linux/omapfb.h: use __kernel_size_t instead of size_t

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:10PM +0200, Mikko Rapeli wrote: > Fixes userspace compilation error: > > error: unknown type name ‘size_t’ > > Signed-off-by: Mikko Rapeli > Cc: linux-o...@vger.kernel.org > Cc: Tomi Valkeinen > Cc: Aaro Koskinen

Re: [PATCH V8 1/2] PCI: handle CRS returned by device after FLR

2017-08-08 Thread Sinan Kaya
On 8/8/2017 5:19 PM, Bjorn Helgaas wrote: > On Sun, Aug 06, 2017 at 10:09:51PM -0400, Sinan Kaya wrote: > > We should include some high-level description of the problem we're > trying to solve here. > > I *think* the problem is that we do something like this: > > - perform an FLR > - sleep

Re: linux-next: Signed-off-by missing for commits in the scsi tree

2017-08-08 Thread James Bottomley
On Wed, 2017-08-09 at 09:27 +1000, Stephen Rothwell wrote: > Hi James, > > The commit series > >   94b76dcac422 ("scsi: qla2xxx: Fix system crash while triggering FW > dump") > to >   b96b8da34c40 ("scsi: aacraid: Fix out of bounds in > aac_get_name_resp") > > is missing Signed-off-bys from its

Re: [PATCH 4/4] scripts/move_maintainer_sections.bash

2017-08-08 Thread Joe Perches
On Tue, 2017-08-08 at 11:33 -0700, Linus Torvalds wrote: > On Mon, Aug 7, 2017 at 10:04 AM, Joe Perches wrote: > > On Sat, 2017-08-05 at 18:45 -0700, Joe Perches wrote: > > > Move MAINTAINERS into a separate directory and reorder it. > > > Separate various blocks of MAINTAINER

Re: [PATCH v5.1 1/2] ARM64: dts: meson-gx: use stable UART bindings with correct gate clock

2017-08-08 Thread Kevin Hilman
Neil Armstrong writes: > From: Helmut Klein > > This patch switches to the stable UART bindings but also add the correct > gate clock to the non-AO UART nodes for GXBB and GXL SoCs. > > Acked-by: Jerome Brunet >

Re: Re: kvm_intel fails to load on Conroe CPUs running Linux 4.12

2017-08-08 Thread Niall Walsh
Hi, On 07/08/17 18:50, Paolo Bonzini wrote: Given the rarity of your machine I'm currently leaning towards _not_ reverting the change. I'll check another non-Xeon Core 2 tomorrow that is from December 2008 (IIRC). If that one also lacks vNMI, or if I get other reports, I suppose I will have

[PATCH] mm/rmap/mmu_notifier: restore mmu_notifier_invalidate_page() semantic

2017-08-08 Thread jglisse
From: Jérôme Glisse Commit c7ab0d2fdc840266b39db94538f74207ec2afbf6 silently modified semantic of mmu_notifier_invalidate_page() this patch restore it to its previous semantic ie allowing to sleep inside invalidate_page() callback. Signed-off-by: Jérôme Glisse

[PATCH] staging: most: hdm-dim2: fix error return code in dim2_probe()

2017-08-08 Thread Gustavo A. R. Silva
platform_get_irq() returns an error code, but the dim2_hdm driver ignores it and always returns -ENODEV. This is not correct and, prevents -EPROBE_DEFER from being propagated properly. Also, notice that platform_get_irq() no longer returns 0 on error:

Re: [PATCH v06 12/36] x86 uapi asm/signal.h: use __kernel_size_t instead of size_t

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:03PM +0200, Mikko Rapeli wrote: > Fixes userspace compilation error: > > error: unknown type name ‘size_t’ > > Signed-off-by: Mikko Rapeli > Cc: Al Viro > Cc: Arnd Bergmann > Cc: H. Peter Anvin

[PATCH] mtd: spi-nor: fix "No newline at end of file"

2017-08-08 Thread matthew . gerlach
From: Matthew Gerlach Add a newline to the end of drivers/spi-nor/Makefile to get rid the message, "No newline at end of file", produced by git. This fix will allow subsequent changes to the file to be able to produce clean patches. Signed-off-by: Matthew

[PATCH v6 05/17] powerpc/vas: Define helpers to init window context

2017-08-08 Thread Sukadev Bhattiprolu
Define helpers to initialize window context registers of the VAS hardware. These will be used in follow-on patches when opening/closing VAS windows. Signed-off-by: Sukadev Bhattiprolu --- Changelog[v6] - Add support for FTW windows and drop the fault window id

[PATCH v6 01/17] powerpc/vas: Define macros, register fields and structures

2017-08-08 Thread Sukadev Bhattiprolu
Define macros for the VAS hardware registers and bit-fields as well as couple of data structures needed by the VAS driver. Signed-off-by: Sukadev Bhattiprolu --- Changelog[v6] - Add some fields for FTW windows Changelog[v4] - [Michael Neuling] Move

[PATCH v6 02/17] powerpc/vas: Move GET_FIELD/SET_FIELD to vas.h

2017-08-08 Thread Sukadev Bhattiprolu
Move the GET_FIELD and SET_FIELD macros to vas.h as VAS and other users of VAS, including NX-842 can use those macros. There is a lot of related code between the VAS/NX kernel drivers and skiboot. For consistency switch the order of parameters in SET_FIELD to match the order in skiboot.

Re: [PATCH] sched/fair: Make PELT signal more accurate

2017-08-08 Thread Joel Fernandes
Hi Vincent, On Mon, Aug 7, 2017 at 6:24 AM, Vincent Guittot wrote: > Hi Joel, > > On 4 August 2017 at 17:40, Joel Fernandes wrote: >> The PELT signal (sa->load_avg and sa->util_avg) are not updated if the amount >> accumulated during a single

[PATCH v6 04/17] powerpc/vas: Define helpers to access MMIO regions

2017-08-08 Thread Sukadev Bhattiprolu
Define some helper functions to access the MMIO regions. We use these in follow-on patches to read/write VAS hardware registers. They are also used to later issue 'paste' instructions to submit requests to the NX hardware engines. Signed-off-by: Sukadev Bhattiprolu

[PATCH v6 07/17] powerpc/vas: Define vas_win_paste_addr()

2017-08-08 Thread Sukadev Bhattiprolu
Define an interface that the NX drivers can use to find the physical paste address of a send window. This interface is expected to be used with the mmap() operation of the NX driver's device. i.e the user space process can use driver's mmap() operation to map the send window's paste address into

Re: [PATCH net-next v2] wan: dscc4: add checks for dma mapping errors

2017-08-08 Thread Francois Romieu
Alexey Khoroshilov : [...] > diff --git a/drivers/net/wan/dscc4.c b/drivers/net/wan/dscc4.c > index 799830f..6a9ffac 100644 > --- a/drivers/net/wan/dscc4.c > +++ b/drivers/net/wan/dscc4.c > @@ -518,23 +518,31 @@ static void dscc4_release_ring(struct dscc4_dev_priv >

linux-next: Signed-off-by missing for commits in the scsi tree

2017-08-08 Thread Stephen Rothwell
Hi James, The commit series 94b76dcac422 ("scsi: qla2xxx: Fix system crash while triggering FW dump") to b96b8da34c40 ("scsi: aacraid: Fix out of bounds in aac_get_name_resp") is missing Signed-off-bys from its committer. It looks like you have rebased a series from Martin ... -- Cheers,

[PATCH RESEND] Input: xpad - fix PowerA init quirk for some gamepad models

2017-08-08 Thread Cameron Gutman
The PowerA gamepad initialization quirk worked with the PowerA wired gamepad I had around (0x24c6:0x543a), but a user reported [0] that it didn't work for him, even though our gamepads shared the same vendor and product IDs. When I initially implemented the PowerA quirk, I wanted to avoid

Re: [PATCH] powerpc: xive: ensure active irqd when setting affinity

2017-08-08 Thread Sukadev Bhattiprolu
Michael Ellerman [m...@ellerman.id.au] wrote: > Sukadev Bhattiprolu writes: > > > From fd0abf5c61b6041fdb75296e8580b86dc91d08d6 Mon Sep 17 00:00:00 2001 > > From: Benjamin Herrenschmidt > > Date: Tue, 1 Aug 2017 20:54:41 -0500 > > Subject:

Re: [PATCH 1/3] autofs - fix AT_NO_AUTOMOUNT not being honored

2017-08-08 Thread Ian Kent
On 08/08/17 21:11, Colin Walters wrote: > On Tue, Aug 8, 2017, at 12:26 AM, Ian Kent wrote: > >> --- a/include/linux/fs.h >> +++ b/include/linux/fs.h >> @@ -3022,8 +3022,7 @@ static inline int vfs_lstat(const char __user *name, >> struct kstat *stat) >> static inline int vfs_fstatat(int dfd,

Re: [PATCH 4.4 40/57] tpm: Provide strong locking for device removal

2017-08-08 Thread Greg Kroah-Hartman
On Wed, Aug 09, 2017 at 12:05:38AM +0300, Jarkko Sakkinen wrote: > On Sun, Aug 06, 2017 at 03:47:49PM +0300, Jarkko Sakkinen wrote: > > On Fri, Aug 04, 2017 at 02:44:18PM -0700, Greg Kroah-Hartman wrote: > > > On Fri, Aug 04, 2017 at 12:59:56PM -0700, Greg Kroah-Hartman wrote: > > > > On Tue, Aug

linux-next: Signed-off-by missing for commit in the v4l-dvb tree

2017-08-08 Thread Stephen Rothwell
Hi Mauro, Commit 05ad2b6dbbb0 ("media: i2c: fix semicolon.cocci warnings") is missing a Signed-off-by from its author. Preumably the author should have been "Fengguang Wu "? -- Cheers, Stephen Rothwell

Re: [PATCH v5.1 2/2] ARM: dts: meson6: use stable UART bindings

2017-08-08 Thread Kevin Hilman
Neil Armstrong writes: > The UART bindings needs specifying a SoC family, use the meson6 family > for the UART nodes like the other nodes. > Switch to the stable UART bindings for meson6 by adding a XTAL node and > using the proper compatible strings. > > Signed-off-by:

Re: [PATCH] tpm: improve tpm_tis send() performance by ignoring burstcount

2017-08-08 Thread Jarkko Sakkinen
On Mon, Aug 07, 2017 at 07:55:49PM +0530, Nayna wrote: > > > On 08/07/2017 05:22 PM, Peter Huewe wrote: > > > > > > Am 7. August 2017 13:46:32 MESZ schrieb Nayna Jain > > : > > > The TPM burstcount status indicates the number of bytes that can > > > be sent to the

Re: [PATCH v06 08/36] uapi asm-generic/shmbuf.h: include fixes

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:43:59PM +0200, Mikko Rapeli wrote: > Include linux/types.h and asm/msgbuf.h and use __kernel_size_t instead > of size_t. > > Fixes userspace compilation errors like: > > error: field ‘shm_perm’ has incomplete type > struct ipc64_perm shm_perm; /* operation perms */ >

Re: [PATCH v06 14/36] arm uapi asm/signal.h: include for size_t in userspace

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:05PM +0200, Mikko Rapeli wrote: > Arnd Bergmann doubts that __kernel_size_t could be used here > so trying to fall back to gcc's . The only architecture where you cannot do this safely is x86 family because of x32 exception. If there is no chance

[PATCH] mm/mmu_notifier: fix deadlock from typo vm_lock_anon_vma()

2017-08-08 Thread jglisse
From: Jérôme Glisse Fix typo introduced by 0c67e6038580e343bd5af12b7ac6548634f05f0d which result in dead lock when mm_take_all_locks() is call (only user being mmu_notifier at this time) Signed-off-by: Jérôme Glisse Cc: Davidlohr Bueso

Re: [PATCH v06 15/36] uapi linux/socket.h: include sys/socket.h in user space

2017-08-08 Thread Dmitry V. Levin
On Sun, Aug 06, 2017 at 06:44:06PM +0200, Mikko Rapeli wrote: > This libc header has sockaddr definition in user space. > > Fixes user space compilation errors like these from kernel headers including > only linux/socket.h: > > error: field ‘ifru_addr’ has incomplete type > struct sockaddr

Re: [PATCH v4] printk: Add monotonic, boottime, and realtime timestamps

2017-08-08 Thread Prarit Bhargava
On 08/08/2017 04:28 AM, Peter Zijlstra wrote: > On Mon, Aug 07, 2017 at 01:36:39PM -0700, Paul E. McKenney wrote: >> On Mon, Aug 07, 2017 at 04:06:09PM -0400, Prarit Bhargava wrote: > >>> peterz? Want to offer a suggestion? The issue is that I'm changing a bool >>> config option to an int and

[PATCH v6 15/17] powerpc/vas: Define window open ioctls API

2017-08-08 Thread Sukadev Bhattiprolu
Define the VAS_TX_WIN_OPEN and VAS_RX_WIN_OPEN ioctl interface. Each user of VAS, like the NX-FTW driver in a follow-on patch, should implement these ioctls. Signed-off-by: Sukadev Bhattiprolu --- arch/powerpc/include/uapi/asm/vas.h | 30

[PATCH v6 14/17] powerpc: Add support for setting SPRN_TIDR

2017-08-08 Thread Sukadev Bhattiprolu
We need the SPRN_TIDR to bet set for use with fast thread-wakeup (core-to-core wakeup). Each thread in a process needs to have a unique id within the process but as explained below, for now, we assign globally unique thread ids to all threads in the system. Signed-off-by: Sukadev Bhattiprolu

[PATCH v6 16/17] powerpc/vas: Implement a simple FTW driver

2017-08-08 Thread Sukadev Bhattiprolu
The Fast Thread Wake-up (FTW) driver provides user space applications an interface to the Core-to-Core functionality in POWER9. The driver provides the device node/ioctl API to applications and uses the external interfaces to the VAS driver to interact with the VAS hardware. A follow-on patch

[PATCH v6 13/17] powerpc/vas: Define copy/paste interfaces

2017-08-08 Thread Sukadev Bhattiprolu
Define interfaces (wrappers) to the 'copy' and 'paste' instructions (which are new in PowerISA 3.0). These are intended to be used to by NX driver(s) to submit Coprocessor Request Blocks (CRBs) to the NX hardware engines. Signed-off-by: Sukadev Bhattiprolu ---

Re: [PATCH v06 33/36] uapi linux/fsmap.h: use __kernel_size_t instead of size_t

2017-08-08 Thread Darrick J. Wong
On Mon, Aug 07, 2017 at 10:20:58PM +0200, Arnd Bergmann wrote: > On Mon, Aug 7, 2017 at 6:45 PM, Darrick J. Wong > wrote: > > On Mon, Aug 07, 2017 at 06:01:43PM +0200, Arnd Bergmann wrote: > >> On Mon, Aug 7, 2017 at 5:54 PM, Darrick J. Wong >

Re: [PATCH 4/7] signal/mips: Document a conflict with SI_USER with SIGFPE

2017-08-08 Thread Maciej W. Rozycki
On Tue, 8 Aug 2017, Eric W. Biederman wrote: > > This is an "impossible" state to reach unless your hardware is on fire. > > One or more of the FCSR Cause bits will have been set (in `fcr31') or the > > FPE exception would not have happened. > > > > Of course there could be a simulator bug,

Re: [PATCH 2/2] mm, oom: fix potential data corruption when oom_reaper races with writer

2017-08-08 Thread Tetsuo Handa
Andrea Arcangeli wrote: > Overall OOM killing to me was reliable also before the oom reaper was > introduced. I don't think so. We spent a lot of time in order to remove possible locations which can lead to failing to invoke the OOM killer when out_of_memory() is called. > > I just did a

Re: linux-next: Signed-off-by missing for commits in the scsi tree

2017-08-08 Thread Stephen Rothwell
Hi James, On Tue, 08 Aug 2017 16:41:36 -0700 James Bottomley wrote: > >  The rule, I believe, is that only the person who transforms the email > to a git tree should be the last signoff in the commit log, even if > they're a tree pulling into another one

Re: [PATCH v5 net-next 00/12] bpf: rewrite value tracking in verifier

2017-08-08 Thread David Miller
From: Daniel Borkmann Date: Tue, 08 Aug 2017 02:46:16 +0200 > On 08/07/2017 04:21 PM, Edward Cree wrote: >> This series simplifies alignment tracking, generalises bounds tracking >> and >> fixes some bounds-tracking bugs in the BPF verifier. Pointer >> arithmetic on >>

[PATCH] gpu: host1x: fix error return code in host1x_probe()

2017-08-08 Thread Gustavo A. R. Silva
platform_get_irq() returns an error code, but the host1x driver ignores it and always returns -ENXIO. This is not correct and, prevents -EPROBE_DEFER from being propagated properly. Notice that platform_get_irq() no longer returns 0 on error:

linux-next: Tree for Aug 8

2017-08-08 Thread Stephen Rothwell
Hi all, Changes since 20170807: The rdma tree gained a conflict against Linus' tree. I again reverted a commit from the staging tree that was causing overnight build failures. The userns tree gained a conflict against the mips tree. Non-merge commits (relative to Linus' tree): 4689 4882

Re: [PATCH] devfreq: add error check for sscanf in userspace governor

2017-08-08 Thread Pavan Kondeti
Hi Santosh, On Mon, Aug 7, 2017 at 6:36 PM, Santosh Mardi wrote: > store_freq function of devfreq userspace governor > executes further, even if error is returned from sscanf, > this will result in setting up wrong frequency value. > > Add proper error check to bail out

Re: [MD] Crash with 4.12+ kernel and high disk load -- bisected to 4ad23a976413: MD: use per-cpu counter for writes_pending

2017-08-08 Thread David R
Quoting Shaohua Li : Spent some time to check this one, unfortunately I can't find how that patch makes rcu stall. the percpu part looks good to me too. Can you double check if reverting 4ad23a976413aa57 makes the issue go away? When the rcu stall happens, what the

Re: [PATCH] i2c: imx: Remove a useless test in 'i2c_imx_init_recovery_info()'

2017-08-08 Thread Julia Lawall
On Tue, 8 Aug 2017, Christophe JAILLET wrote: > Le 07/08/2017 à 09:16, Julia Lawall a écrit : > > > > On Mon, 7 Aug 2017, Uwe Kleine-König wrote: > > > > > On Mon, Aug 07, 2017 at 01:49:53AM +0200, Christophe JAILLET wrote: > > > > 'devm_pinctrl_get()' never returns NULL, so this test can be

[PATCH v5] powerpc/mm: Only read faulting instruction when necessary in do_page_fault()

2017-08-08 Thread Christophe Leroy
Commit a7a9dcd882a67 ("powerpc: Avoid taking a data miss on every userspace instruction miss") has shown that limiting the read of faulting instruction to likely cases improves performance. This patch goes further into this direction by limiting the read of the faulting instruction to the only

  1   2   3   4   5   6   7   8   9   10   >