[PATCH V3 4/6] vhost_net: determine whether or not to use zerocopy at one time

2013-09-02 Thread Jason Wang
Currently, even if the packet length is smaller than VHOST_GOODCOPY_LEN, if upend_idx != done_idx we still set zcopy_used to true and rollback this choice later. This could be avoided by determining zerocopy once by checking all conditions at one time before. Signed-off-by: Jason Wang

[PATCH V3 1/6] vhost_net: make vhost_zerocopy_signal_used() return void

2013-09-02 Thread Jason Wang
None of its caller use its return value, so let it return void. Signed-off-by: Jason Wang --- drivers/vhost/net.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index 969a859..280ee66 100644 --- a/drivers/vhost/net.c +++ b

[PATCH V3 0/6] vhost code cleanup and minor enhancement

2013-09-02 Thread Jason Wang
check based on Michael's suggestion. Jason Wang (6): vhost_net: make vhost_zerocopy_signal_used() return void vhost_net: use vhost_add_used_and_signal_n() in vhost_zerocopy_signal_used() vhost: switch to use vhost_add_used_n() vhost_net: determine whether or not to use zerocopy a

Re: [PATCH v4 2/3] sched: Consider max cost of idle balance per sched domain

2013-09-03 Thread Jason Low
On Mon, 2013-09-02 at 12:24 +0530, Srikar Dronamraju wrote: > If we face a runq lock contention, then domain_cost can go up. > The runq lock contention could be temporary, but we carry the domain > cost forever (i.e till the next reboot). How about averaging the cost + > penalty for unsuccessful b

[PATCH net-next 1/2] tuntap: purge socket error queue on detach

2013-09-03 Thread Jason Wang
Commit eda297729171fe16bf34fe5b0419dfb69060f623 (tun: Support software transmit time stamping) will queue skbs into error queue when tx stamping is enabled. But it forgets to purge the error queue during detach. This patch fixes this. Cc: Richard Cochran Signed-off-by: Jason Wang --- drivers

[PATCH net-next 2/2] tuntap: orphan frags before trying to set tx timestamp

2013-09-03 Thread Jason Wang
eda297729171fe16bf34fe5b0419dfb69060f623 (tun: Support software transmit time stamping). Cc: Richard Cochran Signed-off-by: Jason Wang --- drivers/net/tun.c |9 + 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/net/tun.c b/drivers/net/tun.c index 2dddb1b..af9a096 100644 --- a/drivers

Re: [PATCH V3 4/6] vhost_net: determine whether or not to use zerocopy at one time

2013-09-25 Thread Jason Wang
On 09/23/2013 03:16 PM, Michael S. Tsirkin wrote: > On Thu, Sep 05, 2013 at 10:54:44AM +0800, Jason Wang wrote: >> > On 09/04/2013 07:59 PM, Michael S. Tsirkin wrote: >>> > > On Mon, Sep 02, 2013 at 04:40:59PM +0800, Jason Wang wrote: >>>> > >> Curr

Re: PCI host merge strategy and maintainers

2013-09-26 Thread Jason Cooper
On Wed, Sep 25, 2013 at 05:28:28PM -0600, Bjorn Helgaas wrote: > There are patches for exynos, imx6, mvebu, and tegra on the PCI list, > and I want to sort out how people expect them to be merged. > > My current assumption is the following: ... > drivers/pci/host/pci-mvebu.c ??

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-26 Thread Jason Low
On Wed, Sep 25, 2013 at 3:10 PM, Tim Chen wrote: > We will need the MCS lock code for doing optimistic spinning for rwsem. > Extracting the MCS code from mutex.c and put into its own file allow us > to reuse this code easily for rwsem. > > Signed-off-by: Tim Chen > Signed-off-by: Davidlohr Bueso

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-26 Thread Jason Low
On Thu, 2013-09-26 at 13:06 -0700, Davidlohr Bueso wrote: > On Thu, 2013-09-26 at 12:27 -0700, Jason Low wrote: > > On Wed, Sep 25, 2013 at 3:10 PM, Tim Chen > > wrote: > > > We will need the MCS lock code for doing optimistic spinning for rwsem. > > > Extract

Re: [PATCH 5/7] KGDB/KDB: add support for external NMI handler to call KGDB/KDB.

2013-09-26 Thread Jason Wessel
gt; Reviewed-by: Dimitri Sivanich > Reviewed-by: Hedi Berriche One problem that I pointed out before and then you can add Acked-by: Jason Wessel > --- > include/linux/kdb.h |1 + > include/linux/kgdb.h|1 + > kernel/debug/debug_core.c | 30

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-26 Thread Jason Low
On Thu, 2013-09-26 at 13:40 -0700, Davidlohr Bueso wrote: > On Thu, 2013-09-26 at 13:23 -0700, Jason Low wrote: > > On Thu, 2013-09-26 at 13:06 -0700, Davidlohr Bueso wrote: > > > On Thu, 2013-09-26 at 12:27 -0700, Jason Low wrote: > > > > On Wed, Sep 25, 2013 at 3:1

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-26 Thread Jason Low
On Thu, 2013-09-26 at 14:41 -0700, Tim Chen wrote: > On Thu, 2013-09-26 at 14:09 -0700, Jason Low wrote: > > On Thu, 2013-09-26 at 13:40 -0700, Davidlohr Bueso wrote: > > > On Thu, 2013-09-26 at 13:23 -0700, Jason Low wrote: > > > > On Thu, 2013-09-26 at 13:0

