Re: svn commit: r355600 - in head: share/man/man9 sys/kern sys/sys

2019-12-11 Thread Hans Petter Selasky
On 2019-12-11 10:26, Alexey Dokuchaev wrote: That's one of those things that make Linux so unpleasant to work with. Frankly, I don't think we want that for FreeBSD. The argument for the current callback is still a "void *". container_of() has some type checks at least. It is not a big deal tho

Re: svn commit: r355600 - in head: share/man/man9 sys/kern sys/sys

2019-12-11 Thread Alexey Dokuchaev
On Tue, Dec 10, 2019 at 11:09:41PM +0100, Hans Petter Selasky wrote: > On 2019-12-10 22:58, John Baldwin wrote: > > While here, add to the manpage. > > FYI: > > Linux guys eliminated the "void *c_arg" in their timer implementation by > using container_of() to get callback argument. We could po

Re: svn commit: r355600 - in head: share/man/man9 sys/kern sys/sys

2019-12-11 Thread Hans Petter Selasky
On 2019-12-10 23:47, John Baldwin wrote: You mean passing the pointer to the callout itself and using that to get to the relevant pointer? Yes. --HPS ___ svn-src-all@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-all To u

Re: svn commit: r355600 - in head: share/man/man9 sys/kern sys/sys

2019-12-10 Thread Poul-Henning Kamp
In message , John Baldwin writes: >On 12/10/19 2:09 PM, Hans Petter Selasky wrote: >> On 2019-12-10 22:58, John Baldwin wrote: >>> While here, add to the manpage. >> >> FYI: >> >> Linux guys eliminated the "void *c_arg" in their timer implementation by >> using container_of() to get

Re: svn commit: r355600 - in head: share/man/man9 sys/kern sys/sys

2019-12-10 Thread John Baldwin
On 12/10/19 2:09 PM, Hans Petter Selasky wrote: > On 2019-12-10 22:58, John Baldwin wrote: >> While here, add to the manpage. > > FYI: > > Linux guys eliminated the "void *c_arg" in their timer implementation by > using container_of() to get callback argument. We could possibly do the > same

Re: svn commit: r355600 - in head: share/man/man9 sys/kern sys/sys

2019-12-10 Thread Hans Petter Selasky
On 2019-12-10 22:58, John Baldwin wrote: While here, add to the manpage. FYI: Linux guys eliminated the "void *c_arg" in their timer implementation by using container_of() to get callback argument. We could possibly do the same! --HPS ___ svn-

svn commit: r355600 - in head: share/man/man9 sys/kern sys/sys

2019-12-10 Thread John Baldwin
Author: jhb Date: Tue Dec 10 21:58:30 2019 New Revision: 355600 URL: https://svnweb.freebsd.org/changeset/base/355600 Log: Add a callout_func_t typedef for functions used with callout_*(). This typedef is the same as timeout_t except that it is in the callout namespace and header. Us