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

From	Eduardo Valentin <eduval@amazon.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket
Date	2017-09-08 19:10 +0200
Message-ID	<unuX0-1th-15@gated-at.bofh.it> (permalink)
References	<uhryN-1AG-9@gated-at.bofh.it> <uhvCq-4iI-15@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

David,

On Tue, Aug 22, 2017 at 09:30:30PM -0700, David Miller wrote:
> From: Vallish Vaidyeshwara <vallish@amazon.com>
> Date: Wed, 23 Aug 2017 00:10:25 +0000
> 
> > 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 with real guarantees, please use
> the kernel interfaces which provide this as explained to you as
> feedback by other reviewers.

I understand the kernel provides other interfaces.

> 
> I'm not applying this, sorry.

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, to me, looks like breaking userspace, isn't it?

If the correct recommendation is to use different system calls this should have been mentioned in system call documentation before changing its behavior, not expect the user to figure out after kernel release/upgrade, right?

-- 
All the best,
Eduardo Valentin

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket Eduardo Valentin <eduval@amazon.com> - 2017-09-08 19:10 +0200
  Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket David Miller <davem@davemloft.net> - 2017-09-08 19:20 +0200
    Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket David Miller <davem@davemloft.net> - 2017-09-08 19:30 +0200
      Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket Eduardo Valentin <eduval@amazon.com> - 2017-09-08 21:00 +0200
        Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket Eric Dumazet <eric.dumazet@gmail.com> - 2017-09-08 21:20 +0200
        Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket David Miller <davem@davemloft.net> - 2017-09-08 23:50 +0200
    Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket David Woodhouse <dwmw2@infradead.org> - 2017-09-08 19:30 +0200

csiph-web