Re: [PATCH net] net: mvpp2: allocate zeroed tx descriptors

From: Antoine Tenart Date: Thu, 30 Nov 2017 10:49:46 +0100 > From: Yan Markman > > Reserved and unused fields in the Tx descriptors should be 0. The PPv2 > driver doesn't clear them at run-time (for performance reasons) but > these descriptors aren't zeroed when allocated, which can lead to > u

Re: [PATCH] iw_cxgb4: make pointer reg_workq static

On Thu, Nov 30, 2017 at 01:30:06PM +, Colin King wrote: > From: Colin Ian King > > The pointer reg_workq is local to the source and does not need to be > in global scope, so make it static. > > Cleans up sparse warning: > drivers/infiniband/hw/cxgb4/device.c:69:25: warning: symbol 'reg_workq'

[PATCH] gpio: ftgpio010: Fix platform_get_irq's error checking

The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/gpio/gpio-ftgpio010.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: dd: page allocation failure: order:0, mode:0x1080020(GFP_ATOMIC), nodemask=(null)

On Thu, Nov 30, 2017 at 10:08:04PM +0800, Fengguang Wu wrote: [ 78.848629] dd: page allocation failure: order:0, mode:0x1080020(GFP_ATOMIC), nodemask=(null) [ 78.857841] dd cpuset=/ mems_allowed=0-1 [ 78.862502] CPU: 0 PID: 6131 Comm: dd Tainted: G O 4.15.0-rc1 #1 [ 78.870

[PATCH] chcr: remove unused variables net_device, pi, adap and cntrl

From: Colin Ian King Variables adap, pi and cntrl are assigned but are never read, hence they are redundant and can be removed. Cleans up various clang build warnings. Signed-off-by: Colin Ian King --- drivers/crypto/chelsio/chcr_ipsec.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletio

[PATCH] regmap: use proper part of work_buf for storing val

The map->work_buf is a buffer preallocated in __regmap_init() with size allowing it to store all 3 parts of a buffer - reg, pad and val. While reg and val parts are always properly setup before each transaction, the pad part is left at its default value (zeros). Until it is overwritten, that is.

Re: [PATCH] vfio/iommu_type1: report the IOMMU aperture info

On Thu, 30 Nov 2017 12:34:38 +0100 Pierre Morel wrote: > When userland VFIO defines a new IOMMU for a guest it may > want to specify to the guest the physical limits of > the underlying host IOMMU to avoid access to forbidden > memory ranges. > > Currently, the vfio_iommu_type1 driver does not r

Re: dd: page allocation failure: order:0, mode:0x1080020(GFP_ATOMIC), nodemask=(null)

[ 78.848629] dd: page allocation failure: order:0, mode:0x1080020(GFP_ATOMIC), nodemask=(null) [ 78.857841] dd cpuset=/ mems_allowed=0-1 [ 78.862502] CPU: 0 PID: 6131 Comm: dd Tainted: G O 4.15.0-rc1 #1 [ 78.870437] Call Trace: [ 78.873610] [ 78.876342] dump_stack+0x

[PATCH v2] list_lru: Prefetch neighboring list entries before acquiring lock

The list_lru_del() function removes the given item from the LRU list. The operation looks simple, but it involves writing into the cachelines of the two neighboring list entries in order to get the deletion done. That can take a while if the cachelines aren't there yet, thus prolonging the lock hol

Re: [PATCH, RESEND 1/2] dvb-frontends: fix i2c access helpers for KASAN

On Thu, Nov 30, 2017 at 1:49 PM, Mauro Carvalho Chehab wrote: >> Link: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81715 >> Signed-off-by: Arnd Bergmann >> --- >> I'm undecided here whether there should be a comment pointing >> to PR81715 for each file that the bogus local variable workaround >>

[PATCH v5] lib: optimize cpumask_next_and()

> So I think it really worth to be separated patch. Really, it's > completely nontrivial why adding new function in lib/find_bit.c > requires including asm-generic/bitops/find.h in arm and uncore32 > asm/bitops.h headers (bug?). And why doing that makes you guard > find_first_bit and find_first_zer

Re: dd: page allocation failure: order:0, mode:0x1080020(GFP_ATOMIC), nodemask=(null)

On Thu, Nov 30, 2017 at 02:50:16PM +0100, Michal Hocko wrote: On Thu 30-11-17 21:38:40, Wu Fengguang wrote: Hello, It looks like a regression in 4.15.0-rc1 -- the test case simply run a set of parallel dd's and there seems no reason to run into memory problem. It occurs in 1 out of 4 tests.

Re: [PATCH] rtc: add mxc driver for i.MX53

