[PATCH v2 0/5] perf report: Show inline stack

2016-12-06 Thread Jin Yao
v2: Thanks so much for Arnaldo's comments! The modifications are: 1. Divide v1 patch "perf report: Find the inline stack for a given address" into 2 patches: a. perf report: Refactor common code in srcline.c b. perf report: Find the inline stack for a given address

[PATCH v2 1/5] perf report: Refactor common code in srcline.c

2016-12-06 Thread Jin Yao
Introduce dso__name() and filename_split() out of existing code because these codes will be used in several places in next patch. For filename_split(), it may also solve a potential memory leak in existing code. In existing addr2line(), sep = strchr(filename, ':'); if (sep) {

[PATCH v2 3/5] perf report: Create a new option "--inline"

2016-12-06 Thread Jin Yao
It takes some time to look for inline stack for callgraph addresses. So it provides a new option "--inline" to let user decide if enable this feature. Signed-off-by: Jin Yao --- tools/perf/Documentation/perf-report.txt | 4 tools/perf/builtin-report.c

[PATCH v2 3/5] perf report: Create a new option "--inline"

2016-12-06 Thread Jin Yao
It takes some time to look for inline stack for callgraph addresses. So it provides a new option "--inline" to let user decide if enable this feature. Signed-off-by: Jin Yao --- tools/perf/Documentation/perf-report.txt | 4 tools/perf/builtin-report.c | 2 ++

[PATCH v2 5/5] perf report: Show inline stack in browser mode

2016-12-06 Thread Jin Yao
For example: -0.05% test2test2 [.] main /home/jinyao/perf-dev/test/test2.c:27 (inline) /home/jinyao/perf-dev/test/test2.c:35 (inline) /home/jinyao/perf-dev/test/test2.c:45 (inline) /home/jinyao/perf-dev/test/test2.c:61 (inline) Signed-off-by: Jin Yao

[PATCH v2 5/5] perf report: Show inline stack in browser mode

2016-12-06 Thread Jin Yao
For example: -0.05% test2test2 [.] main /home/jinyao/perf-dev/test/test2.c:27 (inline) /home/jinyao/perf-dev/test/test2.c:35 (inline) /home/jinyao/perf-dev/test/test2.c:45 (inline) /home/jinyao/perf-dev/test/test2.c:61 (inline) Signed-off-by: Jin Yao

[PATCH v2 4/5] perf report: Show inline stack in stdio mode

2016-12-06 Thread Jin Yao
If the address belongs to an inlined function, the source information back to the first non-inlined function will be printed. For example: 0.05% test2test2 [.] main | ---/home/jinyao/perf-dev/test/test2.c:27 (inline) /home/jinyao/perf-dev/test/test2.c:35

[PATCH v2 4/5] perf report: Show inline stack in stdio mode

2016-12-06 Thread Jin Yao
If the address belongs to an inlined function, the source information back to the first non-inlined function will be printed. For example: 0.05% test2test2 [.] main | ---/home/jinyao/perf-dev/test/test2.c:27 (inline) /home/jinyao/perf-dev/test/test2.c:35

[PATCH v2 2/5] perf report: Find the inline stack for a given address

2016-12-06 Thread Jin Yao
It would be useful for perf to support a mode to query the inline stack for a given callgraph address. This would simplify finding the right code in code that does a lot of inlining. The srcline.c has contained the code which supports to translate the address to filename:line_nr. This patch just

[PATCH v2 2/5] perf report: Find the inline stack for a given address

2016-12-06 Thread Jin Yao
It would be useful for perf to support a mode to query the inline stack for a given callgraph address. This would simplify finding the right code in code that does a lot of inlining. The srcline.c has contained the code which supports to translate the address to filename:line_nr. This patch just

Re: [PATCH 10/10] virtio: enable endian checks for sparse builds

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:41, Michael S. Tsirkin wrote: __CHECK_ENDIAN__ isn't on by default presumably because it triggers too many sparse warnings for correct code. But virtio is now clean of these warnings, and we want to keep it this way - enable this for sparse builds. Signed-off-by: Michael S.

Re: [PATCH 10/10] virtio: enable endian checks for sparse builds

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:41, Michael S. Tsirkin wrote: __CHECK_ENDIAN__ isn't on by default presumably because it triggers too many sparse warnings for correct code. But virtio is now clean of these warnings, and we want to keep it this way - enable this for sparse builds. Signed-off-by: Michael S.

[PATCH v2 1/2] misc: atmel-ssc: register as sound DAI if #sound-dai-cells is present

2016-12-06 Thread Peter Rosin
The SSC is currently not usable with the ASoC simple-audio-card, as every SSC audio user has to build a platform driver that may do as little as calling atmel_ssc_set_audio/atmel_ssc_put_audio (which allocates the SSC and registers a DAI with the ASoC subsystem). So, have that happen

[PATCH v2 1/2] misc: atmel-ssc: register as sound DAI if #sound-dai-cells is present

2016-12-06 Thread Peter Rosin
The SSC is currently not usable with the ASoC simple-audio-card, as every SSC audio user has to build a platform driver that may do as little as calling atmel_ssc_set_audio/atmel_ssc_put_audio (which allocates the SSC and registers a DAI with the ASoC subsystem). So, have that happen

Re: [PATCH 1/1] mtd: spi-nor: remove WARN_ONCE() message in spi_nor_write()

2016-12-06 Thread Marek Vasut
On 12/07/2016 12:38 AM, Cyrille Pitchen wrote: > Le 06/12/2016 à 20:00, Marek Vasut a écrit : >> On 12/06/2016 06:14 PM, Cyrille Pitchen wrote: >>> This patch removes the WARN_ONCE() test in spi_nor_write(). >>> This macro triggers the display of a warning message almost every time we >>> use a

Re: [PATCH] net: wireless: realtek: constify rate_control_ops structures

2016-12-06 Thread Jes Sorensen
Larry Finger writes: > On 12/02/2016 03:50 AM, Bhumika Goyal wrote: >> The structures rate_control_ops are only passed as an argument to the >> functions ieee80211_rate_control_{register/unregister}. This argument is >> of type const, so rate_control_ops having this

Re: [PATCH 1/1] mtd: spi-nor: remove WARN_ONCE() message in spi_nor_write()

2016-12-06 Thread Marek Vasut
On 12/07/2016 12:38 AM, Cyrille Pitchen wrote: > Le 06/12/2016 à 20:00, Marek Vasut a écrit : >> On 12/06/2016 06:14 PM, Cyrille Pitchen wrote: >>> This patch removes the WARN_ONCE() test in spi_nor_write(). >>> This macro triggers the display of a warning message almost every time we >>> use a

Re: [PATCH] net: wireless: realtek: constify rate_control_ops structures

2016-12-06 Thread Jes Sorensen
Larry Finger writes: > On 12/02/2016 03:50 AM, Bhumika Goyal wrote: >> The structures rate_control_ops are only passed as an argument to the >> functions ieee80211_rate_control_{register/unregister}. This argument is >> of type const, so rate_control_ops having this property can also be >>

Re: [PATCH] net: return value of skb_linearize should be handled in Linux kernel

2016-12-06 Thread Cong Wang
On Mon, Dec 5, 2016 at 11:10 PM, Zhouyi Zhou wrote: > diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c > b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c > index 2a653ec..ab787cb 100644 > --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c > +++

Re: [PATCH] net: return value of skb_linearize should be handled in Linux kernel

2016-12-06 Thread Cong Wang
On Mon, Dec 5, 2016 at 11:10 PM, Zhouyi Zhou wrote: > diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c > b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c > index 2a653ec..ab787cb 100644 > --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c > +++

[PATCH 5/5] staging/lustre/o2iblnd: Fix misspelled attemps->attempts

2016-12-06 Thread Oleg Drokin
Highlighted by checkpatch: WARNING: 'attemps' may be misspelled - perhaps 'attempts'? #20278: FILE: drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c:3272: + * reconnection attemps. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c

[PATCH 3/5] staging/lustre: Convert all bare unsigned to unsigned int

2016-12-06 Thread Oleg Drokin
Highlighted by relatively new checkpatch test, warnings like: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Oleg Drokin --- drivers/staging/lustre/include/linux/lnet/lnetst.h | 6 +- .../staging/lustre/lustre/include/lprocfs_status.h | 3 +-

[PATCH 5/5] staging/lustre/o2iblnd: Fix misspelled attemps->attempts

2016-12-06 Thread Oleg Drokin
Highlighted by checkpatch: WARNING: 'attemps' may be misspelled - perhaps 'attempts'? #20278: FILE: drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c:3272: + * reconnection attemps. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 2 +- 1 file

[PATCH 3/5] staging/lustre: Convert all bare unsigned to unsigned int

2016-12-06 Thread Oleg Drokin
Highlighted by relatively new checkpatch test, warnings like: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Oleg Drokin --- drivers/staging/lustre/include/linux/lnet/lnetst.h | 6 +- .../staging/lustre/lustre/include/lprocfs_status.h | 3 +-

[PATCH 2/5] staging/lustre/socklnd: Fix whitespace problem

2016-12-06 Thread Oleg Drokin
checkpatch highlighted there are 8 spaces that could be converted to a tab: ERROR: code indent should use tabs where possible +^I^I^I^I^I */$ Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h | 2 +- 1 file changed, 1 insertion(+),

[PATCH 0/5] Lustre style fixes

2016-12-06 Thread Oleg Drokin
These patches fix some more of the low hanging fruits in the style problems highlighted by checkpatch. Now only false positive ERRORs are left. This also converts all bare unsigneds into unsigned ints and a couple of spelling fixes. Please consider. Oleg Drokin (5): staging/lustre/o2iblnd: Add

[PATCH 4/5] staging/lustre/o2iblnd: Fix misspelling intialized->intialized

2016-12-06 Thread Oleg Drokin
Highlighted by checkpatch: + if (!ps->ps_net) /* intialized? */ Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 2/5] staging/lustre/socklnd: Fix whitespace problem

