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

fixup gasket-1.0 build error with mainline kerenl 5.16 (aarch64) #4

Closed
wants to merge 1 commit into from

Conversation

yan-wyb
Copy link

@yan-wyb yan-wyb commented Dec 17, 2021

error message:

DKMS make.log for gasket-1.0 for kernel 5.16.0-rc2 (aarch64)
Fri Dec 17 14:56:02 CST 2021
make: Entering directory '/usr/src/linux-headers-5.16.0-rc2'
CC [M] /var/lib/dkms/gasket/1.0/build/gasket_core.o
CC [M] /var/lib/dkms/gasket/1.0/build/gasket_ioctl.o
CC [M] /var/lib/dkms/gasket/1.0/build/gasket_interrupt.o
CC [M] /var/lib/dkms/gasket/1.0/build/gasket_page_table.o
CC [M] /var/lib/dkms/gasket/1.0/build/gasket_sysfs.o
CC [M] /var/lib/dkms/gasket/1.0/build/apex_driver.o
LD [M] /var/lib/dkms/gasket/1.0/build/gasket.o
LD [M] /var/lib/dkms/gasket/1.0/build/apex.o
MODPOST /var/lib/dkms/gasket/1.0/build/Module.symvers
ERROR: modpost: module gasket uses symbol dma_buf_detach from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_put from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_get from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_unmap_attachment from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_map_attachment from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_attach from namespace DMA_BUF, but does not import it.
make[1]: *** [scripts/Makefile.modpost:134: /var/lib/dkms/gasket/1.0/build/Module.symvers] Error 1
make[1]: *** Deleting file '/var/lib/dkms/gasket/1.0/build/Module.symvers'
make: *** [Makefile:1761: modules] Error 2
make: Leaving directory '/usr/src/linux-headers-5.16.0-rc2'

Signed-off-by: Yan yan-wyb@foxmail.com

error message:

DKMS make.log for gasket-1.0 for kernel 5.16.0-rc2 (aarch64)
Fri Dec 17 14:56:02 CST 2021
make: Entering directory '/usr/src/linux-headers-5.16.0-rc2'
  CC [M]  /var/lib/dkms/gasket/1.0/build/gasket_core.o
  CC [M]  /var/lib/dkms/gasket/1.0/build/gasket_ioctl.o
  CC [M]  /var/lib/dkms/gasket/1.0/build/gasket_interrupt.o
  CC [M]  /var/lib/dkms/gasket/1.0/build/gasket_page_table.o
  CC [M]  /var/lib/dkms/gasket/1.0/build/gasket_sysfs.o
  CC [M]  /var/lib/dkms/gasket/1.0/build/apex_driver.o
  LD [M]  /var/lib/dkms/gasket/1.0/build/gasket.o
  LD [M]  /var/lib/dkms/gasket/1.0/build/apex.o
  MODPOST /var/lib/dkms/gasket/1.0/build/Module.symvers
ERROR: modpost: module gasket uses symbol dma_buf_detach from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_put from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_get from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_unmap_attachment from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_map_attachment from namespace DMA_BUF, but does not import it.
ERROR: modpost: module gasket uses symbol dma_buf_attach from namespace DMA_BUF, but does not import it.
make[1]: *** [scripts/Makefile.modpost:134: /var/lib/dkms/gasket/1.0/build/Module.symvers] Error 1
make[1]: *** Deleting file '/var/lib/dkms/gasket/1.0/build/Module.symvers'
make: *** [Makefile:1761: modules] Error 2
make: Leaving directory '/usr/src/linux-headers-5.16.0-rc2'

Signed-off-by: Yan <yan-wyb@foxmail.com>
@@ -56,6 +56,8 @@
#include "gasket_constants.h"
#include "gasket_core.h"

MODULE_IMPORT_NS(DMA_BUF);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will need #if kernel version >= ... wrapping, or the build will break for older kernels.

I ran into that issue as well. I do have a CLA in place and will send an own PR, addressing this along that. Just that you are not surprised, @yan-wyb.

@mbrooksx
Copy link
Collaborator

See @jan-kiszka 's comment - used his PR for backwards compatibility and an active CLA. Thanks for contributing.

@mbrooksx mbrooksx closed this Feb 22, 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

Successfully merging this pull request may close these issues.

3 participants