[PATCH 7/8] tools lib bpf: fix maps resolution

2016-10-16 Thread Eric Leblond
It is not correct to assimilate the elf data of the maps section to an array of map definition. In fact the sizes differ. The offset provided in the symbol section has to be used instead. This patch fixes a bug causing a elf with two maps not to load correctly. Signed-off-by: Eric Leblond ---

[PATCH 8/8] tools lib bpf: install header file

2016-10-16 Thread Eric Leblond
Makefile was not installing the header file of the library and a manual copy was needed to have a usable library on the system. Signed-off-by: Eric Leblond --- tools/lib/bpf/Makefile | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/tools/lib/bpf/Makefile

[PATCH 5/8] tools lib bpf: add missing functions

2016-10-16 Thread Eric Leblond
Some functions were missing in the library to be able to use it in the case where the userspace is handling the maps in kernel. The patch also renames functions to have a homogeneous naming convention. Signed-off-by: Eric Leblond --- tools/lib/bpf/bpf.c| 35

[PATCH 5/8] tools lib bpf: add missing functions

2016-10-16 Thread Eric Leblond
Some functions were missing in the library to be able to use it in the case where the userspace is handling the maps in kernel. The patch also renames functions to have a homogeneous naming convention. Signed-off-by: Eric Leblond --- tools/lib/bpf/bpf.c| 35

[PATCH 2/8] uapi linux bpf: add max value to enum

2016-10-16 Thread Eric Leblond
It will be used to detect userspace trying to set invalid value. Signed-off-by: Eric Leblond --- include/uapi/linux/bpf.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/bpf.h b/include/uapi/linux/bpf.h index f09c70b..570287f 100644 ---

[PATCH 2/8] uapi linux bpf: add max value to enum

2016-10-16 Thread Eric Leblond
It will be used to detect userspace trying to set invalid value. Signed-off-by: Eric Leblond --- include/uapi/linux/bpf.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/bpf.h b/include/uapi/linux/bpf.h index f09c70b..570287f 100644 --- a/include/uapi/linux/bpf.h +++

Re: [PATCH] watchdog: at91sam9: keep watchdog running in idle mode

2016-10-16 Thread Sylvain Rochet
Hi Guenter, On Sun, Oct 16, 2016 at 08:55:41AM -0700, Guenter Roeck wrote: > On 10/16/2016 08:50 AM, Guenter Roeck wrote: > > On 10/16/2016 08:33 AM, Sylvain Rochet wrote: > > > > > > Looks like this one falls through the cracks, it didn't reach mainline > > > and therefore wasn't applied to

Re: [PATCH] watchdog: at91sam9: keep watchdog running in idle mode

2016-10-16 Thread Sylvain Rochet
Hi Guenter, On Sun, Oct 16, 2016 at 08:55:41AM -0700, Guenter Roeck wrote: > On 10/16/2016 08:50 AM, Guenter Roeck wrote: > > On 10/16/2016 08:33 AM, Sylvain Rochet wrote: > > > > > > Looks like this one falls through the cracks, it didn't reach mainline > > > and therefore wasn't applied to

[PATCH 4/4] AVR32-pio: Replace two seq_printf() calls by seq_puts() in pio_bank_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:18:31 +0200 Strings which did not contain data format specifications should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software.

[PATCH 4/4] AVR32-pio: Replace two seq_printf() calls by seq_puts() in pio_bank_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:18:31 +0200 Strings which did not contain data format specifications should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 3/4] AVR32-pio: Use seq_putc() in pio_bank_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:13:19 +0200 A single character (line break) should be put into a sequence. Thus use the corresponding function "seq_putc". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH 3/4] AVR32-pio: Use seq_putc() in pio_bank_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:13:19 +0200 A single character (line break) should be put into a sequence. Thus use the corresponding function "seq_putc". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- arch/avr32/mach-at32ap/pio.c |

[PATCH 2/4] AVR32-clock: Combine nine seq_printf() calls into one call in clk_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:04:10 +0200 Some data were printed into a sequence by nine separate function calls. Print the same data by a single function call instead. Signed-off-by: Markus Elfring ---

[PATCH 2/4] AVR32-clock: Combine nine seq_printf() calls into one call in clk_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:04:10 +0200 Some data were printed into a sequence by nine separate function calls. Print the same data by a single function call instead. Signed-off-by: Markus Elfring --- arch/avr32/mach-at32ap/clock.c | 28 +++- 1 file

[PATCH 1/4] AVR32-clock: Use seq_putc() in two functions

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 21:51:09 +0200 A single character (line break) should be put into two sequences. Thus use the corresponding function "seq_putc". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH 1/4] AVR32-clock: Use seq_putc() in two functions

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 21:51:09 +0200 A single character (line break) should be put into two sequences. Thus use the corresponding function "seq_putc". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 0/4] AVR32-AT32AP: Fine-tuning for three function implementations

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:30:22 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (4): clock: Use seq_putc() in two functions clock: Combine nine seq_printf() calls into one call in