[PATCH net-next] virtio-net: switch to use XPS to choose txq

2013-09-26 Thread Jason Wang
default mapping when the number of cpus is equal to the number of queues. With XPS support, there's no need for keeping per-cpu vq_index and .ndo_select_queue(), so they were removed also. Cc: Rusty Russell Cc: Michael S. Tsirkin Signed-off-by: Jason Wang --- drivers/net/virtio_net.c |

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-26 Thread Jason Low
On Fri, 2013-09-27 at 08:02 +0200, Ingo Molnar wrote: > * Tim Chen wrote: > > > > If we prefer to optimize this a bit though, perhaps we can first move > > > the node->lock = 0 so that it gets executed after the "if (likely(prev > > > == NULL)) {}" code block and then delete "node->lock = 1" in

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-27 Thread Jason Low
ssignment so that it occurs after the if (likely(prev == NULL)) check. This might also help make it clearer as to how the node->locked variable is used in MCS locks. Signed-off-by: Jason Low --- include/linux/mcslock.h |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/inc

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-27 Thread Jason Low
On Fri, Sep 27, 2013 at 12:38 PM, Tim Chen wrote: > BTW, is the above memory barrier necessary? It seems like the xchg > instruction already provided a memory barrier. > > Now if we made the changes that Jason suggested: > > > /* Init node */ > - node->

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-27 Thread Jason Low
ry barrier so that it is before the "ACCESS_ONCE(next->locked) = 1;". Signed-off-by: Jason Low Signed-off-by: Paul E. McKenney Signed-off-by: Tim Chen --- include/linux/mcslock.h |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/include/linux/mcslock.h

Re: [PATCH v4 1/4] ARM: dts: am335x-bone: add CD for mmc1

2013-09-27 Thread Jason Kridner
On Thu, Sep 12, 2013 at 2:35 PM, Koen Kooi wrote: > From: Alexander Holler > > This enables the use of MMC cards even when no card was inserted at boot. > > Signed-off-by: Alexander Holler > Signed-off-by: Koen Kooi Acked-by: Jason Kridner > --- > arch/arm/boot/dts

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-27 Thread Jason Low
On Fri, Sep 27, 2013 at 7:19 PM, Paul E. McKenney wrote: > On Fri, Sep 27, 2013 at 04:54:06PM -0700, Jason Low wrote: >> On Fri, Sep 27, 2013 at 4:01 PM, Paul E. McKenney >> wrote: >> > Yep. The previous lock holder's smp_wmb() won't keep either the compil

Re: [PATCH net-next] virtio-net: switch to use XPS to choose txq

2013-09-28 Thread Jason Wang
On 09/27/2013 10:35 PM, Michael S. Tsirkin wrote: > On Fri, Sep 27, 2013 at 01:57:24PM +0800, Jason Wang wrote: >> We used to use a percpu structure vq_index to record the cpu to queue >> mapping, this is suboptimal since it duplicates the work of XPS and >> loses all other X

Re: [PATCH V3 4/6] vhost_net: determine whether or not to use zerocopy at one time

2013-09-29 Thread Jason Wang
On 09/26/2013 12:30 PM, Jason Wang wrote: > On 09/23/2013 03:16 PM, Michael S. Tsirkin wrote: >> > On Thu, Sep 05, 2013 at 10:54:44AM +0800, Jason Wang wrote: >>>> >> > On 09/04/2013 07:59 PM, Michael S. Tsirkin wrote: >>>>>> >>> &g

Re: [PATCH net-next] virtio-net: switch to use XPS to choose txq

2013-09-29 Thread Jason Wang
On 09/30/2013 07:35 AM, Rusty Russell wrote: > Jason Wang writes: >> We used to use a percpu structure vq_index to record the cpu to queue >> mapping, this is suboptimal since it duplicates the work of XPS and >> loses all other XPS functionality such as allowing use to c

[net-next PATCH V2] virtio-net: switch to use XPS to choose txq

2013-09-30 Thread Jason Wang
default mapping when the number of cpus is equal to the number of queues. With XPS support, there's no need for keeping per-cpu vq_index and .ndo_select_queue(), so they were removed also. Cc: Rusty Russell Cc: Michael S. Tsirkin Signed-off-by: Jason Wang --- Changes from V1: - use cpuma

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-30 Thread Jason Low
On Mon, 2013-09-30 at 11:51 -0400, Waiman Long wrote: > On 09/28/2013 12:34 AM, Jason Low wrote: > >> Also, below is what the mcs_spin_lock() and mcs_spin_unlock() > >> functions would look like after applying the proposed changes. > >> > >> static

Re: [RFC PATCH] fpga: Introduce new fpga subsystem

