Re: [PATCH 01/13] kdbus: add documentation

2015-02-08 Thread Andy Lutomirski
On Feb 4, 2015 4:16 PM, "David Herrmann" wrote: > > Hi > > On Thu, Feb 5, 2015 at 12:03 AM, Andy Lutomirski wrote: > > I see "latencies" of around 20 microseconds with lockdep and context > > tracking off. For example: > > Without metadata nor memfd transmission, I get 2.5us for kdbus, 1.5us >

Re: [PATCH 01/13] kdbus: add documentation

2015-02-08 Thread Andy Lutomirski
On Feb 4, 2015 4:16 PM, David Herrmann dh.herrm...@gmail.com wrote: Hi On Thu, Feb 5, 2015 at 12:03 AM, Andy Lutomirski l...@amacapital.net wrote: I see latencies of around 20 microseconds with lockdep and context tracking off. For example: Without metadata nor memfd transmission, I get

Re: [PATCH 01/13] kdbus: add documentation

2015-02-04 Thread David Herrmann
Hi On Thu, Feb 5, 2015 at 12:03 AM, Andy Lutomirski wrote: > I see "latencies" of around 20 microseconds with lockdep and context > tracking off. For example: Without metadata nor memfd transmission, I get 2.5us for kdbus, 1.5us for UDS (8k payload). With 8-byte payloads, I get 2.2us and

Re: [PATCH 01/13] kdbus: add documentation

2015-02-04 Thread Andy Lutomirski
On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack wrote: > Hi Andy, > > On 02/02/2015 09:12 PM, Andy Lutomirski wrote: >> On Feb 2, 2015 1:34 AM, "Daniel Mack" wrote: > >>> That's right, but again - if an application wants to gather this kind of >>> information about tasks it interacts with, it can do

Re: [PATCH 01/13] kdbus: add documentation

2015-02-04 Thread Andy Lutomirski
On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack dan...@zonque.org wrote: Hi Andy, On 02/02/2015 09:12 PM, Andy Lutomirski wrote: On Feb 2, 2015 1:34 AM, Daniel Mack dan...@zonque.org wrote: That's right, but again - if an application wants to gather this kind of information about tasks it

Re: [PATCH 01/13] kdbus: add documentation

2015-02-04 Thread David Herrmann
Hi On Thu, Feb 5, 2015 at 12:03 AM, Andy Lutomirski l...@amacapital.net wrote: I see latencies of around 20 microseconds with lockdep and context tracking off. For example: Without metadata nor memfd transmission, I get 2.5us for kdbus, 1.5us for UDS (8k payload). With 8-byte payloads, I get

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Eric W. Biederman
Greg Kroah-Hartman writes: > On Tue, Feb 03, 2015 at 08:47:51PM -0600, Eric W. Biederman wrote: >> Andy Lutomirski writes: >> >> > On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack wrote: >> >> Hi Andy, >> >> >> >> On 02/02/2015 09:12 PM, Andy Lutomirski wrote: >> >>> On Feb 2, 2015 1:34 AM,

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Greg Kroah-Hartman
On Tue, Feb 03, 2015 at 08:47:51PM -0600, Eric W. Biederman wrote: > Andy Lutomirski writes: > > > On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack wrote: > >> Hi Andy, > >> > >> On 02/02/2015 09:12 PM, Andy Lutomirski wrote: > >>> On Feb 2, 2015 1:34 AM, "Daniel Mack" wrote: > >> > That's

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Eric W. Biederman
Andy Lutomirski writes: > On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack wrote: >> Hi Andy, >> >> On 02/02/2015 09:12 PM, Andy Lutomirski wrote: >>> On Feb 2, 2015 1:34 AM, "Daniel Mack" wrote: >> That's right, but again - if an application wants to gather this kind of information about

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Andy Lutomirski
On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack wrote: > Hi Andy, > > On 02/02/2015 09:12 PM, Andy Lutomirski wrote: >> On Feb 2, 2015 1:34 AM, "Daniel Mack" wrote: > >>> That's right, but again - if an application wants to gather this kind of >>> information about tasks it interacts with, it can do

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Daniel Mack
Hi Andy, On 02/02/2015 09:12 PM, Andy Lutomirski wrote: > On Feb 2, 2015 1:34 AM, "Daniel Mack" wrote: >> That's right, but again - if an application wants to gather this kind of >> information about tasks it interacts with, it can do so today by looking >> at /proc or similar sources. Desktop

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Eric W. Biederman
Greg Kroah-Hartman gre...@linuxfoundation.org writes: On Tue, Feb 03, 2015 at 08:47:51PM -0600, Eric W. Biederman wrote: Andy Lutomirski l...@amacapital.net writes: On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack dan...@zonque.org wrote: Hi Andy, On 02/02/2015 09:12 PM, Andy Lutomirski

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Eric W. Biederman
Andy Lutomirski l...@amacapital.net writes: On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack dan...@zonque.org wrote: Hi Andy, On 02/02/2015 09:12 PM, Andy Lutomirski wrote: On Feb 2, 2015 1:34 AM, Daniel Mack dan...@zonque.org wrote: That's right, but again - if an application wants to gather

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Greg Kroah-Hartman
On Tue, Feb 03, 2015 at 08:47:51PM -0600, Eric W. Biederman wrote: Andy Lutomirski l...@amacapital.net writes: On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack dan...@zonque.org wrote: Hi Andy, On 02/02/2015 09:12 PM, Andy Lutomirski wrote: On Feb 2, 2015 1:34 AM, Daniel Mack

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Andy Lutomirski
On Tue, Feb 3, 2015 at 2:09 AM, Daniel Mack dan...@zonque.org wrote: Hi Andy, On 02/02/2015 09:12 PM, Andy Lutomirski wrote: On Feb 2, 2015 1:34 AM, Daniel Mack dan...@zonque.org wrote: That's right, but again - if an application wants to gather this kind of information about tasks it

