Re: [GIT PULL] EFI urgent fixes

2014-09-26 Thread Ingo Molnar
* Matt Fleming wrote: > On Fri, 26 Sep, at 01:35:10PM, Ingo Molnar wrote: > > > > A delta fix would be nice at this point, I've got the x86 side > > tested and besides this build bug it's ready to go to Linus. > > Given that the typo doesn't result in an actual build failure, do you > still

Re: [PATCH v3 1/2] Return a value from printk_ratelimited

2014-09-26 Thread Joe Perches
On Fri, 2014-09-26 at 22:36 -0700, Omar Sandoval wrote: > printk returns an integer; there's no reason for printk_ratelimited to swallow > it. > > Signed-off-by: Omar Sandoval > Acked-by: Paul E. McKenney > --- I'd prefer to keep it the way it is actually. I've submitted several patches to

Re: [PATCH v3 0/2] Move BTRFS RCU string to common library

2014-09-26 Thread Omar Sandoval
On Fri, Sep 26, 2014 at 10:36:47PM -0700, Omar Sandoval wrote: > This patch series makes the generic RCU string library used internally by > BTRFS > accessible by anyone. > > The first patch makes printk_ratelimited pass on the return value from printk. > Version 3 gives the temporary return

[PATCH v3 2/2] Move BTRFS RCU string to common library

2014-09-26 Thread Omar Sandoval
The RCU-friendly string API used internally by BTRFS is generic enough for common use. This doesn't add any new functionality, but instead just moves the code and documents the existing API. Signed-off-by: Omar Sandoval Reviewed-by: Josh Triplett Acked-by: Paul E. McKenney ---

[PATCH v3 0/2] Move BTRFS RCU string to common library

2014-09-26 Thread Omar Sandoval
This patch series makes the generic RCU string library used internally by BTRFS accessible by anyone. The first patch makes printk_ratelimited pass on the return value from printk. Version 3 gives the temporary return variable a unique name to avoid name clashes. The second patch moves the RCU

[PATCH v3 1/2] Return a value from printk_ratelimited

2014-09-26 Thread Omar Sandoval
printk returns an integer; there's no reason for printk_ratelimited to swallow it. Signed-off-by: Omar Sandoval Acked-by: Paul E. McKenney --- include/linux/printk.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/linux/printk.h b/include/linux/printk.h index

Re: [RESEND PATCH] acpi-cpufreq: get the cur_freq from acpi_processor_performance states

2014-09-26 Thread Wang Weidong
On 2014/9/27 7:21, Rafael J. Wysocki wrote: > On Thursday, August 21, 2014 01:55:15 PM Wang Weidong wrote: >> As the initialized freq_tables maybe different from the p-states >> values, so the array index is different as well. >> >> p-states value: [2400 2400 2000 ...], while the freq_tables: >>

[3.10.49 stable kernel] crash in process_one_work