2013-09-30 Thread Jason Gunthorpe
g state data from fpga_config_data to the configuration bus and userspace writes 'operating' once the stream is done to perform the post-configuration actions. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a mess

Re: [tpmdd-devel] [PATCH 09/13] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-09-30 Thread Jason Gunthorpe
On Tue, Sep 24, 2013 at 10:28:41AM -0400, Daniel De Graaf wrote: > On 09/23/2013 06:23 PM, Jason Gunthorpe wrote: > >On Mon, Sep 23, 2013 at 06:00:46PM -0400, Daniel De Graaf wrote: > > > >>In a PC client TPM, normal OS code (as opposed to firmware or microcode) >

Sparse files, sendfile and tmpfs ENOSPC

2013-09-30 Thread Jason Gunthorpe
his problem.. I find this surprising since I would have thought the hole would have just map'd the zero page multiple times? Is this an accounting error someplace? I've seen Hugh's comments in past threads that this area is very complex.. Regards, Jason #include #include #inclu

Re: [tpmdd-devel] [PATCH 09/13] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-09-30 Thread Jason Gunthorpe
ul to talk about how the > ioctl portion of this would need to be set up - if someone is actually > needed. It would be nice to have a user! As I said, we don't use it here. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a messa

[PATCH 0/3] epoll: reduce 'epmutex' lock contention

2013-09-30 Thread Jason Baron
hen under the full load of the benchmark. ... Currently the benchmark stops scaling at around 40-44 sockets but it seems like I found a second unrelated bottleneck. " Jason Baron (3): epoll: optimize EPOLL_CTL_DEL using rcu epoll: Do not take global 'epmutex' for si

[PATCH 1/3] epoll: optimize EPOLL_CTL_DEL using rcu

2013-09-30 Thread Jason Baron
o not take global 'epmutex' for simple topologies", shows a dramatic performance improvement in scalability for SPECjbb. Tested-by: Nathan Zimmer Signed-off-by: Jason Baron --- fs/eventpoll.c | 58 -- 1 file changed, 32 insertions(

[PATCH 3/3] epoll: restore 'struct epitem' size

2013-09-30 Thread Jason Baron
'struct rcu_head'. Note, I've kept this separate from 'epoll: optimize EPOLL_CTL_DEL using rcu' in order to make clear the hack-ish nature of this thing. Tested-by: Nathan Zimmer Signed-off-by: Jason Baron --- fs/eventpoll.c | 32 ++-- 1 file

[PATCH 2/3] epoll: Do not take global 'epmutex' for simple topologies

2013-09-30 Thread Jason Baron
well on just over 40 sockets. ... Currently the benchmark stops scaling at around 40-44 sockets but it seems like I found a second unrelated bottleneck. " Tested-by: Nathan Zimmer Signed-off-by: Jason Baron --- fs/eventpoll.c | 94 ++ 1 file

Re: [PATCH 11/26] ARM: dove: remove custom .init_time hook

2013-09-23 Thread Jason Cooper
h arch/arm calling of_clk_init(NULL) from time_init(), we can now > > >>remove custom .init_time hooks. While at it, also remove some obsolete > > >>includes. > > >> > > >>Signed-off-by: Sebastian Hesselbarth > > >>--- > > >&g

Re: [PATCH 15/26] ARM: kirkwood: remove custom .init_time hook

2013-09-23 Thread Jason Cooper
f-by: Sebastian Hesselbarth > > --- > > Cc: Olof Johansson > > Cc: Arnd Bergmann > > Cc: Jason Cooper > > Cc: Andrew Lunn > > Cc: Russell King > > Cc: linux-arm-ker...@lists.infradead.org > > Cc: linux-kernel@vger.kernel.org > > --- > &

Re: [RFC] eventpoll: Move a kmem_cache_alloc and kmem_cache_free

2013-09-23 Thread Jason Baron
On 09/19/2013 12:37 PM, Nathan Zimmer wrote: > On 09/18/2013 02:09 PM, Jason Baron wrote: >> On 09/13/2013 11:54 AM, Nathan Zimmer wrote: >>> We noticed some scaling issue in the SPECjbb benchmark. Running perf >>> we found that the it was spending lots of time in SYS_e

Re: [RFC] eventpoll: Move a kmem_cache_alloc and kmem_cache_free

2013-09-23 Thread Jason Baron
On 09/22/2013 04:41 PM, Eric Wong wrote: > Jason Baron wrote: >> epoll: reduce usage of global 'epmutex' lock >> >> Epoll file descriptors that are 1 link from a wakeup source and >> are not nested within other epoll descriptors, or pointing to

Re: [RFC PATCH] fpga: Introduce new fpga subsystem

2013-09-23 Thread Jason Gunthorpe
use request firmware at the driver level where the driver somehow knows what FPGA to request. > and there will be this option. But don't try to restrict others > if they want to do it just in the kernel in early phase. Doing the load in kernel early phase doesn't involve the user space s

[PATCH 08/13] tpm: Pull everything related to /dev/tpmX into tpm-dev.c

