Re: [PATCH v2 0/4] ARM: dts: keystone: Add support for new K2G evm

2017-05-10 Thread Santosh Shilimkar
On 4/24/2017 1:22 PM, Franklin S Cooper Jr wrote: This patchset adds support for new K2G Industrial Communication Engine evm. For now only a bare minimal dts which will allow ram boot. Additional peripherals will be added when base K2G SoC patches are upstreamed allowing peripherals to be

Re: linux-next: manual merge of the pm tree with the arm-soc tree

2017-04-24 Thread Santosh Shilimkar
On 4/24/2017 11:02 AM, Dave Gerlach wrote: On 04/21/2017 04:54 PM, Santosh Shilimkar wrote: +Dave, [...] Ok, good, thanks for checking! They are however the commits that contain the silly https://urldefense.proofpoint.com URLs. Can you send a follow-up patch to fix these and use

Re: linux-next: manual merge of the pm tree with the arm-soc tree

2017-04-24 Thread Santosh Shilimkar
On 4/24/2017 11:02 AM, Dave Gerlach wrote: On 04/21/2017 04:54 PM, Santosh Shilimkar wrote: +Dave, [...] Ok, good, thanks for checking! They are however the commits that contain the silly https://urldefense.proofpoint.com URLs. Can you send a follow-up patch to fix these and use

Re: linux-next: manual merge of the pm tree with the arm-soc tree

2017-04-21 Thread Santosh Shilimkar
+Dave, On 4/21/2017 2:44 PM, Arnd Bergmann wrote: On Fri, Apr 21, 2017 at 11:02 PM, santosh.shilim...@oracle.com wrote: On 4/21/17 2:31 AM, Arnd Bergmann wrote: [...] arm-soc/next/drivers: ae3874cc931b ARM: keystone: Drop PM domain support for k2g

Re: linux-next: manual merge of the pm tree with the arm-soc tree

2017-04-21 Thread Santosh Shilimkar
+Dave, On 4/21/2017 2:44 PM, Arnd Bergmann wrote: On Fri, Apr 21, 2017 at 11:02 PM, santosh.shilim...@oracle.com wrote: On 4/21/17 2:31 AM, Arnd Bergmann wrote: [...] arm-soc/next/drivers: ae3874cc931b ARM: keystone: Drop PM domain support for k2g 52835d59fc6c soc: ti: Add

Re: linux-next: manual merge of the pm tree with the arm-soc tree

2017-04-21 Thread Santosh Shilimkar
On 4/21/2017 2:31 PM, Rafael J. Wysocki wrote: On Friday, April 21, 2017 02:02:35 PM santosh.shilim...@oracle.com wrote: On 4/21/17 2:31 AM, Arnd Bergmann wrote: On Fri, Apr 21, 2017 at 8:39 AM, santosh.shilim...@oracle.com [...] I still see two conflicting trees in linux-next as of

Re: linux-next: manual merge of the pm tree with the arm-soc tree

2017-04-21 Thread Santosh Shilimkar
On 4/21/2017 2:31 PM, Rafael J. Wysocki wrote: On Friday, April 21, 2017 02:02:35 PM santosh.shilim...@oracle.com wrote: On 4/21/17 2:31 AM, Arnd Bergmann wrote: On Fri, Apr 21, 2017 at 8:39 AM, santosh.shilim...@oracle.com [...] I still see two conflicting trees in linux-next as of

Re: [PATCH] RDS: IB: ensure an initialized ret is printed in pr_warn message

2017-04-07 Thread Santosh Shilimkar
On 4/7/2017 7:24 AM, Dan Carpenter wrote: Setting it to zero doesn't seem like the right thing, it should be an error code. Oh, heh... Smatch parses this one correctly. "ret" is always initialized but the code is probably buggy in a different way: [...] 561 ibmr =

Re: [PATCH] RDS: IB: ensure an initialized ret is printed in pr_warn message