2016-12-06 Thread Oleg Drokin
checkpatch highlighted there are 8 spaces that could be converted to a tab: ERROR: code indent should use tabs where possible +^I^I^I^I^I */$ Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 0/5] Lustre style fixes

2016-12-06 Thread Oleg Drokin
These patches fix some more of the low hanging fruits in the style problems highlighted by checkpatch. Now only false positive ERRORs are left. This also converts all bare unsigneds into unsigned ints and a couple of spelling fixes. Please consider. Oleg Drokin (5): staging/lustre/o2iblnd: Add

[PATCH 4/5] staging/lustre/o2iblnd: Fix misspelling intialized->intialized

2016-12-06 Thread Oleg Drokin
Highlighted by checkpatch: + if (!ps->ps_net) /* intialized? */ Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c

[PATCH 1/5] staging/lustre/o2iblnd: Add missing space

2016-12-06 Thread Oleg Drokin
checkpatch highlighted missing space before assignment for lock variable. + spinlock_t *lock= _data.kib_connd_lock; Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 1/5] staging/lustre/o2iblnd: Add missing space

2016-12-06 Thread Oleg Drokin
checkpatch highlighted missing space before assignment for lock variable. + spinlock_t *lock= _data.kib_connd_lock; Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 09/10] vsock/virtio: fix src/dst cid format

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:41, Michael S. Tsirkin wrote: These fields are 64 bit, using le32_to_cpu and friends on these will not do the right thing. Fix this up. Cc: sta...@vger.kernel.org Signed-off-by: Michael S. Tsirkin --- net/vmw_vsock/virtio_transport_common.c | 14