Re: [PATCH 01/13] kdbus: add documentation

2015-02-03 Thread Daniel Mack
Hi Andy, On 02/02/2015 09:12 PM, Andy Lutomirski wrote: On Feb 2, 2015 1:34 AM, Daniel Mack dan...@zonque.org wrote: That's right, but again - if an application wants to gather this kind of information about tasks it interacts with, it can do so today by looking at /proc or similar sources.

Re: [PATCH 01/13] kdbus: add documentation

2015-02-02 Thread Andy Lutomirski
On Feb 2, 2015 1:34 AM, "Daniel Mack" wrote: > > Hi Andy, > > On 01/29/2015 01:09 PM, Andy Lutomirski wrote: > > On Jan 29, 2015 6:42 AM, "Daniel Mack" wrote: > > >> As we explained before, currently, D-Bus peers do collect the same > >> information already if they need to have them, but they

Re: [PATCH 01/13] kdbus: add documentation

2015-02-02 Thread Daniel Mack
Hi Andy, On 01/29/2015 01:09 PM, Andy Lutomirski wrote: > On Jan 29, 2015 6:42 AM, "Daniel Mack" wrote: >> As we explained before, currently, D-Bus peers do collect the same >> information already if they need to have them, but they have to do deal >> with the inherit races in such cases. kdbus

Re: [PATCH 01/13] kdbus: add documentation

2015-02-02 Thread Andy Lutomirski
On Feb 2, 2015 1:34 AM, Daniel Mack dan...@zonque.org wrote: Hi Andy, On 01/29/2015 01:09 PM, Andy Lutomirski wrote: On Jan 29, 2015 6:42 AM, Daniel Mack dan...@zonque.org wrote: As we explained before, currently, D-Bus peers do collect the same information already if they need to have

Re: [PATCH 01/13] kdbus: add documentation