2017-04-07 Thread Santosh Shilimkar
On 4/7/2017 7:24 AM, Dan Carpenter wrote: Setting it to zero doesn't seem like the right thing, it should be an error code. Oh, heh... Smatch parses this one correctly. "ret" is always initialized but the code is probably buggy in a different way: [...] 561 ibmr =

Re: [PATCH 0/2] ARM: dts: keystone: Add support for new K2G evm

2017-04-04 Thread Santosh Shilimkar
On 4/3/2017 9:44 AM, santosh.shilim...@oracle.com wrote: Hi Franklin, On 3/30/17 8:29 AM, Franklin S Cooper Jr wrote: This patchset adds support for new K2G Industrial Communication Engine evm. For now only a bare minimal dts which will allow ram boot. Additional peripherals will be added when

Re: [PATCH 0/2] ARM: dts: keystone: Add support for new K2G evm

2017-04-04 Thread Santosh Shilimkar
On 4/3/2017 9:44 AM, santosh.shilim...@oracle.com wrote: Hi Franklin, On 3/30/17 8:29 AM, Franklin S Cooper Jr wrote: This patchset adds support for new K2G Industrial Communication Engine evm. For now only a bare minimal dts which will allow ram boot. Additional peripherals will be added when

Re: [PATCH] RDS: IB: fix ifnullfree.cocci warnings

2017-02-23 Thread Santosh Shilimkar
freeing functions is not needed. Based on checkpatch warning "kfree(NULL) is safe this check is probably not required" and kfreeaddr.cocci by Julia Lawall. Generated by: scripts/coccinelle/free/ifnullfree.cocci Signed-off-by: Fengguang Wu <fengguang...@intel.com> --- A

Re: [PATCH] RDS: IB: fix ifnullfree.cocci warnings

2017-02-23 Thread Santosh Shilimkar
freeing functions is not needed. Based on checkpatch warning "kfree(NULL) is safe this check is probably not required" and kfreeaddr.cocci by Julia Lawall. Generated by: scripts/coccinelle/free/ifnullfree.cocci Signed-off-by: Fengguang Wu --- Acked-by: Santosh Shilimkar

Re: [PATCH] uapi: fix linux/rds.h userspace compilation errors

2017-02-22 Thread Santosh Shilimkar
rx_trace[RDS_MSG_RX_DGRAM_TRACE_MAX]; Fixes: 3289025a("RDS: add receive message trace used by application") Signed-off-by: Dmitry V. Levin <l...@altlinux.org> --- This was part of the patch I submitted other-day with rest of the clean-up. Thanks Dmitry. Acked-by: Santosh Shilimkar

Re: [PATCH] uapi: fix linux/rds.h userspace compilation errors

2017-02-22 Thread Santosh Shilimkar
rx_trace[RDS_MSG_RX_DGRAM_TRACE_MAX]; Fixes: 3289025a("RDS: add receive message trace used by application") Signed-off-by: Dmitry V. Levin --- This was part of the patch I submitted other-day with rest of the clean-up. Thanks Dmitry. Acked-by: Santosh Shilimkar

[net-next][PATCH v2] RDS: Make user visible data types consistent with rest of the kernel

2017-02-20 Thread Santosh Shilimkar
Use "__uX/__sX" kernel types rather than current uint*/int* for entire file. Reviewed-by: Sowmini Varadhan <sowmini.varad...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- v1->v2: As suggested by David Miller, cleaned up entire h

[net-next][PATCH v2] RDS: Make user visible data types consistent with rest of the kernel

2017-02-20 Thread Santosh Shilimkar
Use "__uX/__sX" kernel types rather than current uint*/int* for entire file. Reviewed-by: Sowmini Varadhan Signed-off-by: Santosh Shilimkar --- v1->v2: As suggested by David Miller, cleaned up entire header to use kernel types. include/uapi/linu

[net-next][PATCH v1] RDS: keep data type consistent in the user visible header

