Re: driver model, duplicate names question

2013-07-16 Thread Srinivas Pandruvada
Thanks for the quick response. Here I am creating virtual devices using device_register. I have attached a simple test program, which will give error. This is my intention: $> cd /sys/class/test_class $> ls power_zone_cpu_package_0 power_zone_cpu_package_1 $> cd power_zone_cpu_package_0 $>

Re: [BUG] 3.10 regression: hang on suspend

2013-07-16 Thread Ortwin Glück
Without NetworkManager, no X, on console and with plain jane wpa_supplicant I do echo mem > /sys/power state After that, it still responds to keyboard events: I can switch VT and type on the consoles, but I can not login on a different VT (pressing Enter after the username doesn't return). So

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Aaro Koskinen
Hi, On Tue, Jul 16, 2013 at 11:22:47AM +0300, Felipe Balbi wrote: > > On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: > > > On Tuesday 16 July 2013 12:03 PM, Daniel Mack wrote: > > > > Hi Aaro, > > > > > > > > On 16.07.2013 00:56, Aaro Koskinen wrote: > > > >> Hi, > > >

Re: [Ksummit-2013-discuss] [ATTEND] How to act on LKML (was: [ 00/19] 3.10.1-stable review)

2013-07-16 Thread James Bottomley
On Mon, 2013-07-15 at 15:38 -0700, Linus Torvalds wrote: > On Mon, Jul 15, 2013 at 3:08 PM, Steven Rostedt wrote: > > > > Can we please make this into a Kernel Summit discussion. I highly doubt > > we would solve anything, but it certainly would be a fun segment to > > watch :-) > > I think we

Re: [PATCH] Thermal: Fix lockup of cpu_down()

2013-07-16 Thread Srinivas Pandruvada
On 07/16/2013 11:02 AM, Steven Rostedt wrote: Commit f1a18a105 "Thermal: CPU Package temperature thermal" had code that did a get_online_cpus(), run a loop and then do a put_online_cpus(). The problem is that the loop had an error exit that would skip the put_online_cpus() part. In the error

Re: [Regression][v3.10][v3.11] net: Kill link between CSUM and SG features.

2013-07-16 Thread Pravin Shelar
On Tue, Jul 16, 2013 at 11:09 AM, Joseph Salisbury wrote: > Hi Pravin, > > A bug was opened against the Ubuntu kernel[0]. After a kernel bisect, > it was found that reverting the following commit resolved this bug: > > commit ec5f061564238892005257c83565a0b58ec79295 > Author: Pravin B Shelar >

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Linus Torvalds
On Tue, Jul 16, 2013 at 10:58 AM, Luck, Tony wrote: > > Linux testing is (realistically) done by inflicting changes on gradually wider > sets of end users. However, one thing that people should keep in mind that the testing is often self-selecting. This is particularly true for "obvious fixes".

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Daniel Mack
On 16.07.2013 20:26, Aaro Koskinen wrote: > Hi, > > On Tue, Jul 16, 2013 at 11:22:47AM +0300, Felipe Balbi wrote: >>> On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: On Tuesday 16 July 2013 12:03 PM, Daniel Mack wrote: > Hi Aaro, > > On 16.07.2013 00:56,

[RFC] kbuild: Build samples directory with make modules

2013-07-16 Thread Jim Baxter
If CONFIG_SAMPLES is enabled with any of the sample modules enabled, they are only built if the make command includes uImage and modules (i.e. make uImage modules). This patches allows the kernel and modules to be built separately, this is how openembedded and yocto build the kernel.

Re: driver model, duplicate names question

2013-07-16 Thread Greg KH
On Tue, Jul 16, 2013 at 11:29:42AM -0700, Srinivas Pandruvada wrote: > Thanks for the quick response. Here I am creating virtual devices using > device_register. > I have attached a simple test program, which will give error. > > This is my intention: > > $> cd /sys/class/test_class > $> ls >

Re: [PATCH] Thermal: Fix lockup of cpu_down()

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 11:19 -0700, Srinivas Pandruvada wrote: > Thanks. How did you trigger this error condition? Is it a code review or > you have some way to reproduce? No, my tests do a cpu hotplug stress and the system would hang. I had to bisect it to find the bug and it came to this code.

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Aaro Koskinen
On Tue, Jul 16, 2013 at 08:30:21PM +0200, Daniel Mack wrote: > On 16.07.2013 20:26, Aaro Koskinen wrote: > > Hi, > > > > On Tue, Jul 16, 2013 at 11:22:47AM +0300, Felipe Balbi wrote: > >>> On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: > On Tuesday 16 July 2013 12:03

[PATCH v5] x86: make sure IDT is page aligned

2013-07-16 Thread Kees Cook
Since the IDT is referenced from a fixmap, make sure it is page aligned. Merge with 32-bit one, since it was already aligned to deal with F00F bug. Since bss is cleared before IDT setup, it can live there. This also moves the other *_idt_table variables into common locations. This avoids the risk

RE: DWC2 and/or S3C-HSOTG for STA2X11 board