Re: [PATCH 09/10] vsock/virtio: fix src/dst cid format

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:41, Michael S. Tsirkin wrote: These fields are 64 bit, using le32_to_cpu and friends on these will not do the right thing. Fix this up. Cc: sta...@vger.kernel.org Signed-off-by: Michael S. Tsirkin --- net/vmw_vsock/virtio_transport_common.c | 14 +++--- 1 file

Re: [PATCH 3/5] arm64: dts: exynos5433: Add PPMU dt node

2016-12-06 Thread Chanwoo Choi
On 2016년 12월 07일 04:07, Krzysztof Kozlowski wrote: > On Fri, Dec 02, 2016 at 04:18:05PM +0900, Chanwoo Choi wrote: >> This patch adds PPMU (Platform Performance Monitoring Unit) Device-tree node >> to measure the utilization of each IP in Exynos SoC. >> >> - PPMU_D{0|1}_CPU are used to measure the

Re: [PATCH 3/5] arm64: dts: exynos5433: Add PPMU dt node

2016-12-06 Thread Chanwoo Choi
On 2016년 12월 07일 04:07, Krzysztof Kozlowski wrote: > On Fri, Dec 02, 2016 at 04:18:05PM +0900, Chanwoo Choi wrote: >> This patch adds PPMU (Platform Performance Monitoring Unit) Device-tree node >> to measure the utilization of each IP in Exynos SoC. >> >> - PPMU_D{0|1}_CPU are used to measure the

Re: [PATCH 08/10] vsock/virtio: mark an internal function static

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:41, Michael S. Tsirkin wrote: virtio_transport_alloc_pkt is only used locally, make it static. Signed-off-by: Michael S. Tsirkin --- net/vmw_vsock/virtio_transport_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 08/10] vsock/virtio: mark an internal function static

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:41, Michael S. Tsirkin wrote: virtio_transport_alloc_pkt is only used locally, make it static. Signed-off-by: Michael S. Tsirkin --- net/vmw_vsock/virtio_transport_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 07/10] vsock/virtio: add a missing __le annotation

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: guest cid is read from config space, therefore it's in little endian format and is treated as such, annotate it accordingly. Signed-off-by: Michael S. Tsirkin --- net/vmw_vsock/virtio_transport.c | 2 +- 1 file changed, 1

