Re: [PATCH 5/9] perf utils: add support for arch standard events

On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: SNIP > rc = nftw(ldirname, process_one_file, maxfds, 0); > if (rc && verbose) { > pr_info("%s: Error walking file tree %s\n", prog, ldirname); > - goto empty_map; > + goto free_standard_a

Re: [PATCH 5/9] perf utils: add support for arch standard events

On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: SNIP > /* Call func with each event in the json file */ > int json_events(const char *fn, > int (*func)(void *data, char *name, char *event, char *desc, > @@ -442,6 +550,7 @@ int json_events(const char *fn, > char

Re: [PATCH 5/9] perf utils: add support for arch standard events

On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: SNIP > > +static int is_json_file(const char *name) > +{ > + const char *suffix; > + > + if (strlen(name) < 5) > + return 0; > + > + suffix = name + strlen(name) - 5; > + > + if (strncmp(suffix, ".json", 5)

Re: [PATCH 5/9] perf utils: add support for arch standard events

On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: SNIP > + char *perpkg; > + char *unit; > + char *metric_expr; > + char *metric_name; > + char *metric_group; > + struct list_head list; > + char strings[]; > +}; > + > +static LIST_HEAD(arch_std_events); > + >

Re: x86/microcode/intel: Division by zero panic in 4.9.79 and 4.4.114

On Wed, Feb 7, 2018 at 6:12 PM, Borislav Petkov wrote: > On Wed, Feb 07, 2018 at 04:31:59PM +, Rolf Neugebauer wrote: >> arch/x86/kernel/cpu/microcode/core.c | 28 +++- >> 1 file changed, 19 insertions(+), 9 deletions(-) >> >> diff --git a/arch/x86/kernel/cpu/microcode

Re: [PATCH 1/9] perf vendor events: drop incomplete multiple mapfile support

On Wed, Feb 07, 2018 at 01:44:56AM +0800, John Garry wrote: > Currently jevents supports multiple mapfiles, but this > is only in the form where mapfile basename starts with > 'mapfile.csv' > > At the moment, no architectures actually use multiple mapfiles, > so drop the support for now. > > This

Re: [PATCH 5/9] perf utils: add support for arch standard events

On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: > For some architectures (like arm), there are architecture- > defined events. Sometimes these events may be "recommended" > according to the architecture standard, in that the > implementer is free ignore the "recommendation" and create >

Re: possible deadlock in rtnl_lock (4)

On Thu, Feb 8, 2018 at 9:25 PM, Dmitry Vyukov wrote: > On Thu, Feb 8, 2018 at 10:54 AM, Xin Long wrote: >> On Thu, Feb 8, 2018 at 6:58 AM, syzbot >> wrote: >>> Hello, >>> >>> syzbot hit the following crash on upstream commit >>> a2e5790d841658485d642196dbb0927303d6c22f (Wed Feb 7 06:15:42 2018 +

Re: [PATCH 2/9] perf utils: add support for pmu events vendor sub-directory

On Wed, Feb 07, 2018 at 01:44:57AM +0800, John Garry wrote: > For some architectures (like arm), it is required to support > a vendor sub-directory and not locate all the JSONs for a > specific vendor in the same folder. > > This is because all the events for the same vendor will be > placed in th

Re: [PATCH] scripts/coccinelle/misc/shift.cocci: detect < or > that should be a shift

On Thu, Feb 08, 2018 at 01:53:54PM +0100, Julia Lawall wrote: > This checks for a comparison using < or > between two constants, > considering both explicit constants (1, 2, etc) and macros defined > with #define. False positives are possible in the latter case, when > a macro may have multiple po

Re: [PATCH] scripts/coccinelle/misc/shift.cocci: detect < or > that should be a shift

On Thu, 8 Feb 2018, Dan Carpenter wrote: > On Thu, Feb 08, 2018 at 01:53:54PM +0100, Julia Lawall wrote: > > This checks for a comparison using < or > between two constants, > > considering both explicit constants (1, 2, etc) and macros defined > > with #define. False positives are possible in

Re: [PATCH] ACPI/IORT: Remove linker section for IORT entries again

On Thu, Feb 08, 2018 at 10:37:54AM +0800, Jia He wrote: > > > On 2/7/2018 7:41 PM, Lorenzo Pieralisi Wrote: > >On Tue, Feb 06, 2018 at 08:11:34PM -0800, Jia He wrote: > >>In commit 316ca8804ea8 ("ACPI/IORT: Remove linker section for IORT entries > >>probing"), iort entries was removed in vmlinux.

Re: [PATCH v2 1/3] sched: Stop nohz stats when decayed

On Tue, Feb 06, 2018 at 08:23:05PM +0100, Vincent Guittot wrote: > @@ -9207,13 +9231,15 @@ void nohz_balance_enter_idle(int cpu) > if (!housekeeping_cpu(cpu, HK_FLAG_SCHED)) > return; > > + rq->has_blocked_load = 1; > + > if (rq->nohz_tick_stopped) > - r

Re: [RFC][PATCH 1/6] ftrace: Remove incorrect setting of glob search field

On Tue, 06 Feb 2018 17:43:39 -0500 Steven Rostedt wrote: > From: "Steven Rostedt (VMware)" > > __unregister_ftrace_function_probe() will incorrectly parse the glob filter > because it resets the search variable that was setup by filter_parse_regex(). > > Al Viro reported this: > > After t

Re: [RFC][PATCH 2/6] tracing: Fix parsing of globs with a wildcard at the beginning

On Tue, 06 Feb 2018 17:43:40 -0500 Steven Rostedt wrote: > From: "Steven Rostedt (VMware)" > > Al Viro reported: > > For substring - sure, but what about something like "*a*b" and "a*b"? > AFAICS, filter_parse_regex() ends up with identical results in both > cases - MATCH_GLOB and

[PATCH v3] xenbus: track caller request id

Commit fd8aa9095a95 ("xen: optimize xenbus driver for multiple concurrent xenstore accesses") optimized xenbus concurrent accesses but in doing so broke UABI of /dev/xen/xenbus. Through /dev/xen/xenbus applications are in charge of xenbus message exchange with the correct header and body. Now, afte

Re: [PATCH 01/12] i2c: qup: fixed releasing dma without flush operation completion

Hi Abhishek, On 2/3/2018 1:28 PM, Abhishek Sahu wrote: > The QUP BSLP BAM generates the following error sometimes if the > current I2C DMA transfer fails and the flush operation has been > scheduled > > “bam-dma-engine 7884000.dma: Cannot free busy channel” > > If any I2C error comes during

Re: [PATCH v2 1/3] sched: Stop nohz stats when decayed

On Thu, Feb 08, 2018 at 03:00:05PM +0100, Peter Zijlstra wrote: > On Tue, Feb 06, 2018 at 08:23:05PM +0100, Vincent Guittot wrote: > > > @@ -9207,13 +9231,15 @@ void nohz_balance_enter_idle(int cpu) > > if (!housekeeping_cpu(cpu, HK_FLAG_SCHED)) > > return; > > > > + rq->has_bl

Re: [PATCH 5/9] perf utils: add support for arch standard events

On Thu, 8 Feb 2018 14:54:23 +0100 Jiri Olsa wrote: > On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: > > SNIP > > > static void print_events_table_suffix(FILE *outfp) > > { > > fprintf(outfp, "{\n"); > > @@ -407,6 +469,52 @@ static char *real_event(const char *name, char *even

Re: [PATCH 1/2] trace-cmd: Fix the detection for swig

On Thu, 2018-02-08 at 20:05 +0800, Zamir SUN wrote: > > Hi Vladislav, > > Thanks for the URL. I tried to build from your make8 branch, it works > fine. I must made some copy-paste mistake yesterday. > > Sorry for the noise. > > Thanks. No problem, Zamir. I'm happy that it works. The series ha

Re: [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY

Forgot Cc'ing me ? :) On Thu, Feb 8, 2018 at 7:18 PM, Leo Yan wrote: > Since schedutil kernel thread directly set priority to 0, the macro > SUGOV_KTHREAD_PRIORITY is not used. So remove it. > > Cc: Vikram Mulukutla > Cc: Vincent Guittot > Cc: Daniel Lezcano > Signed-off-by: Leo Yan > --- >

Re: [PATCH 2/8] thermal/drivers/cpu_cooling: Add Software Package Data Exchange (SPDX)

Daniel, On Tue, Jan 23, 2018 at 4:34 PM, Daniel Lezcano wrote: > For license auditing purpose, let's add the SPDX tag. > > Cc: Philippe Ombredanne > Signed-off-by: Daniel Lezcano > --- > drivers/thermal/cpu_cooling.c | 16 +--- > 1 file changed, 1 insertion(+), 15 deletions(-) > >

Re: [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY

On 08/02/2018 14:48, Leo Yan wrote: > Since schedutil kernel thread directly set priority to 0, the macro > SUGOV_KTHREAD_PRIORITY is not used. So remove it. > > Cc: Vikram Mulukutla > Cc: Vincent Guittot > Cc: Daniel Lezcano > Signed-off-by: Leo Yan Acked-by: Daniel Lezcano --

Re: [PATCH 2/8] thermal/drivers/cpu_cooling: Add Software Package Data Exchange (SPDX)

On 08/02/2018 15:05, Philippe Ombredanne wrote: > Daniel, [ ... ] > Sorry for the late reply! > Thank you. > Acked-by: Philippe Ombredanne No worries. Thanks for the ack. -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro:

Re: [RFC][PATCH 5/6] selftests/ftrace: Add some missing glob checks

On Tue, 06 Feb 2018 17:43:43 -0500 Steven Rostedt wrote: > From: "Steven Rostedt (VMware)" > > Al Viro discovered a bug in the glob ftrace filtering code where "*a*b" is > treated the same as "a*b", and functions that would be selected by "*a*b" > but not "a*b" are not selected with "*a*b". >

Re: INFO: task hung in sync_blockdev

On Thu 08-02-18 14:28:08, Dmitry Vyukov wrote: > On Thu, Feb 8, 2018 at 10:28 AM, Jan Kara wrote: > > On Wed 07-02-18 07:52:29, Andi Kleen wrote: > >> > #0: (&bdev->bd_mutex){+.+.}, at: [<40269370>] > >> > __blkdev_put+0xbc/0x7f0 fs/block_dev.c:1757 > >> > 1 lock held by blkid/19199: > >

Re: [PATCH] drm/bridge/synopsys: dsi: Adopt SPDX identifiers

Benjamin, On Wed, Jan 24, 2018 at 9:57 AM, Benjamin Gaignard wrote: > 2018-01-24 0:32 GMT+01:00 Laurent Pinchart > : >> Hi Philippe, >> >> On Tuesday, 23 January 2018 12:25:51 EET Philippe CORNU wrote: >>> On 01/23/2018 12:30 AM, Laurent Pinchart wrote: >>> > On Monday, 22 January 2018 12:26:08

[PATCH][btrfs-next] Btrfs: extent map selftest: fix non-ANSI btrfs_test_extent_map declaration

From: Colin Ian King The function btrfs_test_extent_map requires a void argument to be ANSI C compliant and so it matches the prototype in fs/btrfs/tests/btrfs-tests.h Cleans up sparse warning: fs/btrfs/tests/extent-map-tests.c:346:27: warning: non-ANSI function declaration of function 'btrfs_te

Re: [PATCH 0/4] PCI: Add SPDX tags

Bjorn, On Fri, Jan 26, 2018 at 10:32 PM, Bjorn Helgaas wrote: > b24413180f56 ("License cleanup: add SPDX GPL-2.0 license identifier to > files with no license") added SPDX GPL-2.0 tags to several PCI files. > > These patches add SPDX tags to almost all remaining PCI files. For ease of > reviewin

Re: [PATCH] scripts/coccinelle/misc/shift.cocci: detect < or > that should be a shift

On Thu, Feb 08, 2018 at 02:58:56PM +0100, Julia Lawall wrote: > > > On Thu, 8 Feb 2018, Dan Carpenter wrote: > > > On Thu, Feb 08, 2018 at 01:53:54PM +0100, Julia Lawall wrote: > > > This checks for a comparison using < or > between two constants, > > > considering both explicit constants (1, 2,

Re: [PATCH] watchdog: imx2_wdt: allow setting timeout in devicetree

On 02/08/2018 05:11 AM, Marcus Folkesson wrote: By following best practice described in Documentation/watchdog/watchdog-kernel-api.txt, it also let us to set timout-sec property in devicetree. Signed-off-by: Marcus Folkesson Reviewed-by: Guenter Roeck --- Documentation/devicetree/binding

Re: [PATCH 2/6] nvme-pci: fix the freeze and quiesce for shutdown and reset case

Hi Keith Sorry for bothering you again. ;) There is a dangerous scenario which caused by nvme_wait_freeze in nvme_reset_work. please consider it. nvme_reset_work -> nvme_start_queues -> nvme_wait_freeze if the controller no response, we have to rely on the timeout path. there are issues be

Re: [PATCH] PCI: Add SPDX tag for s390_pci_hpc.c

Bjorn, On Tue, Jan 30, 2018 at 1:40 AM, Bjorn Helgaas wrote: > I previously posted patches [1] to add SPDX tags to all drivers/pci files > except drivers/pci/hotplug/s390_pci_hpc.c. I omitted that one because it > contained only a "License: GPL" comment, which I thought was ambiguous. > > Howeve

Re: INFO: task hung in sync_blockdev

On Thu, Feb 8, 2018 at 3:08 PM, Jan Kara wrote: > On Thu 08-02-18 14:28:08, Dmitry Vyukov wrote: >> On Thu, Feb 8, 2018 at 10:28 AM, Jan Kara wrote: >> > On Wed 07-02-18 07:52:29, Andi Kleen wrote: >> >> > #0: (&bdev->bd_mutex){+.+.}, at: [<40269370>] >> >> > __blkdev_put+0xbc/0x7f0 fs/

[RESEND] [PATCH] x86/smpboot: avoid warning messages while hot-removing physical cpu

From: Masayoshi Mizuma When a physical cpu is hot-removed, the following warning message are shown while the uncore device is removing in uncore_pci_remove(). WARNING: CPU: 120 PID: 5 at arch/x86/events/intel/uncore.c:988 uncore_pci_remove+0xf1/0x110 ... CPU: 120 PID: 5 Comm: kworker/u1024:0 Not

Re: [PATCH v4 0/2] MIPS: Augment CPC support

On Fri, Jan 19, 2018 at 04:40:47PM +0100, Aleksandar Markovic wrote: > From: Aleksandar Markovic > > v3->v4: > > - documentation patch now contains updating of MAINTAINERS file > - rebased to the latest code Thanks, I've applied with the intention of squeezing into 4.16 (I presume this

Re: [PATCH] platform/x86: dell-laptop: Removed duplicates in DMI whitelist

On Thu, Feb 8, 2018 at 12:12 AM, Alexander Abrosimov wrote: > Fixed a mistake in which several entries were duplicated in the DMI list > from the below commit > fe486138 platform/x86: dell-laptop: Add 2-in-1 devices to the DMI whitelist > Pushed to my review and testing queue, thanks! (Would be

Re: [PATCH] PCI: Add SPDX GPL-2.0+ to replace implicit GPL v2 or later statement

On Tue, Jan 30, 2018 at 1:40 AM, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > 7441b0627e22 ("s390/pci: PCI hotplug support via SCLP") added > s390_pci_hpc.c, which included this license information: > > +MODULE_LICENSE("GPL"); > > Based on "git show 7441b0627e22:include/linux/module.h", that

Re: [PATCH net-queue 1/3] Partial revert "e1000e: Avoid receiver overrun interrupt bursts"

On Wed, Feb 7, 2018 at 10:47 PM, Benjamin Poirier wrote: > This partially reverts commit 4aea7a5c5e940c1723add439f4088844cd26196d. > > We keep the fix for the first part of the problem (1) described in the log > of that commit, that is to read ICR in the other interrupt handler. We > remove the fi

Re: [PATCH net-queue 2/3] e1000e: Fix queue interrupt re-raising in Other interrupt.

On Wed, Feb 7, 2018 at 10:47 PM, Benjamin Poirier wrote: > restores the ICS write for rx/tx queue interrupts which was present before > commit 16ecba59bc33 ("e1000e: Do not read ICR in Other interrupt", > v4.5-rc1) but was not restored in commit 4aea7a5c5e94 ("e1000e: Avoid > receiver overrun inte

Re: [PATCH net-queue 3/3] e1000e: Avoid missed interrupts following ICR read.

On Wed, Feb 7, 2018 at 10:47 PM, Benjamin Poirier wrote: > The 82574 specification update errata 12 states that interrupts may be > missed if ICR is read while INT_ASSERTED is not set. Avoid that problem by > setting all bits related to events that can trigger the Other interrupt in > IMS. > > The

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

On Thu, Feb 1, 2018 at 10:14 PM, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file type. >

Re: [PATCH v2] ASoC: use seq_file to dump the contents of dai_list,platform_list and codec_list

On Fri, Jan 26, 2018 at 9:16 AM, Donglin Peng wrote: > On Wed, Jan 24, 2018 at 1:37 AM, Mark Brown wrote: >> On Tue, Jan 23, 2018 at 07:08:15PM +0200, Andy Shevchenko wrote: >> >>> If it's not critical, I would suggest to wait till v4.16-rc1, where I >>> would like to push [1], and switch to DEFI

Re: [PATCH] ARM: dts: ls1021a: add quadspi node

On 2018-02-05 09:03, Shawn Guo wrote: > On Fri, Jan 26, 2018 at 03:20:14PM +0100, Rasmus Villemoes wrote: >> Add a node to device tree repesenting the QuadSPI controller present on >> LS1021a. Driver support has been present since e8c034b2fbe5 (mtd: >> spi-nor: fsl-quadspi: add support for ls1021a)

[PATCH] pinctrl: ocelot: make function ocelot_pinctrl_probe static

From: Colin Ian King The function ocelot_pinctrl_probe is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: drivers/pinctrl/pinctrl-ocelot.c:465:5: warning: symbol 'ocelot_pinctrl_probe' was not declared. Should it be static? Signed-off-by

[PATCH 6/6] ARM: configs: stm32: enable ST MFX and its GPIO expander feature

Signed-off-by: Amelie Delaunay --- arch/arm/configs/stm32_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/configs/stm32_defconfig b/arch/arm/configs/stm32_defconfig index eb3c2e4..aa9feed 100644 --- a/arch/arm/configs/stm32_defconfig +++ b/arch/arm/configs/stm32_defconfig

[PATCH 5/6] ARM: dts: stm32: add joystick support on stm32746g-eval

The joystick on stm32746g-eval uses gpios on MFX gpio expander. Signed-off-by: Amelie Delaunay --- arch/arm/boot/dts/stm32746g-eval.dts | 31 +++ 1 file changed, 31 insertions(+) diff --git a/arch/arm/boot/dts/stm32746g-eval.dts b/arch/arm/boot/dts/stm32746g-eval.dt

[PATCH 2/6] mfd: Add ST Multi-Function eXpander core driver

ST Multi-Function eXpander (MFX) is a slave controller using I2C for communication with the main MCU. Main features are: - 16 fast GPIOs individually configurable in input/output - 8 alternate GPIOs individually configurable in input/output - Main MCU IDD measurement - Resistive touchscreen control

[PATCH 1/6] dt-bindings: mfd: Add ST Multi-Function eXpander driver

This patch adds documentation of device tree bindings for the STMicroelectronics Multi-Function eXpander (MFX). Signed-off-by: Amelie Delaunay --- Documentation/devicetree/bindings/mfd/st-mfx.txt | 51 1 file changed, 51 insertions(+) create mode 100644 Documentation/de

[PATCH 0/6] Introduce STMicroelectronics MultiFunction eXpander

This series adds support for STMicroelectronics MultiFunction eXpander (ST MFX), used on some STM32 discovery and evaluation boards. ST MFX is an STM32L152 slave controller whose firmware embeds the following features: - I/O expander (16 GPIOs + 8 extra if the other features are not enabled), - re

[PATCH 4/6] ARM: dts: stm32: add MFX support on I2C1 on stm32746g-eval

MFX is used as gpio expander on stm32746g-eval. Signed-off-by: Amelie Delaunay --- arch/arm/boot/dts/stm32746g-eval.dts | 17 + 1 file changed, 17 insertions(+) diff --git a/arch/arm/boot/dts/stm32746g-eval.dts b/arch/arm/boot/dts/stm32746g-eval.dts index e74ae59..99739f7 10064

[PATCH 3/6] gpio: Add GPIO support for the ST Multi-Function eXpander

ST Multi-Function eXpander (MFX) can be used as GPIO expander. It has 16 fast GPIOs and can have 8 extra alternate GPIOs when other MFX features are not enabled. Signed-off-by: Amelie Delaunay --- drivers/gpio/Kconfig | 10 + drivers/gpio/Makefile | 1 + dri

[PATCH v1 2/2] drm/panel: Add support for Raydium rm68200 panel driver

This patch adds Raydium Semiconductor Corporation rm68200 5.5" 720x1280 TFT LCD panel driver (MIPI-DSI video mode). Signed-off-by: Philippe Cornu --- drivers/gpu/drm/panel/Kconfig | 8 + drivers/gpu/drm/panel/Makefile| 1 + drivers/gpu/drm/panel/panel-raydium-

[PATCH v1 0/2] drm/panel: Add support for Raydium rm68200 panel

The Raydium Semiconductor Corporation RM68200 is a 5.5" 720x1280 TFT LCD panel connected using a MIPI-DSI video interface. Philippe Cornu (2): dt-bindings/display/panel: Add support for Raydium rm68200 dsi panel drm/panel: Add support for Raydium rm68200 panel driver .../bindings/display/pan

[PATCH v1 1/2] dt-bindings/display/panel: Add support for Raydium rm68200 dsi panel

The Raydium Semiconductor Corporation RM68200 is a 5.5" 720x1280 TFT LCD panel connected using a MIPI-DSI video interface. Signed-off-by: Philippe Cornu --- .../bindings/display/panel/raydium,rm68200.txt | 25 ++ 1 file changed, 25 insertions(+) create mode 100644 Docum

Re: [PATCH v2 1/2] amdgpu/dc/dml: Consolidate redundant CFLAGS

On 2018-02-07 08:51 PM, Matthias Kaehlcke wrote: > Use subdir-ccflags instead of specifying the same flags for every source > file. > > Signed-off-by: Matthias Kaehlcke > Reviewed-by: Guenter Roeck > --- > Changes in v2: > - added 'Reviewed-by: Guenter Roeck ' tag > > drivers/gpu/drm/amd/displ

Re: arch/x86/tools/insn_decoder_test: warning: ffffffff810005ac: 0f ff e9 ud0 %ecx,%ebp

On Thu, 8 Feb 2018 06:04:45 +0800 kbuild test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 7590e37bdaeec25ae325f4ba450be13e2aac6c8d > commit: 10c91577d5e631773a6394e14cf60125389b71ae x86/tools: Standardize > output format of insn_d

[PATCH] ASoC: Intel: Skylake: make function skl_clk_round_rate static

From: Colin Ian King The function skl_clk_round_rate is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: sound/soc/intel/skylake/skl-ssp-clk.c:250:6: warning: symbol 'skl_clk_round_rate' was not declared. Should it be static? Signed-off-b

Re: [PATCH v6] checkpatch.pl: Add SPDX license tag check

even On Fri, Feb 2, 2018 at 8:06 PM, Joe Perches wrote: > On Fri, 2018-02-02 at 12:27 -0600, Rob Herring wrote: >> On Fri, Feb 2, 2018 at 9:49 AM, Igor Stoppa wrote: >> > On 02/02/18 17:40, Rob Herring wrote: >> > > Add SPDX license tag check based on the rules defined in >> > >> > Shouldn't it

Re: [PATCH v7 04/24] mm: Dont assume page-table invariance during faults

On 06/02/2018 21:28, Matthew Wilcox wrote: > On Tue, Feb 06, 2018 at 05:49:50PM +0100, Laurent Dufour wrote: >> From: Peter Zijlstra >> >> One of the side effects of speculating on faults (without holding >> mmap_sem) is that we can race with free_pgtables() and therefore we >> cannot assume the p

[PATCH 0/3] GPU-DRM-GMA500: Adjustments for four function implementations

From: Markus Elfring Date: Thu, 8 Feb 2018 15:27:38 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in two functions Improve four size determinations Delete an unnecessary re

[PATCH] blk: optimization for classic polling

This removes the dependency on interrupts to wake up task. Set task state as TASK_RUNNING, if need_resched() returns true, while polling for IO completion. Earlier, polling task used to sleep, relying on interrupt to wake it up. This made some IO take very long when interrupt-coalescing is enabled

[PATCH 1/3] drm/gma500: Delete an error message for a failed memory allocation in two functions

From: Markus Elfring Date: Thu, 8 Feb 2018 14:55:49 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/gma500/framebuffer.c | 4 +--- drivers/gpu/drm/gma5

[PATCH 2/3] drm/gma500: Improve four size determinations

From: Markus Elfring Date: Thu, 8 Feb 2018 15:08:39 +0100 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was detect

[PATCH 3/3] drm/gma500: Delete an unnecessary return statement in oaktrail_crtc_hdmi_dpms()

From: Markus Elfring Date: Thu, 8 Feb 2018 15:17:48 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected function. Signed-off-by: Markus Elfring --- drivers/

Re: [PATCH] can: m_can: change comparison to bitshift when dealing with a mask

On 02/06/2018 09:52 AM, Wolfram Sang wrote: > Due to a typo, the mask was destroyed by a comparison instead of a bit > shift. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Wolfram Sang Applied to can. tnx, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Indus

Re: [PATCH v6] checkpatch.pl: Add SPDX license tag check

Kate, On Fri, Feb 2, 2018 at 9:18 PM, Kate Stewart wrote: > This is the new way to represent GPLv2 only, as described above. > While the GPL-2.0 and GPL-2.0+ notation is still valid, it is deprecated > in the latest version, so transitioning existing over time will probably > be needed. So I t

Re: [PATCH] can: m_can: change comparison to bitshift when dealing with a mask

On 02/06/2018 09:52 AM, Wolfram Sang wrote: > Due to a typo, the mask was destroyed by a comparison instead of a bit > shift. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Wolfram Sang > --- > Only build tested. Applied to can. Luckily that define is not used :) Tnx, Marc -- Pengutron

Re: [PATCH 2/7] soc: davinci: new genpd driver

On Thursday 08 February 2018 06:57 PM, Bartosz Golaszewski wrote: > 2018-02-08 13:56 GMT+01:00 Sekhar Nori : >> On Thursday 08 February 2018 03:24 PM, Bartosz Golaszewski wrote: >>> 2018-02-08 10:30 GMT+01:00 Sekhar Nori : On Wednesday 07 February 2018 07:15 PM, Bartosz Golaszewski wrote:

Re: [PATCH] opp: cpu: Replace GFP_ATOMIC with GFP_KERNEL in dev_pm_opp_init_cpufreq_table

On 26-01-18, 16:48, Jia-Ju Bai wrote: > After checking all possible call chains to > dev_pm_opp_init_cpufreq_table() here, > my tool finds that this function is never called in atomic context, > namely never in an interrupt handler or holding a spinlock. > And dev_pm_opp_init_cpufreq_table() call

Re: [PATCH v6] checkpatch.pl: Add SPDX license tag check

On Sat, Feb 3, 2018 at 2:41 PM, Igor Stoppa wrote: > > > On 02/02/18 21:06, Joe Perches wrote: >> On Fri, 2018-02-02 at 12:27 -0600, Rob Herring wrote: >>> On Fri, Feb 2, 2018 at 9:49 AM, Igor Stoppa wrote: On 02/02/18 17:40, Rob Herring wrote: > Add SPDX license tag check based on the r

Re: [PATCH 5/9] perf utils: add support for arch standard events

On 08/02/2018 13:55, Jiri Olsa wrote: On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: SNIP + char *perpkg; + char *unit; + char *metric_expr; + char *metric_name; + char *metric_group; + struct list_head list; + char strings[]; +}; + +stati

Re: [PATCH v2 1/2] net, can, ifi: fix "write buffer full" error

On 02/08/2018 07:47 AM, Heiko Schocher wrote: > the driver reads in the ISR first the IRQpending register, > and clears after that in a write *all* bits in it. > > It could happen that the isr register raise bits between > this 2 register accesses, which leads in lost bits ... > > In case it clea

Re: [PATCH v2 05/16] arm64: dts: mt7622: add PMIC MT6380 related nodes

Sean, On Tue, Feb 6, 2018 at 10:52 AM, wrote: > From: Sean Wang > > Enable pwrap and MT6380 on mt7622-rfb1 board. Also add all mt6380 > regulator nodes in an alone file to allow similar boards using MT6380 > able to resue the configuration. > > Signed-off-by: Sean Wang > Cc: Mark Brown > Cc:

Re: INFO: task hung in sync_blockdev

> > It seems multiple processes deadlocked on the bd_mutex. > > Unfortunately there's no backtrace for the lock acquisitions, > > so it's hard to see the exact sequence. > > Well, all in the report points to a situation where some IO was submitted > to the block device and never completed (more e

Re: [PATCH] acpi: osl: Replace GFP_ATOMIC with GFP_KERNEL in acpi_os_execute

On Thu, Feb 8, 2018 at 2:41 PM, Jia-Ju Bai wrote: > > > On 2018/2/8 18:20, Rafael J. Wysocki wrote: >> >> On Thursday, February 8, 2018 11:13:10 AM CET Chris Wilson wrote: >>> >>> Quoting Rafael J. Wysocki (2018-02-08 09:51:41) On Thursday, January 25, 2018 11:13:41 AM CET Jia-Ju Bai wro

[git pull] IOMMU Updates for Linux v4.16

Hi Linus, This time there are not a lot of changes coming from the IOMMU side. That is partly because I returned from my parental leave late in the development process and probably partly because everyone was busy with Spectre and Meltdown mitigation work and didn't find the time for IOMMU work. S

Re: [PATCH v2] printk: Relocate wake_klogd check close to the end of console_unlock()

On (02/08/18 14:04), Petr Mladek wrote: > We mark for waking up klogd whenever we see a new message sequence in > the main loop. However, the actual wakeup is always at the end of the > function and we can easily test for the wakeup condition when we do > the final should-we-repeat check. > > Mov

Re: [PATCH 5/9] perf utils: add support for arch standard events

On Thu, Feb 08, 2018 at 02:45:37PM +, John Garry wrote: > On 08/02/2018 13:55, Jiri Olsa wrote: > > On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: > > > > SNIP > > > > > + char *perpkg; > > > + char *unit; > > > + char *metric_expr; > > > + char *metric_name; > > > + char *metric

Re: x86/microcode/intel: Division by zero panic in 4.9.79 and 4.4.114

On Thu, Feb 08, 2018 at 01:55:48PM +, Rolf Neugebauer wrote: > On the 4.4 kernel, 1f161f67a272c ("x86/microcode: Do the family check > first") does not apply cleanly. Looks like it relies on 309aac77768c0 > ("x86/microcode: Decrease CPUID use") and 7a93a40be23e5 > ("x86/microcode: Remove local

Re: [PATCH v3 01/11] regulator: core: add API to get voltage constraints

On Thu, Feb 08, 2018 at 03:34:56PM +0530, Laxman Dewangan wrote: > I added this API in downstream for the purpose of finding whether rail > output can be changed or not(fixed) based on constraints. > If min and max is same then it can not change. I used this information to > set the pad voltage of

[PATCH v2] drm/bridge/synopsys: dsi: Adopt SPDX identifiers

Add SPDX identifiers to the Synopsys DesignWare MIPI DSI host controller driver. Signed-off-by: Philippe Cornu --- Changes in v2: Update to "GPL-2.0+" following comments from Laurent Pinchart, Benjamin Gaignard & Philippe Ombredanne. drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 6 +- 1 f

Re: [PATCH 2/2] mmc: Add mmc_force_detect_change_begin / _end functions

Hi Ulf, On Wed, Aug 30, 2017 at 03:43:49PM +0200, Ulf Hansson wrote: > On 30 August 2017 at 14:44, Hans de Goede wrote: > > Hi, > > > > > > On 21-07-17 16:35, Quentin Schulz wrote: > >> > >> From: Hans de Goede > >> > >> Some sdio devices have a multiple stage bring-up process. Specifically > >>

Re: [PATCH 5/9] perf utils: add support for arch standard events

On 08/02/2018 13:55, Jiri Olsa wrote: On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote: For some architectures (like arm), there are architecture- defined events. Sometimes these events may be "recommended" according to the architecture standard, in that the implementer is free ignore

Re: [PATCH v7 04/24] mm: Dont assume page-table invariance during faults

On Thu, Feb 08, 2018 at 03:35:58PM +0100, Laurent Dufour wrote: > I reviewed that part of code, and I think I could now change the way > pte_unmap_safe() is checking for the pte's value. Since we now have all the > needed details in the vm_fault structure, I will pass it to > pte_unamp_same() and d

Re: [PATCH v1 1/2] drm/stm: ltdc: add non-alpha color formats

Hi Benjamin, and many thanks for having applied the 2 patches. Philippe :-) On 02/08/2018 10:40 AM, Benjamin Gaignard wrote: > 2018-02-06 10:12 GMT+01:00 Yannick FERTRE : >> Reviewed-by: Yannick Fertré >> >> >> On 02/01/2018 11:42 AM, Philippe Cornu wrote: >>> ltdc supports natively some color fo

Re: [PATCH 1/3] x86/MCE/AMD: Redo function to get SMCA bank type

On Thu, Feb 01, 2018 at 12:48:11PM -0600, Yazen Ghannam wrote: > arch/x86/kernel/cpu/mcheck/mce_amd.c | 15 +++ > 1 file changed, 7 insertions(+), 8 deletions(-) > > diff --git a/arch/x86/kernel/cpu/mcheck/mce_amd.c > b/arch/x86/kernel/cpu/mcheck/mce_amd.c > index 0f32ad242324..4e16a

Re: [PATCH] drm/bridge/synopsys: dsi: Adopt SPDX identifiers

Hi Laurent, Benjamin & Philippe, I sent an updated version of the patch following your comments Big thank you, Philippe :-) On 02/08/2018 03:09 PM, Philippe Ombredanne wrote: > Benjamin, > > On Wed, Jan 24, 2018 at 9:57 AM, Benjamin Gaignard > wrote: >> 2018-01-24 0:32 GMT+01:00 Laurent Pincha

Re: [PATCH v2 1/3] sched: Stop nohz stats when decayed

On 8 February 2018 at 15:00, Peter Zijlstra wrote: > On Tue, Feb 06, 2018 at 08:23:05PM +0100, Vincent Guittot wrote: > >> @@ -9207,13 +9231,15 @@ void nohz_balance_enter_idle(int cpu) >> if (!housekeeping_cpu(cpu, HK_FLAG_SCHED)) >> return; >> >> + rq->has_blocked_load = 1

Re: [PATCH v9 6/8] media: i2c: Add TDA1997x HDMI receiver driver

Hi Tim, I was so hoping I could make a pull request for this, but I still found problems with g/s/query_dv_timings. I strongly suspect that v4l2-compliance would fail if you boot up the system *without* a source connected. And I discovered that I was missing additional checks in the timings test

Re: [PATCH v4 2/2] dt/bindings: Add bindings for Layerscape external irqs

On 2018-02-05 07:07, Rob Herring wrote: >> +Example: >> +scfg: scfg@157 { >> +compatible = "fsl,ls1021a-scfg", "syscon"; >> +... >> +extirq: interrupt-controller { >> +compatible = "fsl,ls1021a-extirq"; >> +#interru

Re: [PATCH 2/6] nvme-pci: fix the freeze and quiesce for shutdown and reset case

On Thu, Feb 08, 2018 at 10:17:00PM +0800, jianchao.wang wrote: > There is a dangerous scenario which caused by nvme_wait_freeze in > nvme_reset_work. > please consider it. > > nvme_reset_work > -> nvme_start_queues > -> nvme_wait_freeze > > if the controller no response, we have to rely on t

Re: [PATCH v5] ARM: sun8i: h2+: add support for Banana Pi M2 Zero board

On Thu, Feb 08, 2018 at 06:06:13PM +0800, Icenowy Zheng wrote: > Banana Pi M2 Zero board is a H2+-based board by Sinovoip, with a form > factor and GPIO holes similar to Raspberry Pi Zero. > > It features: > - Allwinner H2+ SoC > - Single-chip (16-bit) 512MiB DDR3 DRAM > - Ampak AP6212 Wi-Fi/Bluet

Re: [PATCH v2] drm/bridge/synopsys: dsi: Adopt SPDX identifiers

On Thu, Feb 8, 2018 at 3:58 PM, Philippe Cornu wrote: > Add SPDX identifiers to the Synopsys DesignWare MIPI DSI > host controller driver. > > Signed-off-by: Philippe Cornu > --- > Changes in v2: Update to "GPL-2.0+" following comments from Laurent > Pinchart, Benjamin Gaignard & Philippe Ombreda

Re: [PATCH 2/3] x86/MCE/AMD, EDAC/mce_amd: Enumerate Reserved SMCA bank type

On Thu, Feb 01, 2018 at 12:48:12PM -0600, Yazen Ghannam wrote: > From: Yazen Ghannam > > Currently, bank 4 is reserved on Fam17h, so we chose not to initialize > bank 4 in the smca_banks array. This means that when we check if a bank > is initialized, like during boot or resume, we will see that

Re: [PATCH 1/2] serial: 8250: Don't service RX FIFO if interrupts are disabled

On Thu, Feb 8, 2018 at 2:55 PM, Vignesh R wrote: > Currently, data in RX FIFO is read based on UART_LSR register state even > if RDI and RLSI interrupts are disabled in UART_IER register. > This is because when IRQ handler is called due to TX FIFO empty event, > RX FIFO is serviced based on UART_L

Re: [PATCH -mm -v2] mm, swap, frontswap: Fix THP swap if frontswap enabled

On Thu, Feb 8, 2018 at 6:17 PM, Minchan Kim wrote: > On Wed, Feb 07, 2018 at 03:00:35PM +0800, Huang, Ying wrote: >> From: Huang Ying >> >> It was reported by Sergey Senozhatsky that if THP (Transparent Huge >> Page) and frontswap (via zswap) are both enabled, when memory goes low >> so that swap

[PATCH] x86/decoder: Update UD0 to have operands

According to the December 2017 Intel SDM publication, UD0 now has 2 operands by MODRM byte. So it's length should be 3 bytes. Signed-off-by: Masami Hiramatsu Reported-by: kbuild test robot --- arch/x86/lib/x86-opcode-map.txt |3 ++- tools/objtool/arch/x86/lib/x86-opcode-map.tx

[PATCH 3/3] phy: rockchip-typec: force to USB2 if DP at 4 lanes mode

From: Chris Zhong The usb3tousb2_en BIT will be clear to 0 in probe(), it make USB controller work at USB3 mode, and if the USB phy is turned on with DP only mode(4 lanes DP), the rockchip_usb3_phy_power_on() will return directly, so usb3_host_disable and usb3_host_port these 2 BIT will keep a sa

[PATCH 2/3] Documentation: bindings: add usb3-host-disable and usb3-host-port for Rockchip USB Type-C PHY

From: William wu rockchip,usb3-host-disable is the register of type-c phy disable usb3 host rockchip,usb3-host-port is the register of type-c phy usb3 port number Signed-off-by: William wu Signed-off-by: Enric Balletbo i Serra --- Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt |

<    1   2   3   4   5   6   7   8   >