[PATCH 0/4] AVR32-AT32AP: Fine-tuning for three function implementations

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 22:30:22 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (4): clock: Use seq_putc() in two functions clock: Combine nine seq_printf() calls into one call in clk_show() pio: Use seq_putc()

Re: Kildall

2016-10-16 Thread Ove Bjørn Karlsen
Den 10/15/2016 23:19, skrev jdow: On 2016-10-15 05:31, Ove Bjørn Karlsen wrote: Den 10/6/2016 10:37, skrev Ove Bjørn Karlsen: Den 10/3/2016 17:10, skrev Ove Bjørn Karlsen: Den 10/3/2016 14:29, skrev Ove Bjørn Karlsen: I tried Linux in 2010, with the peak of the Ubuntu distribution, Karmic

Re: Kildall

2016-10-16 Thread Ove Bjørn Karlsen
Den 10/15/2016 23:19, skrev jdow: On 2016-10-15 05:31, Ove Bjørn Karlsen wrote: Den 10/6/2016 10:37, skrev Ove Bjørn Karlsen: Den 10/3/2016 17:10, skrev Ove Bjørn Karlsen: Den 10/3/2016 14:29, skrev Ove Bjørn Karlsen: I tried Linux in 2010, with the peak of the Ubuntu distribution, Karmic

Re: [PATCH v5 0/3] drm/nouveau: set DMA mask before mapping scratch page

2016-10-16 Thread Ard Biesheuvel
On 7 October 2016 at 09:12, Alexandre Courbot wrote: > On Fri, Oct 7, 2016 at 12:49 AM, Ard Biesheuvel > wrote: >> This v4 is now a 3 piece series (since v4), after Alexandre pointed out that >> both GF 100 and NV50 are affected by the same issue, and

Re: [PATCH v5 0/3] drm/nouveau: set DMA mask before mapping scratch page

2016-10-16 Thread Ard Biesheuvel
On 7 October 2016 at 09:12, Alexandre Courbot wrote: > On Fri, Oct 7, 2016 at 12:49 AM, Ard Biesheuvel > wrote: >> This v4 is now a 3 piece series (since v4), after Alexandre pointed out that >> both GF 100 and NV50 are affected by the same issue, and that a related issue >> has been solved

RK3288 USB issues with ath9k_htc Wi-Fi dongles on veyron devices

2016-10-16 Thread Paul Kocialkowski
I'm trying to use ath9k_htc Wi-Fi dongles with various rk3288 veyron devices, which currently do not work. I have a bunch of different dongles, that all work nicely with other devices. The kernel I'm running is 4.9-rc1 and the USB connectors I'm using are linked to OTG and HOST1 (both DWC2

RK3288 USB issues with ath9k_htc Wi-Fi dongles on veyron devices

2016-10-16 Thread Paul Kocialkowski
I'm trying to use ath9k_htc Wi-Fi dongles with various rk3288 veyron devices, which currently do not work. I have a bunch of different dongles, that all work nicely with other devices. The kernel I'm running is 4.9-rc1 and the USB connectors I'm using are linked to OTG and HOST1 (both DWC2

Re: [PATCH v2] ARM: dts: rockchip: temporarily remove emmc hs200 speed from rk3288-veyron-speedy.

2016-10-16 Thread Paul Kocialkowski
Hi, Le mardi 27 septembre 2016 à 13:53 -0700, Vagrant Cascadian a écrit : > This essentially mimics what was done with rk3288-veyron-minnie in > commit 984926781122f034d5bc9962815d135b6c4a8e1d. > > The eMMC of the speedy Chromebook also appears to need the same tuning > workaround, as it

Re: [PATCH v2] ARM: dts: rockchip: temporarily remove emmc hs200 speed from rk3288-veyron-speedy.

2016-10-16 Thread Paul Kocialkowski
Hi, Le mardi 27 septembre 2016 à 13:53 -0700, Vagrant Cascadian a écrit : > This essentially mimics what was done with rk3288-veyron-minnie in > commit 984926781122f034d5bc9962815d135b6c4a8e1d. > > The eMMC of the speedy Chromebook also appears to need the same tuning > workaround, as it

Re: About group scheduling for SCHED_DEADLINE

2016-10-16 Thread Luca Abeni
On Mon, 10 Oct 2016 13:08:18 +0200 Peter Zijlstra wrote: > On Mon, Oct 10, 2016 at 12:15:58PM +0200, Peter Zijlstra wrote: > > However, I think there's a third alternative. I have memories of a > > paper from UNC (I'd have to dig through the site to see if I can > > still

Re: About group scheduling for SCHED_DEADLINE

2016-10-16 Thread Luca Abeni
On Mon, 10 Oct 2016 13:08:18 +0200 Peter Zijlstra wrote: > On Mon, Oct 10, 2016 at 12:15:58PM +0200, Peter Zijlstra wrote: > > However, I think there's a third alternative. I have memories of a > > paper from UNC (I'd have to dig through the site to see if I can > > still find it) where they

[PATCH] staging: dgnc: Replace CamelCase namings with underscores.

