Re: [Bugfix] x86/PCI: Release PCI IRQ resource only if PCI device is disabled when unbinding

2015-03-18 Thread Bjorn Helgaas
On Tue, Mar 17, 2015 at 03:37:12PM +0800, Jiang Liu wrote: > To support IOAPIC hot-removal, we need to release PCI interrupt resource > when unbinding PCI device driver. But due to historical reason, > /* > * We would love to complain here if pci_dev->is_enabled is set, that > * the driver

[PATCH v3] dt-bindings: brcm: rationalize Broadcom documentation naming

2015-03-18 Thread Scott Branden
This patchset attempts to standarize the naming of dt-bindings documents based on the Broadcom vendor prefix of brcm. Although there are no guidelines currently present for how to name the dt-bindings document the "vendor,binding.txt" style is in use by some of the other vendors. Acked-by:

Re: [PATCH] dt-bindings: brcm: rationalize Broadcom documentation naming

2015-03-18 Thread Scott Branden
On 15-03-18 03:03 PM, Stephen Warren wrote: On 03/18/2015 03:53 PM, Scott Branden wrote: Hi Stephen, On 15-03-18 12:42 PM, Stephen Warren wrote: On 03/18/2015 01:24 PM, Scott Branden wrote: This patchset attempts to standarize the naming of dt-bindings documents based on the Broadcom vendor

Re: [RFC V2] proc: change /proc/stat show

2015-03-18 Thread Andrew Morton
On Wed, 18 Mar 2015 10:35:39 +0800 "Wang, Yalin" wrote: > This patch change /proc/stat to show each cpu, > we show each present cpus instead of eacn online cpu, > because some cpus are online / offline dynamically, > we should also show its cputime even it is offline, > some lib will read this

Re: seccomp vs ptrace

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:44 PM, Kees Cook wrote: > On Wed, Mar 18, 2015 at 2:42 PM, Andy Lutomirski wrote: >> On Wed, Mar 18, 2015 at 2:38 PM, Kees Cook wrote: >>> On Wed, Mar 18, 2015 at 2:30 PM, Serge E. Hallyn wrote: Hi, I'm writing to ask about The seccomp

Re: [PATCH] PM / Domains: Skip latency measurements if timekeeping is suspended

2015-03-18 Thread Rafael J. Wysocki
On Wednesday, March 18, 2015 05:25:46 PM Geert Uytterhoeven wrote: > The PM Domain code uses ktime_get() to perform various latency > measurements. However, if ktime_get() is called while timekeeping is > suspended, the following warning is printed: > > WARNING: CPU: 0 PID: 1340 at

Re: [PATCH v3 1/2] x86_64,signal: Fix SS handling for signals delivered to 64-bit programs

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:34 PM, Pavel Emelyanov wrote: > On 03/19/2015 12:26 AM, Andy Lutomirski wrote: >> On Wed, Mar 18, 2015 at 1:02 PM, Oleg Nesterov wrote: >>> On 03/18, Andrey Wagin wrote: This patch fixes the problem. Oleg, could you send this path in the criu maillist?

Re: [PATCH] dt-bindings: brcm: rationalize Broadcom documentation naming

2015-03-18 Thread Stephen Warren
On 03/18/2015 03:53 PM, Scott Branden wrote: Hi Stephen, On 15-03-18 12:42 PM, Stephen Warren wrote: On 03/18/2015 01:24 PM, Scott Branden wrote: This patchset attempts to standarize the naming of dt-bindings documents based on the Broadcom vendor prefix of brcm. Although there are no

[PATCH] Revert "net: cx82310_eth: use common match macro"

2015-03-18 Thread Ondrej Zary
This reverts commit 11ad714b98f6d9ca0067568442afe3e70eb94845 because it breaks cx82310_eth. The custom USB_DEVICE_CLASS macro matches bDeviceClass, bDeviceSubClass and bDeviceProtocol but the common USB_DEVICE_AND_INTERFACE_INFO matches bInterfaceClass, bInterfaceSubClass and bInterfaceProtocol

Re: [PATCH 2/2] Staging: lustre: lnet: lnet: router.c: fix useless statements

2015-03-18 Thread Greg KH
On Wed, Mar 18, 2015 at 10:45:47PM +0100, Redha Gouicem wrote: > This patch removes useless returns and elses. > > Signed-off-by: Redha Gouicem > --- > drivers/staging/lustre/lnet/lnet/router.c | 15 --- > 1 file changed, 4 insertions(+), 11 deletions(-) > > diff --git

Re: crypto: img-hash - undefined Kconfig symbols

2015-03-18 Thread Herbert Xu
On Wed, Mar 18, 2015 at 01:02:14PM +, James Hartley wrote: > > Herbert: should I send you a patch to address the issues above, or resubmit > the series with the fixes incorporated? James, your patch has already been merged so please send any fixes on top of the cryptodev tree. Thanks, --

Re: [PATCH v6 2/2] hwspinlock: qcom: Add support for Qualcomm HW Mutex block

2015-03-18 Thread Bjorn Andersson
On Wed 18 Mar 09:45 PDT 2015, Lina Iyer wrote: > On Wed, Mar 18 2015 at 09:56 -0600, Bjorn Andersson wrote: > >On Thu 12 Mar 12:31 PDT 2015, Lina Iyer wrote: > > > >> On Fri, Feb 27 2015 at 15:30 -0700, Bjorn Andersson wrote: [..] > >> >+#define QCOM_MUTEX_NUM_LOCKS 32 > >> > >> Also, talking

Re: [PATCH 1/2] Staging: lustre: lnet: lnet: router.c : fix 80 char limit

2015-03-18 Thread Greg KH
On Wed, Mar 18, 2015 at 10:45:46PM +0100, Redha Gouicem wrote: > This fixes lines longer than 80 char which are not debug or error messages. > > Signed-off-by: Redha Gouicem > --- > drivers/staging/lustre/lnet/lnet/router.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff

Re: [PATCH v8 0/4] hwspinlock core & omap dt support

2015-03-18 Thread Suman Anna
Hi Mark, On 03/12/2015 04:24 AM, Ohad Ben-Cohen wrote: > Hi Suman, > > On Thu, Mar 5, 2015 at 4:01 AM, Suman Anna wrote: >> This is the latest version of the hwspinlock dt support series, >> rebased onto v4.0-rc1 and addressing the long discussion on the >> bindings in v7 [1]. I really hope

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:42 PM, Denys Vlasenko wrote: > On 03/18/2015 10:32 PM, Linus Torvalds wrote: >> On Wed, Mar 18, 2015 at 12:26 PM, Andy Lutomirski >> wrote: crash> disassemble page_fault Dump of assembler code for function page_fault: 0x816834a0 <+0>:

[PATCH v2] dt-bindings: brcm: rationalize Broadcom documentation naming

2015-03-18 Thread Scott Branden
This patchset attempts to standarize the naming of dt-bindings documents based on the Broadcom vendor prefix of brcm. Although there are no guidelines currently present for how to name the dt-bindings document the "vendor,binding.txt" style is in use by some of the other vendors. ---

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Stefan Seyfried
Am 18.03.2015 um 22:49 schrieb Denys Vlasenko: > Stefan, Takashi, can you post your /proc/cpuinfo > and dmesg after boot? susi:~ # cat /proc/cpuinfo processor : 0 vendor_id : GenuineIntel cpu family : 6 model : 23 model name : Intel(R) Core(TM)2 Duo CPU L9400

Re: [PATCH] dt-bindings: brcm: rationalize Broadcom documentation naming

2015-03-18 Thread Scott Branden
Hi Stephen, On 15-03-18 12:42 PM, Stephen Warren wrote: On 03/18/2015 01:24 PM, Scott Branden wrote: This patchset attempts to standarize the naming of dt-bindings documents based on the Broadcom vendor prefix of brcm. Although there are no guidelines currently present for how to name the

Re: [PATCH] blkmq: Fix NULL pointer deref when all reserved tags in use

2015-03-18 Thread Jens Axboe
On 03/18/2015 03:06 PM, Sam Bradshaw wrote: Good catch! But why not put the hctx == NULL check in as a conditional in bt_get() before running the queue? I can't imagine other cases where calling blk_mq_run_hw_queue() with hctx == NULL would be a valid scenario. The change was meant to be

Re: [PATCH 6/8] amd64_edac: enforce synchronous probe

2015-03-18 Thread Tejun Heo
Hello, Dmitry. On Wed, Mar 18, 2015 at 02:41:26PM -0700, Dmitry Torokhov wrote: > You are over-stating the boot order guarantees that storage provides. > Yes, you can scan devices and partitions simultaneously on the same > controller, but it will break if controllers are registered in different

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Denys Vlasenko
Stefan, Takashi, can you post your /proc/cpuinfo and dmesg after boot? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Stefan Seyfried
Am 18.03.2015 um 22:32 schrieb Linus Torvalds: > Is PARAVIRT enabled? The three nop's at the beginning of 'page_fault' > makes me suspect it is, and that that is some paravirt rewriting > area. What does paravirt go for that USERGS_SYSRET64 (or for > SWAPGS_UNSAFE_STACK, for that matter). This

[PATCH 1/2] Staging: lustre: lnet: lnet: router.c : fix 80 char limit

2015-03-18 Thread Redha Gouicem
This fixes lines longer than 80 char which are not debug or error messages. Signed-off-by: Redha Gouicem --- drivers/staging/lustre/lnet/lnet/router.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/router.c

[PATCH 2/2] Staging: lustre: lnet: lnet: router.c: fix useless statements

2015-03-18 Thread Redha Gouicem
This patch removes useless returns and elses. Signed-off-by: Redha Gouicem --- drivers/staging/lustre/lnet/lnet/router.c | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/router.c b/drivers/staging/lustre/lnet/lnet/router.c

Re: [RFC][PATCH 1/9] usb: hcd: Introduce usb_start/stop_hcd()

2015-03-18 Thread Tony Lindgren
* Alan Stern [150318 12:50]: > On Wed, 18 Mar 2015, Roger Quadros wrote: > > > To support OTG we want a mechanism to start and stop > > the HCD from the OTG state machine. Add usb_start_hcd() > > and usb_stop_hcd(). > > > > Signed-off-by: Roger Quadros > > There are a few problems in this

Re: seccomp vs ptrace

2015-03-18 Thread Kees Cook
On Wed, Mar 18, 2015 at 2:42 PM, Andy Lutomirski wrote: > On Wed, Mar 18, 2015 at 2:38 PM, Kees Cook wrote: >> On Wed, Mar 18, 2015 at 2:30 PM, Serge E. Hallyn wrote: >>> Hi, >>> >>> I'm writing to ask about >>> >>> The seccomp check will not be run again after the tracer is >>>

Re: [PATCH V2 2/4] hugetlbfs: add minimum size accounting to subpools

2015-03-18 Thread Andrew Morton
On Mon, 16 Mar 2015 16:53:27 -0700 Mike Kravetz wrote: > The same routines that perform subpool maximum size accounting > hugepage_subpool_get/put_pages() are modified to also perform > minimum size accounting. When a delta value is passed to these > routines, calculate how global reservations

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Denys Vlasenko
On 03/18/2015 10:32 PM, Linus Torvalds wrote: > On Wed, Mar 18, 2015 at 12:26 PM, Andy Lutomirski wrote: >>> >>> crash> disassemble page_fault >>> Dump of assembler code for function page_fault: >>>0x816834a0 <+0>: data32 xchg %ax,%ax >>>0x816834a3 <+3>: data32

Re: [PATCH 2/3] x86: entry_64.S: use PUSH insns to build pt_regs on stack

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:32 PM, Denys Vlasenko wrote: > On 03/18/2015 10:22 PM, Andy Lutomirski wrote: >> On Wed, Mar 18, 2015 at 2:12 PM, Denys Vlasenko wrote: >>> On 03/18/2015 10:01 PM, Andy Lutomirski wrote: On Wed, Mar 18, 2015 at 12:47 PM, Denys Vlasenko wrote: > We lose a

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Stefan Seyfried
Am 18.03.2015 um 22:21 schrieb Andy Lutomirski: > On Wed, Mar 18, 2015 at 2:12 PM, Stefan Seyfried > wrote: >> Am 18.03.2015 um 21:51 schrieb Andy Lutomirski: >>> On Wed, Mar 18, 2015 at 1:05 PM, Stefan Seyfried >>> wrote: >> > The relevant thread's stack is here (see ti in the trace): >

Re: seccomp vs ptrace

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:38 PM, Kees Cook wrote: > On Wed, Mar 18, 2015 at 2:30 PM, Serge E. Hallyn wrote: >> Hi, >> >> I'm writing to ask about >> >> The seccomp check will not be run again after the tracer is >> notified. (This means that seccomp-based sandboxes MUST NOT >>

Re: [PATCH 6/8] amd64_edac: enforce synchronous probe

2015-03-18 Thread Dmitry Torokhov
On Wed, Mar 18, 2015 at 05:02:26PM -0400, Tejun Heo wrote: > Hello, > > On Wed, Mar 18, 2015 at 01:26:05PM -0700, Dmitry Torokhov wrote: > > Tejun, I lost you here. Certainly you are not arguing for going through > > the drivers one by one and making their module init code to engage > >

Re: [PATCH V2 4/4] hugetlbfs: document min_size mount option

2015-03-18 Thread Andrew Morton
On Mon, 16 Mar 2015 16:53:29 -0700 Mike Kravetz wrote: > Update documentation for the hugetlbfs min_size mount option. > > Signed-off-by: Mike Kravetz > --- > Documentation/vm/hugetlbpage.txt | 21 ++--- > 1 file changed, 14 insertions(+), 7 deletions(-) > > diff --git

Re: [PATCH V2 3/4] hugetlbfs: accept subpool min_size mount option and setup accordingly

2015-03-18 Thread Andrew Morton
On Mon, 16 Mar 2015 16:53:28 -0700 Mike Kravetz wrote: > Make 'min_size=' be an option when mounting a hugetlbfs. This option > takes the same value as the 'size' option. min_size can be specified > with specifying size. If both are specified, min_size must be less > that or equal to size

Re: seccomp vs ptrace

2015-03-18 Thread Kees Cook
On Wed, Mar 18, 2015 at 2:30 PM, Serge E. Hallyn wrote: > Hi, > > I'm writing to ask about > > The seccomp check will not be run again after the tracer is > notified. (This means that seccomp-based sandboxes MUST NOT > allow use of ptrace, even of other sandboxed

Re: seccomp vs ptrace

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:30 PM, Serge E. Hallyn wrote: > Hi, > > I'm writing to ask about > > The seccomp check will not be run again after the tracer is > notified. (This means that seccomp-based sandboxes MUST NOT > allow use of ptrace, even of other sandboxed

Re: [PATCH] mm: Use GFP_KERNEL allocation for the page cache in page_cache_read

2015-03-18 Thread NeilBrown
On Wed, 18 Mar 2015 16:45:40 +0100 Michal Hocko wrote: > What do you think about this v2? I cannot say I would like it but I > really dislike the whole mapping_gfp_mask API to be honest. > --- > >From d88010d6f5f59d7eb87b691e27e201d12cab9141 Mon Sep 17 00:00:00 2001 > From: Michal Hocko > Date:

Re: [PATCH v3 1/2] x86_64,signal: Fix SS handling for signals delivered to 64-bit programs

2015-03-18 Thread Pavel Emelyanov
On 03/19/2015 12:26 AM, Andy Lutomirski wrote: > On Wed, Mar 18, 2015 at 1:02 PM, Oleg Nesterov wrote: >> On 03/18, Andrey Wagin wrote: >>> >>> This patch fixes the problem. Oleg, could you send this path in the >>> criu maillist? >> >> Sure, will do. > > We still haven't answered one question:

Re: [PATCH 2/3] x86: entry_64.S: use PUSH insns to build pt_regs on stack

2015-03-18 Thread Denys Vlasenko
On 03/18/2015 10:22 PM, Andy Lutomirski wrote: > On Wed, Mar 18, 2015 at 2:12 PM, Denys Vlasenko wrote: >> On 03/18/2015 10:01 PM, Andy Lutomirski wrote: >>> On Wed, Mar 18, 2015 at 12:47 PM, Denys Vlasenko >>> wrote: We lose a number of large insns there: textdata

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Linus Torvalds
On Wed, Mar 18, 2015 at 12:26 PM, Andy Lutomirski wrote: >> >> crash> disassemble page_fault >> Dump of assembler code for function page_fault: >>0x816834a0 <+0>: data32 xchg %ax,%ax >>0x816834a3 <+3>: data32 xchg %ax,%ax >>0x816834a6 <+6>: data32

seccomp vs ptrace

2015-03-18 Thread Serge E. Hallyn
Hi, I'm writing to ask about The seccomp check will not be run again after the tracer is notified. (This means that seccomp-based sandboxes MUST NOT allow use of ptrace, even of other sandboxed processes, without extreme care; ptracers can use this mechanism to

Re: [PATCH v2 1/3] init: Export name_to_dev_t and mark it const

2015-03-18 Thread Mike Snitzer
On Tue, Feb 10 2015 at 6:20pm -0500, Dan Ehrenberg wrote: > name_to_dev_t is useful in other pieces of code to initialize rootfs. > In the case of dm, the code is sometimes built in a module and other > times used to construct the rootfs; therefore it must be exported > as a symbol. > >

Re: [PATCH v3 1/2] x86_64,signal: Fix SS handling for signals delivered to 64-bit programs

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 1:02 PM, Oleg Nesterov wrote: > On 03/18, Andrey Wagin wrote: >> >> This patch fixes the problem. Oleg, could you send this path in the >> criu maillist? > > Sure, will do. We still haven't answered one question: what's the kernel's position on ABI stability wrt CRIU? We

Re: [PATCH] Documentation: i2o: remove duplicate documentation

2015-03-18 Thread Jonathan Corbet
On Wed, 18 Mar 2015 22:21:47 +0100 Wolfram Sang wrote: > Ping. For consistency, I'd think this would be nice in 4.0. I have it in my pile of docs patches, should get to that before too long. LSFMM kind of slowed everything down. jon -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH V2 1/4] hugetlbfs: add minimum size tracking fields to subpool structure

2015-03-18 Thread Andrew Morton
On Mon, 16 Mar 2015 16:53:26 -0700 Mike Kravetz wrote: > Add a field to the subpool structure to indicate the minimimum > number of huge pages to always be used by this subpool. This > minimum count includes allocated pages as well as reserved pages. > If the minimum number of pages for the

Re: [PATCH 2/3] x86: entry_64.S: use PUSH insns to build pt_regs on stack

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:12 PM, Denys Vlasenko wrote: > On 03/18/2015 10:01 PM, Andy Lutomirski wrote: >> On Wed, Mar 18, 2015 at 12:47 PM, Denys Vlasenko wrote: >>> We lose a number of large insns there: >>> >>> textdata bss dec hex filename >>> 9863 0 0

Re: maps mismatch in /proc/kcore for sparc

2015-03-18 Thread David Ahern
On 3/18/15 2:14 PM, David Miller wrote: Oh I see, you're using older sources which don't have: commit abaf3787ac26ba33e2f75e76b1174c32254c25b0 Author: Paul Gortmaker Date: Thu Jan 23 15:55:45 2014 -0800 fs/proc: don't use module_init for non-modular core

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:12 PM, Stefan Seyfried wrote: > Am 18.03.2015 um 21:51 schrieb Andy Lutomirski: >> On Wed, Mar 18, 2015 at 1:05 PM, Stefan Seyfried >> wrote: > The relevant thread's stack is here (see ti in the trace): 8801013d4000 It could be interesting

Re: [PATCH] Documentation: i2o: remove duplicate documentation

2015-03-18 Thread Wolfram Sang
On Tue, Mar 03, 2015 at 11:07:15AM +0100, Wolfram Sang wrote: > We can remove the i2o documentation because a) the subsystem has been > moved to staging with commit 2cbf7fe2d5d32a (i2o: move to staging) > anyhow and b) the here removed files are present in the subsystem > directory again. There,

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 2:06 PM, Denys Vlasenko wrote: > On 03/18/2015 09:49 PM, Andy Lutomirski wrote: >> On Wed, Mar 18, 2015 at 1:06 PM, Denys Vlasenko wrote: >>> On 03/18/2015 08:26 PM, Andy Lutomirski wrote: Hi Linus- You seem to enjoy debugging these things. Want to give

Re: [PATCH 2/3] x86: entry_64.S: use PUSH insns to build pt_regs on stack

2015-03-18 Thread Denys Vlasenko
On 03/18/2015 10:01 PM, Andy Lutomirski wrote: > On Wed, Mar 18, 2015 at 12:47 PM, Denys Vlasenko wrote: >> We lose a number of large insns there: >> >> textdata bss dec hex filename >> 9863 0 098632687 entry_64_before.o >> 9671 0 0

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Stefan Seyfried
Am 18.03.2015 um 21:51 schrieb Andy Lutomirski: > On Wed, Mar 18, 2015 at 1:05 PM, Stefan Seyfried > wrote: >>> The relevant thread's stack is here (see ti in the trace): >>> >>> 8801013d4000 >>> >>> It could be interesting to see what's there. >>> >>> I don't suppose you want to try to walk

Re: [PATCH] Staging: lustre: ldlm_pool: fix coding style

2015-03-18 Thread Greg KH
On Wed, Mar 18, 2015 at 09:11:22PM +0100, Guillaume Turchini wrote: > Errors and warnings of coding style rules corrected. You need to be a bit more specific and say what exact errors and warnings were fixed. And please only fix one type of error or warning per patch you send out. This patch

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Denys Vlasenko
On 03/18/2015 09:49 PM, Andy Lutomirski wrote: > On Wed, Mar 18, 2015 at 1:06 PM, Denys Vlasenko wrote: >> On 03/18/2015 08:26 PM, Andy Lutomirski wrote: >>> Hi Linus- >>> >>> You seem to enjoy debugging these things. Want to give this a shot? >>> My guess is a vmalloc fault accessing either

Re: [PATCH 3/4] drm/msm: Initial add DSI connector support

2015-03-18 Thread hali
Hi Archit, Thanks for your comments. Please see my response for some comments below. Comments without response will be addressed in patch version 2. I will wait for other comments if any to push patch V2. >> +static int dsi_gpio_init(struct msm_dsi_host *msm_host) >> +{ >> +int ret; >> + >>

Re: [PATCH 3/3] x86: get rid of FIXUP_TOP_OF_STACK/RESTORE_TOP_OF_STACK

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 12:47 PM, Denys Vlasenko wrote: > FIXUP_TOP_OF_STACK is only necessary because we don't save %r11 > to pt_regs->r11 on SYSCALL64 fast path, but we want ptrace to see > it populated. > > Bite the bullet, add a single additional PUSH insn, and remove > FIXUP_TOP_OF_STACK. >

Re: [PATCH] blkmq: Fix NULL pointer deref when all reserved tags in use

2015-03-18 Thread Sam Bradshaw
Good catch! But why not put the hctx == NULL check in as a conditional in bt_get() before running the queue? I can't imagine other cases where calling blk_mq_run_hw_queue() with hctx == NULL would be a valid scenario. The change was meant to be broad in scope. A runtime NULL deref is a

Re: [PATCH 6/8] amd64_edac: enforce synchronous probe

2015-03-18 Thread Tejun Heo
Hello, On Wed, Mar 18, 2015 at 01:26:05PM -0700, Dmitry Torokhov wrote: > Tejun, I lost you here. Certainly you are not arguing for going through > the drivers one by one and making their module init code to engage > async_schedule to continue the device creation in link order (well, > sorta,

Re: [PATCH 2/3] x86: entry_64.S: use PUSH insns to build pt_regs on stack

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 12:47 PM, Denys Vlasenko wrote: > We lose a number of large insns there: > > textdata bss dec hex filename > 9863 0 098632687 entry_64_before.o > 9671 0 0967125c7 entry_64.o > > What's more important, we

Re: Setting reboot type at run time for ARM

2015-03-18 Thread Arun Ramamurthy
On 15-03-17 05:50 PM, Florian Fainelli wrote: On 17/03/15 14:44, Arun Ramamurthy wrote: Hello I would like to specify a reset type just before issuing the reboot command in the kernel. I know the kernel command line parameter can be set as "reboot=w" to indicate warm reset but I want to be

[PATCHv2 5/5] Drivers: Staging: ft1000: checkpatch.pl header file warning fix

2015-03-18 Thread Janakarajan Natarajan
Replaces and with and Signed-off-by: Janakarajan Natarajan --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c index

[PATCHv2 3/5] Drivers: Staging: ft1000: Refactoring if-else statement

2015-03-18 Thread Janakarajan Natarajan
Minor change to convert if-else to if statement Signed-off-by: Janakarajan Natarajan --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c

[PATCHv2 4/5] Drivers: Staging: ft1000: Fix blank line warning

2015-03-18 Thread Janakarajan Natarajan
Merges variable declaration and definition. Signed-off-by: Janakarajan Natarajan --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c

[PATCHv2 2/5] Drivers: Staging: ft1000: Fix extra parenthesis warnings

2015-03-18 Thread Janakarajan Natarajan
Minor changes to remove extra parenthesis Signed-off-by: Janakarajan Natarajan --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCHv2 1/5] Drivers: Staging: ft1000: Single line if-statement changes

2015-03-18 Thread Janakarajan Natarajan
Minor change to remove {} for single line if statements Signed-off-by: Janakarajan Natarajan --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c

[PATCHv2 0/5] ft1000 driver checkpatch.pl fixes

2015-03-18 Thread Janakarajan Natarajan
Minor changes to fix ft1000 driver checkpatch.pl warnings Janakarajan Natarajan (5): Drivers: Staging: ft1000: Single line if-statement changes Drivers: Staging: ft1000: Fix extra parenthesis warnings Drivers: Staging: ft1000: Refactoring if-else statement Drivers: Staging: ft1000: Fix

Re: [PATCH 1/3] x86: get rid of KERNEL_STACK_OFFSET

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 12:47 PM, Denys Vlasenko wrote: > PER_CPU_VAR(kernel_stack) was set up in a way where it points > five stack slots below the top of stack. > > Presumably, it was done to avoid one "sub $5*8,%rsp" > in syscall/sysenter code paths, where iret frame needs to be > created by

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 1:05 PM, Stefan Seyfried wrote: > Hi Andy, > > Am 18.03.2015 um 20:26 schrieb Andy Lutomirski: >> Hi Linus- >> >> You seem to enjoy debugging these things. Want to give this a shot? >> My guess is a vmalloc fault accessing either old_rsp or kernel_stack >> right after

Re: [PATCH] KVM: x86: call irq notifiers with directed EOI

2015-03-18 Thread Bandan Das
Radim Krčmář writes: > kvm_ioapic_update_eoi() wasn't called if directed EOI was enabled. > We need to do that for irq notifiers. (Like with edge interrupts.) > > Fix it by skipping EOI broadcast only. > > Bug: https://bugzilla.kernel.org/show_bug.cgi?id=82211 > Signed-off-by: Radim Krčmář >

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-18 Thread Andy Lutomirski
On Wed, Mar 18, 2015 at 1:06 PM, Denys Vlasenko wrote: > On 03/18/2015 08:26 PM, Andy Lutomirski wrote: >> Hi Linus- >> >> You seem to enjoy debugging these things. Want to give this a shot? >> My guess is a vmalloc fault accessing either old_rsp or kernel_stack >> right after swapgs in syscall

Re: [PATCH 1/3] x86: get rid of KERNEL_STACK_OFFSET

2015-03-18 Thread Borislav Petkov
On Wed, Mar 18, 2015 at 08:47:34PM +0100, Denys Vlasenko wrote: > PER_CPU_VAR(kernel_stack) was set up in a way where it points > five stack slots below the top of stack. > > Presumably, it was done to avoid one "sub $5*8,%rsp" > in syscall/sysenter code paths, where iret frame needs to be >

[PATCH] x86/apic: remove read/write test in verify_local_apic()

2015-03-18 Thread Bandan Das
Most newer processors have the APIC_ID read-only and this test probably fails on most boxes anyway. Fix typos. Signed-off-by: Bandan Das --- Honestly, I don't think this function is really needed. It's called for both UP/MP but the return value isn't used by any of them. Maybe, we should just

[PATCH 3/3] mm: idle memory tracking

2015-03-18 Thread Vladimir Davydov
Knowing the portion of memory that is not used by a certain application or memory cgroup (idle memory) can be useful for partitioning the system efficiently. Currently, the only means to estimate the amount of idle memory provided by the kernel is /proc/PID/clear_refs. However, it has two serious

[PATCH 2/3] proc: add kpagecgroup file

2015-03-18 Thread Vladimir Davydov
/proc/kpagecgroup contains a 64-bit inode number of the memory cgroup each page is charged to, indexed by PFN. Having this information is useful for estimating a cgroup working set size. The file is present if CONFIG_PROC_PAGE_MONITOR && CONFIG_MEMCG. Signed-off-by: Vladimir Davydov ---

[PATCH 1/3] memcg: add page_cgroup_ino helper

2015-03-18 Thread Vladimir Davydov
Hwpoison allows to filter pages by memory cgroup ino. To ahieve that, it calls try_get_mem_cgroup_from_page(), then mem_cgroup_css(), and finally cgroup_ino() on the cgroup returned. This looks bulky. Since in the next patch I need to get the ino of the memory cgroup a page is charged to too, in

[PATCH 0/3] idle memory tracking

2015-03-18 Thread Vladimir Davydov
Hi, Knowing the portion of memory that is not used by a certain application or memory cgroup (idle memory) can be useful for partitioning the system efficiently. Currently, the only means to estimate the amount of idle memory provided by the kernel is /proc/PID/clear_refs. However, it has two

Re: [PATCH] blkmq: Fix NULL pointer deref when all reserved tags in use

2015-03-18 Thread Jens Axboe
On 03/18/2015 02:36 PM, Sam Bradshaw wrote: When allocating from the reserved tags pool, bt_get() is called with a NULL hctx. If all tags are in use, the hw queue is kicked to push out any pending IO, potentially freeing tags, and tag allocation is retried. The problem is that

Re: [PATCH 0/9] qspinlock stuff -v15

2015-03-18 Thread Waiman Long
On 03/16/2015 09:16 AM, Peter Zijlstra wrote: Hi Waiman, As promised; here is the paravirt stuff I did during the trip to BOS last week. All the !paravirt patches are more or less the same as before (the only real change is the copyright lines in the first patch). The paravirt stuff is

[PATCH] blkmq: Fix NULL pointer deref when all reserved tags in use

2015-03-18 Thread Sam Bradshaw
When allocating from the reserved tags pool, bt_get() is called with a NULL hctx. If all tags are in use, the hw queue is kicked to push out any pending IO, potentially freeing tags, and tag allocation is retried. The problem is that blk_mq_run_hw_queue() doesn't check for a NULL hctx. This

Re: [PATCH v3 4/5] i2c: davinci: use bus recovery infrastructure

2015-03-18 Thread Wolfram Sang
Hi, so, the bus recovery patches look fine to me in general. It is only this one question left which I always had with bus recovery. Maybe you guys can join me thinking about it. > @@ -376,8 +366,7 @@ i2c_davinci_xfer_msg(struct i2c_adapter *adap, struct > i2c_msg *msg, int stop) >

Re: [PATCH] staging: lustre: namei.c: coding style: fix 80 characters limit

2015-03-18 Thread gre...@linuxfoundation.org
On Wed, Mar 18, 2015 at 07:53:35PM +, Dilger, Andreas wrote: > On 2015/03/18, 1:07 PM, "Maxime Lorrillere" > wrote: > > >From: Laure Millet > > > >This patch fixes lines over 80 characters > > I think the newer rule is that strings shouldn't be split across lines. > That makes > the first

Re: [RFC PATCH 0/7] evacuate struct page from the block layer

2015-03-18 Thread Andrew Morton
On Mon, 16 Mar 2015 16:25:25 -0400 Dan Williams wrote: > Avoid the impending disaster of requiring struct page coverage for what > is expected to be ever increasing capacities of persistent memory. In > conversations with Rik van Riel, Mel Gorman, and Jens Axboe at the > recently concluded

Re: [PATCH 6/8] amd64_edac: enforce synchronous probe

2015-03-18 Thread Dmitry Torokhov
On Wed, Mar 18, 2015 at 03:51:41PM -0400, Tejun Heo wrote: > Hello, > > On Wed, Mar 18, 2015 at 12:36:22PM -0700, Dmitry Torokhov wrote: > > Because they are not inherently problematic. I mean from the kernel POV > > they work fine, the question is if your userspace can deal with them or > > not.

[PATCH V2 1/1] X86: hyperv: Enable MSR based APIC access

2015-03-18 Thread K. Y. Srinivasan
If the hypervisor supports MSR based access to the APIC registers (EOI, TPR and ICR), implement the MSR based access. Signed-off-by: K. Y. Srinivasan --- Changes from V1: Addressed comments from Ingo Molnar arch/x86/kernel/cpu/mshyperv.c | 62 1

Re: [RFCv3 PATCH 33/48] sched: Energy-aware wake-up task placement

2015-03-18 Thread Sai Gurrappadi
On 03/16/2015 07:47 AM, Morten Rasmussen wrote: > On Fri, Mar 13, 2015 at 10:47:16PM +, Sai Gurrappadi wrote: >> On 02/04/2015 10:31 AM, Morten Rasmussen wrote: >>> +static int energy_aware_wake_cpu(struct task_struct *p) >>> +{ >>> + struct sched_domain *sd; >>> + struct sched_group *sg,

Re: [PATCH] KVM: x86: call irq notifiers with directed EOI

2015-03-18 Thread Radim Krčmář
2015-03-18 15:37-0400, Bandan Das: > Radim Krčmář writes: > > kvm_ioapic_update_eoi() wasn't called if directed EOI was enabled. > > We need to do that for irq notifiers. (Like with edge interrupts.) > > Wow! It's interesting that this path is only hit with Xen as guest. Linux doesn't use

[PATCH v2 1/6] vgaarb: Stub vga_set_legacy_decoding()

2015-03-18 Thread Alex Williamson
vga_set_legacy_decoding() is defined in drivers/gpu/vga/vgaarb.c, which is only compiled with CONFIG_VGA_ARB. A caller would therefore get an undefined symbol if the VGA arbiter is not enabled. Signed-off-by: Alex Williamson Acked-by: Dave Airlie --- include/linux/vgaarb.h |5 + 1

[RFC] gpio: add set_active_low() function to gpio_chip

2015-03-18 Thread David Cohen
Some gpio controllers are capable of programming its pins' active-low state. Let's add this new gpio_chip function for such cases and use it in gpiolib. When set_active_low() is implemented, we no longer need to do soft flips on values from non-raw get functions. Signed-off-by: David Cohen ---

Re: maps mismatch in /proc/kcore for sparc

2015-03-18 Thread David Miller
From: David Ahern Date: Wed, 18 Mar 2015 13:57:59 -0600 > On 3/18/15 1:43 PM, David Miller wrote: >> From: David Ahern >> Date: Wed, 18 Mar 2015 10:56:22 -0600 >> >>> I am trying to understand why the perf tool on sparc fails to convert >>> kernel symbols when /proc/kcore is used. I have

[PATCH v2 3/6] vfio-pci: Add VGA arbiter client

2015-03-18 Thread Alex Williamson
If VFIO VGA access is disabled for the user, either by CONFIG option or module parameter, we can often opt-out of VGA arbitration. We can do this when PCI bridge control of VGA routing is possible. This means that we must have a parent bridge and there must only be a single VGA device below that

[PATCH v2 4/6] vfio-pci: Allow PCI IDs to be specified as module options

2015-03-18 Thread Alex Williamson
This copies the same support from pci-stub for exactly the same purpose, enabling a set of PCI IDs to be automatically added to the driver's dynamic ID table at module load time. The code here is pretty simple and both vfio-pci and pci-stub are fairly unique in being meta drivers, capable of

[PATCH v2 2/6] vfio-pci: Add module option to disable VGA region access

2015-03-18 Thread Alex Williamson
Add a module option so that we don't require a CONFIG change and kernel rebuild to disable VGA support. Not only can VGA support be troublesome in itself, but by disabling it we can reduce the impact to host devices by doing a VGA arbitration opt-out. Signed-off-by: Alex Williamson ---

[PATCH v2 5/6] vfio-pci: Remove warning if try-reset fails

2015-03-18 Thread Alex Williamson
As indicated in the comment, this is not entirely uncommon and causes user concern for no reason. Signed-off-by: Alex Williamson --- drivers/vfio/pci/vfio_pci.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/vfio/pci/vfio_pci.c

[PATCH v2 6/6] vfio-pci: Move idle devices to D3hot power state

2015-03-18 Thread Alex Williamson
We can save some power by putting devices that are bound to vfio-pci but not in use by the user in the D3hot power state. Devices get woken into D0 when opened by the user. Resets return the device to D0, so we need to re-apply the low power state after a bus reset. It's tempting to try to use

[PATCH v2 0/6] vfio-pci: Misc enhancements

2015-03-18 Thread Alex Williamson
v2: - Incorporate comments from Bandan and Bjorn for vfio-pci.ids option - Include necessary vgaarb change, already Ack'd by Dave - Rebase on top of my current next branch - Rename D3 disable parameter There are really 3 separate features added in this series, the first is to opt-out VGA

Re: [PATCH v2] fs/pstore: Optimization function ramoops_init_przs

2015-03-18 Thread Kees Cook
On Tue, Mar 17, 2015 at 6:41 PM, Wang Long wrote: > The value of cxt->record_size does not change in the loop, > so this patch optimize the assign statement by dropping > sz entirely and using cxt->record_size in its place. > > Signed-off-by: Wang Long Thanks! Acked-by: Kees Cook -Kees >

[PATCH] Staging: lustre: ldlm_pool: fix coding style

2015-03-18 Thread Guillaume Turchini
Errors and warnings of coding style rules corrected. Signed-off-by: Guillaume Turchini --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_pool.c

[PATCH] net: ethernet: pcnet32: Setup the SRAM and NOUFLO on Am79C97{3,5}

2015-03-18 Thread Markos Chandras
On a MIPS Malta board, tons of fifo underflow errors have been observed when using u-boot as bootloader instead of YAMON. The reason for that is that YAMON used to set the pcnet device to SRAM mode but u-boot does not. As a result, the default Tx threshold (64 bytes) is now too small to keep the

Re: [PATCH 2/2 v2] tools, update rtctest.c to verify passage of time

2015-03-18 Thread John Stultz
On Wed, Mar 18, 2015 at 12:46 PM, Prarit Bhargava wrote: > rtctest.c checks to see if PIE is functioning by testing if 20 interrupts > occur > at rates from 2HZ to 64HZ. While this check is good, it does not check to > see if the correct amount of time has actually passed. This misses >

Re: [PATCH v10 08/21] ARM64 / ACPI: Introduce early_param "acpi=" to enable/disable ACPI

2015-03-18 Thread Ard Biesheuvel
On 18 March 2015 at 12:35, Lorenzo Pieralisi wrote: > On Wed, Mar 11, 2015 at 12:39:34PM +, Hanjun Guo wrote: >> From: Al Stone >> >> This implements the following policy to decide whether ACPI should >> be used to boot the system: >> - acpi=off: ACPI will not be used to boot the system,

<    1   2   3   4   5   6   7   8   9   10   >