2013-09-23 Thread Jason Gunthorpe
CLASS-dev.c is a common idiom for Linux subsystems This pulls all the code related to the miscdev into tpm-dev.c and makes it static. The identical file_operation structs in the drivers are purged and the tpm common code unconditionally creates the miscdev. Signed-off-by: Jason Gunthorpe

[PATCH 12/13] tpm: st33: Remove chip->data_buffer access from this driver

2013-09-23 Thread Jason Gunthorpe
For some reason this driver thinks that chip->data_buffer needs to be set before it can call tpm_pm_*. This is not true. data_buffer is used only by /dev/tpmX, which is why it is managed exclusively by the fops functions. Cc: Mathias Leblanc Signed-off-by: Jason Gunthorpe --- drivers/char/

[PATCH 13/13] tpm: Make tpm-dev allocate a per-file structure

2013-09-23 Thread Jason Gunthorpe
This consolidates everything that is only used within tpm-dev.c into tpm-dev.c and out of the publicly visible struct tpm_chip. The per-file allocation lays the ground work for someday fixing the strange forced O_EXCL behaviour of the current code. Signed-off-by: Jason Gunthorpe --- drivers

[PATCH 07/13] tpm: Remove tpm_show_caps_1_2

2013-09-23 Thread Jason Gunthorpe
interface the TPM supports at run-time. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/tpm.c | 56 +++-- drivers/char/tpm/tpm.h | 2 -- drivers/char/tpm/tpm_i2c_atmel.c| 2 +- drivers/char/tpm/tpm_i2c_infineon.c | 2 +- drivers/char

[PATCH 06/13] tpm: Remove redundant dev_set_drvdata

2013-09-23 Thread Jason Gunthorpe
TPM drivers should not call dev_set_drvdata (or aliases), only the core code is allowed to call dev_set_drvdata, and it does it during tpm_register_hardware. These extra sets are harmless, but are an anti-pattern that many drivers have copied. Signed-off-by: Jason Gunthorpe --- drivers/char

[PATCH 05/13] tpm: Use container_of to locate the tpm_chip in tpm_open

2013-09-23 Thread Jason Gunthorpe
misc_open sets the file->private_date to the misc_dev when calling open. We can use container_of to go from the misc_dev back to the tpm_chip. Future clean ups will move tpm_open into a new file and this change means we do not have to export the tpm_chip list. Signed-off-by: Jason Guntho

[PATCH 11/13] tpm: Use the ops structure instead of a copy in tpm_vendor_specific

2013-09-23 Thread Jason Gunthorpe
This builds on the last commit to use the ops structure in the core and reduce the size of tpm_vendor_specific. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/tpm-sysfs.c| 2 +- drivers/char/tpm/tpm.c | 35 +-- drivers/char/tpm/tpm.h

[PATCH 10/13] tpm: Create a tpm_class_ops structure and use it in the drivers

2013-09-23 Thread Jason Gunthorpe
This replaces the static initialization of a tpm_vendor_specific structure in the drivers with the standard Linux idiom of providing a const structure of function pointers. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/tpm.c | 10 -- drivers/char/tpm/tpm.h

[PATCH 03/13] tpm: xen-tpmfront: Fix default durations

2013-09-23 Thread Jason Gunthorpe
). Just remove them. Cc: Daniel De Graaf Cc: Konrad Rzeszutek Wilk Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/xen-tpmfront.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/char/tpm/xen-tpmfront.c b/drivers/char/tpm/xen-tpmfront.c index 7a7929b..6f2fe2b 100644 --- a

[PATCH 00/13] TPM cleanup

2013-09-23 Thread Jason Gunthorpe
e device startup (tpm_startup, tpm_get_timeoutes) into core code If this series is successful I may be able to do some of the above as well. Jason Gunthorpe (13): tpm: ibmvtpm: Use %zd formatting for size_t format arguments tpm atmel: Call request_region with the correct base tpm: xen-tpmfr

[PATCH 01/13] tpm: ibmvtpm: Use %zd formatting for size_t format arguments

2013-09-23 Thread Jason Gunthorpe
This suppresses compile warnings on 32 bit builds. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/tpm_ibmvtpm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/char/tpm/tpm_ibmvtpm.c b/drivers/char/tpm/tpm_ibmvtpm.c index 56b07c3..838f043 100644 --- a

[PATCH 02/13] tpm atmel: Call request_region with the correct base

2013-09-23 Thread Jason Gunthorpe
request_region being called with 0 as the base address. I don't know if request_region(0, ..) will fail, if so the driver has been broken since 2006 and we should remove it from the tree as it has no users. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/tpm_atmel.c | 2 +- 1 file chang

[PATCH 04/13] tpm: Store devname in the tpm_chip

2013-09-23 Thread Jason Gunthorpe
Just put the memory directly in the chip structure, rather than in a 2nd dedicated kmalloc. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/tpm.c | 17 ++--- drivers/char/tpm/tpm.h | 1 + 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/char/tpm/tpm.c b

[PATCH 09/13] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-09-23 Thread Jason Gunthorpe
CLASS-sysfs.c is a common idiom for linux subsystems. This pulls all the sysfs attribute functions and related code into tpm-sysfs.c. To support this change some constants are moved from tpm.c to tpm.h and __tpm_pcr_read is made non-static and is called tpm_pcr_read_dev. Signed-off-by: Jason

Re: [tpmdd-devel] [PATCH 09/13] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-09-23 Thread Jason Gunthorpe
On Mon, Sep 23, 2013 at 02:54:21PM -0400, Daniel De Graaf wrote: > On 09/23/2013 02:14 PM, Jason Gunthorpe wrote: > >CLASS-sysfs.c is a common idiom for linux subsystems. > > > >This pulls all the sysfs attribute functions and related code > >into tpm-sysfs.c. To support

Re: [tpmdd-devel] [PATCH 09/13] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-09-23 Thread Jason Gunthorpe
argument. Some thought is needed to determine what locality in-kernel users should be using. I suspect userspace and kernel space should not be forced to the same locality. Should user space be restricted to a subset of localities? What use models do you see with the security label binding mec

[PATCH] tpm: xen-tpmfront: Remove the locality sysfs attribute

2013-09-23 Thread Jason Gunthorpe
The hope is to have a well defined locality API that all the other locality aware drivers can use, perhaps in 3.13. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/xen-tpmfront.c | 29 - 1 file changed, 29 deletions(-) Daniel, if you can Ack this.. Konrad: T

Re: [tpmdd-devel] [PATCH 09/13] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-09-23 Thread Jason Gunthorpe
. Any thoughts on root vs not-root? Would middelware want to use localities? Do you know anyone on the userspace SW side who could look at this? Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More ma

Re: [PATCH v5 1/6] rwsem: check the lock before cpmxchg in down_write_trylock

2013-09-24 Thread Jason Low
Should we do something similar with __down_read_trylock, such as the following? Signed-off-by: Jason Low --- include/asm-generic/rwsem.h |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/include/asm-generic/rwsem.h b/include/asm-generic/rwsem.h index bb1e2cd..47990dc

Re: [PATCH 0/5] dev_ and dynamic_debug cleanups

2012-09-06 Thread Jason Baron
refer have this in one place. Can we add a 'prefix' argument to __dev_printk(), and __netdev_printk() that dynamic debug can use, but is simply empty for dev_printk() and netdev_printk(). Thanks, -Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

Re: [PATCH 0/5] dev_ and dynamic_debug cleanups

2012-09-06 Thread Jason Baron
> >> dev: Add dev_vprintk_emit and dev_printk_emit > > >> device and dynamic_debug: Use dev_vprintk_emit and dev_printk_emit > > >> > > > > > > Ive tested this on 2 builds differing only by DYNAMIC_DEBUG > > > It works for me on x86-64 > >

Re: [PATCH 0/5] dev_ and dynamic_debug cleanups

2012-09-07 Thread Jason Baron
On Thu, Sep 06, 2012 at 11:43:46AM -0700, Joe Perches wrote: > On Thu, 2012-09-06 at 13:51 -0400, Jason Baron wrote: > > On Sun, Aug 26, 2012 at 04:25:25AM -0700, Joe Perches wrote: > > > The recent commit to fix dynamic_debug was a bit unclean. > > > Neaten

Re: [PATCH 0/5] dev_ and dynamic_debug cleanups

2012-09-07 Thread Jason Baron
On Fri, Sep 07, 2012 at 08:12:01AM -0700, Joe Perches wrote: > On Fri, 2012-09-07 at 10:52 -0400, Jason Baron wrote: > > On Thu, Sep 06, 2012 at 11:43:46AM -0700, Joe Perches wrote: > > > On Thu, 2012-09-06 at 13:51 -0400, Jason Baron wrote: > > > > On Sun, Aug 26,

Re: [PATCH v2 1/9] pinctrl: mvebu: pinctrl driver core

2012-09-09 Thread Jason Cooper
On Sun, Sep 02, 2012 at 10:27:29AM +0200, Sebastian Hesselbarth wrote: > there is a v3 of the patch under internal review right now. Sebastian, What is the status of v3? I'd like to get it (and gpio, which depends on this work) in for v3.7, but it's getting close... thx,

Re: [PATCH v3 0/6] ARM: dove: DT support for Marvell Dove

2012-09-09 Thread Jason Cooper
ts > create mode 100644 arch/arm/boot/dts/dove-cubox.dts > create mode 100644 arch/arm/boot/dts/dove-dove-db.dts > create mode 100644 arch/arm/boot/dts/dove.dtsi > > Signed-off-by: Sebastian Hesselbarth > Acked-by: Arnd Bergmann Whole series applied to: git://git.infradead.org

Re: [PATCH 0/5] dev_ and dynamic_debug cleanups

2012-09-10 Thread Jason Baron
On Fri, Sep 07, 2012 at 06:55:50PM -0700, Joe Perches wrote: > On Fri, 2012-09-07 at 11:35 -0400, Jason Baron wrote: > > If nobody else thinks this patch is better, let's at least add a comment in > > __dev_printk() and __netdev_printk() to fix dynamic debug, if these are

Re: [PATCH] [ARM] Use AT() in the linker script to create correct program headers

2012-10-09 Thread Jason Gunthorpe
a. Some other arches page protect the kernel, but that tends to be at odds with the desire to use huge pages for the kernel mapping, and independent of the load headers.. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord

Re: [PATCH] [ARM] Use AT() in the linker script to create correct program headers

2012-10-09 Thread Jason Gunthorpe
be discussed via devicetree-discuss. > > If it is doing anything less trivial than picking a pre-baked DT, the > rationale would need to be carefully argued. I'm not sure there is a great interest in this? What are other folks working on production embedded stuff doing? I suppose t

Re: [PATCH] of: When constructing the bus id consider assigned-addresses as well

2012-11-30 Thread Jason Gunthorpe
ement with the current kernel code is the device_type=pci on soc_bridge. This makes of_match_bus(parent) for soc_device return the PCI structure, which has '.addresses = "assigned-addresses",' So.. how would you like this to look? Jason -- To unsubscribe from this list: send th

Re: [PATCH 1/2] ARM: Dove: Add pinctrl clock to DT

2012-12-01 Thread Jason Cooper
ned-off-by: Sebastian Hesselbarth > > Acked-by: Linus Walleij > > So who's supposed to merge this patch? Me? The MVEBU-maintainer? I'll take it and merge it in once v3.8-rc1 drops. thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel

Re: [PATCH 1/2] ARM: Dove: Add pinctrl clock to DT

2012-12-01 Thread Jason Cooper
On Sat, Dec 01, 2012 at 07:15:55PM +0100, Sebastian Hesselbarth wrote: > On 12/01/2012 07:13 PM, Jason Cooper wrote: > >On Sat, Dec 01, 2012 at 06:07:06PM +0100, Linus Walleij wrote: > >>On Mon, Nov 26, 2012 at 10:18 AM, Sebastian Hesselbarth > >> wrote: > >

[PATCH net-next] tuntap: attach queue 0 before registering netdevice

2012-12-02 Thread Jason Wang
then remove 2046. Solve this problem by attaching queue 0 and set the real number of queues before registering netdevice. Reported-by: Jiri Slaby Tested-by: Jiri Slaby Signed-off-by: Jason Wang --- drivers/net/tun.c | 11 +-- 1 files changed, 5 insertions(+), 6 deletions(-) diff

Re: [net-next rfc v7 2/3] virtio_net: multiqueue support

2012-12-02 Thread Jason Wang
On Sunday, December 02, 2012 06:06:31 PM Michael S. Tsirkin wrote: > On Tue, Nov 27, 2012 at 06:15:59PM +0800, Jason Wang wrote: > > This addes multiqueue support to virtio_net driver. In multiple queue > > modes, the driver expects the number of queue paris is equal to the > >

Re: [net-next rfc v7 2/3] virtio_net: multiqueue support

2012-12-02 Thread Jason Wang
On Monday, December 03, 2012 12:34:08 PM Rusty Russell wrote: > Jason Wang writes: > > +static const struct ethtool_ops virtnet_ethtool_ops; > > + > > +/* > > + * Converting between virtqueue no. and kernel tx/rx queue no. > > + * 0:rx0 1:tx0 2:cvq

Re: [net-next rfc v7 3/3] virtio-net: change the number of queues through ethtool

2012-12-02 Thread Jason Wang
On Sunday, December 02, 2012 06:09:06 PM Michael S. Tsirkin wrote: > On Tue, Nov 27, 2012 at 06:16:00PM +0800, Jason Wang wrote: > > This patch implement the {set|get}_channels method of ethool to allow user > > to change the number of queues dymaically when the device is running.

Re: [net-next RFC] pktgen: don't wait for the device who doesn't free skb immediately after sent

2012-12-02 Thread Jason Wang
On Tuesday, November 27, 2012 08:49:19 AM Stephen Hemminger wrote: > On Tue, 27 Nov 2012 14:45:13 +0800 > > Jason Wang wrote: > > On 11/27/2012 01:37 AM, Stephen Hemminger wrote: > > > On Mon, 26 Nov 2012 15:56:52 +0800 > > > > > > Jason Wang wrote:

Re: [net-next rfc v7 2/3] virtio_net: multiqueue support

2012-12-03 Thread Jason Wang
On 12/03/2012 05:47 PM, Michael S. Tsirkin wrote: > On Mon, Dec 03, 2012 at 02:05:27PM +0800, Jason Wang wrote: >> On Monday, December 03, 2012 12:34:08 PM Rusty Russell wrote: >>> Jason Wang writes: >>>> +static const struct ethtool_ops virtnet_ethtool_ops; >&

Re: [net-next rfc v7 2/3] virtio_net: multiqueue support

2012-12-03 Thread Jason Wang
On 12/03/2012 06:14 PM, Michael S. Tsirkin wrote: > On Tue, Nov 27, 2012 at 06:15:59PM +0800, Jason Wang wrote: >> > - if (!try_fill_recv(&vi->rq, GFP_KERNEL)) >> > - schedule_delayed_work(&vi->rq.refill, 0); >> > + for (i = 0; i &l

Re: [net-next rfc v7 1/3] virtio-net: separate fields of sending/receiving queue from virtnet_info

2012-12-04 Thread Jason Wang
On Monday, December 03, 2012 01:18:48 PM Michael S. Tsirkin wrote: > On Mon, Dec 03, 2012 at 01:15:01PM +0800, Jason Wang wrote: > > > > + > > > > > > > > + /* Work struct for refilling if we run low on memory. */ > > > > > > > > +