2013-07-16 Thread Paul Zimmerman
> From: Federico Vaga [mailto:federico.v...@gmail.com] > Sent: Tuesday, July 16, 2013 5:02 AM > > I have an x86 board made by STMicroelectronics (STA2X11) with the Synopsis > USB-OTG DesignWare 2 on it and connected through the PCI-e bus. > > I know that there are two drivers for the same

Re: [PATCH] Thermal: Fix lockup of cpu_down()

2013-07-16 Thread Srinivas Pandruvada
On 07/16/2013 11:33 AM, Steven Rostedt wrote: On Tue, 2013-07-16 at 11:19 -0700, Srinivas Pandruvada wrote: Thanks. How did you trigger this error condition? Is it a code review or you have some way to reproduce? No, my tests do a cpu hotplug stress and the system would hang. I had to bisect

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

2013-07-16 Thread Raghavendra K T
On 07/16/2013 11:32 AM, Gleb Natapov wrote: On Tue, Jul 16, 2013 at 09:07:53AM +0530, Raghavendra K T wrote: On 07/15/2013 04:06 PM, Gleb Natapov wrote: On Mon, Jul 15, 2013 at 03:20:06PM +0530, Raghavendra K T wrote: On 07/14/2013 06:42 PM, Gleb Natapov wrote: On Mon, Jun 24, 2013 at

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Willy Tarreau
Hi Takashi, On Tue, Jul 16, 2013 at 06:40:39PM +0200, Takashi Iwai wrote: > IMO, one of the reasons is the nature of stable-release: the stable > tree is released soon after reviews of patches, so no actual > regression tests can be done before the release. > > For finding a regression, patch

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread H. Peter Anvin
On 07/16/2013 12:19 AM, David Lang wrote: > On Fri, 12 Jul 2013, Willy Tarreau wrote: > >> And maybe in the end, having 1/10 patch cause a regression is not *that* >> dramatic, and probably less than not fixing the 9 other bugs. In one case >> we rely on -stable to merge the 10 fixes, and on the

Re: [RFC][PATCH 0/9] sched: Power scheduler design proposal

2013-07-16 Thread Arjan van de Ven
On 7/16/2013 10:38 AM, Peter Zijlstra wrote: On Mon, Jul 15, 2013 at 03:52:32PM -0700, Arjan van de Ven wrote: yeah ondemand does this, but ondemand is actually a pretty bad governor. not because of the sampling, but because of its algorithm. Is it good for any class of hardware still out

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

2013-07-16 Thread Raghavendra K T
On 07/16/2013 09:18 PM, Peter Zijlstra wrote: On Tue, Jul 16, 2013 at 09:02:15AM +0300, Gleb Natapov wrote: BTW can NMI handler take spinlocks? No -- that is, yes you can using trylock, but you still shouldn't. Thanks Peter for the clarification. I had started checking few of nmi handlers

