Commit Graph

1140 Commits

Author SHA1 Message Date
Katarzyna Łapińska
362e224dba
Merge pull request #601 from katlapinka/old-version
Workaround for running tests on CAS version with old versioning style
2021-01-25 13:46:35 +01:00
Robert Baldyga
af664f9917
Merge pull request #649 from arutk/zero-md-old-meta
Zero metadata if on-disk version differs from current
2021-01-21 17:54:29 +01:00
Adam Rutkowski
b0ba347ee7 man entry for --zero-metadata command
Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 17:11:31 +01:00
Adam Rutkowski
4c98949cac Add force option to zero-metadata
Force option is used to enforce metadata erasure despite
dirty data, metadata mistmatch and/or dirty shutdown.

Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 15:50:17 -06:00
Adam Rutkowski
68b68db9c0 Zero metadata if on-disk version differs from current
Kernel adapter now returns is_cache_device=1 and newly added
metadata_compatible=0 in case of metadata detected with
differing version (instead of is_cache_device = 0).

This allows zero-superblock command to recognize old
cache instance and clear it.

casadm --script --check-cache-device still returns 'Is cache'='no'
in this case, as this layer only cares about metadata in current
version to be able to detect dirty datas tatus.

Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 14:47:54 -06:00
Robert Baldyga
63eb23ba40
Merge pull request #648 from rafalste/update_copyright
Update copyright statements (2021)
2021-01-21 13:27:51 +01:00
Robert Baldyga
be4920eb4e
Merge pull request #644 from arutk/list
casadm robustness and security fixes
2021-01-21 13:24:13 +01:00
Adam Rutkowski
2b739d1165 Check exclusive device access before probing metadata
Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 14:03:36 -06:00
Adam Rutkowski
5f65d93a36 Replace cache started check with exclusive open in zero superblock.
This check covers the previous case (cache started) as well as
other potential incorrect usages of the command. Also now
the check is made exactly at the moment of opening the
device to write to.

Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 14:03:36 -06:00
Adam Rutkowski
1fcb175fee Fix probing cache device in casadm
Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 14:03:36 -06:00
Adam Rutkowski
1fb5fd9662 Verify whether input device path is actually a block device
This applies to add/momove core, start/stop cache, zero superblock
commands.

Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 14:03:36 -06:00
Slawomir Jankowski
5f5c150c30 Add comment about calling only after normalizing path
Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
2021-01-21 14:03:36 -06:00
Slawomir Jankowski
636e083e22 Remove unneccesary variable
Remove unused `buff` from `print_slow_atomic_cache_start_info()`

Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
2021-01-21 14:03:36 -06:00
Slawomir Jankowski
035ed3e7f9 Fix listing caches with path outside /dev/disk/by-id
Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
2021-01-21 14:03:26 -06:00
Rafal Stefanowski
43f43068ad Update copyright statements (2021)
Signed-off-by: Rafal Stefanowski <rafal.stefanowski@intel.com>
2021-01-21 13:15:38 +01:00
Robert Baldyga
42ac206543
Merge pull request #647 from robertbaldyga/update-tf-20210121
Update TF
2021-01-21 13:00:45 +01:00
Robert Baldyga
78205be75b
Merge pull request #646 from robertbaldyga/update-ocf-20210121
Update OCF: Add missing page lock in cleaner
2021-01-21 10:45:39 +01:00
Robert Baldyga
bf5f224e39 Update TF
Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-21 10:45:12 +01:00
Robert Baldyga
096a2949bd Update OCF: Add missing page lock in cleaner
Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-21 10:07:38 +01:00
Slawomir Jankowski
3df6305254 Change strnlen to sizeof in set_device_path()
Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
2021-01-20 14:14:29 +01:00
Robert Baldyga
59558a8ac6
Merge pull request #643 from mmichal10/use-after-free-fix
Fix `use after free` bug
2021-01-20 11:38:05 +01:00
Michal Mielewczyk
4d61ded26d Fix use after free bug
Signed-off-by: Michal Mielewczyk <michal.mielewczyk@intel.com>
2021-01-20 02:28:19 -05:00
Robert Baldyga
fce70ac4c0
Merge pull request #641 from robertbaldyga/fix-recovery-panic
Fix kernel panic during recovery after force removing core
2021-01-19 19:55:38 +01:00
Robert Baldyga
6ea6ec4d4a
Merge pull request #638 from Ostrokrzew/interrupt_stop
Handle cache stopping interruption
2021-01-19 19:53:09 +01:00
Slawomir Jankowski
a057a0082f Handle cache stopping interruption
Ignore the interruption of the stop operation - will finish asynchronously.
Remove redundant `ocf_queue_put`.
Move creating the `finish_thread` during the cache stop
from the `_cache_mngt_cache_stop_sync` to the `cache_mngt_exit_instance`
and give it a proper handling.

Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
2021-01-19 16:25:11 +01:00
Robert Baldyga
106a37c6a1 Fix kernel panic during recovery after force removing core
Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-19 16:02:43 +01:00
Robert Baldyga
9b3fba2f1a
Merge pull request #637 from mmichal10/core-remove-improvements
update ocf: remove core fine locking
2021-01-18 11:33:36 +01:00
Michal Mielewczyk
36c98086e7 update ocf: remove core fine locking
Signed-off-by: Michal Mielewczyk <michal.mielewczyk@intel.com>
2021-01-18 03:50:07 -05:00
Robert Baldyga
a4753f3f90
Merge pull request #631 from robertbaldyga/more-conservative-mem-check
Restore more conservative free memory calculation method
2021-01-15 21:07:06 +01:00
Robert Baldyga
493caf2b31 configure: Fix coding style
Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-15 10:10:41 +01:00
Robert Baldyga
ca3e1a5984 Revert "modules: use si_mem_available() when possible"
This method produced too optimistic free memory value, which in result
led to oom killer activation. This patch restores more conservative
free memory calculation method.

