Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incompatible with kernels >= 5.19 #29

Closed
Luflosi opened this issue Aug 6, 2022 · 7 comments
Closed

Incompatible with kernels >= 5.19 #29

Luflosi opened this issue Aug 6, 2022 · 7 comments

Comments

@Luflosi
Copy link
Contributor

Luflosi commented Aug 6, 2022

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.

@alexmyczko
Copy link

@mefsaal
Copy link

mefsaal commented Aug 10, 2022

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

@eafer
Copy link
Member

eafer commented Aug 14, 2022

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
Copy link
Member

eafer commented Aug 14, 2022

@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.

@alexmyczko
Copy link

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)

eafer added a commit that referenced this issue Aug 14, 2022
The build is broken for the 5.19 kernel:

  #29

Pages have been replaced by folios in most interfaces, so start using
them. Also the 'flags' argument to grab_cache_page_write_begin() is
gone; instead use memalloc_nofs_save()/memalloc_nofs_restore() to
prevent recursion into the filesystem.

Signed-off-by: Ernesto A. Fernández <ernesto@corellium.com>
@alexmyczko
Copy link

works for me, thank you!

@Luflosi
Copy link
Contributor Author

Luflosi commented Aug 27, 2022

Same here, thanks from me too!

@Luflosi Luflosi closed this as completed Aug 27, 2022
Luflosi added a commit to Luflosi/nixpkgs that referenced this issue Aug 27, 2022
RaitoBezarius pushed a commit to RaitoBezarius/nixpkgs that referenced this issue Sep 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants