Re: [PATCH 2/2] ARM: kirkwood: DT descriptor for Seagate FreeAgent Dockstar

2012-08-14 Thread Jason Cooper
On Sun, Aug 05, 2012 at 04:46:13PM +0200, Sebastian Hesselbarth wrote: This adds the corresponding device tree descriptor for the Seagate FreeAgent Dockstar based on Kirkwood DT. Please note that this patch requires the mach-kirkwood Makefile.boot typo fix to fully apply. Signed-off-by:

[tip:x86/urgent] Revert x86-64/efi: Use EFI to deal with platform wall clock

2012-08-14 Thread tip-bot for H. Peter Anvin
Commit-ID: f026cfa82f628db24b8cea41b9d6202af104cecb Gitweb: http://git.kernel.org/tip/f026cfa82f628db24b8cea41b9d6202af104cecb Author: H. Peter Anvin h...@zytor.com AuthorDate: Tue, 14 Aug 2012 09:53:38 -0700 Committer: H. Peter Anvin h...@zytor.com CommitDate: Tue, 14 Aug 2012 09:58:25

Re: [PATCH v7 1/4] mm: introduce compaction and migration for virtio ballooned pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 10:35:25PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 02:44:05PM -0300, Rafael Aquini wrote: On Mon, Aug 13, 2012 at 11:26:19AM +0300, Michael S. Tsirkin wrote: +static inline bool movable_balloon_page(struct page *page) +{ + return

Re: [PATCH v3 2/6] floppy: do put_disk on current dr if blk_init_queue fails

2012-08-14 Thread Vivek Goyal
On Mon, Aug 13, 2012 at 03:16:23PM -0300, Herton Ronaldo Krzesinski wrote: If blk_init_queue fails, we do not call put_disk on the current dr (dr is decremented first in the error handling loop). Cc: sta...@vger.kernel.org Signed-off-by: Herton Ronaldo Krzesinski

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 03:22:45PM -0300, Rafael Aquini wrote: On Mon, Aug 13, 2012 at 11:41:23AM +0300, Michael S. Tsirkin wrote: @@ -141,7 +151,10 @@ static void fill_balloon(struct virtio_balloon *vb, size_t num) set_page_pfns(vb-pfns + vb-num_pfns, page);

Re: [PATCH] Documentation: dt: hwmon: Update for tmp102

2012-08-14 Thread Guenter Roeck
On Tue, Aug 14, 2012 at 09:53:32PM +0530, Sourav Poddar wrote: Add Documentation for tmp102 temperature sensor. Cc: Benoit Cousson b-cous...@ti.com Cc: Felipe Balbi ba...@ti.com Cc: Santosh Shilimkar santosh.shilim...@ti.com Signed-off-by: Sourav Poddar sourav.pod...@ti.com ---

Re: [PATCH v7 1/4] mm: introduce compaction and migration for virtio ballooned pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 02:44:05PM -0300, Rafael Aquini wrote: On Mon, Aug 13, 2012 at 11:26:19AM +0300, Michael S. Tsirkin wrote: +static inline bool movable_balloon_page(struct page *page) +{ + return (page-mapping page-mapping == balloon_mapping); I am guessing this needs

[PATCH] hardening: add PROT_FINAL prot flag to mmap/mprotect

2012-08-14 Thread Ard Biesheuvel
This patch adds support for the PROT_FINAL flag to the mmap() and mprotect() syscalls. The PROT_FINAL flag indicates that the requested set of protection bits should be final, i.e., it shall not be allowed for a subsequent mprotect call to set protection bits that were not set already. This is

Re: [PATCH 5/5] x86: Move enabling of PSE and PGE out of init_memory_mapping

2012-08-14 Thread Jacob Shin
On Tue, Aug 14, 2012 at 10:49:16AM +0200, Borislav Petkov wrote: On Mon, Aug 13, 2012 at 04:20:40PM -0700, Tejun Heo wrote: I meant the patch should come before the patch making multiple calls to init_memory_mapping(). Ah, okay .. got it. Hm .. for some reason Tejun's emails are no longer

Re: [PATCH 02/19] Input: Improve the events-per-packet estimate

2012-08-14 Thread Dmitry Torokhov
On Tuesday, August 14, 2012 12:32:21 PM Ping Cheng wrote: On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote: Many MT devices send a number of keys along with the mt information. This patch makes sure that there is room for them in the packet buffer.

Re: [patch 3/8] procfs: Add ability to plug in auxiliary fdinfo providers

2012-08-14 Thread Cyrill Gorcunov
On Tue, Aug 14, 2012 at 10:35:58PM +0400, Cyrill Gorcunov wrote: On Tue, Aug 14, 2012 at 07:31:42PM +0100, Al Viro wrote: Initially we considered to inject some show metod to file_operations but since there really a number of file_operations declared inside kernel (and in real the

Re: [PATCH 02/19] Input: Improve the events-per-packet estimate

2012-08-14 Thread Henrik Rydberg
Hi Ping, Long time no see. :-) + /* Make room for KEY and MSC events */ + events += 7; It is nice to get rid of the redundant pieces and to incorporate common functions. Thank you. I have a question about the code above though. Why do we use 7 instead of going through

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 10:51:39PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 03:22:45PM -0300, Rafael Aquini wrote: On Mon, Aug 13, 2012 at 11:41:23AM +0300, Michael S. Tsirkin wrote: @@ -141,7 +151,10 @@ static void fill_balloon(struct virtio_balloon *vb, size_t num)

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-14 Thread Peter Maydell
On 14 August 2012 19:53, Anthony Liguori anth...@codemonkey.ws wrote: Forget about !x86 platforms. They have their own way to do this sort of thing. Think of this feature like a status LED on a motherboard. These are very common and usually controlled by IO ports. Please don't forget !x86

Re: [GIT PULL] VFIO update for 3.6-rc1

2012-08-14 Thread Alex Williamson
On Tue, 2012-08-07 at 13:05 -0600, Alex Williamson wrote: Hi Linus, The following changes since commit 42a579a0f960081cd16fc945036e4780c3ad3202: Merge branches 'timers-urgent-for-linus' and 'perf-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip (2012-08-05

Re: [PATCH v3 3/6] floppy: avoid leaking extra reference to queue on do_floppy_init error handling

2012-08-14 Thread Vivek Goyal
On Tue, Aug 14, 2012 at 04:20:39AM +0100, Ben Hutchings wrote: On Mon, 2012-08-13 at 15:16 -0300, Herton Ronaldo Krzesinski wrote: After commit 3f9a5aa (floppy: Cleanup disk-queue before caling put_disk() if add_disk() was never called), if something fails in the add_disk loop, we

Re: [PATCH v7 1/4] mm: introduce compaction and migration for virtio ballooned pages

2012-08-14 Thread Rafael Aquini
On Tue, Aug 14, 2012 at 10:35:25PM +0300, Michael S. Tsirkin wrote: +/* __isolate_lru_page() counterpart for a ballooned page */ +bool isolate_balloon_page(struct page *page) +{ + if (WARN_ON(!movable_balloon_page(page))) Looks like this actually can happen if the page

Re: [ 07/44] ARM: 7477/1: vfp: Always save VFP state in vfp_pm_suspend on UP

2012-08-14 Thread Herton Ronaldo Krzesinski
On Mon, Aug 13, 2012 at 03:02:14PM -0700, Greg Kroah-Hartman wrote: From: Greg KH gre...@linuxfoundation.org [...] --- a/arch/arm/vfp/vfpmodule.c +++ b/arch/arm/vfp/vfpmodule.c @@ -412,6 +412,12 @@ static int vfp_pm_suspend(void) /* disable, just in case */

Re: [PATCH v7 0/8] Raid: enable talitos xor offload for improving performance

2012-08-14 Thread Dan Williams
On Tue, Aug 14, 2012 at 2:04 AM, Liu Qiang-B32616 b32...@freescale.com wrote: Hi Vinod, Would you like to apply this series from patch 2/8 to 7/8) in your tree? The link as below, http://patchwork.ozlabs.org/patch/176023/ http://patchwork.ozlabs.org/patch/176024/

Re: [PATCH v7 1/4] mm: introduce compaction and migration for virtio ballooned pages

2012-08-14 Thread Rafael Aquini
On Tue, Aug 14, 2012 at 10:48:37PM +0300, Michael S. Tsirkin wrote: E.g. kvm can emulate hyperv so it could in theory have hyperv balloon. This is mm stuff it is best not to tie it to specific drivers. But of course I agree this is not top priority, no need to block submission on this,

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Rafael Aquini
On Tue, Aug 14, 2012 at 10:59:16PM +0300, Michael S. Tsirkin wrote: What if there is more than one balloon device? Is it possible to load this driver twice, or are you foreseeing a future case where this driver will be able to manage several distinct memory balloons for

Re: [PATCH v3 2/6] floppy: do put_disk on current dr if blk_init_queue fails

2012-08-14 Thread Herton Ronaldo Krzesinski
On Tue, Aug 14, 2012 at 03:48:35PM -0400, Vivek Goyal wrote: On Mon, Aug 13, 2012 at 03:16:23PM -0300, Herton Ronaldo Krzesinski wrote: If blk_init_queue fails, we do not call put_disk on the current dr (dr is decremented first in the error handling loop). Cc: sta...@vger.kernel.org

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Rafael Aquini
On Tue, Aug 14, 2012 at 10:51:39PM +0300, Michael S. Tsirkin wrote: What I think you should do is use rcu for access. And here sync rcu before freeing. Maybe an overkill but at least a documented synchronization primitive, and it is very light weight. I liked your suggestion on barriers, as

Re: [PATCH 1/1] boot: Put initcall_debug into its own Kconfig option DEBUG_INITCALL

2012-08-14 Thread Randy Dunlap
On 08/13/2012 06:18 PM, Josh Triplett wrote: On Mon, Aug 13, 2012 at 03:39:54PM -0700, Randy Dunlap wrote: On 08/13/2012 03:08 PM, Thai Bui wrote: Hi all, I am as part of a capstone group at Portland State University is working to tinify the kernel as small as possible. The ultimate goal is

Re: [PATCH 5/5] x86: Move enabling of PSE and PGE out of init_memory_mapping

2012-08-14 Thread Tejun Heo
Hello, On Tue, Aug 14, 2012 at 02:52:48PM -0500, Jacob Shin wrote: You mean pagetable_reserve() right? This is actually needed on every call to init_memory_mapping(), My patch 2/5 changes find_early_table_space() to find just enough space to map start to end. The pagetable_reserve() will

Re: [patch net-next v2 00/15] net: introduce upper device lists and remove dev-master

2012-08-14 Thread Andy Gospodarek
On Tue, Aug 14, 2012 at 05:05:33PM +0200, Jiri Pirko wrote: Hi all. Recent discussion around [net-next] bonding: don't allow the master to become its slave forced me to think about upper-lower device connections. This patchset adds a possibility to record upper device linkage. All

Re: [PATCH v7 1/4] mm: introduce compaction and migration for virtio ballooned pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 05:00:49PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 10:35:25PM +0300, Michael S. Tsirkin wrote: +/* __isolate_lru_page() counterpart for a ballooned page */ +bool isolate_balloon_page(struct page *page) +{ + if

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 05:08:31PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 10:59:16PM +0300, Michael S. Tsirkin wrote: What if there is more than one balloon device? Is it possible to load this driver twice, or are you foreseeing a future case where this driver

Re: [PATCH] iio: lm3533-als: Fix build warnings

2012-08-14 Thread Jonathan Cameron
On 08/02/2012 11:10 AM, Axel Lin wrote: Fix below build warnings: CC [M] drivers/iio/light/lm3533-als.o drivers/iio/light/lm3533-als.c:667:8: warning: initialization from incompatible pointer type [enabled by default] drivers/iio/light/lm3533-als.c:667:8: warning: (near initialization for

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Rafael Aquini
On Tue, Aug 14, 2012 at 11:24:01PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 05:08:31PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 10:59:16PM +0300, Michael S. Tsirkin wrote: What if there is more than one balloon device? Is it possible to load this

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 05:11:13PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 10:51:39PM +0300, Michael S. Tsirkin wrote: What I think you should do is use rcu for access. And here sync rcu before freeing. Maybe an overkill but at least a documented synchronization primitive, and

Re: [PATCH] drivers/iio/adc/at91_adc.c: use devm_ functions

2012-08-14 Thread Jonathan Cameron
Lars-Peter, Are you happy with this updated version? Can't immediately find any response from you to it. Jonathan From: Julia Lawall julia.law...@lip6.fr The various devm_ functions allocate memory that is released when a driver detaches. This patch uses these functions for data that is

Re: [patch net-next v2 00/15] net: introduce upper device lists and remove dev-master

2012-08-14 Thread Ben Hutchings
On Tue, 2012-08-14 at 16:19 -0400, Andy Gospodarek wrote: On Tue, Aug 14, 2012 at 05:05:33PM +0200, Jiri Pirko wrote: Hi all. Recent discussion around [net-next] bonding: don't allow the master to become its slave forced me to think about upper-lower device connections. This

Re: [PATCH] epoll: Improved support for multi-threaded clients

2012-08-14 Thread Christof Meerwald
Hi Paton, On Thu, Aug 02, 2012 at 06:37:06PM -0700, Paton J. Lewis wrote: [...] My first concern is about code clarity. Using a custom event to delete an event type (either EPOLLIN or EPOLLOUT) from an epoll item requires that functionality to be split across two areas of code: the code that

Re: [PATCHSET] timer: clean up initializers and implement irqsafe timers

2012-08-14 Thread Thomas Gleixner
On Tue, 14 Aug 2012, Tejun Heo wrote: On Tue, Aug 14, 2012 at 08:55:16PM +0200, Thomas Gleixner wrote: * mod_delayed_work() can't be used from IRQ handlers. This function does not exist. So what? It makes the workqueue users messy. It's difficult to get completely correct and subtle

[PATCH] staging: comedi: adl_pci7x3x: fix pointer-to-int-cast warning

2012-08-14 Thread H Hartley Sweeten
This driver uses the void *private variable in the comedi_subdevice to pass the offset needed to read/write the appropriate register to get/set the channels for the subdevice. The adl_pci7x3x_do_insn_bits() and adl_pci7x3x_di_insn_bits() functions were retrieving this offset by casting the

Re: efi_pstore: question about how to remove create_sysfs_entry() from a write callback.

2012-08-14 Thread Mike Waychison
On Tue, Aug 14, 2012 at 12:29 PM, Seiji Aguchi seiji.agu...@hds.com wrote: Mike, Previous pseudo code was not correct. More precisely, it is as follows. But, we still need to alloc memory dynamically in the part of Save information about new entry because a workqueue can't know how many

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 05:29:50PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 11:24:01PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 05:08:31PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 10:59:16PM +0300, Michael S. Tsirkin wrote: What if there is more

Re: [PATCH 02/19] Input: Improve the events-per-packet estimate

2012-08-14 Thread Ping Cheng
On Tue, Aug 14, 2012 at 12:53 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Tuesday, August 14, 2012 12:32:21 PM Ping Cheng wrote: On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote: Many MT devices send a number of keys along with the mt information. This

[PATCH] sched: Support compiling out real-time scheduling with REALTIME_SCHED.

2012-08-14 Thread Trevor Brandt
Adds support for compiling out the real-time scheduler (SCHED_FIFO and SCHED_RR) to save space. Changes sched_set_stop_task to use SCHED_NORMAL rather than SCHED_FIFO, since the kernel only uses this function as a fake scheduling priority for userspace to read to avoid exposing the stop class to

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 11:49:06PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 05:29:50PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 11:24:01PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 05:08:31PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at

Re: [GIT PULL] Disintegrate the User API from the kernel headers

2012-08-14 Thread David Howells
Oliver Hartkopp socket...@hartkopp.net wrote: The defines are relevant for upcoming tools and drivers (for hardware that will become available in Q4/2012). So we're currently not breaking any Userspace API. After it turned out that one of the bits was misleading i wanted to fix the bits ASAP,

Re: [PATCH 1/1] boot: Put initcall_debug into its own Kconfig option DEBUG_INITCALL

2012-08-14 Thread Josh Triplett
On Tue, Aug 14, 2012 at 01:13:00PM -0700, Randy Dunlap wrote: On 08/13/2012 06:18 PM, Josh Triplett wrote: On Mon, Aug 13, 2012 at 03:39:54PM -0700, Randy Dunlap wrote: In any case, do you object to the introduction of a Kconfig option at all, or to that option defaulting to off? In

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Rik van Riel
On 08/14/2012 04:54 PM, Michael S. Tsirkin wrote: To clarify, the global state that this patch adds, is ugly even if we didn't support multiple balloons yet. So I don't think I can accept such a patch. Rusty has a final word here, maybe he thinks differently. Before deciding that does not

Re: [PATCH 0/5] Call netif_carrier_off() after register_netdev()

2012-08-14 Thread David Miller
From: Ilya Shchepetkov shchepet...@ispras.ru Date: Tue, 14 Aug 2012 14:28:50 +0400 Hi, There are several patches on the subject: 31bde1ceaa873bcaecd49e829bfabceacc4c512d c55ad8e56b983f03589b38b4504b5d1f41161ff8 e826eafa65c6f1f7c8db5a237556cebac57ebcc5

Re: [PATCH v2 30/31] arm64: Build infrastructure

2012-08-14 Thread Sam Ravnborg
diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig new file mode 100644 index 000..1ce3d04 --- /dev/null +++ b/arch/arm64/Kconfig @@ -0,0 +1,261 @@ +config ARM64 + def_bool y + select OF + select OF_EARLY_FLATTREE + select IRQ_DOMAIN + select HAVE_AOUT +

Re: [PATCHSET] timer: clean up initializers and implement irqsafe timers

2012-08-14 Thread Thomas Gleixner
On Tue, 14 Aug 2012, Tejun Heo wrote: On Tue, Aug 14, 2012 at 09:16:16PM +0200, Thomas Gleixner wrote: Why the hell are you trying to rush stuff which affects a well maintained part of the kernel through your own tree w/o having the courtesy of contacting the maintainer politely instead of

[PATCH 3/3] apple-gmux: Add display mux support

2012-08-14 Thread Seth Forshee
From: Andreas Heider andr...@meetr.de Add support for the gmux display muxing functionality and register a mux handler with vga_switcheroo. Signed-off-by: Andreas Heider andr...@meetr.de Signed-off-by: Seth Forshee seth.fors...@canonical.com --- drivers/platform/x86/apple-gmux.c | 222

[PATCH 0/3] Display switching support for Apple laptops

2012-08-14 Thread Seth Forshee
This series adds display switching support for Apple laptops. The first two patches contain preparatory changes to vga_switcheroo, and the third contains the changes to support display switching with the gmux. While these patches will allow switching the display mux, most Macbook owners will not

[PATCH 1/3] vga_switcheroo: Don't require handler init callback

2012-08-14 Thread Seth Forshee
This callback is a no-op in nouveau, and the upcoming apple-gmux switcheroo support won't require it either. Rather than forcing drivers to stub it out, just make it optional and remove the callback from nouveau. Signed-off-by: Seth Forshee seth.fors...@canonical.com ---

[PATCH 2/3] vga_switcheroo: Remove assumptions about registration/unregistration ordering

2012-08-14 Thread Seth Forshee
vga_switcheroo assumes that the handler will be registered before the last client, otherwise switching will not be enabled. Likewise it's assumed that the handler will not be unregistered without at least one client also being unregistered, otherwise switching will remain enabled despite no longer

Re: [PATCH 02/19] Input: Improve the events-per-packet estimate

2012-08-14 Thread Ping Cheng
On Tue, Aug 14, 2012 at 1:01 PM, Henrik Rydberg rydb...@euromail.se wrote: Hi Ping, Long time no see. :-) + /* Make room for KEY and MSC events */ + events += 7; It is nice to get rid of the redundant pieces and to incorporate common functions. Thank you. I have a question

Re: [PATCH v2] iio: adc: add new lp8788 adc driver

2012-08-14 Thread Jonathan Cameron
On 08/10/2012 08:06 AM, Kim, Milo wrote: Patch v2. (a) Use iio_priv() for private data rather than allocating data (b) Support raw and scale inferface for iio consumer (c) Make inline function for lp8788_adc_read_raw() (d) For better readability, use fixed number for shift and mask

Re: [PATCH 02/19] Input: Improve the events-per-packet estimate

2012-08-14 Thread Dmitry Torokhov
On Tue, Aug 14, 2012 at 01:50:38PM -0700, Ping Cheng wrote: On Tue, Aug 14, 2012 at 12:53 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Tuesday, August 14, 2012 12:32:21 PM Ping Cheng wrote: On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote: Many MT

Re: [PATCH net-next 0/7] sctp: network namespace support Part 2: per net tunables

2012-08-14 Thread David Miller
Come on Vlad, please review this stuff some time this century. If you want inclusion to be dependent upon your review, then the onus is on you to review it in a timely manner. And you are not doing so here. I'm not letting Eric's patches rot in patchwork for more than a week, this is

Re: yama_ptrace_access_check(): possible recursive locking detected

2012-08-14 Thread Kees Cook
On Thu, Aug 9, 2012 at 6:52 PM, Fengguang Wu fengguang...@intel.com wrote: On Thu, Aug 09, 2012 at 06:39:34PM -0700, Kees Cook wrote: Hi, So, after taking a closer look at this, I cannot understand how it's possible. Yama's task_lock call is against current, not child, which is what

[PATCH] Init: main: fixed a brace coding style issue

2012-08-14 Thread Valerio Baudo =
From: Valerio Baudo vb.xy...@gmail.com Fixed a coding style issue. Signed-off-by: Valerio Baudo vb.xy...@gmail.com --- init/main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/init/main.c b/init/main.c index e60679d..3951ab4 100644 --- a/init/main.c +++ b/init/main.c

Re: [PATCH 1/1] boot: Put initcall_debug into its own Kconfig option DEBUG_INITCALL

2012-08-14 Thread Randy Dunlap
On 08/14/2012 01:55 PM, Josh Triplett wrote: On Tue, Aug 14, 2012 at 01:13:00PM -0700, Randy Dunlap wrote: On 08/13/2012 06:18 PM, Josh Triplett wrote: On Mon, Aug 13, 2012 at 03:39:54PM -0700, Randy Dunlap wrote: In any case, do you object to the introduction of a Kconfig option at all, or

Re: [patch 3/8] procfs: Add ability to plug in auxiliary fdinfo providers

2012-08-14 Thread Al Viro
On Tue, Aug 14, 2012 at 11:56:49PM +0400, Cyrill Gorcunov wrote: Al, does the patch below looks better? If so I'll fix up the rest. struct file_operations { struct module *owner; + struct seq_operations *fdinfo_ops; IDGI. Why on the earth do you need the whole iterator? All it

Re: [PATCH v7 2/2] kvm: KVM_EOIFD, an eventfd for EOIs

2012-08-14 Thread Alex Williamson
On Tue, 2012-08-14 at 11:35 +0300, Michael S. Tsirkin wrote: On Mon, Aug 13, 2012 at 09:09:43PM -0600, Alex Williamson wrote: On Tue, 2012-08-14 at 02:00 +0300, Michael S. Tsirkin wrote: On Mon, Aug 13, 2012 at 04:41:05PM -0600, Alex Williamson wrote: On Tue, 2012-08-14 at 01:06 +0300,

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Rafael Aquini
On Tue, Aug 14, 2012 at 11:49:06PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 05:29:50PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 11:24:01PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 05:08:31PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at

[PATCH 1/3] PM / Runtime: Fix rpm_resume() return value for power.no_callbacks set

2012-08-14 Thread Rafael J. Wysocki
For devices whose power.no_callbacks flag is set, rpm_resume() should return 1 if the device's parent is already active, so that the callers of pm_runtime_get() don't think that they have to wait for the device to resume (asynchronously) in that case (the core won't queue up an asynchronous

[PATCH 0/3] PM / Runtime: rpm_suspend() and rpm_resume() fixes

2012-08-14 Thread Rafael J. Wysocki
Hi, The following three patches fix minor issues in rpm_suspend() and rpm_resume(). They are on top of the current linux-next branch of the linux-pm.git tree, but they should apply to the mainline too. [1/3] - Fix rpm_resume() return value for power.no_callbacks set and active parent.

[PATCH 2/3] PM / Runtime: Clear power.deferred_resume on success in rpm_suspend()

2012-08-14 Thread Rafael J. Wysocki
The power.deferred_resume can only be set if the runtime PM status of device is RPM_SUSPENDING and it should be cleared after its status has been changed, regardless of whether or not the runtime suspend has been successful. However, it only is cleared on suspend failure, while it may remain set

[PATCH 3/3] PM / Runtime: Check device PM QoS setting before no callbacks check

2012-08-14 Thread Rafael J. Wysocki
If __dev_pm_qos_read_value(dev) returns a negative value, rpm_suspend() should return -EPERM for dev even if its power.no_callbacks flag is set. For this to happen, the device's power.no_callbacks flag has to be checked after the PM QoS check, so modify the code accordingly. Signed-off-by:

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 04:56:59PM -0400, Rik van Riel wrote: On 08/14/2012 04:54 PM, Michael S. Tsirkin wrote: To clarify, the global state that this patch adds, is ugly even if we didn't support multiple balloons yet. So I don't think I can accept such a patch. Rusty has a final word

RE: efi_pstore: question about how to remove create_sysfs_entry() from a write callback.

2012-08-14 Thread Seiji Aguchi
Something like this (pseudo) may work: /* Loop until we have all entries in efivars. */ while (1) { variable_name = kzalloc(1024, GFP_KERNEL ); spin_lock_irqsave(efivars-lock); bool found = false; while (1) { ret = ops-get_next_variable(variable_name) if (ret ==

Re: [PATCHSET] timer: clean up initializers and implement irqsafe timers

2012-08-14 Thread Tejun Heo
Hello, Thomas. On Tue, Aug 14, 2012 at 10:43:30PM +0200, Thomas Gleixner wrote: It makes the workqueue users messy. It's difficult to get completely correct and subtle errors are difficult to detect / verify. Ah, the function which does not exist makes the users messy. Interesting

[PATCH] [media] media: coda: add const qualifiers

2012-08-14 Thread Emil Goode
The commit 98d7bbb9 changed *of_device_id.data to const which introduced warnings in various places that have mostly been fixed. This patch fixes one such warning by introducing two const qualifiers. GCC warning: drivers/media/video/coda.c:1785:16: warning: assignment discards ‘const’

Re: [PATCH] net: Fix incorrect comment in netif_tx_stop_queue()

2012-08-14 Thread David Miller
From: Ilya Shchepetkov shchepet...@ispras.ru Date: Mon, 13 Aug 2012 18:43:34 +0400 netif_stop_queue() can be called before register_netdev() because now TX queues are allocated inside alloc_netdev_mqs(). (since ed9af2e839c06c18f721da2c768fbb444c4a10e5 commit) Found by Linux Driver

Re: [BUGFIX -v3 3/4] PCI/PM: Fix config reg access for D3cold and bridge suspending

2012-08-14 Thread Rafael J. Wysocki
Sorry for the delay. On Friday, August 10, 2012, Huang Ying wrote: On Wed, 2012-08-08 at 09:07 +0800, Huang Ying wrote: This patch fixes the following bug: http://marc.info/?l=linux-pcim=134338059022620w=2 Where lspci does not work properly if a device and the corresponding parent

Re: [patch 3/8] procfs: Add ability to plug in auxiliary fdinfo providers

2012-08-14 Thread Cyrill Gorcunov
On Tue, Aug 14, 2012 at 10:27:21PM +0100, Al Viro wrote: On Tue, Aug 14, 2012 at 11:56:49PM +0400, Cyrill Gorcunov wrote: Al, does the patch below looks better? If so I'll fix up the rest. struct file_operations { struct module *owner; + struct seq_operations *fdinfo_ops;

Re: [PATCHSET] timer: clean up initializers and implement irqsafe timers

2012-08-14 Thread Tejun Heo
Hello, Thomas. On Tue, Aug 14, 2012 at 11:03:33PM +0200, Thomas Gleixner wrote: Why should -next have different rules to mainline? It's faster paced and trees revert. The message specifically was a ping for objection and I was waiting for further response and would have waited until early next

Re: [PATCH v7 2/2] kvm: KVM_EOIFD, an eventfd for EOIs

2012-08-14 Thread Alex Williamson
On Tue, 2012-08-14 at 15:35 +0300, Avi Kivity wrote: On 08/12/2012 12:33 PM, Michael S. Tsirkin wrote: Michael, would the interface be more acceptable to you if we added separate ioctls to allocate and free some representation of an irq source ID, gsi pair? For instance, an ioctl might

Re: mq: INFO: possible circular locking dependency detected

2012-08-14 Thread Jan Kara
On Wed 08-08-12 15:54:39, Wu Fengguang wrote: On Wed, Aug 08, 2012 at 03:17:38PM +0800, Fengguang Wu wrote: On Tue, Aug 07, 2012 at 07:39:55AM +0100, Al Viro wrote: On Tue, Aug 07, 2012 at 01:04:12PM +0800, Fengguang Wu wrote: FYI, here is a different back trace on that commit.

[PATCH] regulator: twl-regulator: fix up VINTANA1/VINTANA2

2012-08-14 Thread Aaro Koskinen
It seems commit 2098e95ce9bb039ff2e7bf836df358d18a176139 (regulator: twl: adapt twl-regulator driver to dt) accidentally deleted VINTANA1. Also the same commit defines VINTANA2 twice with TWL4030_ADJUSTABLE_LDO and TWL4030_FIXED_LDO. This patch changes the fixed one to be VINTANA1. I noticed this

Re: mq: INFO: possible circular locking dependency detected

2012-08-14 Thread Al Viro
On Wed, Aug 15, 2012 at 12:09:27AM +0200, Jan Kara wrote: OK, this is a theoretical deadlock since mqueue (virtual) filesystem cannot really be frozen. But we should convert the mqueue code to deal with new lock ordering of i_mutex and mnt_want_write() anyway to make lockdep happy if

Re: [PATCH 1/1] boot: Put initcall_debug into its own Kconfig option DEBUG_INITCALL

2012-08-14 Thread Josh Triplett
On Tue, Aug 14, 2012 at 02:25:07PM -0700, Randy Dunlap wrote: Surely. I have no gross objection to this specific patch. OK. Thanks! PATCHv2 will hide the option behind EMBEDDED and make it default to y. - Josh Triplett -- To unsubscribe from this list: send the line unsubscribe linux-kernel

Re: [PATCH] epoll: Improved support for multi-threaded clients

2012-08-14 Thread Paton J. Lewis
At 8/14/2012 01:21 PM, Christof Meerwald wrote: Hi Paton, On Thu, Aug 02, 2012 at 06:37:06PM -0700, Paton J. Lewis wrote: [...] My first concern is about code clarity. Using a custom event to delete an event type (either EPOLLIN or EPOLLOUT) from an epoll item requires that functionality to

Re: [PATCH RFT 1/2] gpio: gpio-ml-ioh: Use spinlock for register access protection

2012-08-14 Thread Linus Walleij
On Sun, Jul 29, 2012 at 4:54 AM, Axel Lin axel@gmail.com wrote: gpio_chip.can_sleep is 0, but current code uses mutex in ioh_gpio_set, ioh_gpio_get and ioh_gpio_direction_input functions. Thus those functions are not callable from interrupt context. This patch converts mutex into

Re: [PATCH 0/4] promote zcache from staging

2012-08-14 Thread Seth Jennings
On 07/27/2012 01:18 PM, Seth Jennings wrote: zcache is the remaining piece of code required to support in-kernel memory compression. The other two features, cleancache and frontswap, have been promoted to mainline in 3.0 and 3.5. This patchset promotes zcache from the staging tree to

Re: [PATCH RFT 2/2] gpio: gpio-pch: Use spinlock for register access protection

2012-08-14 Thread Linus Walleij
On Sun, Jul 29, 2012 at 4:55 AM, Axel Lin axel@gmail.com wrote: gpio_chip.can_sleep is 0, but current code uses mutex in pch_gpio_set pch_gpio_get and pch_gpio_direction_input functions. Thus those functions are not callable from interrupt context. This patch converts mutex into spinlock.

Re: [patch 3/8] procfs: Add ability to plug in auxiliary fdinfo providers

2012-08-14 Thread Cyrill Gorcunov
On Wed, Aug 15, 2012 at 01:56:16AM +0400, Cyrill Gorcunov wrote: struct file_operations { struct module *owner; + struct seq_operations *fdinfo_ops; IDGI. Why on the earth do you need the whole iterator? All it takes is show_fdinfo(struct seq_file *m, struct file *f); have

Re: Bug in UDF, setting larger file size when data in ICB

2012-08-14 Thread Jan Kara
On Mon 23-07-12 17:43:37, Ian Abbott wrote: On 2012-07-23 16:19, Ian Abbott wrote: Hi folks, in fs/udf/inode.c, udf_setsize(), if the new size is larger than the old size and the old data is in the ICB and the new size still fits in the ICB (iinfo-i_alloc_type == ICBTAG_FLAG_AD_IN_ICB and

Re: mq: INFO: possible circular locking dependency detected

2012-08-14 Thread Jan Kara
On Tue 14-08-12 23:13:31, Al Viro wrote: On Wed, Aug 15, 2012 at 12:09:27AM +0200, Jan Kara wrote: OK, this is a theoretical deadlock since mqueue (virtual) filesystem cannot really be frozen. But we should convert the mqueue code to deal with new lock ordering of i_mutex and

Re: [patch net-next v2 01/15] net: introduce upper device lists

2012-08-14 Thread Ben Hutchings
On Tue, 2012-08-14 at 17:05 +0200, Jiri Pirko wrote: This lists are supposed to serve for storing pointers to all upper devices. Eventually it will replace dev-master pointer which is used for bonding, bridge, team but it cannot be used for vlan, macvlan where there might be multiple upper

[PATCH] udf: fix retun value on error path in udf_load_logicalvol

2012-08-14 Thread Nikola Pajkovsky
In case we detect a problem and bail out, we fail to set ret to a nonzero value, and udf_load_logicalvol will mistakenly report success. Signed-off-by: Nikola Pajkovsky npajk...@redhat.com --- fs/udf/super.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

[PATCH 1/4] x86: Move enabling of PSE and PGE out of init_memory_mapping

2012-08-14 Thread Jacob Shin
Depending on the platform, init_memory_mapping() may be called multiple times. Move it out to setup_arch() to avoid writing to cr4 on every call. Signed-off-by: Jacob Shin jacob.s...@amd.com --- arch/x86/kernel/setup.c | 10 ++ arch/x86/mm/init.c | 10 -- 2 files

[PATCH 2/4] x86: find_early_table_space based on memory ranges that are being mapped

2012-08-14 Thread Jacob Shin
Current logic finds enough space for direct mapping page tables from 0 to end. Instead, we only need to find enough space to cover mr[0].start to mr[nr_range].end -- the range that is actually being mapped by init_memory_mapping() This patch also reportedly fixes suspend/resume issue reported in:

[PATCH 3/4] x86: Only direct map addresses that are marked as E820_RAM

2012-08-14 Thread Jacob Shin
Currently direct mappings are created for [ 0 to max_low_pfnPAGE_SHIFT ) and [ 4GB to max_pfnPAGE_SHIFT ), which may include regions that are not backed by actual DRAM. This is fine for holes under 4GB which are covered by fixed and variable range MTRRs to be UC. However, we run into trouble on

[PATCH V3 0/4] x86: Create direct mappings for E820_RAM only

2012-08-14 Thread Jacob Shin
Currently kernel direct mappings are created for all pfns between [ 0 to max_low_pfn ) and [ 4GB to max_pfn ). When we introduce memory holes, we end up mapping memory ranges that are not backed by physical DRAM. This is fine for lower memory addresses which can be marked as UC by fixed/variable

[PATCH 4/4] x86: Fixup code testing if a pfn is direct mapped

2012-08-14 Thread Jacob Shin
Update code that previously assumed pfns [ 0 - max_low_pfn_mapped ) and [ 4GB - max_pfn_mapped ) were always direct mapped, to now look up pfn_mapped ranges instead. Signed-off-by: Jacob Shin jacob.s...@amd.com --- arch/x86/kernel/cpu/amd.c |6 +- arch/x86/platform/efi/efi.c |8

Re: [PATCHSET] timer: clean up initializers and implement irqsafe timers

2012-08-14 Thread Thomas Gleixner
Tejun, On Tue, 14 Aug 2012, Tejun Heo wrote: On Tue, Aug 14, 2012 at 11:03:33PM +0200, Thomas Gleixner wrote: Why should -next have different rules to mainline? It's faster paced and trees revert. The message specifically was a Nonsense. It's a tree which gets stuff which is cooked and

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-14 Thread Michael S. Tsirkin
On Tue, Aug 14, 2012 at 06:34:13PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 11:49:06PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at 05:29:50PM -0300, Rafael Aquini wrote: On Tue, Aug 14, 2012 at 11:24:01PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 14, 2012 at

Re: [PATCH 0/5] Call netif_carrier_off() after register_netdev()

2012-08-14 Thread Ben Hutchings
On Tue, 2012-08-14 at 14:00 -0700, David Miller wrote: From: Ilya Shchepetkov shchepet...@ispras.ru Date: Tue, 14 Aug 2012 14:28:50 +0400 Hi, There are several patches on the subject: 31bde1ceaa873bcaecd49e829bfabceacc4c512d c55ad8e56b983f03589b38b4504b5d1f41161ff8

[net PATCH v3 1/3] net: netprio: fix files lock and remove useless d_path bits

2012-08-14 Thread John Fastabend
Add lock to prevent a race with a file closing and also remove useless and ugly sscanf code. The extra code was never needed and the case it supposedly protected against is in fact handled correctly by sock_from_file as pointed out by Al Viro. CC: Neil Horman nhor...@tuxdriver.com Reported-by: Al

[net PATCH v3 2/3] net: netprio: fd passed in SCM_RIGHTS datagram not set correctly

2012-08-14 Thread John Fastabend
A socket fd passed in a SCM_RIGHTS datagram was not getting updated with the new tasks cgrp prioidx. This leaves IO on the socket tagged with the old tasks priority. To fix this add a check in the scm recvmsg path to update the sock cgrp prioidx with the new tasks value. Thanks to Al Viro for

[net PATCH v3 3/3] net: netprio: fix cgrp create and write priomap race

2012-08-14 Thread John Fastabend
A race exists where creating cgroups and also updating the priomap may result in losing a priomap update. This is because priomap writers are not protected by rtnl_lock. Move priority writer into rtnl_lock()/rtnl_unlock(). CC: Neil Horman nhor...@tuxdriver.com Reported-by: Al Viro

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-14 Thread Marcelo Tosatti
On Tue, Aug 14, 2012 at 02:35:34PM -0500, Anthony Liguori wrote: Marcelo Tosatti mtosa...@redhat.com writes: On Tue, Aug 14, 2012 at 01:53:01PM -0500, Anthony Liguori wrote: Marcelo Tosatti mtosa...@redhat.com writes: On Tue, Aug 14, 2012 at 05:55:54PM +0300, Yan Vugenfirer wrote:

Re: [PATCH V2 0/5] x86: Create direct mappings for E820_RAM only

2012-08-14 Thread Jacob Shin
On Tue, Aug 14, 2012 at 04:34:39PM +0800, Dave Young wrote: On 08/14/2012 05:46 AM, Jacob Shin wrote: Currently kernel direct mappings are created for all pfns between [ 0 to max_low_pfn ) and [ 4GB to max_pfn ). When we introduce memory holes, we end up mapping memory ranges that are not

<    7   8   9   10   11   12   13   14   >