2017-02-17 Thread Santosh Shilimkar
of the user header. Reported-by: Sowmini Varadhan <sowmini.varad...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- V1: Fixed the email id typo include/uapi/linux/rds.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/include/uapi

[net-next][PATCH v1] RDS: keep data type consistent in the user visible header

2017-02-17 Thread Santosh Shilimkar
of the user header. Reported-by: Sowmini Varadhan Signed-off-by: Santosh Shilimkar --- V1: Fixed the email id typo include/uapi/linux/rds.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/include/uapi/linux/rds.h b/include/uapi/linux/rds.h index 3833113..a9e4c02 100644

[net-next][PATCH] RDS: keep data type consistent in the user visible header

2017-02-17 Thread Santosh Shilimkar
From: Santosh Shilimkar <ssant...@kernel.org> rds.h is exported to /usr/include/rds.h, so u8, u64 leads to errors like below. /usr/include/linux/rds.h:197: error: expected specifier-qualifier-list before 'u8' /usr/include/linux/rds.h:202: error: expected specifier-qualifier-list befor

[net-next][PATCH] RDS: keep data type consistent in the user visible header

2017-02-17 Thread Santosh Shilimkar
From: Santosh Shilimkar rds.h is exported to /usr/include/rds.h, so u8, u64 leads to errors like below. /usr/include/linux/rds.h:197: error: expected specifier-qualifier-list before 'u8' /usr/include/linux/rds.h:202: error: expected specifier-qualifier-list before 'u8' Fix it by following

Re: [PATCH v3 0/4] ARM: K2G: Add support for TI-SCI Generic PM Domains

2017-01-19 Thread Santosh Shilimkar
On 1/19/2017 10:59 AM, Dave Gerlach wrote: Santosh, On 01/19/2017 11:51 AM, santosh.shilim...@oracle.com wrote: On 1/4/17 2:06 PM, Dave Gerlach wrote: [...] Am going to send pull request over weekend, so if you would like me take the series via arm-soc tree, please get Rafaels, ack and let

Re: [PATCH v3 0/4] ARM: K2G: Add support for TI-SCI Generic PM Domains

2017-01-19 Thread Santosh Shilimkar
On 1/19/2017 10:59 AM, Dave Gerlach wrote: Santosh, On 01/19/2017 11:51 AM, santosh.shilim...@oracle.com wrote: On 1/4/17 2:06 PM, Dave Gerlach wrote: [...] Am going to send pull request over weekend, so if you would like me take the series via arm-soc tree, please get Rafaels, ack and let

Re: [PATCH 2/2] ARM: DTS: Fix register map for virt-capable GIC

2017-01-18 Thread Santosh Shilimkar
Signed-off-by: Marc Zyngier <marc.zyng...@arm.com> --- arch/arm/boot/dts/imx6ul.dtsi| 4 ++-- arch/arm/boot/dts/keystone-k2g.dtsi | 2 +- arch/arm/boot/dts/keystone.dtsi | 2 +- For keystone bits, Acked-by: Santosh Shilimkar <ssant...@kernel.org>

Re: [PATCH 2/2] ARM: DTS: Fix register map for virt-capable GIC

2017-01-18 Thread Santosh Shilimkar
Signed-off-by: Marc Zyngier --- arch/arm/boot/dts/imx6ul.dtsi| 4 ++-- arch/arm/boot/dts/keystone-k2g.dtsi | 2 +- arch/arm/boot/dts/keystone.dtsi | 2 +- For keystone bits, Acked-by: Santosh Shilimkar

Re: [PATCH RESEND 1/2] memory: aemif: allow passing device lookup table as platform data

2017-01-13 Thread Santosh Shilimkar
<nsek...@ti.com> Acked-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- Can you please take this one patch via your driver core ? Regards, Santosh

Re: [PATCH RESEND 1/2] memory: aemif: allow passing device lookup table as platform data