Re: [PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-07-16 Thread Alan Stern
On Tue, 16 Jul 2013, boris brezillon wrote: > >> + uclk = clk_get(>dev, "usb_clk"); > >> + if (IS_ERR(uclk)) { > >> + uclk = NULL; > >> + dev_warn(>dev, "failed to get usb_clk\n"); > >> + } > > Is this really what you want for backward compatibility? > Here are some

Re: driver model, duplicate names question

2013-07-16 Thread Srinivas Pandruvada
Hi, I am assigned to do add a powercap class. There are several technologies, which will allow to add a power budget to an individual device. For example, you can set a power budget to a individual physical cpu package, each core and uncore devices, GPUs, DRAM etc. +The Power Capping

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 11:29 -0700, Linus Torvalds wrote: > Anyway, the point I'm making is that Q is limited and often even > actively misleading ("Hey, I have three tested-by's, so it must be > fine"), and we might actually want to have a new class of > "non-critical patch that might be worth

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Willy Tarreau
On Tue, Jul 16, 2013 at 11:29:15AM -0700, Linus Torvalds wrote: > There have been tons of obvious patches that turned out to simply be > wrong - often for very non-obvious reasons. Even when they are small. > And the problems seldom get caught in early testing, often exactly > because of this

Re: [PATCH v5] x86: make sure IDT is page aligned

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 11:34 -0700, Kees Cook wrote: > Since the IDT is referenced from a fixmap, make sure it is page aligned. > Merge with 32-bit one, since it was already aligned to deal with F00F > bug. Since bss is cleared before IDT setup, it can live there. This also > moves the other

[RFC PATCH 4/4] tracing: Change ftrace_event_filter_fops to rely on event_mutex/i_private

2013-07-16 Thread Oleg Nesterov
1. Shift mutex_lock(event_mutex) from print/apply_event_filter to the callers. And rename them just in case. 2. Change the callers, event_filter_read() and event_filter_write() to read call = i_private under this mutex and abort if it is NULL. 3. Remove .open from ftrace_event_filter_fops.

[RFC PATCH 3/4] tracing: Kill tracing_open/release_generic_file

2013-07-16 Thread Oleg Nesterov
Kill tracing_open_generic_file() and tracing_release_generic_file(), they are racy anyway. Instead, change event_enable_read() and event_enable_write() to rely on event_mutex and file_inode(filp)->i_private != NULL check. trace_array_get() goes away. NOTE! this is actually wrong until we change

[RFC PATCH 0/4] tracing: fix open/delete fixes

2013-07-16 Thread Oleg Nesterov
Hello. Completely untested and _incomplete_. This ignores instance_delete() and ftrace_event_format_fops, at least. But I am not going to even try to finish this series unless you tell me that you agree with this approach. I have no idea what else could I miss. I probably understand no more

[RFC PATCH 2/4] tracing: Turn "id"->i_private into call->event.type

2013-07-16 Thread Oleg Nesterov
ftrace_event_id_fops and event_id_read() is overcomplicated. 1. Change event_create_dir() to pass "data = call->event.type" to debugfs_create_file(). This means that ftrace_event_id_fops doesn't need .open() and event_id_read() can simply print (int)i_private 2. event_id_read() has no

[RFC PATCH 1/4] tracing: Change remove_event_from_tracers() to clear d_subdirs->i_private

2013-07-16 Thread Oleg Nesterov
Preparation to make the next patches more understandable. The caller of trace_remove_event_call() is going to free call/files, this means that every opened id/filter/enable/format file will use the already freed memory via filp->private_data / inode->i_private. Change remove_event_from_tracers()

Re: [PATCH 1/8] mm: drop actor argument of do_generic_file_read()

2013-07-16 Thread Matthew Wilcox
On Mon, Jul 15, 2013 at 01:47:47PM +0300, Kirill A. Shutemov wrote: > From: "Kirill A. Shutemov" > > There's only one caller of do_generic_file_read() and the only actor is > file_read_actor(). No reason to have a callback parameter. > > Signed-off-by: Kirill A. Shutemov > Acked-by: Dave

Re: driver model, duplicate names question

2013-07-16 Thread Greg KH
A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Tue, Jul 16, 2013 at 11:54:31AM -0700, Srinivas Pandruvada wrote: > Hi, > > I am assigned to do add a powercap class. There are several > technologies, which will allow to add a power budget to

Re: [PATCH] um/configs: don't use devtmpfs in defconfig

2013-07-16 Thread Al Viro
On Tue, Jul 16, 2013 at 11:42:55PM +0530, Ramkumar Ramachandra wrote: > Leave aside the fact that I could not find the uml-utils upstream [1], > and didn't have a /uml/port-helper to connect the xterms for a second; > I didn't even understand what was supposed to happen. Why do we spawn > xterms,

configurable partial slab support on UP

2013-07-16 Thread Uwe Kleine-König
Hello, (sorry for breaking the mail threading, but I failed to find the right mail to reply to.) while doing make oldconfig on an !SMP config using 3.11-rc1 I was asked if I want CONFIG_SLUB_CPU_PARTIAL. AFAICT this setting only affects SMP machines? If so I suggest to make the choice depend on

Re: [RFC][PATCH 0/4] tracing/kprobes/uprobes: Fix race between opening probe event files and deleting probe

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 18:38 +0200, Oleg Nesterov wrote: > On 07/15, Oleg Nesterov wrote: > > > > So. As Masami pointed out, this is not enough. Probably we can add more > > hacks, but I'd like to discuss the alternative approach. > > > > Note also that this ref count has the unfortunate property,

Re: [PATCH 3/7] alpha: Modernize lib/mpi/longlong.h

2013-07-16 Thread Richard Henderson
On 07/16/2013 10:19 AM, Joe Perches wrote: > On Tue, 2013-07-16 at 10:04 -0700, Richard Henderson wrote: >> Remove the compile warning for __udiv_qrnnd not having a prototype. >> Use the __builtin_alpha_umulh introduced in gcc 4.0. > > Isn't gcc 3.x still a supported compiler? Further

Re: [PATCH] ndisc: bool initializations should use true and false

2013-07-16 Thread David Miller
From: Daniel Baluta Date: Sat, 13 Jul 2013 11:26:51 +0300 > Signed-off-by: Daniel Baluta Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-07-16 Thread boris brezillon
On 16/07/2013 20:47, Alan Stern wrote: On Tue, 16 Jul 2013, boris brezillon wrote: + uclk = clk_get(>dev, "usb_clk"); + if (IS_ERR(uclk)) { + uclk = NULL; + dev_warn(>dev, "failed to get usb_clk\n"); + } Is this really what you want for backward

Re: [PATCH] ethtool: fixed trailing statements in ethtool

2013-07-16 Thread David Miller
From: Dragos Foianu Date: Sat, 13 Jul 2013 14:43:00 +0100 > Applied fixes suggested by checkpatch.pl. > > Signed-off-by: Dragos Foianu Applied, but only so that life is easier for the next person who makes changes to this file and wants to validate things with checkpatch. -- To unsubscribe

Re: [PATCH] net/irda: fixed style issues in irlan_eth

2013-07-16 Thread David Miller
From: Dragos Foianu Date: Sat, 13 Jul 2013 15:03:55 +0100 > Applied fixes suggested by checkpatch.pl > > Signed-off-by: Dragos Foianu Applied. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info

[RFC] sched: Limit idle_balance() when it is being used too frequently

2013-07-16 Thread Jason Low
When running benchmarks on an 8 socket 80 core machine with a 3.10 kernel, there can be a lot of contention in idle_balance() and related functions. On many AIM7 workloads in which CPUs go idle very often and idle balance gets called a lot, it is actually lowering performance. Since idle balance

Re: [RFC][PATCH 0/9] sched: Power scheduler design proposal

2013-07-16 Thread Peter Zijlstra
On Tue, Jul 16, 2013 at 11:44:15AM -0700, Arjan van de Ven wrote: > the interaction is "using the scheduler data using the scheduler provided > function". I'm so not following. > So I don't just want something that makes sense for todays Intel ;-) > We need something that has an interface that

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Greg Kroah-Hartman
On Tue, Jul 16, 2013 at 02:41:24PM -0400, Steven Rostedt wrote: > On Tue, 2013-07-16 at 11:29 -0700, Linus Torvalds wrote: > > > Anyway, the point I'm making is that Q is limited and often even > > actively misleading ("Hey, I have three tested-by's, so it must be > > fine"), and we might

