linux-apfs-rw icon indicating copy to clipboard operation
linux-apfs-rw copied to clipboard

Incompatible with kernels >= 5.19

Open Luflosi opened this issue 3 years ago • 6 comments

Here is the build log of compiling the latest master version:

make -C /nix/store/0rdnj4x5qa093cf5zi78knljwa05raw0-linux-5.19-dev/lib/modules/5.19.0/build M=/build/source
make[1]: Entering directory '/nix/store/0rdnj4x5qa093cf5zi78knljwa05raw0-linux-5.19-dev/lib/modules/5.19.0/build'
  CC [M]  /build/source/btree.o
  CC [M]  /build/source/compress.o
  CC [M]  /build/source/dir.o
  CC [M]  /build/source/extents.o
  CC [M]  /build/source/file.o
/build/source/file.c: In function 'apfs_file_mmap':
/build/source/file.c:109:28: error: 'const struct address_space_operations' has no member named 'readpage'
  109 |         if (!mapping->a_ops->readpage)
      |                            ^~
make[2]: *** [/nix/store/0rdnj4x5qa093cf5zi78knljwa05raw0-linux-5.19-dev/lib/modules/5.19.0/source/scripts/Makefile.build:249: /build/source/file.o] Error 1
make[1]: *** [/nix/store/0rdnj4x5qa093cf5zi78knljwa05raw0-linux-5.19-dev/lib/modules/5.19.0/source/Makefile:1843: /build/source] Error 2
make[1]: Leaving directory '/nix/store/0rdnj4x5qa093cf5zi78knljwa05raw0-linux-5.19-dev/lib/modules/5.19.0/build'
make: *** [Makefile:16: default] Error 2

Looks like someone wrote a patch (that I didn't test) for a slightly older version of this kernel module: https://bugs.launchpad.net/ubuntu/+source/linux-apfs-rw/+bug/1982115.

Luflosi avatar Aug 06 '22 11:08 Luflosi

true https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1016770

alexmyczko avatar Aug 07 '22 07:08 alexmyczko

Hi i made some diff/patch file test based on debian patch if someone wanna try are free. now i dont have any APFS now in my pc and dont have time

apfs-rw.zip

mefsaal avatar Aug 10 '22 02:08 mefsaal

true https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1016770

@alexmyczko That looks like a separate issue, and it should have been fixed a couple of weeks ago. I guess the package is outdated? It says 20220214 right?

eafer avatar Aug 14 '22 17:08 eafer

@mefsaal The debian patch seems to have made a mistake with the version checks, and it would break the build for 5.18. For instance, mpage_read_folio() was only added for 5.19.

eafer avatar Aug 14 '22 18:08 eafer

So I guess I'll wait for that new patch to be fixed, and update the debian package right after that... (preferably when the patch is merged to main/master)

alexmyczko avatar Aug 14 '22 18:08 alexmyczko

works for me, thank you!

alexmyczko avatar Aug 15 '22 14:08 alexmyczko

Same here, thanks from me too!

Luflosi avatar Aug 27 '22 16:08 Luflosi