2017-01-13 Thread Santosh Shilimkar
but nodes created from within the aemif driver can't access the lookup table. Create a platform data structure that holds a pointer to of_dev_auxdata so that we can use it with of_platform_populate(). Signed-off-by: Bartosz Golaszewski Acked-by: Sekhar Nori Acked-by: Santosh Shilimkar --- Can

Re: [PATCH 0/5] Reset Controller Nodes for TI Keystone platforms

2017-01-09 Thread Santosh Shilimkar
On 1/9/2017 11:43 AM, Suman Anna wrote: Hi Santosh, This patch adds the reset controller nodes and the corresponding reset data for TI Keystone 66AK2H, 66AK2L and 66AK2E SoCs. These resets are for the DSPs on these SoCs, and are the last dependencies before the keystone remoteproc driver can be

Re: [PATCH 0/5] Reset Controller Nodes for TI Keystone platforms

2017-01-09 Thread Santosh Shilimkar
On 1/9/2017 11:43 AM, Suman Anna wrote: Hi Santosh, This patch adds the reset controller nodes and the corresponding reset data for TI Keystone 66AK2H, 66AK2L and 66AK2E SoCs. These resets are for the DSPs on these SoCs, and are the last dependencies before the keystone remoteproc driver can be

Re: [PATCH 0/5] Use mmio-sram driver for Keystone MSMC RAM

2017-01-06 Thread Santosh Shilimkar
On 1/6/2017 11:10 AM, Suman Anna wrote: Hi Santosh, OK. Its good to add the code at least with one active user of it. Since this has to anyway wait for another merge window, please post the users of it so that I can pull the combined patchset. Are you going to post new patchset or you

Re: [PATCH 0/5] Use mmio-sram driver for Keystone MSMC RAM

2017-01-06 Thread Santosh Shilimkar
On 1/6/2017 11:10 AM, Suman Anna wrote: Hi Santosh, OK. Its good to add the code at least with one active user of it. Since this has to anyway wait for another merge window, please post the users of it so that I can pull the combined patchset. Are you going to post new patchset or you

[net-next][PATCH] RDS: validate the requested traces user input against max supported

2017-01-06 Thread Santosh Shilimkar
Larger than supported value can lead to array read/write overflow. Reported-by: Colin Ian King <colin.k...@canonical.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- net/rds/af_rds.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/rds/af_rds

[net-next][PATCH] RDS: validate the requested traces user input against max supported

2017-01-06 Thread Santosh Shilimkar
Larger than supported value can lead to array read/write overflow. Reported-by: Colin Ian King Signed-off-by: Santosh Shilimkar --- net/rds/af_rds.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/rds/af_rds.c b/net/rds/af_rds.c index fd821740..b405f77 100644 --- a/net/rds/af_rds.c

Re: [PATCH v2] soc: ti: Drop wait from wkup_m3_rproc_boot_thread

2017-01-05 Thread Santosh Shilimkar
On 1/5/2017 1:08 PM, Suman Anna wrote: On 01/03/2017 05:41 PM, Sarangdhar Joshi wrote: The function wkup_m3_rproc_boot_thread waits for asynchronous firmware loading to parse the resource table before calling rproc_boot(). However, as the resource table parsing has been moved to rproc_boot(),

Re: [PATCH v2] soc: ti: Drop wait from wkup_m3_rproc_boot_thread

2017-01-05 Thread Santosh Shilimkar
On 1/5/2017 1:08 PM, Suman Anna wrote: On 01/03/2017 05:41 PM, Sarangdhar Joshi wrote: The function wkup_m3_rproc_boot_thread waits for asynchronous firmware loading to parse the resource table before calling rproc_boot(). However, as the resource table parsing has been moved to rproc_boot(),

Re: [PATCH 1/2] soc: ti: knav_dma: fix typos in trace message

