Package Details: kvdo-dkms 6.2.2.18-1

Git Clone URL: https://aur.archlinux.org/kvdo-dkms.git (read-only)
Package Base: kvdo-dkms
Description: A pair of kernel modules which provide pools of deduplicated and/or compressed block storage
Upstream URL: https://github.com/dm-vdo/kvdo
Licenses: GPL2
Provides: kvdo
Submitter: zapp-brannigan
Maintainer: zapp-brannigan
Last Packager: zapp-brannigan
Votes: 2
Popularity: 0.008026
First Submitted: 2018-08-19 18:48
Last Updated: 2019-10-17 16:24

Dependencies (0)

Required by (1)

Sources (2)

Latest Comments

balwierz commented on 2019-07-27 09:53

Module does not compile with the current gcc + kernel sources:

/var/lib/dkms/kvdo/6.2.1.102/build/vdo/base/heap.c: In function ‘swapElements’:
/var/lib/dkms/kvdo/6.2.1.102/build/vdo/base/heap.c:52:3: error: ISO C90 forbids variable length array ‘temp’ [-Werror=vla]
   52 |   byte temp[heap->elementSize];
      |   ^~~~
cc1: all warnings being treated as errors
make[2]: *** [scripts/Makefile.build:279: /var/lib/dkms/kvdo/6.2.1.102/build/vdo/base/heap.o] Error 1
make[2]: *** Waiting for unfinished jobs....
/var/lib/dkms/kvdo/6.2.1.102/build/uds/util/pathBuffer.c: In function ‘initializePathBufferSprintf’:
/var/lib/dkms/kvdo/6.2.1.102/build/uds/util/pathBuffer.c:91:3: error: ISO C90 forbids array ‘buf’ whose size can’t be evaluated [-Werror=vla]
   91 |   char buf[DEFAULT_PATH_BUFFER_SIZE];
      |   ^~~~
cc1: all warnings being treated as errors

nyoxi commented on 2019-02-28 19:24

I see. linux-lts has HZ set to 100, that's why the assert works there.

zapp-brannigan commented on 2019-02-19 19:56

It works with linux-lts (or any kernel below 4.20) for now. As you already noticed, the module is broken with kernel 4.20. I hope they will update their sources soon.

nyoxi commented on 2019-02-19 19:38

Build fails with current kernel because of [1]. But that is fixable [2].

More importantly, build fails on this assert [3]: STATIC_ASSERT((MSEC_PER_SEC % HZ) == 0);

MSEC_PER_SEC is 1000 and HZ is 300 on Arch kernels. Which makes me wonder, has this module ever worked with stock Arch kernel?

[1] https://github.com/dm-vdo/kvdo/issues/17 [2] https://gist.github.com/rhawalsh/5ef42b2dbf4084f4f8baa1761fe0d60a#file-0001-removed_variable-length-arrays-patch [3] https://github.com/dm-vdo/kvdo/blob/8df6805f0c4eddae59c9edcebeeab885c835f93c/vdo/kernel/histogram.c#L600