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

2017-09-25 Thread Thomas Gleixner
On Wed, 20 Sep 2017, Vallish Vaidyeshwara wrote: > On Sat, Sep 16, 2017 at 11:47:56AM +0200, Thomas Gleixner wrote: > > > So if we need to replace all 'legacy' timers to high resolution timer, > > > because some application was _relying_ on jiffies being kind of precise, > > > maybe it is better to

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-09-16 Thread Thomas Gleixner
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 > > > Date: Fri, 08 Sep 2017 18:23:22 +0100 > > > > > > > I don't know that anyone's e

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

2017-09-08 Thread David Miller
From: Eduardo Valentin Date: Fri, 8 Sep 2017 11:55:21 -0700 > I agree. I would prefer to understand here what is the technical > reason not to accept these patches other than "use other system > calls". I explained this, let me reiterate: And most importantly, letting the k

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

2017-09-08 Thread Eric Dumazet
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 > > Date: Fri, 08 Sep 2017 18:23:22 +0100 > > > > > I don't know that anyone's ever tried saying "show me the chapter > and > > > verse

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

2017-09-08 Thread Eduardo Valentin
Hello, On Fri, Sep 08, 2017 at 10:26:45AM -0700, David Miller wrote: > From: David Woodhouse > Date: Fri, 08 Sep 2017 18:23:22 +0100 > > > I don't know that anyone's ever tried saying "show me the chapter and > > verse of the documentation" > > Do you know why I brought this up? Because the pe

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

2017-09-08 Thread David Miller
From: David Woodhouse Date: Fri, 08 Sep 2017 18:23:22 +0100 > I don't know that anyone's ever tried saying "show me the chapter and > verse of the documentation" Do you know why I brought this up? Because the person I am replying to told me that the syscall documentation should have suggested t

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

2017-09-08 Thread David Woodhouse
On Fri, 2017-09-08 at 10:16 -0700, David Miller wrote: > From: Eduardo Valentin > Date: Fri, 8 Sep 2017 10:04:09 -0700 > > > > > However, this is a clear, the system call, from the net subsystem, > > has changed in behavior across kernel versions. From application / > > userspace perspective, ch

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

2017-09-08 Thread David Miller
From: Eduardo Valentin Date: Fri, 8 Sep 2017 10:04:09 -0700 > However, this is a clear, the system call, from the net subsystem, > has changed in behavior across kernel versions. From application / > userspace perspective, changing the system call without clear > documentation or deprecation path

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

2017-09-08 Thread Eduardo Valentin
David, 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 test > > timeout accura

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

2017-08-29 Thread David Laight
From: Vallish Vaidyeshwara > Sent: 27 August 2017 21:47 ... > I respect the decision not to upstream this patch series, however I > wanted to provide additional details. Application wanting high > resolution events with real guarantees is not the case, but the case > here is regression in system ca

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 test > > timeout accuracy on th

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

2017-08-22 Thread David Miller
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 test > timeout accuracy on these sockets. This is not reasonable. If you want high resolution events

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

2017-08-22 Thread Vallish Vaidyeshwara
v1->v2: - Cong Wang pointed out MAX_SCHEDULE_TIMEOUT wraparound, fixed the patch 1/2 to accommodate MAX_SCHEDULE_TIMEOUT wait time - Changed format of printing total time from float to long in selftests patch 2/2 Hello Dave, Resending the patch series to include netdev mailing list with a