2016-10-16 Thread Pankaj Bharadiya
Replace CamelCase names with underscores to comply with the standard kernel coding style. Signed-off-by: Pankaj Bharadiya --- drivers/staging/dgnc/dgnc_tty.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git

[PATCH] staging: dgnc: Replace CamelCase namings with underscores.

2016-10-16 Thread Pankaj Bharadiya
Replace CamelCase names with underscores to comply with the standard kernel coding style. Signed-off-by: Pankaj Bharadiya --- drivers/staging/dgnc/dgnc_tty.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c

Re: About group scheduling for SCHED_DEADLINE

2016-10-16 Thread Luca Abeni
Hi Peter, first of all, sorry for the delay in my answer On Mon, 10 Oct 2016 12:15:58 +0200 Peter Zijlstra wrote: > On Sun, Oct 09, 2016 at 09:39:38PM +0200, Luca Abeni wrote: > > > So, I started to think about this, and here are some ideas to start > > a discussion: > >

Re: About group scheduling for SCHED_DEADLINE

2016-10-16 Thread Luca Abeni
Hi Peter, first of all, sorry for the delay in my answer On Mon, 10 Oct 2016 12:15:58 +0200 Peter Zijlstra wrote: > On Sun, Oct 09, 2016 at 09:39:38PM +0200, Luca Abeni wrote: > > > So, I started to think about this, and here are some ideas to start > > a discussion: > > 1) First of all, we

[PATCH 1/2] perf bench futex: cache align the worer struct

2016-10-16 Thread Sebastian Andrzej Siewior
It popped up in perf testing that the worker consumes some amount of CPU. It boils down to the increment of `ops` which causes cache line bouncing between the individual threads. The patch aligns the struct by 256 bytes to ensure that not a cache line is shared among CPUs. 128 byte is the x86

[PATCH 1/2] perf bench futex: cache align the worer struct

2016-10-16 Thread Sebastian Andrzej Siewior
It popped up in perf testing that the worker consumes some amount of CPU. It boils down to the increment of `ops` which causes cache line bouncing between the individual threads. The patch aligns the struct by 256 bytes to ensure that not a cache line is shared among CPUs. 128 byte is the x86

[PATCH 2/2] perf bench futex: add NUMA support

2016-10-16 Thread Sebastian Andrzej Siewior
By default the application uses malloc() and all available CPUs. This patch introduces NUMA support which means: - memory is allocated node local via numa_alloc_local() - all CPUs of the specified NUMA node are used. This is also true if the number of threads set is greater than the number of

[PATCH 2/2] perf bench futex: add NUMA support

2016-10-16 Thread Sebastian Andrzej Siewior
By default the application uses malloc() and all available CPUs. This patch introduces NUMA support which means: - memory is allocated node local via numa_alloc_local() - all CPUs of the specified NUMA node are used. This is also true if the number of threads set is greater than the number of

[PATCH] ARM64-cpuinfo: Combine six calls for sequence output into one seq_printf() call in c_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 20:48:28 +0200 Some data were printed into a sequence by six separate function calls. Print the same data by a single function call instead. Signed-off-by: Markus Elfring ---

[PATCH] ARM64-cpuinfo: Combine six calls for sequence output into one seq_printf() call in c_show()

2016-10-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 16 Oct 2016 20:48:28 +0200 Some data were printed into a sequence by six separate function calls. Print the same data by a single function call instead. Signed-off-by: Markus Elfring --- arch/arm64/kernel/cpuinfo.c | 19 +++ 1 file changed, 11

Re: [PATCH V3 00/11] block-throttle: add .high limit

2016-10-16 Thread Paolo Valente
> Il giorno 14 ott 2016, alle ore 20:35, Tejun Heo ha scritto: > > Hello, Paolo. > > On Fri, Oct 14, 2016 at 07:13:41PM +0200, Paolo Valente wrote: >> That said, your 'thus' seems a little too strong: "bfq does not yet >> handle fast SSDs, thus we need something else". What

Re: [PATCH V3 00/11] block-throttle: add .high limit

2016-10-16 Thread Paolo Valente
> Il giorno 14 ott 2016, alle ore 20:35, Tejun Heo ha scritto: > > Hello, Paolo. > > On Fri, Oct 14, 2016 at 07:13:41PM +0200, Paolo Valente wrote: >> That said, your 'thus' seems a little too strong: "bfq does not yet >> handle fast SSDs, thus we need something else". What about the >>

Re: [PATCH 0/5] Replace data type declarations with variables of same types in several source files.

2016-10-16 Thread Sergio Paracuellos
El 2016年10月16日 a las 10:24, Alexander Alemayhu escribió: On Mon, Oct 10, 2016 at 04:02:17PM +0200, Sergio Paracuellos wrote: This patch series replaces several sizeof(struct XXX) stuff in favour of sizeof(var) which is the preferred one. Why not squash this series into one patch? The

Re: [PATCH 0/5] Replace data type declarations with variables of same types in several source files.