Re: [net-next rfc v7 1/3] virtio-net: separate fields of sending/receiving queue from virtnet_info

2012-12-04 Thread Jason Wang
On Tuesday, December 04, 2012 02:13:11 PM Rusty Russell wrote: > Jason Wang writes: > > On Monday, December 03, 2012 12:25:42 PM Rusty Russell wrote: > >> > + > >> > +/* Work struct for refilling if we run low on memory. */ > >> > +s

Re: [net-next rfc v7 2/3] virtio_net: multiqueue support

2012-12-04 Thread Jason Wang
On Monday, December 03, 2012 01:11:18 PM Michael S. Tsirkin wrote: > On Mon, Dec 03, 2012 at 06:01:58PM +0800, Jason Wang wrote: > > On 12/03/2012 05:47 PM, Michael S. Tsirkin wrote: > > > On Mon, Dec 03, 2012 at 02:05:27PM +0800, Jason Wang wrote: > > >> On Monday,

Re: [net-next rfc v7 2/3] virtio_net: multiqueue support

2012-12-04 Thread Jason Wang
On Tuesday, December 04, 2012 09:35:03 AM Michael S. Tsirkin wrote: > On Mon, Dec 03, 2012 at 06:30:49PM +0800, Jason Wang wrote: > > On 12/03/2012 06:14 PM, Michael S. Tsirkin wrote: > > > On Tue, Nov 27, 2012 at 06:15:59PM +0800, Jason Wang wrote: > > >> >

[PATCH net-next 0/3] Multiqueue support for virtio-net

2012-12-04 Thread Jason Wang
ml.org/lkml/2012/6/25/120 - RFC V2: http://lwn.net/Articles/467283/ Perf Numbers: Will do some basic test and post as a reply to this mail. Jason Wang (3): virtio-net: separate fields of sending/receiving queue from virtnet_info virtio_net: multiqueue support virtio-net: change the nu

[PATCH net-next 1/3] virtio-net: separate fields of sending/receiving queue from virtnet_info

2012-12-04 Thread Jason Wang
-off-by: Jason Wang --- drivers/net/virtio_net.c | 271 +- 1 files changed, 149 insertions(+), 122 deletions(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index 8262232..266f712 100644 --- a/drivers/net/virtio_net.c +++ b/drivers

[PATCH net-next 3/3] virtio-net: change the number of queues through ethtool

2012-12-04 Thread Jason Wang
This patch implement the ethtool_{set|get}_channels method of ethool to allow user to change the number of queues dymaically when the device is running. This would let the user to configure it on demand. Signed-off-by: Jason Wang --- drivers/net/virtio_net.c | 44

[PATCH net-next 2/3] virtio_net: multiqueue support

2012-12-04 Thread Jason Wang
processor id. - smp affinity hint were set to the vcpu that owns the queue pairs. Signed-off-by: Krishna Kumar Signed-off-by: Jason Wang --- drivers/net/virtio_net.c| 472 ++- include/uapi/linux/virtio_net.h | 16 ++ 2 files changed, 385 insertions

Re: [net-next RFC] pktgen: don't wait for the device who doesn't free skb immediately after sent

2012-12-04 Thread Jason Wang
On Monday, December 03, 2012 08:01:11 AM Stephen Hemminger wrote: > On Mon, 03 Dec 2012 14:45:46 +0800 > > Jason Wang wrote: > > On Tuesday, November 27, 2012 08:49:19 AM Stephen Hemminger wrote: > > > On Tue, 27 Nov 2012 14:45:13 +0800 > > > > > > Jaso

Re: [PATCH net-next 2/3] virtio_net: multiqueue support

2012-12-04 Thread Jason Wang
On Tuesday, December 04, 2012 03:24:22 PM Michael S. Tsirkin wrote: > I found some bugs, see below. > Also some style nitpicking, this is not mandatory to address. Thanks for the reviewing. > > On Tue, Dec 04, 2012 at 07:07:57PM +0800, Jason Wang wrote: > > This addes mul

Re: [PATCH net-next 3/3] virtio-net: change the number of queues through ethtool

2012-12-04 Thread Jason Wang
On Tuesday, December 04, 2012 03:49:59 PM Michael S. Tsirkin wrote: > On Tue, Dec 04, 2012 at 07:07:58PM +0800, Jason Wang wrote: > > This patch implement the ethtool_{set|get}_channels method of ethool to > > allow user to change the number of queues dymaically when the device

Re: [PATCH net-next 2/3] virtio_net: multiqueue support

2012-12-04 Thread Jason Wang
On 12/04/2012 11:11 PM, Michael S. Tsirkin wrote: > On Tue, Dec 04, 2012 at 10:45:33PM +0800, Jason Wang wrote: >> On Tuesday, December 04, 2012 03:24:22 PM Michael S. Tsirkin wrote: >>> I found some bugs, see below. >>> Also some style nitpicking, this is not mandator

[PATCH net-next v2 0/3] Multiqueue support in virtio-net

2012-12-05 Thread Jason Wang
V2: http://lwn.net/Articles/467283/ Perf Numbers: Will do some basic test and post as a reply to this mail. Jason Wang (3): virtio-net: separate fields of sending/receiving queue from virtnet_info virtio_net: multiqueue support virtio-net: support changing the number of queue pairs t

[PATCH net-next v2 1/3] virtio-net: separate fields of sending/receiving queue from virtnet_info

2012-12-05 Thread Jason Wang
-off-by: Jason Wang --- drivers/net/virtio_net.c | 282 ++ 1 files changed, 158 insertions(+), 124 deletions(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index 8262232..0dcaee7 100644 --- a/drivers/net/virtio_net.c +++ b/drivers

[PATCH net-next v2 3/3] virtio-net: support changing the number of queue pairs through ethtool

2012-12-05 Thread Jason Wang
This patch implements the ethtool_{set|get}_channels method of virtio-net to allow user to change the number of queues when the device is running on demand. Signed-off-by: Jason Wang --- drivers/net/virtio_net.c | 43 +++ 1 files changed, 43 insertions

[PATCH net-next v2 2/3] virtio_net: multiqueue support

2012-12-05 Thread Jason Wang
. - smp affinity hint to the cpu that owns the queue pairs. This could be used with the flow steering support of the device to guarantee the packets of a single flow is handled by the same cpu. Signed-off-by: Krishna Kumar Signed-off-by: Jason Wang --- drivers/net/virtio_net.c| 473

[PATCH 1/3] dynamic_debug: Fix vpr_ logging styles

2012-12-05 Thread Jason Baron
From: Joe Perches vpr_info_dq should be a function and vpr_info should have a do {} while (0) Add missing newlines to pr_s. Miscellaneous neatening too. braces, coalescing formats, alignments, etc... Signed-off-by: Joe Perches Signed-off-by: Jason Baron --- lib/dynamic_debug.c | 118

[PATCH 3/3] dynamic_debug: add pr_errs before -EINVALs

2012-12-05 Thread Jason Baron
-off-by: Jason Baron --- lib/dynamic_debug.c | 47 +++ 1 files changed, 35 insertions(+), 12 deletions(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index c0869f1..da820f2 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_debug.c

[PATCH 2/3] dynamic_debug: dynamic hex dump

2012-12-05 Thread Jason Baron
ong prefix would impact readability. To provide line/file etc. information, use pr_debug or similar before/after print_hex_dump_debug() Signed-off-by: Vladimir Kondratiev Signed-off-by: Jason Baron --- Documentation/dynamic-debug-howto.txt | 15 +-- include/linux/dynamic_debug.h

[PATCH 0/3] dynamic_debug: Add print_hex_dump_bytes/debug support and cleanups

2012-12-05 Thread Jason Baron
Hi Greg, Here's a collection of the latest dyanmic debug patches that I have pending. Thanks, -Jason Jim Cromie (1): dynamic_debug: add pr_errs before -EINVALs Joe Perches (1): dynamic_debug: Fix vpr_ logging styles Vladimir Kondratiev (1): dynamic_debug: dynamic hex

Re: [PATCH 0/3] dynamic_debug: Add print_hex_dump_bytes/debug support and cleanups

2012-12-05 Thread Jason Baron
On Wed, Dec 05, 2012 at 02:05:06PM -0800, Joe Perches wrote: > On Wed, 2012-12-05 at 16:48 -0500, Jason Baron wrote: > > Here's a collection of the latest dynamic debug patches that I have > > pending. > > Any update on the jump table support? > I have patch

Re: [PATCH V3 0/3] Add clock framework for armada 370/XP

2012-10-17 Thread Jason Cooper
clk-core.h > create mode 100644 drivers/clk/mvebu/clk-cpu.c > create mode 100644 drivers/clk/mvebu/clk-cpu.h > create mode 100644 drivers/clk/mvebu/clk.c Would it be okay if we took this through the mvebu tree? It looks like the only potential merge conflict we would have would be in dri

Re: [PATCH] dynamic_debug: Remove unnecessary __used

2012-10-18 Thread Jason Baron
debug __aligned(8) \ > __attribute__((section("__verbose"))) name = { \ > .modname = KBUILD_MODNAME, \ > .function = __func__, \ > > Greg, Pls pull this. Acked-by: Ja

Re: [PULL REQ] IXP4xx changes for Linux 3.7

2012-10-19 Thread Jason Cooper
nslu2, gateworks boards). This way, you could see the process first hand without having to do the labor. We could do this for a few release cycles so you can see how it goes. After that, we can reassess things. If you still don't wish to lend your experience, so be it. But I hope we can c

<    1   2   3   4   5   6   7   8   9   10   >