[PATCH 17/22] perf mem: Fix --all-user/--all-kernel options

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Removing extra '--' prefix. Signed-off-by: Jiri Olsa Cc: David Ahern Cc: Namhyung Kim Cc: Peter Zijlstra Fixes: ad16511b0e40 ("perf mem: Add -U/-K (--all-user/--all-kernel) options") Link: http://lkml.kernel.org/r/1481538943-21874-2-git-send-email-jo...@kernel.org Signed-off

[PATCH 14/22] samples/bpf: Move open_raw_sock to separate header

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer This function was declared in libbpf.c and was the only remaining function in this library, but has nothing to do with BPF. Shift it out into a new header, sock_example.h, and include it from the relevant samples. Signed-off-by: Joe Stringer Cc: Alexei Starovoitov Cc: Daniel

[PATCH resend] IB/core: fix unmap_sg argument

2016-12-13 Thread Sebastian Ott
Hi, using dapltest on s390 I ran into the following warning: [ 20.781709] mlx4_core :00:00.0: DMA-API: device driver frees DMA sg list with different entry count [map count=2] [unmap count=1] [ 20.781760] [ cut here ] [ 20.781767] WARNING: CPU: 4 PID: 1063 at li

[PATCH 01/22] perf tools: Move headers check into bash script

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa To make it nicer and easily maintainable. Also moving the check into fixdep sub make, so its output is not scattered around the build output. Removing extra $$ from mman*.h checks. Signed-off-by: Jiri Olsa Tested-by: Arnaldo Carvalho de Melo Cc: David Ahern Cc: Namhyung Kim

[PATCH 05/22] perf sched timehist: Skip non-idle events when necessary

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim Sometimes it only focuses on idle-related events like upcoming idle-hist feature. In this case we don't want to see other event to reduce noise. Signed-off-by: Namhyung Kim Acked-by: David Ahern Cc: Andi Kleen Cc: Jiri Olsa Cc: Minchan Kim Cc: Peter Zijlstra Link: http:

[PATCH 07/22] perf sched timehist: Show callchains for idle stat

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim When --idle-hist option is used with --summary, it now shows idle stats with callchains like below: Idle stats by callchain: CPU 0: 902.195 msec Idle time (msec)Count Callchains --- --

Re: [PATCH] Coccinelle: uslee_range: ensure delta not zero

2016-12-13 Thread Julia Lawall
issue > > > a warning if delta is 0. > > > > > > Signed-off-by: Nicholas Mc Guire > > > --- > > > > > > As of 4.9.0 this finds about 20 cases - all of which look like the > > > should be passing a range. > > > > > > Patch

[PATCH 18/22] perf evsel: Use variable instead of repeating lengthy FD macro

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa It's more readable and will ease up following patches. Signed-off-by: Jiri Olsa Cc: David Ahern Cc: Namhyung Kim Cc: Peter Zijlstra Link: http://lkml.kernel.org/r/1481538943-21874-3-git-send-email-jo...@kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/util

[PATCH 15/22] perf sdt: Add scanning of sdt probles arguments

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Alexis Berlemont During a "perf buildid-cache --add" command, the section ".note.stapsdt" of the "added" binary is scanned in order to list the available SDT markers available in a binary. The parts containing the probes arguments were left unscanned. The whole section is now parsed; the p

[PATCH 03/22] perf sched timehist: Introduce struct idle_time_data

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim The struct idle_time_data is to keep idle stats with callchains entering to the idle task. The normal thread_runtime calculation is done transparently since it extends the struct thread_runtime. Signed-off-by: Namhyung Kim Acked-by: David Ahern Cc: Andi Kleen Cc: Jiri Olsa

[PATCH 22/22] samples/bpf: Drop unnecessary build targets.

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer Commit f72179ef11db ("samples/bpf: Switch over to libbpf") added these two makefile changes that were unnecessary for switching samples to use libbpf. The extra make is already handled by the build dependency, and libbpf target doesn't build because it lacks main(). Remove thes

[PATCH 10/22] tools lib bpf: Add flags to bpf_create_map()

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer Commit 6c905981743 ("bpf: pre-allocate hash map elements") introduces map_flags to bpf_attr for BPF_MAP_CREATE command. Expose this new parameter in libbpf. By exposing it, users can access flags such as whether or not to preallocate the map. Signed-off-by: Joe Stringer Acke

[PATCH 12/22] samples/bpf: Switch over to libbpf

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer Now that libbpf under tools/lib/bpf/* is synced with the version from samples/bpf, we can get rid most of the libbpf library here. Signed-off-by: Joe Stringer Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Wang Nan Link: http://lkml.kernel.org/r/20161209024620.31660-6-...@

[PATCH 06/22] perf sched timehist: Add -I/--idle-hist option

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim The --idle-hist option is to analyze system idle state so which process makes cpu to go idle. If this option is specified, non-idle events will be skipped and processes switching to/from idle will be shown. This option is mostly useful when used with --summary(-only) option.

[PATCH 13/22] samples/bpf: Remove perf_event_open() declaration

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer This declaration was made in samples/bpf/libbpf.c for convenience, but there's already one in tools/perf/perf-sys.h. Reuse that one. Committer notes: Testing it: $ make -j4 O=../build/v4.9.0-rc8+ samples/bpf/ make[1]: Entering directory '/home/build/v4.9.0-rc8+' CHK

[PATCH 16/22] perf tools: Remove some needless __maybe_unused

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo I.e. those parameters/functions _are_ used, so ditch that misleading attribute. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: http://lkml.kernel.org/n/tip-13cqtjh0yojg5gzvpq1zz...@git.kernel.org Signed-off-by: Arnaldo Carv

[PATCH 11/22] samples/bpf: Make samples more libbpf-centric

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer Switch all of the sample code to use the function names from tools/lib/bpf so that they're consistent with that, and to declare their own log buffers. This allow the next commit to be purely devoted to getting rid of the duplicate library in samples/bpf. Signed-off-by: Joe Str

[PATCH 20/22] perf evsel: Allow to ignore missing pid

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Adding perf_evsel::ignore_missing_cpu_thread bool. When set true, it allows perf to ignore error of missing pid of perf event syscall. We remove missing thread id from the thread_map, so the rest of the processing like ioctl and mmap won't get disturbed with -1 fd. The reason f

[PATCH 19/22] perf thread_map: Add thread_map__remove function

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Add thread_map__remove function to remove thread from thread map. Add automated test also. Committer notes: Testing it: # perf test "Remove thread map" 39: Remove thread map : Ok # perf test -v "Remove thread map" 39: Remove thread map

[PATCH 09/22] tools lib bpf: use __u32 from linux/types.h

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer Fixes the following issue when building without access to 'u32' type: ./tools/lib/bpf/bpf.h:27:23: error: unknown type name ‘u32’ Signed-off-by: Joe Stringer Acked-by: Wang Nan Cc: Alexei Starovoitov Cc: Daniel Borkmann Link: http://lkml.kernel.org/r/20161209024620.31660-

[PATCH 08/22] tools lib bpf: Sync {tools,}/include/uapi/linux/bpf.h

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Joe Stringer The tools version of this header is out of date; update it to the latest version from the kernel headers. Signed-off-by: Joe Stringer Acked-by: Wang Nan Cc: Alexei Starovoitov Cc: Daniel Borkmann Link: http://lkml.kernel.org/r/20161209024620.31660-2-...@ovn.org Signed-off-

Re: [PATCH 5/7] blk-mq-sched: add framework for MQ capable IO schedulers

2016-12-13 Thread Jens Axboe
On 12/13/2016 06:56 AM, Bart Van Assche wrote: > On 12/08/2016 09:13 PM, Jens Axboe wrote: >> +/* >> + * Empty set >> + */ >> +static struct blk_mq_ops mq_sched_tag_ops = { >> +.queue_rq = NULL, >> +}; > > Hello Jens, > > Would "static struct blk_mq_ops mq_sched_tag_ops;" have been suff

Re: [PATCH resend] ARM: dts: sun8i: Support DTB build for NanoPi M1

2016-12-13 Thread Maxime Ripard
On Tue, Dec 13, 2016 at 08:18:15AM +0900, Milo Kim wrote: > The commit 10efbf5f1633 ("ARM: dts: sun8i: Add dts file for NanoPi M1 SBC") > introduced NanoPi M1 board but it's missing in Allwinner H3 DTB build. > > Signed-off-by: Milo Kim Applied, thanks! Maxime -- Maxime Ripard, Free Electrons

Re: [PATCH v1] KVM: x86: avoid redundant REQ_EVENT

2016-12-13 Thread Radim Krčmář
2016-12-12 23:20+0300, Roman Kagan: > On Mon, Dec 12, 2016 at 05:29:43PM +0100, Radim Krčmář wrote: >> 2016-12-12 17:02+0300, Denis Plotnikov: >> > When processing KVM_REQ_EVENT, apic_update_ppr is called which may set >> > KVM_REQ_EVENT again if the recalculated value of PPR becomes smaller >> > t

[PATCH 02/22] perf sched timehist: Split is_idle_sample()

2016-12-13 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim The is_idle_sample() function actually does more than determining whether sample come from idle task. Split the callchain part into save_task_callchain() to make it clearer. Also checking prev_pid from trace data looks preferred than just checking sample->pid since it's possi

Re: [PATCH v2 1/2] net: rfkill: Cleanup error handling in rfkill_init()

2016-12-13 Thread Johannes Berg
On Thu, 2016-12-08 at 08:30 +0100, Michał Kępień wrote: > Use a separate label per error condition in rfkill_init() to make it > a bit cleaner and easier to extend. applied. johannes

Re: [PATCH v2 2/2] net: rfkill: Add rfkill-any LED trigger

2016-12-13 Thread Johannes Berg
On Thu, 2016-12-08 at 08:30 +0100, Michał Kępień wrote: > Add a new "global" (i.e. not per-rfkill device) LED trigger, rfkill- > any, > which may be useful on laptops with a single "radio LED" and multiple > radio transmitters.  The trigger is meant to turn a LED on whenever > there is at least one

Re: [RFC PATCH 0/3] staging: remove fbdev drivers

2016-12-13 Thread Laurent Pinchart
Hi Daniel, On Thursday 08 Dec 2016 11:10:05 Daniel Vetter wrote: > On Thu, Dec 08, 2016 at 12:01:19PM +1100, Benjamin Herrenschmidt wrote: > > On Wed, 2016-11-23 at 10:03 +0200, Tomi Valkeinen wrote: > > > Hi, > > > > > > Since the fbdev framework is in maintenance mode and all new display > > >

Re: [PATCHSET/RFC v2] blk-mq scheduling framework

2016-12-13 Thread Jens Axboe
On Tue, Dec 13 2016, Paolo Valente wrote: > > > Il giorno 08 dic 2016, alle ore 21:13, Jens Axboe ha scritto: > > > > As a followup to this posting from yesterday: > > > > https://marc.info/?l=linux-block&m=148115232806065&w=2 > > > > this is version 2. I wanted to post a new one fairly quickl