Re: [PATCH 07/10] vsock/virtio: add a missing __le annotation

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: guest cid is read from config space, therefore it's in little endian format and is treated as such, annotate it accordingly. Signed-off-by: Michael S. Tsirkin --- net/vmw_vsock/virtio_transport.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH 06/10] vhost: add missing __user annotations

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: Several vhost functions were missing __user annotations on pointers, causing sparse warnings. Fix this up. Signed-off-by: Michael S. Tsirkin --- drivers/vhost/vhost.c | 10 +- 1 file changed, 5 insertions(+), 5

Re: [PATCH 06/10] vhost: add missing __user annotations

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: Several vhost functions were missing __user annotations on pointers, causing sparse warnings. Fix this up. Signed-off-by: Michael S. Tsirkin --- drivers/vhost/vhost.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff

Re: [PATCH 04/10] drm/virtio: annotate virtio_gpu_queue_ctrl_buffer_locked

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: virtio_gpu_queue_ctrl_buffer_locked is called with ctrlq.qlock taken, it releases and acquires this lock. This causes a sparse warning. Add appropriate annotations for sparse context checking. Signed-off-by: Michael S. Tsirkin

Re: [PATCH 05/10] vhost: make interval tree static inline

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: vhost_umem_interval_tree is only used locally within vhost.c, mark it static. As some functions generated go unused, this triggers warnings unless we also mark it inline. Signed-off-by: Michael S. Tsirkin ---

Re: [PATCH 04/10] drm/virtio: annotate virtio_gpu_queue_ctrl_buffer_locked

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: virtio_gpu_queue_ctrl_buffer_locked is called with ctrlq.qlock taken, it releases and acquires this lock. This causes a sparse warning. Add appropriate annotations for sparse context checking. Signed-off-by: Michael S. Tsirkin ---

Re: [PATCH 05/10] vhost: make interval tree static inline

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: vhost_umem_interval_tree is only used locally within vhost.c, mark it static. As some functions generated go unused, this triggers warnings unless we also mark it inline. Signed-off-by: Michael S. Tsirkin --- drivers/vhost/vhost.c | 2 +- 1

Re: [RFC][PATCH 1/3 v2] usb: dwc2: Add extcon support to dwc2 driver

2016-12-06 Thread John Stultz
On Tue, Dec 6, 2016 at 7:54 PM, Chanwoo Choi wrote: > Hi John, > > I give a some guide for extcon API. > This patch uses the deprecated extcon API (extcon_get_cable_state_). > So, I recommend that you better to use following extcon API: > - extcon_get_cable_state_() ->

Re: [RFC][PATCH 1/3 v2] usb: dwc2: Add extcon support to dwc2 driver

2016-12-06 Thread John Stultz
On Tue, Dec 6, 2016 at 7:54 PM, Chanwoo Choi wrote: > Hi John, > > I give a some guide for extcon API. > This patch uses the deprecated extcon API (extcon_get_cable_state_). > So, I recommend that you better to use following extcon API: > - extcon_get_cable_state_() -> extcon_get_state() > -

Re: [PATCH 02/10] drm/virtio: fix endianness in primary_plane_update

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: virtio_gpu_cmd_transfer_to_host_2d expects x and y parameters in LE, but virtio_gpu_primary_plane_update passes in the CPU format instead. Signed-off-by: Michael S. Tsirkin --- drivers/gpu/drm/virtio/virtgpu_plane.c | 4 ++--

Re: [PATCH 03/10] drm/virtio: fix lock context imbalance

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: When virtio_gpu_free_vbufs exits due to list empty, it does not drop the free_vbufs lock that it took. list empty is not expected to happen anyway, but it can't hurt to fix this and drop the lock. Signed-off-by: Michael S. Tsirkin

Re: [PATCH 01/10] virtio_console: drop unused config fields

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: struct ports_device includes a config field including the whole virtio_console_config, but only max_nr_ports in there is ever updated or used. The rest is unused and in fact does not even mirror the device config. Drop everything except

Re: [PATCH 02/10] drm/virtio: fix endianness in primary_plane_update

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: virtio_gpu_cmd_transfer_to_host_2d expects x and y parameters in LE, but virtio_gpu_primary_plane_update passes in the CPU format instead. Signed-off-by: Michael S. Tsirkin --- drivers/gpu/drm/virtio/virtgpu_plane.c | 4 ++-- 1 file changed,

Re: [PATCH 03/10] drm/virtio: fix lock context imbalance

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: When virtio_gpu_free_vbufs exits due to list empty, it does not drop the free_vbufs lock that it took. list empty is not expected to happen anyway, but it can't hurt to fix this and drop the lock. Signed-off-by: Michael S. Tsirkin ---

Re: [PATCH 01/10] virtio_console: drop unused config fields