Re: driver model, duplicate names question

2013-07-16 Thread Srinivas Pandruvada
On 07/16/2013 12:04 PM, Greg KH wrote: A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top Glad to learn something new today. On Tue, Jul 16, 2013 at 11:54:31AM -0700, Srinivas Pandruvada wrote: Hi, I am assigned to do add a powercap class. There

Re: arm: Only load TLS values when needed

2013-07-16 Thread André Hentschel
Hi Jonathan, First, thank you for your review. Am 16.07.2013 19:31, schrieb Jonathan Austin: > Hi André, > > On 15/07/13 18:14, André Hentschel wrote: >> From: André Hentschel >> >> This patch intents to reduce loading instructions when the resulting value >> is not used. >> It's a follow up

Re: [RFC][PATCH 0/4] tracing/kprobes/uprobes: Fix race between opening probe event files and deleting probe

2013-07-16 Thread Oleg Nesterov
On 07/16, Steven Rostedt wrote: > > On Tue, 2013-07-16 at 18:38 +0200, Oleg Nesterov wrote: > > On 07/15, Oleg Nesterov wrote: > > > > > > So. As Masami pointed out, this is not enough. Probably we can add more > > > hacks, but I'd like to discuss the alternative approach. > > > > > > Note also

Re: [RFC] sched: Limit idle_balance() when it is being used too frequently

2013-07-16 Thread Rik van Riel
On 07/16/2013 03:21 PM, Jason Low wrote: When running benchmarks on an 8 socket 80 core machine with a 3.10 kernel, there can be a lot of contention in idle_balance() and related functions. On many AIM7 workloads in which CPUs go idle very often and idle balance gets called a lot, it is actually

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Takashi Iwai
At Tue, 16 Jul 2013 09:42:34 -0700 (PDT), David Lang wrote: > > On Tue, 16 Jul 2013, Takashi Iwai wrote: > > > At Tue, 16 Jul 2013 00:19:16 -0700 (PDT), > > David Lang wrote: > >> > >> On Fri, 12 Jul 2013, Willy Tarreau wrote: > >> > >>> And maybe in the end, having 1/10 patch cause a regression

Re: [PATCH] um/configs: don't use devtmpfs in defconfig

2013-07-16 Thread Ramkumar Ramachandra
Richard Weinberger wrote: > http://user-mode-linux.sourceforge.net/downloads.html I found a uml_utilities_20070815.tar.bz2. 2007, seriously? Is there no interest in maintaining it? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

[PATCH RFC] allow some kernel filesystems to be mounted in a user namespace

2013-07-16 Thread Serge Hallyn
All the files will be owned by host root, so there's no security concern in allowing this. (These are mounted by default by mountall, and if permission is denied then by default container boot will hang) Signed-off-by: Serge Hallyn --- fs/debugfs/inode.c | 1 + fs/fuse/control.c | 1 +

Re: [PATCH] um/configs: don't use devtmpfs in defconfig

2013-07-16 Thread Richard Weinberger
Am 16.07.2013 21:29, schrieb Ramkumar Ramachandra: > Richard Weinberger wrote: >> http://user-mode-linux.sourceforge.net/downloads.html > > I found a uml_utilities_20070815.tar.bz2. 2007, seriously? Is there > no interest in maintaining it? What patches are missing? Maintaining != Applying a

Re: [ATTEND] How to act on LKML

2013-07-16 Thread H. Peter Anvin
On 07/16/2013 09:58 AM, Stefano Stabellini wrote: > > Because Linux is the most widely used kernel, it's everywhere from > embedded devices to supercomputers. > Many different companies make a business on Linux and pay people to work > on it (not FreeBSD or NetBSD). But that's different from what