2016-10-16 Thread Sergio Paracuellos
El 2016年10月16日 a las 10:24, Alexander Alemayhu escribió: On Mon, Oct 10, 2016 at 04:02:17PM +0200, Sergio Paracuellos wrote: This patch series replaces several sizeof(struct XXX) stuff in favour of sizeof(var) which is the preferred one. Why not squash this series into one patch? The

Re: [PATCH] kconfig.h: remove config_enabled() macro

2016-10-16 Thread hpa
On October 16, 2016 4:07:58 AM PDT, Masahiro Yamada wrote: >The use of config_enabled() is ambiguous. For config options, >IS_ENABLED(), IS_REACHABLE(), etc. will make intention clearer. >Sometimes config_enabled() has been used for non-config options >because it

Re: [PATCH] kconfig.h: remove config_enabled() macro

2016-10-16 Thread hpa
On October 16, 2016 4:07:58 AM PDT, Masahiro Yamada wrote: >The use of config_enabled() is ambiguous. For config options, >IS_ENABLED(), IS_REACHABLE(), etc. will make intention clearer. >Sometimes config_enabled() has been used for non-config options >because it is useful to check whether the

Re: [tip:x86/urgent] x86/cpufeature: Add AVX512_4VNNIW and AVX512_4FMAPS features

2016-10-16 Thread hpa
On October 16, 2016 9:35:57 AM PDT, Borislav Petkov wrote: >On Sun, Oct 16, 2016 at 09:02:51AM -0700, h...@zytor.com wrote: >> No, please. That would be worse than the disease. > >Why not? > >I did that recently with a bunch of leaves and there were no issues: > >2ccd71f1b278

Re: [tip:x86/urgent] x86/cpufeature: Add AVX512_4VNNIW and AVX512_4FMAPS features

