summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Add astyle formatting script/config (in progress)formattingSimon Rettberg2021-03-062-0/+51
|
* [BUILD] Add Github workflow to validate (lint) source codeManuel Bentele2021-03-051-0/+34
|
* [BUILD] Disable lint/formatting for non-kernel for nowSimon Rettberg2021-03-052-23/+43
|
* [BUILD] Make lint/format optionalSimon Rettberg2021-03-051-1/+13
|
* [BUILD] Add support in CMake to validate (lint) the source codeManuel Bentele2021-03-0413-33/+321
|
* [BUILD] Add build support for Github actionsManuel Bentele2021-03-022-1/+66
|
* [DOC] replace tabs with whitespacesManuel Bentele2021-02-101-16/+16
|
* [BUILD] add support for AlmaLinux 8Manuel Bentele2021-02-101-0/+18
|
* [BUILD] add support for FreeBSD 12.2 and upcoming 13.xManuel Bentele2020-12-141-3/+3
|
* [BUILD] add support for Linux LTS kernel 5.10.xManuel Bentele2020-12-141-2/+2
|
* [FUSE] Fix build: Add dnbd3-build to dependenciesSimon Rettberg2020-12-081-1/+1
|
* [BUILD] Fix dnbd3-client build, Fix source-only buildSimon Rettberg2020-12-022-2/+2
|
* [BUILD] Include branch and build timestamp in binariesSimon Rettberg2020-12-029-14/+29
|
* [CLIENT] print help and version number correctlyManuel Bentele2020-12-021-8/+5Star
| | | | | | This change prevents the dnbd3-client to print the help text twice if the help parameter is submitted. In addition to that, correct exit codes are set after the help text is printed and the program terminates.
* [SERVER] replaced non-existent FUSE define to match CMake's build definesManuel Bentele2020-11-271-3/+3
|
* [SERVER] Fix warningsSimon Rettberg2020-11-234-7/+9
|
* [BUILD] remove automatic search of includes for 'libatomic'Manuel Bentele2020-11-231-10/+2Star
| | | | | | This patch removes the automatic search of includes for 'libatomic' since the includes for this library are part of the C11 standard shipped with each compiler.
* [BUILD] update search paths for 'libatomic' to support build on FreeBSDManuel Bentele2020-11-233-3/+11
|
* [BUILD] add CMake find package search to find 'libatomic' automaticallyManuel Bentele2020-11-232-0/+53
|
* [BUILD] increase minimum CMake version to be compliant with CMake 3.19.xManuel Bentele2020-11-231-1/+1
|
* [KERNEL] Fix race condition for request_queuereceive in receive threadSimon Rettberg2020-11-201-8/+7Star
| | | | | | | | | | Formerly, the request that was about to be received was looked up in the receive queue without removing it, then the request payload was received from the socket while the lock was not being held, and finally, the lock was required again and the request removed from the queue. This is dangrous as another thread can concurrently take the request from the queue while the receive thread reads the payload from the socket, leading to a double-free by calling blk_mq_end_request twice.
* [KERNEL] Cleanup thread cleanup, fix closing of device when busySimon Rettberg2020-11-203-90/+124
|
* [BUILD] add support for atomic operations not supported by hardwareManuel Bentele2020-11-203-3/+17
| | | | | | | | | | | | | | This change links the dnbd3-server with 'libatomic' to add support for atomic operations not supported by hardware (especially 32-bit hardware architectures, such as ARM). Thus the dnbd3-server can also run on a Raspberry Pi 1 running Rasperry Pi OS. Note that the dnbd3-server is still linked to the libatomic, even if the hardware supports atomic operations. In this case, the compiler resolves atomic operations and replaces them automatically with specific built-in functions. This unnecessary linkage can be removed in the future if the GCC supports an upcoming option called automatic linking of libatomic (--enable-autolink-libatomic).
* [KERNEL] Fix several connect/disconnect race conditionsSimon Rettberg2020-11-193-188/+193
| | | | | | | | | | | | | | | | | | | Previously disconnect was protected against concurrent calls, but connect wasn't. It was easy to crash the kernel when calling connect and disconnect IOCTLs in a tight loop concurrently. A global lock was introduced to make sure only one caller can change the connection state at a time. dev->connection_lock needs to be aquired when calling dnbd3_net_connect or _disconnect. This atomic_t based locking mechanism should be turned into a mutex in a next step, relying on mutex_trylock for cases where we don't have the cmpxchg-schedule() loop. Along the way it was noticed that the send/receive timeouts don't apply to kernel_connect, which might have been the case in older 3.x kernel versions. A crude workaround using nonblocking connect has been introduced to emulate this, but a clean solution for this is welcomed. Also, devices are now properly closed on module unload.
* [KERNEL] add support for Linux kernel 4.19 on Ubuntu 18.04 (GCC 7.5)Manuel Bentele2020-11-162-8/+22
| | | | | | | This patch replaces the __VA_OPT__ varadic macro for the C preprocessor with a builtin GCC extension for optional varadic arguments. Based on this change, the dnbd3 Linux kernel module can be also built with GCC 7.5 on a fresh Ubuntu 18.04 with Linux kernel version 4.19.x.
* cmake: Add default include dir for fuseSimon Rettberg2020-11-131-1/+1
| | | | | | | Without this, the FindFuse module will silently set the wrong include path for fuse if we don't have pkg-config installed, leading to compile errors. We could either make pkg-config a requirement, or try to fall back to its default path.
* cmake: No sign conversion warningsSimon Rettberg2020-11-131-1/+1
|
* [SERVER] Fix log calls in thread debug codeSimon Rettberg2020-11-131-5/+7
|
* cmake: Fix passing -Defines to ccSimon Rettberg2020-11-131-3/+3
|
* [BUILD] add support to locate build directory out of Git directoryManuel Bentele2020-11-133-7/+11
| | | | | This patch adds support to locate the build directory for generated artifacts out of the original checked out Git source code repository.
* [BUILD] add CMake toolchain files to cross-compile for ppc and aarch64Manuel Bentele2020-11-113-0/+68
|
* [BUILD] build docker containers only if build of dnbd3-server is enabledManuel Bentele2020-11-102-4/+5
| | | | | | | | | This patch fixes the issue that a Docker container for the dnbd3 components (including the dnbd3-server) can be built with missing dnbd3-server's binary. Since the created Docker container starts the missing binary, it will fail. Therefore, a built of the Docker image is only possible if building of the dnbd3-server is enabled (CMAKE option DNBD3_SERVER=ON).
* [CLIENT] display proper connected server if multiple servers specifiedManuel Bentele2020-11-102-14/+20
| | | | | | | With this change, the dnbd3-client is able to display the selected server (choosen by the Linux kernel module) after successful probing. Therefore, the ioctl call OPEN returns the selected server host as index of the specified dnbd3-server hosts after successful probing.
* [KERNEL] add manual switching of dnbd3-server to specified serverManuel Bentele2020-11-104-56/+159
| | | | | | This patch adds the feature to manually switch the dnbd3-server to a specified server. The switching is implemented by the use of the ioctl call SWITCH.
* [KERNEL] protect read-only access to server listManuel Bentele2020-11-091-6/+9
| | | | | | | | | | This patch completes the changes from the last commit [1] and protects in addition the read-only access to the alternative list of new servers. Otherwise a race condition between ioctl ADD_SRV/REM_SRV (process context) and dnbd3 discover thread would be possible since the 'new_servers_num' field is changed in both contexts. [1] - DNBD3 commit db5a93a43fea2aa3c3bbaf8c48d238e60d834676
* [KERNEL] protect access to server list if alt server is added/removedManuel Bentele2020-11-091-0/+3
| | | | | | This patch locks the protecting spinlock of the DNBD3 device's new server list. Otherwise a race condition between ioctl ADD_SRV/REM_SRV (process context) and dnbd3 discover thread would be possible.
* [KERNEL]: change probing logic to select first available serverManuel Bentele2020-11-092-16/+25
| | | | | | | This patch changes the probing logic to select the first available dnbd3-server submitted by the ioctl OPEN call. All other specified dnbd3-servers are added to the alternative server list and do not have to be available.
* [KERNEL, CLIENT]: submit and probe multiple dnbd3-server with ioctl OPENManuel Bentele2020-11-065-105/+268
| | | | | | | | | | The ioctl OPEN call for DNBD3 devices exposed by the dnbd3 Linux kernel module, is extended with a fixed array of dnbd3 hosts. The fixed array allows the dnbd3-client to submit host information (IP address and port) of multiple dnbd3-servers. This information is used to probe all submitted dnbd3-servers and add them to the alternative dnbd3-server list. If at least one dnbd3-server is not reachable, the OPEN ioctl call will abort with an error code.
* [BUILD] add CMake support to build docker images based on ArchlinuxManuel Bentele2020-11-024-9/+46
|
* [CLIENT] format output of program help textManuel Bentele2020-10-301-7/+7
|
* [CLIENT] add documentation for adding/removing dnbd3-server to helpManuel Bentele2020-10-302-10/+9Star
|
* [BUILD] do not ignore C flags that are passed to CMakeManuel Bentele2020-10-301-1/+1
| | | | | | | | C flags which are set by the environment variable CFLAGS have been ignored by CMake since the corresponding CMake variable CMAKE_C_FLAGS was not extended with those passed flags, but have been overwritten by a fixed set of flags. This patch fixes the overwrite of CMAKE_C_FLAGS and extends its content with the C flags passed in the environment variable.
* [FUSE] removed unnecessary cmdln argument check for sticky modeManuel Bentele2020-10-301-6/+0Star
|
* [BUILD] add build options to enable/disable build of dnbd3 componentsManuel Bentele2020-10-305-7/+22
|
* [CLIENT] add feature to ignore propagated alternative servers (sticky mode)Manuel Bentele2020-10-292-6/+29
|
* [DOC] update setup description of docker containers with network linksManuel Bentele2020-10-291-7/+16
|
* [BUILD] add CMake support to build docker images based on Ubuntu 20.04Manuel Bentele2020-10-295-1/+113
|
* [SERVER] Update nextSave timestamp at start of functionSimon Rettberg2020-10-291-4/+6
| | | | | | | | | saveLoadAllCacheMaps() is called frequently, and a 'full' run can take some time. If we only update the nextSave timestamp when we're done, we might already have a concurrent call to the function, which will also do a 'full' run, since the timestamp is not updated yet. This doesn't break anything, but leads to even more disk activity, which is probably already high, given that the previous run is not done yet.
* [BUILD] do not print verbose CMake messages in unsupported CMake versionsv2.3Manuel Bentele2020-10-272-16/+7Star
|
* [BUILD] fix broken version header packaging for source packagesManuel Bentele2020-10-273-11/+44
|