Re: [PATCH v2 1/3] x86, boot: add mmio serial during compressed boot

2013-07-16 Thread Kees Cook
On Fri, Jul 12, 2013 at 11:47 PM, Yinghai Lu wrote: > On Fri, Jul 12, 2013 at 1:38 PM, Kees Cook wrote: >> Allows "console=uart[8250],mmio[32],0xADDR[,BAUDn8[,BASE_BAUD]]" to >> be recognized during compressed boot early console setup, and during >> boot console setup. Replaces defines with

Re: [PATCH] virtio-net: put virtio net header inline with data

2013-07-16 Thread David Miller
From: Rusty Russell Date: Mon, 15 Jul 2013 11:13:25 +0930 > From: Michael S. Tsirkin > > For small packets we can simplify xmit processing > by linearizing buffers with the header: > most packets seem to have enough head room > we can use for this purpose. > Since existing hypervisors require

Re: splice vs execve lockdep trace.

2013-07-16 Thread Ben Myers
Hi Dave, Linus, On Tue, Jul 16, 2013 at 04:03:51PM +1000, Dave Chinner wrote: > On Mon, Jul 15, 2013 at 08:25:14PM -0700, Linus Torvalds wrote: > > On Mon, Jul 15, 2013 at 7:38 PM, Dave Jones wrote: > > > > > > The recent trinity changes shouldn't have really made > > > any notable difference

Re: [PATCH v2 1/3] x86, boot: add mmio serial during compressed boot

2013-07-16 Thread H. Peter Anvin
On 07/16/2013 12:31 PM, Kees Cook wrote: > > Could the first step be documenting the limitation? I've found this > patch extremely useful for my case already, and I imagine there might > be other people that need the early mmio stuff to. Generally the > compressed boot serial console stuff is

Re: [PATCH 3.11-rc1] mlx5 core: Fix __udivdi3 when compiling for 32 bit arches

2013-07-16 Thread David Miller
From: Randy Dunlap Date: Mon, 15 Jul 2013 08:52:18 -0700 > On 07/15/13 07:56, Tim Gardner wrote: >> Cc: Eli Cohen >> Signed-off-by: Tim Gardner > > I reported this last week and Eli wrote: > > "I have this fixed in my tree and we run the driver on i386. I will check on > Sunday why it is

Re: driver model, duplicate names question

2013-07-16 Thread Greg KH
On Tue, Jul 16, 2013 at 12:33:03PM -0700, Srinivas Pandruvada wrote: > >> +Example Sys-FS Interface > >> + > >> +/sys/class/power_cap/intel-rapl > >> +├── package-0 > >> +│ ├── constraint-0 > >> +│ │ ├── name > >> +│ │ ├── power_limit_uw > >> +│ │ └── time_window_us > >> +│ ├──

Re: [RFC] kbuild: Build samples directory with make modules

2013-07-16 Thread Sam Ravnborg
On Tue, Jul 16, 2013 at 07:29:50PM +0100, Jim Baxter wrote: > If CONFIG_SAMPLES is enabled with any of the sample > modules enabled, they are only built if the make command > includes uImage and modules (i.e. make uImage modules). With the current location samples are built when you build

Re: [PATCH RFC] allow some kernel filesystems to be mounted in a user namespace

2013-07-16 Thread Al Viro
On Tue, Jul 16, 2013 at 02:29:20PM -0500, Serge Hallyn wrote: > All the files will be owned by host root, so there's no security > concern in allowing this. Files owned by root != very bad things can't be done by non-root. Especially for debugfs, which is very much a "don't even think about

[PATCH/v2] [trivial] treewide: Fix typo in printk

2013-07-16 Thread Masanari Iida
Correct spelling typo in printk Signed-off-by: Masanari Iida --- drivers/crypto/ux500/hash/hash_core.c | 2 +- drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c | 4 ++-- drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c | 2 +- drivers/net/wireless/cw1200/wsm.c

Re: [PATCH v2 1/3] x86, boot: add mmio serial during compressed boot

2013-07-16 Thread Kees Cook
On Tue, Jul 16, 2013 at 12:36 PM, H. Peter Anvin wrote: > On 07/16/2013 12:31 PM, Kees Cook wrote: >> >> Could the first step be documenting the limitation? I've found this >> patch extremely useful for my case already, and I imagine there might >> be other people that need the early mmio stuff

Re: [PATCH] zram percpu implementation

2013-07-16 Thread Valdis . Kletnieks
On Wed, 17 Jul 2013 00:27:34 +0530, mani said: > This patch will create a percpu structures compression algo. > 1. Takes extra memory for workspace buffers. > I haven't seen any performance gain with this need to find the > root cause. My first guess is that the higher-level zram stuff is