2016-10-16 Thread hpa
On October 16, 2016 9:35:57 AM PDT, Borislav Petkov wrote: >On Sun, Oct 16, 2016 at 09:02:51AM -0700, h...@zytor.com wrote: >> No, please. That would be worse than the disease. > >Why not? > >I did that recently with a bunch of leaves and there were no issues: > >2ccd71f1b278 ("x86/cpufeature:

Re: [PATCH v3 2/9] hwmon: (core) New hwmon registration API

2016-10-16 Thread Guenter Roeck
Hi Jean, On 10/07/2016 05:32 AM, Jean Delvare wrote: [ ... ] + name = (char *)template; + } else { + name = devm_kzalloc(dev, strlen(template) + 16, GFP_KERNEL); + if (!name) + return ERR_PTR(-ENOMEM); +

Re: [PATCH v3 2/9] hwmon: (core) New hwmon registration API

2016-10-16 Thread Guenter Roeck
Hi Jean, On 10/07/2016 05:32 AM, Jean Delvare wrote: [ ... ] + name = (char *)template; + } else { + name = devm_kzalloc(dev, strlen(template) + 16, GFP_KERNEL); + if (!name) + return ERR_PTR(-ENOMEM); +

[PATCH] staging/lustre/llite: Move unstable_stats from sysfs to debugfs

2016-10-16 Thread Oleg Drokin
It's multiple values per file, so it has no business being in sysfs, besides it was assuming seqfile anyway. Introduced by commit d806f30e639b ("staging: lustre: osc: revise unstable pages accounting") Signed-off-by: Oleg Drokin ---

[PATCH] staging/lustre/llite: Move unstable_stats from sysfs to debugfs

2016-10-16 Thread Oleg Drokin
It's multiple values per file, so it has no business being in sysfs, besides it was assuming seqfile anyway. Introduced by commit d806f30e639b ("staging: lustre: osc: revise unstable pages accounting") Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/lproc_llite.c | 34

Re: [PATCH v2] console: Don't prefer first registered if DT specifies stdout-path

2016-10-16 Thread Andreas Schwab
On Aug 09 2016, Paul Burton wrote: > Fix this by not automatically preferring the first registered console if > one is specified by the device tree. This allows consoles to be > registered but not enabled, and once the driver for the console selected > by stdout-path

Re: [PATCH v2] console: Don't prefer first registered if DT specifies stdout-path

2016-10-16 Thread Andreas Schwab
On Aug 09 2016, Paul Burton wrote: > Fix this by not automatically preferring the first registered console if > one is specified by the device tree. This allows consoles to be > registered but not enabled, and once the driver for the console selected > by stdout-path calls of_console_check() the

[PATCH v2 3/3] Coccinelle: misc: Add support for devm variant in all modes

2016-10-16 Thread Vaishali Thakkar
Add missing support for the devm_request_threaded_irq in the rules of context, report and org modes. Misc: To be consistent with other scripts, change confidence level of the script to 'Moderate'. Signed-off-by: Vaishali Thakkar ---

[PATCH v2 3/3] Coccinelle: misc: Add support for devm variant in all modes

2016-10-16 Thread Vaishali Thakkar
Add missing support for the devm_request_threaded_irq in the rules of context, report and org modes. Misc: To be consistent with other scripts, change confidence level of the script to 'Moderate'. Signed-off-by: Vaishali Thakkar --- scripts/coccinelle/misc/irqf_oneshot.cocci | 10

cygnus-pcm.c:undefined reference to `bad_dma_ops'

2016-10-16 Thread kbuild test robot
Hi Mark, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1001354ca34179f3db924eb66672442a173147dc commit: 3ceeda1cbee9f93bb5537c9b840d1f7e767d7c01 Merge remote-tracking branches 'asoc/topic/cs53l30',

cygnus-pcm.c:undefined reference to `bad_dma_ops'

2016-10-16 Thread kbuild test robot
Hi Mark, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1001354ca34179f3db924eb66672442a173147dc commit: 3ceeda1cbee9f93bb5537c9b840d1f7e767d7c01 Merge remote-tracking branches 'asoc/topic/cs53l30',

Re: [PATCH 11/12] jump_label: declare jump table as external array

2016-10-16 Thread Peter Zijlstra
On Sun, Oct 16, 2016 at 06:50:55PM +0200, Vegard Nossum wrote: > On 10/16/2016 06:25 PM, Peter Zijlstra wrote: > >On Sun, Oct 16, 2016 at 05:16:15PM +0200, Vegard Nossum wrote: > >>Cc: Jason Baron > >>Cc: Peter Zijlstra > >>Cc: Steven Rostedt

Re: [PATCH 11/12] jump_label: declare jump table as external array

2016-10-16 Thread Peter Zijlstra
On Sun, Oct 16, 2016 at 06:50:55PM +0200, Vegard Nossum wrote: > On 10/16/2016 06:25 PM, Peter Zijlstra wrote: > >On Sun, Oct 16, 2016 at 05:16:15PM +0200, Vegard Nossum wrote: > >>Cc: Jason Baron > >>Cc: Peter Zijlstra > >>Cc: Steven Rostedt > >>Signed-off-by: Vegard Nossum > > > >NAK,

Re: [PATCH 5/7] ramoops: Split ftrace buffer space into per-CPU zones

2016-10-16 Thread Joel Fernandes
Hi Kees, On Mon, Oct 10, 2016 at 4:59 PM, Kees Cook wrote: > On Sun, Oct 9, 2016 at 10:15 AM, Joel Fernandes wrote: >> On Fri, Oct 7, 2016 at 10:28 PM, Joel Fernandes wrote: >>> If FTRACE_PER_CPU flag is passed to ramoops pdata,

Re: [PATCH 5/7] ramoops: Split ftrace buffer space into per-CPU zones

2016-10-16 Thread Joel Fernandes
Hi Kees, On Mon, Oct 10, 2016 at 4:59 PM, Kees Cook wrote: > On Sun, Oct 9, 2016 at 10:15 AM, Joel Fernandes wrote: >> On Fri, Oct 7, 2016 at 10:28 PM, Joel Fernandes wrote: >>> If FTRACE_PER_CPU flag is passed to ramoops pdata, split the space into >>> multiple zones depending on the number

Re: [PATCH 00/12] external array access helpers

2016-10-16 Thread Vegard Nossum
On 10/16/2016 06:14 PM, Greg Kroah-Hartman wrote: On Sun, Oct 16, 2016 at 05:16:04PM +0200, Vegard Nossum wrote: Hi, The first two patches in the series fix the concrete bug (a boot crash when using gcc 7.0+) by defining new wrappers for arrays defined in linker scripts. These two patches

Re: [PATCH 00/12] external array access helpers

2016-10-16 Thread Vegard Nossum
On 10/16/2016 06:14 PM, Greg Kroah-Hartman wrote: On Sun, Oct 16, 2016 at 05:16:04PM +0200, Vegard Nossum wrote: Hi, The first two patches in the series fix the concrete bug (a boot crash when using gcc 7.0+) by defining new wrappers for arrays defined in linker scripts. These two patches

possible if/else bug in dc395x.c

2016-10-16 Thread Nicholas Mc Guire
srb->end_message, CHECK_CONDITION); (line numbers from 4.9.0-rc1, localversion-next next-20161016) The if and else are identical, these lines date back to pre-git (so git blame assigns them to the initial 2.6.12-rc2 import). Not sure if it makes much sense trying t

possible if/else bug in dc395x.c

2016-10-16 Thread Nicholas Mc Guire
srb->end_message, CHECK_CONDITION); (line numbers from 4.9.0-rc1, localversion-next next-20161016) The if and else are identical, these lines date back to pre-git (so git blame assigns them to the initial 2.6.12-rc2 import). Not sure if it makes much sense trying t

[PATCH v2 0/3] Coccinelle: misc: Improve the script for more accurate results

2016-10-16 Thread Vaishali Thakkar
Few changes to improve the results given by the irqf_oneshot.cocci: - Change in the matching rules to eliminate false postives in the patch mode - Change in the context mode to eliminate false postives in the context mode - Support for the missing devm_request_threaded_irq in context, report

[PATCH v2 2/3] Coccinelle: misc: Improve the result given by context mode

