summaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Merged in sandervrijders/ouroboros/be-fix-conn (pull request #291)0.3dimitri staessens2016-10-274-46/+58
|\ | | | | | | ipcpd: normal: Fix bugs in connection establishment
| * ipcpd: normal: Fix bugs in connection establishmentSander Vrijders2016-10-274-46/+58
| | | | | | | | This fixes some bugs in connection establishment over the normal IPCP.
* | Merged in dstaesse/ouroboros/be-dealloc (pull request #290)Sander Vrijders2016-10-272-81/+129
|\ \ | |/ |/| | | lib: Clean up flow if alloc_res fails
| * lib: Clean up flow if alloc_res failsdimitri staessens2016-10-272-81/+129
| | | | | | | | | | | | | | | | | | If alloc_res fails all resources will be cleaned, except for the assigned flow_descriptor, which must be released with a dealloc call. Calling dealloc after a failed flow_alloc will not try to destroy the already cleaned up flow in the IRMd and IPCPs. Also fixes some memleaks in the oping client.
* | Merged in dstaesse/ouroboros/be-irmd (pull request #289)Sander Vrijders2016-10-271-1/+1
|\| | | | | | | irmd: Fix check in irm_flow
| * irmd: Fix check in irm_flowdimitri staessens2016-10-271-1/+1
| |
* | Merged in dstaesse/ouroboros/be-udp (pull request #287)Sander Vrijders2016-10-271-11/+10
|\ \ | |/ |/| | | ipcpd: Fix bad lock in shim-udp
| * ipcpd: Fix bad lock in shim-udpdimitri staessens2016-10-261-11/+10
| |
* | irmd: Fix cleanup of pending flowsdimitri staessens2016-10-272-3/+19
|/
* lib, ipcpd: Further stabilization of flowsdimitri staessens2016-10-268-183/+92
| | | | | | | | | | | The steps for flow deallocation have been further refined. An operation ipcp_flow_fini() which wait for all SDUs to be read from a flow has been added. The shim IPCPs and the local IPCP have been adapted to this new API. Deallocation messages have been removed from the shim IPCPs, since there is insufficient state synchronisation between them to make this work reliably.
* Merged in sandervrijders/ouroboros/be-fix-udp-query (pull request #286)dimitri staessens2016-10-261-0/+5
|\ | | | | | | ipcpd: shim-udp: Add check for directory entry
| * ipcpd: shim-udp: Add check for directory entrySander Vrijders2016-10-261-0/+5
| | | | | | | | | | | | A check was missing in the shim UDP if the destination was already present in the directory, in which case no further queries were needed.
* | irmd: Fix wrong unlocksSander Vrijders2016-10-261-2/+2
| | | | | | | | This fixes 2 wrong unlocks in the bind API method.
* | tools: irm: Bind IPCP name to IPCP-ISander Vrijders2016-10-263-3/+18
|/ | | | | | This will invoke the bind operation when using the irm tool to create a new IPCP. The new IPCP will be bound to the IPCP name as specified by the administrator.
* Merged in sandervrijders/ouroboros/be-ipcp-threads (pull request #283)dimitri staessens2016-10-263-33/+50
|\ | | | | | | ipcpd: Add threadpool for main loop
| * ipcpd: Add threadpool for main loopSander Vrijders2016-10-263-33/+50
| | | | | | | | | | | | | | This adds a threadpool for the main loop of the IPCPs. Before there was a single thread handling each request, which could result in starvation since performing name queries at the same time as enrolling a normal IPCP was impossible.
* | Merged in dstaesse/ouroboros/be-qos (pull request #279)Sander Vrijders2016-10-265-5/+12
|\ \ | |/ |/| | | lib: API for accepting flows with QoS
| * lib: API for accepting flows with QoSdimitri staessens2016-10-255-5/+12
| |
* | Merged in sandervrijders/ouroboros/be-directory (pull request #280)dimitri staessens2016-10-2615-276/+604
|\ \ | | | | | | | | | lib, irmd, ipcpd: Add name querying to IPCPs
| * | lib, irmd, ipcpd: Add name querying to IPCPsSander Vrijders2016-10-2615-276/+604
| | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the ability to query IPCPs if a name can be reached through them, e.g. if a name is available in a DIF. This means that in the shim-udp a DNS query is performed, in the shim-eth-llc an ARP-like query has been added, in the local a check is done to see if the name is registered, and in the normal currently no application is reachable through it.
* | | irmd: Add log messages for flow allocationdimitri staessens2016-10-261-2/+9
| | | | | | | | | | | | | | | | | | In the rewrite of the flow allocator for per-port ring buffers, the messages were removed because they were not accurate anymore. This adds messages when a port is allocated.
* | | ipcpd: Fix memory leaksdimitri staessens2016-10-252-31/+35
| |/ |/| | | | | | | Fixes memory leaks associated with the dif_name (which doesn't need to be maintained in shim DIFs) and the fqueue for the N + 1 flows.
* | Merge branch 'be' of bitbucket.org:ouroboros-rina/ouroboros into bedimitri staessens2016-10-243-45/+78
|\ \
| * \ Merged in dstaesse/ouroboros/be-oping (pull request #277)Sander Vrijders2016-10-243-17/+26
| |\ \ | | |/ | |/| | | | tools: Fix memory leaks in oping
| * | Merged in dstaesse/ouroboros/be-fix-alloc (pull request #276)Sander Vrijders2016-10-244-42/+26
| |\ \ | | | | | | | | | | | | lib: Stabilise flow allocation
| * | | ipcpd: Fix log message and lockdimitri staessens2016-10-231-3/+2
| | | | | | | | | | | | | | | | | | | | Remote requests for a flow that's locally deallocated should not generate an error. Sending messages should not be under lock.
| * | | ipcpd: Improve locking in ipcp-datadimitri staessens2016-10-232-42/+76
| | | | | | | | | | | | | | | | This solves some potential data races in the shims.
* | | | ipcpd: Fix memleaks in local IPCPdimitri staessens2016-10-241-11/+14
| |_|/ |/| |
* | | tools: Fix memory leaks in opingdimitri staessens2016-10-243-17/+26
| |/ |/|
* | lib: Stabilise flow allocationdimitri staessens2016-10-234-42/+26
|/ | | | | | | Deallocation was reverted to a synchronoous operation between the AP, IRMd and IPCP in order to avoid inconsistent states of the port_id. Fixes some memory leaks, particularly the shm_flow_set is now closed upon deallocation.
* dev, ipcp: Fix setting default flow optionsdimitri staessens2016-10-232-9/+3
| | | | | The default flow options were not set when a flow allocation request arrives remotely. This went unnoticed due to a previous bug.
* build: Remove the gethostbyname checkdimitri staessens2016-10-231-17/+0
| | | | | gethostbyname is included in POSIX 2001. Ouroboros requires at least POSIX 2001 and recommends POSIX 2008 for stable builds.
* tools: Fix setting flags in oping tooldimitri staessens2016-10-231-1/+1
|
* ipcpd: Fix unused variabledimitri staessens2016-10-231-1/+4
|
* lib: Fix closing applications with active flowsdimitri staessens2016-10-221-1/+2
| | | | The rdrbuff was previously closed before the remaining SDUs were removed.
* lib: Revert bad fqueue modificationdimitri staessens2016-10-222-2/+5
| | | | | An untested (and wrong) modification to the fqueue was mistakenly added to a previous commit. This corrects the bad code.
* irmd, tools: Fix missing initializersdimitri staessens2016-10-225-16/+18
| | | | | | | | | | | The BSD compiler requires explicit initializers for all fields with the -Wmissing-field-initializers flag enabled On branch be-fix-build # Changes to be committed: # modified: src/irmd/main.c # modified: src/tools/irm/irm.c # modified: src/tools/irm/irm_bind.c # modified: src/tools/irm/irm_ipcp.c # modified: src/tools/irm/irm_unbind.c # # Untracked files: # killouroboros.sh # start.sh #
* lib: Fix wrong comparisondimitri staessens2016-10-221-2/+2
| | | | FLOW_O_RDONLY is 0, so the previous statement would always be false.
* build: Fix missing includedimitri staessens2016-10-221-0/+1
| | | | | | CheckLibraryExists was not included. Apparently the CHECK_FUNCTION_EXISTS call fails with the strict prototyping flag turned on.
* build: Compile with strict conversiondimitri staessens2016-10-2251-268/+296
| | | | | This has the code checked with -Wcast-qual and -Wconversion flags. These flags were removed because SWIG generated code fails.
* build: Comply with -Wextra compiler flagdimitri staessens2016-10-2122-13/+114
| | | | | This reduces the risk for some bugs, for instance due to signed/unsigned mismatches and unused variables.
* Merged in dstaesse/ouroboros/be-demux (pull request #267)Sander Vrijders2016-10-2119-1070/+1537
|\ | | | | | | lib: Demultiplex the fast path
| * lib: Move rbuff creation/destruction to IRMddimitri staessens2016-10-216-84/+105
| | | | | | | | | | This stabilises flow allocation now that the rbuffs are created upon flow allocation. Only the IRMd can sync this process sufficiently.
| * lib: Demultiplex the fast pathdimitri staessens2016-10-2117-1051/+1497
| | | | | | | | | | | | | | | | The fast path will now use an incoming ring buffer per flow per process. This necessitated the development of a new method for the asynchronous io call, which is now based on an event queue system for scalability (fqueue). The ipcpd's and tools have been updated to this API.
* | ipcpd: normal: Avoid allocating flow over unenrolled IPCP.Sander Vrijders2016-10-171-0/+10
|/ | | | | There was no check to see if the normal IPCP was enrolled or not when a flow allocation request is done.
* lib, dev: Implement read/write options for flowsdimitri staessens2016-10-162-0/+19
| | | | | | | Added the missing implementation of setting read/write options for flows. This allows applications to block the fast path for remotes. IPCPs can use this to block the fast path for the N + 1 flow when receiving remote deallocation requests.
* lib: Stabilize fast flow deallocation over local IPCPdimitri staessens2016-10-153-49/+49
|
* lib: Fix initialisation of flow_setdimitri staessens2016-10-151-2/+3
| | | | The safe copy was not correctly initialised.
* lib: Fix flow deallocation logicdimitri staessens2016-10-137-34/+35
| | | | | Both the N and N-1 entity must now call flow_dealloc for the port_id to be released.
* ipcpd: normal: Improve upon the internal designSander Vrijders2016-10-1211-574/+411
| | | | | | | | This commit will remove the RMT component from the normal IPCP, as some of its functionality would else be duplicated in the FMGR. Now all reading from flows, either N-1 or N+1 is done in the FMGR, then either passed to the FRCT or a lookup is performed in the PFF (not there yet) and the PDU is forwarded.