| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|\
| |
| |
| | |
ipcpd: fixed the shim-udp grinding the cpu
|
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/ |
|
|
|
|
|
| |
non-blocking write would return when the buffer was full
functions in dev now check validity of input file descripters
|
|\
| |
| |
| | |
Be cbr
|
| |
| |
| |
| |
| | |
This adds a cleanup handler for the main message loop, which closes
the socket upon pthread_cancel so that the ipcp can exit cleanly.
|
| |
| |
| |
| |
| | |
This adds a threadpool to cbr, so that it is not overflooded with
client requests.
|
|\ \
| |/
|/|
| | |
lib: allow running shm_du_map in single block mode
|
| |
| |
| |
| |
| | |
this allows disabling support for SDU's that are larger than a block
in the DU MAP by defining the variable SHM_MAP_SINGLE_BLOCK
|
| |
| |
| |
| |
| | |
the CBR tool would stop if the buffer is full. Now it will continue
sending SDU's until the duration expires.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
The tool sends constant bandwidth traffic between a client and a
server. cbr --help for more info.
Adds time_utils.h to the library containing useful functions for
arithmetic with timespec and timeval structures.
|
|/
|
|
| |
Fixes some memleaks in dev.c in the case of error conditions.
|
|
|
|
|
| |
This makes sure only one shm_du_map is running in the system.
Starting a second irmd would invalidate the shared memory map.
|
|\
| |
| |
| | |
irmd: Change read lock to write lock
|
| |
| |
| |
| |
| | |
A read lock was taken instead of a write lock, which resulted in
double frees.
|
|/
|
|
| |
Conf was not checked on bootstrap.
|
|
|
|
|
|
| |
The fast path has been rewritten to have certainty to read the correct
flow. Deallocation will not release port_id's or fd's until they are
explicitly released locally.
|
|
|
|
|
| |
This removes an install command from the build, since the protobuf
header file was being installed in the source directory.
|
| |
|
| |
|
|
|
|
|
|
| |
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.
|
| |
|
|
|
|
|
|
| |
dev.c: read now only reads an SDU if is is for the correct port_id
shm_ap_rbuff: added a function peek() that returns the port_id of the
tail.
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
some debug code for locking and a sleep check are removed from the
shim-udp.
|
|
|
|
|
|
|
|
|
| |
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();
|
|
|
|
|
|
|
| |
There was a check missing upon garbage collection. If the whole DU map
has been used once, all DUs will have their garbage bit set to
one. This meant that the garbage collector would keep moving the tail
indefinitely.
|
|
|
|
|
| |
When moving the tail pointer in the DU map, blocks was never reset to
zero. This caused it to sometimes clean up too many DUs.
|
| |
|
|
|
|
|
|
| |
Adds a missing modulo calculation that would result in index -1 being
returned instead of zero when the ringbuffer skipped back to the
beginning.
|
|\
| |
| |
| | |
lib: dev: small locking improvements
|
| |
| |
| |
| |
| |
| | |
This avoids holding the long locks during accept() and alloc_res()
calls. Small reduction of time of other locks held. No changes in
locking logic.
|
| |\ |
|
|\ \ \
| | | |
| | | |
| | | | |
irmd: new locking implementation
|
| | | |
| | | |
| | | |
| | | | |
unlock is now called before the object is destroyed.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This locking should be more consistent, It now has three locks, one
guarding flows and port_id's, one guarding the registered apps and
ipcps, and one guarding the overall state of the irmd. There are two
additional mutexes guarding the condition variables.
|
|\ \ \ \
| |_|/ /
|/| | |
| | | | |
ipcpd: shim-udp: Revised locking
|
| | |/
| |/| |
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/
|
|
|
| |
fd and flows are now all protected by the flows_lock while the overall
state of the AP is protected by the data_lock.
|
|
|
|
| |
The application can call accept(fd, NULL, NULL);
|
|
|
|
|
| |
cleanup of file descriptors for the shared memory.
also adds a missing NULL check and some invalidation in dev.c
|
|
|
|
|
| |
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.
|
| |
|
|
|
|
| |
Used simpler pointer arithmetic to calculate the file offset pointers.
|
|
|
|
|
| |
Port_id's weren't correctly released. Also removes remaining debug
logs from the library.
|
|
|
|
|
| |
The fd_to_port_id function has no internal locking and must now be
called under lock.
|
|\
| |
| |
| | |
lib, ipcpd, irmd: Add QoS cube definition
|