Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket

2017-09-20 Thread Vallish Vaidyeshwara
On Sat, Sep 16, 2017 at 11:47:56AM +0200, Thomas Gleixner wrote: > On Fri, 8 Sep 2017, Eric Dumazet wrote: > > On Fri, 2017-09-08 at 11:55 -0700, Eduardo Valentin wrote: > > > Hello, > > > > > > On Fri, Sep 08, 2017 at 10:26:45AM -0700, David Miller wrote: > > > > From: David Woodhouse > > > > Da

Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket

2017-08-27 Thread Vallish Vaidyeshwara
On Tue, Aug 22, 2017 at 09:30:30PM -0700, David Miller wrote: > From: Vallish Vaidyeshwara > Date: Wed, 23 Aug 2017 00:10:25 + > > > I am submitting 2 patch series to enable hires timer to timeout > > datagram sockets (AF_UNIX & AF_INET domain) and test code to tes

[PATCH v2 2/2] selftests/net: add test to verify datagram socket timeout

2017-08-22 Thread Vallish Vaidyeshwara
AF_INET socket is the same kernel function used by AF_UNIX as well which is __skb_wait_for_more_packets(). Reported-by: Manjula Peiris Reviewed-by: Eduardo Valentin Reviewed-by: Anchal Agarwal Signed-off-by: Vallish Vaidyeshwara --- tools/testing/selftests/net/Makefile | 3

[PATCH v2 1/2] net: enable high resolution timer mode to timeout datagram sockets

2017-08-22 Thread Vallish Vaidyeshwara
y: Manjula Peiris Reviewed-by: Eduardo Valentin Reviewed-by: Anchal Agarwal Signed-off-by: Vallish Vaidyeshwara --- net/core/datagram.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/net/core/datagram.c b/net/core/datagram.c index ee5647b..832c147 100644 ---

[PATCH v2 0/2] enable hires timer to timeout datagram socket

2017-08-22 Thread Vallish Vaidyeshwara
timeout behavior related to patch 1 Vallish Vaidyeshwara (2): net: enable high resolution timer mode to timeout datagram sockets selftests/net: add test to verify datagram socket timeout net/core/datagram.c| 14 ++- tools/testing/selftests/net/M

Re: [PATCH RESEND 0/2] enable hires timer to timeout datagram socket

2017-08-22 Thread Vallish Vaidyeshwara
On Tue, Aug 22, 2017 at 08:23:11AM +0200, Richard Cochran wrote: > On Mon, Aug 21, 2017 at 06:22:10PM +0000, Vallish Vaidyeshwara wrote: > > AWS Lambda is affected by this change in behavior in > > system call. Following links has more information: > > https://en.wikipedi

Re: [PATCH RESEND 1/2] net: enable high resolution timer mode to timeout datagram sockets

2017-08-22 Thread Vallish Vaidyeshwara
On Mon, Aug 21, 2017 at 01:10:34PM -0700, Cong Wang wrote: > On Fri, Aug 18, 2017 at 11:44 AM, Vallish Vaidyeshwara > wrote: > > - *timeo_p = schedule_timeout(*timeo_p); > > + /* Wait using highres timer */ > > + expires = ktime_add_ns(ktime_get(), j

Re: [PATCH RESEND 0/2] enable hires timer to timeout datagram socket

2017-08-21 Thread Vallish Vaidyeshwara
On Sun, Aug 20, 2017 at 01:47:45AM +, Vallish Vaidyeshwara wrote: > On Sat, Aug 19, 2017 at 08:21:45AM +0200, Richard Cochran wrote: > > On Fri, Aug 18, 2017 at 10:27:56PM +, Vallish Vaidyeshwara wrote: > > > We have a on-demand application that uses long timeouts and nee

Re: [PATCH RESEND 0/2] enable hires timer to timeout datagram socket

2017-08-19 Thread Vallish Vaidyeshwara
On Sat, Aug 19, 2017 at 08:21:45AM +0200, Richard Cochran wrote: > On Fri, Aug 18, 2017 at 10:27:56PM +0000, Vallish Vaidyeshwara wrote: > > We have a on-demand application that uses long timeouts and needs to react > > to > > events within milliseconds. > Hello Ri

Re: [PATCH RESEND 0/2] enable hires timer to timeout datagram socket

2017-08-18 Thread Vallish Vaidyeshwara
On Fri, Aug 18, 2017 at 10:18:54PM +0200, Richard Cochran wrote: > On Fri, Aug 18, 2017 at 06:44:08PM +0000, Vallish Vaidyeshwara wrote: > > There has been a behavior change in 4.9 kernel with refactoring of Kernel > > timer wheel in 4.8. We have a use case wherein our

[PATCH RESEND 2/2] selftests/net: add test to verify datagram socket timeout

2017-08-18 Thread Vallish Vaidyeshwara
AF_INET socket is the same kernel function used by AF_UNIX as well which is __skb_wait_for_more_packets(). Reported-by: Manjula Peiris Reviewed-by: Eduardo Valentin Reviewed-by: Anchal Agarwal Signed-off-by: Vallish Vaidyeshwara --- tools/testing/selftests/net/Makefile | 3

[PATCH RESEND 0/2] enable hires timer to timeout datagram socket

2017-08-18 Thread Vallish Vaidyeshwara
and AF_INET domain Patch 2: Test code to report regression in timeout behavior related to patch 1 Vallish Vaidyeshwara (2): net: enable high resolution timer mode to timeout datagram sockets selftests/net: add test to verify datagram socket timeout net/core/datagr

[PATCH RESEND 1/2] net: enable high resolution timer mode to timeout datagram sockets

2017-08-18 Thread Vallish Vaidyeshwara
y: Manjula Peiris Reviewed-by: Eduardo Valentin Reviewed-by: Anchal Agarwal Signed-off-by: Vallish Vaidyeshwara --- net/core/datagram.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/net/core/datagram.c b/net/core/datagram.c index ee5647b..c89a104 100644 ---