RE: [PATCH 5/5] Documentation: fsl-quadspi: Add fsl, ls1012a-qspi compatible string

2016-12-13 Thread Yao Yuan
On Thu, Dec 13, 2016 at 05:23:02PM +0800, Rob Herring wrote: > On Thu, Dec 08, 2016 at 05:23:04PM +0800, Yuan Yao wrote: > > From: Yuan Yao > > Same problem in this subject too. > > > > > new compatible string: "fsl,ls1012a-qspi". > > > > Signed-off-by: Yuan Yao > > --- > > Documentation/devic

Re: [PATCH 5/7] blk-mq-sched: add framework for MQ capable IO schedulers

2016-12-13 Thread Jens Axboe
On Tue, Dec 13 2016, Bart Van Assche wrote: > On 12/08/2016 09:13 PM, Jens Axboe wrote: > >+static inline void blk_mq_sched_put_request(struct request *rq) > >+{ > >+struct request_queue *q = rq->q; > >+struct elevator_queue *e = q->elevator; > >+ > >+if (e && e->type->mq_ops.put_reques

Re: [PATCH V8 3/3] irqchip: qcom: Add IRQ combiner driver

2016-12-13 Thread Agustin Vega-Frias
On 2016-12-07 13:16, Marc Zyngier wrote: Hi Agustin, On 29/11/16 22:57, Agustin Vega-Frias wrote: Driver for interrupt combiners in the Top-level Control and Status Registers (TCSR) hardware block in Qualcomm Technologies chips. An interrupt combiner in this block combines a set of interrupts

Re: [PATCH v2] crypto: sun4i-ss: support the Security System PRNG

2016-12-13 Thread PrasannaKumar Muralidharan
> What do you think about those two solutions ? I prefer the second solution's idea of using two files (/dev/hwrng and /dev/hwprng). Upon having a quick glance it looks like (based on current_rng == prng check) that your current implementation allows only one rng device to be in use at a time. It

[PATCH] Documentation: note that KCOV is supported since gcc 4.5

2016-12-13 Thread Dmitry Vyukov
Now that we have gcc plugins and in particular sancov plugin, KCOV can be used with gcc 4.5+. Note this in the docs. I've tested CONFIG_KCOV with gcc 4.8.4 (stock for ubuntu 14.04) and it works. It even emits very comparable number of coverage callbacks: 420153 vs 422211 for gcc 7.0. Cc: Jonathan

Re: [PATCH v2] kasan: Support for r/w instrumentation control

2016-12-13 Thread Andrey Ryabinin
On 12/13/2016 05:13 PM, Dmitry Vyukov wrote: > On Tue, Dec 13, 2016 at 2:59 PM, Andrey Ryabinin > wrote: >> On 12/13/2016 12:38 PM, Dmitry Vyukov wrote: >>> On Tue, Dec 13, 2016 at 10:20 AM, Andrey Ryabinin >>> wrote: On 12/13/2016 11:58 AM, Dmitry Vyukov wrote: > --- a/

Re: [PATCH] selftests: ftrace: Shift down default message verbosity

2016-12-13 Thread Steven Rostedt
On Tue, 13 Dec 2016 07:06:18 -0700 Shuah Khan wrote: > On 12/12/2016 08:23 PM, Masami Hiramatsu wrote: > > Ping? > > > > On Fri, 25 Nov 2016 21:58:48 +0900 > > Masami Hiramatsu wrote: > > > >> Shift down default message verbosity, so that do not show > >> error results in stdout by default.

[PATCH v2 1/3] drivers: crypto: Add Support for Octeon-tx CPT Engine

2016-12-13 Thread George Cherian
Enable the Physical Function diver for the Cavium Crypto Engine (CPT) found in Octeon-tx series of SoC's. CPT is the Cryptographic Acceleration Unit. CPT includes microcoded GigaCypher symmetric engines (SEs) and asymmetric engines (AEs). Signed-off-by: George Cherian --- drivers/crypto/cavium/c

Re: [PATCH V8 3/3] irqchip: qcom: Add IRQ combiner driver

2016-12-13 Thread Marc Zyngier
On 13/12/16 15:23, Agustin Vega-Frias wrote: > On 2016-12-07 13:16, Marc Zyngier wrote: >> Hi Agustin, >> >> On 29/11/16 22:57, Agustin Vega-Frias wrote: >>> Driver for interrupt combiners in the Top-level Control and Status >>> Registers (TCSR) hardware block in Qualcomm Technologies chips. >>> >>

[RFT PATCH] [media] partial revert of "[media] tvp5150: add HW input connectors support"

2016-12-13 Thread Javier Martinez Canillas
Commit f7b4b54e6364 ("[media] tvp5150: add HW input connectors support") added input signals support for the tvp5150, but the approach was found to be incorrect so the corresponding DT binding commit 82c2ffeb217a ("[media] tvp5150: document input connectors DT bindings") was reverted. This left th

Re: [PATCH v9 2/3] mfd: Kconfig: MFD_SUN4I_GPADC depends on !TOUCHSCREN_SUN4I_GPADC

2016-12-13 Thread Maxime Ripard
On Tue, Dec 13, 2016 at 03:33:31PM +0100, Quentin Schulz wrote: > MFD_SUN4I_GPADC and TOUCHSCREEN_SUN4I are incompatible (both are drivers > for Allwinner SoCs' ADC). This makes sure TOUCHSCREEN_SUN4I isn't > enabled while MFD_SUN4I_GPADC is enabled. > > Signed-off-by: Quentin Schulz Acked-by: M

Re: [PATCH v9 3/3] iio: adc: add support for Allwinner SoCs ADC

2016-12-13 Thread Maxime Ripard
On Tue, Dec 13, 2016 at 03:33:32PM +0100, 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 chann

[PATCH] perf diff: bug fix, donot overwrite valid build id

2016-12-13 Thread kan . liang
From: Kan Liang Fixes a perf diff regression issue which was introduced by commit 5baecbcd9c9a ("perf symbols: we can now read separate debug-info files based on a build ID") The binary name could be same when perf diff different binaries. Build id is used to distinguish between them. However, t

Re: [PATCH v2] crypto: sun4i-ss: support the Security System PRNG

2016-12-13 Thread Corentin Labbe
On Tue, Dec 13, 2016 at 08:53:54PM +0530, PrasannaKumar Muralidharan wrote: > > What do you think about those two solutions ? > > I prefer the second solution's idea of using two files (/dev/hwrng and > /dev/hwprng). Upon having a quick glance it looks like (based on > current_rng == prng check) t

Re: [PATCH v9 1/3] ARM: sunxi_defconfig: Add CONFIG_THERMAL_OF

2016-12-13 Thread Maxime Ripard
On Tue, Dec 13, 2016 at 03:33:30PM +0100, Quentin Schulz wrote: > This enables CONFIG_THERMAL_OF by default for sunxi_defconfig. It is > required to get Allwinner SoCs' temperature from the GPADC driver. > > The Allwinner SoCs all have an ADC that can also act as a touchscreen > controller and a t

Re: usb/core: warning in usb_create_ep_devs/sysfs_create_dir_ns

2016-12-13 Thread Alan Stern
On Tue, 13 Dec 2016, Dmitry Vyukov wrote: > >> > If it is > >> > not a bug in kernel source code, then it must not produce a WARNING. > > > > What about a memory allocation failure? The memory management part of > > the kernel produces a WARNING message if an allocation fails and the > > caller

Re: [patch 2/2] x86/tsc: Force TSC_ADJUST register to value >= zero

2016-12-13 Thread Thomas Gleixner
On Tue, 13 Dec 2016, Peter Zijlstra wrote: > On Tue, Dec 13, 2016 at 01:14:17PM -, Thomas Gleixner wrote: > > @@ -429,8 +447,13 @@ void check_tsc_sync_target(void) > > * that the warp is not longer detectable when the observed warp > > * value is used. In the worst case the adjustment

Re: [PATCH 2/6] clk: sunxi-ng: set the parent rate when adjustin CPUX clock on A33

2016-12-13 Thread Maxime Ripard
On Tue, Dec 13, 2016 at 11:22:48PM +0800, Icenowy Zheng wrote: > The CPUX clock on A33, which is for the Cortex-A7 cores, is designed to > be changeable by changing the rate of PLL_CPUX. > > Add CLK_SET_RATE_PARENT flag to this clock. > > Signed-off-by: Icenowy Zheng Acked-by: Maxime Ripard T

Re: CVE-2016-7097 causes acl leak

2016-12-13 Thread Mark Salyzyn
On 12/12/2016 10:26 PM, Cong Wang wrote: On Mon, Dec 12, 2016 at 4:26 PM, Mark Salyzyn wrote: The leaks were introduced in 9p, gfs2, jfs and xfs drivers only. Only the 9p case is obvious to me: diff --git a/fs/9p/acl.c b/fs/9p/acl.c index b3c2cc7..082d227 100644 --- a/fs/9p/acl.c +++ b/fs/9p

Re: memory_hotplug: zone_can_shift() returns boolean value

2016-12-13 Thread Reza Arbab
On Mon, Dec 12, 2016 at 03:29:04PM -0500, Yasuaki Ishimatsu wrote: --- a/drivers/base/memory.c +++ b/drivers/base/memory.c @@ -410,15 +410,13 @@ static ssize_t show_valid_zones(struct device *dev, sprintf(buf, "%s", zone->name); /* MMOP_ONLINE_KERNEL */ - zone_shift = zone_

[PATCH 1/2] pwm: pca9685: fix period change with same duty cycle

2016-12-13 Thread Clemens Gruber
When first implementing support for changing the output frequency, an optimization was added to continue the PWM after changing the prescaler without having to reprogram the ON and OFF registers for the duty cycle, in case the duty cycle stayed the same. This was flawed, because we compared the abs

[PATCH 2/2] pwm: pca9685: fix prescaler initialization

2016-12-13 Thread Clemens Gruber
Until now, we assumed that the period is the hardware default of 1/200Hz at probe time, but if the period was changed and the user reboots, this assumption is wrong. Solution: Check if the prescaler is set to the hardware default. If not, reprogram the prescaler at first configuration. Cc: # v4.

Re: [PATCH v6 3/8] PWM: add pwm-stm32 DT bindings

2016-12-13 Thread Rob Herring
On Tue, Dec 13, 2016 at 5:11 AM, Lee Jones wrote: > On Mon, 12 Dec 2016, Rob Herring wrote: > >> On Fri, Dec 09, 2016 at 03:15:14PM +0100, Benjamin Gaignard wrote: >> > Define bindings for pwm-stm32 >> > >> > version 6: >> > - change st,breakinput parameter format to make it usuable on stm32f7 too

Re: [PATCH 3/6] ARM: dts: sun8i: add a cpu0 label to cpu@0 node on A23/33

2016-12-13 Thread Maxime Ripard
On Tue, Dec 13, 2016 at 11:22:49PM +0800, Icenowy Zheng wrote: > A "cpu0" label is needed on cpu@0 for cpufreq-dt to work. > > Add such a label, in order to prepare for cpufreq support of A23/33. > > Signed-off-by: Icenowy Zheng Applied, thanks! Maxime -- Maxime Ripard, Free Electrons Embedde

Re: [PATCH] selftests: ftrace: Shift down default message verbosity

2016-12-13 Thread Shuah Khan
On 12/13/2016 08:25 AM, Steven Rostedt wrote: > On Tue, 13 Dec 2016 07:06:18 -0700 > Shuah Khan wrote: > >> On 12/12/2016 08:23 PM, Masami Hiramatsu wrote: >>> Ping? >>> >>> On Fri, 25 Nov 2016 21:58:48 +0900 >>> Masami Hiramatsu wrote: >>> Shift down default message verbosity, so that d

Re: [PATCH] sched/pid fix use-after free in task_tgid_vnr

2016-12-13 Thread Oleg Nesterov
On 12/13, Eric W. Biederman wrote: > > Oleg Nesterov writes: > > > In this case current->group_leader or parent/real_parent can point to the > > exited/freed tasks. I already said this many times, ee really need to > > nullify > > them in __unhash_process() but this needs a lot of (mostly simple)

Re: [PATCH] staging: android: ion: return -ENOMEM in ion_cma_heap allocation failure

2016-12-13 Thread Laura Abbott
On 12/08/2016 09:05 PM, Jaewon Kim wrote: > Initial Commit 349c9e138551 ("gpu: ion: add CMA heap") returns -1 in > allocation > failure. The returned value is passed up to userspace through ioctl. So user > can > misunderstand error reason as -EPERM(1) rather than -ENOMEM(12). > > This patch sim

Re: [PATCH -v4 00/10] FUTEX_UNLOCK_PI wobbles

2016-12-13 Thread Peter Zijlstra
On Tue, Dec 13, 2016 at 09:36:38AM +0100, Peter Zijlstra wrote: > The basic idea is to, like requeue PI, break the rt_mutex_lock() function into > pieces, such that we can enqueue the waiter while holding hb->lock, wait for > acquisition without hb->lock and can remove the waiter, on failure, whil

Re: CVE-2016-7097 causes acl leak

2016-12-13 Thread Jan Kara
On Tue 13-12-16 07:55:23, Mark Salyzyn wrote: > On 12/12/2016 10:26 PM, Cong Wang wrote: > >On Mon, Dec 12, 2016 at 4:26 PM, Mark Salyzyn wrote: > >>The leaks were introduced in 9p, gfs2, jfs and xfs drivers only. > > > >Only the 9p case is obvious to me: > > > >diff --git a/fs/9p/acl.c b/fs/9p/ac

Re: version 3.18.44 to 3.18.45 introduced a bug in "drivers/scsi/megaraid/megaraid_sas_base.c"

2016-12-13 Thread Randy Dunlap
[adding other lists + gregkh] On 12/13/16 02:56, Dashi DS1 Cao wrote: > --- a/drivers/scsi/megaraid/megaraid_sas_base.c > +++ b/drivers/scsi/megaraid/megaraid_sas_base.c > @@ -1614,16 +1614,13 @@ megasas_queue_command(struct Scsi_Host *shost, struct > scsi_cmnd *scmd) > goto out_

Re: [PATCH v6 2/2] crypto: add virtio-crypto driver

2016-12-13 Thread Halil Pasic
On 12/12/2016 11:05 PM, Michael S. Tsirkin wrote: > On Mon, Dec 12, 2016 at 06:54:07PM +0800, Herbert Xu wrote: >> On Mon, Dec 12, 2016 at 06:25:12AM +, Gonglei (Arei) wrote: >>> Hi, Michael & Herbert >>> >>> Because the virtio-crypto device emulation had been in QEMU 2.8, >>> would you pleas

Re: [PATCH] vfio/pci: Support error recovery

2016-12-13 Thread Michael S. Tsirkin
On Mon, Dec 12, 2016 at 08:39:48PM -0700, Alex Williamson wrote: > On Tue, 13 Dec 2016 05:15:13 +0200 > "Michael S. Tsirkin" wrote: > > > On Mon, Dec 12, 2016 at 03:43:13PM -0700, Alex Williamson wrote: > > > > So just don't do it then. Topology must match between host and guest, > > > > except m

Re: [PATCH 3/6] ARM: dts: sun8i: add a cpu0 label to cpu@0 node on A23/33

2016-12-13 Thread Sudeep Holla
On 13/12/16 15:22, Icenowy Zheng wrote: > A "cpu0" label is needed on cpu@0 for cpufreq-dt to work. > IIUC any label should be fine and I don't see anything in the driver looking for such label name. All I see is it looks for cpu0 regulator for *legacy* DTs > Add such a label, in order to prep

[for-next][PATCH 4/5] kprobes/trace: Fix kprobe selftest for newer gcc

2016-12-13 Thread Steven Rostedt
From: Marcin Nowakowski Commit 265a5b7ee3eb ("kprobes/trace: Fix kprobe selftest for gcc 4.6") has added __used attribute to kprobe_trace_selftest_target to ensure that the method is listed in kallsyms table. However, even though the method remains in the kernel image, the actual call is optimiz

[for-next][PATCH 1/5] tracing: Use SOFTIRQ_OFFSET for softirq dectection for more accurate results

2016-12-13 Thread Steven Rostedt
From: Pavankumar Kondeti The 's' flag is supposed to indicate that a softirq is running. This can be detected by testing the preempt_count with SOFTIRQ_OFFSET. The current code tests the preempt_count with SOFTIRQ_MASK, which would be true even when softirqs are disabled but not serving a softir

[for-next][PATCH 0/5] tracing: Hopefully last update for merge window

2016-12-13 Thread Steven Rostedt
Some last minute additions. git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next Head SHA1: 772b020609258144b80a853d7305b5723125f4ca Marcin Nowakowski (2): tracing/kprobes: Add a helper method to return number of probe hits kprobes/trace: Fix kprobe self

Re: [PATCH v2 5/5] staging: lustre: headers: use proper byteorder functions in lustre_idl.h

2016-12-13 Thread Oleg Drokin
On Dec 13, 2016, at 3:31 AM, Dan Carpenter wrote: > It used to be that great swathes of Lustre were used in both user space > and kernel space. We had huge unused modules in the kernel that were > only used for user space. Huh? There was nothing of the sort. There were huge parts of code that w

[for-next][PATCH 3/5] tracing/kprobes: Add a helper method to return number of probe hits

2016-12-13 Thread Steven Rostedt
From: Marcin Nowakowski The number of probe hits is stored in a percpu variable and therefore can't be read directly. Add a helper method trace_kprobe_nhit() that performs the required calculation. It will be used in a follow-up commit that changes kprobe selftests to verify the number of probe

[for-next][PATCH 5/5] selftests: ftrace: Shift down default message verbosity

2016-12-13 Thread Steven Rostedt
From: Masami Hiramatsu Shift down default message verbosity, where it does not show error results in stdout by default. Since that behavior is the same as giving the --quiet option, this patch removes --quiet and makes --verbose increasing verbosity. In other words, this changes verbosity option

[for-next][PATCH 2/5] tracing/rb: Init the CPU mask on allocation

2016-12-13 Thread Steven Rostedt
From: Sebastian Andrzej Siewior Before commit b32614c03413 ("tracing/rb: Convert to hotplug state machine") the allocated cpumask was initialized to the mask of ONLINE or POSSIBLE CPUs. After the CPU hotplug changes the buffer initialisation moved to trace_rb_cpu_prepare() but I forgot to initial

Re: [PATCHSET/RFC v2] blk-mq scheduling framework

2016-12-13 Thread Paolo Valente
> Il giorno 13 dic 2016, alle ore 16:17, Jens Axboe ha scritto: > > On Tue, Dec 13 2016, Paolo Valente wrote: >> >>> Il giorno 08 dic 2016, alle ore 21:13, Jens Axboe ha scritto: >>> >>> As a followup to this posting from yesterday: >>> >>> https://marc.info/?l=linux-block&m=148115232806065&

Re: [PATCH v8 1/3] perf annotate: Show raw form for jump instruction with indirect target

2016-12-13 Thread Ravi Bangoria
Hi Arnaldo, Can you please review 2nd and 3rd patch. -Ravi On Monday 05 December 2016 09:26 PM, Ravi Bangoria wrote: > For jump instructions that does not include target address as direct > operand, show the original disassembled line for them. This is needed > for certain powerpc jump instructi

Re: [PATCH v8 2/3] perf annotate: Support jump instruction with target as second operand

2016-12-13 Thread Arnaldo Carvalho de Melo
Em Mon, Dec 05, 2016 at 09:26:46PM +0530, Ravi Bangoria escreveu: > +++ b/tools/perf/util/annotate.c > @@ -223,8 +223,12 @@ bool ins__is_call(const struct ins *ins) > static int jump__parse(struct arch *arch __maybe_unused, struct ins_operands > *ops, struct map *map __maybe_unused) > { >

Re: usb/core: warning in usb_create_ep_devs/sysfs_create_dir_ns

2016-12-13 Thread Dmitry Vyukov
On Tue, Dec 13, 2016 at 4:52 PM, Alan Stern wrote: > On Tue, 13 Dec 2016, Dmitry Vyukov wrote: > >> >> > If it is >> >> > not a bug in kernel source code, then it must not produce a WARNING. >> > >> > What about a memory allocation failure? The memory management part of >> > the kernel produces

RE: [PATCH 3/5] Documentation: dt: mtd: add chip support for "jedec, spi-nor"

2016-12-13 Thread Yao Yuan
On Thu, Dec 13, 2016 at 05:23:02PM +0800, Rob Herring wrote: > On Thu, Dec 08, 2016 at 05:23:02PM +0800, Yuan Yao wrote: > > From: Yuan Yao > > The compatible string is wrong in the subject. > > > > > "sst25wf040b" and "en25s64" are also chip compatible with SPI NOR flash. > > > > Signed-off-by:

Re: [PATCH v5] cgroup: Add new capability to allow a process to migrate other tasks between cgroups

2016-12-13 Thread John Stultz
On Tue, Dec 13, 2016 at 1:47 AM, Michael Kerrisk (man-pages) wrote: > On 13 December 2016 at 02:39, John Stultz wrote: > So, back to the discussion of silos. I understand the argument for > wanting a new silo. But, in that case can we at least try not to make > it a single-use silo? > > How about

Re: version 3.18.44 to 3.18.45 introduced a bug in "drivers/scsi/megaraid/megaraid_sas_base.c"

2016-12-13 Thread Greg Kroah-Hartman
On Tue, Dec 13, 2016 at 08:08:27AM -0800, Randy Dunlap wrote: > [adding other lists + gregkh] > > > On 12/13/16 02:56, Dashi DS1 Cao wrote: > > --- a/drivers/scsi/megaraid/megaraid_sas_base.c > > +++ b/drivers/scsi/megaraid/megaraid_sas_base.c > > @@ -1614,16 +1614,13 @@ megasas_queue_command(str

[PATCH v2 0/4] KVM: x86: allow hotplug of VCPU with APIC ID over 0xff

2016-12-13 Thread Radim Krčmář
v2 removes the capability that let userspace know of changes. v1: http://www.spinics.net/lists/kvm/msg141944.html > The problem is described in [4/4]. > > [1/4] is a prerequisite to allow a cleanup in [2/4]. > [2/4] hopefully makes [4/4] easier to understand. > [3/4] fixes mistakes from dealing w

Re: [for-next][PATCH 5/5] selftests: ftrace: Shift down default message verbosity

2016-12-13 Thread Steven Rostedt
[ Rebased to add Shuah's ack ] >From 3dbb16b87b57bb1088044ad2a0432e4769075002 Mon Sep 17 00:00:00 2001 From: Masami Hiramatsu Date: Fri, 25 Nov 2016 21:58:48 +0900 Subject: [PATCH] selftests: ftrace: Shift down default message verbosity Shift down default message verbosity, where it does not sh

[PATCH v2 1/4] KVM: x86: use delivery to self in hyperv synic

2016-12-13 Thread Radim Krčmář
Interrupt to self can be sent without knowing the APIC ID. Reviewed-by: David Hildenbrand Signed-off-by: Radim Krčmář --- v2: r-b David arch/x86/kvm/hyperv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/kvm/hyperv.c b/arch/x86/kvm/hyperv.c index 99cde5220e0

[PATCH v2 2/4] KVM: x86: replace kvm_apic_id with kvm_{x,x2}apic_id

2016-12-13 Thread Radim Krčmář
There were three calls sites: - recalculate_apic_map and kvm_apic_match_physical_addr, where it would only complicate implementation of x2APIC hotplug; - in apic_debug, where it was still somewhat preserved, but keeping the old function just for apic_debug was not worth it Signed-off-by: R

[PATCH v2 3/4] KVM: x86: make interrupt delivery fast and slow path behave the same

2016-12-13 Thread Radim Krčmář
Slow path tried to prevent IPIs from x2APIC VCPUs from being delivered to xAPIC VCPUs and vice-versa. Make slow path behave like fast path, which never distinguished that. Signed-off-by: Radim Krčmář --- arch/x86/kvm/lapic.c | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-)

Re: [PATCH] selftests: ftrace: Shift down default message verbosity

2016-12-13 Thread Steven Rostedt
On Tue, 13 Dec 2016 09:03:19 -0700 Shuah Khan wrote: > Acked-by: Shuah Khan thanks! -- Steve

[PATCH v2 4/4] KVM: x86: allow hotplug of VCPU with APIC ID over 0xff

2016-12-13 Thread Radim Krčmář
LAPIC after reset is in xAPIC mode, which poses a problem for hotplug of VCPUs with high APIC ID, because reset VCPU is waiting for INIT/SIPI, but there is no way to uniquely address it using xAPIC. >From many possible options, we chose the one that also works on real hardware: accepting interrupt

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Roland Scheidegger
Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: > Roland reported interesting TSC ADJUST register wreckage on his DELL > machine, which seems to populate that MSR with a random number generator. FWIW, I thought about the actual values some more and I don't actually think they are all that random a

Re: version 3.18.44 to 3.18.45 introduced a bug in "drivers/scsi/megaraid/megaraid_sas_base.c"

2016-12-13 Thread Randy Dunlap
On 12/13/16 08:30, Greg Kroah-Hartman wrote: > On Tue, Dec 13, 2016 at 08:08:27AM -0800, Randy Dunlap wrote: >> [adding other lists + gregkh] >> >> >> On 12/13/16 02:56, Dashi DS1 Cao wrote: >>> --- a/drivers/scsi/megaraid/megaraid_sas_base.c >>> +++ b/drivers/scsi/megaraid/megaraid_sas_base.c >>>

Re: [PATCH 3/6] ARM: dts: sun8i: add a cpu0 label to cpu@0 node on A23/33

2016-12-13 Thread Chen-Yu Tsai
On Wed, Dec 14, 2016 at 12:09 AM, Sudeep Holla wrote: > > > On 13/12/16 15:22, Icenowy Zheng wrote: >> A "cpu0" label is needed on cpu@0 for cpufreq-dt to work. >> > > IIUC any label should be fine and I don't see anything in the driver > looking for such label name. All I see is it looks for cpu0

[PATCH v2 0/3] Add Support for Cavium Cryptographic Accelerarion Unit

2016-12-13 Thread George Cherian
This series adds the support for Cavium Cryptographic Accelerarion Unit (CPT) CPT is available in Cavium's Octeon-Tx SoC series. The series was tested with ecryptfs and dm-crypt for in kerne

Re: [PATCH v5] cgroup: Add new capability to allow a process to migrate other tasks between cgroups

2016-12-13 Thread Casey Schaufler
On 12/13/2016 1:47 AM, Michael Kerrisk (man-pages) wrote: > Hi John, > > On 13 December 2016 at 02:39, John Stultz wrote: >> This patch adds CAP_GROUP_MIGRATE and logic to allows a process > s/CAP_GROUP_MIGRATE/CAP_CGROUP_MIGRATE/ > >> to migrate other tasks between cgroups. >> >> In Android (wher

Re: [PATCH] keys/encrypted: Fix two crypto-on-the-stack bugs

2016-12-13 Thread Andy Lutomirski
[add some people who might know] On Tue, Dec 13, 2016 at 4:20 AM, David Laight wrote: > From: Andy Lutomirski >> Sent: 12 December 2016 20:53 >> The driver put a constant buffer of all zeros on the stack and >> pointed a scatterlist entry at it in two places. This doesn't work >> with virtual st

Re: version 3.18.44 to 3.18.45 introduced a bug in "drivers/scsi/megaraid/megaraid_sas_base.c"

2016-12-13 Thread James Bottomley
On Tue, 2016-12-13 at 08:33 -0800, Randy Dunlap wrote: > On 12/13/16 08:30, Greg Kroah-Hartman wrote: > > I don't maintain 3.18-stable :) > > > > thanks, > > > > greg k-h > > > > Thanks. My bad. > > adding Sasha. This was all covered here: https://www.spinics.net/lists/stable/msg150608.html

Re: [PATCH] orinoco: Use shash instead of ahash for MIC calculations

2016-12-13 Thread Andy Lutomirski
On Tue, Dec 13, 2016 at 3:35 AM, Kalle Valo wrote: > Andy Lutomirski writes: > >> Eric Biggers pointed out that the orinoco driver pointed scatterlists >> at the stack. >> >> Fix it by switching from ahash to shash. The result should be >> simpler, faster, and more correct. >> >> Cc: sta...@vger

[PATCH] i2c: designware: Cleaning and comment style fixes.

2016-12-13 Thread Luis Oliveira
- Misspelling of some words - Comment format fix - Minor fix in coding style Signed-off-by: Luis Oliveira --- The purpose of this patch is to fix some comments and styling issues in the existing code. What is being made here is: - Sorted the headers files - Corrected the commentary format - Rev

Re: version 3.18.44 to 3.18.45 introduced a bug in "drivers/scsi/megaraid/megaraid_sas_base.c"

2016-12-13 Thread Greg Kroah-Hartman
On Tue, Dec 13, 2016 at 08:43:41AM -0800, James Bottomley wrote: > On Tue, 2016-12-13 at 08:33 -0800, Randy Dunlap wrote: > > On 12/13/16 08:30, Greg Kroah-Hartman wrote: > > > I don't maintain 3.18-stable :) > > > > > > thanks, > > > > > > greg k-h > > > > > > > Thanks. My bad. > > > > adding

Re: [PATCH] keys/encrypted: Fix two crypto-on-the-stack bugs

2016-12-13 Thread David Howells
Andy Lutomirski wrote: > After all, rodata is ordinary memory, is backed by struct page, etc. Is that actually true? I thought some arches excluded the kernel image from the page struct array to make the array consume less memory. David

Re: memory_hotplug: zone_can_shift() returns boolean value

2016-12-13 Thread Yasuaki Ishimatsu
Hi Reza, Thank you for your review. On 12/13/2016 10:54 AM, Reza Arbab wrote: On Mon, Dec 12, 2016 at 03:29:04PM -0500, Yasuaki Ishimatsu wrote: --- a/drivers/base/memory.c +++ b/drivers/base/memory.c @@ -410,15 +410,13 @@ static ssize_t show_valid_zones(struct device *dev, sprintf(buf, "%

Re: [PATCH v2 1/7] selftest: sync: basic tests for sw_sync framework

2016-12-13 Thread Shuah Khan
Hi Gustavo, On 11/01/2016 10:18 AM, Gustavo Padovan wrote: > Hi Emilio, > > 2016-10-19 Emilio López : > >> These tests are based on the libsync test suite from Android. >> This commit lays the ground for future tests, as well as includes >> tests for a variety of basic allocation commands. >> >>

Re: [PATCH v2 2/2] mfd: axp20x: Fix AXP806 access errors on cold boot

2016-12-13 Thread Mark Brown
On Fri, Dec 09, 2016 at 11:20:18AM +, Lee Jones wrote: > Is the following valid/necessary? > On Wed, 23 Nov 2016, Chen-Yu Tsai wrote: > > The AXP806 supports either master/standalone or slave mode. > > Slave mode allows sharing the serial bus, even with multiple > > AXP806 which all have the

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Thomas Gleixner
On Tue, 13 Dec 2016, Roland Scheidegger wrote: > Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: > > Roland reported interesting TSC ADJUST register wreckage on his DELL > > machine, which seems to populate that MSR with a random number generator. > > FWIW, I thought about the actual values some

Re: [PATCH v5] cgroup: Add new capability to allow a process to migrate other tasks between cgroups

2016-12-13 Thread John Stultz
On Tue, Dec 13, 2016 at 8:39 AM, Casey Schaufler wrote: > On 12/13/2016 1:47 AM, Michael Kerrisk (man-pages) wrote: >> Hi John, >> >> On 13 December 2016 at 02:39, John Stultz wrote: >>> This patch adds CAP_GROUP_MIGRATE and logic to allows a process >> s/CAP_GROUP_MIGRATE/CAP_CGROUP_MIGRATE/ >>

Re: [PATCH 3/6] ARM: dts: sun8i: add a cpu0 label to cpu@0 node on A23/33

2016-12-13 Thread Sudeep Holla
On 13/12/16 16:31, Chen-Yu Tsai wrote: > On Wed, Dec 14, 2016 at 12:09 AM, Sudeep Holla wrote: >> >> >> On 13/12/16 15:22, Icenowy Zheng wrote: >>> A "cpu0" label is needed on cpu@0 for cpufreq-dt to work. >>> >> >> IIUC any label should be fine and I don't see anything in the driver >> looking

<    1   2   3   4   5   6   7   8   9   10   >