Re: [RFC][PATCH 0/4] tracing/kprobes/uprobes: Fix race between opening probe event files and deleting probe

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 21:22 +0200, Oleg Nesterov wrote: > But tracing_open_generic_file()->trace_array_get() is racy, no? > Somehow we need to ensure it is safe to use file / file->tr. Ah crap. Yeah, I see what you mean. It's the race that I described with deleting private data and using it on

Re: [PATCH v8] ARM: clocksource: add support for MOXA ART SoCs

2013-07-16 Thread Daniel Lezcano
On 07/16/2013 04:44 PM, Jonas Jensen wrote: > This patch adds an clocksource driver for the main timer(s) > found on MOXA ART SoCs. > > Signed-off-by: Jonas Jensen > --- Ok, I will ask you two more things before taking your patch. I missed them at the previous review, sorry for asking a new

Re: [PATCH v2 1/3] x86, boot: add mmio serial during compressed boot

2013-07-16 Thread H. Peter Anvin
On 07/16/2013 12:40 PM, Kees Cook wrote: >> >> I think setting up a dynamic #PF handler is the right thing for the >> decompressor, we already did for the kernel proper. > > I'm not sure how to accomplish this yet. I'm still trying to > understand how the page tables are arranged. :) > Look at

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 12:11 -0700, Greg Kroah-Hartman wrote: > People mark stable patches that way already today with a: > Cc: stable # delay for 3.12-rc4 > or some such wording. I take those and don't apply them until the noted > release happens, so you can do this if needed. I guess

Re: [RFC PATCH 2/4] tracing: Turn "id"->i_private into call->event.type

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 20:57 +0200, Oleg Nesterov wrote: > ftrace_event_id_fops and event_id_read() is overcomplicated. > > 1. Change event_create_dir() to pass "data = call->event.type" >to debugfs_create_file(). > >This means that ftrace_event_id_fops doesn't need .open() >and

Re: [PATCH RFC] allow some kernel filesystems to be mounted in a user namespace

2013-07-16 Thread Serge E. Hallyn
Quoting Al Viro (v...@zeniv.linux.org.uk): > On Tue, Jul 16, 2013 at 02:29:20PM -0500, Serge Hallyn wrote: > > All the files will be owned by host root, so there's no security > > concern in allowing this. > > Files owned by root != very bad things can't be done by non-root. > Especially for

Re: [RFC PATCH 3/4] tracing: Kill tracing_open/release_generic_file

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 20:57 +0200, Oleg Nesterov wrote: > Kill tracing_open_generic_file() and tracing_release_generic_file(), > they are racy anyway. > > Instead, change event_enable_read() and event_enable_write() to rely > on event_mutex and file_inode(filp)->i_private != NULL check. > >

Re: [Regression][v3.10][v3.11] net: Kill link between CSUM and SG features.

2013-07-16 Thread Joseph Salisbury
On 07/16/2013 02:29 PM, Pravin Shelar wrote: > On Tue, Jul 16, 2013 at 11:09 AM, Joseph Salisbury > wrote: >> Hi Pravin, >> >> A bug was opened against the Ubuntu kernel[0]. After a kernel bisect, >> it was found that reverting the following commit resolved this bug: >> >> commit

Re: [RFC][PATCH 0/9] sched: Power scheduler design proposal

2013-07-16 Thread Arjan van de Ven
On 7/16/2013 12:21 PM, Peter Zijlstra wrote: Suppose a 2 cpu system, one cpu is running 3/4 throttle, the other is running at half speed. Both cpus are equally utilized. A new task comes on. Where do we run it? We need to know that there's head-room on the 1/2 speed cpu and should crank its

Re: [ANNOUNCE] iproute2 3.10

2013-07-16 Thread Christian Hesse
Stephen Hemminger on Tue, 2013/07/16 10:17: > This is update for iproute2 tools for 3.10 kernel. > [...] > > Iproute2 package is available at: > http://kernel.org/pub/linux/utils/net/iproute2/iproute2-3.10.0.tar.gz > > You can download the source from: >

Re: [PATCH net 2/2] macvtap: do not assume 802.1Q when send vlan packets

2013-07-16 Thread David Miller
From: Jason Wang Date: Tue, 16 Jul 2013 13:36:34 +0800 > The hard-coded 8021.q proto will break 802.1ad traffic. So switch to use > vlan->proto. > > Cc: Basil Gor > Signed-off-by: Jason Wang Applied. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

Re: [PATCH net 1/2] macvtap: fix the missing ret value of TUNSETQUEUE

2013-07-16 Thread David Miller
From: Jason Wang Date: Tue, 16 Jul 2013 13:36:33 +0800 > Commit 441ac0fcaadc76ad09771812382345001dd2b813 > (macvtap: Convert to using rtnl lock) forget to return what > macvtap_ioctl_set_queue() returns to its caller. This may break multiqueue API > by always falling through to TUNGETFEATURES. >

cpufreq: possible circular locking detected on suspend

2013-07-16 Thread Sergey Senozhatsky
Hello, extracted as a new thread from https://lkml.org/lkml/2013/7/16/344 cpufreq midifications are: > Srivatsa S. Bhat wrote: > > 1. Patch given in: https://lkml.org/lkml/2013/7/11/661 >(Just apply patch 1, not the entire patchset). > > 2. Apply the patch shown below, on top of the above

