Re: [RFC PATCH v2 -mm] provide estimated available memory in /proc/meminfo

2013-11-07 Thread Andrew Morton
On Thu, 7 Nov 2013 16:21:32 -0500 Johannes Weiner wrote: > > Subject: provide estimated available memory in /proc/meminfo > > > > Many load balancing and workload placing programs check /proc/meminfo > > to estimate how much free memory is available. They generally do this > > by adding up "free

Re: [RFC PATCH 0/4] wire up CPU features to udev based module loading

2013-11-07 Thread Andi Kleen
> - the module aliases host tool has no arch specific dependencies at > all except having x86cpu as one of the entries: would you mind > dropping the x86 prefix there? Or rather add dependencies on $ARCH? > (If we drop it there, we basically end up with 'cpu:' everywhere) Should be fine. > - in t

Re: [PATCH 2/4] irq_work: Provide a irq work that can be processed on any cpu

2013-11-07 Thread Frederic Weisbecker
On Thu, Nov 07, 2013 at 11:19:04PM +0100, Jan Kara wrote: > On Thu 07-11-13 23:13:39, Frederic Weisbecker wrote: > > But then, who's going to process that work if every CPUs is idle? > Have a look into irq_work_queue(). There is: > /* > * If the work is not "lazy" or the tick is

Re: [PATCH 05/11] ARM: Introduce CPU_METHOD_OF_DECLARE() for cpu hotplug/smp

2013-11-07 Thread Stephen Boyd
On 11/06/13 17:50, Josh Cartwright wrote: > On Fri, Nov 01, 2013 at 03:08:53PM -0700, Stephen Boyd wrote: >> diff --git a/arch/arm/kernel/devtree.c b/arch/arm/kernel/devtree.c >> index f35906b..71a8592 100644 >> --- a/arch/arm/kernel/devtree.c >> +++ b/arch/arm/kernel/devtree.c >> @@ -25,6 +25,7 @@

[PATCH] platform: add chrome platform directory

2013-11-07 Thread Olof Johansson
It makes sense to split out the Chromebook/Chromebox hardware platform drivers to a separate subdirectory, since some of it will be shared between ARM and x86. This moves over the existing chromeos_laptop driver without making any other changes, and adds appropriate Kconfig entries for the new dir

usb-serial lockdep trace in linus' current tree.

2013-11-07 Thread Dave Jones
Seeing this since todays USB merge. WARNING: CPU: 0 PID: 226 at kernel/lockdep.c:2740 lockdep_trace_alloc+0xc5/0xd0() DEBUG_LOCKS_WARN_ON(irqs_disabled_flags(flags)) Modules linked in: usb_debug(+) kvm_intel kvm crct10dif_pclmul crc32c_intel ghash_clmulni_intel microcode(+) pcspkr serio_raw CPU:

Re: [PATCH v3 3/5] MCS Lock: Barrier corrections

2013-11-07 Thread Michel Lespinasse
On Thu, Nov 7, 2013 at 2:21 PM, Peter Zijlstra wrote: > On Thu, Nov 07, 2013 at 01:15:51PM -0800, Tim Chen wrote: >> Michel, are you planning to do an implementation of >> load-acquire/store-release functions of various architectures? > > A little something like this: > http://marc.info/?l=linux-a

Re: [PATCH 3/4] printk: Defer printing to irq work when we printed too much

2013-11-07 Thread Frederic Weisbecker
2013/11/7 Jan Kara : > A CPU can be caught in console_unlock() for a long time (tens of seconds > are reported by our customers) when other CPUs are using printk heavily > and serial console makes printing slow. Despite serial console drivers > are calling touch_nmi_watchdog() this triggers softloc

Re: [BUG][ext2] XIP does not work on ext2

2013-11-07 Thread Andiry Xu
On Thu, Nov 7, 2013 at 2:20 PM, Jan Kara wrote: > On Thu 07-11-13 13:50:09, Andiry Xu wrote: >> On Thu, Nov 7, 2013 at 1:07 PM, Jan Kara wrote: >> > On Thu 07-11-13 12:14:13, Andiry Xu wrote: >> >> On Wed, Nov 6, 2013 at 1:18 PM, Jan Kara wrote: >> >> > On Tue 05-11-13 17:28:35, Andiry Xu wrote:

Re: [PATCH 2/4] irq_work: Provide a irq work that can be processed on any cpu

2013-11-07 Thread Jan Kara
On Thu 07-11-13 23:23:14, Frederic Weisbecker wrote: > On Thu, Nov 07, 2013 at 11:19:04PM +0100, Jan Kara wrote: > > On Thu 07-11-13 23:13:39, Frederic Weisbecker wrote: > > > But then, who's going to process that work if every CPUs is idle? > > Have a look into irq_work_queue(). There is: > >

Re: [RFC PATCH 0/4] wire up CPU features to udev based module loading

2013-11-07 Thread H. Peter Anvin
On 11/07/2013 02:15 PM, Ard Biesheuvel wrote: > > That would involve repurposing/generalizing a bit more of the existing > x86-only code than I did the first time around, but if you (as x86 > maintainers) are happy with that, I'm all for it. > > I do have a couple of questions then > - the module

Re: [PATCH 0/3 - V2] Introducing Device Tree Overlays

2013-11-07 Thread Guenter Roeck
On Thu, Nov 07, 2013 at 10:06:31PM +0200, Pantelis Antoniou wrote: > Hi Sebastian, > > On Nov 7, 2013, at 9:25 PM, Sebastian Andrzej Siewior wrote: > > > On 06.11.13, Guenter Roeck wrote: > > |… > > thanks for the explanation. > > > >> We use DT overlays to describe the hardware on those boards