2017-01-05 Thread Santosh Shilimkar
On 1/5/2017 11:08 AM, Murali Karicheri wrote: On 12/20/2016 08:01 PM, Santosh Shilimkar wrote: On 12/20/2016 2:24 PM, Murali Karicheri wrote: This patch fixes some typos in the trace message Signed-off-by: Murali Karicheri <m-kariche...@ti.com> Signed-off-by: Sekhar Nori <nsek.

Re: [PATCH 1/2] soc: ti: knav_dma: fix typos in trace message

2017-01-05 Thread Santosh Shilimkar
On 1/5/2017 11:08 AM, Murali Karicheri wrote: On 12/20/2016 08:01 PM, Santosh Shilimkar wrote: On 12/20/2016 2:24 PM, Murali Karicheri wrote: This patch fixes some typos in the trace message Signed-off-by: Murali Karicheri Signed-off-by: Sekhar Nori --- Patch 1 and 2 looks fine. Will pick

Re: [PATCH v2 4/4] ARM: dts: keystone: Add "ti,da830-uart" compatible string

2017-01-05 Thread Santosh Shilimkar
On 1/5/2017 1:04 AM, Sekhar Nori wrote: Hi Santosh, On Thursday 05 January 2017 03:30 AM, Santosh Shilimkar wrote: On 1/4/2017 12:30 PM, David Lechner wrote: The TI Keystone SoCs have extra UART registers beyond the standard 8250 registers, so we need a new compatible string to indicate

Re: [PATCH v2 4/4] ARM: dts: keystone: Add "ti,da830-uart" compatible string

2017-01-05 Thread Santosh Shilimkar
On 1/5/2017 1:04 AM, Sekhar Nori wrote: Hi Santosh, On Thursday 05 January 2017 03:30 AM, Santosh Shilimkar wrote: On 1/4/2017 12:30 PM, David Lechner wrote: The TI Keystone SoCs have extra UART registers beyond the standard 8250 registers, so we need a new compatible string to indicate

Re: [PATCH v3 0/4] ARM: K2G: Add support for TI-SCI Generic PM Domains

2017-01-04 Thread Santosh Shilimkar
On 1/4/2017 12:55 PM, Dave Gerlach wrote: Hi, This is v3 of the series to add support for TI-SCI Generic PM Domains. Previous versions can be found here: v2: https://www.spinics.net/lists/kernel/msg2364612.html v1: http://www.spinics.net/lists/arm-kernel/msg525204.html This version is rebased

Re: [PATCH v3 0/4] ARM: K2G: Add support for TI-SCI Generic PM Domains

2017-01-04 Thread Santosh Shilimkar
On 1/4/2017 12:55 PM, Dave Gerlach wrote: Hi, This is v3 of the series to add support for TI-SCI Generic PM Domains. Previous versions can be found here: v2: https://www.spinics.net/lists/kernel/msg2364612.html v1: http://www.spinics.net/lists/arm-kernel/msg525204.html This version is rebased

Re: [PATCH v2 4/4] ARM: dts: keystone: Add "ti,da830-uart" compatible string

2017-01-04 Thread Santosh Shilimkar
On 1/4/2017 12:30 PM, David Lechner wrote: The TI Keystone SoCs have extra UART registers beyond the standard 8250 registers, so we need a new compatible string to indicate this. Also, at least one of these registers uses the full 32 bits, so we need to specify reg-io-width in addition to

Re: [PATCH v2 4/4] ARM: dts: keystone: Add "ti,da830-uart" compatible string

2017-01-04 Thread Santosh Shilimkar
On 1/4/2017 12:30 PM, David Lechner wrote: The TI Keystone SoCs have extra UART registers beyond the standard 8250 registers, so we need a new compatible string to indicate this. Also, at least one of these registers uses the full 32 bits, so we need to specify reg-io-width in addition to

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

2017-01-02 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 <santosh.shilim...@oracle.com> --- net/rds/ib.h | 4 +++- net/rds/i

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

2017-01-02 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 <santosh.shilim...@oracle.com> --- net/rds/rdma.c | 3 ++-

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