2014-09-26 Thread Arun KS
Hello Tejun, I m seen the following crash in 3.10 kernel workqueue. [ 1133.893817] Unable to handle kernel NULL pointer dereference at virtual address 0004 [ 1133.893821] pgd = c0004000 [ 1133.893827] [0004] *pgd= [ 1133.893834] Internal error: Oops: 5 [#1] PREEMPT SMP ARM [

Re: [OPW kernel] checkpatch: Does anyone care that comments are freeform aligned?

2014-09-26 Thread Joe Perches
On Fri, 2014-09-26 at 16:03 -0700, j...@joshtriplett.org wrote: > On Wed, Sep 24, 2014 at 11:41:14AM -0700, Joe Perches wrote: > > On Wed, 2014-09-24 at 07:47 +0200, Julia Lawall wrote: > > > In the following patch extract, one line is indented with spaces rather > > > than tabs. Is it

Re: [PATCH v2] vfs: Don't exchange "short" filenames unconditionally.

2014-09-26 Thread Al Viro
On Fri, Sep 26, 2014 at 05:44:42PM +0100, Al Viro wrote: > FWIW, the longer I'm looking at that, the more it seems that __d_move() and > __d_materialise_dentry() ought to be merged. The thing is, both callers of > the latter are followed by the same sequence: >

Re: [PATCH v2 0/3] fuse: Add support for mounts from pid/user namespaces

2014-09-26 Thread Seth Forshee
On Fri, Sep 26, 2014 at 06:41:33PM -0700, Eric W. Biederman wrote: > >> I am on the fence about what to do when a uid from the filesystem server > >> or for other filesystems the on-disk data structures does not map, but > >> make_bad_inode is simpler in conception. So make_bad_inode seems like >

[PATCH] regmap: fix NULL pointer dereference in _regmap_write/read

2014-09-26 Thread Pankaj Dubey
If LOG_DEVICE is defined and map->dev is NULL it will lead to NULL pointer dereference. This patch fixes this issue by adding check for dev->NULL in all such places in regmap.c Signed-off-by: Pankaj Dubey --- drivers/base/regmap/regmap.c |4 ++-- 1 file changed, 2 insertions(+), 2

Re: [PATCH v2] next: openrisc: Fix build

2014-09-26 Thread Stefan Kristiansson
On Fri, Sep 26, 2014 at 07:51:49PM +0200, Jonas Bonn wrote: > On 09/26/2014 06:05 PM, Guenter Roeck wrote: > > openrisc:defconfig fails to build in next-20140926 with the following error. > > > > In file included from arch/openrisc/kernel/signal.c:31:0: > > ./arch/ope

Re: [PATCH v4] power: reset: use restart_notifier mechanism for msm-poweroff

2014-09-26 Thread Pramod Gurav
On 27-09-2014 01:55 AM, Sebastian Reichel wrote: > Hi, > > On Thu, Sep 25, 2014 at 05:03:51PM +0530, Pramod Gurav wrote: >> This change replaces use of arm_pm_restart with recently introduced >> reset mechanism in Linux kernel called restart_notifier. >> >> Choosing priority 128, which is default

Re: [PATCH] mmc: atmel-mci: fix mismatched section on atmci_cleanup_slot

2014-09-26 Thread Pramod Gurav
Hi Arnd, On Sat, Sep 27, 2014 at 1:04 AM, Arnd Bergmann wrote: > As of 528bc7808f4e ("mmc: atmel-mci: Release mmc resources on failure in > probe"), > the atmci_probe() function calls atmci_cleanup_slot in the failure path. > > This causes a new warning whenever the driver is built: > >

[PATCH] net: llc: check for device before dereferencing

2014-09-26 Thread Sasha Levin
] PGD 5edcd067 PUD 5edce067 PMD 0 [ 430.551939] Oops: [#1] PREEMPT SMP DEBUG_PAGEALLOC KASAN [ 430.551939] Dumping ftrace buffer: [ 430.551939](ftrace buffer empty) [ 430.551939] Modules linked in: [ 430.551939] CPU: 2 PID: 9395 Comm: trinity-c261 Not tainted 3.17.0-rc6-next-20140926

Re: [PATCH v2 0/3] fuse: Add support for mounts from pid/user namespaces

2014-09-26 Thread Eric W. Biederman
Seth Forshee writes: > On Thu, Sep 25, 2014 at 12:14:01PM -0700, Eric W. Biederman wrote: >> Sorry iattr_to_setattr look for from_kuid and from_kgid. >> >> The call path is >> fuse_setattr >>fuse_do_setattr >> iattr_to_fattr. > > Bah. Sorry, I misread that originally and thought you

Re: [PATCH 0/5] HID: wacom: introduce generic HID handling

2014-09-26 Thread Jason Gerecke
On Fri, Sep 26, 2014 at 4:03 AM, Jiri Kosina wrote: > On Tue, 23 Sep 2014, Benjamin Tissoires wrote: > >> Hi guys, >> >> So, this patch series aims at supporting natively any future HID compliant >> wacom >> tablet. Those found on the various laptops (ISDv4/5) already are HID >> compliant >>

[PATCH] media, platform, LLVMLinux: Remove nested function from ti-vpe

2014-09-26 Thread Behan Webster
Replace the use of nested functions where a normal function will suffice. Nested functions are not liked by upstream kernel developers in general. Their use breaks the use of clang as a compiler, and doesn't make the code any better. This code now works for both gcc and clang. Signed-off-by:

[PATCH 2/2] arm, fbdev, omap2, LLVMLinux: Remove nested function from omapfb

2014-09-26 Thread Behan Webster
Replace the use of nested functions where a normal function will suffice. Nested functions are not liked by upstream kernel developers in general. Their use breaks the use of clang as a compiler, and doesn't make the code any better. This code now works for both gcc and clang. Signed-off-by:

[PATCH 0/2] LLVMLinux: Patches to enable the kernel to be compiled with clang/LLVM

2014-09-26 Thread Behan Webster
Replace the use of nested functions where a normal function will suffice. Nested functions are not liked by upstream kernel developers in general. Their use breaks the use of clang as a compiler, and doesn't make the code any better. This code now works for both gcc and clang. The LLVMLinux

[PATCH 1/2] arm, fbdev, omap2, LLVMLinux: Remove nested function from omap2 dss

2014-09-26 Thread Behan Webster
Replace the use of nested functions where a normal function will suffice. Nested functions are not liked by upstream kernel developers in general. Their use breaks the use of clang as a compiler, and doesn't make the code any better. This code now works for both gcc and clang. Signed-off-by:

[PATCH] md, sysfs, LLVMLinux: Remove nested function from bcache sysfs

2014-09-26 Thread Behan Webster
Replace the use of nested functions where a normal function will suffice. Nested functions are not liked by upstream kernel developers in general. Their use breaks the use of clang as a compiler, and doesn't make the code any better. This code now works for both gcc and clang. Signed-off-by:

[PATCH 02/10] power/reset: xgene: Return -ENOMEM if out of memory

2014-09-26 Thread Guenter Roeck
It is customary to return an error code of -ENOMEM if the system is out of memory. Also, in that case, the infrastructure will report an error, so it is unnecessary to report it again. Cc: Loc Ho Signed-off-by: Guenter Roeck --- drivers/power/reset/xgene-reboot.c | 6 ++ 1 file changed, 2

[PATCH 1/2] DT: add binding for mxs regulator

2014-09-26 Thread Stefan Wahren
This patch adds the Device tree bindings for the Freescale MXS on-chip regulators. Signed-off-by: Stefan Wahren --- .../bindings/regulator/mxs-regulator.txt | 36 1 file changed, 36 insertions(+) create mode 100644

[PATCH 0/2] regulator: add support for mxs regulator

2014-09-26 Thread Stefan Wahren
This patch series adds support for Freescale i.MX23, i.xM28 on-chip regulators: vddd, vdda, vddio This driver based on the Freescale high level [1] and low level driver [2], but contains the following changes: * devicetree support * fix for regulator modes * drop support for overall_current and

[PATCH 2/2] regulator: add mxs regulator driver

2014-09-26 Thread Stefan Wahren
This patch adds driver support for Freescale i.MX23, i.MX28 on-chip regulators. The driver supports the following regulators: vddd, vdda and vddio. Signed-off-by: Stefan Wahren --- drivers/regulator/Kconfig | 11 ++ drivers/regulator/Makefile|1 +

[PATCH 08/10] power/reset: keystone: Register with kernel restart handler

2014-09-26 Thread Guenter Roeck
Register with kernel restart handler instead of setting arm_pm_restart directly. Move notifier registration to the end of the probe function to avoid having to implement error handling. Cc: Ivan Khoronzhuk Cc: Santosh Shilimkar Signed-off-by: Guenter Roeck ---

[PATCH 04/10] power/reset: xgene: Use local variable dev instead of pdev->dev

2014-09-26 Thread Guenter Roeck
Using a local variable dev to point to the device is simpler then repeatedly dereferencing pdev->dev. Cc: Loc Ho Signed-off-by: Guenter Roeck --- drivers/power/reset/xgene-reboot.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git

[PATCH 05/10] power/reset: xgene: Use mdelay instead of jiffies based timeout

2014-09-26 Thread Guenter Roeck
jiffies are not running at this stage of system shutdown, meaning an error in the reset function would never be reported. Replace with mdelay(). Cc: Loc Ho Signed-off-by: Guenter Roeck --- drivers/power/reset/xgene-reboot.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff

[PATCH 06/10] power/reset: xgene: Register with kernel restart handler

2014-09-26 Thread Guenter Roeck
Register with kernel restart handler instead of setting arm_pm_restart directly. This patch also addresses the following compile warning. drivers/power/reset/xgene-reboot.c: In function 'xgene_reboot_probe': drivers/power/reset/xgene-reboot.c:77:17: warning: assignment from incompatible

[PATCH 07/10] power/reset: axxia: Register with kernel restart handler

2014-09-26 Thread Guenter Roeck
Register with kernel restart handler instead of setting arm_pm_restart directly. Cc: Anders Berg Signed-off-by: Guenter Roeck --- drivers/power/reset/axxia-reset.c | 19 ++- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/drivers/power/reset/axxia-reset.c

Re: [PATCH] clk, ti, LLVMLinux: Move __init outside of type definition

2014-09-26 Thread Behan Webster
On 09/26/14 17:55, Felipe Balbi wrote: On Fri, Sep 26, 2014 at 05:31:48PM -0700, Behan Webster wrote: As written, the __init for ti_clk_get_div_table is in the middle of the return type. The gcc documentation indicates that section attributes should be added to the end of the function

[PATCH 10/10] power/reset: brcmstb: Register with kernel restart handler

2014-09-26 Thread Guenter Roeck
Register with kernel restart handler instead of setting arm_pm_restart directly. Cc: Marc Carino Cc: Brian Norris Signed-off-by: Guenter Roeck --- drivers/power/reset/brcmstb-reboot.c | 26 ++ 1 file changed, 18 insertions(+), 8 deletions(-) diff --git

[PATCH 09/10] power/reset: hisi: Register with kernel restart handler

2014-09-26 Thread Guenter Roeck
Register with kernel restart handler instead of setting arm_pm_restart directly. Cc: Haojian Zhuang Signed-off-by: Guenter Roeck --- drivers/power/reset/hisi-reboot.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/drivers/power/reset/hisi-reboot.c

[PATCH 03/10] power/reset: xgene: Drop devm_kfree

2014-09-26 Thread Guenter Roeck
Calling devm_kfree is unnecessary. Drop it. Cc: Loc Ho Signed-off-by: Guenter Roeck --- drivers/power/reset/xgene-reboot.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/power/reset/xgene-reboot.c b/drivers/power/reset/xgene-reboot.c index 20cf7c3..aac0287 100644 ---

[PATCH 01/10] power/reset: vexpress: Register with kernel restart handler

2014-09-26 Thread Guenter Roeck
Use the kernel restart handler instead of setting arm_pm_restart directly. This allows for more than one restart handler in the system. Signed-off-by: Guenter Roeck --- drivers/power/reset/vexpress-poweroff.c | 33 - 1 file changed, 24 insertions(+), 9

[PATCH 00/10] power/reset: Register drivers with restart handler

2014-09-26 Thread Guenter Roeck
Convert drivers to use the kernel restart handler instead of setting arm_pm_restart directly. This patch set depends on the kernel restart handler patchset submitted earlier. Patch 01/10 was tested with qemu. All other patches were compile tested only. Some of the restart handlers loop forever

Re: [PATCH] clk, ti, LLVMLinux: Move __init outside of type definition

2014-09-26 Thread Felipe Balbi
On Fri, Sep 26, 2014 at 05:31:48PM -0700, Behan Webster wrote: > As written, the __init for ti_clk_get_div_table is in the middle of the return > type. > > The gcc documentation indicates that section attributes should be added to the > end of the function declaration: > > extern void foobar

Re: linux-next: Tree for Sep 26 (media/pci/pt3)

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 10:01:47 -0700 Randy Dunlap escreveu: > On 09/26/14 04:10, Stephen Rothwell wrote: > > Hi all, > > > > There will be no linux-next release on Monday. > > > > This has not been a good day :-( > > > > Changes since 20140925: > > > on x86_64: > when CONFIG_MODULES is not

[PATCH] clk, ti, LLVMLinux: Move __init outside of type definition

2014-09-26 Thread Behan Webster
As written, the __init for ti_clk_get_div_table is in the middle of the return type. The gcc documentation indicates that section attributes should be added to the end of the function declaration: extern void foobar (void) __attribute__ ((section ("bar"))); However gcc seems to be very

Re: [PATCH v13 0/9] Per-user clock constraints

2014-09-26 Thread Stephen Boyd
On 09/26/14 16:20, Mike Turquette wrote: > Quoting Tomeu Vizoso (2014-09-26 01:09:20) >> On 09/26/2014 03:29 AM, Stephen Boyd wrote: >> >>> We already have the consumer/provider split in the struct clk_hw and >>> struct clk separation. Why don't we just use struct clk_hw throughout >>> the

Re: [PATCH] clk: add gpio gated clock

2014-09-26 Thread Mike Turquette
Quoting Jyri Sarha (2014-09-11 01:44:24) > On 09/10/2014 01:14 AM, Mike Turquette wrote: > > Quoting Jyri Sarha (2014-09-05 05:21:34) > >> The added gpio-gate-clock is a basic clock that can be enabled and > >> disabled trough a gpio output. The DT binding document for the clock > >> is also

[PATCH 1/8] perf, tools: Support handling complete branch stacks as histograms

2014-09-26 Thread Andi Kleen
From: Andi Kleen Currently branch stacks can be only shown as edge histograms for individual branches. I never found this display particularly useful. This implements an alternative mode that creates histograms over complete branch traces, instead of individual branches, similar to how normal

[PATCH 2/8] perf, tools: Add --branch-history option to report

2014-09-26 Thread Andi Kleen
From: Andi Kleen Add a --branch-history option to perf report that changes all the settings necessary for using the branches in callstacks. This is just a short cut to make this nicer to use, it does not enable any functionality by itself. v2: Change sort order. Rename option to

[PATCH 8/8] tools, perf: Add asprintf replacement

2014-09-26 Thread Andi Kleen
From: Andi Kleen asprintf corrupts memory on some older glibc versions. Provide a replacement. This fixes various segfaults with --branch-history on older Fedoras. Signed-off-by: Andi Kleen --- tools/perf/Makefile.perf| 1 + tools/perf/builtin-report.c | 3 ++-

[PATCH 7/8] tools, perf: Make srcline output address with -v

2014-09-26 Thread Andi Kleen
From: Andi Kleen When -v is specified always print the hex address for the srcline. Signed-off-by: Andi Kleen --- tools/perf/util/srcline.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/tools/perf/util/srcline.c b/tools/perf/util/srcline.c index

[PATCH 6/8] tools, perf: Make get_srcline fall back to sym+offset

2014-09-26 Thread Andi Kleen
From: Andi Kleen When the source line is not found fall back to sym + offset. This is generally much more useful than a raw address. For this we need to pass in the symbol from the caller. For some callers it's awkward to compute, so we stay at the old behaviour. Signed-off-by: Andi Kleen ---

Implement lbr-as-callgraph v10

2014-09-26 Thread Andi Kleen
This has been reviewed before. Any change to merge it now? [Rebase. Fix one conflict with an earlier tip change] [Just a repost after a rebase] [Even more review feedback and some bugs addressed.] [Only port to changes in perf/core. No other changes.] [Rebase to latest perf/core] [Another rebase.

[PATCH 3/8] perf, tools: Enable printing the srcline in the history

2014-09-26 Thread Andi Kleen
From: Andi Kleen For lbr-as-callgraph we need to see the line number in the history, because many LBR entries can be in a single function, and just showing the same function name many times is not useful. When the history code is configured to sort by address, also try to resolve the address to

[PATCH 5/8] perf, tools: Support source line numbers in annotate

2014-09-26 Thread Andi Kleen
From: Andi Kleen With srcline key/sort'ing it's useful to have line numbers in the annotate window. This patch implements this. Use objdump -l to request the line numbers and save them in the line structure. Then the browser displays them for source lines. The line numbers are not displayed by

[PATCH 4/8] perf, tools: Only print base source file for srcline

2014-09-26 Thread Andi Kleen
From: Andi Kleen For perf report with --sort srcline only print the base source file name. This makes the results generally fit much better to the screen. The path is usually not that useful anyways because it is often from different systems. Signed-off-by: Andi Kleen ---

[PATCH 1/2] Use faster check for modules in backtrace on 64bit

2014-09-26 Thread Andi Kleen
From: Andi Kleen On my workstation which has a lot of modules loaded: $ lsmod | wc -l 80 backtrace from the NMI for perf record -g can take a quite long time. This leads to frequent messages like: perf interrupt took too long (7852 > 7812), lowering kernel.perf_event_max_sample_rate to 16000

Re: [PATCH] x86 : Ensure X86_FLAGS_NT is cleared on syscall entry

2014-09-26 Thread Linus Torvalds
On Fri, Sep 26, 2014 at 3:00 PM, Chuck Ebbert wrote: > > I don't get it. Why isn't this patch acceptable, at least on x86-64 > where NT is never valid? Why do you think it's not acceptable? Why do you raise a stink *one* day after the patch - that seems to not be very important - is sent out?

Mount propagation issue

2014-09-26 Thread Ross Lagerwall
Hi, I have encountered a strange bug (on 3.14, 3.16.3 and master) regarding mount namespaces and a particular directory. Somehow, mounts are being mounted successfully but disappearing when the process dies, but only for /mnt/puppy, not for any other directory in the system: # grep puppy

Optimize backtrace code for perf PMI handler.

2014-09-26 Thread Andi Kleen
While doing perf record -g I regularly exceed the time quote for the NMI: This leads to perf shortening the period, which causes various problems. This patchkit optimizes two sources of longer latencies in the NMI backtrace code. There's probably more work needed to fix other sources of

[PATCH 2/2] x86: Only do a single page fault for copy_from_user_nmi

2014-09-26 Thread Andi Kleen
From: Andi Kleen When copy_from_user_nmi faults the copy_user_tail code ends up "replaying" the page faults to compute the exact tail bytes, (added with 112958). So we do an expensive page fault. And then we do it *again*. This ends up being very expensive in the PMI handler for any page fault

Re: [PATCH v2 2/2] Move BTRFS RCU string to common library

2014-09-26 Thread josh
On Fri, Sep 19, 2014 at 02:01:30AM -0700, Omar Sandoval wrote: > The RCU-friendy string API used internally by BTRFS is generic enough for > common use. This doesn't add any new functionality, but instead just moves the > code and documents the existing API. > > Signed-off-by: Omar Sandoval One

Re: [PATCH] clk: prevent erronous parsing of children during rate change

2014-09-26 Thread Mike Turquette
Quoting Tero Kristo (2014-09-26 00:18:55) > On 09/26/2014 04:35 AM, Stephen Boyd wrote: > > On 09/23/14 06:38, Tero Kristo wrote: > >> On 09/22/2014 10:18 PM, Stephen Boyd wrote: > >>> On 08/21, Tero Kristo wrote: > /* Skip children who will be reparented to another clock */ >

Re: [PATCH] partitions: efi.c: fix formatting and function readability

2014-09-26 Thread Davidlohr Bueso
On Fri, 2014-09-26 at 16:53 +0100, Matt Fleming wrote: > (No response from Davidlohr, try another address) > > On Wed, 03 Sep, at 11:07:20PM, Joseph Poirier wrote: > > From: jpoirier > > > > Fixed coding style issues, refactored the protective MBR check > > function for readability, and removed

Re: [PATCH v13 0/9] Per-user clock constraints

2014-09-26 Thread Mike Turquette
Quoting Tomeu Vizoso (2014-09-26 01:09:20) > On 09/26/2014 03:29 AM, Stephen Boyd wrote: > > On 09/24/14 01:27, Tomeu Vizoso wrote: > >> On 09/23/2014 10:59 PM, Stephen Boyd wrote: > >>> > >>> Any thoughts on my comments on patch set #10[1]? It seems like we can > >>> avoid having a flag day to

Re: [PATCH 1/1] jffs2: fix sparse warning: unexpected unlock

2014-09-26 Thread josh
On Mon, Sep 22, 2014 at 11:12:50AM -0700, Brian Norris wrote: > + linux-sparse > > On Thu, Sep 18, 2014 at 08:46:16PM +0200, Fabian Frederick wrote: > > fs/jffs2/summary.c:846:5: warning: context imbalance in > > 'jffs2_sum_write_sumnode' - unexpected unlock > > > > Signed-off-by: Fabian

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Alexei Starovoitov
On Fri, Sep 26, 2014 at 3:41 PM, Andy Lutomirski wrote: > On Fri, Sep 26, 2014 at 3:26 PM, Alexei Starovoitov wrote: >> On Fri, Sep 26, 2014 at 3:07 PM, Andy Lutomirski wrote: >>> On Fri, Sep 26, 2014 at 3:03 PM, Alexei Starovoitov >>> wrote: On Fri, Sep 26, 2014 at 2:47 PM, Andy

[PATCH] PCI: Intel 10G NIC ACS quirks

2014-09-26 Thread Alex Williamson
Intel has verified there is no peer-to-peer between functions for the below selection of 82598, 82599, and X520 10G NICs. These NICs lack an ACS capability, so we're not able to determine this isolation without the help of quirks. Generalize the Solarflare quirk and add these. Signed-off-by:

Re: [OPW kernel] checkpatch: Does anyone care that comments are freeform aligned?

2014-09-26 Thread josh
On Wed, Sep 24, 2014 at 11:41:14AM -0700, Joe Perches wrote: > On Wed, 2014-09-24 at 07:47 +0200, Julia Lawall wrote: > > In the following patch extract, one line is indented with spaces rather > > than tabs. Is it intentional that checkpatch doesn't complain about this, > > I guess due to the

Re: [RESEND PATCH] acpi-cpufreq: get the cur_freq from acpi_processor_performance states

2014-09-26 Thread Rafael J. Wysocki
On Thursday, August 21, 2014 01:55:15 PM Wang Weidong wrote: > As the initialized freq_tables maybe different from the p-states > values, so the array index is different as well. > > p-states value: [2400 2400 2000 ...], while the freq_tables: > [2400 2000 ... CPUFREQ_TABLE_END]. After setted the

Re: [PATCH v12 08/12] PCI: OF: Introduce helper function for retrieving PCI domain numbers

2014-09-26 Thread Liviu Dudau
On Fri, Sep 26, 2014 at 10:53:26PM +0100, Rob Herring wrote: > On Fri, Sep 26, 2014 at 4:20 PM, wrote: > > On Fri, Sep 26, 2014 at 01:20:39PM -0500, Rob Herring wrote: > >> On Tue, Sep 23, 2014 at 2:01 PM, Liviu Dudau wrote: > >> > Add pci_get_new_domain_nr() to allocate a new domain number >

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Andy Lutomirski
On Fri, Sep 26, 2014 at 3:26 PM, Alexei Starovoitov wrote: > On Fri, Sep 26, 2014 at 3:07 PM, Andy Lutomirski wrote: >> On Fri, Sep 26, 2014 at 3:03 PM, Alexei Starovoitov >> wrote: >>> On Fri, Sep 26, 2014 at 2:47 PM, Andy Lutomirski >>> wrote: Can't you just disallow the 1-byte

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Alexei Starovoitov
On Fri, Sep 26, 2014 at 3:07 PM, Andy Lutomirski wrote: > On Fri, Sep 26, 2014 at 3:03 PM, Alexei Starovoitov wrote: >> On Fri, Sep 26, 2014 at 2:47 PM, Andy Lutomirski wrote: >>> >>> Can't you just disallow the 1-byte write to the stack? >> >> of course not. >> That would be extremely limiting

Re: BUG: sleeping function called from invalid context at drivers/cpufreq/cpufreq.c:370

2014-09-26 Thread Rafael J. Wysocki
On Friday, September 26, 2014 09:54:00 AM Mike Galbraith wrote: > On Fri, 2014-09-26 at 08:24 +0200, Mike Galbraith wrote: > > (the hazards of multitasking.. post escaped early, and went to mostly > > the wrong folks) > > > > > > While testing some scheduler patches, the below pcc-cpufreq > >

RE: [PATCH] x86 : Ensure X86_FLAGS_NT is cleared on syscall entry

2014-09-26 Thread Anish Bhatt
> -Original Message- > From: Chuck Ebbert [mailto:cebbert.l...@gmail.com] > Sent: Friday, September 26, 2014 3:01 PM > To: Anish Bhatt > Cc: linux-kernel@vger.kernel.org; x...@kernel.org; t...@linutronix.de; > mi...@redhat.com; h...@zytor.com; sebast...@fds-team.de; Linus > Torvalds >

[PATCH 1/1] [RFC] kconfig: warn if an unknown symbol is selected

2014-09-26 Thread Paul Bolle
A select of an unknown symbol is basically treated as a nop and is silently skipped. This is annoying if the selected symbol contains a typo. It can also hide the fact that a treewide symbol cleanup was only done partially. There are also a few cases were this might have been done on purpose. But

[PATCH 0/1] [RFC] kconfig: warn if an unknown symbol is selected

2014-09-26 Thread Paul Bolle
em. Then I ran kconfig on all (500+) in tree defconfig files as of next-20140926. These are the select statements warns about when doing that. 1) arch/arm/Kconfig:429:warning: 'ARCH_EFM32' selects unknown symbol 'NO_DMA' Unknown on arm. Patch submitted. 2) arch/arm/mach-meson/Kconfig:

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Andy Lutomirski
On Fri, Sep 26, 2014 at 3:03 PM, Alexei Starovoitov wrote: > On Fri, Sep 26, 2014 at 2:47 PM, Andy Lutomirski wrote: >> >> Can't you just disallow the 1-byte write to the stack? > > of course not. > That would be extremely limiting to users. > Can you actually see yourself living with stack that

[PATCH] libata-sff: Fix controllers with no ctl port

2014-09-26 Thread Ondrej Zary
Currently, ata_sff_softreset is skipped for controllers with no ctl port. But that also skips ata_sff_dev_classify required for device detection. This means that libata is currently broken on controllers with no ctl port. No device connected: [1.872480] pata_isapnp 01:01.02: activated [

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Alexei Starovoitov
On Fri, Sep 26, 2014 at 2:47 PM, Andy Lutomirski wrote: > > Can't you just disallow the 1-byte write to the stack? of course not. That would be extremely limiting to users. Can you actually see yourself living with stack that only allows 8-byte writes/reads? The stack usage will increase a lot,

Re: [PATCH] x86 : Ensure X86_FLAGS_NT is cleared on syscall entry

2014-09-26 Thread Chuck Ebbert
On Thu, 25 Sep 2014 12:42:51 -0700 Anish Bhatt wrote: > The MSR_SYSCALL_MASK, which is responsible for clearing specific EFLAGS on > syscall entry, should also clear the nested task (NT) flag to be safe from > userspace injection. Without this fix the application segmentation > faults on

[PATCH v1 2/5] driver-core: enable drivers to opt-out of async probe

2014-09-26 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" We'll soon add generic support for asynchronous probe, before that gets merged lets let drivers annotate if they should never probe asynchronously. Cc: Tejun Heo Cc: Arjan van de Ven Cc: Greg Kroah-Hartman Cc: Doug Thompson Cc: Borislav Petkov Cc: Mauro Carvalho

[PATCH v1 3/5] amd64_edac: enforce synchronous probe

2014-09-26 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" While testing asynchronous PCI probe on this driver I noticed it failed so enforce just synchronouse probe for now. Asynchronous probe is not used by default and requires userepace intervention. Patches for its support will be merged later. [ 14.411083] AMD64 EDAC

[PATCH v1 4/5] driver-core: generalize freeing driver private member

2014-09-26 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" This will be used later. Cc: Tejun Heo Cc: Arjan van de Ven Cc: Greg Kroah-Hartman Cc: linux-kernel@vger.kernel.org Signed-off-by: Luis R. Rodriguez --- drivers/base/bus.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git

[PATCH v1 5/5] driver-core: add driver asynchronous probe support

2014-09-26 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" Some init systems may wish to express the desire to have device drivers run their device driver's bus probe() run asynchronously. This implements support for this and allows userspace to request async probe as a preference through a generic shared device driver module

[PATCH v1 0/5] driver-core: async probe support

2014-09-26 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" What started off as a witch hunt for a solution to the systemd kernel-module-loading-timeout-and-it-killing-drivers-issue (TM) ended up in our last discussions [0] agreement for us to add to the kernel what systemd expected us to be doing: asynchronous probing. This

[PATCH v1 1/5] module: add extra argument for parse_params() callback

2014-09-26 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" This adds an extra argument onto parse_params() to be used as a way to make the unused callback a bit more useful and generic by allowing the caller to pass on a data structure of its choice. An example use case is to allow us to easily make module parameters for every

Re: [PATCH v12 08/12] PCI: OF: Introduce helper function for retrieving PCI domain numbers

2014-09-26 Thread Rob Herring
On Fri, Sep 26, 2014 at 4:20 PM, wrote: > On Fri, Sep 26, 2014 at 01:20:39PM -0500, Rob Herring wrote: >> On Tue, Sep 23, 2014 at 2:01 PM, Liviu Dudau wrote: >> > Add pci_get_new_domain_nr() to allocate a new domain number >> > and of_get_pci_domain_nr() to retrieve the PCI domain number >> >

Re: [git pull] vfs fixes

2014-09-26 Thread Joachim Eastwood
On 26 September 2014 23:28, Joachim Eastwood wrote: > On 26 September 2014 22:58, Al Viro wrote: >> On Fri, Sep 26, 2014 at 10:46:14PM +0200, Joachim Eastwood wrote: >>> On 14 September 2014 21:47, Al Viro wrote: >>> > double iput() on failure exit in lustre, racy removal of spliced dentries

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Andy Lutomirski
On Fri, Sep 26, 2014 at 2:46 PM, Alexei Starovoitov wrote: > On Fri, Sep 26, 2014 at 1:42 PM, Andy Lutomirski wrote: >> On Fri, Sep 26, 2014 at 1:09 PM, Alexei Starovoitov >> wrote: >>> On Fri, Sep 26, 2014 at 12:51 PM, Andy Lutomirski >>> wrote: On Fri, Sep 26, 2014 at 12:34 PM, Andy

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Andy Lutomirski
On Fri, Sep 26, 2014 at 2:25 PM, Alexei Starovoitov wrote: > On Fri, Sep 26, 2014 at 1:39 PM, Andy Lutomirski wrote: >>> not quite. there is a distinction between key and value. >>> They both come from map definition and correspond to key_size >>> and value_size, so they have to have two

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Alexei Starovoitov
On Fri, Sep 26, 2014 at 1:42 PM, Andy Lutomirski wrote: > On Fri, Sep 26, 2014 at 1:09 PM, Alexei Starovoitov wrote: >> On Fri, Sep 26, 2014 at 12:51 PM, Andy Lutomirski >> wrote: >>> On Fri, Sep 26, 2014 at 12:34 PM, Andy Lutomirski >>> wrote: >>> >>> To add one more point: >>> >>> With the

Re: [PATCH v3 00/17] Cross-architecture definitions of relaxed MMIO accessors

2014-09-26 Thread Arnd Bergmann
On Friday 26 September 2014 14:46:08 Russell King - ARM Linux wrote: > > Obviously, this does nothing for the: > > include/asm-generic/io.h:804:29: error: redefinition of 'virt_to_bus' > include/asm-generic/io.h:809:21: error: redefinition of 'bus_to_virt' > > errors which are also reported in

Re: [PATCH] ACPI / i915: Update the condition to ignore firmware backlight change request

2014-09-26 Thread Rafael J. Wysocki
On Friday, September 26, 2014 10:30:08 AM Aaron Lu wrote: > Some of the Thinkpads' firmware will issue a backlight change request > through i915 operation region unconditionally on AC plug/unplug, the > backlight level used is arbitrary and thus should be ignored. This is > handled by commit

Re: [PATCH v12 08/12] PCI: OF: Introduce helper function for retrieving PCI domain numbers

2014-09-26 Thread liviu
On Fri, Sep 26, 2014 at 01:20:39PM -0500, Rob Herring wrote: > On Tue, Sep 23, 2014 at 2:01 PM, Liviu Dudau wrote: > > Add pci_get_new_domain_nr() to allocate a new domain number > > and of_get_pci_domain_nr() to retrieve the PCI domain number > > of a given device from DT. Host bridge drivers or

Re: [PATCH v2] x86: Quark: Add if/else to setup_arch for Quark TLB bug

2014-09-26 Thread Bryan O'Donoghue
On 26/09/14 21:59, Henrique de Moraes Holschuh wrote: I'm confused, now. Wasn't the other patch -- which just added a comment -- the one selected as a better fix, because there is absolutely no point in calling __flush_tlb() on Quark X1000 *right after* you just flushed the TLB [on these

Re: [git pull] vfs fixes

2014-09-26 Thread Joachim Eastwood
On 26 September 2014 22:58, Al Viro wrote: > On Fri, Sep 26, 2014 at 10:46:14PM +0200, Joachim Eastwood wrote: >> On 14 September 2014 21:47, Al Viro wrote: >> > double iput() on failure exit in lustre, racy removal of spliced dentries >> > from ->s_anon in __d_materialise_dentry() plus a bunch

Re: irq mask swapping during suspend/resume

2014-09-26 Thread Rafael J. Wysocki
On Friday, September 26, 2014 11:39:28 PM Rafael J. Wysocki wrote: > On Friday, September 26, 2014 09:58:39 AM Eric Caruso wrote: > > I was putting together a prototype for this, and ran into a design > > issue. It's not obvious how to get from the struct wakeup_source to > > places where we hold

Re: eBPF verifier thoughts (Re: [PATCH v15 net-next 00/11] eBPF syscall, verifier, testsuite)

2014-09-26 Thread Alexei Starovoitov
On Fri, Sep 26, 2014 at 1:39 PM, Andy Lutomirski wrote: >> not quite. there is a distinction between key and value. >> They both come from map definition and correspond to key_size >> and value_size, so they have to have two different corresponding >> _internal_ types 'ptr_to_map_key' and

[PATCH] dma: imx-sdma: fix another incorrect __init annotation

2014-09-26 Thread Arnd Bergmann
In e34b731faa7d1 ("dma: imx-sdma: Remove spurious __init annotation on sdma_probe()"), Mark found an extraneous __init label and fixed it. However, he missed another one, because now we get this other warning: WARNING: drivers/dma/imx-sdma.o(.text+0x3bb4): Section mismatch in reference from the

Re: [PATCH 01/11 v6] coresight: add CoreSight core layer framework

2014-09-26 Thread Mathieu Poirier
On 23 September 2014 23:45, Greg KH wrote: > On Thu, Sep 18, 2014 at 05:09:11PM -0600, Mathieu Poirier wrote: >> On 12 September 2014 12:16, Greg KH wrote: >> > On Fri, Sep 12, 2014 at 11:41:44AM -0600, Mathieu Poirier wrote: >> >> Good morning and thanks for the review. Pls see comments below.

Re: irq mask swapping during suspend/resume

2014-09-26 Thread Rafael J. Wysocki
On Friday, September 26, 2014 09:58:39 AM Eric Caruso wrote: > I was putting together a prototype for this, and ran into a design > issue. It's not obvious how to get from the struct wakeup_source to > places where we hold all of the relevant device information or irq > information. If we were to

Re: [PATCH v5 RESEND 1/3] ARM: EXYNOS: Move code from hotplug.c to platsmp.c

2014-09-26 Thread Arnd Bergmann
On Friday 05 September 2014, Krzysztof Kozlowski wrote: > The commit only moves code around with one additional observable change: > the hotplug.c was compiled with custom CFLAGS (-march=armv7-a). These > CFLAGS are not necessary any more. This turns out to be wrong, and your change broke

Re: Stupid NVIDIA 3D vgaarb.c patch

2014-09-26 Thread Ilia Mirkin
On Fri, Sep 26, 2014 at 5:08 PM, Aaron Plattner wrote: > On 09/23/2014 01:29 PM, Benjamin Herrenschmidt wrote: >> >> On Mon, 2014-09-22 at 13:43 -0700, Linus Torvalds wrote: >>> >>> Adding proper people and mailing lists.. >>> >>> The PCI_CLASS_DISPLAY_VGA test goes back to the very beginning by

  1   2   3   4   5   6   7   8   9   10   >