2015-02-02 Thread Daniel Mack
Hi Andy, On 01/29/2015 01:09 PM, Andy Lutomirski wrote: On Jan 29, 2015 6:42 AM, Daniel Mack dan...@zonque.org wrote: As we explained before, currently, D-Bus peers do collect the same information already if they need to have them, but they have to do deal with the inherit races in such

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Andy Lutomirski
On Jan 29, 2015 6:42 AM, "Daniel Mack" wrote: > > On 01/29/2015 12:25 PM, Andy Lutomirski wrote: > > On Jan 29, 2015 3:53 AM, "Daniel Mack" wrote: > > >> Also note that if a receiving peer opts in for a certain piece of > >> metadata, it should do that that for a good reason, because it needs >

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Daniel Mack
On 01/29/2015 12:25 PM, Andy Lutomirski wrote: > On Jan 29, 2015 3:53 AM, "Daniel Mack" wrote: >> Also note that if a receiving peer opts in for a certain piece of >> metadata, it should do that that for a good reason, because it needs >> that data to process a request. Letting kdbus do the work

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Andy Lutomirski
On Jan 29, 2015 3:53 AM, "Daniel Mack" wrote: > > Hi Andy, > > On 01/27/2015 05:03 PM, Andy Lutomirski wrote: > > On Tue, Jan 27, 2015 at 7:05 AM, David Herrmann > > wrote: > > >> A 16byte copy does not affect the performance of kdbus message > >> transactions in any way that matters. > > >

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Daniel Mack
Hi Andy, On 01/27/2015 05:03 PM, Andy Lutomirski wrote: > On Tue, Jan 27, 2015 at 7:05 AM, David Herrmann wrote: >> A 16byte copy does not affect the performance of kdbus message >> transactions in any way that matters. > What are the performance goals of kdbus? How fast is it ever intended >

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Daniel Mack
Hi Andy, On 01/27/2015 05:03 PM, Andy Lutomirski wrote: On Tue, Jan 27, 2015 at 7:05 AM, David Herrmann dh.herrm...@gmail.com wrote: A 16byte copy does not affect the performance of kdbus message transactions in any way that matters. What are the performance goals of kdbus? How fast is it

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Daniel Mack
On 01/29/2015 12:25 PM, Andy Lutomirski wrote: On Jan 29, 2015 3:53 AM, Daniel Mack dan...@zonque.org wrote: Also note that if a receiving peer opts in for a certain piece of metadata, it should do that that for a good reason, because it needs that data to process a request. Letting kdbus do

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Andy Lutomirski
On Jan 29, 2015 6:42 AM, Daniel Mack dan...@zonque.org wrote: On 01/29/2015 12:25 PM, Andy Lutomirski wrote: On Jan 29, 2015 3:53 AM, Daniel Mack dan...@zonque.org wrote: Also note that if a receiving peer opts in for a certain piece of metadata, it should do that that for a good reason,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-29 Thread Andy Lutomirski
On Jan 29, 2015 3:53 AM, Daniel Mack dan...@zonque.org wrote: Hi Andy, On 01/27/2015 05:03 PM, Andy Lutomirski wrote: On Tue, Jan 27, 2015 at 7:05 AM, David Herrmann dh.herrm...@gmail.com wrote: A 16byte copy does not affect the performance of kdbus message transactions in any way

Re: [PATCH 01/13] kdbus: add documentation

2015-01-28 Thread Michael Kerrisk (man-pages)
Hello Daniel, On 01/27/2015 07:14 PM, Daniel Mack wrote: > Hi Michael, > > On 01/27/2015 06:53 PM, Michael Kerrisk (man-pages) wrote: >> On 01/27/2015 04:23 PM, David Herrmann wrote: > >>> I only expect a handful of users to call the ioctls directly. The >>> libraries that implement the

Re: [PATCH 01/13] kdbus: add documentation

