Re: [3.10.4] NFS locking panic, plus persisting NFS shutdown panic from 3.9.*

2013-08-05 Thread Jeff Layton
On Mon, 05 Aug 2013 15:48:01 +0100 Nix wrote: > On 5 Aug 2013, Jeff Layton stated: > > > On Sun, 04 Aug 2013 16:40:58 +0100 > > Nix wrote: > > > >> I just got this panic on 3.10.4, in the middle of a large parallel > >> compilation (of Chromium, as it happens) over NFSv3: > >> > >> [16364.5275

Re: [PATCH] WIP: HACK: LPAE, BOOTMEM and NO_BOOTMEM

2013-08-05 Thread Tejun Heo
Hello, On Fri, Aug 02, 2013 at 05:06:02PM -0400, Santosh Shilimkar wrote: > Looking at the situation, how about proceeding with patch updating > the bootmem API signatures to use phys_addr_t which can unblock me > to get my machine working. I'm not sure about that. No matter how you play it, it

Re: [PATCH 8/9] perf kvm: debug for missing vmexit/vmentry event

2013-08-05 Thread David Ahern
On 8/5/13 2:53 AM, Xiao Guangrong wrote: On 08/03/2013 04:05 AM, David Ahern wrote: Expected to have missing events for each vcpu when perf is started. After that should not have missing events. Sorry, i can not understand what's this info used for. For debugging. Things that should not happ

Re: [PATCH] fs/nfs/inode.c: adjust code alignment

2013-08-05 Thread Myklebust, Trond
On Mon, 2013-08-05 at 16:47 +0200, Julia Lawall wrote: > From: Julia Lawall > > Signed-off-by: Julia Lawall > > --- > > This patch adjusts the code so that the alignment matches the current > semantics. I have no idea if it is the intended semantics, though. Should > the call to nfs_setsecur

Re: [PATCH 9/9] perf kvm stat report: Add option to analyze specific VM

2013-08-05 Thread David Ahern
On 8/5/13 2:57 AM, Xiao Guangrong wrote: On 08/03/2013 04:05 AM, David Ahern wrote: Add an option to analyze a specific VM within a data file. This allows the collection of kvm events for all VMs and then analyze data for each VM (or set of VMs) individually. Interesting. But how can we know

Re: percpu: kernel BUG at mm/percpu.c:579!

2013-08-05 Thread Tejun Heo
e ] > [ 274.821320] kernel BUG at mm/percpu.c:579! Looks like double free. > [ 274.821848] invalid opcode: [#1] PREEMPT SMP DEBUG_PAGEALLOC > [ 274.822467] Modules linked in: > [ 274.823240] CPU: 13 PID: 58 Comm: rcuos/13 Tainted: GW > 3.11.0-rc4-next-20130

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Paolo Bonzini
> Only thing I am thinking is about KVM_FEATURE_PV_UNHALT, and > KVM_HC_KICK_CPU definition in the below hunk, that is needed by guest > as well. may be this header file change can be a separate patch so that > duplicate can be handled easily during merge? Sure, good idea. Paolo > I do testing o

Re: [PATCH v2 RESEND 13/18] x86, numa, mem_hotplug: Skip all the regions the kernel resides in.

2013-08-05 Thread Tejun Heo
On Mon, Aug 05, 2013 at 02:22:47PM +0800, Tang Chen wrote: > I have resent the v2 patch-set. Would you please give some more > comments about the memblock and x86 booting code modification ? Patch 13 still seems corrupt. Is it a problem on my side maybe? Nope, gmane raw message is corrupt too.

Re: [GIT PULL] tracing: final fixes for events and some

2013-08-05 Thread Steven Rostedt
On Mon, 2013-08-05 at 16:32 +0200, Ingo Molnar wrote: > * Steven Rostedt wrote: > > > Linus, > > > > Oleg Nesterov has been working hard in closing all the holes that can > > lead to race conditions between deleting an event and accessing an event > > debugfs file. This included a fix to the d

Re: [PATCH] regmap: cache: Make sure to sync the last register in a block

2013-08-05 Thread Mark Brown
On Mon, Aug 05, 2013 at 11:21:29AM +0200, Lars-Peter Clausen wrote: > regcache_sync_block_raw_flush() expects the address of the register after last > register that needs to be synced as its parameter. But the last call to > regcache_sync_block_raw_flush() in regcache_sync_block_raw() passes the >

Re: [PATCH 2/2] HID: hid-logitech-dj, querying_devices was never set

2013-08-05 Thread Benjamin Tissoires
On Mon, Aug 5, 2013 at 3:22 PM, Jiri Kosina wrote: > On Fri, 2 Aug 2013, Benjamin Tissoires wrote: > >> > Could you please elaborate? (and put an elaborate description to revert >> > commit log perhaps?) >> >> Sure, so here is the revert commit log: >> >> -- >> >> Commit "HID: hid-logitech-dj, que

Re: [PATCH 7/9] perf kvm: option to print events that exceed a threshold

2013-08-05 Thread David Ahern
On 8/5/13 2:39 AM, Xiao Guangrong wrote: On 08/03/2013 04:05 AM, David Ahern wrote: This is useful to spot high latency blips. Yes, it is a good idea. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: N

percpu: kernel BUG at mm/percpu.c:579!

2013-08-05 Thread Sasha Levin
ALLOC [ 274.822467] Modules linked in: [ 274.823240] CPU: 13 PID: 58 Comm: rcuos/13 Tainted: GW 3.11.0-rc4-next-20130805-sasha-2-gf6cc217 #3975 [ 274.824464] task: 880220cb3000 ti: 880220cba000 task.ti: 880220cba000 [ 274.825442] RIP: 0010:[] [] pcpu_free_area+0xd8/

[PATCH] drivers/scsi/lpfc/lpfc_init.c: adjust code alignment

2013-08-05 Thread Julia Lawall
From: Julia Lawall Adjust code alignment so that each statement is lined up with its neighbor. In the second case, it could be desirable to put the call to lpfc_destroy_vport_work_array under the if. The call, is, however, safe in case vports is NULL, so the patch just adjusts the indentation t

[PATCH] net/vmw_vsock/af_vsock.c: drop unneeded semicolon

2013-08-05 Thread Julia Lawall
From: Julia Lawall Drop the semicolon at the end of the list_for_each_entry loop header. Signed-off-by: Julia Lawall --- Not tested, but I can't imagine how the current code could work, since vsk should end up pointing to a dummy value. net/vmw_vsock/af_vsock.c |2 +- 1 file changed, 1 i

[PATCH] fs/nfs/inode.c: adjust code alignment

2013-08-05 Thread Julia Lawall
From: Julia Lawall Signed-off-by: Julia Lawall --- This patch adjusts the code so that the alignment matches the current semantics. I have no idea if it is the intended semantics, though. Should the call to nfs_setsecurity also be under the else? fs/nfs/inode.c |2 +- 1 file changed, 1

Re: [3.10.4] NFS locking panic, plus persisting NFS shutdown panic from 3.9.*

2013-08-05 Thread Nix
On 5 Aug 2013, Jeff Layton stated: > On Sun, 04 Aug 2013 16:40:58 +0100 > Nix wrote: > >> I just got this panic on 3.10.4, in the middle of a large parallel >> compilation (of Chromium, as it happens) over NFSv3: >> >> [16364.527516] BUG: unable to handle kernel NULL pointer dereference at >> 0

[PATCH] adjust code alignment

2013-08-05 Thread Julia Lawall
These patches result from the following semantic patch (http://coccinelle.lip6.fr/), which checks for successive statements that are not aligned. @bad@ statement S; expression e; position p1,p; @@ S@p1 e@p; @script:ocaml@ p1 << bad.p1; p << bad.p; @@ if not ((List.hd p1).line = (List.hd p).line

Re: [PATCHv2] Add per-process flag to control thp

2013-08-05 Thread Peter Zijlstra
On Sat, Aug 03, 2013 at 07:01:02PM +0200, Oleg Nesterov wrote: > On 08/02, Alex Thorlton wrote: > > > > This patch implements functionality to allow processes to disable the use of > > transparent hugepages through the prctl syscall. > > > > We've determined that some jobs perform significantly bet

Re: [PATCH 6/9] perf kvm: add min and max stats to display

2013-08-05 Thread David Ahern
On 8/5/13 2:09 AM, Xiao Guangrong wrote: On 08/03/2013 04:05 AM, David Ahern wrote: Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Namhyung Kim Cc: Xiao Guangrong Cc: Runzhen Wang --- tools/perf/buil

Re: Linux 3.11-rc4

2013-08-05 Thread Oleg Nesterov
On 08/05, Felipe Contreras wrote: > > On Mon, Aug 5, 2013 at 8:29 AM, Oleg Nesterov wrote: > > > > Could you please run wine under strace > > > > strace -f -e ptrace -o LOG wine ... > > > > and show the result? > > Sure. Thanks. > Note that the crash might have happened some time before

Re: [PATCH 5/9] perf kvm: add live mode - v3

2013-08-05 Thread David Ahern
On 8/5/13 1:53 AM, Xiao Guangrong wrote: Hi David, Thanks for your nice job! I got some questions. On 08/03/2013 04:05 AM, David Ahern wrote: static int kvm_events_hash_fn(u64 key) { return key & (EVENTS_CACHE_SIZE - 1); @@ -472,7 +501,11 @@ static bool handle_end_event(struct per

Re: [alsa-devel] [PATCH] ASoC: codecs: da9055: Update driver name to fix breakage due to pmic driver with same name

2013-08-05 Thread Mark Brown
On Mon, Aug 05, 2013 at 01:25:31PM +0530, Ashish Chavan wrote: > On Mon, 2013-07-29 at 17:01 +0100, Mark Brown wrote: > > Well, it's a very unusual hardware design choice to have multiple I2C > > endpoints in a single physical chip. > I hope to see more of such devices in near future. There's pr

Re: [PATCH] acpi: video: improve quirk check

2013-08-05 Thread Felipe Contreras
On Mon, Aug 5, 2013 at 9:04 AM, Rafael J. Wysocki wrote: >> In my machine I think the issue is slightly different, I think _BCM is >> failing, at least until enabling the _DOS thing, but at the end of the >> day it's the same thing for the check; _BQC is always returning the >> same value, and th

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Borislav Petkov
On Mon, Aug 05, 2013 at 04:27:44PM +0200, Laszlo Ersek wrote: > I wouldn't call the design of SetVirtualAddressMap() braindead. Ok, I've always wondered and you could probably shed some light on the matter: why is SetVirtualAddressMap() a call-once only? Why can't I simply call it again and update

[PATCH v7] dmaengine: Add MOXA ART DMA engine driver

2013-08-05 Thread Jonas Jensen
nge "#dma-cells" to "<2>" Applies to next-20130805 .../devicetree/bindings/dma/moxa,moxart-dma.txt| 21 + drivers/dma/Kconfig| 7 + drivers/dma/Makefile | 1 + drivers/dma/mox

Re: [PATCH V2 4/4] x86: correctly detect hypervisor

2013-08-05 Thread Konrad Rzeszutek Wilk
On Mon, Aug 05, 2013 at 11:38:14AM +0800, Jason Wang wrote: > On 07/25/2013 04:54 PM, Jason Wang wrote: > > We try to handle the hypervisor compatibility mode by detecting hypervisor > > through a specific order. This is not robust, since hypervisors may > > implement > > each others features. > >

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Raghavendra K T
On 08/05/2013 07:35 PM, Paolo Bonzini wrote: I guess you'd want to carry this in the KVM tree or so - maybe in a separate branch because it changes Xen as well? It changes KVM host and guest side, XEN and common x86 spinlock code. I think it would be best to merge common x86 spinlock bits and g

Re: [GIT PULL] tracing: final fixes for events and some

2013-08-05 Thread Ingo Molnar
* Steven Rostedt wrote: > Linus, > > Oleg Nesterov has been working hard in closing all the holes that can > lead to race conditions between deleting an event and accessing an event > debugfs file. This included a fix to the debugfs system (acked by Greg > Kroah-Hartman). We think that all t

Re: O_TMPFILE fs corruption (Re: Linux 3.11-rc4)

2013-08-05 Thread Al Viro
On Sun, Aug 04, 2013 at 08:45:16PM -0700, Linus Torvalds wrote: > The patch looks right to me - we should pass in similar flags for the > create case as for tmpfile to the filesystem. > > But let's make sure we're all on the same page. Al? ACK. It used to happen as a side effect of O_CREAT being

Re: Linux 3.11-rc4

2013-08-05 Thread Felipe Contreras
On Mon, Aug 5, 2013 at 8:29 AM, Oleg Nesterov wrote: > On 08/04, Felipe Contreras wrote: >> >> I found a regression while running all v3.11-rcX kernels; Starcract II >> through wine crashes. > > Thanks... just to clarify, Starcract crashes, wine or kernel? It's Starcraft. In fact, it detects the

[GIT PULL] tracing: final fixes for events and some

2013-08-05 Thread Steven Rostedt
Linus, Oleg Nesterov has been working hard in closing all the holes that can lead to race conditions between deleting an event and accessing an event debugfs file. This included a fix to the debugfs system (acked by Greg Kroah-Hartman). We think that all the holes have been patched and hopefully

Re: [PATCH 15/23] cgroup: make hierarchy iterators deal with cgroup_subsys_state instead of cgroup

2013-08-05 Thread Vivek Goyal
On Thu, Aug 01, 2013 at 05:49:53PM -0400, Tejun Heo wrote: > cgroup is currently in the process of transitioning to using css > (cgroup_subsys_state) as the primary handle instead of cgroup in > subsystem API. For hierarchy iterators, this is beneficial because > > * In most cases, css is the onl

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Laszlo Ersek
On 08/05/13 16:03, Borislav Petkov wrote: > On Mon, Aug 05, 2013 at 03:39:31PM +0200, Laszlo Ersek wrote: >> My question was: is my understanding correct that you only see this >> problem with "-enable-kvm"? Because, >> >> On 08/01/13 18:49, Borislav Petkov wrote: >>> so I'm seeing this funny thing

Re: [PATCH 12/23] cgroup: pass around cgroup_subsys_state instead of cgroup in file methods

2013-08-05 Thread Vivek Goyal
On Thu, Aug 01, 2013 at 05:49:50PM -0400, Tejun Heo wrote: > cgroup is currently in the process of transitioning to using struct > cgroup_subsys_state * as the primary handle instead of struct cgroup. > Please see the previous commit which converts the subsystem methods > for rationale. > > This p

Re: [PATCH 1/7] EFI stub documentation updates

2013-08-05 Thread Dave Martin
On Fri, Aug 02, 2013 at 02:29:02PM -0700, Roy Franz wrote: > The ARM kernel also has an EFI stub which works largely the same way > as the x86 stub, so move the documentation out of x86 directory and > update to reflect that it is generic, and add ARM specific text. > > Signed-off-by: Roy Franz >

Re: [PATCH 6/7] Add EFI stub for ARM

2013-08-05 Thread Dave Martin
On Fri, Aug 02, 2013 at 02:29:07PM -0700, Roy Franz wrote: > This patch adds EFI stub support for the ARM Linux kernel. The EFI stub > operations similarly to the x86 stub: it is a shim between the EFI firmware > and the normal zImage entry point, and sets up the environment that the > zImage is e

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Paolo Bonzini
> > On Mon, Aug 05, 2013 at 11:46:03AM +0200, Ingo Molnar wrote: > > > Acked-by: Ingo Molnar > > > > > > I guess you'd want to carry this in the KVM tree or so - maybe in a > > > separate branch because it changes Xen as well? > > > > It changes KVM host and guest side, XEN and common x86 spinlo

[PATCH] ACPI / platform: Drop unnecessary label

2013-08-05 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The create_dev label in acpi_create_platform_device() is not necessary, because the if statement causing the jump to it to happen may be rearranged to avoid that jump. Rework the code accordingly (no functional changes should result from that). Signed-off-by: Rafael J. W

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Borislav Petkov
On Mon, Aug 05, 2013 at 03:39:31PM +0200, Laszlo Ersek wrote: > My question was: is my understanding correct that you only see this > problem with "-enable-kvm"? Because, > > On 08/01/13 18:49, Borislav Petkov wrote: > > so I'm seeing this funny thing where an EFI region changes when we > > enter

[PATCH] ACPI / PM: Walk physical_node_list under physical_node_lock

2013-08-05 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The list of physical devices corresponding to an ACPI device object is walked by acpi_system_wakeup_device_seq_show() and physical_device_enable_wakeup() without taking that object's physical_node_lock mutex. Since each of those functions may be run at any time as a resul

[tip:x86/paravirt] xen: Switch to use hypervisor_cpuid_base()

2013-08-05 Thread tip-bot for Jason Wang
Commit-ID: 448ac44d561f0aba5baac54bac62fe40d07230b6 Gitweb: http://git.kernel.org/tip/448ac44d561f0aba5baac54bac62fe40d07230b6 Author: Jason Wang AuthorDate: Thu, 25 Jul 2013 16:54:33 +0800 Committer: H. Peter Anvin CommitDate: Mon, 5 Aug 2013 06:34:09 -0700 xen: Switch to use hypervis

[tip:x86/paravirt] x86: Correctly detect hypervisor

2013-08-05 Thread tip-bot for Jason Wang
Commit-ID: 9df56f19a500bea90d160be1bf77e4fbcd204d3f Gitweb: http://git.kernel.org/tip/9df56f19a500bea90d160be1bf77e4fbcd204d3f Author: Jason Wang AuthorDate: Thu, 25 Jul 2013 16:54:35 +0800 Committer: H. Peter Anvin CommitDate: Mon, 5 Aug 2013 06:35:33 -0700 x86: Correctly detect hyper

[tip:x86/paravirt] x86, kvm: Switch to use hypervisor_cpuid_base( )

2013-08-05 Thread tip-bot for Jason Wang
Commit-ID: 1085ba7f552d84aa8ac0ae903fa8d0cc2ff9f79d Gitweb: http://git.kernel.org/tip/1085ba7f552d84aa8ac0ae903fa8d0cc2ff9f79d Author: Jason Wang AuthorDate: Thu, 25 Jul 2013 16:54:34 +0800 Committer: H. Peter Anvin CommitDate: Mon, 5 Aug 2013 06:34:33 -0700 x86, kvm: Switch to use hyp

[tip:x86/paravirt] x86: Introduce hypervisor_cpuid_base()

2013-08-05 Thread tip-bot for Jason Wang
Commit-ID: 96e39ac0e9d18020c8c5a41b64b8fe86c26575ab Gitweb: http://git.kernel.org/tip/96e39ac0e9d18020c8c5a41b64b8fe86c26575ab Author: Jason Wang AuthorDate: Thu, 25 Jul 2013 16:54:32 +0800 Committer: H. Peter Anvin CommitDate: Mon, 5 Aug 2013 06:33:54 -0700 x86: Introduce hypervisor_c

aio: kernel BUG at fs/aio.c:646!

2013-08-05 Thread Sasha Levin
Hi all, While fuzzing with trinity inside a KVM tools guest running latest -next kernel, I've stumbled on the following spew caused by a new BUG() added in "aio: fix io_destroy() regression by using call_rcu()". [ 9646.599640] kernel BUG at fs/aio.c:646! [ 9646.600119] invalid opcode: [#1]

Re: i915 INFO: trying to register non-static key.

2013-08-05 Thread Johannes Stezenbach
Hi, wrt to $Subject, I get this with 3.10.5: [4.342638] i915 :00:02.0: setting latency timer to 64 [4.409045] INFO: trying to register non-static key. [4.409164] the code is fine but needs lockdep annotation. [4.409278] turning off the locking correctness validator. [4.409

Re: [PATCH] acpi: video: improve quirk check

2013-08-05 Thread Rafael J. Wysocki
On Sunday, August 04, 2013 09:19:56 AM Felipe Contreras wrote: > On Sun, Aug 4, 2013 at 9:19 AM, Rafael J. Wysocki wrote: > > On Sunday, August 04, 2013 01:42:49 AM Felipe Contreras wrote: > > >> Personally I think there are better ways to fix the code for the > >> synthetic case than what you pa

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Ingo Molnar
* Gleb Natapov wrote: > On Mon, Aug 05, 2013 at 11:46:03AM +0200, Ingo Molnar wrote: > > Acked-by: Ingo Molnar > > > > I guess you'd want to carry this in the KVM tree or so - maybe in a > > separate branch because it changes Xen as well? > > It changes KVM host and guest side, XEN and commo

Re: i915 INFO: trying to register non-static key.

2013-08-05 Thread Johannes Stezenbach
On Mon, Aug 05, 2013 at 03:27:29PM +0200, Borislav Petkov wrote: > On Mon, Aug 05, 2013 at 03:23:53PM +0200, Johannes Stezenbach wrote: > > wrt to $Subject, I get this with 3.10.5: > > > > [4.342638] i915 :00:02.0: setting latency timer to 64 > > [4.409045] INFO: trying to register non

[PATCH v2 8/8] exec: cleanup the error handling in search_binary_handler()

2013-08-05 Thread Oleg Nesterov
The error hanling and ret-from-loop look confusing and inconsistent. - "retval >= 0" simply returns - "!bprm->file" returns too but with read_unlock() because binfmt_lock was already re-acquired - "retval != -ENOEXEC || bprm->mm == NULL" does "break" and relies on the same check after the m

[PATCH v2 3/8] exec: proc_exec_connector() should be called only once

2013-08-05 Thread Oleg Nesterov
A separate one-liner with the minor fix. PROC_EVENT_EXEC reports the "exec" event, but this message is sent at least twice if search_binary_handler() is called by ->load_binary() recursively, say, load_script(). Move it to exec_binprm(), this is "depth == 0" code too. Signed-off-by: Oleg Nestero

[PATCH v2 5/8] exec: kill ->load_binary != NULL check in search_binary_handler()

2013-08-05 Thread Oleg Nesterov
search_binary_handler() checks ->load_binary != NULL for no reason, this method should be always defined. Turn this check into WARN_ON() and move it into __register_binfmt(). Also, kill the function pointer. The current code looks confusing, as if ->load_binary can go away after read_unlock(&binfm

[PATCH v2 7/8] exec: don't retry if request_module() fails

2013-08-05 Thread Oleg Nesterov
A separate one-liner for better documentation. It doesn't make sense to retry if request_module() fails to exec /sbin/modprobe, add the additional "request_module() < 0" check. However, this logic still doesn't look exactly right: 1. It would be better to check "request_module() != 0", the user

[PATCH v2 6/8] exec: cleanup the CONFIG_MODULES logic

2013-08-05 Thread Oleg Nesterov
search_binary_handler() uses "for (try=0; try<2; try++)" to avoid "goto" but the code looks too complicated and horrible imho. We still need to check "try == 0" before request_module() and add the additional "break" for !CONFIG_MODULES case. Kill this loop and use a simple "bool need_retry" + "got

[PATCH v2 1/8] exec: introduce exec_binprm() for "depth == 0" code

2013-08-05 Thread Oleg Nesterov
task_pid_nr_ns() and trace/ptrace code in the middle of the recursive search_binary_handler() looks confusing and imho annoying. We only need this code if "depth == 0", lets add a simple helper which calls search_binary_handler() and does trace_sched_process_exec() + ptrace_event(). The patch also

[PATCH v2 2/8] exec: kill "int depth" in search_binary_handler()

2013-08-05 Thread Oleg Nesterov
Nobody except search_binary_handler() should touch ->recursion_depth, "int depth" buys nothing but complicates the code, kill it. Probably we should also kill "fn" and the !NULL check, ->load_binary should be always defined. And it can not go away after read_unlock() or this code is buggy anyway.

[PATCH v2 4/8] exec: move allow_write_access/fput to exec_binprm()

2013-08-05 Thread Oleg Nesterov
When search_binary_handler() succeeds it does allow_write_access() and fput(), then it clears bprm->file to ensure the caller will not do the same. We can simply move this code to exec_binprm() which is called only once. In fact we could move this to free_bprm() and remove the same code in do_exec

[PATCH v2 0/8] exec: cleanup search_binary_handler()

2013-08-05 Thread Oleg Nesterov
To remind, by discussion with Zach fs-binfmts-add-a-name-field-to-the-binfmt-struct.patch fs-binfmts-better-handling-of-binfmt-loops.patch fs-binfmts-whitespace-fixes-with-scripts-cleanfile.patch should be dropped. Zach is working on the new version on top of these changes

Re: [PATCH 1/3] mips/kvm: Improve code formatting in arch/mips/kvm/kvm_locore.S

2013-08-05 Thread Gleb Natapov
On Mon, Aug 05, 2013 at 03:21:57PM +0200, Ralf Baechle wrote: > On Mon, Aug 05, 2013 at 02:17:01PM +0100, James Hogan wrote: > > > > > On 01/08/13 21:22, David Daney wrote: > > > From: David Daney > > > > > > No code changes, just reflowing some comments and consistently using > > > tabs and sp

Re: [PATCH v2 00/18] Arrange hotpluggable memory as ZONE_MOVABLE.

2013-08-05 Thread Tang Chen
Hi hpa, I'm sorry but I don't quite following it. On 08/05/2013 09:07 PM, H. Peter Anvin wrote: On 08/01/2013 12:06 AM, Tang Chen wrote: This patch-set aims to solve some problems at system boot time to enhance memory hotplug functionality. [Background] The Linux kernel cannot migrate pages

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Laszlo Ersek
On 08/05/13 15:02, Borislav Petkov wrote: > On Mon, Aug 05, 2013 at 01:27:16PM +0200, Laszlo Ersek wrote: >>> --- before 2013-07-31 22:20:52.316039492 +0200 >>> +++ after 2013-07-31 22:21:30.960731706 +0200 >>> @@ -9,7 +9,7 @@ efi: mem07: type=2, attr=0xf, range=[0x0 >>> efi: mem08: typ

Re: Linux 3.11-rc4

2013-08-05 Thread Oleg Nesterov
On 08/04, Felipe Contreras wrote: > > I found a regression while running all v3.11-rcX kernels; Starcract II > through wine crashes. Thanks... just to clarify, Starcract crashes, wine or kernel? > The culprit is fab840f (ptrace: PTRACE_DETACH > should do flush_ptrace_hw_breakpoint(child)), I reve

Re: [PATCH v8 06/12] ARM: dts: Add description of System MMU of Exynos SoCs

2013-08-05 Thread Marek Szyprowski
Hello, On 8/5/2013 3:09 PM, Bartlomiej Zolnierkiewicz wrote: On Monday, August 05, 2013 08:16:40 PM Cho KyongHo wrote: > > -Original Message- > > From: Bartlomiej Zolnierkiewicz [mailto:b.zolnier...@samsung.com] > > Sent: Saturday, August 03, 2013 2:14 AM > > > > Hi, > > > > On Friday, J

Re: [PATCH v3 1/2] ARM: OMAP: Add secure function omap_smc3() which calling instruction smc #1

2013-08-05 Thread Dave Martin
On Sun, Aug 04, 2013 at 10:45:00AM +0200, Pali Rohár wrote: > Here is new version (v3) of omap secure part patch: > > Other secure functions omap_smc1() and omap_smc2() calling instruction smc #0 > but Nokia RX-51 board needs to call smc #1 for PPA access. > > Signed-off-by: Ivaylo Dimitrov > Si

Re: [PATCH 19/23] truncate: support huge pages

2013-08-05 Thread Jan Kara
On Sun 04-08-13 05:17:21, Kirill A. Shutemov wrote: > From: "Kirill A. Shutemov" > > truncate_inode_pages_range() drops whole huge page at once if it's fully > inside the range. > > If a huge page is only partly in the range we zero out the part, > exactly like we do for partial small pages. >

Re: i915 INFO: trying to register non-static key.

2013-08-05 Thread Borislav Petkov
On Mon, Aug 05, 2013 at 03:23:53PM +0200, Johannes Stezenbach wrote: > wrt to $Subject, I get this with 3.10.5: > > [4.342638] i915 :00:02.0: setting latency timer to 64 > [4.409045] INFO: trying to register non-static key. > [4.409164] the code is fine but needs lockdep annotation

Re: [PATCH v2 RESEND 05/18] x86, ACPICA: Split acpi_boot_table_init() into two parts.

2013-08-05 Thread Tang Chen
Hi Rafael, On 08/05/2013 09:26 PM, Rafael J. Wysocki wrote: .. I think I understand what you're trying to achieve and I don't have objections agaist the goal, but the matter is *how* to do that. Why don't you do something like this: (1) Introduce two new functions that will each do part of

Re: [PATCH 2/2] HID: hid-logitech-dj, querying_devices was never set

2013-08-05 Thread Jiri Kosina
On Fri, 2 Aug 2013, Benjamin Tissoires wrote: > > Could you please elaborate? (and put an elaborate description to revert > > commit log perhaps?) > > Sure, so here is the revert commit log: > > -- > > Commit "HID: hid-logitech-dj, querying_devices was never set" activate > a flag which guarant

Re: [PATCH 1/3] mips/kvm: Improve code formatting in arch/mips/kvm/kvm_locore.S

2013-08-05 Thread Ralf Baechle
On Mon, Aug 05, 2013 at 02:17:01PM +0100, James Hogan wrote: > > On 01/08/13 21:22, David Daney wrote: > > From: David Daney > > > > No code changes, just reflowing some comments and consistently using > > tabs and spaces. Object code is verified to be unchanged. > > > > Signed-off-by: David

Re: [PATCH 1/3] mips/kvm: Improve code formatting in arch/mips/kvm/kvm_locore.S

2013-08-05 Thread James Hogan
Hi David, On 01/08/13 21:22, David Daney wrote: > From: David Daney > > No code changes, just reflowing some comments and consistently using > tabs and spaces. Object code is verified to be unchanged. > > Signed-off-by: David Daney > Acked-by: Ralf Baechle > + /* Put the saved pointer

RE: [PATCH v8 10/12] iommu/exynos: add bus notifier for registering System MMU

2013-08-05 Thread Cho KyongHo
> -Original Message- > From: Bartlomiej Zolnierkiewicz [mailto:b.zolnier...@samsung.com] > Sent: Saturday, August 03, 2013 2:32 AM > > Hi, > > On Friday, July 26, 2013 08:30:17 PM Cho KyongHo wrote: > > When a device driver is registered, all constructs to handle System MMU > > is prepare

Re: [PATCH v2 RESEND 07/18] x86, ACPI: Also initialize signature and length when parsing root table.

2013-08-05 Thread Rafael J. Wysocki
On Monday, August 05, 2013 09:33:32 AM Tang Chen wrote: > Hi Rafael, > > On 08/02/2013 09:03 PM, Rafael J. Wysocki wrote: > > On Friday, August 02, 2013 05:14:26 PM Tang Chen wrote: > >> Besides the phys addr of the acpi tables, it will be very convenient if > >> we also have the signature of each

Re: [PATCH v2 RESEND 05/18] x86, ACPICA: Split acpi_boot_table_init() into two parts.

2013-08-05 Thread Rafael J. Wysocki
On Monday, August 05, 2013 11:21:51 AM Tang Chen wrote: > Hi Rafael, > > On 08/02/2013 09:00 PM, Rafael J. Wysocki wrote: > .. > >> This patch splits acpi_boot_table_init() into two steps: > >> 1. Parse RSDT, which cannot be overrided, and initialize > >> acpi_gbl_root_table_list. (step 1

Re: [PATCH 1/2] OF: Add helper for matching against linux,stdout-path

2013-08-05 Thread Sascha Hauer
On Mon, Aug 05, 2013 at 07:59:01AM -0500, Rob Herring wrote: > On 08/05/2013 07:40 AM, Sascha Hauer wrote: > > devicetrees may have a linux,stdout-path property in the chosen > > node describing the console device. This adds a helper function > > to match a device against this property so a driver

Re: [PATCH 1/2] ARM: remove dmacap,memset from Device tree binding

2013-08-05 Thread Sebastian Hesselbarth
On 08/05/13 14:26, Jason Cooper wrote: On Mon, Aug 05, 2013 at 10:53:41AM +0100, Mark Rutland wrote: I'm not entirely sure on this. The property described is still true for the device, even if the kernel's not using that information now. My feeling would be to keep them, unless there's a plan to

Re: [PATCH v8 06/12] ARM: dts: Add description of System MMU of Exynos SoCs

2013-08-05 Thread Bartlomiej Zolnierkiewicz
On Monday, August 05, 2013 08:16:40 PM Cho KyongHo wrote: > > -Original Message- > > From: Bartlomiej Zolnierkiewicz [mailto:b.zolnier...@samsung.com] > > Sent: Saturday, August 03, 2013 2:14 AM > > > > Hi, > > > > On Friday, July 26, 2013 08:28:19 PM Cho KyongHo wrote: > > > Signed-off-b

Re: [PATCH v2 00/18] Arrange hotpluggable memory as ZONE_MOVABLE.

2013-08-05 Thread H. Peter Anvin
On 08/01/2013 12:06 AM, Tang Chen wrote: > This patch-set aims to solve some problems at system boot time > to enhance memory hotplug functionality. > > [Background] > > The Linux kernel cannot migrate pages used by the kernel because > of the kernel direct mapping. Since va = pa + PAGE_OFFSET, i

Proposal

2013-08-05 Thread Lisbeth Karlsson
I seek your consent to present you as the next of kin to the account of a deceased client for claims valued at Fourteen million, three hundred thousand Pounds. Contact me through my private email for details: halawani@ovi.com Mr. Halawani, Azhar Abdulrahim. Solicitors Legal Services Barr

Re: [GIT PULL] urgent fix

2013-08-05 Thread H. Peter Anvin
On 08/05/2013 06:02 AM, Matt Fleming wrote: > On Tue, 30 Jul, at 08:52:12PM, Ingo Molnar wrote: >> Pulled into tip:x86/urgent, thanks Matt! > > Thanks Ingo. > > Any chance we can get this to Linus for -rc5? The ARM EFI boot stub > patches depend on the bug fix in this pull request. > Yes, we sh

Re: [PATCH] clk: tegra30: Don't wait for PLL_U lock bit

2013-08-05 Thread Tuomas Tynkkynen
On 08/05/2013 09:38 AM, Prashant Gaikwad wrote: > On Thursday 01 August 2013 02:34 AM, Stephen Warren wrote: >> On 07/31/2013 11:42 AM, Tuomas Tynkkynen wrote: >>> The lock bit on PLL_U does not seem to be working correctly and >>> sometimes never gets set when waiting for the PLL to come up. >>> R

Re: Cannot hot remove a memory device (patch)

2013-08-05 Thread Rafael J. Wysocki
On Monday, August 05, 2013 04:59:20 PM Yasuaki Ishimatsu wrote: > (2013/08/05 13:00), Yasuaki Ishimatsu wrote: > > (2013/08/04 9:37), Toshi Kani wrote: > >> On Sat, 2013-08-03 at 03:01 +0200, Rafael J. Wysocki wrote: > >>> On Friday, August 02, 2013 06:04:40 PM Toshi Kani wrote: > On Sat, 2013

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Borislav Petkov
On Mon, Aug 05, 2013 at 01:27:16PM +0200, Laszlo Ersek wrote: > > --- before 2013-07-31 22:20:52.316039492 +0200 > > +++ after 2013-07-31 22:21:30.960731706 +0200 > > @@ -9,7 +9,7 @@ efi: mem07: type=2, attr=0xf, range=[0x0 > > efi: mem08: type=7, attr=0xf, > > range=[0x400

Re: [GIT PULL] urgent fix

2013-08-05 Thread Matt Fleming
On Tue, 30 Jul, at 08:52:12PM, Ingo Molnar wrote: > Pulled into tip:x86/urgent, thanks Matt! Thanks Ingo. Any chance we can get this to Linus for -rc5? The ARM EFI boot stub patches depend on the bug fix in this pull request. -- Matt Fleming, Intel Open Source Technology Center -- To unsubscrib

Re: [PATCH 1/2] OF: Add helper for matching against linux,stdout-path

2013-08-05 Thread Rob Herring
On 08/05/2013 07:40 AM, Sascha Hauer wrote: > devicetrees may have a linux,stdout-path property in the chosen > node describing the console device. This adds a helper function > to match a device against this property so a driver can call > add_preferred_console for a matching device. > > Signed-o

Re: [PATCH 09/23] cgroup: add subsys backlink pointer to cftype

2013-08-05 Thread Vivek Goyal
On Thu, Aug 01, 2013 at 05:49:47PM -0400, Tejun Heo wrote: > cgroup is transitioning to using css (cgroup_subsys_state) instead of > cgroup as the primary subsystem handle. The cgroupfs file interface > will be converted to use css's which requires finding out the > subsystem from cftype so that t

[GIT PULL] s390 patches for 3.11-rc5

2013-08-05 Thread Martin Schwidefsky
Hi Linus, please pull from the 'for-linus' branch of git://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git for-linus to receive the following updates: Enable LZ4 compression for the kernel image, add the machine id for the new zBC12 model, fix an issue with hanging dasd devices, c

Re: [PATCH 08/23] cgroup: pass around cgroup_subsys_state instead of cgroup in subsystem methods

2013-08-05 Thread Vivek Goyal
On Thu, Aug 01, 2013 at 05:49:46PM -0400, Tejun Heo wrote: > cgroup is currently in the process of transitioning to using struct > cgroup_subsys_state * as the primary handle instead of struct cgroup * > in subsystem implementations for the following reasons. > > * With unified hierarchy, subsyste

Re: [3.10.4] NFS locking panic, plus persisting NFS shutdown panic from 3.9.*

2013-08-05 Thread Jeff Layton
On Sun, 04 Aug 2013 16:40:58 +0100 Nix wrote: > I just got this panic on 3.10.4, in the middle of a large parallel > compilation (of Chromium, as it happens) over NFSv3: > > [16364.527516] BUG: unable to handle kernel NULL pointer dereference at > 0008 > [16364.527571] IP: [] nlmcln

[PATCH 2/2] serial: i.MX: evaluate linux,stdout-path property

2013-08-05 Thread Sascha Hauer
devicetrees may have the linux,stdout-path property to specify the console. This patch adds support to the i.MX serial driver for this. Signed-off-by: Sascha Hauer --- drivers/tty/serial/imx.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/im

[PATCH 1/2] OF: Add helper for matching against linux,stdout-path

2013-08-05 Thread Sascha Hauer
devicetrees may have a linux,stdout-path property in the chosen node describing the console device. This adds a helper function to match a device against this property so a driver can call add_preferred_console for a matching device. Signed-off-by: Sascha Hauer --- drivers/of/base.c | 26 ++

[PATCH v6] linux,stdout-path helper

2013-08-05 Thread Sascha Hauer
The following adds a helper for matching the linux,stdout-path property in the chosen node and makes use of it in the i.MX serial This is a series originally created by me. It was hijacked and then abandoned by Jean-Christophe for v5. I reverted the changes by Jean-Christophe since they introduced

Re: [PATCH] OPP: Export opp_add()

2013-08-05 Thread Rafael J. Wysocki
On Monday, August 05, 2013 07:29:09 AM Nishanth Menon wrote: > minor nitpick. > $subject: PM / OPP: Those are things I can easily fix up when I'm applying the patch. > On 08/03/2013 04:25 AM, Viresh Kumar wrote: > > Export opp_add() so that modules can use it. > > > > Signed-off-by: Viresh Kumar

Re: [PATCH] OPP: Export opp_add()

2013-08-05 Thread Nishanth Menon
minor nitpick. $subject: PM / OPP: On 08/03/2013 04:25 AM, Viresh Kumar wrote: Export opp_add() so that modules can use it. Signed-off-by: Viresh Kumar --- drivers/base/power/opp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/base/power/opp.c b/drivers/base/power/opp.c index

Re: [PATCH 1/2] ARM: remove dmacap,memset from Device tree binding

2013-08-05 Thread Jason Cooper
On Mon, Aug 05, 2013 at 10:53:41AM +0100, Mark Rutland wrote: > On Sun, Aug 04, 2013 at 02:09:04AM +0100, Jason Cooper wrote: > > DT Maintainers, > > > > It's been a week with no comment. Shall I assume it's ok to apply > > this? > > Apologies for the delay, still catching up on binding review.

Re: [PATCH v2 5/5] efi: x86: make efi_lookup_mapped_addr() a common function

2013-08-05 Thread Matt Fleming
On Fri, 02 Aug, at 05:56:22PM, Leif Lindholm wrote: > efi_lookup_mapped_addr() is a handy utility for other platforms than > x86. Move it from arch/x86 to drivers/firmware. Add memmap pointer > to global efi structure, and initialise it on x86. > > Signed-off-by: Leif Lindholm > --- > arch/x86/p

[PATCH] nohz: put proper symbol in the unstable schedclock warning

2013-08-05 Thread Jiri Kosina
Since e12d0271 ("nohz: Warn if the machine can not perform nohz_full") Kernel triggers a WARN() in case sched clock is marked as unstable. Make it output a proper Kconfig symbol (NO_HZ_FULL) when informing user about this. Signed-off-by: Jiri Kosina --- kernel/time/tick-sched.c |2 +- 1 f

Re: [PATCH v2 4/5] efi: ia64: use common code for (U)EFI configuration scanning

2013-08-05 Thread Matt Fleming
On Fri, 02 Aug, at 05:56:21PM, Leif Lindholm wrote: > This patch makes ia64 use the new common code for configuration table > scanning. It also removes the local definition of the global "efi" > data structure. > > Signed-off-by: Leif Lindholm > --- > arch/ia64/kernel/efi.c | 54 > ++-

Re: [PATCH v2 3/5] efi: x86: use common code for (U)EFI configuration scanning

2013-08-05 Thread Matt Fleming
On Fri, 02 Aug, at 05:56:20PM, Leif Lindholm wrote: > This patch makes x86 use the new common code for configuration table > scanning. It also removes the local definition of the global "efi" > data structure. > > Signed-off-by: Leif Lindholm > --- > arch/x86/platform/efi/efi.c | 96 > ---

Re: [alsa-devel] [PATCH v4] ASoC: core: init delayed_work for codec-codec links

2013-08-05 Thread Richard Fitzgerald
We must init the delayed_work for codec-codec links otherwise shutting down the DAI chain will fault when calling flush_delayed_work_sync() on the linked DAI. Signed-off-by: Richard Fitzgerald --- sound/soc/soc-core.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff -

<    1   2   3   4   5   6   7   8   >