Request the return value of the operation from the kernel and check if
it's an error, to inform the user about e.g. not having sufficient
permission.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Now that we only have one socket for our context we can simplify the
name of the struct member.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Now that we have moved all operations to the control socket we can
remove the ns handler. The query function in the library is racy, as
there might be non-response messages in our incoming buffer when we
start reading, so this is dropped too - rather than just being
translated to the new interface.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Convert qrtr-lookup to use the control port for queries, allowing us to
drop the ns service altogether.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Introduce a subscription based lookup model, in order for a client to be
able to listen to a future spawning service. The client can register any
number of queries with the ns and the ns will post lookup-results as
services are registred.
As a special case the list of lookup-results that is sent out as a
result of the registration of a new lookup will be finished by an empty
lookup response, allowing tools like qrtr-lookup to terminate as the
entire list of currently registed services is received.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Replace the implementation of the two library functions with calls to
the new APIs, so that we can remove the server-side implementation of
the previous messages.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
The BYE control message signals that a node is gone, drop all services
from our register and propagate this message to all local servies - to
allow them to clean up any resources associated with potential remote
clients.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Following the downstream kernel we should propagate a del_client to all
local services as the dying socket might have acted - or once acted - as
a client as well.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Upon receiving a DEL_CLIENT some local or remote client has been
disconnected. If this is a local server we remove the server from the
lists and announce the removal to the remotes. Otherwise we need to
inform local servers that one of their clients might be gone, so we
propagate a DEL_SERVER message to all locally registered services.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Add a new set of operations to libqrtr for sending registration events
to the control service, rather than using the nameservice port.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Move the qrtr_ctrl_cmd struct to the shared header file, order to
facilitate sending control messages directly from the library.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
A number of operations broadcasts messages, so store a static
broadcast socketaddr in the context to facilitate this.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Store the local node id in the "context, rather than requesting the
local node id from the kernel every time we need it.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
In order to easily shoot down all services when a remote node goes away,
or list all local services when announcing inject a "node" layer above
the services map.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
With newer kernels AF_QIPCRTR is already defined in the system header
files, so don't define it unconditionaly.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
The kernel will broadcast the presence of the qrtr-ns to all locally
opened sockets and any remote sockets will be notified through other
means; so drop the reset notification.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
This component is compiled in Debian and OE where we have multilib support
enabled by default, so let's use proper variables for Makefile so that distro
can better configure the destination folders.
Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org>
New macro to be used to deploy header files when 'installing'. Use it as:
$(call add-inc-target,lib,foo.h)
to install file lib/foo.h
Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org>
Build libraries with proper soname, real and linker names, using symlinks
appropriately.
As such, the generic macro 'add-target' can no longer be used for both
binary and libaries.
Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org>
The final version of QRTR that was merged into v4.7 of the Linux kernel
requires a configuration step where we set the local node address.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Extract the libqrtr source and header file to a separate directory,
making it more convinient to include the header file from other
projects.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
The instance is made up of 16 bits of version and 16 bits of instance,
the first is used upon matching services and must often be specified by
service implementations, so make it convenient to specify these.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Upon receiving a HELLO from the remote we need to advertise our
registered services for it to know about them.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
As node 0 is made a valid id in the kernel we're not allowed to specify
node 0 when binding our sockets. For now just hard code 1, as that's
where we are.
Either the kernel should define a reserved node id or we should have a
convenient way of specifying the local node, but for now this is good
enough.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