2015-01-28 Thread Michael Kerrisk (man-pages)
Hello Daniel, On 01/27/2015 07:14 PM, Daniel Mack wrote: Hi Michael, On 01/27/2015 06:53 PM, Michael Kerrisk (man-pages) wrote: On 01/27/2015 04:23 PM, David Herrmann wrote: I only expect a handful of users to call the ioctls directly. The libraries that implement the payload-marshaling,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Daniel Mack
Hi Michael, On 01/27/2015 06:53 PM, Michael Kerrisk (man-pages) wrote: > On 01/27/2015 04:23 PM, David Herrmann wrote: >> I only expect a handful of users to call the ioctls directly. The >> libraries that implement the payload-marshaling, in particular. It's a >> similar situation with netlink.

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Michael Kerrisk (man-pages)
Hi David, On 01/27/2015 04:05 PM, David Herrmann wrote: > Hi > > On Mon, Jan 26, 2015 at 3:46 PM, Michael Kerrisk (man-pages) > wrote: >> Hello Greg, >> >> On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: >>> On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: While I

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Michael Kerrisk (man-pages)
On 01/27/2015 04:23 PM, David Herrmann wrote: > Hi > > On Mon, Jan 26, 2015 at 5:45 PM, Michael Kerrisk (man-pages) > wrote: >> On 01/26/2015 04:26 PM, Tom Gundersen wrote: >>> On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) >>> wrote: 2. Is the API to be invoked directly by

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Andy Lutomirski
On Tue, Jan 27, 2015 at 7:05 AM, David Herrmann wrote: > Hi > > On Mon, Jan 26, 2015 at 3:46 PM, Michael Kerrisk (man-pages) > wrote: >> Hello Greg, >> >> On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: >>> On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: While I agree

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread David Herrmann
Hi On Mon, Jan 26, 2015 at 5:45 PM, Michael Kerrisk (man-pages) wrote: > On 01/26/2015 04:26 PM, Tom Gundersen wrote: >> On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) >> wrote: >>> 2. Is the API to be invoked directly by applications or is intended to >>>be used only behind

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread David Herrmann
Hi On Mon, Jan 26, 2015 at 3:46 PM, Michael Kerrisk (man-pages) wrote: > Hello Greg, > > On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: >> On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: >>> While I agree that there should be a way for userspace to get the list of >>>

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Daniel Mack
Hi Michael, On 01/27/2015 06:53 PM, Michael Kerrisk (man-pages) wrote: On 01/27/2015 04:23 PM, David Herrmann wrote: I only expect a handful of users to call the ioctls directly. The libraries that implement the payload-marshaling, in particular. It's a similar situation with netlink.

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread David Herrmann
Hi On Mon, Jan 26, 2015 at 3:46 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: Hello Greg, On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: While I agree that there should be a way for userspace to get the

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread David Herrmann
Hi On Mon, Jan 26, 2015 at 5:45 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: On 01/26/2015 04:26 PM, Tom Gundersen wrote: On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: 2. Is the API to be invoked directly by applications or is

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Andy Lutomirski
On Tue, Jan 27, 2015 at 7:05 AM, David Herrmann dh.herrm...@gmail.com wrote: Hi On Mon, Jan 26, 2015 at 3:46 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: Hello Greg, On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Michael Kerrisk (man-pages)
On 01/27/2015 04:23 PM, David Herrmann wrote: Hi On Mon, Jan 26, 2015 at 5:45 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: On 01/26/2015 04:26 PM, Tom Gundersen wrote: On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: 2. Is the API

Re: [PATCH 01/13] kdbus: add documentation