2016-10-16 Thread Vaishali Thakkar
To eliminate false positives given by the context mode, add necessary arguments for the function request_threaded_irq. Signed-off-by: Vaishali Thakkar --- scripts/coccinelle/misc/irqf_oneshot.cocci | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: MD-RAID: Use seq_putc() in three status functions?

2016-10-16 Thread SF Markus Elfring
> Yes. - I got such an impression. Correction: * Is it more efficient to call the function "seq_putc" for the desired data processing for a single character than to pass it to the function "seq_printf" in a string? * Will the required data transfer shrink a bit for the affected functions

[PATCH v2 1/3] Coccinelle: misc: Improve the matching of rules

2016-10-16 Thread Vaishali Thakkar
Currently because of the left associativity of the operators, pattern IRQF_ONESHOT | flags does not match with the pattern when we have more than one flag after the disjunction. This eventually results in giving false positives by the script. This patch eliminates these FPs by improving the rule.

[PATCH v2 2/3] Coccinelle: misc: Improve the result given by context mode

2016-10-16 Thread Vaishali Thakkar
To eliminate false positives given by the context mode, add necessary arguments for the function request_threaded_irq. Signed-off-by: Vaishali Thakkar --- scripts/coccinelle/misc/irqf_oneshot.cocci | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: MD-RAID: Use seq_putc() in three status functions?

2016-10-16 Thread SF Markus Elfring
> Yes. - I got such an impression. Correction: * Is it more efficient to call the function "seq_putc" for the desired data processing for a single character than to pass it to the function "seq_printf" in a string? * Will the required data transfer shrink a bit for the affected functions

[PATCH v2 1/3] Coccinelle: misc: Improve the matching of rules

2016-10-16 Thread Vaishali Thakkar
Currently because of the left associativity of the operators, pattern IRQF_ONESHOT | flags does not match with the pattern when we have more than one flag after the disjunction. This eventually results in giving false positives by the script. This patch eliminates these FPs by improving the rule.

[PATCH v2 0/3] Coccinelle: misc: Improve the script for more accurate results

2016-10-16 Thread Vaishali Thakkar
Few changes to improve the results given by the irqf_oneshot.cocci: - Change in the matching rules to eliminate false postives in the patch mode - Change in the context mode to eliminate false postives in the context mode - Support for the missing devm_request_threaded_irq in context, report

Re: [PATCH] wusb: Stop using the stack for sg crypto scratch space

2016-10-16 Thread Andy Lutomirski
On Thu, Oct 6, 2016 at 10:25 AM, Andy Lutomirski wrote: > Pointing an sg list at the stack is verboten and, with > CONFIG_VMAP_STACK=y, will malfunction. Use kmalloc for the wusb > crypto stack space instead. > > Untested -- I'm not entirely convinced that this hardware exists

Re: [PATCH] wusb: Stop using the stack for sg crypto scratch space

2016-10-16 Thread Andy Lutomirski
On Thu, Oct 6, 2016 at 10:25 AM, Andy Lutomirski wrote: > Pointing an sg list at the stack is verboten and, with > CONFIG_VMAP_STACK=y, will malfunction. Use kmalloc for the wusb > crypto stack space instead. > > Untested -- I'm not entirely convinced that this hardware exists in > the wild.

Re: [PATCH 57/80] staging: lustre: osc: revise unstable pages accounting

2016-10-16 Thread Oleg Drokin
On Oct 16, 2016, at 11:14 AM, Greg Kroah-Hartman wrote: > Digging up an old email... > > On Tue, Aug 16, 2016 at 04:19:10PM -0400, James Simmons wrote: >> From: Jinshan Xiong >> >> A few changes are made in this patch for unstable pages tracking: >> >> 1. Remove

Re: [PATCH 57/80] staging: lustre: osc: revise unstable pages accounting

2016-10-16 Thread Oleg Drokin
On Oct 16, 2016, at 11:14 AM, Greg Kroah-Hartman wrote: > Digging up an old email... > > On Tue, Aug 16, 2016 at 04:19:10PM -0400, James Simmons wrote: >> From: Jinshan Xiong >> >> A few changes are made in this patch for unstable pages tracking: >> >> 1. Remove kernel NFS unstable pages

Re: virtio-rng scatterlist null pointer dereference with VMAP_STACK=y

2016-10-16 Thread Andy Lutomirski
On Sun, Oct 16, 2016 at 9:59 AM, Andy Lutomirski wrote: > On Sat, Oct 15, 2016 at 5:21 PM, Matt Mullins wrote: >> With VMAP_STACK=y and HW_RANDOM_VIRTIO=y, I get the following crash: >> >> [1.470437] BUG: unable to handle kernel NULL pointer dereference

Re: [PATCH] ARM: at91/dt: pullup dbgu rx instead of tx

2016-10-16 Thread Sylvain Rochet
Hi Peter, On Sun, Oct 16, 2016 at 12:57:09PM +0200, Peter Rosin wrote: > Hi again, > > I forgot about this, and it's been a year. But isn't it time to > upstream those pull-up fixes that Sylvain provided? Thank you for your reminder, actually I waited an answer on this nice subject:

Re: virtio-rng scatterlist null pointer dereference with VMAP_STACK=y

2016-10-16 Thread Andy Lutomirski
On Sun, Oct 16, 2016 at 9:59 AM, Andy Lutomirski wrote: > On Sat, Oct 15, 2016 at 5:21 PM, Matt Mullins wrote: >> With VMAP_STACK=y and HW_RANDOM_VIRTIO=y, I get the following crash: >> >> [1.470437] BUG: unable to handle kernel NULL pointer dereference at >> (null) >> [1.473350] IP:

Re: [PATCH] ARM: at91/dt: pullup dbgu rx instead of tx

2016-10-16 Thread Sylvain Rochet
Hi Peter, On Sun, Oct 16, 2016 at 12:57:09PM +0200, Peter Rosin wrote: > Hi again, > > I forgot about this, and it's been a year. But isn't it time to > upstream those pull-up fixes that Sylvain provided? Thank you for your reminder, actually I waited an answer on this nice subject:

Re: MD-RAID: Use seq_putc() in three status functions?

2016-10-16 Thread SF Markus Elfring
> Does it improve code? Does it improve anything? Yes. - I got such an impression. * Is it more efficient to call the function "seq_printf" for the desired data processing for a single character than to pass it to the function "" in a string? * Will the required data transfer shrink a bit

Re: MD-RAID: Use seq_putc() in three status functions?

2016-10-16 Thread SF Markus Elfring
> Does it improve code? Does it improve anything? Yes. - I got such an impression. * Is it more efficient to call the function "seq_printf" for the desired data processing for a single character than to pass it to the function "" in a string? * Will the required data transfer shrink a bit

Re: [PATCH 1/1] dmaengine: imx-sdma - correct the dma transfer residue calculation

2016-10-16 Thread Marek Vasut
On 10/11/2016 01:13 PM, Nandor Han wrote: > The residue calculation was taking in consideration that dma > transaction status will be always retrieved in the dma callback > used to inform that dma transfer is complete. However this is not > the case for all subsystems that use dma. Some subsystems

Re: [PATCH 1/1] dmaengine: imx-sdma - correct the dma transfer residue calculation

2016-10-16 Thread Marek Vasut
On 10/11/2016 01:13 PM, Nandor Han wrote: > The residue calculation was taking in consideration that dma > transaction status will be always retrieved in the dma callback > used to inform that dma transfer is complete. However this is not > the case for all subsystems that use dma. Some subsystems

Re: virtio-rng scatterlist null pointer dereference with VMAP_STACK=y

2016-10-16 Thread Andy Lutomirski
On Sat, Oct 15, 2016 at 5:21 PM, Matt Mullins wrote: > With VMAP_STACK=y and HW_RANDOM_VIRTIO=y, I get the following crash: > > [1.470437] BUG: unable to handle kernel NULL pointer dereference at > (null) > [1.473350] IP: [] sg_init_one+0x65/0x90 > [1.474658] PGD 0

Re: virtio-rng scatterlist null pointer dereference with VMAP_STACK=y

2016-10-16 Thread Andy Lutomirski
On Sat, Oct 15, 2016 at 5:21 PM, Matt Mullins wrote: > With VMAP_STACK=y and HW_RANDOM_VIRTIO=y, I get the following crash: > > [1.470437] BUG: unable to handle kernel NULL pointer dereference at > (null) > [1.473350] IP: [] sg_init_one+0x65/0x90 > [1.474658] PGD 0 > [1.475169]

Re: [PATCH] MD-RAID: Use seq_putc() in three status functions

2016-10-16 Thread Hannes Reinecke
On 10/16/2016 10:20 AM, SF Markus Elfring wrote: From: Markus Elfring Date: Sun, 16 Oct 2016 10:10:28 +0200 A single character (a closing square bracket) should be put into a sequence at the end in these functions. Thus use the corresponding function "seq_putc".

Re: [PATCH] MD-RAID: Use seq_putc() in three status functions

2016-10-16 Thread Hannes Reinecke
On 10/16/2016 10:20 AM, SF Markus Elfring wrote: From: Markus Elfring Date: Sun, 16 Oct 2016 10:10:28 +0200 A single character (a closing square bracket) should be put into a sequence at the end in these functions. Thus use the corresponding function "seq_putc". This issue was detected also

Re: [PATCH 11/12] jump_label: declare jump table as external array

2016-10-16 Thread Vegard Nossum
On 10/16/2016 06:25 PM, Peter Zijlstra wrote: On Sun, Oct 16, 2016 at 05:16:15PM +0200, Vegard Nossum wrote: Cc: Jason Baron Cc: Peter Zijlstra Cc: Steven Rostedt Signed-off-by: Vegard Nossum NAK,

Re: [PATCH 11/12] jump_label: declare jump table as external array