2016-12-06 Thread Jason Wang
On 2016年12月06日 23:40, Michael S. Tsirkin wrote: struct ports_device includes a config field including the whole virtio_console_config, but only max_nr_ports in there is ever updated or used. The rest is unused and in fact does not even mirror the device config. Drop everything except

[net-next][PATCH v2 10/18] RDS: IB: track and log active side endpoint in connection

2016-12-06 Thread Santosh Shilimkar
Useful to know the active and passive end points in a RDS IB connection. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h| 3 +++ net/rds/ib_cm.c | 11 +++ 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/net/rds/ib.h b/net/rds/ib.h

[net-next][PATCH v2 08/18] RDS: IB: split the mr registration and invalidation path

2016-12-06 Thread Santosh Shilimkar
MR invalidation in RDS is done in background thread and not in data path like registration. So break the dependency between them which helps to remove the performance bottleneck. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h | 4 +++- net/rds/ib_cm.c |

[net-next][PATCH v2 08/18] RDS: IB: split the mr registration and invalidation path

2016-12-06 Thread Santosh Shilimkar
MR invalidation in RDS is done in background thread and not in data path like registration. So break the dependency between them which helps to remove the performance bottleneck. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h | 4 +++- net/rds/ib_cm.c | 9 +++--

[net-next][PATCH v2 10/18] RDS: IB: track and log active side endpoint in connection

2016-12-06 Thread Santosh Shilimkar
Useful to know the active and passive end points in a RDS IB connection. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h| 3 +++ net/rds/ib_cm.c | 11 +++ 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/net/rds/ib.h b/net/rds/ib.h index f14c26d..97e7696 100644 ---

[net-next][PATCH v2 06/18] RDS: RDMA: start rdma listening after init

2016-12-06 Thread Santosh Shilimkar
From: Qing Huang This prevents RDS from handling incoming rdma packets before RDS completes initializing its recv/send components. Signed-off-by: Qing Huang Signed-off-by: Santosh Shilimkar ---

linux-next: manual merge of the staging tree with the net-next tree

2016-12-06 Thread Stephen Rothwell
Hi Greg, Today's linux-next merge of the staging tree got a conflict in: drivers/staging/slicoss/slicoss.c between commit: a52ad514fdf3 ("net: deprecate eth_change_mtu, remove usage") from the net-next tree and commit: 0af72df267f2 ("staging: slicoss: remove the staging driver") from

[net-next][PATCH v2 11/18] RDS: IB: add few useful cache stasts

2016-12-06 Thread Santosh Shilimkar
Tracks the ib receive cache total, incoming and frag allocations. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h | 7 +++ net/rds/ib_recv.c | 6 ++ net/rds/ib_stats.c | 2 ++ 3 files changed, 15 insertions(+) diff --git a/net/rds/ib.h

[net-next][PATCH v2 04/18] RDS: IB: make the transport retry count smallest

2016-12-06 Thread Santosh Shilimkar
Transport retry is not much useful since it indicate packet loss in fabric so its better to failover fast rather than longer retry. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/rds/ib.h

[net-next][PATCH v2 15/18] RDS: add stat for socket recv memory usage

2016-12-06 Thread Santosh Shilimkar
From: Venkat Venkatsubra Tracks the receive side memory added to scokets and removed from sockets. Signed-off-by: Venkat Venkatsubra Signed-off-by: Santosh Shilimkar --- net/rds/rds.h | 3 +++

[net-next][PATCH v2 18/18] RDS: IB: add missing connection cache usage info

2016-12-06 Thread Santosh Shilimkar
rds-tools already support it. Signed-off-by: Santosh Shilimkar --- include/uapi/linux/rds.h | 1 + net/rds/ib.c | 1 + 2 files changed, 2 insertions(+) diff --git a/include/uapi/linux/rds.h b/include/uapi/linux/rds.h index 3833113..410ae3c 100644 ---

[net-next][PATCH v2 14/18] RDS: IB: fix panic due to handlers running post teardown

2016-12-06 Thread Santosh Shilimkar
Shutdown code reaping loop takes care of emptying the CQ's before they being destroyed. And once tasklets are killed, the hanlders are not expected to run. But because of core tasklet code issues, tasklet handler could still run even after tasklet_kill, RDS IB shutdown code already reaps the CQs

[net-next][PATCH v2 16/18] RDS: make message size limit compliant with spec

2016-12-06 Thread Santosh Shilimkar
From: Avinash Repaka RDS support max message size as 1M but the code doesn't check this in all cases. Patch fixes it for RDMA & non-RDMA and RDS MR size and its enforced irrespective of underlying transport. Signed-off-by: Avinash Repaka

[net-next][PATCH v2 13/18] RDS: RDMA: Fix the composite message user notification

2016-12-06 Thread Santosh Shilimkar
When application sends an RDS RDMA composite message consist of RDMA transfer to be followed up by non RDMA payload, it expect to be notified *only* when the full message gets delivered. RDS RDMA notification doesn't behave this way though. Thanks to Venkat for debug and root casuing the issue

[net-next][PATCH v2 06/18] RDS: RDMA: start rdma listening after init

2016-12-06 Thread Santosh Shilimkar
From: Qing Huang This prevents RDS from handling incoming rdma packets before RDS completes initializing its recv/send components. Signed-off-by: Qing Huang Signed-off-by: Santosh Shilimkar --- net/rds/rdma_transport.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff

linux-next: manual merge of the staging tree with the net-next tree

2016-12-06 Thread Stephen Rothwell
Hi Greg, Today's linux-next merge of the staging tree got a conflict in: drivers/staging/slicoss/slicoss.c between commit: a52ad514fdf3 ("net: deprecate eth_change_mtu, remove usage") from the net-next tree and commit: 0af72df267f2 ("staging: slicoss: remove the staging driver") from

[net-next][PATCH v2 11/18] RDS: IB: add few useful cache stasts

2016-12-06 Thread Santosh Shilimkar
Tracks the ib receive cache total, incoming and frag allocations. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h | 7 +++ net/rds/ib_recv.c | 6 ++ net/rds/ib_stats.c | 2 ++ 3 files changed, 15 insertions(+) diff --git a/net/rds/ib.h b/net/rds/ib.h index 97e7696..4987387

[net-next][PATCH v2 04/18] RDS: IB: make the transport retry count smallest

2016-12-06 Thread Santosh Shilimkar
Transport retry is not much useful since it indicate packet loss in fabric so its better to failover fast rather than longer retry. Signed-off-by: Santosh Shilimkar --- net/rds/ib.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/rds/ib.h b/net/rds/ib.h index

[net-next][PATCH v2 15/18] RDS: add stat for socket recv memory usage

2016-12-06 Thread Santosh Shilimkar
From: Venkat Venkatsubra Tracks the receive side memory added to scokets and removed from sockets. Signed-off-by: Venkat Venkatsubra Signed-off-by: Santosh Shilimkar --- net/rds/rds.h | 3 +++ net/rds/recv.c | 4 2 files changed, 7 insertions(+) diff --git a/net/rds/rds.h

[net-next][PATCH v2 18/18] RDS: IB: add missing connection cache usage info

2016-12-06 Thread Santosh Shilimkar
rds-tools already support it. Signed-off-by: Santosh Shilimkar --- include/uapi/linux/rds.h | 1 + net/rds/ib.c | 1 + 2 files changed, 2 insertions(+) diff --git a/include/uapi/linux/rds.h b/include/uapi/linux/rds.h index 3833113..410ae3c 100644 --- a/include/uapi/linux/rds.h +++

[net-next][PATCH v2 14/18] RDS: IB: fix panic due to handlers running post teardown

2016-12-06 Thread Santosh Shilimkar
Shutdown code reaping loop takes care of emptying the CQ's before they being destroyed. And once tasklets are killed, the hanlders are not expected to run. But because of core tasklet code issues, tasklet handler could still run even after tasklet_kill, RDS IB shutdown code already reaps the CQs

[net-next][PATCH v2 16/18] RDS: make message size limit compliant with spec

2016-12-06 Thread Santosh Shilimkar
From: Avinash Repaka RDS support max message size as 1M but the code doesn't check this in all cases. Patch fixes it for RDMA & non-RDMA and RDS MR size and its enforced irrespective of underlying transport. Signed-off-by: Avinash Repaka Signed-off-by: Santosh Shilimkar --- net/rds/rdma.c |

[net-next][PATCH v2 13/18] RDS: RDMA: Fix the composite message user notification

2016-12-06 Thread Santosh Shilimkar
When application sends an RDS RDMA composite message consist of RDMA transfer to be followed up by non RDMA payload, it expect to be notified *only* when the full message gets delivered. RDS RDMA notification doesn't behave this way though. Thanks to Venkat for debug and root casuing the issue

[net-next][PATCH v2 12/18] RDS: IB: Add vector spreading for cqs

2016-12-06 Thread Santosh Shilimkar
Based on available device vectors, allocate cqs accordingly to get better spread of completion vectors which helps performace great deal.. Signed-off-by: Santosh Shilimkar --- net/rds/ib.c| 11 +++ net/rds/ib.h| 5 + net/rds/ib_cm.c | 40

[net-next][PATCH v2 03/18] RDS: IB: include faddr in connection log