2017-01-02 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 v3 08/17] RDS: IB: split the mr registration and invalidation path

2017-01-02 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/rds

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

2017-01-02 Thread Santosh Shilimkar
to the user by appropriate error code. Before returning to the user, lets trigger the connection so that its ready for the next retry. Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- net/rds/send.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff

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

2017-01-02 Thread Santosh Shilimkar
From: Qing Huang <qing.hu...@oracle.com> This prevents RDS from handling incoming rdma packets before RDS completes initializing its recv/send components. Signed-off-by: Qing Huang <qing.hu...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.co

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

2017-01-02 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

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

2017-01-02 Thread Santosh Shilimkar
to the user by appropriate error code. Before returning to the user, lets trigger the connection so that its ready for the next retry. Signed-off-by: Santosh Shilimkar --- net/rds/send.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/net/rds/send.c b/net/rds/send.c

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

2017-01-02 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 <wei.lin.g...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- net/rds/bind.c | 4 ++-- 1 fil

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

2017-01-02 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 <santosh.shilim...@oracle.com> --- net/rds/ib.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/rds/

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

2017-01-02 Thread Santosh Shilimkar
Also use pr_* for it. Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- 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

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

2017-01-02 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 b

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

2017-01-02 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 45ac8e8

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

2017-01-02 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 v3 05/17] RDS: RDMA: fix the ib_map_mr_sg_zbva() argument

2017-01-02 Thread Santosh Shilimkar
Fixes warning: Using plain integer as NULL pointer Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- 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/n

[net-next][PATCH v3 00/17] net: RDS updates

2017-01-02 Thread Santosh Shilimkar
ve message trace used by application (2017-01-02 14:02:59 -0800) Avinash Repaka (1): RDS: make message size limit compliant with spec Qing Huang (1): RDS: RDMA: start rdma listening after init Santosh Shilimkar (14): RDS: log the address

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

2017-01-02 Thread Santosh Shilimkar
' was not declared. Should it be static? Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- net/rds/connection.c | 10 +- net/rds/send.c | 4 ++-- net/rds/tcp_listen.c | 1 + 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/net/rds/connection.c b/n

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

2017-01-02 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 v3 00/17] net: RDS updates

2017-01-02 Thread Santosh Shilimkar
ve message trace used by application (2017-01-02 14:02:59 -0800) Avinash Repaka (1): RDS: make message size limit compliant with spec Qing Huang (1): RDS: RDMA: start rdma listening after init Santosh Shilimkar (14): RDS: log the address

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

2017-01-02 Thread Santosh Shilimkar
' was not declared. Should it be static? Signed-off-by: Santosh Shilimkar --- net/rds/connection.c | 10 +- net/rds/send.c | 4 ++-- net/rds/tcp_listen.c | 1 + 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/net/rds/connection.c b/net/rds/connection.c index fe9d31c..0e04dcc

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

2017-01-02 Thread Santosh Shilimkar
for future without need of change in the interface. Reviewed-by: Sowmini Varadhan <sowmini.varad...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- include/uapi/linux/rds.h | 33 + net/rds/af_rds.c

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

2017-01-02 Thread Santosh Shilimkar
..@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- net/rds/rdma.c | 9 - net/rds/rds.h | 3 +++ net/rds/send.c | 31 +++ 3 files changed, 42 insertions(+), 1 deletion(-) diff --git a/net/rds/rdma.c b/net/rds/rdma.c index 1

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

2017-01-02 Thread Santosh Shilimkar
. Reviewed-by: Venkat Venkatsubra <venkat.x.venkatsu...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- net/rds/ib_send.c | 25 +++-- net/rds/rdma.c| 10 ++ net/rds/rds.h | 1 + net/rds/send.c| 4 +++- 4 files

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

