summaryrefslogtreecommitdiff
path: root/src/ipcpd/shim-udp
Commit message (Collapse)AuthorAgeFilesLines
* lib: robust locking in shared memory and crash recoverydimitri staessens2016-07-021-10/+10
| | | | | | | | | | | | | | This PR enhances the shared memory providing recovery if a process crashes. It adds a SHM_DU_TIMEOUT_MICROS variable, setting an expiration time for SDU's when shared memory is full. If an application doesn't read a blocking SDU within this time, the shared memory will be cleansed of all SDU's for this application and the application's rbuff will be cleared. Some refactoring of the API's. Fixed wrong pthread checks in IRMd. Fixes #13 Fixes #14
* lib, irmd, ipcpd: Change pid to apiSander Vrijders2016-06-282-21/+18
| | | | Changes the variable name as well to API instead of PID.
* lib, irmd, ipcpd: Change of IRM APISander Vrijders2016-06-281-24/+5
| | | | | | | | | | | | | | This changes the IRM API after discussions with Dimitri. The register operation is now split into a bind and register operation. The same for unregister; unbind and unregister. PIDs are now used as the application instance name. A name for a PID is only provided for scriptability in bash. It is therefore also no longer passed down to the IPCP. Every operation on an IPCP through the IRM API has to use the PID. Quering of the PIDs by name is possible. The IRM tool has been updated to use this new API as well. A subcommand 'ipcp' has been added for operations that take effect on IPCPs only. Fixes #12
* ipcpd: fix deadlock in shim-udpdimitri staessens2016-06-281-13/+25
| | | | | shim UDP would deadlock upon a dealloc call. fd_clr cannot be called under wrlock.
* ipcpd: fixes to shim-udpdimitri staessens2016-06-271-48/+60
| | | | | getsockname() can give bad behaviour when haphazardly looping fd's, added check that the fd has a flow. Replaced LOG_DBGF with LOG_DBG.
* build: correct sandboxingdimitri staessens2016-06-271-1/+1
| | | | | paths cannot start with "/" or PREFIX will be omitted. PREFIX must be set before the project() call.
* build: Fix installation prefixSander Vrijders2016-06-271-1/+1
| | | | | | The installation prefix was taking a trailing backslash, while it is common not to do so. Fixed it so that any trailing backslash is removed by the compilation and installation scripts.
* lib, irmd, ipcp: robust mutexesdimitri staessens2016-06-211-110/+109
| | | | | | | | Update to POSIX 200112L to allow use of robust mutexes in the shm_du_map. Removed the implementation of the rw_lock in favor of pthread_rwlock_t. Placeholder for the shm_du_map_sanitize function.
* build: Change install directories and set correct permissionsSander Vrijders2016-06-201-1/+1
| | | | | | | | | This sets the correct install directories for all the binaries, library and header files. It also sets the right permissions on the sockets and shared memory so that regular users can also use the ouroboros library. Root privileges are required to run the irmd. Fixes #7
* ipcpd: fixed missing unlock in shim UDPdimitri staessens2016-06-161-1/+5
| | | | Unlock missing when compiled without DNS support.
* irmd: clean up stale pending flowsdimitri staessens2016-06-161-1/+1
| | | | | | | Adds a thread that cleans up stale pending flows. Compile time option IRMD_FLOW_TIMEOUT allows setting the timeout period. Default timeout is 5 seconds. The thread wakes up 20 times per period to check the flows.
* ipcpd: fixed locking and cleanupdimitri staessens2016-06-161-0/+1
| | | | | The ipcpds didn't unlock the flows lock when the dst_name is unbeknownst to the IRMd.
* lib, irmd, tools, ipcpd: updates to dev API.dimitri staessens2016-06-132-11/+2
| | | | | | | | | | | | | | | The registration function has been moved to the irm tool, applications now need to be registered by an administrator. Currently only supports one instance per registered name, and an AP can be registered under only one name. The irmd can now start a registered server application on demand. For the full functionality of the tool, execute "irm register". AP name removed from flow allocation. Flow allocation does not send the source ap name as it is quite useless. The accept() call now only returns the AE name.
* ipcpd: fixes shutdown lockup in local and shim-udpdimitri staessens2016-05-291-6/+5
|
* lib: shm_ap_rbuff: sleep when read on empty rbuffdimitri staessens2016-05-281-1/+0
| | | | | When the ring buffer is empty, a read call will sleep. A write call on an empty ring buffer will wake up sleeping readers.
* ipcpd: cleanup of IPC processesdimitri staessens2016-05-281-5/+10
| | | | | | | This stops IPC processes from hanging due to bad locks. It first informs all threads that the IPCP is shutting down, then cancels all threads. Cancellation is still required because threads may be in accept() or receivefrom().
* ipcpd: shim-udp changed to blocking writedimitri staessens2016-05-271-14/+11
| | | | | The shim UDP did a non-blocking write call, causing loss of SDU's. This has been changed to a blocking write call.
* ipcpd: fixed the shim-udp grinding the cpudimitri staessens2016-05-271-3/+4
| | | | | | | | In linux, the select call changes the timeout variable to the time not spent in sleep, causing the timeout to decrease and that loop to grind the cpu. The timeval is now reset to the correct value every loop. Also removed a LOG_ERR message that was hampering performance.
* ipcpd: shim-udp: fixed missing NULL checkdimitri staessens2016-05-221-0/+3
| | | | Conf was not checked on bootstrap.
* ipcpd: Remove install command from buildSander Vrijders2016-05-221-2/+0
| | | | | This removes an install command from the build, since the protobuf header file was being installed in the source directory.
* lib: removed fd cleaning functiondimitri staessens2016-05-221-3/+4
|
* ipcpd: homogenous debug messages for deallocationdimitri staessens2016-05-211-8/+1
|
* lib, irmd, ipcpd: Flow deallocation over shim UDPdimitri staessens2016-05-212-52/+153
| | | | | | The shim UDP now supports deallocating a flow end-to-end. Contains some stability fixes for flow allocation and some missing close() calls in lib/sockets.
* lib, ipcpd: fixes segfaults in dev.c and shim-udpdimitri staessens2016-05-201-2/+1
|
* ipcpd: shim-udp: read fd on client sidedimitri staessens2016-05-201-0/+13
| | | | | There could be a theoretical race condition that the server sends a message before the FD is set on the client side, which is resolved.
* ipcpd: shim-ipcp: fixed race conditionsdimitri staessens2016-05-201-40/+57
| | | | | | | | | | | There were two race conditions. The first was where client received the response message before the client UDP port was connected to the server UDP port and the first message of the client was sent to the LISTEN port instead of the server application port. This was solved by reordering the operations so the response is sent after the port is correctly connected. The second race condition is that the FD is not added to the set in time at the server side before the first SDU arrives. This was solved by adding a variable proteceted by a mutex.
* ipcpd: shim-udp: removed debug codedimitri staessens2016-05-191-12/+0
| | | | | some debug code for locking and a sleep check are removed from the shim-udp.
* Implementation of a full flow allocator for the shim UDP.dimitri staessens2016-05-183-249/+532
| | | | | | | | | It uses UDP port 0x0D1F on all hosts to send and receive flow allocation messages. It supports communication between server and client AP over a single shim IPCP. Implementation of full flow deallocation is pending. Both the client and the server still have to call flow_dealloc();
* lib: Add missing modulo calculationSander Vrijders2016-05-161-2/+1
| | | | | | Adds a missing modulo calculation that would result in index -1 being returned instead of zero when the ringbuffer skipped back to the beginning.
* fixed two unlocks, removed the thread_lock.dimitri staessens2016-05-151-13/+3
|
* ipcpd: shim-udp: Revised lockingdimitri staessens2016-05-151-94/+97
| | | | | | | | | | Simplified locking to take only two locks: the first lock guards the state of the ipcp. This lock must be held for writing on bootstrap and closing, and held for reading during all other operations. The second lock guards operations on flows, and must be held for writing during allocation and deallocation, and held for reading when sending sdu's. After adding a fd to FD_SET, the shim will wait for 1 ms to ensure that the FD is added to the select call.
* ipcpd: Move lock in ipcp_udp_listenerSander Vrijders2016-05-151-2/+2
| | | | | This takes a lock before notifying the IRMd of a new flow request, to avoid race conditions when trying to retrieve the fd by port-id.
* ipcpd: shim-udp: added missing checksdimitri staessens2016-05-151-10/+10
|
* ipcpd: shim-udp: fixed locking.dimitri staessens2016-05-141-15/+21
| | | | | The fd_to_port_id function has no internal locking and must now be called under lock.
* Merged in sandervrijders/ouroboros/be (pull request #84)dimitri staessens2016-05-141-7/+8
|\ | | | | | | lib, ipcpd, irmd: Add QoS cube definition
| * lib, ipcpd, irmd: Add QoS cube definitionSander Vrijders2016-05-141-7/+8
| | | | | | | | | | | | | | | | | | | | This adds the QoS cube definition, which is an enum to select which QoS is needed in the IPCP. An application has to use the qos_spec in qos.h to define what it needs. The IRMd will map this unto a qos cube definition. Some headers are now also no longer installed on the system, since they are only to be used within the irmd and ipcps.
* | ipcpd: shim-udp: log error when failed sending SDUdimitri staessens2016-05-141-1/+2
| |
* | ipcpd: shim-udp: complete lockingdimitri staessens2016-05-141-103/+352
|/ | | | | | | Added necessary locks for the shim-udp. This PR also improves thread management, the main thread now starts a mainloop thread, which spawns sdu handler threads when it the IPCP is enrolled. If the IPCP exits the enrolled state, the sdu loop is cancelled.
* ipcpd: threading and cleanup of ipcpd_udpdimitri staessens2016-05-111-98/+40
| | | | | The shim_udp now correctly destroys its ringbuffer on exit. Now handles sdu's in a single thread using select() with a 10 us timeout.
* ipcpd: Fix unitialized valueSander Vrijders2016-05-101-2/+2
| | | | | A value that was unitialized was being treated as an error in the release build with gcc.
* ipcpd: Add nslookup to shim UDPSander Vrijders2016-05-103-25/+156
| | | | | This adds nslookup support to the shim UDP, so that a custom DNS server can be queried.
* ipcpd: Allow disabling of DNSSander Vrijders2016-05-103-29/+79
| | | | | | This allows someone to disable the use of DNS in the shim UDP. It also adds a config file specific for the shim UDP (which holds the nsupdate location). Certain defines were also moved to the global config file.
* ipcpd-udp: fixed missing closedimitri staessens2016-05-081-0/+1
| | | | caused the shim to run out of file descriptors fast...
* lib: changed port_id to intdimitri staessens2016-05-081-12/+12
| | | | | returning -1 as uint32_t leads to bugs. also changed types in GPB to sint to use zigzag encoding.
* irmd: flow allocation and fast pathdimitri staessens2016-05-072-152/+384
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit has a first implementation of flow allocation (the "slow path") and read/write (the "fast path") for ouroboros. It provides basic but unstable communications over the shared memory. It required a lot of changes all over the stack, and fixes a number of previously undetected issues. This PR still need heavy revision regarding data model, locking and cleanup. lib/dev: modifications to the API. It now uses an ap_init() call to set the AP name and sets the Instance ID to the pid of the process. It also binds the AP to the shared memory and creates tables for mappings in the fast path. A call to ap_fini() releases the resources. lib/shm_ap_rbuff: added ring buffer for data exchange between processes in the fast path. It passes an index in the shm_du_map. lib/shm_du_map: rewrote API to work with calls from dev.c. Garbage collector added. Tests updated to new API. ipcpd/ipcp-data: removed everything related to flows, as these are universal for all ap's and kept in ap_data (dev.c), or similar structs for shim ipcps. shim-udp: added flow allocator and read/write functions and shm elements. irmd: revised data model and structures necessary for flow allocation. tools: echo updated to new dev.h API. messaging system was updated to comply with new flow allocation messages. All exchanges use pid and port_id to bootstrap the fast path.
* ipcpd: Add registration to DNS serverSander Vrijders2016-04-272-19/+120
| | | | | This adds support for adding the application name to a BIND DNS server through the nsupdate utility.
* lib: client-side flow allocationdimitri staessens2016-04-271-1/+12
| | | | | flow allocation now propagates on the client side up to the IPCP. added UNKNOWN_AP and UNKNOWN_AE definitions to dev.h
* ipcpd: Hide IRMd messages and fix formattingSander Vrijders2016-04-271-39/+14
| | | | | This abstracts away the IRMd messages by calling the functions from ipcp.h in the ouroboros library. It also fixes some formatting issues.
* ipcpd: Fix shim-udp-testSander Vrijders2016-04-271-2/+2
| | | | | This fixes the shim-udp-test, which was using a non null-terminated string, which caused it to fail on some systems.
* shim-udp: resolve dst_name using DNSdimitri staessens2016-04-263-20/+37
| | | | | | This commits adds the gethostbyname call to resolve the dst_name for a flow allocation using DNS. The DNS server should be specified using the method mandated by the system (e.g. added to /etc/resolv.conf).