2015-01-27 Thread Michael Kerrisk (man-pages)
Hi David, On 01/27/2015 04:05 PM, David Herrmann wrote: Hi On Mon, Jan 26, 2015 at 3:46 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: Hello Greg, On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: While

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Michael Kerrisk (man-pages)
On 01/26/2015 04:26 PM, Tom Gundersen wrote: > Hi Michael, > > On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) > wrote: >> 2. Is the API to be invoked directly by applications or is intended to >>be used only behind specific libraries? You seem to be saying that >>the latter

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread christoph Hellwig
On Mon, Jan 26, 2015 at 04:26:53PM +0100, Tom Gundersen wrote: > The way I read this is that there will (probably) be a handful of > users, namely the existing dbus libraries: libdus, sd-bus, glib, Qt, > ell, and maybe a few others. However, third-party developers will not > know/care about the

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Tom Gundersen
Hi Michael, On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) wrote: > 2. Is the API to be invoked directly by applications or is intended to >be used only behind specific libraries? You seem to be saying that >the latter is the case (here, I'm referring to your comment above

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Michael Kerrisk (man-pages)
Hello Greg, On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: > On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: >> While I agree that there should be a way for userspace to get the list of >> supported operations, userspace apps will only actually care about that >> once,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Michael Kerrisk (man-pages)
Hi Greg, First of all, I seem to have offended you. That was not my intention. It's certainly not my intent to disparage you or your work (or for that matter, the other kdbus developers). Insofar as any of the wordings I've used suggested otherwise, I do apologize. I'll comment on various

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Michael Kerrisk (man-pages)
Hi Greg, First of all, I seem to have offended you. That was not my intention. It's certainly not my intent to disparage you or your work (or for that matter, the other kdbus developers). Insofar as any of the wordings I've used suggested otherwise, I do apologize. I'll comment on various

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Michael Kerrisk (man-pages)
On 01/26/2015 04:26 PM, Tom Gundersen wrote: Hi Michael, On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: 2. Is the API to be invoked directly by applications or is intended to be used only behind specific libraries? You seem to be saying that

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Tom Gundersen
Hi Michael, On Mon, Jan 26, 2015 at 3:42 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: 2. Is the API to be invoked directly by applications or is intended to be used only behind specific libraries? You seem to be saying that the latter is the case (here, I'm referring to

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread christoph Hellwig
On Mon, Jan 26, 2015 at 04:26:53PM +0100, Tom Gundersen wrote: The way I read this is that there will (probably) be a handful of users, namely the existing dbus libraries: libdus, sd-bus, glib, Qt, ell, and maybe a few others. However, third-party developers will not know/care about the

Re: [PATCH 01/13] kdbus: add documentation

2015-01-26 Thread Michael Kerrisk (man-pages)
Hello Greg, On 01/23/2015 05:08 PM, Greg Kroah-Hartman wrote: On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: While I agree that there should be a way for userspace to get the list of supported operations, userspace apps will only actually care about that once, when they

Re: [PATCH 01/13] kdbus: add documentation

2015-01-24 Thread Ahmed S. Darwish
On Fri, Jan 23, 2015 at 09:19:46PM +0800, Greg Kroah-Hartman wrote: > On Fri, Jan 23, 2015 at 08:28:20AM +0200, Ahmed S. Darwish wrote: > > On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: > > > From: Daniel Mack > > > > > > kdbus is a system for low-latency, low-overhead,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-24 Thread Ahmed S. Darwish
On Fri, Jan 23, 2015 at 09:19:46PM +0800, Greg Kroah-Hartman wrote: On Fri, Jan 23, 2015 at 08:28:20AM +0200, Ahmed S. Darwish wrote: On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org kdbus is a system for low-latency,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: > While I agree that there should be a way for userspace to get the list of > supported operations, userspace apps will only actually care about that > once, when they begin talking to kdbus, because (ignoring the live kernel >

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Thu, Jan 22, 2015 at 11:18:50AM +0100, Michael Kerrisk (man-pages) wrote: > >> And that process seems to be frequent and ongoing even now. (And > >> it's to your great credit that the API/ABI breaks are clearly and honestly > >> marked in the kdbus.h changelog.) All of this lightens the

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Fri, Jan 23, 2015 at 09:19:46PM +0800, Greg Kroah-Hartman wrote: > On Fri, Jan 23, 2015 at 08:28:20AM +0200, Ahmed S. Darwish wrote: > > On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: > > > From: Daniel Mack > > > > > > kdbus is a system for low-latency, low-overhead,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Fri, Jan 23, 2015 at 08:28:20AM +0200, Ahmed S. Darwish wrote: > On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: > > From: Daniel Mack > > > > kdbus is a system for low-latency, low-overhead, easy to use > > interprocess communication (IPC). > > > > The interface to all

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Michael Kerrisk (man-pages)
Hi David, On 01/22/2015 02:46 PM, David Herrmann wrote: > Hi Michael > > On Thu, Jan 22, 2015 at 11:18 AM, Michael Kerrisk (man-pages) > wrote: >> On 01/21/2015 05:58 PM, Daniel Mack wrote: > Also, the context the kdbus commands operate on originate from a > mountable special-purpose

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Fri, Jan 23, 2015 at 09:19:46PM +0800, Greg Kroah-Hartman wrote: On Fri, Jan 23, 2015 at 08:28:20AM +0200, Ahmed S. Darwish wrote: On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org kdbus is a system for low-latency,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Michael Kerrisk (man-pages)
Hi David, On 01/22/2015 02:46 PM, David Herrmann wrote: Hi Michael On Thu, Jan 22, 2015 at 11:18 AM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: On 01/21/2015 05:58 PM, Daniel Mack wrote: Also, the context the kdbus commands operate on originate from a mountable

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Fri, Jan 23, 2015 at 08:28:20AM +0200, Ahmed S. Darwish wrote: On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org kdbus is a system for low-latency, low-overhead, easy to use interprocess communication (IPC). The interface to

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Thu, Jan 22, 2015 at 09:49:00AM -0500, Austin S Hemmelgarn wrote: While I agree that there should be a way for userspace to get the list of supported operations, userspace apps will only actually care about that once, when they begin talking to kdbus, because (ignoring the live kernel

Re: [PATCH 01/13] kdbus: add documentation

2015-01-23 Thread Greg Kroah-Hartman
On Thu, Jan 22, 2015 at 11:18:50AM +0100, Michael Kerrisk (man-pages) wrote: And that process seems to be frequent and ongoing even now. (And it's to your great credit that the API/ABI breaks are clearly and honestly marked in the kdbus.h changelog.) All of this lightens the burden of API

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread Ahmed S. Darwish
On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: > From: Daniel Mack > > kdbus is a system for low-latency, low-overhead, easy to use > interprocess communication (IPC). > > The interface to all functions in this driver is implemented via ioctls > on files exposed through a

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread Austin S Hemmelgarn
On 2015-01-22 08:46, David Herrmann wrote: Hi Michael On Thu, Jan 22, 2015 at 11:18 AM, Michael Kerrisk (man-pages) wrote: * API oddities such as the 'kernel_flags' fields. Why do I need to be told what flags the kernel supports on *every* operation? If we only returned EINVAL on invalid

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread David Herrmann
Hi Michael On Thu, Jan 22, 2015 at 11:18 AM, Michael Kerrisk (man-pages) wrote: > On 01/21/2015 05:58 PM, Daniel Mack wrote: Also, the context the kdbus commands operate on originate from a mountable special-purpose file system. >>> >>> It's not clear to me how this point implies any

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread Michael Kerrisk (man-pages)
Hi Daniel, On 01/21/2015 05:58 PM, Daniel Mack wrote: > Hi Michael, > > On 01/21/2015 11:32 AM, Michael Kerrisk (man-pages) wrote: >> On 01/20/2015 07:23 PM, Daniel Mack wrote: > >>> It's rather an optional driver than a core kernel feature. >> >> Given the various things that I've seen said

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread David Herrmann
Hi Michael On Thu, Jan 22, 2015 at 11:18 AM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: On 01/21/2015 05:58 PM, Daniel Mack wrote: Also, the context the kdbus commands operate on originate from a mountable special-purpose file system. It's not clear to me how this point

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread Michael Kerrisk (man-pages)
Hi Daniel, On 01/21/2015 05:58 PM, Daniel Mack wrote: Hi Michael, On 01/21/2015 11:32 AM, Michael Kerrisk (man-pages) wrote: On 01/20/2015 07:23 PM, Daniel Mack wrote: It's rather an optional driver than a core kernel feature. Given the various things that I've seen said about kdbus,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread Austin S Hemmelgarn
On 2015-01-22 08:46, David Herrmann wrote: Hi Michael On Thu, Jan 22, 2015 at 11:18 AM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: * API oddities such as the 'kernel_flags' fields. Why do I need to be told what flags the kernel supports on *every* operation? If we only

Re: [PATCH 01/13] kdbus: add documentation

2015-01-22 Thread Ahmed S. Darwish
On Fri, Jan 16, 2015 at 11:16:05AM -0800, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org kdbus is a system for low-latency, low-overhead, easy to use interprocess communication (IPC). The interface to all functions in this driver is implemented via ioctls on files exposed

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Daniel Mack
Hi Michael, On 01/21/2015 11:32 AM, Michael Kerrisk (man-pages) wrote: > On 01/20/2015 07:23 PM, Daniel Mack wrote: >> It's rather an optional driver than a core kernel feature. > > Given the various things that I've seen said about kdbus, the > preceding sentence makes little sense to me: > >

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Theodore Ts'o
On Wed, Jan 21, 2015 at 11:32:59AM +0100, Michael Kerrisk (man-pages) wrote: > It's rather an optional driver than a core kernel feature. > > Given the various things that I've seen said about kdbus, the > preceding sentence makes little sense to me: > > * kdbus will be the framework supporting

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Hello Daniel, On 01/20/2015 07:23 PM, Daniel Mack wrote: > On 01/20/2015 02:53 PM, Michael Kerrisk (man-pages) wrote: >> This is an enormous and complex API. Why is the API ioctl() based, >> rather than system-call-based? Have we learned nothing from the hydra >> that the futex() multiplexing

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Hi David, On 01/20/2015 03:31 PM, David Herrmann wrote: > Hi Michael > > On Tue, Jan 20, 2015 at 2:53 PM, Michael Kerrisk (man-pages) > wrote: >> On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: >>> From: Daniel Mack >>> >>> kdbus is a system for low-latency, low-overhead, easy to use >>>

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Daniel Mack
On 01/21/2015 10:07 AM, Michael Kerrisk (man-pages) wrote: > On 01/20/2015 02:58 PM, Michael Kerrisk (man-pages) wrote: >>> +Also, if the connection allowed for file descriptor to be passed >>> +(KDBUS_HELLO_ACCEPT_FD), and if the message contained any, they will be >>> +installed into the

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Daniel, On 01/20/2015 02:58 PM, Michael Kerrisk (man-pages) wrote: > On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: >> From: Daniel Mack >> [...] >> +offset field contains the location of the new message inside the receiver's >> +pool. The message is stored as struct kdbus_msg at this

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Daniel Mack
Hi Michael, On 01/21/2015 09:57 AM, Michael Kerrisk (man-pages) wrote: > On 01/20/2015 06:50 PM, Daniel Mack wrote: >> I've addressed all but the below issues, following your suggestions. > > Are your changes already visible somewhere? Yes, in the upstream repo for the standalone module, which

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Hi Daniel, On 01/20/2015 06:50 PM, Daniel Mack wrote: > Hi Michael, > > Thanks a lot for for intense review of the documentation. Much appreciated. > > I've addressed all but the below issues, following your suggestions. Are your changes already visible somewhere? > On 01/20/2015 02:58 PM,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Daniel Mack
Hi Michael, On 01/21/2015 11:32 AM, Michael Kerrisk (man-pages) wrote: On 01/20/2015 07:23 PM, Daniel Mack wrote: It's rather an optional driver than a core kernel feature. Given the various things that I've seen said about kdbus, the preceding sentence makes little sense to me: * kdbus

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Hi David, On 01/20/2015 03:31 PM, David Herrmann wrote: Hi Michael On Tue, Jan 20, 2015 at 2:53 PM, Michael Kerrisk (man-pages) mtk.manpa...@gmail.com wrote: On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org kdbus is a system for low-latency,

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Hello Daniel, On 01/20/2015 07:23 PM, Daniel Mack wrote: On 01/20/2015 02:53 PM, Michael Kerrisk (man-pages) wrote: This is an enormous and complex API. Why is the API ioctl() based, rather than system-call-based? Have we learned nothing from the hydra that the futex() multiplexing syscall

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Daniel Mack
Hi Michael, On 01/21/2015 09:57 AM, Michael Kerrisk (man-pages) wrote: On 01/20/2015 06:50 PM, Daniel Mack wrote: I've addressed all but the below issues, following your suggestions. Are your changes already visible somewhere? Yes, in the upstream repo for the standalone module, which we

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Daniel Mack
On 01/21/2015 10:07 AM, Michael Kerrisk (man-pages) wrote: On 01/20/2015 02:58 PM, Michael Kerrisk (man-pages) wrote: +Also, if the connection allowed for file descriptor to be passed +(KDBUS_HELLO_ACCEPT_FD), and if the message contained any, they will be +installed into the receiving

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Hi Daniel, On 01/20/2015 06:50 PM, Daniel Mack wrote: Hi Michael, Thanks a lot for for intense review of the documentation. Much appreciated. I've addressed all but the below issues, following your suggestions. Are your changes already visible somewhere? On 01/20/2015 02:58 PM, Michael

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Michael Kerrisk (man-pages)
Daniel, On 01/20/2015 02:58 PM, Michael Kerrisk (man-pages) wrote: On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org [...] +offset field contains the location of the new message inside the receiver's +pool. The message is stored as struct kdbus_msg at

Re: [PATCH 01/13] kdbus: add documentation

2015-01-21 Thread Theodore Ts'o
On Wed, Jan 21, 2015 at 11:32:59AM +0100, Michael Kerrisk (man-pages) wrote: It's rather an optional driver than a core kernel feature. Given the various things that I've seen said about kdbus, the preceding sentence makes little sense to me: * kdbus will be the framework supporting

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Johannes Stezenbach
Hi David, On Tue, Jan 20, 2015 at 06:00:28PM +0100, David Herrmann wrote: [big snip] > These are just examples off the top of my head, but I think they're > already pretty convincing. Thank you for writing this up. This is the information I was looking for which puts kdbus into context and

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Daniel Mack
On 01/20/2015 02:53 PM, Michael Kerrisk (man-pages) wrote: > This is an enormous and complex API. Why is the API ioctl() based, > rather than system-call-based? Have we learned nothing from the hydra > that the futex() multiplexing syscall became? (And kdbus is an order > of magnitude more

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Daniel Mack
Hi Michael, Thanks a lot for for intense review of the documentation. Much appreciated. I've addressed all but the below issues, following your suggestions. On 01/20/2015 02:58 PM, Michael Kerrisk (man-pages) wrote: >> +and KDBUS_CMD_ENDPOINT_MAKE (see above). >> + >> +Following items

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread David Herrmann
Hi On Tue, Jan 20, 2015 at 5:08 PM, Johannes Stezenbach wrote: > However, let me repeat and rephrase my previous questions: > Is there a noticable or measurable improvement from using kdbus? > IOW, is the added complexity of kdbus worth the result? > > I have stated my believe that current usage

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Johannes Stezenbach
Hi all, On Tue, Jan 20, 2015 at 03:53:53PM +0100, Djalal Harouni wrote: > On Tue, Jan 20, 2015 at 09:42:59AM -0500, Josh Boyer wrote: > > On Tue, Jan 20, 2015 at 9:31 AM, David Herrmann > > wrote: > > > > > > If you run a 3.18 kernel, you can install kdbus.ko from our repository > > > and boot

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Djalal Harouni
Hi, On Tue, Jan 20, 2015 at 09:42:59AM -0500, Josh Boyer wrote: > On Tue, Jan 20, 2015 at 9:31 AM, David Herrmann wrote: > > Hi Michael > > > > On Tue, Jan 20, 2015 at 2:53 PM, Michael Kerrisk (man-pages) > > wrote: > >> On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: > >>> From: Daniel Mack

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Josh Boyer
On Tue, Jan 20, 2015 at 9:31 AM, David Herrmann wrote: > Hi Michael > > On Tue, Jan 20, 2015 at 2:53 PM, Michael Kerrisk (man-pages) > wrote: >> On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: >>> From: Daniel Mack >>> >>> kdbus is a system for low-latency, low-overhead, easy to use >>>

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread David Herrmann
Hi Michael On Tue, Jan 20, 2015 at 2:53 PM, Michael Kerrisk (man-pages) wrote: > On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: >> From: Daniel Mack >> >> kdbus is a system for low-latency, low-overhead, easy to use >> interprocess communication (IPC). >> >> The interface to all functions in

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Michael Kerrisk (man-pages)
On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: > From: Daniel Mack > > kdbus is a system for low-latency, low-overhead, easy to use > interprocess communication (IPC). > > The interface to all functions in this driver is implemented via ioctls > on files exposed through a filesystem called

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Michael Kerrisk (man-pages)
On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: > From: Daniel Mack > > kdbus is a system for low-latency, low-overhead, easy to use > interprocess communication (IPC). > > The interface to all functions in this driver is implemented via ioctls > on files exposed through a filesystem called

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Michael Kerrisk (man-pages)
On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org kdbus is a system for low-latency, low-overhead, easy to use interprocess communication (IPC). The interface to all functions in this driver is implemented via ioctls on files exposed through a

Re: [PATCH 01/13] kdbus: add documentation

2015-01-20 Thread Michael Kerrisk (man-pages)
On 01/16/2015 08:16 PM, Greg Kroah-Hartman wrote: From: Daniel Mack dan...@zonque.org kdbus is a system for low-latency, low-overhead, easy to use interprocess communication (IPC). The interface to all functions in this driver is implemented via ioctls on files exposed through a

  1   2   >