2016-12-06 Thread Santosh Shilimkar
Also use pr_* for it. Signed-off-by: Santosh Shilimkar --- net/rds/ib_cm.c | 19 +-- net/rds/ib_recv.c | 4 ++-- net/rds/ib_send.c | 4 ++-- 3 files changed, 13 insertions(+), 14 deletions(-) diff --git a/net/rds/ib_cm.c b/net/rds/ib_cm.c index

[net-next][PATCH v2 01/18] RDS: log the address on bind failure

2016-12-06 Thread Santosh Shilimkar
It's useful to know the IP address when RDS fails to bind a connection. Thus, adding it to the error message. Orabug: 21894138 Reviewed-by: Wei Lin Guay Signed-off-by: Santosh Shilimkar --- net/rds/bind.c | 4 ++-- 1 file changed, 2

[net-next][PATCH v2 12/18] RDS: IB: Add vector spreading for cqs

2016-12-06 Thread Santosh Shilimkar
Based on available device vectors, allocate cqs accordingly to get better spread of completion vectors which helps performace great deal.. Signed-off-by: Santosh Shilimkar --- net/rds/ib.c| 11 +++ net/rds/ib.h| 5 + net/rds/ib_cm.c | 40

[net-next][PATCH v2 03/18] RDS: IB: include faddr in connection log

2016-12-06 Thread Santosh Shilimkar
Also use pr_* for it. Signed-off-by: Santosh Shilimkar --- net/rds/ib_cm.c | 19 +-- net/rds/ib_recv.c | 4 ++-- net/rds/ib_send.c | 4 ++-- 3 files changed, 13 insertions(+), 14 deletions(-) diff --git a/net/rds/ib_cm.c b/net/rds/ib_cm.c index 5b2ab95..b9da1e5 100644 ---

[net-next][PATCH v2 01/18] RDS: log the address on bind failure

2016-12-06 Thread Santosh Shilimkar
It's useful to know the IP address when RDS fails to bind a connection. Thus, adding it to the error message. Orabug: 21894138 Reviewed-by: Wei Lin Guay Signed-off-by: Santosh Shilimkar --- net/rds/bind.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/rds/bind.c

[net-next][PATCH v2 09/18] RDS: RDMA: silence the use_once mr log flood

2016-12-06 Thread Santosh Shilimkar
In absence of extension headers, message log will keep flooding the console. As such even without use_once we can clean up the MRs so its not really an error case message so make it debug message Signed-off-by: Santosh Shilimkar --- net/rds/rdma.c | 3 ++- 1 file

[net-next][PATCH v2 17/18] RDS: add receive message trace used by application

2016-12-06 Thread Santosh Shilimkar
Socket option to tap receive path latency in various stages in nano seconds. It can be enabled on selective sockets using using SO_RDS_MSG_RXPATH_LATENCY socket option. RDS will return the data to application with RDS_CMSG_RXPATH_LATENCY in defined format. Scope is left to add more trace points

[net-next][PATCH v2 09/18] RDS: RDMA: silence the use_once mr log flood

2016-12-06 Thread Santosh Shilimkar
In absence of extension headers, message log will keep flooding the console. As such even without use_once we can clean up the MRs so its not really an error case message so make it debug message Signed-off-by: Santosh Shilimkar --- net/rds/rdma.c | 3 ++- 1 file changed, 2 insertions(+), 1

[net-next][PATCH v2 17/18] RDS: add receive message trace used by application

2016-12-06 Thread Santosh Shilimkar
Socket option to tap receive path latency in various stages in nano seconds. It can be enabled on selective sockets using using SO_RDS_MSG_RXPATH_LATENCY socket option. RDS will return the data to application with RDS_CMSG_RXPATH_LATENCY in defined format. Scope is left to add more trace points

[net-next][PATCH v2 00/18] net: RDS updates

2016-12-06 Thread Santosh Shilimkar
v1->v2: Re-aligned indentation in patch 'RDS: mark few internal functions Series consist of: - RDMA transport fixes for map failure, listen sequence, handler panic and composite message notification. - Couple of sparse fixes. - Message logging improvements for bind failure, use once mr

[net-next][PATCH v2 02/18] RDS: mark few internal functions static to make sparse build happy

2016-12-06 Thread Santosh Shilimkar
Fixes below warnings: warning: symbol 'rds_send_probe' was not declared. Should it be static? warning: symbol 'rds_send_ping' was not declared. Should it be static? warning: symbol 'rds_tcp_accept_one_path' was not declared. Should it be static? warning: symbol 'rds_walk_conn_path_info' was not

[net-next][PATCH v2 00/18] net: RDS updates