2017-01-02 Thread Santosh Shilimkar
. Reviewed-by: Venkat Venkatsubra Signed-off-by: Santosh Shilimkar --- net/rds/ib_send.c | 25 +++-- net/rds/rdma.c| 10 ++ net/rds/rds.h | 1 + net/rds/send.c| 4 +++- 4 files changed, 29 insertions(+), 11 deletions(-) diff --git a/net/rds/ib_send.c b/net

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

2017-01-02 Thread Santosh Shilimkar
for future without need of change in the interface. Reviewed-by: Sowmini Varadhan Signed-off-by: Santosh Shilimkar --- include/uapi/linux/rds.h | 33 + net/rds/af_rds.c | 28 net/rds/ib_recv.c| 4 net/rds/rds.h

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

2017-01-02 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/rd

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

2017-01-02 Thread Santosh Shilimkar
Tracks the ib receive cache total, incoming and frag allocations. Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- 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/n

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

2017-01-02 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 5f02b4d..c62e551

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

2017-01-02 Thread Santosh Shilimkar
From: Venkat Venkatsubra <venkat.x.venkatsu...@oracle.com> Tracks the receive side memory added to scokets and removed from sockets. Signed-off-by: Venkat Venkatsubra <venkat.x.venkatsu...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> ---

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

2017-01-02 Thread Santosh Shilimkar
Useful to know the active and passive end points in a RDS IB connection. Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- 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/rd

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

2017-01-02 Thread Santosh Shilimkar
. Reviewed-by: Wengang <wen.gang.w...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- net/rds/ib.h| 1 + net/rds/ib_cm.c | 12 2 files changed, 13 insertions(+) diff --git a/net/rds/ib.h b/net/rds/ib.h index 1fe9f79..5404589 100644 -

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

2017-01-02 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 b/net/rds

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

2017-01-02 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..5f02b4d 100644

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

2017-01-02 Thread Santosh Shilimkar
. Reviewed-by: Wengang Signed-off-by: Santosh Shilimkar --- net/rds/ib.h| 1 + net/rds/ib_cm.c | 12 2 files changed, 13 insertions(+) diff --git a/net/rds/ib.h b/net/rds/ib.h index 1fe9f79..5404589 100644 --- a/net/rds/ib.h +++ b/net/rds/ib.h @@ -185,6 +185,7 @@ struct

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

2017-01-02 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 <santosh.shilim...@oracle.com> --- net/rds/ib.c| 11 +++ net/rds/ib.h| 5 + net/rds/ib_cm.

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

2017-01-02 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

Re: [PATCH 1/2] soc: ti: knav_dma: fix typos in trace message

2016-12-20 Thread Santosh Shilimkar
On 12/20/2016 2:24 PM, Murali Karicheri wrote: This patch fixes some typos in the trace message Signed-off-by: Murali Karicheri Signed-off-by: Sekhar Nori --- Patch 1 and 2 looks fine. Will pick them for next merge window. Thanks !! Regards, Santosh

Re: [PATCH 1/2] soc: ti: knav_dma: fix typos in trace message

2016-12-20 Thread Santosh Shilimkar
On 12/20/2016 2:24 PM, Murali Karicheri wrote: This patch fixes some typos in the trace message Signed-off-by: Murali Karicheri Signed-off-by: Sekhar Nori --- Patch 1 and 2 looks fine. Will pick them for next merge window. Thanks !! Regards, Santosh

Re: [PATCH] RDS: use rb_entry()

2016-12-20 Thread Santosh Shilimkar
On 12/20/2016 6:02 AM, Geliang Tang wrote: To make the code clearer, use rb_entry() instead of container_of() to deal with rbtree. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- Looks fine. Acked-by: Santosh Shilimkar <santosh.shilim...@oracle.com>

Re: [PATCH] RDS: use rb_entry()

2016-12-20 Thread Santosh Shilimkar
On 12/20/2016 6:02 AM, Geliang Tang wrote: To make the code clearer, use rb_entry() instead of container_of() to deal with rbtree. Signed-off-by: Geliang Tang --- Looks fine. Acked-by: Santosh Shilimkar