Re: [PATCH] kmod: Run usermodehelpers only on cpus allowed for kthreadd V2

2013-11-07 Thread Frederic Weisbecker
On Thu, Nov 07, 2013 at 04:43:11PM +, Christoph Lameter wrote: > usermodehelper() threads can currently run on all processors. > This is an issue for low latency cores. Spawnig a new thread causes > cpu holdoffs in the range of hundreds of microseconds to a few > milliseconds. Not good for core

Re: [PATCH 2/4] irq_work: Provide a irq work that can be processed on any cpu

2013-11-07 Thread Frederic Weisbecker
On Thu, Nov 07, 2013 at 11:50:34PM +0100, Jan Kara wrote: > On Thu 07-11-13 23:23:14, Frederic Weisbecker wrote: > > On Thu, Nov 07, 2013 at 11:19:04PM +0100, Jan Kara wrote: > > > On Thu 07-11-13 23:13:39, Frederic Weisbecker wrote: > > > > But then, who's going to process that work if every CPUs

[PATCH v2] ext4: Fix reading of extended tv_sec (bug 23732)

2013-11-07 Thread David Turner
On Thu, 2013-11-07 at 17:03 +0100, Jan Kara wrote: > So I'm somewhat wondering: Previously we decoded tv_nsec regardless of > tv_sec size. After your patch we do it only if sizeof(time->tv_sec) > 4. Is > this an intended change? Why is it OK? This is an error. Here is a corrected version of the

Re: [PATCH 1/2] LEDS: tca6507 - fix bugs in parsing of device-tree configuration.

2013-11-07 Thread Bryan Wu
On Thu, Oct 31, 2013 at 7:33 PM, NeilBrown wrote: > > > 1/ The led_info array must be allocated to allow the full number > of LEDs even if not all are present. The array maybe be sparsely > filled but it is indexed by device address so we must at least > allocate as many slots as the highes

Re: [PATCH] staging: zsmalloc: Ensure handle is never 0 on success

2013-11-07 Thread Olav Haugan
On 11/6/2013 7:06 PM, Greg KH wrote: > On Wed, Nov 06, 2013 at 04:00:02PM -0800, Olav Haugan wrote: >> On 11/5/2013 5:56 PM, Greg KH wrote: >>> On Tue, Nov 05, 2013 at 04:54:12PM -0800, Olav Haugan wrote: zsmalloc encodes a handle using the page pfn and an object index. On some hardware p

Re: [PATCH 3/4] printk: Defer printing to irq work when we printed too much

2013-11-07 Thread Jan Kara
On Thu 07-11-13 23:43:52, Frederic Weisbecker wrote: > 2013/11/7 Jan Kara : > > A CPU can be caught in console_unlock() for a long time (tens of seconds > > are reported by our customers) when other CPUs are using printk heavily > > and serial console makes printing slow. Despite serial console dri

Re: [PATCH 3/4] printk: Defer printing to irq work when we printed too much

2013-11-07 Thread Steven Rostedt
On Thu, 7 Nov 2013 23:43:52 +0100 Frederic Weisbecker wrote: > When a message takes tens of seconds to be printed, it usually means > we are in trouble somehow :) > I wonder what printk source can trigger such a high volume. The only ones that I'm aware of is the prints from sysrq. Which includ

Re: [PATCH 0/3 - V2] Introducing Device Tree Overlays

2013-11-07 Thread Guenter Roeck
On Thu, Nov 07, 2013 at 09:46:26PM +0100, Sebastian Andrzej Siewior wrote: > On 07.11.13, Pantelis Antoniou wrote: > > Hi Sebastian, > Hi Pantelis, > > > FWIW DT has been ported to x86. And is present on arm/powerpc/mips/arc and > > possibly > > others. > > Yes, I know. I am the one that did the

Re: [PATCH 2/4] irq_work: Provide a irq work that can be processed on any cpu

2013-11-07 Thread Jan Kara
On Thu 07-11-13 23:54:10, Frederic Weisbecker wrote: > On Thu, Nov 07, 2013 at 11:50:34PM +0100, Jan Kara wrote: > > On Thu 07-11-13 23:23:14, Frederic Weisbecker wrote: > > > On Thu, Nov 07, 2013 at 11:19:04PM +0100, Jan Kara wrote: > > > > On Thu 07-11-13 23:13:39, Frederic Weisbecker wrote: > >

Re: [PATCH 1/2] dmaengine: add msm bam dma driver

2013-11-07 Thread Andy Gross
On Thu, Oct 31, 2013 at 04:46:21PM -0500, Andy Gross wrote: > On Thu, Oct 31, 2013 at 10:29:48PM +0530, Vinod Koul wrote: > > On Fri, Oct 25, 2013 at 03:24:02PM -0500, Andy Gross wrote: > > > > This should be sent to dmaeng...@vger.kernel.org > > I'll add the list when I send the second iteratio

Re: [PATCH v2] ext4: Fix reading of extended tv_sec (bug 23732)

2013-11-07 Thread Jan Kara
On Thu 07-11-13 17:54:24, David Turner wrote: > On Thu, 2013-11-07 at 17:03 +0100, Jan Kara wrote: > > So I'm somewhat wondering: Previously we decoded tv_nsec regardless of > > tv_sec size. After your patch we do it only if sizeof(time->tv_sec) > 4. Is > > this an intended change? Why is it OK?

common location for devicetree files

