[PATCH] ACPI / button: Do not propagate wakeup-from-suspend events

2014-07-18 Thread Rafael J. Wysocki
From: Rafael J. Wysocki During system suspend mark ACPI buttons (other than the lid) as "suspended" and if in that state, report wakeup events on button events, but do not propagate those events up the stack. This prevents systems from being turned off after a button-triggered wakeup from the

Re: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread John Stultz
On 07/18/2014 03:09 PM, Stephen Boyd wrote: > During suspend we call sched_clock_poll() to update the epoch and > accumulated time and reprogram the sched_clock_timer to fire > before the next wrap-around time. Unfortunately, > sched_clock_poll() doesn't restart the timer, instead it relies > on

[PATCH] PM / sleep: Move platform suspend operations to separate functions

2014-07-18 Thread Rafael J. Wysocki
From: Rafael J. Wysocki After the introduction of freeze_ops it makes more sense to move all of the platform suspend operations to separate functions that each will do all of the necessary checks and choose the right callback to execute istead of doing all that in the core code which makes it

Re: How to automate checkpatch && get_maintainers && git send-email of commits range?

2014-07-18 Thread Theodore Ts'o
On Fri, Jul 18, 2014 at 05:38:30PM +0300, Andrey Utkin wrote: > Is there script for automated checkpatch.pl && get_maintainers.pl && > git send-email for range of commits? I see none. Would it be welcome > to submit such one to kernel tree? Too much automation can be a really bad thing. You

Re: [PATCH v2 3/7] seccomp: Allow arch code to provide seccomp_data

2014-07-18 Thread Kees Cook
On Fri, Jul 18, 2014 at 2:18 PM, Andy Lutomirski wrote: > populate_seccomp_data is expensive: it works by inspecting > task_pt_regs and various other bits to piece together all the > information, and it's does so in multiple partially redundant steps. > > Arch-specific code in the syscall entry

[PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread Stephen Boyd
During suspend we call sched_clock_poll() to update the epoch and accumulated time and reprogram the sched_clock_timer to fire before the next wrap-around time. Unfortunately, sched_clock_poll() doesn't restart the timer, instead it relies on the hrtimer layer to do that and during suspend we

[PATCH] random: check for increase of entropy_count because of signed conversion

2014-07-18 Thread Theodore Ts'o
From: Hannes Frederic Sowa The expression entropy_count -= ibytes << (ENTROPY_SHIFT + 3) could actually increase entropy_count if during assignment of the unsigned expression on the RHS (mind the -=) we reduce the value modulo 2^width(int) and assign it to entropy_count. Trinity found this. [

Re: [PATCH 1/2] workqueue: remove unneeded test before wake up next worker

2014-07-18 Thread Tejun Heo
On Wed, Jul 16, 2014 at 06:09:58PM +0800, Lai Jiangshan wrote: > In this code: > if ((worker->flags & WORKER_UNBOUND) && need_more_worker(pool)) > wake_up_worker(pool); > > the first test is unneeded. Even the first test is removed, it doesn't affect > the wake-up logic when

Re: Power-managing devices that are not of interest at some point in time

2014-07-18 Thread Rafael J. Wysocki
On Friday, July 18, 2014 02:45:40 PM Dmitry Torokhov wrote: > On Friday, July 18, 2014 11:59:18 PM Rafael J. Wysocki wrote: > > On Friday, July 18, 2014 02:26:21 PM Dmitry Torokhov wrote: > > > On Friday, July 18, 2014 04:09:46 PM Alan Stern wrote: > > > > On Fri, 18 Jul 2014, Patrik Fimml wrote:

Re: [PATCH v5 1/8] of: Add NVIDIA Tegra SATA controller binding

2014-07-18 Thread Tejun Heo
On Fri, Jul 18, 2014 at 05:06:04PM +0200, Thierry Reding wrote: > I think the following patches can go through your tree without causing > breakage through unresolved dependencies: > > [PATCH v5 1/8] of: Add NVIDIA Tegra SATA controller binding > [PATCH v3 6/8] ata: ahci_platform:

Re: [PATCH] random: check for increase of entropy_count because of signed conversion

2014-07-18 Thread Theodore Ts'o
On Fri, Jul 18, 2014 at 05:25:04PM -0400, Theodore Ts'o wrote: > > As indicated by credit_entropy_bits entropy_count cannot get negative, > > so I don't see any reason to include a check for entropy_count < 0 > > here. Do you agree? > > No, the check is important; after we subtract ibytes <<

AW: Early microcode not being loaded

2014-07-18 Thread Conrad Kostecki
> On Fri, Jul 18, 2014 at 08:40:42PM +, Conrad Kostecki wrote: > > Where is the problem? Why is the microcode not being loaded early? > > CONFIG_MICROCODE_INTEL_EARLY=y set in .config? Yes! Galactica tmp # grep -i microcode /boot/config-3.15.5 CONFIG_MICROCODE=y CONFIG_MICROCODE_INTEL=y #

RE: [BISECTED][REGRESSION] Loading Hyper-V network drivers is racy in 3.14+ on Hyper-V 2012 R2

2014-07-18 Thread Haiyang Zhang
> -Original Message- > From: Sitsofe Wheeler [mailto:sits...@gmail.com] > Sent: Tuesday, July 15, 2014 1:09 AM > To: Haiyang Zhang > Cc: KY Srinivasan; David S. Miller; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org; net...@vger.kernel.org > Subject: Re:

Re: [PATCH 0/6] PM / Hibernate: Memory bitmap scalability improvements

2014-07-18 Thread Rafael J. Wysocki
On Friday, July 18, 2014 01:57:17 PM Joerg Roedel wrote: > Hi, > > here is a patch set to improve the scalability of the memory > bitmap implementation used for hibernation. The current > implementation does not scale well to machines with several > TB of memory. A resume on those machines may

Re: Power-managing devices that are not of interest at some point in time

2014-07-18 Thread Dmitry Torokhov
On Friday, July 18, 2014 11:59:18 PM Rafael J. Wysocki wrote: > On Friday, July 18, 2014 02:26:21 PM Dmitry Torokhov wrote: > > On Friday, July 18, 2014 04:09:46 PM Alan Stern wrote: > > > On Fri, 18 Jul 2014, Patrik Fimml wrote: > > > > On Fri, Jul 18, 2014 at 03:00:46PM -0400, Alan Stern wrote:

Re: [PATCH] random: check for increase of entropy_count because of signed conversion

2014-07-18 Thread Hannes Frederic Sowa
Hi, On Fri, Jul 18, 2014, at 23:25, Theodore Ts'o wrote: > On Wed, Jul 16, 2014 at 09:18:15PM +0200, Hannes Frederic Sowa wrote: > > The expression entropy_count -= ibytes << (ENTROPY_SHIFT + 3) could > > actually increase entropy_count if during assignment of the unsigned > > expression on the

Re: Power-managing devices that are not of interest at some point in time

2014-07-18 Thread Rafael J. Wysocki
On Friday, July 18, 2014 02:26:21 PM Dmitry Torokhov wrote: > On Friday, July 18, 2014 04:09:46 PM Alan Stern wrote: > > On Fri, 18 Jul 2014, Patrik Fimml wrote: > > > On Fri, Jul 18, 2014 at 03:00:46PM -0400, Alan Stern wrote: [cut] > > > I'm not sure what the appropriate action for a video

Re: [GIT PULL] at91: fixes for 3.16 #2

2014-07-18 Thread Olof Johansson
On Fri, Jul 18, 2014 at 7:12 AM, Nicolas Ferre wrote: > Arnd, Olof, Kevin, > > This is the latest regressions that we found while testing the at91sam9n12 and > at91sam9x5 platforms following our move to CCF. > I created a pull-request this time because I have 3 patches: there should be > no

Re: WARNING: CPU: 1 PID: 495 at mm/slab_common.c:69 kmem_cache_create+0x1a9/0x330()

2014-07-18 Thread poma
On 18.07.2014 22:03, poma wrote: On 18.07.2014 16:20, Christoph Lameter wrote: On Fri, 18 Jul 2014, poma wrote: I guess someone working over the summertime. :) Cache names should not contain blanks. I guess the WARN_ON(strchr(name, ' ')); /* It confuses parsers */ was triggered?

Re: WARNING: CPU: 1 PID: 495 at mm/slab_common.c:69 kmem_cache_create+0x1a9/0x330()

2014-07-18 Thread poma
On 18.07.2014 22:16, poma wrote: On 18.07.2014 22:07, James Bottomley wrote: On Fri, 2014-07-18 at 22:01 +0200, poma wrote: On 18.07.2014 16:17, Christoph Hellwig wrote: On Fri, Jul 18, 2014 at 05:21:04PM +0400, Vladimir Davydov wrote: Slab warns, because the name of the cache being created

Re: [PATCH 4/6] x86-mce: Add spinlocks to prevent duplicated MCP and CMCI reports.

2014-07-18 Thread Borislav Petkov
On Fri, Jul 18, 2014 at 02:23:04PM -0700, Tony Luck wrote: > On Thu, Jul 17, 2014 at 3:50 AM, Borislav Petkov wrote: > > Well, maybe it is about time we tracked shared banks. > > For cpus that support CMCI and the MCi_CTL2 registers we do track > sharing. Only one cpu gets to be the "owner" of a

[PATCH v4 2/7] printk: initialize syslog_prev and console_prev

2014-07-18 Thread Alex Elder
Two global variables, "syslog_prev" and "console_prev", maintain a copy of the flags value used in the log record most recently formatted for syslog or the console, respectively. Initially there is no previous formatted log record, and these variables simply have an initial value 0. And

[PATCH v4 0/7] printk: start simplifying some flags

2014-07-18 Thread Alex Elder
Each log record has a "flags" field. The flags keep track of, for instance, whether the record was saved in its entirety (as opposed to being one of multiple records that should be merged as a single unit). A log record's flags field alone is not currently sufficient to know how the record

[PATCH v4 3/7] printk: LOG_CONT and LOG_NEWLINE are opposites

2014-07-18 Thread Alex Elder
Two log record flags--LOG_CONT and LOG_NEWLINE--are mutually exclusive. That is, one or the other is always set, but they are never both set at the same time in a log record flags field. What follows is a great deal of explanation that aims to prove this assertion. Having that knowledge allows

[PATCH v4 4/7] printk: honor LOG_PREFIX in devkmsg_read()

2014-07-18 Thread Alex Elder
In devkmsg_read(), a variable "cont" holds a character that's used to indicate whether a given log line is a "continuation", that is, whether a log record should be merged with the one before or after it. If a record should be merged with its successor (but not its predecessor) that character is

[PATCH v4 1/7] printk: report dropped messages on separate line

2014-07-18 Thread Alex Elder
It is possible for the log to be filled too quickly for the consoles to be able to keep up. This is detected in console_unlock(), and when it occurs, a message is inserted to note the event. When reviewing some nearby code, Petr Mládek suggested it might be nicer if this message were placed on a

[PATCH v4 5/7] printk: honor LOG_PREFIX in msg_print_text()

2014-07-18 Thread Alex Elder
This patch fixes a problem similar to what was addressed in the previous patch. All paths that read and format log records (for consoles, and for reading via syslog and /dev/kmsg) go through msg_print_text(). That function starts with some logic to determine whether the given log record when

[PATCH v4 7/7] printk: correct some more typos

2014-07-18 Thread Alex Elder
This patch corrects a few more typographical errors in "printk.c". Signed-off-by: Alex Elder Reviewed-by: Petr Mládek --- kernel/printk/printk.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index a5ad81c..fd1ecd4

[PATCH v4 6/7] printk: insert newline for truncated records

2014-07-18 Thread Alex Elder
If a log record has LOG_PREFIX set, its predecessor record should be terminated if it was marked LOG_CONT. In devkmsg_read(), this condition was being ignored, which would lead to such records showing up combined when reading /dev/kmsg. Fix this oversight. We should similarly insert a newline in

Re: Power-managing devices that are not of interest at some point in time

2014-07-18 Thread Dmitry Torokhov
On Friday, July 18, 2014 04:09:46 PM Alan Stern wrote: > On Fri, 18 Jul 2014, Patrik Fimml wrote: > > On Fri, Jul 18, 2014 at 03:00:46PM -0400, Alan Stern wrote: > > > "Quiescing" is the wrong word. "Quiescing a device" means stopping the > > > device from doing anything, which isn't what you

Re: [PATCH] random: check for increase of entropy_count because of signed conversion

2014-07-18 Thread Theodore Ts'o
On Wed, Jul 16, 2014 at 09:18:15PM +0200, Hannes Frederic Sowa wrote: > The expression entropy_count -= ibytes << (ENTROPY_SHIFT + 3) could > actually increase entropy_count if during assignment of the unsigned > expression on the RHS (mind the -=) we reduce the value modulo > 2^width(int) and

Re: Early microcode not being loaded

2014-07-18 Thread Borislav Petkov
On Fri, Jul 18, 2014 at 08:40:42PM +, Conrad Kostecki wrote: > Where is the problem? Why is the microcode not being loaded early? CONFIG_MICROCODE_INTEL_EARLY=y set in .config? You can also try with a recent distro kernel and initrd which has microcode in it to rule out you're doing

Re: [PATCH 4/6] x86-mce: Add spinlocks to prevent duplicated MCP and CMCI reports.

2014-07-18 Thread Tony Luck
On Thu, Jul 17, 2014 at 3:50 AM, Borislav Petkov wrote: > Well, maybe it is about time we tracked shared banks. For cpus that support CMCI and the MCi_CTL2 registers we do track sharing. Only one cpu gets to be the "owner" of a bank that supports CMCI (the first one to find it and set bit 30 in

Re: [PATCH 2/4] ARM: add IPI tracepoints

2014-07-18 Thread Steven Rostedt
On Fri, 18 Jul 2014 16:55:42 -0400 (EDT) Nicolas Pitre wrote: > > Here's the patch I have at the head of the series now, with the above > ugliness changed to an unconditional __tracepoint_string attribute. > I was thinking of something like this. Feel free to add this to your series. --

[PATCH v2 3/7] seccomp: Allow arch code to provide seccomp_data

2014-07-18 Thread Andy Lutomirski
populate_seccomp_data is expensive: it works by inspecting task_pt_regs and various other bits to piece together all the information, and it's does so in multiple partially redundant steps. Arch-specific code in the syscall entry path can do much better. Admittedly this adds a bit of additional

[PATCH v2 5/7] x86: Split syscall_trace_enter into two phases

2014-07-18 Thread Andy Lutomirski
This splits syscall_trace_enter into syscall_trace_enter_phase1 and syscall_trace_enter_phase2. Only phase 2 has full pt_regs, and only phase 2 is permitted to modify any of pt_regs except for orig_ax. The intent is that phase 1 can be called from the syscall fast path. Signed-off-by: Andy

[PATCH v2 6/7] x86_64,entry: Treat regs->ax the same in fastpath and slowpath syscalls

2014-07-18 Thread Andy Lutomirski
For slowpath syscalls, we initialize regs->ax to -ENOSYS and stick the syscall number into regs->orig_ax prior to any possible tracing and syscall execution. This is user-visible ABI used by ptrace syscall emulation and seccomp. For fastpath syscalls, there's no good reason not to do the same

[PATCH v2 4/7] x86,x32,audit: Fix x32's AUDIT_ARCH wrt audit

2014-07-18 Thread Andy Lutomirski
is_compat_task() is the wrong check for audit arch; the check should be is_ia32_task(): x32 syscalls should be AUDIT_ARCH_X86_64, not AUDIT_ARCH_I386. CONFIG_AUDITSYSCALL is currently incompatible with x32, so this has no visible effect. Signed-off-by: Andy Lutomirski ---

[PATCH v2 7/7] x86_64,entry: Use split-phase syscall_trace_enter for 64-bit syscalls

2014-07-18 Thread Andy Lutomirski
On KVM on my box, this reduces the overhead from an always-accept seccomp filter from ~130ns to ~17ns. Most of that comes from avoiding IRET on every syscall when seccomp is enabled. In extremely approximate hacked-up benchmarking, just bypassing IRET saves about 80ns, so there's another 43ns of

[PATCH v2 2/7] seccomp: Refactor the filter callback and the API

2014-07-18 Thread Andy Lutomirski
The reason I did this is to add a seccomp API that will be usable for an x86 fast path. The x86 entry code needs to use a rather expensive slow path for a syscall that might be visible to things like ptrace. By splitting seccomp into two phases, we can check whether we need the slow path and

[PATCH v2 1/7] seccomp,x86,arm,mips,s390: Remove nr parameter from secure_computing

2014-07-18 Thread Andy Lutomirski
The secure_computing function took a syscall number parameter, but it only paid any attention to that parameter if seccomp mode 1 was enabled. Rather than coming up with a kludge to get the parameter to work in mode 2, just remove the parameter. To avoid churn in arches that don't have seccomp

[PATCH v2 0/7] Two-phase seccomp and x86 tracing changes

2014-07-18 Thread Andy Lutomirski
This is both a cleanup and a speedup. It reduces overhead due to installing a trivial seccomp filter by 87%. The speedup comes from avoiding the full syscall tracing mechanism for filters that don't return SECCOMP_RET_TRACE. This series works by splitting the seccomp hooks into two phases. The

[GIT PULL] irqchip: Core changes for v3.17 (round 3)

2014-07-18 Thread Jason Cooper
Thomas, Here are the core changes I've gathered so far for v3.17. irq-gic.c has been a little adventuresome this cycle. There's still the possibility of some more code making it in for gic (assuming we don't run out of time), which is why I merged a tag into core instead of the branch. As

[3.13.y.z extended stable] Linux 3.13.11.5

2014-07-18 Thread Kamal Mostafa
I am announcing the release of the Linux 3.13.11.5 kernel. The updated 3.13.y tree can be found at: git://kernel.ubuntu.com/ubuntu/linux.git linux-3.13.y and can be browsed at: http://kernel.ubuntu.com/git?p=ubuntu/linux.git;h=refs/heads/linux-3.13.y;a=shortlog The diff from v3.13.11.4 is

[PATCH -v4] random: introduce getrandom(2) system call

2014-07-18 Thread Theodore Ts'o
The getrandom(2) system call was requested by the LibreSSL Portable developers. It is analoguous to the getentropy(2) system call in OpenBSD. The rationale of this system call is to provide resiliance against file descriptor exhaustion attacks, where the attacker consumes all available file

Re: [PATCH 3/5] net/netfilter/ipvs/ip_vs_ctl.c: drop argument range check just before the check for equality

2014-07-18 Thread Andrey Utkin
2014-07-18 23:48 GMT+03:00 Julian Anastasov : > The above check ensures the set_arglen[] value (some > struct size) does not exceed the arg[MAX_ARG_LEN] space. You can check > commit 04bcef2a83f40c ("ipvs: Add boundary check on ioctl arguments") > for more info. Thanks for info. What

Re: [PATCH] kbuild: devtest - new make target for build all and run tests

2014-07-18 Thread Greg KH
On Fri, Jul 18, 2014 at 02:44:34PM -0600, Shuah Khan wrote: > Add a new devtest make target to enable developer testing. This > new target does full build (make all) and then runs selftests. > > Signed-off-by: Shuah Khan Nice to see this. Acked-by: Greg Kroah-Hartman -- To unsubscribe from

Re: [PATCH v3 1/3] asm-generic/io.h: Implement generic {read,write}s*()

2014-07-18 Thread Sam Ravnborg
> > All these things are not introduced by your patch but now that > you show some love and care for this file maybe we could take > the next step and bring more order to the current semi chaos? And I just noticed... These are never used - and only defined a few places: #define inb_p(addr)

Re: [PATCH 3/5] net/netfilter/ipvs/ip_vs_ctl.c: drop argument range check just before the check for equality

2014-07-18 Thread Julian Anastasov
Hello, On Fri, 18 Jul 2014, Andrey Utkin wrote: > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=80601 > Reported-by: David Binderman > Signed-off-by: Andrey Utkin > --- > net/netfilter/ipvs/ip_vs_ctl.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git

Re: [PATCH] parisc: Remove FIXME comment

2014-07-18 Thread James Bottomley
On Fri, 2014-07-18 at 16:37 -0400, Nicholas Krause wrote: > The comment for size of frame not being needed is incorrect , the > function called needs this parameter. Actually, that's not correct. The point of the FIXME is that fram_size is only used in a debug print and could be eliminated since

Re: [PATCH v3 1/3] asm-generic/io.h: Implement generic {read,write}s*()

2014-07-18 Thread Sam Ravnborg
On Wed, Jul 16, 2014 at 01:01:22PM +0200, Thierry Reding wrote: > From: Thierry Reding > > Currently driver writers need to use io{read,write}{8,16,32}_rep() when > accessing FIFO registers portably. This is bad for two reasons: it is > inconsistent with how other registers are accessed using

Re: linux-next: build warning after merge of the tip tree

2014-07-18 Thread H. Peter Anvin
On 07/18/2014 01:20 PM, Andy Lutomirski wrote: >> >> The reason this is a concern is that: (x > x + n) and its variants is >> often used to mean (x > INT_MAX - n) without the type knowledge, but >> that is actually invalid standard C because signed types are not >> guaranteed to wrap. > > Right,

Re: [PATCH 4/4] (RFC) X86: add IPI tracepoints

2014-07-18 Thread Nicolas Pitre
On Fri, 18 Jul 2014, Steven Rostedt wrote: > On Fri, 18 Jul 2014 01:18:55 -0400 > Nicolas Pitre wrote: > > > > diff --git a/arch/x86/kernel/smp.c b/arch/x86/kernel/smp.c > > index be8e1bde07..e154d176cf 100644 > > --- a/arch/x86/kernel/smp.c > > +++ b/arch/x86/kernel/smp.c > > @@ -31,6 +31,12

Early microcode not being loaded

2014-07-18 Thread Conrad Kostecki
Hi! I'am trying to use the feature of loading early an intel microcode. According to the documentation (early-microcode.txt) that should be possible. I am currently using my own initrd, so I've created a second one, as the docs specified and merged it with my own. Galactica linux # file

Re: [PATCH 2/4] ARM: add IPI tracepoints

2014-07-18 Thread Nicolas Pitre
On Fri, 18 Jul 2014, Steven Rostedt wrote: > On Fri, 18 Jul 2014 01:18:53 -0400 > Nicolas Pitre wrote: > > > > -#ifdef CONFIG_IRQ_WORK > > -void arch_irq_work_raise(void) > > -{ > > - if (is_smp()) > > - smp_cross_call(cpumask_of(smp_processor_id()), IPI_IRQ_WORK); > > -} > >

[GIT PULL] EFI changes for v3.17

2014-07-18 Thread Matt Fleming
Hi guys, please queue up the following changes for v3.17. The following changes since commit 7171511eaec5bf23fb06078f59784a3a0626b38f: Linux 3.16-rc1 (2014-06-15 17:45:28 -1000) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git

[PATCH v2] mmc: mmci: Add qcom dml support to the driver.

2014-07-18 Thread Srinivas Kandagatla
On Qualcomm APQ8064 SOCs, SD card controller has an additional glue called DML (Data Mover Local/Lite) to assist dma transfers. This hardware needs to be setup before any dma transfer is requested. DML itself is not a DMA engine, its just a gule between the SD card controller and dma controller.

[GIT PULL] irqchip: Fixes for v3.16 (round #2)

2014-07-18 Thread Jason Cooper
Thomas, Here's all of the fixes I have queued up for v3.16. Nothing else is allowed to break ;-) This is an incremental pull request on the irqchip/urgent branch from tags/irqchip-urgent-3.16 up to tags/irqchip-urgent-3.16-2. Please pull. thx, Jason. The following changes since commit

Re: [PATCH] parisc: Remove FIXME comment

2014-07-18 Thread Helge Deller
On 07/18/2014 10:37 PM, Nicholas Krause wrote: > The comment for size of frame not being needed is incorrect , the > function called needs this parameter. Thanks for the patch Nicholas. It has been queued up: https://patchwork.kernel.org/patch/4587631/ and

[GIT PULL] please pull infiniband.git

2014-07-18 Thread Roland Dreier
Hi Linus, Please pull from git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband.git tags/rdma-for-linus InfiniBand/RDMA fixes for 3.16 - cxgb4 hardware driver regression fixes - mlx5 hardware driver regression

[PATCH] kbuild: devtest - new make target for build all and run tests

2014-07-18 Thread Shuah Khan
Add a new devtest make target to enable developer testing. This new target does full build (make all) and then runs selftests. Signed-off-by: Shuah Khan --- Makefile | 11 +++ 1 file changed, 11 insertions(+) diff --git a/Makefile b/Makefile index f3c543d..1ef3128 100644 --- a/Makefile

Re: How to automate checkpatch && get_maintainers && git send-email of commits range?

2014-07-18 Thread Greg KH
On Fri, Jul 18, 2014 at 06:24:15PM +0300, Andrey Utkin wrote: > 2014-07-18 17:46 GMT+03:00 Benoit Taine : > > On 18/07/2014 17:38, Andrey Utkin wrote: > >> Is there script for automated checkpatch.pl && get_maintainers.pl && > >> git send-email for range of commits? I see none. Would it be welcome

Re: [PATCH 1/1] checkpatch.pl: 1-tab relative indent for conditionals/blocks

2014-07-18 Thread Joe Perches
On Fri, 2014-07-18 at 12:40 -0700, Gregory Fong wrote: > The following would incorrectly pass checkpatch: > > void foo(void) > { > if (a) { > something; > somethingelse; > } else { > messed_up_indentation; > }

[PATCH] parisc: Remove FIXME comment

2014-07-18 Thread Nicholas Krause
The comment for size of frame not being needed is incorrect , the function called needs this parameter. Signed-off-by: Nicholas Krause --- arch/parisc/kernel/signal.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/parisc/kernel/signal.c b/arch/parisc/kernel/signal.c index

Re: [PATCH v6 4/5] PCI: add PCI controller for keystone PCIe h/w

2014-07-18 Thread Murali Karicheri
Rob, On 07/18/2014 03:31 PM, Rob Herring wrote: On Fri, Jul 18, 2014 at 10:14 AM, Murali Karicheri wrote: --- Cut --- + +Optional properties:- + phys: phandle to Generic Keystone SerDes phy for PCI + phy-names: name of the Generic Keystine SerDes phy for PCI + - If boot

Re: [PATCH 4/4] (RFC) X86: add IPI tracepoints

2014-07-18 Thread Steven Rostedt
On Fri, 18 Jul 2014 01:18:55 -0400 Nicolas Pitre wrote: > diff --git a/arch/x86/kernel/smp.c b/arch/x86/kernel/smp.c > index be8e1bde07..e154d176cf 100644 > --- a/arch/x86/kernel/smp.c > +++ b/arch/x86/kernel/smp.c > @@ -31,6 +31,12 @@ > #include > #include > #include > + > +#define

Re: [RFC] sched_clock: Track monotonic raw clock

2014-07-18 Thread Peter Zijlstra
On Fri, Jul 18, 2014 at 12:46:29PM -0700, John Stultz wrote: > On 07/18/2014 12:34 PM, Peter Zijlstra wrote: > > On Fri, Jul 18, 2014 at 12:25:48PM -0700, John Stultz wrote: > >> Also, assuming we someday will merge the x86 sched_clock logic into > >> the generic sched_clock code, we'll have to

Re: linux-next: build warning after merge of the tip tree

2014-07-18 Thread Andy Lutomirski
On Fri, Jul 18, 2014 at 1:15 PM, H. Peter Anvin wrote: > On 07/18/2014 01:08 PM, Andy Lutomirski wrote: >> >> i isn't an index in to the syms array at all. This code is completely >> wrong. See the patch I sent in reply to Stephen's original email. >> >> But, to your earlier point, presumably

Re: [PATCH v6 4/5] PCI: add PCI controller for keystone PCIe h/w

2014-07-18 Thread Murali Karicheri
On 07/18/2014 03:50 PM, Arnd Bergmann wrote: On Friday 18 July 2014 14:31:39 Rob Herring wrote: + + Example: + pcie_msi_intc: msi-interrupt-controller { + interrupt-controller; + #interrupt-cells =<1>; + interrupt-parent

Re: WARNING: CPU: 1 PID: 495 at mm/slab_common.c:69 kmem_cache_create+0x1a9/0x330()

2014-07-18 Thread poma
On 18.07.2014 22:07, James Bottomley wrote: On Fri, 2014-07-18 at 22:01 +0200, poma wrote: On 18.07.2014 16:17, Christoph Hellwig wrote: On Fri, Jul 18, 2014 at 05:21:04PM +0400, Vladimir Davydov wrote: Slab warns, because the name of the cache being created contains spaces. The "bad" cache

Re: Blackfin Build fixs

2014-07-18 Thread Nick Krause
On Wed, Jul 16, 2014 at 8:40 PM, Nick Krause wrote: > On Wed, Jul 16, 2014 at 3:12 AM, Steven Miao wrote: >> Hi Nick, >> >> On Sun, Jul 13, 2014 at 6:18 AM, Nick Krause wrote: >>> On Fri, Jul 11, 2014 at 12:18 PM, Nick Krause wrote: Hey Steven, I having been testing what builds are

Re: linux-next: build warning after merge of the tip tree

2014-07-18 Thread H. Peter Anvin
On 07/18/2014 01:08 PM, Andy Lutomirski wrote: > > i isn't an index in to the syms array at all. This code is completely > wrong. See the patch I sent in reply to Stephen's original email. > > But, to your earlier point, presumably this could warn: > > for (int i = 0; i < 10; i++) > if

[PATCH 2/2] Staging: comedi: adl_pci9118: line over 80 fixed

2014-07-18 Thread Sam Asadi
A 'line over 80 characters' issue fixed. Signed-off-by: Sam Asadi --- drivers/staging/comedi/drivers/adl_pci9118.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/adl_pci9118.c b/drivers/staging/comedi/drivers/adl_pci9118.c index

[PATCH 1/2] Staging: comedi: adl_pci9118: commenting style issue fixed

2014-07-18 Thread Sam Asadi
A 'quoted string split across lines' issue fixed, while a better use of language applied to the comment. Signed-off-by: Sam Asadi --- drivers/staging/comedi/drivers/adl_pci9118.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH v2 04/10] proc: convert /proc/$PID/cmdline to seq_file interface

2014-07-18 Thread Alexey Dobriyan
Signed-off-by: Alexey Dobriyan --- fs/proc/base.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) --- a/fs/proc/base.c +++ b/fs/proc/base.c @@ -200,9 +200,16 @@ static int proc_root_link(struct dentry *dentry, struct path *path) return result; } -static int

Re: [PATCH 04/10] proc: fix PAGE_SIZE limit of /proc/$PID/cmdline

2014-07-18 Thread Alexey Dobriyan
On Thu, Jul 17, 2014 at 03:53:46PM -0700, Andrew Morton wrote: > On Thu, 17 Jul 2014 00:32:47 +0300 Alexey Dobriyan > wrote: > > > Convert /proc/$PID/cmdline to seq_file interface. > > > > XXX > > Unsure what XXX signifies. > > > This one must be buggy. > > > > seq_file buffer is

Re: Power-managing devices that are not of interest at some point in time

2014-07-18 Thread Alan Stern
On Fri, 18 Jul 2014, Patrik Fimml wrote: > On Fri, Jul 18, 2014 at 03:00:46PM -0400, Alan Stern wrote: > > "Quiescing" is the wrong word. "Quiescing a device" means stopping the > > device from doing anything, which isn't what you want. You want to > > ignore any activity the device may

Re: linux-next: build warning after merge of the tip tree

2014-07-18 Thread Andy Lutomirski
On Fri, Jul 18, 2014 at 1:05 PM, H. Peter Anvin wrote: > On 07/18/2014 12:57 PM, Andy Lutomirski wrote: >> >> This particular warning is IMO in a particularly dumb category: GCC >> optimizes some code and then warns about a construct that wasn't there >> in the original code. In this case, I

[PATCH percpu/for-3.17 2/2] blk-throttle: replace custom async percpu alloc mechanism with percpu_pool

2014-07-18 Thread Tejun Heo
>From 0a4bd997ba9725565883c688d7dcee8264abf71c Mon Sep 17 00:00:00 2001 From: Tejun Heo Date: Fri, 18 Jul 2014 16:07:14 -0400 throtl_pd_init() is called under the queue lock but needs to allocate the percpu stats area. This is currently handled by queueing it on a list and scheduling a work

[PATCH percpu/for-3.17 1/2] percpu: implement percpu_pool

2014-07-18 Thread Tejun Heo
Hello, So, I have another case where percpu allocation needs to be performed from an atomic context (also on the IO path), so I wrote up a simple percpu alloc cache which is filled asynchronously and replaced blk-throttle's custom implementation with it. I still think it's quite unlikely that we

Re: WARNING: CPU: 1 PID: 495 at mm/slab_common.c:69 kmem_cache_create+0x1a9/0x330()

2014-07-18 Thread James Bottomley
On Fri, 2014-07-18 at 22:01 +0200, poma wrote: > On 18.07.2014 16:17, Christoph Hellwig wrote: > > On Fri, Jul 18, 2014 at 05:21:04PM +0400, Vladimir Davydov wrote: > >> Slab warns, because the name of the cache being created contains spaces. > >> The "bad" cache is created by

Re: linux-next: build warning after merge of the tip tree

2014-07-18 Thread H. Peter Anvin
On 07/18/2014 12:57 PM, Andy Lutomirski wrote: > > This particular warning is IMO in a particularly dumb category: GCC > optimizes some code and then warns about a construct that wasn't there > in the original code. In this case, I think it unrolled a loop and > discovered that one iteration

Re: [PATCH 2/4] ARM: add IPI tracepoints

2014-07-18 Thread Steven Rostedt
On Fri, 18 Jul 2014 01:18:53 -0400 Nicolas Pitre wrote: > -#ifdef CONFIG_IRQ_WORK > -void arch_irq_work_raise(void) > -{ > - if (is_smp()) > - smp_cross_call(cpumask_of(smp_processor_id()), IPI_IRQ_WORK); > -} > +static const char *ipi_types[NR_IPI] > +#ifdef CONFIG_TRACING >

Re: WARNING: CPU: 1 PID: 495 at mm/slab_common.c:69 kmem_cache_create+0x1a9/0x330()

2014-07-18 Thread poma
On 18.07.2014 16:20, Christoph Lameter wrote: On Fri, 18 Jul 2014, poma wrote: I guess someone working over the summertime. :) Cache names should not contain blanks. I guess the WARN_ON(strchr(name, ' ')); /* It confuses parsers */ was triggered? I can only guess also. ;) poma

Re: WARNING: CPU: 1 PID: 495 at mm/slab_common.c:69 kmem_cache_create+0x1a9/0x330()

2014-07-18 Thread poma
On 18.07.2014 16:17, Christoph Hellwig wrote: On Fri, Jul 18, 2014 at 05:21:04PM +0400, Vladimir Davydov wrote: Slab warns, because the name of the cache being created contains spaces. The "bad" cache is created by scsi_get_host_cmd_pool. Its name (pool->cmd_name) is initialized by

Re: Build Errors when building usb directory

2014-07-18 Thread Nick Krause
On Fri, Jul 18, 2014 at 2:10 PM, Randy Dunlap wrote: > On 07/18/2014 10:45 AM, Nick Krause wrote: >> Hey Greg and others, >> When I built the usb directory today to check a patch I am also >> sending to. I seem to hitting >> a few compiler errors and a lot of warnings. I am going to attach a >>

Intel Iris Graphics Faster on Windows

2014-07-18 Thread Nick Krause
This article, http://www.phoronix.com/scan.php?page=article=intel_iris5100_winlin=4 shows that iris graphics are faster on windows. I was wondering if this is correct and we need to improve them. Cheers Nick -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

Re: linux-next: build warning after merge of the tip tree

2014-07-18 Thread Andy Lutomirski
On Fri, Jul 18, 2014 at 12:16 PM, H. Peter Anvin wrote: > On 07/17/2014 10:00 PM, Stephen Rothwell wrote: >> Hi all, >> >> After merging the tip tree, today's linux-next build (x86_64 >> allmodconfig) produced these warnings: >> >> In file included from arch/x86/vdso/vdso2c.c:161:0: >>

Re: [PATCH] irqchip: gicv2m: Clean up logic for detecting MSI support

2014-07-18 Thread Jason Cooper
On Fri, Jul 18, 2014 at 08:26:36AM -0500, suravee.suthikulpa...@amd.com wrote: > From: Suravee Suthikulpanit > > It's not quite clear that msi-controller is already checked > by of_msi_chip_add. So, this patch add a note to clarify. > > Also, clean up redundant logic and unnecessary pr_info. >

Re: [PATCH] usb-core: Remove Fix mes in file hcd.c

2014-07-18 Thread Nick Krause
On Fri, Jul 18, 2014 at 3:09 PM, Alan Stern wrote: > On Fri, 18 Jul 2014, Nicholas Krause wrote: > >> I am removing two fix mes in this file as after dicussing then it seems >> there is no reason to check against Null for usb_device as it can never >> be NULL and this is check is therefore not

Re: Sparc: Fix Mes in highmem.c

2014-07-18 Thread Nick Krause
On Fri, Jul 18, 2014 at 2:20 PM, Sam Ravnborg wrote: > On Fri, Jul 18, 2014 at 01:06:41PM -0400, Nick Krause wrote: >> On Fri, Jul 18, 2014 at 10:08 AM, Steven Rostedt wrote: >> > On Thu, Jul 17, 2014 at 11:05:12PM -0400, Nick Krause wrote: >> >> On Thu, Jul 17, 2014 at 1:38 PM, Nick Krause

Re: [PATCH v6 4/5] PCI: add PCI controller for keystone PCIe h/w

2014-07-18 Thread Arnd Bergmann
On Friday 18 July 2014 14:31:39 Rob Herring wrote: > > + > > + Example: > > + pcie_msi_intc: msi-interrupt-controller { > > + interrupt-controller; > > + #interrupt-cells = <1>; > > + interrupt-parent = <>; > > +

Re: [RFC] sched_clock: Track monotonic raw clock

2014-07-18 Thread John Stultz
On 07/18/2014 12:34 PM, Peter Zijlstra wrote: > On Fri, Jul 18, 2014 at 12:25:48PM -0700, John Stultz wrote: >> Also, assuming we someday will merge the x86 sched_clock logic into >> the generic sched_clock code, we'll have to handle cases where they >> aren't the same. > I prefer that to not

Re: [PATCH v2] ARM: EXYNOS: Fix build with PM_SLEEP=n

2014-07-18 Thread Kukjin Kim
On 07/16/14 20:59, Tomasz Figa wrote: Hi Krzysztof, On 14.07.2014 09:45, Krzysztof Kozlowski wrote: Fix building of exynos defconfig with disabled PM_SLEEP: CONFIG_PM_SLEEP=n CONFIG_PM_SLEEP_SMP=n CONFIG_SUSPEND=n by moving functions for power up/down of CPU and cluster to platsmp.c The build

Re: [PATCH 13/19] ARM: s5pv210: move debug-macro.S into the common space

2014-07-18 Thread Kukjin Kim
On 07/16/14 09:56, Tomasz Figa wrote: On 16.07.2014 02:53, Kukjin Kim wrote: Kukjin Kim wrote: On 07/05/14 02:48, Tomasz Figa wrote: Move debug-macro.S from mach/include to include/debug where all other common debug macros are. Signed-off-by: Tomasz Figa --- arch/arm/Kconfig.debug

[PATCH 1/1] checkpatch.pl: 1-tab relative indent for conditionals/blocks

2014-07-18 Thread Gregory Fong
The following would incorrectly pass checkpatch: void foo(void) { if (a) { something; somethingelse; } else { messed_up_indentation; } } Assume single-tab indentation of blocks to catch this. Also add "else"

Re: [PATCH v3 3/7] printk: LOG_CONT and LOG_NEWLINE are opposites

2014-07-18 Thread Alex Elder
On 07/18/2014 09:14 AM, Alex Elder wrote: > Two log record flags--LOG_CONT and LOG_NEWLINE--are mutually > exclusive. That is, one or the other is always set, but they are > never both set at the same time in a log record flags field. What > follows is a great deal of explanation that aims to

Re: [RFC] sched_clock: Track monotonic raw clock

2014-07-18 Thread Peter Zijlstra
On Fri, Jul 18, 2014 at 12:25:48PM -0700, John Stultz wrote: > Also, assuming we someday will merge the x86 sched_clock logic into > the generic sched_clock code, we'll have to handle cases where they > aren't the same. I prefer that to not happen. I spend quite a bit of time and effort to make

Re: [PATCH v6 4/5] PCI: add PCI controller for keystone PCIe h/w

2014-07-18 Thread Rob Herring
On Fri, Jul 18, 2014 at 10:14 AM, Murali Karicheri wrote: > keystone PCIe controller is based on v3.65 version of the > designware h/w. Main differences are > 1. No ATU support > 2. Legacy and MSI irq functions are implemented in >application register space >

[PATCH 2/2] iio: exynos-adc: add experimental touchscreen support

2014-07-18 Thread Arnd Bergmann
This adds support for the touchscreen on Samsung s3c64xx. The driver is completely untested but shows roughly how it could be done, following the example of the at91 driver. Open questions include: - compared to the old plat-samsung/adc driver, there is no support for prioritizing ts over

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