2016-10-16 Thread Vegard Nossum
On 10/16/2016 06:25 PM, Peter Zijlstra wrote: On Sun, Oct 16, 2016 at 05:16:15PM +0200, Vegard Nossum wrote: Cc: Jason Baron Cc: Peter Zijlstra Cc: Steven Rostedt Signed-off-by: Vegard Nossum NAK, -ENOCHANGELOG. Hi Peter, It's true I didn't put an RFC tag on this (mostly because

Re: [PATCH 4/4] x86, hotplug: Use hlt instead of mwait when resuming from hibernation

2016-10-16 Thread Andy Lutomirski
On Sat, Oct 8, 2016 at 3:31 AM, Rafael J. Wysocki wrote: > On Fri, Oct 7, 2016 at 9:47 PM, Andy Lutomirski wrote: >> On 06/25/2016 09:19 AM, Chen Yu wrote: >>> >>> Here's the story of what the problem is, why this >>> happened, and why this patch looks like

Re: [PATCH 4/4] x86, hotplug: Use hlt instead of mwait when resuming from hibernation

2016-10-16 Thread Andy Lutomirski
On Sat, Oct 8, 2016 at 3:31 AM, Rafael J. Wysocki wrote: > On Fri, Oct 7, 2016 at 9:47 PM, Andy Lutomirski wrote: >> On 06/25/2016 09:19 AM, Chen Yu wrote: >>> >>> Here's the story of what the problem is, why this >>> happened, and why this patch looks like this: >>> >>> Stress test from Varun

Re: fs/xfs/xfs_ondisk.h:96:2: error: call to '__compiletime_assert_96' declared with attribute error: XFS: sizeof(xfs_dir2_sf_entry_t) is wrong, expected 3

2016-10-16 Thread Christoph Hellwig
> head: 1001354ca34179f3db924eb66672442a173147dc That's Linux 4.9-rc1 > 30cbc591 Darrick J. Wong 2016-03-09 @86 > XFS_CHECK_STRUCT_SIZE(xfs_dir2_data_unused_t, 6); But that's not how xfs_ondisk.h in 4.9-rc1 looks like > 30cbc591 Darrick J. Wong 2016-03-09 @96 >

Re: fs/xfs/xfs_ondisk.h:96:2: error: call to '__compiletime_assert_96' declared with attribute error: XFS: sizeof(xfs_dir2_sf_entry_t) is wrong, expected 3

2016-10-16 Thread Christoph Hellwig
> head: 1001354ca34179f3db924eb66672442a173147dc That's Linux 4.9-rc1 > 30cbc591 Darrick J. Wong 2016-03-09 @86 > XFS_CHECK_STRUCT_SIZE(xfs_dir2_data_unused_t, 6); But that's not how xfs_ondisk.h in 4.9-rc1 looks like > 30cbc591 Darrick J. Wong 2016-03-09 @96 >

Re: [PATCH 08/10] fault injection: down_{read,write}_trylock() fault injection

2016-10-16 Thread Peter Zijlstra
On Sun, Oct 16, 2016 at 05:56:10PM +0200, Vegard Nossum wrote: > Cc: Peter Zijlstra > Cc: Ingo Molnar > Signed-off-by: Vegard Nossum Guess what... NAK

Re: [PATCH 08/10] fault injection: down_{read,write}_trylock() fault injection

2016-10-16 Thread Peter Zijlstra
On Sun, Oct 16, 2016 at 05:56:10PM +0200, Vegard Nossum wrote: > Cc: Peter Zijlstra > Cc: Ingo Molnar > Signed-off-by: Vegard Nossum Guess what... NAK

Re: [tip:x86/urgent] x86/cpufeature: Add AVX512_4VNNIW and AVX512_4FMAPS features

2016-10-16 Thread Borislav Petkov
On Sun, Oct 16, 2016 at 09:02:51AM -0700, h...@zytor.com wrote: > No, please. That would be worse than the disease. Why not? I did that recently with a bunch of leaves and there were no issues: 2ccd71f1b278 ("x86/cpufeature: Move some of the scattered feature bits to x86_capability") There

Re: [tip:x86/urgent] x86/cpufeature: Add AVX512_4VNNIW and AVX512_4FMAPS features

2016-10-16 Thread Borislav Petkov
On Sun, Oct 16, 2016 at 09:02:51AM -0700, h...@zytor.com wrote: > No, please. That would be worse than the disease. Why not? I did that recently with a bunch of leaves and there were no issues: 2ccd71f1b278 ("x86/cpufeature: Move some of the scattered feature bits to x86_capability") There

Re: [PATCH 07/10] fault injection: down_trylock() fault injection

2016-10-16 Thread Peter Zijlstra
On Sun, Oct 16, 2016 at 05:56:09PM +0200, Vegard Nossum wrote: > Cc: Peter Zijlstra > Cc: Ingo Molnar > Signed-off-by: Vegard Nossum Very tiring this.. NAK -ENOCHANGELOG

Re: [PATCH 09/10] fault injection: spin_trylock() fault injection

2016-10-16 Thread Peter Zijlstra
On Sun, Oct 16, 2016 at 05:56:11PM +0200, Vegard Nossum wrote: > Cc: Peter Zijlstra > Cc: Ingo Molnar > Signed-off-by: Vegard Nossum *sigh*

<    1   2   3   4   5   >