connection: kill unneeded include

use the kernel's generate_random_uuid()

TODO: update

use uid_eq()

namespace: remove debug print

connection: fix queue entry locking

Don't access the queue entry's members without holder the connection's
lock.

connection: kill unused variable

connection: remember the reply in the queue item

When an implementor takes over a name from an activator, we must make
sure the overtaking connection is actually able to answer to the waiting
connection.

connection: pass back the full queue element from kdbus_conn_queue_insert

That way, we can fiddle with the contents of the queue entries in
more flexible ways.

connection: assorted sync_reply fixes and cleanups

connection: fix kernel doc comment

increase hash tables for connections and names

limit the number of buses and connections per user

connection: delete timer on error

do not shortcut handle state with KDBUS_CMD_BYEBYE, it leaks the namespace ref

connection: delay sending messages to monitor connections

Only supply messages to monitor connections in case
kdbus_conn_queue_insert() for the actual destination succeeded.

connection: add comment about wait_event

kerneldoc: s/Returns/Return/, s/any value/negative errno/, move "Return:" to last line

properly drop references of endpoint and activator connection

connection: only schedule timeout scan for async operation

test/test-kdbus-sync.c: kill unused variable

kdbus.h: document two new possible error values

handle: fix potential mem leak when kdbus_conn_kmsg_send() fails

test: add test for sync message ping-pong

connection: add syncronous message reply mechanism

Sending a message with (KDBUS_MSG_FLAGS_EXPECT_REPLY |
KDBUS_MSG_FLAGS_SYNC_REPLY) will result in the KDBUS_CMD_MSG_SEND ioctl
will block until the receiving connection has replied or the specified
timeout is hit.

When the round-trip succeeded, .offset_reply in the sender's message is
set to the offset where the reply can be found, and the message is
accepted and received automatically. The sender is still in charge of
freeing the message though.

This breaks with Greg's original matra that we followed elsewhere
in the design of kbus: "nothing blocks, ever". But hey, it an optional
feature after all :)

kdbus.h, message: prepare for synchronous operations (ABI break)

connection: let kdbus_conn_queue_insert() hand back the used offset

... so we can store the answer offset in case of synchronous operations.

connection: split receive logic

This allows us to call the receive logic internally which is needed to
automatically receive a message in synchronous operation.

fix kerneldoc

kdbus.h: fix comments

update RECV ioctl and implement priority queues (ABI break)

TODO: update

bus: make conn_seq_last atomic64_t

conn->id is now assigned outside of the bus' lock, so an atomic64_t is
nicer.

test: test-kdbus-timeout: cleanups, less verbosity

connection: fix error unwinding of kdbus_conn_new()

It's not safe to unref a half-baked connection, so conduct a real error
unwinding in case of errors.

test: test-timeout: add dirty hack for loop mode

connection: kick off timeout scan for conn_src, not conn_dst

conn_src is the one we added the reply list to, so this one should be
kicked off.

Note that the first invocation of the timeout scanner will unlikely do
any real work anyway. The reason for calling it here is that it will
implicitly calculate a new deadline, and arm the timer.

connection: rework timeout logic

No functional change actually, just a code reordering that makes the
function more readable and saves us one level of indentation.

test: improve reply timeout test

test: add test for reply timeout

kdbus, endpoint: abort if device_register fails

If device_register() fails then jump to exit to clean things up and
return.

Signed-off-by: Djalal Harouni <tixxdz@opendz.org>

namespace use 64 bit message seqnum

pool: remove debug function

allow to set and query connections and memfds a human-readable names (ABI break)

export namespace-global message sequence number (ABI break)

test: use bool instead of int

identify anon inode as [kdbus-memfd]

util: fix typo

namespace: spare one format variable in dev_set_name()

kerneldoc: remove unused ENOTCONN

connection: Fix timeout

KDBUS_ITEM_REPLY_TIMEOUT have to be sent to src_id.

names: switch locking order

pool: use own lock to protect the slices lists

Merge pull request #12 from michelecurti/patch-1

kdbus.txt: fix typo

fix typo

TODO: update

policy: require privileges for seeting the endpoint's policy

test: return error from loop

test: silence a Wmaybe-uninitialized report on ret variable

Silence the gcc Wmaybe-uninitialized report on ret variable, and avoid
an extra check since run_tests() already returns EXIT_FAILURE or
EXIT_SUCCESS

Signed-off-by: Djalal Harouni <tixxdz@opendz.org>

endpoint: don't kfree a PTR_ERR

Signed-off-by: Djalal Harouni <tixxdz@opendz.org>

TODO: update

move devname validation to util.c

split internal.h into defaults.h and util.h

fix whitespace and comments

TODO: update

validate incoming names for buses, namespaces, endpoints

require privileges to activate or monitor

TODO: update

Merge pull request #7 from Keruspe/master

Makefile: fix typo

Makefile: s/name/uname/

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

TODO: update

various 80 column warnings fixed up

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

policy.c: use linux/uaccess.h not asm/uaccess.h

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

more checkpatch space "fixes"

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

namespace.c: fix bug with trailing ; on an if statement

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

notify.c: switch/case statement alignment.

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

notify.c: space checkpatch "fixes"

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

kdbus.h: fix stupid checkpatch warnings about spaces in comments.

ugh, trivial crud...

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

fix semicolon in if statement

Signed-off-by: Govindarajulu Varadarajan <govindarajulu90@gmail.com>

Fix some stupid checkpatch warnings in connection.h and names.h

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

limit the amount of request for a reply a connection can issue

test: benchmark - use sizeof(struct timeval)

Simon Peeters 12:41 PM:
I am happy to let you all know that the error is not in the kdbus
code, but in the test code which has 16 hardcoded as the size of struct
timeval.

test: cast addr to uintptr_t

Makefile: fix typo

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

grant CAP_IPC_OWNER-like access to the owner of the bus

do not accept "kernel messages" from userspace

kdbus.h: fix kernel doc

kdbus.h: KDBUS_ITEM_PRIORITY is not implemented and probably not even a good idea, let's remove it

use module name for printk

print better error message

TODO: update

memfd: use KBUILD_MODNAME

allow to compile a separate kernel module with a named extension

running:
  make EXT=2

will create a:
  kdbus2.ko

and the tests will run against:
  /dev/kdbus2/control

This allows us to run a new and separated kernel modules while the original
kdbus.ko is in use in the booted system.

fix kernel doc

handle timeouts only handled for expected replies, not for queued messages

TODO: update

connection: fix access-after-free

update cookie and cookie_reply logic

return error in kdbus_name_release()

do not deny writing messages to an activator connection