Re: [RFC PATCH v5 1/1] drivers: mfd: vexpress: add Serial Power Controller (SPC) support

2013-07-16 Thread Rob Herring
On 07/16/2013 11:05 AM, Lorenzo Pieralisi wrote: > The TC2 versatile express core tile integrates a logic block that provides the > interface between the dual cluster test-chip and the M3 microcontroller that > carries out power management. The logic block, called Serial Power Controller > (SPC),

Re: [ATTEND] How to act on LKML (was: [ 00/19] 3.10.1-stable review)

2013-07-16 Thread Martin Steigerwald
Am Montag, 15. Juli 2013, 15:50:03 schrieb Sarah Sharp: > On Mon, Jul 15, 2013 at 03:38:42PM -0700, Linus Torvalds wrote: > > On Mon, Jul 15, 2013 at 3:08 PM, Steven Rostedt > > wrote: > > > Can we please make this into a Kernel Summit discussion. I highly doubt > > > we would solve anything,

Re: driver model, duplicate names question

2013-07-16 Thread Srinivas Pandruvada
On 07/16/2013 12:32 PM, Greg KH wrote: On Tue, Jul 16, 2013 at 12:33:03PM -0700, Srinivas Pandruvada wrote: +Example Sys-FS Interface + +/sys/class/power_cap/intel-rapl +├── package-0 +│ ├── constraint-0 +│ │ ├── name +│ │ ├── power_limit_uw +│ │ └── time_window_us +│ ├──

Re: [ANNOUNCE] iproute2 3.10

2013-07-16 Thread Stephen Hemminger
On Tue, 16 Jul 2013 21:56:55 +0200 Christian Hesse wrote: > Stephen Hemminger on Tue, 2013/07/16 10:17: > > This is update for iproute2 tools for 3.10 kernel. > > [...] > > > > Iproute2 package is available at: > > http://kernel.org/pub/linux/utils/net/iproute2/iproute2-3.10.0.tar.gz > > > >

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread Willy Tarreau
On Tue, Jul 16, 2013 at 03:43:09PM -0400, Steven Rostedt wrote: > On Tue, 2013-07-16 at 12:11 -0700, Greg Kroah-Hartman wrote: > > > People mark stable patches that way already today with a: > > Cc: stable # delay for 3.12-rc4 > > or some such wording. I take those and don't apply them

[PATCH] audit: listen in all network namespaces

2013-07-16 Thread Richard Guy Briggs
Convert audit from only listening in init_net to use register_pernet_subsys() to dynamically manage the netlink socket list. Signed-off-by: Richard Guy Briggs --- kernel/audit.c | 64 ++- kernel/audit.h |4 +++ 2 files changed, 57

Re: [RFC][PATCH 0/9] sched: Power scheduler design proposal

2013-07-16 Thread Peter Zijlstra
On Tue, Jul 16, 2013 at 12:57:34PM -0700, Arjan van de Ven wrote: > then the question of how much remaining capacity; this is a hard one, and not > just > for Intel. Almost all mobile devices today are thermally constrained, ARM and > Intel > alike (at least the higher performance ones)... the

Re: splice vs execve lockdep trace.

2013-07-16 Thread Linus Torvalds
On Tue, Jul 16, 2013 at 12:33 PM, Ben Myers wrote: >> > >> > And looking more at that, I'm actually starting to think this is an >> > XFS locking problem. XFS really should not call back to splice while >> > holding the inode lock. .. that was misleading, normally "inode lock" would be i_lock,

Re: [Intel-gfx] [PATCH] drm/i915: fix long-standing SNB regression in power consumption after resume

2013-07-16 Thread Jesse Barnes
On Tue, 16 Jul 2013 10:06:54 -0700 Jesse Barnes wrote: > On Tue, 16 Jul 2013 11:34:25 +0400 > Konstantin Khlebnikov wrote: > > I've tested that patch and it really works for me. If you want change > > something for other hardware or > > extend range where forcewake is held prease do it in a

Re: [RFC] sched: Limit idle_balance() when it is being used too frequently

2013-07-16 Thread Peter Zijlstra
On Tue, Jul 16, 2013 at 12:21:03PM -0700, Jason Low wrote: > When running benchmarks on an 8 socket 80 core machine with a 3.10 kernel, > there can be a lot of contention in idle_balance() and related functions. > On many AIM7 workloads in which CPUs go idle very often and idle balance > gets

Re: [RFC][PATCH 0/9] sched: Power scheduler design proposal

