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

Compile error #1

Open
yuri-ccp opened this issue Aug 15, 2017 · 3 comments · Fixed by #2
Open

Compile error #1

yuri-ccp opened this issue Aug 15, 2017 · 3 comments · Fixed by #2

Comments

@yuri-ccp
Copy link

Hello,

I was trying to compile this module using the guide but when I tried to make the module in a CloudLinux 7 server I received this error:

make -C src
make[1]: Entrando no diretório `/root/zfs-quota/src'
make -C /usr/src/kernels/3.10.0-714.10.2.lve1.4.59.el7.x86_64 SUBDIRS=`pwd` modules
make[2]: Entrando no diretório `/usr/src/kernels/3.10.0-714.10.2.lve1.4.59.el7.x86_64'
  CC [M]  /root/zfs-quota/src/handle.o
/root/zfs-quota/src/handle.c: In function ‘get_zfsh’:
/root/zfs-quota/src/handle.c:38:17: error: ‘const struct super_operations’ has no member named ‘get_quota_root’
  return sb->s_op->get_quota_root(sb)->i_sb->s_fs_info;
                 ^
/root/zfs-quota/src/handle.c:42:1: warning: control reaches end of non-void function [-Wreturn-type]
 }
 ^
make[3]: ** [/root/zfs-quota/src/handle.o] Erro 1
make[2]: ** [_module_/root/zfs-quota/src] Erro 2
make[2]: Saindo do diretório `/usr/src/kernels/3.10.0-714.10.2.lve1.4.59.el7.x86_64'
make[1]: ** [modules] Erro 2
make[1]: Saindo do diretório `/root/zfs-quota/src'
make: ** [src] Erro 2

Can you help me?

Thanks

@paboldin
Copy link
Contributor

CL7 kernel has CONFIG_VE=y but is not a copy of OpenVZ, so it lacks get_quota_root function in super_operations structure.

We should rather to check whether get_quota_root is available during the configuration stage.

@paboldin
Copy link
Contributor

Please try the code from pull request #2

@mrqwer88 mrqwer88 reopened this Aug 30, 2017
@mrqwer88
Copy link
Contributor

Hello, @yuri-ccp

Are this fix work for you good?

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 a pull request may close this issue.

3 participants