| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| ... | |
| | |
| |
| |
| |
| | |
A read lock was taken instead of a write lock, which resulted in
double frees.
|
| |\ \
| |/
|/|
| | |
ipcpd: shim-udp: fixed missing NULL check
|
| |/
|
|
| |
Conf was not checked on bootstrap.
|
| |\
| |
| |
| | |
lib, ipcpd, irmd: fixes deallocation and fast path
|
| |/
|
|
|
|
| |
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.
|
| |\
| |
| |
| | |
ipcpd: Remove install command from build
|
| |/
|
|
|
| |
This removes an install command from the build, since the protobuf
header file was being installed in the source directory.
|
| |\
| |
| |
| | |
lib, irmd, ipcpd: Flow deallocation over shim UDP
|
| | | |
|
| | | |
|
| |/
|
|
|
|
| |
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.
|
| |\
| |
| |
| | |
fixes segfaults in dev.c and in shim-udp
|
| |/ |
|
| |\
| |
| |
| | |
lib: allow parallel connections
|
| |/
|
|
|
|
| |
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.
|
| |\
| |
| |
| | |
ipcpd: Full flow allocation for the shim UDP
|
| | |
| |
| |
| |
| | |
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();
|
| |\
| |
| |
| | |
lib: Add missing check upon garbage collection
|
| | |\
| |/
|/| |
|
| |\ \
| | |
| | |
| | | |
lib: Fixed incorrect comparison
|
| | |/
| |
| |
| |
| |
| |
| | |
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.
|
| |\
| |
| |
| | |
irmd: fixed bad locks
|
| |/ |
|
| |\
| |
| |
| | |
lib: Add missing modulo calculation
|
| |/
|
|
|
|
| |
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.
|
| |\ \ \
| |/ /
|/| /
| |/ |
lib: dev.c: Revised locking
|
| |/
|
|
|
| |
fd and flows are now all protected by the flows_lock while the overall
state of the AP is protected by the data_lock.
|
| |\
| |
| |
| | |
lib: allow accepting any flows
|
| | |
| |
| |
| | |
The application can call accept(fd, NULL, NULL);
|
| |\|
| |
| |
| | |
lib: cleanup file descriptors for shared memory
|
| |/
|
|
|
| |
cleanup of file descriptors for the shared memory.
also adds a missing NULL check and some invalidation in dev.c
|
| |\
| |
| |
| | |
ipcpd: Move lock in ipcp_udp_listener
|
| |/
|
|
|
| |
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 checks
|
| |/ |
|
| |\
| |
| |
| | |
ipcpd: shim-udp: fixed locking.
|
| | |
| |
| |
| | |
Used simpler pointer arithmetic to calculate the file offset pointers.
|