On Thu, Nov 30, 2017 at 2:36 PM, Patrick Brünn wrote: >>From: Philippe Ombredanne [mailto:pombreda...@nexb.com] >>Sent: Donnerstag, 30. November 2017 09:18 >>> +/* + * Copyright (C) 2004-2011 Freescale Semiconductor, Inc. All Rights >>Reserved. + */ + +/* + * The code

Re: [PATCH] x86/unwind_guess: Prevent using UNWINDER_GUESS=y with STACKDEPOT=y

On Thu, Nov 30, 2017 at 03:35:54PM +0300, Andrey Ryabinin wrote: > Stackdepot doesn't work well with CONFIG_UNWINDER_GUESS=y. > The 'guess' unwinder generate awfully large and inaccurate stacktraces, > thus stackdepot can't deduplicate stacktraces because they all look like > unique. Eventually sta

Re: WARNING: CPU: 1 PID: 185 at drivers/gpu/drm/i915/intel_display.c:14422 intel_modeset_init+0x3dc/0xf60 [i915]

Op 30-11-17 om 14:15 schreef Fengguang Wu: > Hello, > > This happens in mainline kernel v4.15-rc1 on LENOVO IdeaPad U410. > It at least dates back to v4.11 . > > It occurs in 72 out of 72 boots. Can I get a boot log with drm.debug=0x1f? I don't have enough information to see what's going on. :) ~

Re: [Xen-devel] [PATCH] Xen/pciback: Implement PCI slot or bus reset with 'do_flr' SysFS attribute

On 11/30/2017 2:29 AM, Jan Beulich wrote: On 29.11.17 at 20:44, wrote: So, we will use the following sequence to reset the requested device/function. - FLR (as first option) - BUS/SLOT reset (as fall-back option) if FLR is not supported or any issue with FLR It looks to me as if the slot re

Re: [PATCH] list_lru: Prefetch neighboring list entries before acquiring lock

On 11/29/2017 07:42 PM, Dave Chinner wrote: > On Wed, Nov 29, 2017 at 01:53:19PM -0800, Andrew Morton wrote: >> On Wed, 29 Nov 2017 09:17:34 -0500 Waiman Long wrote: >> >>> The list_lru_del() function removes the given item from the LRU list. >>> The operation looks simple, but it involves writing

Re: [PATCH] of: overlay: fix memory leak of ovcs on error exit path

Hi Dan, On Thu, Nov 30, 2017 at 1:50 PM, Dan Carpenter wrote: > On Thu, Nov 30, 2017 at 07:14:45AM -0500, Frank Rowand wrote: >> On 11/29/17 14:17, Colin King wrote: >> > From: Colin Ian King >> > >> > Currently if the call to of_resolve_phandles fails then then ovcs >> > is not kfree'd on the e

Re: dd: page allocation failure: order:0, mode:0x1080020(GFP_ATOMIC), nodemask=(null)

On Thu 30-11-17 21:38:40, Wu Fengguang wrote: > Hello, > > It looks like a regression in 4.15.0-rc1 -- the test case simply run a > set of parallel dd's and there seems no reason to run into memory problem. > > It occurs in 1 out of 4 tests. This is an atomic allocations. So the failure really d

Re: [PATCH v2] clk: stm32-h7: fix copyright

Benjamin: On Thu, Nov 30, 2017 at 2:05 PM, Benjamin Gaignard wrote: > 2017-11-30 13:56 GMT+01:00 Philippe Ombredanne : >> On Thu, Nov 30, 2017 at 1:47 PM, Benjamin Gaignard >> wrote: >>> 2017-11-30 12:28 GMT+01:00 Philippe Ombredanne : On Thu, Nov 30, 2017 at 9:41 AM, Benjamin Gaignard

[PATCH net-next 1/5] libbpf: add ability to guess program type based on section name

The bpf_prog_load() function will guess program type if it's not specified explicitly. This functionality will be used to implement loading of different programs without asking a user to specify the program type. In first order it will be used by bpftool. Signed-off-by: Roman Gushchin Cc: Alexei

[PATCH net-next 5/5] bpftool: implement cglist command

Implement cgattach command to list bpf progrrams attached to the given cgroup: Example: $ ./bpftool cgattach dev_cgroup.o /sys/fs/cgroup/user.slice/ device $ ./bpftool cglist /sys/fs/cgroup/user.slice/ ID AttachType Name 1device bpf_prog1 Signed-off-by: Roman Gushchin

Re: [PATCH] gpio: ath79: add missing MODULE_DESCRIPTION/LICENSE

On Mon, Nov 20, 2017 at 9:54 PM, Jesse Chan wrote: > This change resolves a new compile-time warning > when built as a loadable module: > > WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-ath79.o > see include/linux/module.h for more information > > This adds the license as "GPL v

[string_selftest_init] watchdog: BUG: soft lockup - CPU#0 stuck for 23s! [swapper:1]

Hello, FYI this happens in mainline kernel 4.15.0-rc1. It at least dates back to v4.14-rc1 . It occurs in 2 out of 3 boots. [ 107.032959] Asymmetric key parser 'x509' registered [ 107.058214] Block layer SCSI generic (bsg) driver version 0.4 loaded (major 250) [ 107.078095] io scheduler noop

[PATCH net-next 4/5] bpftool: implement cgdetach command

Implement cgdetach command, which allows to detach the bpf program from a cgroup. It takes program id and attach type as arguments. Example: $ ./bpftool cgdetach /sys/fs/cgroup/user.slice/ device 1 Signed-off-by: Roman Gushchin Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Jakub Kicinski Cc:

[PATCH net-next 3/5] bpftool: implement cgattach command

This patch add the cgattach command to bpftool. It allows to load a bpf program from a binary file and attach it to a given cgroup. Example: $ bpftool cgattach ./mybpfprog.o /sys/fs/cgroup/user.slice/ ingress Signed-off-by: Roman Gushchin Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Jakub Ki

[PATCH net-next 0/5] bpftool: cgroup bpf operations

This patchset adds basic cgroup bpf operations to bpftool. Right now there is no convenient way to perform these operations. The /samples/bpf/load_sock_ops.c implements attach/detacg operations, but only for BPF_CGROUP_SOCK_OPS programs. Bps (part of bcc) implements bpf introspection, but lacks an

[PATCH net-next 2/5] libbpf: prefer global symbols as bpf program name source

Libbpf picks the name of the first symbol in the corresponding elf section to use as a program name. But without taking symbol's scope into account it may end's up with some local label as a program name. E.g.: $ bpftool cglist /sys/fs/cgroup/system.slice/tmp.mount/ 16 device LBB0_1

Re: [PATCH] list_lru: Prefetch neighboring list entries before acquiring lock

On 11/29/2017 07:53 PM, Minchan Kim wrote: > Hello, > > On Wed, Nov 29, 2017 at 09:17:34AM -0500, Waiman Long wrote: >> The list_lru_del() function removes the given item from the LRU list. >> The operation looks simple, but it involves writing into the cachelines >> of the two neighboring list ent

Re: [PATCH] rtc: add mxc driver for i.MX53

Hi, On 30/11/2017 at 13:36:22 +, Patrick Brünn wrote: > >From: Philippe Ombredanne [mailto:pombreda...@nexb.com] > >Sent: Donnerstag, 30. November 2017 09:18 > >> > >>> +/* > >>> + * Copyright (C) 2004-2011 Freescale Semiconductor, Inc. All Rights > >Reserved. > >>> + */ > >>> + > >>> +/* > >>

Re: [PATCH] pinctrl: pxa: pxa2xx: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

On Mon, Nov 20, 2017 at 9:58 PM, Jesse Chan wrote: > This change resolves a new compile-time warning > when built as a loadable module: > > WARNING: modpost: missing MODULE_LICENSE() in > drivers/pinctrl/pxa/pinctrl-pxa2xx.o > see include/linux/module.h for more information > > This adds the lic

[PATCH] MAINTAINERS: Remove bogus wildcards for CHAR and MISC DRIVERS

With the wildcards present, this entry does not cover subdirectories. Signed-off-by: Geert Uytterhoeven --- MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 4737de9f41bff570..b888d6f241e6f1d9 100644 --- a/MAINTAINERS +++ b/MAINTA

Re: [PATCH v3 6/6] cpufreq: schedutil: ignore sugov kthreads

Hi, On 30/11/17 11:47, Patrick Bellasi wrote: > In system where multiple CPUs shares the same frequency domain a small > workload on a CPU can still be subject to frequency spikes, generated by > the activation of the sugov's kthread. > > Since the sugov kthread is a special RT task, which goal i

[PATCH 1/3] clk: mvebu: armada-37xx-periph: cosmetic changes

This patches fixes few cosmetic issues such as alignment, blank lines and required space. Signed-off-by: Gregory CLEMENT --- drivers/clk/mvebu/armada-37xx-periph.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/driv

[PATCH 3/3] clk: mvebu: armada-37xx-periph: add DVFS support for cpu clocks

When DVFS is enabled the CPU clock setting is done using an other set of registers. These Power Management registers are exposed through a syscon as they will also be used by other drivers such as the cpufreq. This patch add the possibility to modify the CPU frequency using the associate load lev

[PATCH 2/3] clk: mvebu: armada-37xx-periph: prepare cpu clk to be used with DVFS

When DVFS will be enabled then the cpu clk will use a different set of register at run time. That means that we won't be able to use the common callback and need to use our own ones. This patch prepares this change by switching on our own set of callbacks without modifying the behavior of the cloc

Re: [PATCH] gpio: iop: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

On Mon, Nov 20, 2017 at 9:54 PM, Jesse Chan wrote: > This change resolves a new compile-time warning > when built as a loadable module: > > WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-iop.o > see include/linux/module.h for more information > > This adds the license as "GPL", w

[PATCH 0/3] Add DVFS support on CPU clock for Armada 37xx

Hi, This small series is needed to use DVFS on Armada 37xx. When DVFS is enabled the CPU clock setting is done using an other set of registers from the North Bridge Power Management block. The series adds the possibility to modify the CPU frequency using the associate load level matching the targ

Re: [PATCH 1/6] perf: Add new type PERF_TYPE_PROBE

On Thu, Nov 30, 2017 at 01:43:06AM +, Song Liu wrote: > I added two fixed types (PERF_TYPE_KPROBE and PERF_TYPE_UPROBE) in the new > version. I know that perf doesn't need them any more. But currently bcc still > relies on these fixed types to use the probes/tracepoints. Yeah, sorry, that's

Re: [PATCH] of: overlay: fix memory leak of ovcs on error exit path

Hi Colin, Rob, On 11/30/17 07:18, Colin Ian King wrote: > On 30/11/17 12:14, Frank Rowand wrote: >> On 11/29/17 14:17, Colin King wrote: >>> From: Colin Ian King >>> >>> Currently if the call to of_resolve_phandles fails then then ovcs >>> is not kfree'd on the error exit path. Rather than try a

[PATCH/trivial] ledtrig-activity: Grammar s/a immediate/an immediate/

Fixes: 7df4f9a9f0667ee6 ("leds: ledtrig-activity: Add a system activity LED trigger") Signed-off-by: Geert Uytterhoeven --- drivers/leds/trigger/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/leds/trigger/Kconfig b/drivers/leds/trigger/Kconfig index bb090216b

RE: [PATCH] rtc: add mxc driver for i.MX53

>From: Philippe Ombredanne [mailto:pombreda...@nexb.com] >Sent: Donnerstag, 30. November 2017 09:18 >> >>> +/* >>> + * Copyright (C) 2004-2011 Freescale Semiconductor, Inc. All Rights >Reserved. >>> + */ >>> + >>> +/* >>> + * The code contained herein is licensed under the GNU General Public >>> +

Re: [PATCH v3 5/6] cpufreq: schedutil: relax rate-limiting while running RT/DL tasks

Hi, On 30/11/17 11:47, Patrick Bellasi wrote: > The policy in use for RT/DL tasks sets the maximum frequency when a task > in these classes calls for a cpufreq_update_util(). However, the > current implementation is still enforcing a frequency switch rate > limiting when these tasks are running.

[PATCH/trivial] spi: Fix double "when"

Signed-off-by: Geert Uytterhoeven --- include/linux/spi/spi.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h index 7b2170bfd6e7dae4..bc6bb325d1bf7c03 100644 --- a/include/linux/spi/spi.h +++ b/include/linux/spi/spi.h @@ -126,7

Re: [PATCH v18 05/10] xbitmap: add more operations

Tetsuo Handa wrote: > > + > > + if (ebit >= BITS_PER_LONG) > > + continue; > > (I don't understand how radix tree works, but generally this patchset looks > fuzzy > to me about boundary cases. Thus, I want to confirm that this is not an > overlook.) >

[PATCH/trivial] skbuff: Grammar s/are can/can/, s/change/changes/

Signed-off-by: Geert Uytterhoeven --- include/linux/skbuff.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index bc486ef23f20f91c..a38c80e9f91efee0 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h @@ -1406,8 +

[PATCH v6] mfd: syscon: Add hardware spinlock support

Some system control registers need hardware spinlock to synchronize between the multiple subsystems, so we should add hardware spinlock support for syscon. Signed-off-by: Baolin Wang Acked-by: Rob Herring --- Changes since v5: - Fix the case that hwspinlock is not enabled. Changes since v4: -

RE: IT Service Desk Support

Dear Colleague, Note that Microsoft Outlook Windows Updates is scheduled for today and installed on all users' workstations. Please Click here to Updates your account, Your device will prompt to restart after the updat

[PATCH/trivial] samsung-laptop: Grammar s/are can/can/

Signed-off-by: Geert Uytterhoeven --- Documentation/ABI/testing/sysfs-driver-samsung-laptop | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/ABI/testing/sysfs-driver-samsung-laptop b/Documentation/ABI/testing/sysfs-driver-samsung-laptop index 63c1ad0212fc8624..34

Re: [PATCH V1] drivers:mtd:spi-nor:checkup FSR error bits

Hi Bean, Le 11/11/2017 à 21:49, Bean Huo (beanhuo) a écrit : > For the Micron SPI NOR, when the erase/program operation fails, especially, To be verified but I think you'd rather remove "the" words in this case: "For Micron SPI NOR memories, when erase/program operation fails, especially ..."

Re: [PATCH v2 2/2] pinctrl: meson-axg: Add new pinctrl driver for Meson AXG SoC

On Mon, Nov 20, 2017 at 11:08 AM, Yixun Lan wrote: > From: Xingyu Chen > > Add new pinctrl driver for Amlogic's Meson-AXG SoC. > > Reviewed-by: Neil Armstrong > Signed-off-by: Xingyu Chen > Signed-off-by: Yixun Lan Patch applied with the ACKs. Nice work! Yours, Linus Walleij

[PATCH 0/3] eeprom: at25: Add DT support for 25lc040

Hi all, Some "atmel,at25" compatible SPI EEPROMs (e.g. Microchip 25lc040) use an odd number of address bits. This patch series adds support for instantiating such devices from DT. Do EEPROMs using 17 or 25 address bits, as mentioned in include/linux/spi/eeprom.h, really exist? Or should

[PATCH 1/3] eeprom: at25: Add DT support for EEPROMs with odd address bits

Certain EEPROMS have a size that is larger than the number of address bytes would allow, and store the MSB of the address in bit 3 of the instruction byte. This can be described in platform data using EE_INSTR_BIT3_IS_ADDR, or in DT using the obsolete legacy "at25,addr-mode" property. But currentl

[PATCH 3/3] dt-bindings: eeprom: at25: Document device-specific compatible values

Document the recommended presence of a device-specific compatible value, and list examples that are already in use or soon will be. This will allow checkpatch to validate compatible values in DTS. Update the example to match current best practices (generic node name, specific compatible value firs

[PATCH] iw_cxgb4: make pointer reg_workq static

From: Colin Ian King The pointer reg_workq is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: drivers/infiniband/hw/cxgb4/device.c:69:25: warning: symbol 'reg_workq' was not declared. Should it be static? Signed-off-by: Colin Ian King -

[PATCH 2/3] dt-bindings: eeprom: at25: Grammar s/are can/can/

Signed-off-by: Geert Uytterhoeven --- Documentation/devicetree/bindings/eeprom/at25.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/eeprom/at25.txt b/Documentation/devicetree/bindings/eeprom/at25.txt index d00779e4ab4377b9..9b178280938684

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

Hi, On 30/11/17 11:47, Patrick Bellasi wrote: > Currently schedutil updates are triggered for the RT class using a single > call place, which is part of the rt::update_curr_rt() used in: > > - dequeue_task_rt: > but it does not make sense to set the schedutil's SCHED_CPUFREQ_RT in > case the

Re: [PATCH v2 1/2] pinctrl: meson-axg: Introduce a pinctrl pinmux ops for Meson-AXG SoC

On Mon, Nov 20, 2017 at 11:08 AM, Yixun Lan wrote: > From: Xingyu Chen > > The pin controller has been updated in the Amlogic Meson AXG series, > which use continuous 4-bit register to select function for each pin. > In order to support this, a new pinmux operations "meson_axg_pmx_ops" > has bee

Re: [EXT] Re: [PATCH net] net: phylink: fix link state on phy-connect

On Thu, Nov 30, 2017 at 10:10:18AM +, Russell King - ARM Linux wrote: > On Thu, Nov 30, 2017 at 08:51:21AM +, Yan Markman wrote: > > The phylink_stop is called before phylink_disconnect_phy > > You could see in mvpp2.c: > > > > mvpp2_stop_dev() { > > phylink_stop(port->phylink); > > }

Re: [PATCH v2 1/2] documentation: Add compatibles for Amlogic Meson AXG pin controllers

On Mon, Nov 20, 2017 at 11:08 AM, Yixun Lan wrote: > From: Xingyu Chen > > Add compatibles for Amlogic Meson AXG pin controllers > > Reviewed-by: Neil Armstrong > Signed-off-by: Xingyu Chen > Signed-off-by: Yixun Lan Patch applied with Rob's ACK. Yours, Linus Walleij

Re: [PATCH net,stable v2] vhost: fix skb leak in handle_rx()

On Thu, Nov 30, 2017 at 10:46:17AM +0800, Jason Wang wrote: > > > On 2017年11月29日 23:31, Michael S. Tsirkin wrote: > > On Wed, Nov 29, 2017 at 09:23:24AM -0500,w...@redhat.com wrote: > > > From: Wei Xu > > > > > > Matthew found a roughly 40% tcp throughput regression with commit > > > c67df11f(v

[PATCH v2 06/10] trace-cmd: Extr. profile-specific code from record_trace

This patch extracts the code in record_trace() under if (IS_PROFILE(ctx)): now that profile has its own function, that code could be moved there after removing the IS_PROFILE(ctx) checks, clearly. Signed-off-by: Vladislav Valtchev (VMware) --- trace-record.c | 18 -- 1 file chang

[PATCH v2 04/10] trace-cmd: Rename trace_profile() to do_trace_profile()

The purpose of this renaming is to make room for a new trace_profile() function, that will handle directly the 'profile' command, following the internal convention COMMAND_NAME -> trace_{COMMAND_NAME}. Clearly, in the next steps the top-level trace_profile() function will be made to call do_trace_

[PATCH v2 05/10] trace-cmd: Making start,extract,stream,profile separate funcs

This simple patch make the above-mentioned commands independent from 'record'. The point of doing so is to follow the convention of one entry-point per command that is followed by most of trace-cmd's code. Ultimately that aims to prevent the complexity to concentrate in a single point, making the c

[PATCH v2 02/10] trace-cmd: Replacing cmd flags w/ a trace_cmd enum

This patch replaces 5 mutually exclusive flag variables (extract, start, stream, record, profile) in trace_record() with a more convenient enum. The point of doing that is to make the code simpler and easier to maintain. Signed-off-by: Vladislav Valtchev (VMware) --- trace-record.c | 94

Re: [PATCH v3 3/6] cpufreq: schedutil: update CFS util only if used

Hi, On 30/11/17 11:47, Patrick Bellasi wrote: > Currently the utilization of the FAIR class is collected before locking > the policy. Although that should not be a big issue for most cases, we > also don't really know how much latency there can be between the > utilization reading and its usage. >

Re: [PATCH] vfio/iommu_type1: report the IOMMU aperture info

Hi Pierre, On 30/11/17 12:34, Pierre Morel wrote: > When userland VFIO defines a new IOMMU for a guest it may > want to specify to the guest the physical limits of > the underlying host IOMMU to avoid access to forbidden > memory ranges. > > Currently, the vfio_iommu_type1 driver does not report t

[PATCH v2 01/10] trace-cmd: Extract parse_record_options() from trace_record()

In this patch a huge part of trace_record(), dealing with parsing the command line options, has been extracted in a separate function. That allows the body of trace_record() to be focused only on the proper actions involved in a given type of tracing (record, stream, etc.) reducing, this way, the s

[PATCH v2 07/10] trace-cmd: Mov init_common_record_context in parse_record_options

This short patch moves init_common_record_context() into parse_record_option() after checking that no trace_* function in trace-record.c really needs to do work after init_common_record_context() but before parse_record_option(). In particular, two statements in trace_profile() have been moved afte

Re: [PATCH v2] PM / Domains: Remove obsolete "samsung,power-domain" check

On 30 November 2017 at 12:54, Geert Uytterhoeven wrote: > Currently the generic PM Domain code code checks for the presence of > both (generic) "power-domains" and (Samsung Exynos legacy) > "samsung,power-domain" properties in all device tree nodes representing > devices. > > There are two issues

Re: [PATCH v4] component: add debugfs support

On 11/30/2017 01:54 PM, Russell King - ARM Linux wrote: On Thu, Nov 30, 2017 at 01:48:32PM +0100, Maciej Purski wrote: Currently there is no information in any vfs about which devices a master component consists of, what makes debugging hard if one of the component devices fails to register.

[PATCH v2 08/10] trace-cmd: Introducing get_trace_cmd_type()

This patch aims to reduce the size of common_record_commads_code() by removing a relatively long 'else if' sequence that was used to do the mapping between the current trace command and the trace_type used by it. Signed-off-by: Vladislav Valtchev (VMware) --- trace-record.c | 34

[PATCH v2 10/10] trace-cmd: Fork record_trace() for the extract case

This patch inlines record_trace() into trace_extract() by removing the code not related to extract, by replacing IS_EXTRACT(ctx) with true and then removing the dead code, as well as the if statements when their condition always evalutates to true. The opposite change [IS_EXTRACT(ctx) evaluated as

[PATCH v2 03/10] trace-cmd: Extracting record_trace()

This patch moves in a separate function almost all the code in trace_record() after the call of parse_record_options(). This is the last necessary preparation step before removing the command-multiplexing code from trace_record and allowing the commands 'start', 'extract', 'stream' and 'profile' to

[PATCH v2 09/10] trace-cmd: Extract finalize_record_trace()

This patch splits record_trace() in two parts by moving its finalization part in a separate function. This will also allow splitting out trace-cmd extract code from trace-cmd record code, by using a shared function. Signed-off-by: Vladislav Valtchev (VMware) --- trace-record.c | 48 +

[PATCH v2 00/10] trace-cmd: Refactoring trace_record()

The following series of patches is a refactoring of trace_record() trying to reduce as much as possible its complexity but, at the same time, without making risky changes. All the patches are relatively small and potentially no-brainer steps towards the final shape of the code. The rationale for t

Re: [RFC] bpf: offload: report device information for offloaded programs

Hi, Jakub, please, read comments below. On 30.11.2017 03:22, Jakub Kicinski wrote: > Report to the user ifindex and namespace information of offloaded > programs. Always set dev_bound to true if program was loaded for > a device which has been since removed. Specify the namespace > using dev/in

Re: KASAN: use-after-free Read in sock_release

On Thu, Nov 30, 2017 at 02:07:19AM +, Al Viro wrote: > Incidentally, grepping for sys_close() shows another piece of fun in > net/netfilter/xt_bpf.c. Folks, ONCE DESCRIPTOR IS INSTALLED, THAT'S > IT; THERE'S NO REMOVING IT ON FAILURE EXITS. sys_close() should > never, ever be used that way.

Re: [PATCH v3 2/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

Hi, On 30/11/17 11:47, Patrick Bellasi wrote: [...] > @@ -340,6 +349,7 @@ static void sugov_update_shared(struct update_util_data > *hook, u64 time, > struct sugov_policy *sg_policy = sg_cpu->sg_policy; > unsigned long util, max; > unsigned int next_f; > + bool rt_mode; >

Re: module: add debugging alias parsing support

+++ Luis R. Rodriguez [29/11/17 18:36 -0800]: Debugging modules can often lead to an alias question. We purposely don't have alias parsing support upstream as this is all dealt with in userpace with the assumption that in-kernel we just process aliases and userspace Does The Right Thing (TM) abou

Re: [PATCH 6/6] x86/mm/kaiser: Optimize __native_flush_tlb

On Thu, Nov 30, 2017 at 01:43:19PM +0100, Peter Zijlstra wrote: > Now the problem is that flush_tlb_kernel_range() is implemented using > either __flush_tlb_all() or __flush_tlb_single(), and it is that last > use that is buggered. > > So at the very least we need the below to cure things, but the

Re: [PATCH] usb: dwc2: gadget: fix dwc2_check_param_tx_fifo_sizes

Hi Minas, On 11/30/2017 10:43 AM, Minas Harutyunyan wrote: > Hi Amelie, > > On 11/27/2017 6:42 PM, Amelie Delaunay wrote: >> In case of OTG mode, with an OTG adapter plugged, the >> dwc2_check_param_tx_fifo_sizes function reads DPTXFSIZN registers while >> the controller is in Host mode, and, bec

Re: [PATCH v3 1/6] cpufreq: schedutil: reset sg_cpus's flags at IDLE enter

Hi, On 30/11/17 11:47, Patrick Bellasi wrote: [...] > diff --git a/kernel/sched/cpufreq_schedutil.c > b/kernel/sched/cpufreq_schedutil.c > index 2f52ec0f1539..67339ccb5595 100644 > --- a/kernel/sched/cpufreq_schedutil.c > +++ b/kernel/sched/cpufreq_schedutil.c > @@ -347,6 +347,12 @@ static void

Re: [PATCH v2 0/3] KVM: x86: kvm_mwait_in_guest() cleanup

On Wed, Nov 29, 2017 at 10:23:40PM +0100, Radim Krčmář wrote: > This is a rebased version of an old series that simplified > kvm_mwait_in_guest: https://www.spinics.net/lists/kvm/msg149238.html > > AMD errata 400 patch was dropped thanks to Boris's review; > [2/3] got an expanded commit message an

Re: [PATCH] mmc_test: use ktime_get_ts64 for timestamps

On 27 November 2017 at 12:53, Arnd Bergmann wrote: > Calling getnstimeofday() can suffer from time jumps and from the > y2038 overflow, so it is not appropriate here. Using ktime_get_ts64() > solves both problems. Using ktime_get() with ktime_t timestamps would > also work, but it seems that we ma

Re: [PATCH] mmc: block: make function mmc_cqe_issue_type static

On 30 November 2017 at 12:37, Colin King wrote: > From: Colin Ian King > > The function mmc_cqe_issue_type is local to the source and does > not need to be in global scope, so make it static. > > Cleans up sparse warning: > drivers/mmc/core/queue.c:62:21: warning: symbol 'mmc_cqe_issue_type' > wa

Re: [PATCH] mmc: sdhci-of-arasan: Add sdhci_arasan_set_power

On 28 November 2017 at 01:02, Milan Stevanovic wrote: > The power register needs to have a valid voltage set > even when the power supply is managed by an external regulator. > > Signed-off-by: Milan Stevanovic Thanks, applied for next! Kind regards Uffe > --- > drivers/mmc/host/sdhci-of-ara

RE: [PATCH v2] MIPS: Add nonxstack=on|off kernel parameter

Hi James, > > We do have PT_GNU_STACK flags set correctly, this feature is required to > > workaround CPU revisions which do not have RIXI support. > > RIXI support can be discovered programatically from CP0_Config3.RXI > (cpu_has_rixi in asm/cpu-features.h), so I don't follow why CPUs without >

[PATCH 3/3, V2] kernel: set_groups doesn't call groups_sort anymore.

When the group_info is cached (e.g. sunrpc) there's the possibility that threads calling set_groups will attempt to do so simultaneously. Moving the responsibility of sorting to the caller of set_groups, or in the case of nfsd, to the point where it is received from rpc.mountd avoids this issue.

[PATCH 1/3, V2] kernel: make groups_sort globally visible

In preparation to move group_info sorting to the caller, make group_sort globally visible. Signed-off-by: Thiago Rafael Becker --- include/linux/cred.h | 1 + kernel/groups.c | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/include/linux/cred.h b/include/linux/cred.h

[PATCH 0/3, V2] Move groups_sort outisde of set_groups

In cases where group_info is cached (e.g. sunrpc), multiplpe threads may call set_groups with a freshly created group_info cache (e.g. nfsd), and attempt to sort them simultaneously, which configures a race condition that can overwrite some groups in the cache and lead to errors. In the case of nfs

[PATCH 2/3, V2] kernel: Move groups_sort to the caller of set_groups.

The responsibility for calling groups_sort is now on the caller of set_groups. Signed-off-by: Thiago Rafael Becker --- kernel/groups.c | 1 + kernel/uid16.c| 1 + net/sunrpc/svcauth_unix.c | 7 +++ 3 files changed, 9 insertions(+) diff --git a/kernel/groups.c b/kernel/

Re: [BUILD FAILURE] nfs4state.c fails to compile with gcc 4.5.4

On 11/30/2017 06:46 AM, Trond Myklebust wrote: > On Wed, 2017-11-29 at 23:05 -0500, Steven Rostedt wrote: >> On Wed, 29 Nov 2017 19:44:33 -0800 >> Linus Torvalds wrote: >> >> I keep older compilers around to test ftrace before -mfentry was >> introduced. >> >> I wonder if I should just add a con

Re: [PATCH v2] clk: stm32-h7: fix copyright

2017-11-30 13:56 GMT+01:00 Philippe Ombredanne : > On Thu, Nov 30, 2017 at 1:47 PM, Benjamin Gaignard > wrote: >> 2017-11-30 12:28 GMT+01:00 Philippe Ombredanne : >>> On Thu, Nov 30, 2017 at 9:41 AM, Benjamin Gaignard >>> wrote: Uniformize STMicroelectronics copyrights header Add SPDX i

Re: [ 0.003333] BUG: KASAN: use-after-scope in console_unlock+0x605/0xcc0

On 11/30/2017 11:29 AM, Sergey Senozhatsky wrote: > On (11/30/17 09:16), Dmitry Vyukov wrote: > [..] >>> to be honest, this backtrace hardly makes any sense to me. >>> >>> vprintk_emit() >>> reserve_standard_io_resources() >>> __flush_tlb_all() >>>vprintk_emit() >>> __down_trylock_cons

[PATCH] ARM: dts: imx6sx-sdb: Add DRM panel, power-enable and backlight support

It is preferred to use the panel compatible string rather than passing the LCD timming in the device tree. So pass the "sii,43wvf1g" compatible string, a new driver for the Seiko Panel, which describes the parallel LCD. This new driver needs two power supply, so pass 'dvdd-supply' and 'avdd-su

[PATCH] ARM: dts: imx6sl-evk: Add DRM panel, power-enable and backlight support

From: Marco Frank It is preferred to use the panel compatible string rather than passing the LCD timming in the device tree. So pass the "sii,43wvf1g" compatible string, a new driver for the Seiko Panel, which describes the parallel LCD. This new driver needs two power supply, so pass 'dvdd-s

Re: general protection fault in show_timer

+syzbot back, please don't drop it, it's reading On Thu, Nov 30, 2017 at 1:20 PM, Alexey Dobriyan wrote: > [cc security@] > 100% oops with interrupts disabled by nobody > or kernel memory read > > > On 11/30/17, Dmitry Vyukov wrote: >> On Thu, Nov 30, 2017 at 12:31 PM, Dmitry Vyukov wrote: >>>

Re: general protection fault in show_timer

On Thu, 30 Nov 2017, Alexey Dobriyan wrote: > [cc security@] > 100% oops with interrupts disabled by nobody > or kernel memory read > [nods] > you named the bug already > > "notify" directly comes from userspace struct sigevent::sigev_notify > without adult supervision. > > Reproducer is timer_c

Re: [PATCH v2] clk: stm32-h7: fix copyright

On Thu, Nov 30, 2017 at 1:47 PM, Benjamin Gaignard wrote: > 2017-11-30 12:28 GMT+01:00 Philippe Ombredanne : >> On Thu, Nov 30, 2017 at 9:41 AM, Benjamin Gaignard >> wrote: >>> Uniformize STMicroelectronics copyrights header >>> Add SPDX identifier >>> >>> Signed-off-by: Benjamin Gaignard >>> Ac

<    4   5   6   7   8   9   10   11   12   >