[PATCH v6 3/3] mmc: dw_mmc: Set timeout to max upon resume

2013-08-22 Thread Doug Anderson
The TMOUT register is initted to 0x at probe time but isn't initted after suspend/resume. Add an init of this value. No problems were observed without this (it will also get initted in __dw_mci_start_request if there is data to send), but it makes the register dump before and after suspen

Re: [PATCH 2/2] powerpc/iommu: check dev->iommu_group before remove a device from iommu_group

2013-08-22 Thread Alex Williamson
On Thu, 2013-08-22 at 23:41 +0800, Wei Yang wrote: > On Thu, Aug 22, 2013 at 09:28:23AM -0600, Alex Williamson wrote: > >On Thu, 2013-08-22 at 15:52 +0800, Wei Yang wrote: > >> On Thu, Aug 22, 2013 at 05:23:34PM +1000, Alexey Kardashevskiy wrote: > >> >On 08/19/2013 11:55 AM, Wei Yang wrote: > >> >

[PATCH v6 0/3] mmc: dw_mmc: fixes for suspend/resume on exynos

2013-08-22 Thread Doug Anderson
This series of patches addresses some suspend/resume problems with dw_mmc on exynos platforms, espeically exynos5420. This patchset was tested on the current ToT Chromeos 3.8 tree (which has lots of backports from 3.10/3.11) and on ToT Linux (v3.11-rc6). I have confirmed basic booting and that SD

Re: [PATCH] sysfs: add sysfs_create/remove_groups()

2013-08-22 Thread Greg Kroah-Hartman
On Wed, Aug 21, 2013 at 10:55:19PM -0600, Anthony Foiani wrote: > Greg Kroah-Hartman writes: > > > + * sysfs_create_groups - given a directory kobject, create a bunch of > > attribute groups > > + * @kobj: The kobject to create the group on > > + * @groups:The attribute groups to create

[PATCH 1/2] net/cadence/macb: add support for dt phy definition

2013-08-22 Thread Boris BREZILLON
The macb driver only handle PHY description through platform_data (macb_platform_data). Thus, when using dt you cannot define phy properties like phy address or phy irq pin. This patch makes use of the of_mdiobus_register to add support for phy device definition using dt. A fallback to the autosca

Re: [PATCH v4 1/4] mmc: dw_mmc: Invalidate cache of current_speed after suspend/resume

2013-08-22 Thread Doug Anderson
Seungwon, On Wed, Aug 21, 2013 at 5:54 PM, Doug Anderson wrote: >> Doug, your analysis is right. >> But, let me suggest another approach. >> After step #1, core layer actually call mmc_power_off because slot is >> empthy(get_cd() is '0'). >> Then, set_ios is requested with 'ios->clock'. >> Howev

[for-next][PATCH 2/5] tracing/syscalls: Annotate raw_init function with __init

2013-08-22 Thread Steven Rostedt
From: Li Zefan init_syscall_trace() can only be called during kernel bootup only, so we can mark it and the functions it calls as __init. Link: http://lkml.kernel.org/r/51528e89.6080...@huawei.com Signed-off-by: Li Zefan Signed-off-by: Steven Rostedt --- kernel/trace/trace_syscalls.c | 10

[for-next][PATCH 1/5] tracing: Add __tracepoint_string() to export string pointers

2013-08-22 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" There are several tracepoints (mostly in RCU), that reference a string pointer and uses the print format of "%s" to display the string that exists in the kernel, instead of copying the actual string to the ring buffer (saves time and ring buffer space). But this

Re: [PATCH] tile: support ftrace on tilegx