This reverts commit 1e9b7a4262.

Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-14 15:16:09 +01:00
Robert Baldyga
f967596b47
Merge pull request #625 from robertbaldyga/metadata-probe-error-handle
Handle error in ocf_metadata_probe() complete callback properly
2021-01-13 18:00:52 +01:00
Robert Baldyga
02c3910677 Handle error in ocf_metadata_probe() complete callback properly
Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-13 15:20:54 +01:00
Robert Baldyga
722767b06f
Merge pull request #624 from mmichal10/remove-free-stat
Remove `Free` entry from ioclass usage stats
2021-01-12 13:02:47 +01:00
Michal Mielewczyk
b4a34fe9c9 tests: update ioclass stats tests
`Free` is no longer a part of the ioclass usage stats

Signed-off-by: Michal Mielewczyk <michal.mielewczyk@intel.com>
2021-01-12 05:11:16 -05:00
Michal Mielewczyk
0e1ba12756 test api: class for ioclass usage stats
Signed-off-by: Michal Mielewczyk <michal.mielewczyk@intel.com>
2021-01-12 05:11:16 -05:00
Michal Mielewczyk
6cbce57106 casadm: don't print Free entry for ioclass stats
Signed-off-by: Michal Mielewczyk <michal.mielewczyk@intel.com>
2021-01-12 05:11:09 -05:00
Robert Baldyga
cbc1de7a37
Merge pull request #614 from Ostrokrzew/message
Move message about by-id path higher
2021-01-08 12:25:36 +01:00
Robert Baldyga
58438c3a5d
Merge pull request #617 from robertbaldyga/update-after-tf-api-changes
tests: Update tests after test-framework API changes
2021-01-08 12:00:55 +01:00
Slawomir Jankowski
fb8fb63f06 Add path sanitizing
to `print_slow_atomic_cache_start_info` after removing check from lower
function `_check_cache_device`.

Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
2021-01-08 11:45:52 +01:00
Robert Baldyga
76aaaef62f
Merge pull request #621 from robertbaldyga/update-ocf-2021
Update OCF - seq cutoff fix
2021-01-08 11:16:57 +01:00
Robert Baldyga
3d1d503fef Update OCF - seq cutoff fix
Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-08 11:06:35 +01:00
Robert Baldyga
9f23f47c9d
Merge pull request #618 from rafalste/fix_weak_modules_path_resolving
Fix resolving modules path for weak-modules
2021-01-07 14:26:14 +01:00
Rafal Stefanowski
87ea8ced35 Fix resolving modules path for weak-modules
Signed-off-by: Rafal Stefanowski <rafal.stefanowski@intel.com>
2021-01-07 10:26:38 +01:00
Robert Baldyga
22b4288f3d tests: Update tests after test-framework API changes
Signed-off-by: Robert Baldyga <robert.baldyga@intel.com>
2021-01-05 20:35:58 +01:00
Robert Baldyga
5d5b6ae594
Merge pull request #616 from Ostrokrzew/api_fix
Fix recent change in functional tests API
2021-01-05 18:31:46 +01:00
Robert Baldyga
f8687bb53b
Merge pull request #615 from mmichal10/uninitialized_access
casadm: prevent acccess to uninitialized variables
2021-01-05 16:53:00 +01:00
Slawomir Jankowski
e2ea2f1b30 Fix recent change in functional tests API
Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
2021-01-05 16:44:47 +01:00
Michal Mielewczyk
92ab19999e casadm: prevent acccess to uninitialized variables
Signed-off-by: Michal Mielewczyk <michal.mielewczyk@intel.com>
2021-01-05 09:54:17 -05:00
Slawomir Jankowski
be87b59303 Move message about by-id path higher
Not every usage of `set_device_path()` needs to print message.
Sometimes it's internal operation and can contain misleading info.

Signed-off-by: Slawomir Jankowski <slawomir.jankowski@intel.com>
2021-01-05 14:40:35 +01:00