[PATCH] vfio: fix crash on rmmod

2013-06-05 Thread Alexey Kardashevskiy
devtmpfs_delete_node() calls devnode() callback with mode==NULL but vfio still tries to write there. The patch fixes this. Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru --- Steps to reproduce on freshly booted system with no devices given to VFIO: modprobe vfio rmmod vfio_iommu_spapr_tce

Re: [PATCH] usbnet: improve/fix status interrupt endpoint interval

2013-06-05 Thread Oliver Neukum
On Tuesday 04 June 2013 20:28:30 Andreas Mohr wrote: From 307685fe8e6dfc8181e30167b9c31479332cb22f Mon Sep 17 00:00:00 2001 From: Andreas Mohr and...@users.sourceforge.net Date: Sun, 2 Jun 2013 20:37:05 +0200 Subject: [PATCH] usbnet: improve/fix status interrupt endpoint interval tweaking.

Re: [PATCH v2 02/10] zram: avoid invalid memory access in zram_exit()

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 12:06:00AM +0800, Jiang Liu wrote: Memory for zram-disk object may have already been freed after returning from destroy_device(zram), then it's unsafe for zram_reset_device(zram) to access zram-disk again. We can't solve this bug by flipping the order of

[PATCH 0/4 v3] KVM: PPC: IOMMU in-kernel handling

2013-06-05 Thread Alexey Kardashevskiy
Ben, ping! :) This series has tiny fixes (capability and ioctl numbers, changed documentation, compile errors in some configuration). More details are in the commit messages. Rebased on v3.10-rc4. Alexey Kardashevskiy (4): KVM: PPC: Add support for multiple-TCE hcalls powerpc: Prepare to

[PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-06-05 Thread Alexey Kardashevskiy
This allows the host kernel to handle H_PUT_TCE, H_PUT_TCE_INDIRECT and H_STUFF_TCE requests without passing them to QEMU, which should save time on switching to QEMU and back. Both real and virtual modes are supported - whenever the kernel fails to handle TCE request, it passes it to the virtual

[PATCH 1/4] KVM: PPC: Add support for multiple-TCE hcalls

2013-06-05 Thread Alexey Kardashevskiy
This adds real mode handlers for the H_PUT_TCE_INDIRECT and H_STUFF_TCE hypercalls for QEMU emulated devices such as IBMVIO devices or emulated PCI. These calls allow adding multiple entries (up to 512) into the TCE table in one call which saves time on transition to/from real mode. This adds a

[PATCH 2/4] powerpc: Prepare to support kernel handling of IOMMU map/unmap

2013-06-05 Thread Alexey Kardashevskiy
The current VFIO-on-POWER implementation supports only user mode driven mapping, i.e. QEMU is sending requests to map/unmap pages. However this approach is really slow, so we want to move that to KVM. Since H_PUT_TCE can be extremely performance sensitive (especially with network adapters where

[PATCH 4/4] KVM: PPC: Add hugepage support for IOMMU in-kernel handling

2013-06-05 Thread Alexey Kardashevskiy
This adds special support for huge pages (16MB). The reference counting cannot be easily done for such pages in real mode (when MMU is off) so we added a list of huge pages. It is populated in virtual mode and get_page is called just once per a huge page. Real mode handlers check if the

Re: [PATCH v2 03/10] zram: use zram-lock to protect zram_free_page() in swap free notify path

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 12:06:01AM +0800, Jiang Liu wrote: zram_free_page() is protected by down_write(zram-lock) when called by zram_bvec_write(), but there's no such protection when called by zram_slot_free_notify(), which may cause wrong states to zram object. There are two possible

Re: [PATCH v2 04/10] zram: destroy all devices on error recovery path in zram_init()

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 12:06:02AM +0800, Jiang Liu wrote: On error recovery path of zram_init(), it leaks the zram device object causing the failure. So change create_device() to free allocated resources on error path. Signed-off-by: Jiang Liu jiang@huawei.com Cc:

Re: [patch] mm, memcg: add oom killer delay

2013-06-05 Thread David Rientjes
On Tue, 4 Jun 2013, Michal Hocko wrote: I'm not sure a userspace oom notifier would want to keep a preallocated buffer around that is mlocked in memory for all possible lengths of this file. Well, an oom handler which allocates memory under the same restricted memory doesn't make much

Re: [PATCH v2 05/10] zram: avoid double free in function zram_bvec_write()

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 12:06:03AM +0800, Jiang Liu wrote: When doing a patial write and the whole page is filled with zero, zram_bvec_write() will free uncmem twice. Signed-off-by: Jiang Liu jiang@huawei.com Cc: sta...@vger.kernel.org Acked-by: Minchan Kim minc...@kernel.org -- Kind

Re: [PATCH v2 06/10] zram: avoid access beyond the zram device

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 12:06:04AM +0800, Jiang Liu wrote: Function valid_io_request() should verify the entire request doesn't exceed the zram device, otherwise it will cause invalid memory access. Right but you need to explain what invalid memory access is and what's the result from that to

Re: [PATCHv13 2/4] zbud: add to mm/

2013-06-05 Thread Bob Liu
Hi Seth, On 06/04/2013 04:33 AM, Seth Jennings wrote: zbud is an special purpose allocator for storing compressed pages. It is designed to store up to two compressed pages per physical page. While this design limits storage density, it has simple and deterministic reclaim properties that

[net-next rfc V3 1/9] macvtap: fix a possible race between queue selection and changing queues

2013-06-05 Thread Jason Wang
Complier may generate codes that re-read the vlan-numvtaps during macvtap_get_queue(). This may lead a race if vlan-numvtaps were changed in the same time and which can lead unexpected result (e.g. very huge value). We need prevent the compiler from generating such codes by adding an

Re: [PATCH-v2] Set irq thread to RT priority on creation

2013-06-05 Thread Ivo Sieben
Hi Thomas, 2013/6/3 Thomas Gleixner t...@linutronix.de: The question is why there is data present in the UART when the UART driver has not initialized the UART. Up to the point where the UART is opened and the interrupt handler is installed the receiver should be disabled. Also there is the

[net-next rfc V3 0/9] Multiqueue API for macvtap

2013-06-05 Thread Jason Wang
Hi all: This series implements a v3 of fully tuntap compatiable API which could be used by userspace to manage multiple macvtap queues. The main parts is to add TUNSETQUEUE ioctl support for macvtap. Patch 1 - 5 was some tuntap compatibility and misc cleanups. Patch 6 removes the linear search

[net-next rfc V3 4/9] macvtap: introduce macvtap_get_vlan()

2013-06-05 Thread Jason Wang
Factor out the device holding logic to a macvtap_get_vlan(), this will be also used by multiqueue API. Signed-off-by: Jason Wang jasow...@redhat.com --- drivers/net/macvtap.c | 27 --- 1 files changed, 20 insertions(+), 7 deletions(-) diff --git a/drivers/net/macvtap.c

[net-next rfc V3 5/9] macvlan: change the max number of queues to 16

2013-06-05 Thread Jason Wang
Macvtap should be at least compatible with tap, so change the max number to 16. Signed-off-by: Jason Wang jasow...@redhat.com --- include/linux/if_macvlan.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/linux/if_macvlan.h b/include/linux/if_macvlan.h index

[net-next rfc V3 2/9] macvtap: do not add self to waitqueue if doing a nonblock read

2013-06-05 Thread Jason Wang
There's no need to add self to waitqueue if doing a nonblock read. This could help to avoid the spinlock contention. Signed-off-by: Jason Wang jasow...@redhat.com --- drivers/net/macvtap.c |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/net/macvtap.c

[net-next rfc V3 6/9] macvtap: eliminate linear search

2013-06-05 Thread Jason Wang
Linear search were used in both get_slot() and macvtap_get_queue(), this is because: - macvtap didn't reshuffle the array of taps when create or destroy a queue, so when adding a new queue, macvtap must do linear search to find a location for the new queue. This will also complicate the

[net-next rfc V3 8/9] macvtap: add TUNSETQUEUE ioctl

2013-06-05 Thread Jason Wang
This patch adds TUNSETQUEUE ioctl to let userspace can temporarily disable or enable a queue of macvtap. This is used to be compatible at API layer of tuntap to simplify the userspace to manage the queues. This is done through introducing a linked list to track all taps while using vlan-taps array

[net-next rfc V3 9/9] macvtap: enable multiqueue flag

2013-06-05 Thread Jason Wang
To notify the userspace about our capability of multiqueue. Signed-off-by: Jason Wang jasow...@redhat.com --- drivers/net/macvtap.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/net/macvtap.c b/drivers/net/macvtap.c index 355e6ad..b907aee 100644 ---

[net-next rfc V3 7/9] macvtap: allow TUNSETIFF to create multiqueue device

2013-06-05 Thread Jason Wang
Though the queue were in fact created by open(), we still need to add this check to be compatible with tuntap which can let mgmt software use a single API to manage queues. This patch only validates the device name and moves the TUNSETIFF to a helper. Signed-off-by: Jason Wang jasow...@redhat.com

[net-next rfc V3 3/9] macvlan: switch to use IS_ENABLED()

2013-06-05 Thread Jason Wang
Acked-by: Michael S. Tsirkin m...@redhat.com Signed-off-by: Jason Wang jasow...@redhat.com --- include/linux/if_macvlan.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/linux/if_macvlan.h b/include/linux/if_macvlan.h index 84dde1d..e47ad46 100644 ---

[PATCH] eCryptfs: Check return of filemap_write_and_wait during fsync

2013-06-05 Thread Tyler Hicks
Error out of ecryptfs_fsync() if filemap_write_and_wait() fails. Signed-off-by: Tyler Hicks tyhi...@canonical.com Cc: Paul Taysom tay...@chromium.org Cc: Olof Johansson ol...@chromium.org --- After giving Paul's patch one more look, I noticed that we were ignoring filemap_write_and_wait()'s

Re: [PATCH 0/2] introduce list_for_each_entry_del

2013-06-05 Thread Arne Jansen
On 05.06.2013 04:09, Jörn Engel wrote: On Tue, 4 June 2013 14:44:35 -0400, Jörn Engel wrote: Or while_list_drain? I'm fine with while_list_drain, although a name starting with list_ like all other list macros would be nice. How about just list_drain? The next question is where to put it in the

Re: [PATCH v2 07/10] zram: optimize memory operations with clear_page()/copy_page()

2013-06-05 Thread Minchan Kim
Looks good but although we can know your intention easily with only subject, it would be better to add body in description. More questionable thing is I'm not sure Greg accepts this optimization patch(NOT bug fix) because he claimed he will not accept any patches from zram/zsmalloc except plain

Re: [PATCH 07/10] pinctrl: add pinctrl driver for Rockchip SoCs

2013-06-05 Thread Linus Walleij
On Tue, Jun 4, 2013 at 2:05 PM, Heiko Stübner he...@sntech.de wrote: +PULL_AUTO (1 0): indicate this pin needs a pull setting for SoCs + that determine the pull up or down themselfs Hm, never saw that before... Citing the original gpio driver: /*

Re: [Xen-devel] [PATCH] PVH: vcpu info placement, load selectors, and remove debug printk.

2013-06-05 Thread Jan Beulich
On 04.06.13 at 23:53, Mukesh Rathor mukesh.rat...@oracle.com wrote: Following OK? : if (xen_feature(XENFEAT_auto_translated_physmap)) { switch_to_new_gdt(0); asm volatile ( pushq %%rax\n leaq

Re: [PATCH v2 08/10] zram: protect sysfs handler from invalid memory access

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 12:06:06AM +0800, Jiang Liu wrote: Use zram-init_lock to protect access to zram-meta, otherwise it may cause invalid memory access if zram-meta has been freed by zram_reset_device(). This issue may be triggered by: Thread 1: while true; do cat mem_used_total; done

Re: [PATCH 3/3] firmware loader: allow distribution to choose default search paths

2013-06-05 Thread Takashi Iwai
At Wed, 5 Jun 2013 13:42:49 +0800, Ming Lei wrote: For some distributions(e.g. android), firmware images aren't put under kernel built-in search paths, so introduce one Kconfig option to allow distributions or users to choose its specific default search paths, which are always tried before

[PATCH 0/3] workqueue: code refine/clean for workqueue

2013-06-05 Thread Michael Wang
Code refine/clean patch set for workqueue. Michael Wang (3): [PATCH 1/3] workqueue: move the internal helper from .h to .c [PATCH 2/3] workqueue: remove the unused helper in .h [PATCH 3/3] workqueue: add a check point in pwq_activate_delayed_work() ---

[PATCH 1/3] workqueue: move the internal helper from .h to .c

2013-06-05 Thread Michael Wang
alloc_workqueue_attrs(), free_workqueue_attrs() and apply_workqueue_attrs() are only used internally, move them to .c and make them static. CC: Tejun Heo t...@kernel.org Signed-off-by: Michael Wang wang...@linux.vnet.ibm.com --- include/linux/workqueue.h |5 - kernel/workqueue.c|

Re: [PATCH 10/10] arm: add basic support for Rockchip RK3066a boards

2013-06-05 Thread Thomas Petazzoni
Dear Heiko Stübner, On Mon, 3 Jun 2013 01:02:20 +0200, Heiko Stübner wrote: index 000..a2d8c70 --- /dev/null +++ b/arch/arm/mach-rockchip/Makefile.boot @@ -0,0 +1,3 @@ +zreladdr-$(CONFIG_ARCH_ROCKCHIP) := 0x60408000 +params_phys-$(CONFIG_ARCH_ROCKCHIP) := 0x60088000

[PATCH 3/3] workqueue: add a check point in pwq_activate_delayed_work()

2013-06-05 Thread Michael Wang
get_work_pwq() is possible to return NULL, add a check point for that in the context inside pwq_activate_delayed_work(). CC: Tejun Heo t...@kernel.org Signed-off-by: Michael Wang wang...@linux.vnet.ibm.com --- kernel/workqueue.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff

Re: [PATCH v2 09/10] zram: minor code cleanup

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 12:06:07AM +0800, Jiang Liu wrote: Minor code cleanup for zram. Signed-off-by: Jiang Liu jiang@huawei.com --- drivers/staging/zram/zram_drv.c | 11 --- drivers/staging/zram/zram_sysfs.c | 3 +-- 2 files changed, 5 insertions(+), 9 deletions(-)

Re: Please add to stable: module: don't unlink the module until we've removed all exposure.

2013-06-05 Thread Tejun Heo
Hello, On Wed, Jun 05, 2013 at 01:47:43PM +0930, Rusty Russell wrote: I have some printk debugging in (see bottom of email) and was using a serial console, so things were probably running a bit slower than on most systems. Here is trace from my kernel with local patches and not so much

Re: [RFC 0/5] Enable f2fs support inline data

2013-06-05 Thread Jaegeuk Kim
Hi Haicheng, 2013-06-04 (화), 14:01 +0800, Haicheng Li: Hi Jaegeuk Namjae, Sure, we'll address your comments. And this version is RFC, just wanna to make sure this feature is meaningful for f2fs project, and there is no obvious mistake, e.g. missing some critical path. IMO, it is worth to

Re: [PATCH 0/3] workqueue: code refine/clean for workqueue

2013-06-05 Thread Michael Wang
On 06/05/2013 03:10 PM, Michael Wang wrote: Code refine/clean patch set for workqueue. Michael Wang (3): [PATCH 1/3] workqueue: move the internal helper from .h to .c [PATCH 2/3] workqueue: remove the unused helper in .h [PATCH 3/3] workqueue: add a check point in

[PATCH 4/3] workqueue: code refine in wqattrs_equal()

2013-06-05 Thread Michael Wang
code refine to save some line. CC: Tejun Heo t...@kernel.org Signed-off-by: Michael Wang wang...@linux.vnet.ibm.com --- kernel/workqueue.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/kernel/workqueue.c b/kernel/workqueue.c index ee8e29a..5fd4791 100644 ---

Re: [PATCH 3/3] workqueue: add a check point in pwq_activate_delayed_work()

2013-06-05 Thread Tejun Heo
On Wed, Jun 05, 2013 at 03:11:35PM +0800, Michael Wang wrote: get_work_pwq() is possible to return NULL, add a check point for that in the context inside pwq_activate_delayed_work(). CC: Tejun Heo t...@kernel.org Signed-off-by: Michael Wang wang...@linux.vnet.ibm.com ---

Re: [PATCH 1/3] workqueue: move the internal helper from .h to .c

2013-06-05 Thread Tejun Heo
On Wed, Jun 05, 2013 at 03:10:43PM +0800, Michael Wang wrote: alloc_workqueue_attrs(), free_workqueue_attrs() and apply_workqueue_attrs() are only used internally, move them to .c and make them static. CC: Tejun Heo t...@kernel.org Signed-off-by: Michael Wang wang...@linux.vnet.ibm.com

Re: [PATCH v2] arch/*/asm/include/bitops.h: api issue, find_*_bit() defination are different with each other

2013-06-05 Thread Uwe Kleine-König
Hello, On Wed, Jun 05, 2013 at 09:13:01AM +0800, Chen Gang wrote: For arm and m68k, they customize find_*_bit(), but the API is different with 'generic'. avr32, s390, and unicore32 also customize find_*_bit(), but the API is the same with 'generic', and the left architectures all use

Re: [PATCH 4/3] workqueue: code refine in wqattrs_equal()

2013-06-05 Thread Tejun Heo
On Wed, Jun 05, 2013 at 03:14:04PM +0800, Michael Wang wrote: code refine to save some line. CC: Tejun Heo t...@kernel.org Signed-off-by: Michael Wang wang...@linux.vnet.ibm.com --- kernel/workqueue.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git

[PATCH 2/3] workqueue: remove the unused helper in .h

2013-06-05 Thread Michael Wang
__cancel_delayed_work(), flush_work_sync() and flush_delayed_work_sync() are no longer used by anyone, just remove them. CC: Tejun Heo t...@kernel.org Signed-off-by: Michael Wang wang...@linux.vnet.ibm.com --- include/linux/workqueue.h | 27 --- 1 files changed, 0

Re: [PATCH 2/3] workqueue: remove the unused helper in .h

2013-06-05 Thread Tejun Heo
On Wed, Jun 05, 2013 at 03:11:08PM +0800, Michael Wang wrote: __cancel_delayed_work(), flush_work_sync() and flush_delayed_work_sync() are no longer used by anyone, just remove them. These are deprecated interfaces which used to be necessary because they functioned differently. Keeping them

Re: [PATCH] arch: m68k: include: asm: the 3rd parameter of 'insl' and 'outsl' need ' 2'

2013-06-05 Thread schmitz
Chen, On 06/03/2013 06:48 PM, Geert Uytterhoeven wrote: This _only_ applies to use of insl/outsl macros in parport_pc.h, which is only used by Q40 on m68k. I see no reason to change anything in io.h to cope with this warning. I guess your meaning is :

Re: [v5][PATCH 5/6] mm: vmscan: batch shrink_page_list() locking operations

2013-06-05 Thread Hillf Danton
On Tue, Jun 4, 2013 at 1:07 PM, Minchan Kim minc...@kernel.org wrote: On Tue, Jun 04, 2013 at 09:17:26AM +0800, Hillf Danton wrote: On Tue, Jun 4, 2013 at 4:02 AM, Dave Hansen d...@sr71.net wrote: +/* + * pages come in here (via remove_list) locked and leave unlocked + * (on either

Re: [patch -v4 4/8] memcg: enhance memcg iterator to support predicates

2013-06-05 Thread Michal Hocko
On Tue 04-06-13 13:54:26, Tejun Heo wrote: Hey, On Tue, Jun 04, 2013 at 10:48:07PM +0200, Michal Hocko wrote: I really don't think memcg can afford to add more mess than there already is. Let's try to get things right with each change, please. Is this really about inside vs.

Re: Please add to stable: module: don't unlink the module until we've removed all exposure.

2013-06-05 Thread Rusty Russell
Greg KH gre...@linuxfoundation.org writes: On Mon, Jun 03, 2013 at 10:17:17AM -0400, Joe Lawrence wrote: [Cc: sta...@vger.kernel.org] Third time is a charm? The stable address was incorrect from the first msg in this thread, but the relevant bits remain quoted below... Really? I'm

Re: [PATCH] Generic percpu refcounting

2013-06-05 Thread Rusty Russell
Kent Overstreet koverstr...@google.com writes: This implements a refcount with similar semantics to atomic_get()/atomic_dec_and_test() - but percpu. It also implements two stage shutdown, as we need it to tear down the percpu counts. Before dropping the initial refcount, you must call

Re: [PATCH 0/3] workqueue: code refine/clean for workqueue

2013-06-05 Thread Michael Wang
On 06/05/2013 03:10 PM, Michael Wang wrote: Code refine/clean patch set for workqueue. Michael Wang (3): [PATCH 1/3] workqueue: move the internal helper from .h to .c [PATCH 2/3] workqueue: remove the unused helper in .h [PATCH 3/3] workqueue: add a check point in

Re: [PATCH] Generic percpu refcounting

2013-06-05 Thread Tejun Heo
On Wed, Jun 05, 2013 at 05:05:09PM +0930, Rusty Russell wrote: I think this should be rcu_read_lock(), which is currently equivalent but theoretically different. I suggested using preempt_disable/enable() along with sched RCU. I thought that was better for some reason but I was probably

trigger_all_cpu_backtrace() is ignored on x86

2013-06-05 Thread Michel Lespinasse
Hi, I am having a funny issue with code that tries to use trigger_all_cpu_backtrace(). I would expect this function to dump backtraces on architectures that support it, including x86. However as it turns out, include/linux/nmi.h includes asm/irq.h but not asm/nmi.h, so it misses the

[PATCH] vhost_net: clear msg.control for non-zerocopy case during tx

2013-06-05 Thread Jason Wang
When we decide not use zero-copy, msg.control should be set to NULL otherwise macvtap/tap may set zerocopy callbacks which may decrease the kref of ubufs wrongly. Bug were introduced by commit cedb9bdce099206290a2bdd02ce47a7b253b6a84 (vhost-net: skip head management if no outstanding). This

[char-misc 3.10] mei: me: clear interrupts on the resume path

2013-06-05 Thread Tomas Winkler
We need to clear pending interrupts on the resume path. This brings the device into defined state before starting the reset flow This should solve suspend/resume issues: mei_me : wait hw ready failed. status = 0x0 mei_me : version message write failed Signed-off-by: Tomas Winkler

Re: Handling NUMA page migration

2013-06-05 Thread Michal Hocko
On Tue 04-06-13 23:54:45, Frank Mehnert wrote: On Tuesday 04 June 2013 20:17:02 Frank Mehnert wrote: On Tuesday 04 June 2013 16:02:30 Michal Hocko wrote: On Tue 04-06-13 14:14:45, Frank Mehnert wrote: On Tuesday 04 June 2013 13:58:07 Robin Holt wrote: This is probably more

[PATCH 0/2 v4] net/macb: RX path enhancement

2013-06-05 Thread Nicolas Ferre
Here is the patch series for modifying the RX path in macb driver. This change applies on GEM variant of the Cadence IP and introduces function pointers to match the path to the proper adapter. The move to RX buffers adapted to MTU and that can be DMAed directly in SKB is done in two steps but can

[PATCH 1/2 v4] net/macb: increase RX buffer size for GEM

2013-06-05 Thread Nicolas Ferre
Macb Ethernet controller requires a RX buffer of 128 bytes. It is highly sub-optimal for Gigabit-capable GEM that is able to use a bigger DMA buffer. Change this constant and associated macros with data stored in the private structure. RX DMA buffer size has to be multiple of 64 bytes as indicated

Re: [v5][PATCH 5/6] mm: vmscan: batch shrink_page_list() locking operations

2013-06-05 Thread Minchan Kim
On Wed, Jun 05, 2013 at 03:28:27PM +0800, Hillf Danton wrote: On Tue, Jun 4, 2013 at 1:07 PM, Minchan Kim minc...@kernel.org wrote: On Tue, Jun 04, 2013 at 09:17:26AM +0800, Hillf Danton wrote: On Tue, Jun 4, 2013 at 4:02 AM, Dave Hansen d...@sr71.net wrote: +/* + * pages come in here

[PATCH 2/2 v4] net/macb: change RX path for GEM

2013-06-05 Thread Nicolas Ferre
GEM is able to adapt its DMA buffer size, so change the RX path to take advantage of this possibility and remove all kind of memcpy in this path. This modification introduces function pointers for managing differences between MACB and GEM adapter type. Signed-off-by: Nicolas Ferre

Re: [PATCH 21/21] clk: ux500: Supply provider look-up functionality to support Device Tree

2013-06-05 Thread Lee Jones
On Tue, 04 Jun 2013, Arnd Bergmann wrote: On Tuesday 04 June 2013, Linus Walleij wrote: The whole thing is very different from other DT clock things I've seen, usually you add a compatible node for each clock type, and a node for each physical gate. But there may be several ways to skin

Re: [patch -v4 4/8] memcg: enhance memcg iterator to support predicates

2013-06-05 Thread Tejun Heo
Hey, Michal. On Wed, Jun 05, 2013 at 09:37:28AM +0200, Michal Hocko wrote: Tejun, I do not have infinite amount of time and this is barely a priority for the patchset. The core part is to be able to skip nodes/subtrees which are not worth reclaiming, remember? I have already expressed my

Re: NOHZ: WARNING: at arch/x86/kernel/smp.c:123 native_smp_send_reschedule, round 2

2013-06-05 Thread Jiri Kosina
On Wed, 5 Jun 2013, Michael Wang wrote: Just to not let this thread sleep -- I am seeing this as well, even with current Linus' tree (git HEAD == aa4f608). Have you tried this: diff --git a/drivers/cpufreq/cpufreq_governor.c b/drivers/cpufreq/cpufreq_governor.c index 443442d..449be88

[RESEND][PATCH] cpufreq: stats: Add 'load_table' sysfs file to show accumulated data of CPU

2013-06-05 Thread Chanwoo Choi
This patch add new sysfs file to show previous accumulated data of CPU load as following path. This sysfs file is used to judge the correct system state or determine suitable system resource on user-space. - /sys/devices/system/cpu/cpu0/cpufreq/stats/load_table This sysfs file include following

Re: NOHZ: WARNING: at arch/x86/kernel/smp.c:123 native_smp_send_reschedule, round 2

2013-06-05 Thread Michael Wang
On 06/05/2013 04:08 PM, Jiri Kosina wrote: On Wed, 5 Jun 2013, Michael Wang wrote: Just to not let this thread sleep -- I am seeing this as well, even with current Linus' tree (git HEAD == aa4f608). Have you tried this: diff --git a/drivers/cpufreq/cpufreq_governor.c

[PATCH v2] sched: fix clear NOHZ_BALANCE_KICK

2013-06-05 Thread Vincent Guittot
I have faced a sequence where the Idle Load Balance was sometime not triggered for a while on my platform. CPU 0 and CPU 1 are running tasks and CPU 2 is idle CPU 1 kicks the Idle Load Balance CPU 1 selects CPU 2 as the new Idle Load Balancer CPU 2 sets NOHZ_BALANCE_KICK for CPU 2 CPU 2 sends a

Re: [PATCH v4 2/6] staging: android: binder: fix binder interface for 64bit compat layer

2013-06-05 Thread Serban Constantinescu
On 05/06/13 00:58, Arve Hjønnevåg wrote: On Tue, Jun 4, 2013 at 1:54 AM, Serban Constantinescu serban.constantine...@arm.com wrote: On 03/06/13 22:41, Arve Hjønnevåg wrote: On Wed, May 22, 2013 at 3:12 AM, Serban Constantinescu serban.constantine...@arm.com wrote: The changes in this patch

[PATCH] perf/x86/intel: Fix sparse warning

2013-06-05 Thread Yan, Zheng
From: Yan, Zheng zheng.z@intel.com Signed-off-by: Yan, Zheng zheng.z@intel.com --- arch/x86/kernel/cpu/perf_event_intel_uncore.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/cpu/perf_event_intel_uncore.c

Re: Handling NUMA page migration

2013-06-05 Thread Frank Mehnert
On Wednesday 05 June 2013 09:54:54 Michal Hocko wrote: On Tue 04-06-13 23:54:45, Frank Mehnert wrote: On Tuesday 04 June 2013 20:17:02 Frank Mehnert wrote: On Tuesday 04 June 2013 16:02:30 Michal Hocko wrote: On Tue 04-06-13 14:14:45, Frank Mehnert wrote: On Tuesday 04 June 2013

Re: [PATCH v3] include/linux/skbuff.h: using '(__u16) ~0U' instead of '~0U'

2013-06-05 Thread David Miller
From: Chen Gang gang.c...@asianux.com Date: Wed, 05 Jun 2013 08:54:22 +0800 Both 'transport_header' and 'mac_header' are __u16, which are never equal to '~0U'. So need use '(__u16) ~0U' instead of '~0U'. The related warning (with EXTRA_CFLAGS=-W ARCH=m68k for allmodconfig)

Re: [RFC][PATCH 0/2] dma-buf: add importer private data for reimporting

2013-06-05 Thread Daniel Vetter
On Wed, Jun 05, 2013 at 11:52:59AM +0900, 김승우 wrote: On 2013년 06월 04일 21:55, Daniel Vetter wrote: On Tue, Jun 04, 2013 at 07:42:22PM +0900, 김승우 wrote: On 2013년 06월 01일 00:29, Daniel Vetter wrote: On Fri, May 31, 2013 at 07:22:24PM +0900, 김승우 wrote: Hello Daniel, Thanks for

Re: [PATCH v4 03/22] x86, ACPI, mm: Kill max_low_pfn_mapped

2013-06-05 Thread Tang Chen
Hi yinghai, HPA, On 04/12/2013 08:55 AM, Yinghai Lu wrote: Now we have arch_pfn_mapped array, and max_low_pfn_mapped should not be used anymore. I'm rebasing this patch-set to the latest kernel, and improving the comment. But I didn't find any arch_pfn_mapped array in the kernel. Would you

Re: [PATCH] net: core: move mac_pton() to lib/net_utils.c

2013-06-05 Thread Arnd Bergmann
On Tuesday 04 June 2013 17:43:23 David Miller wrote: From: Andy Shevchenko andy.shevche...@gmail.com Date: Tue, 4 Jun 2013 19:46:26 +0300 Since we have at least one user of this function outside of CONFIG_NET scope, we have to provide this function independently. The proposed solution

Re: [PATCH v2 2/3] cpufreq: Remove unused function __cpufreq_driver_getavg

2013-06-05 Thread Viresh Kumar
On 4 June 2013 20:36, Stratos Karafotis strat...@semaphore.gr wrote: On 06/04/2013 08:19 AM, Viresh Kumar wrote: Should this be done in 3/3 ? acpi-cpufreq does not use mperf after 2/3. Why should we compile it with CONFIG_X86_ACPI_CPUFREQ? Do you want me to move the change in 3/3? I

Re: [PATCH] net: core: move mac_pton() to lib/net_utils.c

2013-06-05 Thread David Miller
From: Arnd Bergmann a...@arndb.de Date: Wed, 05 Jun 2013 10:41:45 +0200 On Tuesday 04 June 2013 17:43:23 David Miller wrote: From: Andy Shevchenko andy.shevche...@gmail.com Date: Tue, 4 Jun 2013 19:46:26 +0300 Since we have at least one user of this function outside of CONFIG_NET scope,

Re: [PATCH v1 3/4] dma: dw: split driver to library part and platform code

2013-06-05 Thread Viresh Kumar
On 4 June 2013 18:06, Andy Shevchenko andriy.shevche...@linux.intel.com wrote: To simplify the driver development let's split driver to library and platform code parts. It helps us to add PCI driver in future. Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com ---

[PATCH] cpufreq: prevent 'policy-cpus' become offline in __gov_queue_work()

2013-06-05 Thread Michael Wang
Jiri Kosina jkos...@suse.cz and Borislav Petkov b...@alien8.de reported the warning: [ 51.616759] [ cut here ] [ 51.621460] WARNING: at arch/x86/kernel/smp.c:123 native_smp_send_reschedule+0x58/0x60() [ 51.629638] Modules linked in: ext2 vfat fat loop

Re: [patch -v4 4/8] memcg: enhance memcg iterator to support predicates

2013-06-05 Thread Michal Hocko
On Wed 05-06-13 01:05:45, Tejun Heo wrote: Hey, Michal. On Wed, Jun 05, 2013 at 09:37:28AM +0200, Michal Hocko wrote: Tejun, I do not have infinite amount of time and this is barely a priority for the patchset. The core part is to be able to skip nodes/subtrees which are not worth

Re: [PATCH] cpufreq: prevent 'policy-cpus' become offline in __gov_queue_work()

2013-06-05 Thread Viresh Kumar
On 5 June 2013 14:19, Michael Wang wang...@linux.vnet.ibm.com wrote: Jiri Kosina jkos...@suse.cz and Borislav Petkov b...@alien8.de reported the warning: [ 51.616759] [ cut here ] [ 51.621460] WARNING: at arch/x86/kernel/smp.c:123

Re: [RFC PATCH v1 6/8] zram: avoid access beyond the zram device

2013-06-05 Thread Jerome Marchand
On 06/04/2013 05:09 PM, Jiang Liu wrote: On Tue 04 Jun 2013 09:15:43 PM CST, Jerome Marchand wrote: On 06/03/2013 05:42 PM, Jiang Liu wrote: Function valid_io_request() should verify the entire request doesn't exceed the zram device, otherwise it will cause invalid memory access.

Re: [PATCH V2] hwrng: use platform_{get,set}_drvdata()

2013-06-05 Thread Herbert Xu
On Wed, May 29, 2013 at 09:47:29AM +0900, Jingoo Han wrote: Use the wrapper functions for getting and setting the driver data using platform_device instead of using dev_{get,set}_drvdata() with pdev-dev, so we can directly pass a struct platform_device. Also, unnecessary dev_set_drvdata() is

Re: [PATCH] hw_rng: nomadik: use clk_prepare_enable()

2013-06-05 Thread Herbert Xu
On Mon, Jun 03, 2013 at 02:02:09AM +0200, Linus Walleij wrote: The Nomadik HW RNG driver has seen some rust and is not preparing the clock before use. Fix this up so we get rid of runtime complaints from the clock subsystem. Signed-off-by: Linus Walleij linus.wall...@linaro.org Patch

[PATCH] tuntap: fix a possible race between queue selection and changing queues

2013-06-05 Thread Jason Wang
Complier may generate codes that re-read the tun-numqueues during tun_select_queue(). This may be a race if vlan-numqueues were changed in the same time and can lead unexpected result (e.g. very huge value). We need prevent the compiler from generating such codes by adding an ACCESS_ONCE() to

Re: [PATCH v9 1/9] ARM: davinci: move private EDMA API to arm/common

2013-06-05 Thread Sekhar Nori
Hi Russell, On 3/6/2013 9:45 PM, Matt Porter wrote: Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP (specifically AM33xx) as well. Signed-off-by: Matt Porter mpor...@ti.com Acked-by: Sekhar Nori nsek...@ti.com Can you please ack this movement of davinci private EDMA

Re: Handling NUMA page migration

2013-06-05 Thread Frank Mehnert
On Wednesday 05 June 2013 10:34:13 Frank Mehnert wrote: On Wednesday 05 June 2013 09:54:54 Michal Hocko wrote: On Tue 04-06-13 23:54:45, Frank Mehnert wrote: On Tuesday 04 June 2013 20:17:02 Frank Mehnert wrote: On Tuesday 04 June 2013 16:02:30 Michal Hocko wrote: On Tue 04-06-13

Re: [PATCH] ARM: Orion: Hoist bridge interrupt handling out of the timer

2013-06-05 Thread Sebastian Hesselbarth
On 06/04/13 19:26, Jason Gunthorpe wrote: On Sun, Dec 09, 2012 at 02:06:48PM +0100, Sebastian Hesselbarth wrote: The main irq controller will be required for sure, but for the secondary irq controller we had a discussion long ago. IIRC Gregory proposed to have shared irqs handled by timer and

Re: [PATCH] cpufreq: prevent 'policy-cpus' become offline in __gov_queue_work()

2013-06-05 Thread Michael Wang
Hi, Viresh On 06/05/2013 04:53 PM, Viresh Kumar wrote: [snip] It was caused by the policy-cpus changed during the process of __gov_queue_work(), in other word, cpu offline happened. This patch will use get/put_online_cpus() to prevent the offline happen inside __gov_queue_work(), after

Re: [patch -v4 4/8] memcg: enhance memcg iterator to support predicates

2013-06-05 Thread Tejun Heo
Hey, Michal. On Wed, Jun 05, 2013 at 10:52:39AM +0200, Michal Hocko wrote: One of the core jobs of being a maintainer is ensuring the code stays in readable and maintainable state. As you might know I am playing the maintainer role for around year and a half and there were many

Re: [PATCH v3 0/8] Nvram-to-pstore

2013-06-05 Thread Aruna Balakrishnaiah
Hi Ben, On Saturday 01 June 2013 10:55 AM, Benjamin Herrenschmidt wrote: Another question... Should the core pstore fail to unlink partitions that don't have an -erase callback ? IE. Why would you let anyone erase the OFW common partition for example ? That means that userspace tools can no

[PATCH 0/6] keucr: checkpatch fixes

2013-06-05 Thread Johannes Schilling
hi, we started cleaning up a bit in drivers/staging/keucr, resolved a lot of checkpatch warnings and errors. [PATCH 1/6] keucr: remove some unneccesary typedefs in ../init.c the two unused typedefed structures ADDRESS_T and SSFDCTYPE_T are removed, in other files typedefs are removed and

[PATCH 1/6] keucr: remove some unneccesary typedefs

2013-06-05 Thread Johannes Schilling
resolves checkpatch.pl warning do not add new typedefs Signed-off-by: Laura Lawniczak laura.lawnic...@googlemail.com Signed-off-by: Johannes Schilling of82e...@cip.cs.fau.de --- drivers/staging/keucr/init.c |2 +- drivers/staging/keucr/smil.h | 16

[PATCH 2/6] keucr: fix some camelCase warnings

2013-06-05 Thread Johannes Schilling
resolves checkpatch warning Avoid CamelCase Signed-off-by: Laura Lawniczak laura.lawnic...@googlemail.com Signed-off-by: Johannes Schilling of82e...@cip.cs.fau.de --- drivers/staging/keucr/init.c |8 drivers/staging/keucr/init.h |8 2 files changed, 8 insertions(+), 8

[PATCH 4/6] keucr: make error messages less aggressive

2013-06-05 Thread Johannes Schilling
here's a flower: ⚘. have a very happy day ☺ Signed-off-by: Laura Lawniczak laura.lawnic...@googlemail.com Signed-off-by: Johannes Schilling of82e...@cip.cs.fau.de --- drivers/staging/keucr/smilsub.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

[GIT] Networking

2013-06-05 Thread David Miller
1) Fix timeouts with direct mode authentication in mac80211, from Stanislaw Gruszka. 2) Aggregation sessions can deadlock in ath9k, from Felix Fietkau. 3) Netfilter's xt_addrtype doesn't work with ipv6 due to route lookups creating undesirable cache entries, from Florian Westphal. 4) Fix

Re: [PATCH] vhost_net: clear msg.control for non-zerocopy case during tx

2013-06-05 Thread Michael S. Tsirkin
On Wed, Jun 05, 2013 at 03:40:46PM +0800, Jason Wang wrote: When we decide not use zero-copy, msg.control should be set to NULL otherwise macvtap/tap may set zerocopy callbacks which may decrease the kref of ubufs wrongly. Bug were introduced by commit

[PATCH 6/6] keucr: use more specific max_t(int, ..

2013-06-05 Thread Johannes Schilling
as advised by checkpatch, changed generic max(..) to max_t(int, .. Signed-off-by: Laura Lawniczak laura.lawnic...@googlemail.com Signed-off-by: Johannes Schilling of82e...@cip.cs.fau.de --- drivers/staging/keucr/transport.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 3/6] keucr: migrate printk to dev_dbg/info/warn/err

2013-06-05 Thread Johannes Schilling
as per suggestion of checkpatch.pl; this resolves warnings like Prefer netdev_info .. then dev_info .. to printk. Signed-off-by: Laura Lawniczak laura.lawnic...@googlemail.com Signed-off-by: Johannes Schilling of82e...@cip.cs.fau.de --- drivers/staging/keucr/init.c | 62

[PATCH 5/6] keucr: fix some alignment- and whitespace-problems

2013-06-05 Thread Johannes Schilling
resolves checkpatch errors and warnings regarding whitespace around operators, line lengths and indentation. Signed-off-by: Laura Lawniczak laura.lawnic...@googlemail.com Signed-off-by: Johannes Schilling of82e...@cip.cs.fau.de --- drivers/staging/keucr/scsiglue.c |5 +-

  1   2   3   4   5   6   7   8   9   10   >