Re: [PATCH] ARM: keystone: dts: fix netcp clocks and add names

2016-12-08 Thread Santosh Shilimkar
rii Strashko <grygorii.stras...@ti.com> --- Acked-by: Santosh Shilimkar <ssant...@kernel.org> Hi Arnd, Can you please pick this fix as well if possible in your non critical fixes branch ? Regards, Santosh

Re: [PATCH] ARM: keystone: dts: fix netcp clocks and add names

2016-12-08 Thread Santosh Shilimkar
On 12/8/2016 1:11 PM, Grygorii Strashko wrote: From: Murali Karicheri Fix the pa clock to point to the clkpa which has clock rate of 1/3 of PA PLL clock and add clock names. Signed-off-by: Murali Karicheri Signed-off-by: Grygorii Strashko --- Acked-by: Santosh Shilimkar Hi Arnd, Can you

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 9:36 AM, David Miller wrote: [...] What does the newer tool do on an older kernel if it doesn't see the fields? Does it check the size of the structure given back to it, and conditionally handle the older vs. the newer layout? It must do this. Right but the rds-tool doesn't

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 9:36 AM, David Miller wrote: [...] What does the newer tool do on an older kernel if it doesn't see the fields? Does it check the size of the structure given back to it, and conditionally handle the older vs. the newer layout? It must do this. Right but the rds-tool doesn't

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 9:05 AM, David Miller wrote: From: Santosh Shilimkar <santosh.shilim...@oracle.com> Date: Wed, 7 Dec 2016 08:44:04 -0800 On 12/7/2016 7:55 AM, David Miller wrote: From: Santosh Shilimkar <santosh.shilim...@oracle.com> Date: Tue, 6 Dec 2016 20:01:56 -0800

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 9:05 AM, David Miller wrote: From: Santosh Shilimkar Date: Wed, 7 Dec 2016 08:44:04 -0800 On 12/7/2016 7:55 AM, David Miller wrote: From: Santosh Shilimkar Date: Tue, 6 Dec 2016 20:01:56 -0800 What level of compatability exists here? If we run an old tool on a new kernel

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 7:55 AM, David Miller wrote: From: Santosh Shilimkar <santosh.shilim...@oracle.com> Date: Tue, 6 Dec 2016 20:01:56 -0800 rds-tools already support it. Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.com> --- include/uapi/linux/rds.h | 1 +

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 7:55 AM, David Miller wrote: From: Santosh Shilimkar Date: Tue, 6 Dec 2016 20:01:56 -0800 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

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 7:53 AM, David Miller wrote: From: Santosh Shilimkar <santosh.shilim...@oracle.com> Date: Tue, 6 Dec 2016 20:01:48 -0800 @@ -181,6 +181,9 @@ struct rds_ib_connection { /* Batched completions */ unsigned inti_unsignaled_wrs; + + /* Endpoin

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

2016-12-07 Thread Santosh Shilimkar
On 12/7/2016 7:53 AM, David Miller wrote: From: Santosh Shilimkar Date: Tue, 6 Dec 2016 20:01:48 -0800 @@ -181,6 +181,9 @@ struct rds_ib_connection { /* Batched completions */ unsigned inti_unsignaled_wrs; + + /* Endpoint role in connection */ + int

[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 <santosh.shilim...@oracle.com> --- 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/rd

[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 <santosh.shilim...@oracle.com> --- net/rds/ib.h | 4 +++- net/rds/i

[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/rds

[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 <qing.hu...@oracle.com> This prevents RDS from handling incoming rdma packets before RDS completes initializing its recv/send components. Signed-off-by: Qing Huang <qing.hu...@oracle.com> Signed-off-by: Santosh Shilimkar <santosh.shilim...@oracle.co

[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 <santosh.shilim...@oracle.com> --- 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/n

[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 <santosh.shilim...@oracle.com> --- net/rds/ib.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/rds/

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