2013-11-07 Thread Kumar Gala
As we start having more sharing of device trees between architectures (arm & arm64, arm & powerpc, guessing maybe mips & arm) we need dts to live in location that I was wondering what people felt about doing: arch/dts// as a common location that could be shared. I'm up for other sugg

Re: [PATCH 3/4] printk: Defer printing to irq work when we printed too much

2013-11-07 Thread Frederic Weisbecker
On Thu, Nov 07, 2013 at 11:57:33PM +0100, Jan Kara wrote: > On Thu 07-11-13 23:43:52, Frederic Weisbecker wrote: > > 2013/11/7 Jan Kara : > > > A CPU can be caught in console_unlock() for a long time (tens of seconds > > > are reported by our customers) when other CPUs are using printk heavily > >

linux-next: reminder

2013-11-07 Thread Stephen Rothwell
Hi all, Just a reminder to *not* add any v3.14 material to linux-next until after v3.13-rc1 has been released. Also, please resist the temptation to rebase your trees before sending them upstream. -- Cheers, Stephen Rothwells...@canb.auug.org.au pgpYb_OjfDphf.pgp Descripti

[PATCH v3] ext4: Fix reading of extended tv_sec (bug 23732)

2013-11-07 Thread David Turner
On Fri, 2013-11-08 at 00:14 +0100, Jan Kara wrote: > Still unnecessary type cast here (but that's a cosmetic issue). ... > Otherwise the patch looks good. You can add: > Reviewed-by: Jan Kara Thanks. A version with this correction and the reviewed-by follows. -- In ext4, the bottom two bits of

Re: [PATCH v4 00/17] ARM: at91: move to common clk framework

2013-11-07 Thread Mike Turquette
Quoting Nicolas Ferre (2013-10-18 01:18:04) > On 11/10/2013 09:37, Boris BREZILLON : > > Hello, > > > > This patch series is the 4th version of the new at91 clock implementation > > (using common clk framework). > > Mike, DT maintainers, > > Can you have a look at this series converting the AT91

Re: [RFC PATCH 00/11] Embeddable Position Independent Executable

2013-11-07 Thread Kevin Hilman
On Tue, Sep 17, 2013 at 5:43 AM, Russ Dill wrote: > This patch adds support for and demonstrates the usage of an embedded > position independent executable (PIE). The goal is to allow the use of C > code in situations where carefully written position independent assembly > was previously required.

Re: [PATCH 2/4] irq_work: Provide a irq work that can be processed on any cpu

2013-11-07 Thread Steven Rostedt
On Fri, 8 Nov 2013 00:01:11 +0100 Jan Kara wrote: > On Thu 07-11-13 23:54:10, Frederic Weisbecker wrote: > > So if the current CPU can handle it, what is the problem? > I hope this gets cleared out in my other email. But to make sure: If > other CPUs are idle (i.e. not appending to the printk

Re: [PATCH 3/4] printk: Defer printing to irq work when we printed too much

2013-11-07 Thread Steven Rostedt
On Fri, 8 Nov 2013 00:21:51 +0100 Frederic Weisbecker wrote: > Ok I see now. > > But then this irq_work based solution won't work if, say, you run in full > dynticks > mode. Also the hook on the timer interrupt is something that I wish we get rid > of on archs that can trigger self-IPIs. Do w

Re: linux-next: manual merge of the gpio tree with the arm-soc and cortex trees

2013-11-07 Thread Stephen Rothwell
Hi Uwe, On Wed, 6 Nov 2013 09:41:17 +0100 Uwe Kleine-König wrote: > > BTW, "cortex" is a bit too general. My tree is only about Cortex-M. Also > you can drop -2.6 from the URL. Updated both, thanks. -- Cheers, Stephen Rothwells...@canb.auug.org.au pgpDNpGVBonys.pgp Descr

Re: [PATCH 3/3] DT: proc: Add runtime overlay interface in /proc

2013-11-07 Thread delicious quinoa
On Tue, Nov 5, 2013 at 12:41 PM, Pantelis Antoniou wrote: > + > + pr_info("%s: Applied #%d overlay segments @%d\n", __func__, > + od->ovinfo_cnt, od->id); > + This could be pr_debug so that we get normally get silence unless something fails, like insmod. Also please t

Re: [PATCH 2/2] LEDS: tca6502: add device-tree support for GPIO configuration.

2013-11-07 Thread Bryan Wu
On Thu, Oct 31, 2013 at 7:41 PM, NeilBrown wrote: > [PATCH 2/2] LEDS: tca6502: add device-tree support for GPIO configuration. I think it should be tca6507, right? Typo? For other parts of this patch, I'm OK for them. And just a quick scan of the leds-tca6507, I found bunch of typos in the com

[PATCH linux-next] cifs: Use data structures to compute NTLMv2 response offsets

2013-11-07 Thread Tim Gardner
A bit of cleanup plus some gratuitous variable renaming. I think using structures instead of numeric offsets makes this code much more understandable. Also added a comment about current time range expected by the server. Cc: Jeff Layton Cc: Steve French Signed-off-by: Tim Gardner --- The comm

Re: [PATCH] ARM: OMAP2+: omap_device: maintain sane runtime pm status around suspend/resume

2013-11-07 Thread Kevin Hilman
indicates the device status. As a result of this change > any further calls to pm_runtime_get* would return -EACCES (since > disable_depth is 1). On resume, we restore the clocks and runtime > status exactly as we suspended with. > > Reported-by: J Keerthy > Signed-off-by: Nishanth Meno

Re: [PATCH 3/4] printk: Defer printing to irq work when we printed too much

2013-11-07 Thread Frederic Weisbecker
On Thu, Nov 07, 2013 at 06:37:17PM -0500, Steven Rostedt wrote: > On Fri, 8 Nov 2013 00:21:51 +0100 > Frederic Weisbecker wrote: > > > > Offloading to a workqueue would be perhaps better, and writing to the serial > > console could then be done with interrupts enabled, preemptible context, > > e

Re: [PATCH 2/2] LEDS: tca6502: add device-tree support for GPIO configuration.

2013-11-07 Thread NeilBrown
On Thu, 7 Nov 2013 15:39:00 -0800 Bryan Wu wrote: > On Thu, Oct 31, 2013 at 7:41 PM, NeilBrown wrote: > > > > [PATCH 2/2] LEDS: tca6502: add device-tree support for GPIO configuration. > > I think it should be tca6507, right? Typo? Obviously I'm still dreaming of the Apple IIe. Yes, a typo.

Re: [PATCH 2/2] LEDS: tca6502: add device-tree support for GPIO configuration.

2013-11-07 Thread Bryan Wu
On Thu, Nov 7, 2013 at 3:39 PM, Bryan Wu wrote: > On Thu, Oct 31, 2013 at 7:41 PM, NeilBrown wrote: >> > > [PATCH 2/2] LEDS: tca6502: add device-tree support for GPIO configuration. > > I think it should be tca6507, right? Typo? > I fixed this typo and queue up this patch in my devel branch. -B

Re: [PATCH 3/4] printk: Defer printing to irq work when we printed too much

2013-11-07 Thread Frederic Weisbecker
On Thu, Nov 07, 2013 at 06:37:17PM -0500, Steven Rostedt wrote: > On Fri, 8 Nov 2013 00:21:51 +0100 > Frederic Weisbecker wrote: > > > Ok I see now. > > > > But then this irq_work based solution won't work if, say, you run in full > > dynticks > > mode. Also the hook on the timer interrupt is

Re: kernel BUG at kernel/kallsyms.c:222!

2013-11-07 Thread Rusty Russell
Axel Lin writes: > 2013/11/7 Ming Lei : >> Hi, >> >> On Thu, Nov 7, 2013 at 10:47 AM, Axel Lin wrote: >>> >>> hi Ming, >>> Seems CONFIG_PAGE_OFFSET is not configurabe in "make menuconfig". >>> And I found CONFIG_PAGE_OFFSET=0xC000 for all below configs... >>> $ make at91_dt_defconfig; grep C

Re: [trivial PATCH] module.h: Remove unnecessary semicolon

2013-11-07 Thread Rusty Russell
Joe Perches writes: > On Thu, 2013-11-07 at 12:32 +1030, Rusty Russell wrote: >> Joe Perches writes: >> > This semicolon isn't necessary, remove it. >> > >> > Signed-off-by: Joe Perches >> >> This is a terrible description. Really bad. > > I'd've preferred no description. Me too. >> First, i

[PATCH 01/11] random: fix typos / spelling errors in comments

2013-11-07 Thread Greg Price
Cc: "Theodore Ts'o" Signed-off-by: Greg Price --- drivers/char/random.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/char/random.c b/drivers/char/random.c index 7a744d3..ef3e15b 100644 --- a/drivers/char/random.c +++ b/drivers/char/random.c @@ -348,12 +348,

Re: [trivial PATCH] module.h: Remove unnecessary semicolon

2013-11-07 Thread Joe Perches
On Fri, 2013-11-08 at 09:41 +1030, Rusty Russell wrote: > Joe Perches writes: > > On Thu, 2013-11-07 at 12:32 +1030, Rusty Russell wrote: > >> Joe Perches writes: > >> > This semicolon isn't necessary, remove it. > >> > > >> > Signed-off-by: Joe Perches > >> > >> This is a terrible description.

[PATCH 00/11] random: code cleanups

2013-11-07 Thread Greg Price
Hi Ted, hi all, I recently read through the random number generator's code. This series has fixes for some minor things I spotted. Four of the patches touch comments only. Four simplify code without changing its behavior (total diffstat: 35 insertions, 73 deletions), and one is a trivial signedn

[PATCH 02/11] random: fix comment on proc_do_uuid

2013-11-07 Thread Greg Price
There's only one function here now, as uuid_strategy is long gone. Also make the bit about "If accesses via ..." clearer. Cc: "Theodore Ts'o" Signed-off-by: Greg Price --- drivers/char/random.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/char/random.c b/d

[PATCH 03/11] random: fix description of get_random_bytes

2013-11-07 Thread Greg Price
After this remark was written, commit d2e7c96af added a use of arch_get_random_long() inside the get_random_bytes codepath. The main point stands, but it needs to be reworded. Cc: "Theodore Ts'o" Signed-off-by: Greg Price --- drivers/char/random.c | 5 +++-- 1 file changed, 3 insertions(+), 2 d

[PATCH 06/11] random: fix comment on "account"

2013-11-07 Thread Greg Price
This comment didn't quite keep up as extract_entropy() was split into four functions. Put each bit by the function it describes. Cc: "Theodore Ts'o" Signed-off-by: Greg Price --- drivers/char/random.c | 31 +-- 1 file changed, 21 insertions(+), 10 deletions(-) diff

[PATCH 05/11] random: declare trickle_count unsigned

2013-11-07 Thread Greg Price
We cheerfully overflow these variables (at least where "int" is 32 bits wide), so pedantically they should be unsigned. Cc: "Theodore Ts'o" Signed-off-by: Greg Price --- drivers/char/random.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/random.c b/drivers/cha

[PATCH 10/11] random: pull 'min' check in accounting to inside lockless update

2013-11-07 Thread Greg Price
Since 902c098a3 ("random: use lockless techniques in the interrupt path") we have protected entropy_count only with cmpxchg, not the lock. In 10b3a32d2 ("random: fix accounting race condition") we put a cmpxchg-retry loop around most of the logic in account(), but the enforcement of the 'min' para

[PATCH 11/11] random: simplify accounting code

2013-11-07 Thread Greg Price
With this we handle "reserved" in just one place. As a bonus the code becomes less nested, and the "wakeup_write" flag variable becomes unnecessary. The variable "flags" was already unused. This code behaves identically to the previous version except in two pathological cases that don't occur.

[PATCH 09/11] random: forget lock in lockless accounting

2013-11-07 Thread Greg Price
The only mutable data accessed here is ->entropy_count, but since 902c098a3 ("random: use lockless techniques in the interrupt path") that member isn't protected by the lock anyway. Since 10b3a32d2 ("random: fix accounting race condition") we use cmpxchg to protect our accesses to ->entropy_count

[PATCH 04/11] random: simplify loop in random_read

2013-11-07 Thread Greg Price
The loop condition never changes until just before a break, so we might as well write it as a constant. Also since v2.6.33-rc7~40^2~2 ("random: drop weird m_time/a_time manipulation") we don't do anything after the loop finishes, so the 'break's might as well return directly. Some other simplific

[PATCH 07/11] random: simplify accounting code slightly

2013-11-07 Thread Greg Price
This commit makes the very boring simplifications so that the next commit, which is a little trickier, is isolated and easy to review. No change in behavior here at all. Cc: "Theodore Ts'o" Cc: Jiri Kosina Signed-off-by: Greg Price --- drivers/char/random.c | 10 -- 1 file changed, 4 i

RE: Bench for testing scheduler

2013-11-07 Thread Rowand, Frank
Hi Vincent, Thanks for creating some benchmark numbers! On Thursday, November 07, 2013 5:33 AM, Vincent Guittot [vincent.guit...@linaro.org] wrote: > > On 7 November 2013 12:32, Catalin Marinas wrote: > > Hi Vincent, > > > > (for whatever reason, the text is wrapped and results hard to read)

Re: linux-next: manual merge of the block tree with the tree

2013-11-07 Thread Dave Kleikamp
On 11/07/2013 01:25 PM, Kent Overstreet wrote: > On Thu, Nov 07, 2013 at 01:20:26PM -0600, Dave Kleikamp wrote: >> I ended up replacing a call to bio_iovec_idx() with __bvec_iter_bvec() >> since the former was removed. It's not very elegant, but it works. I'm >> open to suggestions on a cleaner fi

[PATCH 08/11] random: simplify accounting logic

2013-11-07 Thread Greg Price
This logic is exactly equivalent to the old logic, but it should be easier to see what it's doing. The equivalence depends on one fact from outside this function: when 'r->limit' is false, 'reserved' is zero. (Well, two facts; the other is that 'reserved' is never negative.) Which is a good thin

Re: [PATCH] net: x86: bpf: don't forget to free sk_filter (v2)

2013-11-07 Thread David Miller
From: Andrey Vagin Date: Thu, 7 Nov 2013 08:35:12 +0400 > sk_filter isn't freed if bpf_func is equal to sk_run_filter. > > This memory leak was introduced by v3.12-rc3-224-gd45ed4a4 > "net: fix unsafe set_memory_rw from softirq". > > Before this patch sk_filter was freed in sk_filter_release_r

Re: [PATCH] ARM: OMAP2+: omap_device: maintain sane runtime pm status around suspend/resume

2013-11-07 Thread Nishanth Menon
nded with. Reported-by: J Keerthy Signed-off-by: Nishanth Menon Acked-by: Rajendra Nayak --- patch baseline: V3.12 tag (also applies on linux-next next-20131107 tag) Logs from 3.12 based vendor kernel: Before: http://pastebin.com/m5KxnB7a After: http://pastebin.com/8AfX4e7r The discussion on cpufre

Re: usb-serial lockdep trace in linus' current tree.

2013-11-07 Thread Johan Hovold
On Thu, Nov 07, 2013 at 05:37:28PM -0500, Dave Jones wrote: > Seeing this since todays USB merge. > > WARNING: CPU: 0 PID: 226 at kernel/lockdep.c:2740 > lockdep_trace_alloc+0xc5/0xd0() > DEBUG_LOCKS_WARN_ON(irqs_disabled_flags(flags)) > Modules linked in: usb_debug(+) kvm_intel kvm crct10dif_pclm

[PATCH 1/3] drivers/bus: Add Legacy PM OPS usage check and warning to bus_register()

2013-11-07 Thread Shuah Khan
Add Legacy PM OPS usage checks to bus_register() function. If Legacy PM OPS usage is found, print warning message to indicate the driver code needs updating to use Dev PM OPS interfaces. This will help serve as a way to track drivers that still use Legacy PM OPS and fix them. The Legacy PM OPS che

[PATCH 3/3] driver: Add Legacy PM OPS usage check and warning to driver_register()

2013-11-07 Thread Shuah Khan
Add Legacy PM OPS usage checks to driver_register() function. If Legacy PM OPS usage is found, print warning message to indicate the driver code needs updating to use Dev PM OPS interfaces. This will help serve as a way to track drivers that still use Legacy PM OPS and fix them. The Legacy PM OPS

[PATCH 2/3] drivers/class: Add Legacy PM OPS usage check and warning to __class_register()

2013-11-07 Thread Shuah Khan
Add Legacy PM OPS usage checks to __class_register() function. If Legacy PM OPS usage is found, print warning message to indicate the driver code needs updating to use Dev PM OPS interfaces. This will help serve as a way to track drivers that still use Legacy PM OPS and fix them. The Legacy PM OPS

[PATCH 0/3] Add Legacy PM OPS usage checks to class, bus, and driver register functions

2013-11-07 Thread Shuah Khan
Add Legacy PM OPS usage checks to class, bus, and driver register functions. If Legacy PM OPS usage is found, print warning message to indicate the driver code needs updating to use Dev PM OPS interfaces. This will help serve as a way to track drivers that still use Legacy PM OPS and fix them. Thi

Re: [patch] net: make ndev->irq signed for error handling

2013-11-07 Thread David Miller
From: Dan Carpenter Date: Thu, 7 Nov 2013 10:48:49 +0300 > There is a bug in cpsw_probe() where we do: > > ndev->irq = platform_get_irq(pdev, 0); > if (ndev->irq < 0) { > > The problem is that "ndev->irq" is unsigned so the error handling > doesn't work. I have changed it to a regu

Congratulation!!! Confirm Mail Receipt!!

2013-11-07 Thread Qatar Foundation
This is to inform you that you were among the lucky beneficiary selected to receive this donations award sum of $ 1Million US DOLLAR each as charity donations/aid from the Qatar Foundation to promote your business and personal Interest. Kindly revert back for claims processing via email: qatarf

Re: usb-serial lockdep trace in linus' current tree.

2013-11-07 Thread Dave Jones
On Fri, Nov 08, 2013 at 01:09:03AM +0100, Johan Hovold wrote: > A recent change in usb-serial used the wrong memory-allocation flag in > write(), which results in a > > [5.979005] BUG: sleeping function called from invalid context at > /home/johan/src/linux/linux-nu/mm/dmapool.c:3

Re: [RFC PATCH v2 -mm] provide estimated available memory in /proc/meminfo

2013-11-07 Thread Minchan Kim
Hi Hannes, Rik. On Thu, Nov 07, 2013 at 04:21:32PM -0500, Johannes Weiner wrote: > On Thu, Nov 07, 2013 at 10:13:45AM -0500, Rik van Riel wrote: > > > > > > fs/proc/meminfo.c | 36 > > > > 1 file changed, 36 insertions(+) > > > > > > Documentation/filesystem

Re: [PATCH 3/4] ARM: pinctrl: Add Broadcom Capri pinctrl driver

2013-11-07 Thread Sherman Yin
On 13-11-06 01:40 AM, Linus Walleij wrote: On Wed, Nov 6, 2013 at 3:02 AM, Sherman Yin wrote: When Linus asked me to try using generic pinconf instead, I ran into problems with this feature due to how the generic pinconf properties are defined differently than my properties - perhaps this featu

[PATCH] doc: devicetree: Add bindings documentation for omap-des driver

2013-11-07 Thread Joel Fernandes
Add documentation for the generic OMAP DES crypto module describing the device tree bindings. Signed-off-by: Joel Fernandes --- .../devicetree/bindings/crypto/omap-des.txt| 28 ++ 1 file changed, 28 insertions(+) create mode 100644 Documentation/devicetree/bindings/c

Re: [PATCH] ARM: OMAP2+: omap_device: maintain sane runtime pm status around suspend/resume

2013-11-07 Thread Kevin Hilman
ee. Makes sense. >> >> That being said, I agree that omap_device should still be catching this >> case in order to find/fix driver races like this. > >> >>> To prevent this from happening, we should ensure that runtime_status >>> exactly indicates th

Re: [PATCH] doc: devicetree: Add bindings documentation for omap-des driver

2013-11-07 Thread Santosh Shilimkar
On Thursday 07 November 2013 07:37 PM, Joel Fernandes wrote: > Add documentation for the generic OMAP DES crypto module describing the device > tree bindings. > > Signed-off-by: Joel Fernandes > --- Thanks Joel for picking this up. FWIW, Acked-by: Santosh Shilimkar -- To unsubscribe from this

Re: kernel BUG at kernel/kallsyms.c:222!

2013-11-07 Thread Ming Lei
On Fri, Nov 8, 2013 at 7:44 AM, Rusty Russell wrote: > Axel Lin writes: >> 2013/11/7 Ming Lei : >>> Hi, >>> >>> On Thu, Nov 7, 2013 at 10:47 AM, Axel Lin wrote: hi Ming, Seems CONFIG_PAGE_OFFSET is not configurabe in "make menuconfig". And I found CONFIG_PAGE_OFFSET=0xC00

Re: [RFC PATCH RESEND 1/2] clk: add clk accuracy retrieval support

2013-11-07 Thread Mike Turquette
Quoting Boris BREZILLON (2013-10-13 10:17:10) > The clock accuracy is expressed in ppb (parts per billion) and represents > the possible clock drift. > Say you have a clock (e.g. an oscillator) which provides a fixed clock of > 20MHz with an accuracy of +- 20Hz. This accuracy expressed in ppb is >

[PATCH 3/3] panic: enable local IRQs for restart timeout too

2013-11-07 Thread Felipe Contreras
Signed-off-by: Felipe Contreras --- kernel/panic.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/kernel/panic.c b/kernel/panic.c index 5a0bdaa..e32919f 100644 --- a/kernel/panic.c +++ b/kernel/panic.c @@ -133,6 +133,8 @@ void panic(const char *fmt, ...) bust_spin

[PATCH 2/3] panic: improve panic_timeout calculation

2013-11-07 Thread Felipe Contreras
We want to calculate the blinks per second, and instead of making it 5 (1000 / (3600 / 18)), let's make it 4, so the user can see two blinks per second. Signed-off-by: Felipe Contreras --- kernel/panic.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/panic.c b/k

[PATCH 1/3] panic: setup panic_timeout early

2013-11-07 Thread Felipe Contreras
Otherwise we might not reboot when the user needs it the most (early on). Signed-off-by: Felipe Contreras --- kernel/panic.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/kernel/panic.c b/kernel/panic.c index b6c482c..46e37ff 100644 --- a/kernel/panic.c +++ b

[PATCH 0/3] panic: improvements

2013-11-07 Thread Felipe Contreras
Hi, While debugging a hang after panic in my laptop (I cannot hard-reboot, or unplug the battery), I noticed the panic code has some areas of improvment. Only the first patch is really needed, the rest seem like good to have. Felipe Contreras (3): panic: setup panic_timeout early panic: impr

Re: [PATCH] net: don't forget to free sk_filter

2013-11-07 Thread Keller, Jacob E
On Wed, 2013-11-06 at 11:28 -0800, Eric Dumazet wrote: > On Wed, 2013-11-06 at 20:19 +0100, Daniel Borkmann wrote: > > > When you send v2 with Alexei's feedback, please also be more specific > > in your subject like "net: x86: bpf: don't forget to free sk_filter" > > or the like. Also it's enough

Re: [PATCH v2] kernel/rcutorture.c: be sure of enough memory for result printing

2013-11-07 Thread Chen Gang
On 11/08/2013 04:59 AM, Paul E. McKenney wrote: > On Thu, Nov 07, 2013 at 10:30:25AM +0800, Chen Gang wrote: >> > If the contents is more than 4096 bytes (e.g. if have 1K cpus), current >> > sprintf() will cause memory overflow. And this fix patch is to be sure >> > of memory large enough. >> > >>

Re: [PATCH] net: don't forget to free sk_filter

2013-11-07 Thread Eric Dumazet
On Fri, 2013-11-08 at 00:56 +, Keller, Jacob E wrote: > Has there been any mention of a forthcoming patch > to ./Documentation/SubmittingPatches which documents this? I do not remember if this point was raised, but you certainly can submit a patch ! Thanks -- To unsubscribe from this list:

linux-next: manual merge of the vfs tree with Linus' tree

2013-11-07 Thread Stephen Rothwell
Hi Al, Today's linux-next merge of the vfs tree got a conflict in drivers/usb/core/file.c between commit 469271f8c48f ("drivers: usb: core: {file,hub,sysfs,usb}.c: Whitespace fixes") from Linus' tree and commit e84f9e57b90c ("consolidate the reassignments of ->f_op in ->open() instances") from the

Re: [PATCH v3 9/9] ARM: add initial support for Marvell Berlin SoCs

2013-11-07 Thread Jisheng Zhang
Dear Sebastian, On Thu, 7 Nov 2013 13:22:35 -0800 Sebastian Hesselbarth wrote: > On 11/07/2013 05:20 PM, Arnd Bergmann wrote: > > On Thursday 07 November 2013, Sebastian Hesselbarth wrote: > >> Actually, IIRC smp_twd does not compile without SMP set, so the above > >> should at least be 'HAVE_AR

Re: [PATCH v3 3/5] MCS Lock: Barrier corrections

2013-11-07 Thread Tim Chen
On Thu, 2013-11-07 at 14:43 -0800, Michel Lespinasse wrote: > On Thu, Nov 7, 2013 at 2:21 PM, Peter Zijlstra wrote: > > On Thu, Nov 07, 2013 at 01:15:51PM -0800, Tim Chen wrote: > >> Michel, are you planning to do an implementation of > >> load-acquire/store-release functions of various architectu

[PATCH] ARM: Save/Restore 64-bit TTBR registers on LPAE suspend/resume

2013-11-07 Thread Stephen Boyd
From: Mahesh Sivasubramanian LPAE enabled kernels use the 64-bit version of TTBR0 and TTBR1 registers. If we're running an LPAE kernel, fill the upper half of TTBR0 with 0 because we're setting it to the idmap here (the idmap is guaranteed to be < 4Gb) and fully restore TTBR1 instead of just rest

Re: [trivial PATCH] module.h: Remove unnecessary semicolon

2013-11-07 Thread Rusty Russell
Joe Perches writes: > On Fri, 2013-11-08 at 09:41 +1030, Rusty Russell wrote: >> Joe Perches writes: >> > That's a trust issue. >> > I've done it. It isn't necessary. >> >> WTF? Now you just said it's not necessary, I *know* I can't trust you. > > "It" in this case is the grep that I did > pri

Re: [PATCH] ARM: Use udiv/sdiv for __aeabi_{u}idiv library functions

2013-11-07 Thread Rob Herring
On Thu, Nov 7, 2013 at 1:20 PM, Stephen Boyd wrote: > If we're running on a v7 ARM CPU, detect if the CPU supports the > sdiv/udiv instructions and replace the signed and unsigned > division library functions with an sdiv/udiv instruction. [snip] > diff --git a/arch/arm/lib/Makefile b/arch/arm/l

Re: [PATCH] x86: make reboot task only run on the appropriate processor

2013-11-07 Thread Dave Young
On 11/05/13 at 05:16pm, Baoquan He wrote: > Currently system always reboot after below message when execute "kexec -e". > > [0.572119] smpboot: Booting Node 0, Processors # 1 OK > > In commit 1b3a5d02ee070c8f994b9b6370f486601e0f, reboot= handling was > moved to kerne/reboot.c. Howeve

Re: [PATCH] ARM: pxa/colibri: fix build with CONFIG_USB_OHCI_HCD=y

2013-11-07 Thread Marek Vasut
Dear Dmitry Artamonow, > Fix following build failure observed when support for > OHCI host controller is enabled. > > CC arch/arm/mach-pxa/colibri-evalboard.o > arch/arm/mach-pxa/colibri-evalboard.c: In function 'colibri_ohci_init': > arch/arm/mach-pxa/colibri-evalboard.c:68: error: implic

Re: [PATCH] PCI: Introduce two new MSI infrastructure calls for masking/unmasking.

2013-11-07 Thread Zhenzhong Duan
On 2013-11-07 07:51, Bjorn Helgaas wrote: [+cc Thomas, Ingo, Peter, x86 list] On Wed, Nov 6, 2013 at 2:16 PM, Konrad Rzeszutek Wilk wrote: Certain platforms do not allow writes in the MSI-X bars to setup or tear down vector values. To combat against the generic code trying to write to that an

Re: [PATCH linux-next] cifs: Use data structures to compute NTLMv2 response offsets

2013-11-07 Thread Jeff Layton
On Thu, 7 Nov 2013 16:40:57 -0700 Tim Gardner wrote: > A bit of cleanup plus some gratuitous variable renaming. I think using > structures instead of numeric offsets makes this code much more > understandable. > > Also added a comment about current time range expected by > the server. > > Cc:

Re: [trivial PATCH] module.h: Remove unnecessary semicolon

2013-11-07 Thread Joe Perches
On Fri, 2013-11-08 at 11:56 +1030, Rusty Russell wrote: > Joe Perches writes: > > On Fri, 2013-11-08 at 09:41 +1030, Rusty Russell wrote: > >> Joe Perches writes: > >> > That's a trust issue. > >> > I've done it. It isn't necessary. > >> > >> WTF? Now you just said it's not necessary, I *know*

Re: linux-next: manual merge of the block tree with the tree

2013-11-07 Thread Stephen Rothwell
Hi all, On Thu, 07 Nov 2013 18:04:57 -0600 Dave Kleikamp wrote: > > Can you please drop the aio-direct tree for the time being? OK, I was afraid of this, but, yes, I can drop it. I am not quite sure what affect this will have on Andrew's tree, though (hopefully not too much). This is a bit di

[PATCH v2] staging: zsmalloc: Ensure handle is never 0 on success

2013-11-07 Thread Olav Haugan
zsmalloc encodes a handle using the pfn and an object index. On hardware platforms with physical memory starting at 0x0 the pfn can be 0. This causes the encoded handle to be 0 and is incorrectly interpreted as an allocation failure. To prevent this false error we ensure that the encoded handle wi

Re: [PATCH] staging: zsmalloc: Ensure handle is never 0 on success

2013-11-07 Thread Greg KH
On Thu, Nov 07, 2013 at 09:06:26AM -0800, Luigi Semenzato wrote: > If I may add my usual 2c (and some news): > > zram is used by default on all Chrome OS devices. I can't say how > many devices, but it's not a small number, google it, and it's an > important market, low-end laptops for education

Re: linux-next: manual merge of the block tree with the tree

2013-11-07 Thread Kent Overstreet
On Fri, Nov 08, 2013 at 12:53:07PM +1100, Stephen Rothwell wrote: > Hi all, > > On Thu, 07 Nov 2013 18:04:57 -0600 Dave Kleikamp > wrote: > > > > Can you please drop the aio-direct tree for the time being? > > OK, I was afraid of this, but, yes, I can drop it. I am not quite sure > what affect

[3.8.y.z extended stable] Linux 3.8.13.13 stable review

2013-11-07 Thread Kamal Mostafa
This is the start of the review cycle for the Linux 3.8.13.13 stable kernel. This version contains 91 new patches, summarized below. The new patches are posted as replies to this message and also available in this git branch: http://kernel.ubuntu.com/git?p=ubuntu/linux.git;h=linux-3.8.y-review;a

[PATCH 3.8 04/91] ACPICA: Fix for a Store->ArgX when ArgX contains a reference to a field.

2013-11-07 Thread Kamal Mostafa
3.8.13.13 -stable review patch. If anyone has any objections, please let me know. -- From: Bob Moore commit 4be4be8fee2ee99a52f94f90d03d2f287ee1db86 upstream. This change fixes a problem where a Store operation to an ArgX object that contained a reference to a field object di

[PATCH 3.8 81/91] ALSA: hda - Add a fixup for ASUS N76VZ

2013-11-07 Thread Kamal Mostafa
3.8.13.13 -stable review patch. If anyone has any objections, please let me know. -- From: Takashi Iwai commit 6fc16e58adf50c0f1e4478538983fb5ff6f453d4 upstream. ASUS N76VZ needs the same fixup as N56VZ for supporting the boost speaker. Bugzilla: https://bugzilla.novell.com/

[PATCH 3.8 08/91] net: Add skb_unclone() helper function.

2013-11-07 Thread Kamal Mostafa
3.8.13.13 -stable review patch. If anyone has any objections, please let me know. -- From: Pravin B Shelar commit 14bbd6a565e1bcdc240d44687edb93f721cfdf99 upstream. This function will be used in next GRE_GSO patch. This patch does not change any functionality. Signed-off-by:

<    1   2   3   4   5   6   7   >