arddennis
commented
8 years ago According to a mail conversation bio_get_nr_vecs was removed and now it is not necessary. At least from this words: "We can always fill up the bio now, no need to estimate the possible size based on queue parameters." I can assume that I can use something like BIO_MAX_PAGES (256) instead of bio_get_nr_vecs(). Also bio_get_nr_vecs found only several times in the code. So it should be not hard to replace. Also, there are more errors shown when you compile against 4.3. For example, function misc_deregister currently void , but in code it tries to return its result as int.
int eio_delete_misc_device()
{
return misc_deregister(&eio_misc);
}Such coud be easy fixed by adding "return 1" and removing return from misc_deregister... I am not sure this is right way, but 10 minutes seach didn't give me an answer what this function returned before.
The same is with bio_endio which does not return any value and has only one argument...
I made changes to the code to correct compilation errors and managed to compie kernel modules for 4.3, but this simply not enough. Code needs to be checked thoughtfully. I get errors during cache creation:
[Errno 22] Invalid argument And this
enhanceio: Setting mode to write through get_policy: policy 2 found
enhanceio_lru: eio_lru_instance_init: created new instance of LRUenhanceio: Setting replacement policy to lru (2)
enhanceio: md_create: Could not read superblock sector 128 error 4096 for cache "*****"
storcium
flrichar
qm3ster
elmystico
sjlongland
raxip
I was trying to compile enhanceio on 4.3 without success. There's problem caused by this commit on kernel (have a look at bio.h changes "block: add a bi_error field to struct bio") also by this commit (bio.h also, "remove bio_get_nr_vecs()"). I couldn't fix it myself easly- few enhaceio source files are affected, it would be good someone experienced had a look. Also, I'm not sure how this could be fixed without breaking older kernels support... fork?