2016-12-06 Thread Santosh Shilimkar
v1->v2: Re-aligned indentation in patch 'RDS: mark few internal functions Series consist of: - RDMA transport fixes for map failure, listen sequence, handler panic and composite message notification. - Couple of sparse fixes. - Message logging improvements for bind failure, use once mr

[net-next][PATCH v2 02/18] RDS: mark few internal functions static to make sparse build happy

2016-12-06 Thread Santosh Shilimkar
Fixes below warnings: warning: symbol 'rds_send_probe' was not declared. Should it be static? warning: symbol 'rds_send_ping' was not declared. Should it be static? warning: symbol 'rds_tcp_accept_one_path' was not declared. Should it be static? warning: symbol 'rds_walk_conn_path_info' was not

[net-next][PATCH v2 05/18] RDS: RDMA: fix the ib_map_mr_sg_zbva() argument

2016-12-06 Thread Santosh Shilimkar
Fixes warning: Using plain integer as NULL pointer Signed-off-by: Santosh Shilimkar --- net/rds/ib_frmr.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/net/rds/ib_frmr.c b/net/rds/ib_frmr.c index d921adc..66b3d62 100644 ---

[net-next][PATCH v2 07/18] RDS: RDMA: return appropriate error on rdma map failures

2016-12-06 Thread Santosh Shilimkar
The first message to a remote node should prompt a new connection even if it is RDMA operation. For RDMA operation the MR mapping can fail because connections is not yet up. Since the connection establishment is asynchronous, we make sure the map failure because of unavailable connection reach to

[net-next][PATCH v2 05/18] RDS: RDMA: fix the ib_map_mr_sg_zbva() argument

2016-12-06 Thread Santosh Shilimkar
Fixes warning: Using plain integer as NULL pointer Signed-off-by: Santosh Shilimkar --- net/rds/ib_frmr.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/net/rds/ib_frmr.c b/net/rds/ib_frmr.c index d921adc..66b3d62 100644 --- a/net/rds/ib_frmr.c +++ b/net/rds/ib_frmr.c

[net-next][PATCH v2 07/18] RDS: RDMA: return appropriate error on rdma map failures

2016-12-06 Thread Santosh Shilimkar
The first message to a remote node should prompt a new connection even if it is RDMA operation. For RDMA operation the MR mapping can fail because connections is not yet up. Since the connection establishment is asynchronous, we make sure the map failure because of unavailable connection reach to

[PATCH net-next 0/2] Initial driver for Synopsys DWC XLGMAC

2016-12-06 Thread Jie Deng
This series provides the support for 25/40/50/100 GbE devices using Synopsys DWC Enterprise Ethernet (XLGMAC). The first patch adds support for Synopsys XLGMII. The second patch provides the initial driver for Synopsys XLGMAC The driver has three layers by refactoring AMD XGBE. dwc-eth-xxx.x

[PATCH net-next 1/2] net: phy: add extension of phy-mode for XLGMII

2016-12-06 Thread Jie Deng
This patch adds phy-mode support for Synopsys XLGMAC Signed-off-by: Jie Deng --- Documentation/devicetree/bindings/net/ethernet.txt | 1 + include/linux/phy.h| 3 +++ 2 files changed, 4 insertions(+) diff --git

[PATCH net-next 0/2] Initial driver for Synopsys DWC XLGMAC

2016-12-06 Thread Jie Deng
This series provides the support for 25/40/50/100 GbE devices using Synopsys DWC Enterprise Ethernet (XLGMAC). The first patch adds support for Synopsys XLGMII. The second patch provides the initial driver for Synopsys XLGMAC The driver has three layers by refactoring AMD XGBE. dwc-eth-xxx.x

[PATCH net-next 1/2] net: phy: add extension of phy-mode for XLGMII

2016-12-06 Thread Jie Deng
This patch adds phy-mode support for Synopsys XLGMAC Signed-off-by: Jie Deng --- Documentation/devicetree/bindings/net/ethernet.txt | 1 + include/linux/phy.h| 3 +++ 2 files changed, 4 insertions(+) diff --git

[PATCH] staging/lustre/osc: Revert erroneous list_for_each_entry_safe use

2016-12-06 Thread Oleg Drokin
I have been having a lot of unexplainable crashes in osc_lru_shrink lately that I could not see a good explanation for and then I found this patch that slip under the radar somehow that incorrectly converted while loop for lru list iteration into list_for_each_entry_safe totally ignoring that in

[PATCH] staging/lustre/osc: Revert erroneous list_for_each_entry_safe use

2016-12-06 Thread Oleg Drokin
I have been having a lot of unexplainable crashes in osc_lru_shrink lately that I could not see a good explanation for and then I found this patch that slip under the radar somehow that incorrectly converted while loop for lru list iteration into list_for_each_entry_safe totally ignoring that in

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