2013-07-16 Thread Arjan van de Ven
On 7/16/2013 1:17 PM, Peter Zijlstra wrote: On Tue, Jul 16, 2013 at 12:57:34PM -0700, Arjan van de Ven wrote: then the question of how much remaining capacity; this is a hard one, and not just for Intel. Almost all mobile devices today are thermally constrained, ARM and Intel alike (at least

Re: [PATCH v5] x86: make sure IDT is page aligned

2013-07-16 Thread Yinghai Lu
On Tue, Jul 16, 2013 at 11:34 AM, Kees Cook wrote: > Since the IDT is referenced from a fixmap, make sure it is page aligned. > Merge with 32-bit one, since it was already aligned to deal with F00F > bug. Since bss is cleared before IDT setup, it can live there. This also > moves the other

ext4_da_release_space:1333: ext4_da_release_space: ino 12, to_free 1 with only 0 reserved data blocks

2013-07-16 Thread Dave Jones
I've seen this happen a few times this week.. EXT4-fs (sdb1): mounted filesystem with ordered data mode. Opts: (null) EXT4-fs warning (device sdb1): ext4_da_release_space:1333: ext4_da_release_space: ino 12, to_free 1 with only 0 reserved data blocks [ cut here ]

Re: [PATCH v5] x86: make sure IDT is page aligned

2013-07-16 Thread Kees Cook
On Tue, Jul 16, 2013 at 1:21 PM, Yinghai Lu wrote: > On Tue, Jul 16, 2013 at 11:34 AM, Kees Cook wrote: >> Since the IDT is referenced from a fixmap, make sure it is page aligned. >> Merge with 32-bit one, since it was already aligned to deal with F00F >> bug. Since bss is cleared before IDT

Re: [Ksummit-2013-discuss] When to push bug fixes to mainline

2013-07-16 Thread David Lang
On Tue, 16 Jul 2013, H. Peter Anvin wrote: On 07/16/2013 12:19 AM, David Lang wrote: On Fri, 12 Jul 2013, Willy Tarreau wrote: And maybe in the end, having 1/10 patch cause a regression is not *that* dramatic, and probably less than not fixing the 9 other bugs. In one case we rely on -stable

Re: [RFC][PATCH 0/9] sched: Power scheduler design proposal

2013-07-16 Thread Arjan van de Ven
On 7/16/2013 1:17 PM, Peter Zijlstra wrote: On Tue, Jul 16, 2013 at 12:57:34PM -0700, Arjan van de Ven wrote: then the question of how much remaining capacity; this is a hard one, and not just for Intel. Almost all mobile devices today are thermally constrained, ARM and Intel alike (at least

Re: [PATCH v5] x86: make sure IDT is page aligned

2013-07-16 Thread Steven Rostedt
On Tue, 2013-07-16 at 13:28 -0700, Kees Cook wrote: > On Tue, Jul 16, 2013 at 1:21 PM, Yinghai Lu wrote: > > On Tue, Jul 16, 2013 at 11:34 AM, Kees Cook wrote: > >> Since the IDT is referenced from a fixmap, make sure it is page aligned. > >> Merge with 32-bit one, since it was already aligned

Re: [PATCH RFC v4] media: OF: add "sync-on-green-active" property

2013-07-16 Thread Sylwester Nawrocki
Hi Prabhakar, On 07/16/2013 07:19 PM, Prabhakar Lad wrote: From: "Lad, Prabhakar" This patch adds 'sync-on-green-active' property as part of endpoint property. Signed-off-by: Lad, Prabhakar --- Changes for v4: 1: Fixed review comments pointed by Sylwester. Changes for v3: 1:

[PATCH 3/6] kconfig/[mn]conf: shorten title in search-box

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" No need to repeat the 'CONFIG_' string in the title, once is explicit enough. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Cc: Jean Delvare --- scripts/kconfig/mconf.c | 4 ++-- scripts/kconfig/nconf.c | 4 ++-- 2 files changed, 4 insertions(+), 4

[PATCH 5/6] kconfig: don't allocate n+1 elements in temporary array

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" The temporary array that stores the search results is not NULL-terminated, so there is no reason to allocate n+1 elements. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Reviewed-by: Jean Delvare --- scripts/kconfig/symbol.c | 2 +- 1 file changed, 1

[PATCH 6/6] kconfig: simplify symbol-search code

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" There is no need for a double indirection in the temporary array that stores the internediate search results. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Reviewed-by: Jean Delvare --- scripts/kconfig/symbol.c | 30 +++--- 1 file

[PATCH 4/6] kconfig: minor style fixes in symbol-search code

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" Two minor style fixes: - no space before/after parenthesis in function definition - no {} for single-line if() And one grammar fix in a comment. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Cc: Jean Delvare Reviewed-by: Jean Delvare ---

[PATCH 2/6] kconfig: avoid multiple calls to strlen

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" Calls to strlen are costly, so avoid calling strln as much as we can. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Cc: Jean Delvare Reviewed-by: Jean Delvare --- scripts/kconfig/symbol.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-)

[pull request] Pull request for branch yem-kconfig-rc-fixes

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" Hello Michal, Please pull these post-rc1 kconfig cleanups, all after review and comments by Jean: - simplify and cleanup the symbol-search code - better documentation about the symbols ordering in search results - eye-candy in the title of the search-box in [mn]conf

<    8   9   10   11   12   13   14   15   16   >