2013-08-22 Thread Steven Rostedt
On Fri, 9 Aug 2013 13:26:09 -0400 Tony Lu wrote: > diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl > index 858966a..a674fd5 100755 > --- a/scripts/recordmcount.pl > +++ b/scripts/recordmcount.pl > @@ -364,6 +364,10 @@ if ($arch eq "x86_64") { > } elsif ($arch eq "blackfin") { >

[PATCH 1/4] ozwpan: staging: Fix crash for race condition.

2013-08-22 Thread Rupesh Gujare
Do not allocate a port to new device or process URB when its status is yet to be read. This avoids race condition when USB core read hub status a bit late, while new device tries to acquire port. Signed-off-by: Rupesh Gujare --- drivers/staging/ozwpan/ozhcd.c |5 +++-- 1 file changed, 3 inse

[PATCH 2/4] staging: ozwpan: Check error condition before creating endpoint.

2013-08-22 Thread Rupesh Gujare
Check if interface number is correct before creating an end point. Signed-off-by: Rupesh Gujare --- drivers/staging/ozwpan/ozhcd.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c index 4682d78..d5a3900 100644 --- a/drivers/

[PATCH 3/4] staging: ozwpan: Increment reference counter.

2013-08-22 Thread Rupesh Gujare
Increment PD reference counter, on every timer event so that we do not loose PD object by mistake. Signed-off-by: Rupesh Gujare --- drivers/staging/ozwpan/ozproto.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/staging/ozwpan/ozproto.c b/drivers/staging/ozwpa

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Wed, 2013-08-21 at 23:13 +0100, Alexandre Belloni wrote: > You are not so wrong. There is indeed actually only one reference > voltage (and that is 1.85V). But, before feeding the voltage to the ADC > channels, you sometimes have a divider. Then, after the channel muxing, > you can add a by 2 d

Re: [PATCH 10/13] tracing/uprobes: Fetch args before reserving a ring buffer

2013-08-22 Thread Steven Rostedt
On Fri, 09 Aug 2013 18:56:54 +0900 Masami Hiramatsu wrote: > (2013/08/09 17:45), Namhyung Kim wrote: > > From: Namhyung Kim > > > > Fetching from user space should be done in a non-atomic context. So > > use a temporary buffer and copy its content to the ring buffer > > atomically. > > > > Wh

Re: [PATCH 3/6] mm/hwpoison: fix num_poisoned_pages error statistics for thp

2013-08-22 Thread Naoya Horiguchi
On Thu, Aug 22, 2013 at 05:48:24PM +0800, Wanpeng Li wrote: > There is a race between hwpoison page and unpoison page, memory_failure > set the page hwpoison and increase num_poisoned_pages without hold page > lock, and one page count will be accounted against thp for num_poisoned_pages. > Howeve

[for-next][PATCH 3/5] tracing: Kill trace_create_file_ops() and friends

2013-08-22 Thread Steven Rostedt
From: Oleg Nesterov trace_create_file_ops() allocates the copy of id/filter/format/enable file_operations to set "f_op->owner = mod" for fops_get(). However after the recent changes there is no reason to prevent rmmod even if one of these files is opened. A file operation can do nothing but fail

[for-next][PATCH 5/5] tracing: Kill the !CONFIG_MODULES code in trace_events.c

2013-08-22 Thread Steven Rostedt
From: Oleg Nesterov Move trace_module_nb under CONFIG_MODULES and kill the dummy trace_module_notify(). Imho it doesn't make sense to define "struct notifier_block" and its .notifier_call just to avoid "ifdef" in event_trace_init(), and all other !CONFIG_MODULES code has already gone away. Link:

[for-next][PATCH 0/5] tracing: few updates

2013-08-22 Thread Steven Rostedt
I final have time to start looking at my INBOX. There's several patches that need to be pushed for 3.12. They are mostly clean ups and not anything urgent. I'm still in the process of going through my INBOX (backwards in time) so I may find new stuff as well. -- Steve git://git.kernel.org/pub/

[for-next][PATCH 4/5] tracing: Dont pass file_operations array to event_create_dir()

2013-08-22 Thread Steven Rostedt
From: Oleg Nesterov Now that event_create_dir() and __trace_add_new_event() always use the same file_operations we can kill these arguments and simplify the code. Link: http://lkml.kernel.org/r/20130731173135.ga31...@redhat.com Signed-off-by: Oleg Nesterov Signed-off-by: Steven Rostedt --- k

Re: [PATCH 1/1] ACPI / PNP: Fix incorrect placement of __initdata

2013-08-22 Thread Sachin Kamat
[+cc Russell] On 22 August 2013 21:45, Bjorn Helgaas wrote: > [+cc linux-kernel] > > On Thu, Aug 22, 2013 at 1:44 AM, Sachin Kamat wrote: >> __initdata should be placed between the variable name and equal >> sign for the variable to be placed in the intended section. > > I'm not sure this is tru

Re: [BUG REPORT]kernel panic with kmemcheck config

2013-08-22 Thread Dave Hansen
On 08/21/2013 08:58 PM, Libin wrote: > I test it on IBM System x3850 X5 platform, and also trigger oops in boot > process. But if don't config the kmemcheck, it can boot up normally. > Hardware information and oops information as following: > > [0.205976] BUG: unable to handle kernel paging re

Dear Beneficiary

2013-08-22 Thread The Weir Foundation
Dear Beneficiary My Wife Christine and I are donating £550,000 to you.Contact us via my wife email at (funds.christineweir.fundat...@gmail.com) for further details. Best Regards, Colin & Christine Weir• Copyright ©2013• The Weir Foundation• All Rights Reserved• -- To unsubscribe from this

[PATCH 4/4] staging: ozwpan: Create deferred work to destroy PD object.

2013-08-22 Thread Rupesh Gujare
Currently we call oz_pd_destroy() from softirq context, where we try to destroy relevant data structures, as well we kill a tasklet which always result in following kernel warning. [12279.262194] Attempt to kill tasklet from interrupt [12279.262202] Attempt to kill tasklet from interrupt This pat

[PATCH v6 2/3] mmc: dw_mmc: Honor requests to set the clock to 0 (turn off clock)

2013-08-22 Thread Doug Anderson
Previously the dw_mmc driver would ignore any requests to disable the card's clock. This doesn't seem like a good thing in general, but had one extra bad side effect in the following situtation: * mmc core would set clk to 400kHz at boot time while initting * mmc core would set clk to 0 since no c

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Thu, 2013-08-22 at 09:05 +0100, Hector Palacios wrote: > This was what I originally submitted but it then looked like it would better > fit in > the DeviceTree. The spear-adc seemed to use a similar approach: I'm not sure if it's the best example... I much more prefer the AD7303's way: Docum

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Thu, 2013-08-22 at 07:17 +0100, Jonathan Cameron wrote: > I would favour option 2 though some of the discussions going on at the moment > about > bindings might result in a generic description of this and any other bits of > analog front end. Any link to this discussion? devicet...@vger.kerne

Re: [PATCH] idr: Use this_cpu_ptr() for percpu_ida

2013-08-22 Thread Christoph Lameter
On Wed, 21 Aug 2013, Andrew Morton wrote: > The lock is needed so that one cpu can steal tags from another cpu's cache. > See (the needlessly inlined!) steal_tags(). Stealing tags could also be done via IPIs or some other things. -- To unsubscribe from this list: send the line "unsubscribe linux

Re: [RFC PATCH v2 3/4] powerpc: refactor of_get_cpu_node to support other architectures

2013-08-22 Thread Sudeep KarkadaNagesha
On 22/08/13 14:59, Mark Rutland wrote: > On Mon, Aug 19, 2013 at 02:56:10PM +0100, Sudeep KarkadaNagesha wrote: >> On 19/08/13 14:02, Rob Herring wrote: >>> On 08/19/2013 05:19 AM, Mark Rutland wrote: On Sat, Aug 17, 2013 at 11:09:36PM +0100, Benjamin Herrenschmidt wrote: > On Sat, 2013-08

Re: PATCH? fix unshare(NEWPID) && vfork()

2013-08-22 Thread Oleg Nesterov
On 08/21, Oleg Nesterov wrote: > > Still. Can't we make a single check? Like the initial patch I sent, but > this one moves the check into copy_process() and checks CLONE_* first. > Looks a bit simpler. And more understandable to me but this is subjective. Seriously. CLONE_NEWPID and task_active_p

Re: [PATCH 00/16] slab: overload struct slab over struct page to reduce memory usage

2013-08-22 Thread Christoph Lameter
On Thu, 22 Aug 2013, Joonsoo Kim wrote: > And this patchset change a management method of free objects of a slab. > Current free objects management method of the slab is weird, because > it touch random position of the array of kmem_bufctl_t when we try to > get free object. See following example.

Re: [PATCH v2 1/2] mbcache: decoupling the locking of local from global data

2013-08-22 Thread Linus Torvalds
On Thu, Aug 22, 2013 at 8:54 AM, T Makphaibulchoke wrote: > > +#defineMB_LOCK_HASH_CHAIN(hash_head) hlist_bl_lock(hash_head) > +#defineMB_UNLOCK_HASH_CHAIN(hash_head) hlist_bl_unlock(hash_head) > +#ifdef MB_CACHE_DEBUG > +#defineMB_LOCK_BLOCK_HASH(ce) do { Please don't d

Re: [PATCH] idr: Use this_cpu_ptr() for percpu_ida

2013-08-22 Thread Jens Axboe
On 08/22/2013 10:44 AM, Christoph Lameter wrote: > On Wed, 21 Aug 2013, Andrew Morton wrote: > >> The lock is needed so that one cpu can steal tags from another cpu's cache. >> See (the needlessly inlined!) steal_tags(). > > Stealing tags could also be done via IPIs or some other things. That is

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Lars-Peter Clausen
On 08/22/2013 06:41 PM, Pawel Moll wrote: > > On Wed, 2013-08-21 at 23:13 +0100, Alexandre Belloni wrote: >> You are not so wrong. There is indeed actually only one reference >> voltage (and that is 1.85V). But, before feeding the voltage to the ADC >> channels, you sometimes have a divider. Then,

Re: [PATCH 3/6] mm/hwpoison: fix num_poisoned_pages error statistics for thp

2013-08-22 Thread Naoya Horiguchi
On Thu, Aug 22, 2013 at 12:43:08PM -0400, Naoya Horiguchi wrote: > On Thu, Aug 22, 2013 at 05:48:24PM +0800, Wanpeng Li wrote: > > There is a race between hwpoison page and unpoison page, memory_failure > > set the page hwpoison and increase num_poisoned_pages without hold page > > lock, and one

[PATCH 1/2] tick: broadcast: Deny per-cpu clockevents from being broadcast sources

2013-08-22 Thread Stephen Boyd
On most ARM systems the per-cpu clockevents are truly per-cpu in the sense that they can't be controlled on any other CPU besides the CPU that they interrupt. If one of these clockevents were to become a broadcast source we will run into a lot of trouble because the broadcast source is enabled on t

Re: [RFC PATCH] mmc: Enable wakeup_sources for mmc core

2013-08-22 Thread Zoran Markovic
Ulf, > I got confirmation from Broadcom that all cell phone reference designs > have card insert/removal configured as a wakeup IRQ. Unless our > customers change that - which I doubt - this results in a considerable > number of products implementing this feature. > > Please let me know how you wis

[PATCH 2/2] clockevents: Prefer clockevents that don't suffer from FEAT_C3_STOP

2013-08-22 Thread Stephen Boyd
On some ARM systems there are two sets of per-cpu timers: the TWD timers and the global timers. The TWD timers are rated at 350 and the global timers are rated at 300 but the TWD timers suffer from FEAT_C3_STOP while the arm global timers don't. The tick device selection logic doesn't consider the

Re: [for-next][PATCH 1/5] tracing: Add __tracepoint_string() to export string pointers

2013-08-22 Thread Paul E. McKenney
On Thu, Aug 22, 2013 at 12:32:26PM -0400, Steven Rostedt wrote: > From: "Steven Rostedt (Red Hat)" > > There are several tracepoints (mostly in RCU), that reference a string > pointer and uses the print format of "%s" to display the string that > exists in the kernel, instead of copying the actua

Re: [RFC PATCH] pm: prevent suspend until power supply events are processed

2013-08-22 Thread Zoran Markovic
Any opinions on this patch? Regards, Zoran On 2 August 2013 13:38, Zoran Markovic wrote: > This patch, originally authored by Arve Hjonnevag and Todd Poynor, > prevents the system from entering suspend mode until the power > supply plug, unplug, or any other change of state event is fully > proce

[PATCH 0/3] namespaces && fork fixes/cleanups

2013-08-22 Thread Oleg Nesterov
Hello. The discussion was a bit confusing. I decided to split the changes to simplify the documentation/review. Eric, Andy, could you please ack/nack explicitly? I guess it is too late for 3.11, hopefully Andrew can take these patches if they pass the review. OTOH, 1/3 fixes the regression, so

Re: [RFC PATCHv3] drivers: power: Detect device suspend/resume lockup and log event in pstore.

2013-08-22 Thread Zoran Markovic
Rafael, I haven't seen any other proposals/alternatives on how to do this. Is there anything else we should do to get this upstream? I believe this is a valuable debug feature for Android and it now explicitly depends on pstore... Thanks, Zoran On 30 July 2013 13:38, Zoran Markovic wrote: > From:

[PATCH] vfio: Add O_CLOEXEC flag to vfio device fd

2013-08-22 Thread Alex Williamson
Add the default O_CLOEXEC flag for device file descriptors. This is generally considered a safer option as it allows the user a race free option to decide whether file descriptors are inherited across exec, with the default avoiding file descriptor leaks. Reported-by: Yann Droneaud Signed-off-by

[PATCH 2/3] pidns: kill the unnecessary CLONE_NEWPID in copy_process()

2013-08-22 Thread Oleg Nesterov
8382fcac "pidns: Outlaw thread creation after unshare(CLONE_NEWPID)" nacks CLONE_NEWPID if the forking process unshared pid_ns. This is correct but unnecessary, copy_pid_ns() does the same check. Test-case (needs root or CLONE_NEWUSER should be added): static int child(void *arg)

[PATCH 1/3] pidns: fix vfork() after unshare(CLONE_NEWPID)

2013-08-22 Thread Oleg Nesterov
8382fcac "pidns: Outlaw thread creation after unshare(CLONE_NEWPID)" nacks CLONE_VM if the forking process unshared pid_ns, this obviously breaks vfork: int main(void) { assert(unshare(CLONE_NEWUSER | CLONE_NEWPID) == 0); assert(vfork() >= 0);

[PATCH 3/3] fork: unify and tighten up CLONE_NEWUSER/CLONE_NEWPID checks

2013-08-22 Thread Oleg Nesterov
do_fork() denies CLONE_THREAD | CLONE_PARENT if NEWUSER | NEWPID. Then later copy_process() denies CLONE_SIGHAND if the new process will be in a different pid namespace (task_active_pid_ns() doesn't match current->nsproxy->pid_ns). This looks confusing and inconsistent. CLONE_NEWPID is very simil

Re: [RFC/RFT 00/33] Add new staging driver for RTL8188EU

2013-08-22 Thread Greg KH
On Wed, Aug 21, 2013 at 10:33:42PM -0500, Larry Finger wrote: > Initially, the only Linux driver for the RTL8188EU chips incorporated > in devices such as the TP-Link TL-WN725N was one sent to me privately > by the Realtek engineers with permission to submit it to the Linux kernel. > Not only did i

Re: [for-next][PATCH 1/5] tracing: Add __tracepoint_string() to export string pointers

2013-08-22 Thread Steven Rostedt
On Thu, 22 Aug 2013 10:06:23 -0700 "Paul E. McKenney" wrote: > On Thu, Aug 22, 2013 at 12:32:26PM -0400, Steven Rostedt wrote: > > From: "Steven Rostedt (Red Hat)" > > > > Cc: Paul E. McKenney > > Signed-off-by: Steven Rostedt > > Acked-by: Paul E. McKenney > Since you and I are both usi

Re: [RFC 02/17] ARM: call clk_of_init from time_init

2013-08-22 Thread Arnd Bergmann
On Thursday 22 August 2013, Mike Turquette wrote: > Some timers need to know their clock rate for the timer setup. This > could be hard-coded (e.g. for some timer driven from a fixed 32Khz > source) but it could also be from a clock capable of different rates > that needs to be calculated. > > I r

Re: [PATCH 3/3] thinkpad_acpi: Wire unused micmute LED to capslock

2013-08-22 Thread Jason A. Donenfeld
On Thu, Aug 22, 2013 at 5:39 PM, Matthew Garrett wrote: > I think there's a risk of user confusion here, in that it's now possible > for the mic mute light to be lit despite the internal microphone still > being active. That seems like surprising behaviour. Yeah, that is a legitimate concern. Pat

Re: [PATCH 3/4] arm: atmel: sama5: enable kernel uncompress info output

2013-08-22 Thread Jean-Christophe PLAGNIOL-VILLARD
On 15:16 Thu 04 Jul , Bo Shen wrote: > enable kernel uncompress info output, which will show as following: > ---8>--- > Uncompressing Linux... done, booting the kernel. > ---<8--- squash it with the precedent patch Best Regards, J, > > Signed-off-by: Bo Shen > --- > arch/arm/mach-at91/incl

Re: [3.10-rc1 PATCH] devtmpfs: Fix kmemcheck warning.

2013-08-22 Thread Dave Hansen
cc'ing Al and Kay who have the most commits in devtmpfs... On 05/14/2013 05:02 AM, Tetsuo Handa wrote: > I got below warning. > > WARNING: kmemcheck: Caught 8-bit read from uninitialized memory > (88007ae384d8) > d884e37a0088006f665f64657669 >i i i

Re: [PATCH 2/2] clockevents: Prefer clockevents that don't suffer from FEAT_C3_STOP

2013-08-22 Thread Santosh Shilimkar
On Thursday 22 August 2013 01:06 PM, Stephen Boyd wrote: > On some ARM systems there are two sets of per-cpu timers: the TWD > timers and the global timers. The TWD timers are rated at 350 and > the global timers are rated at 300 but the TWD timers suffer from > FEAT_C3_STOP while the arm global ti

Re: [PATCH 2/2] clockevents: Prefer clockevents that don't suffer from FEAT_C3_STOP

2013-08-22 Thread Stephen Boyd
On 08/22/13 10:33, Santosh Shilimkar wrote: > On Thursday 22 August 2013 01:06 PM, Stephen Boyd wrote: >> On some ARM systems there are two sets of per-cpu timers: the TWD >> timers and the global timers. The TWD timers are rated at 350 and >> the global timers are rated at 300 but the TWD timers s

Re: [for-next][PATCH 1/5] tracing: Add __tracepoint_string() to export string pointers

2013-08-22 Thread Paul E. McKenney
On Thu, Aug 22, 2013 at 01:27:53PM -0400, Steven Rostedt wrote: > On Thu, 22 Aug 2013 10:06:23 -0700 > "Paul E. McKenney" wrote: > > > On Thu, Aug 22, 2013 at 12:32:26PM -0400, Steven Rostedt wrote: > > > From: "Steven Rostedt (Red Hat)" > > > > > > > Cc: Paul E. McKenney > > > Signed-off-by:

Re: 3.9.0 dmesg reports that my NIC is hanging

2013-08-22 Thread Bjorn Helgaas
On Tue, May 7, 2013 at 3:07 PM, John wrote: > > > > > - Original Message - >> From: Bjorn Helgaas >> To: John >> Cc: lkml ; Jeff Kirsher >> ; "e1000-de...@lists.sourceforge.net" >> >> Sent: Monday, May 6, 2013 1:29 PM >> Subject: Re: 3.9.0 dmesg reports that my NIC is hanging >> >> [+

Re: [PATCH 4/4] staging: ozwpan: Create deferred work to destroy PD object.

2013-08-22 Thread Sergei Shtylyov
Hello. On 08/22/2013 08:38 PM, Rupesh Gujare wrote: Currently we call oz_pd_destroy() from softirq context, where we try to destroy relevant data structures, as well we kill a tasklet which always result in following kernel warning. [12279.262194] Attempt to kill tasklet from interrupt [1227

Re: [PATCH 2/2] clockevents: Prefer clockevents that don't suffer from FEAT_C3_STOP

2013-08-22 Thread Santosh Shilimkar
On Thursday 22 August 2013 01:40 PM, Stephen Boyd wrote: > On 08/22/13 10:33, Santosh Shilimkar wrote: >> On Thursday 22 August 2013 01:06 PM, Stephen Boyd wrote: >>> On some ARM systems there are two sets of per-cpu timers: the TWD >>> timers and the global timers. The TWD timers are rated at 350

Re: [PATCH 02/16] slab: change return type of kmem_getpages() to struct page

2013-08-22 Thread Christoph Lameter
On Thu, 22 Aug 2013, Joonsoo Kim wrote: > @@ -2042,7 +2042,7 @@ static void slab_destroy_debugcheck(struct kmem_cache > *cachep, struct slab *slab > */ > static void slab_destroy(struct kmem_cache *cachep, struct slab *slabp) > { > - void *addr = slabp->s_mem - slabp->colouroff; > + s

Re: [PATCH v3 0/3] HID: Win 8 multitouch panels detection in core

2013-08-22 Thread Henrik Rydberg
Hi Benjamin, thanks for the patches, it all looks fine now. > this is the v3 of this patch series. > > Srinivas, I still did not added your tested-by as I made minors modifications > of > the patches 1 and 2. Now only the patch 1 will impact sensor_hub, and patch 2 > will not impact this. Maybe

Re: [PATCH 2/2] w1: use default attribute groups for w1 slave devices

2013-08-22 Thread Greg Kroah-Hartman
On Thu, Aug 22, 2013 at 04:02:39AM +0400, Evgeniy Polyakov wrote: > hi Greg > > 22.08.2013, 00:24, "Greg Kroah-Hartman" : > > As we have 2 sysfs files for the w1 slave devices, let the driver core > > create / destroy them automatically by setting the default attribute > > group for them, saving c

Re: [PATCH 6/6] MIPS: ath79: simplify platform_get_resource_byname/devm_ioremap_resource

2013-08-22 Thread Gabor Juhos
2013.08.19. 10:51 keltezéssel, Julia Lawall írta: > From: Julia Lawall > > Remove unneeded error handling on the result of a call to > platform_get_resource_byname when the value is passed to > devm_ioremap_resource. > > A simplified version of the semantic patch that makes this change is as >

Re: [PATCH 05/16] slab: remove cachep in struct slab_rcu

2013-08-22 Thread Christoph Lameter
On Thu, 22 Aug 2013, Joonsoo Kim wrote: > We can get cachep using page in struct slab_rcu, so remove it. Ok but this means that we need to touch struct page. Additional cacheline in cache footprint. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a messag

Re: [PATCH v3 08/12] perf, persistent: Exposing persistent events using sysfs

2013-08-22 Thread Vince Weaver
On Thu, 22 Aug 2013, Robert Richter wrote: > From: Robert Richter > > Expose persistent events in the system to userland using sysfs. Perf > tools are able to read existing pmu events from sysfs. Now we use a > persistent pmu as an event container containing all registered > persistent events of

Re: [PATCH 1/3] pidns: fix vfork() after unshare(CLONE_NEWPID)

2013-08-22 Thread Andy Lutomirski
On Thu, Aug 22, 2013 at 10:09 AM, Oleg Nesterov wrote: > 8382fcac "pidns: Outlaw thread creation after unshare(CLONE_NEWPID)" > nacks CLONE_VM if the forking process unshared pid_ns, this obviously > breaks vfork: > > int main(void) > { > assert(unshare(CLONE_NEWUSE

Re: [PATCH 2/3] pidns: kill the unnecessary CLONE_NEWPID in copy_process()

2013-08-22 Thread Andy Lutomirski
On Thu, Aug 22, 2013 at 10:10 AM, Oleg Nesterov wrote: > 8382fcac "pidns: Outlaw thread creation after unshare(CLONE_NEWPID)" > nacks CLONE_NEWPID if the forking process unshared pid_ns. This is > correct but unnecessary, copy_pid_ns() does the same check. > > Test-case (needs root or CLONE_NEWUSE

Re: [PATCH 01/11] w1: add attribute groups to struct w1_family_ops

2013-08-22 Thread Greg Kroah-Hartman
On Thu, Aug 22, 2013 at 04:05:04AM +0400, Рустафа Джамурахметов wrote: > Hi > > 22.08.2013, 02:45, "Greg Kroah-Hartman" : > > This lets w1 slave drivers declare an attribute group, and not have to > > create/destroy sysfs files directly.  All w1 slave drivers will be fixed > > to use this field up

Re: [PATCH 04/16] slab: remove nodeid in struct slab

2013-08-22 Thread Christoph Lameter
On Thu, 22 Aug 2013, Joonsoo Kim wrote: > @@ -1099,8 +1098,7 @@ static void drain_alien_cache(struct kmem_cache *cachep, > > static inline int cache_free_alien(struct kmem_cache *cachep, void *objp) > { > - struct slab *slabp = virt_to_slab(objp); > - int nodeid = slabp->nodeid; > +

Re: [PATCH 3/3] fork: unify and tighten up CLONE_NEWUSER/CLONE_NEWPID checks

2013-08-22 Thread Andy Lutomirski
On Thu, Aug 22, 2013 at 10:10 AM, Oleg Nesterov wrote: > do_fork() denies CLONE_THREAD | CLONE_PARENT if NEWUSER | NEWPID. > > Then later copy_process() denies CLONE_SIGHAND if the new process > will be in a different pid namespace (task_active_pid_ns() doesn't > match current->nsproxy->pid_ns). >

Re: [PATCH v3 12/12] [RFC] perf, persistent: ioctl functions to control persistency

2013-08-22 Thread Vince Weaver
On Thu, 22 Aug 2013, Robert Richter wrote: > From: Robert Richter > > Implementing ioctl functions to control persistent events. There are > functions to detach or attach an event to or from a process. The > PERF_EVENT_IOC_DETACH ioctl call makes an event persistent. After > closing the event's

Re: [PATCH 09/16] slab: use __GFP_COMP flag for allocating slab pages

2013-08-22 Thread Christoph Lameter
On Thu, 22 Aug 2013, Joonsoo Kim wrote: > If we use 'struct page' of first page as 'struct slab', there is no > advantage not to use __GFP_COMP. So use __GFP_COMP flag for all the cases. Ok that brings it in line with SLUB and SLOB. > @@ -2717,17 +2701,8 @@ static void slab_put_obj(struct kmem_c

[ANNOUNCE] 3.10.9-rt5

2013-08-22 Thread Sebastian Andrzej Siewior
Dear RT folks! I'm pleased to announce the v3.10.9-rt5 patch set. Changes since v3.10.9-rt4 - swait fixes from Steven. It fixed the issues with CONFIG_RCU_NOCB_CPU where the system suddenly froze and RCU wasn't doing its job anymore - hwlat improvements by Steven Known issues: - SLAB su

Re: [PATCH 3/3] fork: unify and tighten up CLONE_NEWUSER/CLONE_NEWPID checks

2013-08-22 Thread Oleg Nesterov
On 08/22, Andy Lutomirski wrote: > > On Thu, Aug 22, 2013 at 10:10 AM, Oleg Nesterov wrote: > > do_fork() denies CLONE_THREAD | CLONE_PARENT if NEWUSER | NEWPID. > > > > Then later copy_process() denies CLONE_SIGHAND if the new process > > will be in a different pid namespace (task_active_pid_ns()

Re: [ANNOUNCE] 3.10.9-rt5

2013-08-22 Thread Sebastian Andrzej Siewior
* Sebastian Andrzej Siewior | 2013-08-22 20:21:27 [+0200]: >Changes since v3.10.9-rt4 >- swait fixes from Steven. It fixed the issues with CONFIG_RCU_NOCB_CPU > where the system suddenly froze and RCU wasn't doing its job Also a big thanks to Carsten Emde and Paul E. McKenney for finding and de

Asus F5RL laptop unable to resume from S3 because of radeon module

2013-08-22 Thread Ondrej Zary
Hello, resume from suspend-to-RAM (S3) on Asus F5RL laptop does not work. According to many reports found by Google, it was always been that and there is no fix or workaround. Did some tests: radeon module loaded (usual state): After "echo mem>/sys/power/state", the laptop suspends correctly (pow

Re: [PATCH 1/3] pidns: fix vfork() after unshare(CLONE_NEWPID)

2013-08-22 Thread Oleg Nesterov
On 08/22, Andy Lutomirski wrote: > > On Thu, Aug 22, 2013 at 10:09 AM, Oleg Nesterov wrote: > > 8382fcac "pidns: Outlaw thread creation after unshare(CLONE_NEWPID)" > > nacks CLONE_VM if the forking process unshared pid_ns, this obviously > > breaks vfork: > > > > int main(void) > >

Re: pull request: bluetooth-next 2013-08-21

2013-08-22 Thread John W. Linville
On Wed, Aug 21, 2013 at 06:12:18PM +0200, Gustavo Padovan wrote: > Hi John, > > Patches to 3.12, here we have: > > * implementation of a proper tty_port for RFCOMM devices, this fixes some > issues people were seeing lately in the kernel. > * Add voice_setting option for SCO, it is used for SCO C

Re: [PATCH 3/3] fork: unify and tighten up CLONE_NEWUSER/CLONE_NEWPID checks

2013-08-22 Thread Andy Lutomirski
On Thu, Aug 22, 2013 at 11:15 AM, Oleg Nesterov wrote: > On 08/22, Andy Lutomirski wrote: >> >> On Thu, Aug 22, 2013 at 10:10 AM, Oleg Nesterov wrote: >> > do_fork() denies CLONE_THREAD | CLONE_PARENT if NEWUSER | NEWPID. >> > >> > Then later copy_process() denies CLONE_SIGHAND if the new process

Re: [PATCH 2/2] clockevents: Prefer clockevents that don't suffer from FEAT_C3_STOP

2013-08-22 Thread Stephen Boyd
On 08/22, Santosh Shilimkar wrote: > On Thursday 22 August 2013 01:40 PM, Stephen Boyd wrote: > > On 08/22/13 10:33, Santosh Shilimkar wrote: > >> On Thursday 22 August 2013 01:06 PM, Stephen Boyd wrote: > >>> On some ARM systems there are two sets of per-cpu timers: the TWD > >>> timers and the gl

Re: [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier.

2013-08-22 Thread Tejun Heo
Hello, On Thu, Aug 22, 2013 at 09:52:09AM -0600, Toshi Kani wrote: > I understand that you are concerned about stability of the ACPI stuff, > which I think is a valid point, but most of (if not all) of the > ACPI-related issues come from ACPI namespace/methods, which is a very > different thing.

[RESEND/PATCH] acpi: video: remove unused code

2013-08-22 Thread Felipe Contreras
_BCM_use_index and _BCL_use_index are never used and probably never will. Signed-off-by: Felipe Contreras --- drivers/acpi/video.c | 12 1 file changed, 12 deletions(-) diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c index e1284b8..e6be27e 100644 --- a/drivers/acpi/video.c

[PATCH 0/2] Rename nsproxy.pid_ns and fix a related security bug

2013-08-22 Thread Andy Lutomirski
In: commit 92f28d973cce45ef5823209aab3138eb45d8b349 Author: Eric W. Biederman Date: Fri Mar 15 01:03:33 2013 -0700 scm: Require CAP_SYS_ADMIN over the current pidns to spoof pids. Eric fell for my bogus claim that nsproxy->pid_ns was the current' process's pid ns. This is

[PATCH 1/2] net: Check the correct namespace when spoofing pid over SCM_RIGHTS

2013-08-22 Thread Andy Lutomirski
This is a security bug. The follow-up will fix nsproxy to discourage this type of issue from happening again. Cc: sta...@vger.kernel.org Signed-off-by: Andy Lutomirski --- net/core/scm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/core/scm.c b/net/core/scm.c index 03

Re: [PATCH 3/3] fork: unify and tighten up CLONE_NEWUSER/CLONE_NEWPID checks

2013-08-22 Thread Oleg Nesterov
On 08/22, Andy Lutomirski wrote: > > On Thu, Aug 22, 2013 at 11:15 AM, Oleg Nesterov wrote: > > > > Yes. Before this patch do_fork() did: > > > > if (clone_flags & (CLONE_NEWUSER | CLONE_NEWPID)) { > > if (clone_flags & (CLONE_THREAD|CLONE_PARENT)) > >

Re: [GIT PULL] at91: device tree update for 3.12 #1

2013-08-22 Thread Kevin Hilman
Nicolas Ferre writes: > Arnd, Olof, Kevin, > > Here is the first pull request about DT on AT91 for 3.12. > It is based on 3.11-rc5 as I saw that the branch is coming from this tag on > your side. > > Thanks, best regards, > > The following changes since commit d4e4ab86bcba5a72779c43dc1459f71fea3d

[PATCH 2/2] Rename nsproxy.pid_ns to nsproxy.pid_ns_for_children

2013-08-22 Thread Andy Lutomirski
nsproxy.pid_ns is *not* the task's pid namespace. The name should clarify that. This makes it more obvious that setns on a pid namespace is weird -- it won't change the pid namespace shown in procfs. Signed-off-by: Andy Lutomirski --- include/linux/nsproxy.h | 6 +- kernel/fork.c

Re: [PATCH] irqchip: gic: Don't complain in gic_get_cpumask() if UP system

2013-08-22 Thread Stephen Boyd
On 07/17, Stephen Boyd wrote: > On 07/17/13 15:53, Nicolas Pitre wrote: > > On Wed, 17 Jul 2013, Stephen Boyd wrote: > > > >> On 07/17/13 15:34, Nicolas Pitre wrote: > >>> On Wed, 17 Jul 2013, Stephen Boyd wrote: > >>> > On 07/12/13 05:10, Stephen Boyd wrote: > > On 07/12, Javi Merino wrot

Re: [PATCH v2] vfs: Tighten up linkat(..., AT_EMPTY_PATH)

2013-08-22 Thread Linus Torvalds
On Wed, Aug 21, 2013 at 12:14 PM, Andy Lutomirski wrote: > > So let's be careful for now: only allow linkat(..., AT_EMPTY_PATH) > if the target is I_LINKABLE. So I really detest this just because it's such a special case. Now this is only useful for that one special case, and the thing very funda

Re: [ANNOUNCE] 3.10.9-rt5

2013-08-22 Thread Steven Rostedt
On Thu, 22 Aug 2013 20:21:27 +0200 Sebastian Andrzej Siewior wrote: > Known issues: > > - SLAB support not working > > - The cpsw network driver shows some issues. > > - bcache does not compile. For now, it may be prudent to just make bcache depend on !PREEMPT_RT. I can sen

Re: [PATCH v2] vfs: Tighten up linkat(..., AT_EMPTY_PATH)

2013-08-22 Thread Willy Tarreau
On Thu, Aug 22, 2013 at 11:48:10AM -0700, Linus Torvalds wrote: > On Wed, Aug 21, 2013 at 12:14 PM, Andy Lutomirski wrote: > > > > So let's be careful for now: only allow linkat(..., AT_EMPTY_PATH) > > if the target is I_LINKABLE. > > So I really detest this just because it's such a special case.

Re: [PATCH 2/2] Rename nsproxy.pid_ns to nsproxy.pid_ns_for_children

2013-08-22 Thread Oleg Nesterov
On 08/22, Andy Lutomirski wrote: > > nsproxy.pid_ns is *not* the task's pid namespace. The name should clarify > that. > > This makes it more obvious that setns on a pid namespace is weird -- > it won't change the pid namespace shown in procfs. > > ... > > + * The pid namespace is an exception --

Re: [PATCH v2] vfs: Tighten up linkat(..., AT_EMPTY_PATH)

2013-08-22 Thread Andy Lutomirski
On Thu, Aug 22, 2013 at 11:53 AM, Willy Tarreau wrote: > On Thu, Aug 22, 2013 at 11:48:10AM -0700, Linus Torvalds wrote: >> On Wed, Aug 21, 2013 at 12:14 PM, Andy Lutomirski >> wrote: >> > >> > So let's be careful for now: only allow linkat(..., AT_EMPTY_PATH) >> > if the target is I_LINKABLE. >

Re: [PATCH 4/6] mm/hwpoison: don't set migration type twice to avoid hold heavy contend zone->lock

2013-08-22 Thread Naoya Horiguchi
On Thu, Aug 22, 2013 at 05:48:25PM +0800, Wanpeng Li wrote: > Set pageblock migration type will hold zone->lock which is heavy contended > in system to avoid race. However, soft offline page will set pageblock > migration type twice during get page if the page is in used, not hugetlbfs > page an

Re: [PATCH] irqchip: gic: Don't complain in gic_get_cpumask() if UP system

2013-08-22 Thread Nicolas Pitre
On Thu, 22 Aug 2013, Stephen Boyd wrote: > On 07/17, Stephen Boyd wrote: > > On 07/17/13 15:53, Nicolas Pitre wrote: > > > On Wed, 17 Jul 2013, Stephen Boyd wrote: > > > > > >> On 07/17/13 15:34, Nicolas Pitre wrote: > > >>> On Wed, 17 Jul 2013, Stephen Boyd wrote: > > >>> > > On 07/12/13 05:

[PATCH] usb: acm gadget: Null termintate strings table

2013-08-22 Thread Graham Williams
The gadget strings table should be null terminated. usb_gadget_get_string() loops through the table expecting a null at the end of the list. Signed-off-by: Graham --- drivers/usb/gadget/f_acm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/gadget/f_acm.c b/drivers/usb/gadget/f_ac

Re: [PATCH] xfs: Register hotcpu notifier after initialization

2013-08-22 Thread Ben Myers
On Mon, Aug 19, 2013 at 05:21:29PM -0500, Ben Myers wrote: > On Mon, Aug 19, 2013 at 10:56:44PM +0200, Richard Weinberger wrote: > > Currently the code initializizes mp->m_icsb_mutex and other things > > _after_ register_hotcpu_notifier(). > > As the notifier takes mp->m_icsb_mutex it can happen >

Re: [PATCH v5 1/1] ASoc: kirkwood: add DT support to the mvebu audio subsystem

2013-08-22 Thread Mark Brown
On Wed, Aug 14, 2013 at 12:27:33PM +0200, Jean-Francois Moine wrote: > This patch adds DT support to the audio subsystem of the mvebu family > (Kirkwood, Dove, Armada 370). Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 3/3] fork: unify and tighten up CLONE_NEWUSER/CLONE_NEWPID checks

2013-08-22 Thread Andy Lutomirski
On Thu, Aug 22, 2013 at 11:32 AM, Oleg Nesterov wrote: > On 08/22, Andy Lutomirski wrote: >> >> On Thu, Aug 22, 2013 at 11:15 AM, Oleg Nesterov wrote: >> > >> > Yes. Before this patch do_fork() did: >> > >> > if (clone_flags & (CLONE_NEWUSER | CLONE_NEWPID)) { >> > if (clo

Re: [PATCH 11/12] pid: rewrite task helper functions avoiding task->pid and task->tgid

2013-08-22 Thread Oleg Nesterov
On 08/20, Richard Guy Briggs wrote: > > static inline int is_global_init(struct task_struct *tsk) > { > - return tsk->pid == 1; > + return task_pid_nr(tsk) == 1; > } Probably it would be better to simply kill it. Almost every usage is wrong. > static inline bool is_idle_task(const str

[PATCH] hwpoison: always unset MIGRATE_ISOLATE before returning from soft_offline_page()

2013-08-22 Thread Naoya Horiguchi
Soft offline code expects that MIGRATE_ISOLATE is set on the target page only during soft offlining work. But currenly it doesn't work as expected when get_any_page() fails and returns negative value. In the result, end users can have unexpectedly isolated pages. This patch just fixes it. Signed-o

Re: [PATCH v2] vfs: Tighten up linkat(..., AT_EMPTY_PATH)

2013-08-22 Thread Linus Torvalds
On Thu, Aug 22, 2013 at 12:05 PM, Andy Lutomirski wrote: > > Does this work for the procfs case? As far as I understand it (which > isn't saying much), it goes through the symlink-following path. Right. The /proc case is still separate, and we really should do something about that too. But again

<    1   2   3   4   5   6   7   >