linux-cifs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Building ksmbd as external module on kernel < 5.15
@ 2021-09-25  9:15 Ralph Boehme
  2021-09-25  9:51 ` Ralph Boehme
  0 siblings, 1 reply; 3+ messages in thread
From: Ralph Boehme @ 2021-09-25  9:15 UTC (permalink / raw)
  To: COMMON INTERNET FILE SYSTEM SERVER; +Cc: Steve French, Namjae Jeon


[-- Attachment #1.1: Type: text/plain, Size: 991 bytes --]

Hi!

I noticed two problems building ksmbd as external module from 
github.com:smfrench/smb3-kernel.git:

1. Need to manually export CONFIG_SMB_SERVER=m even though .config 
contains it.

2. Use of lookup_one() not present in kernels before 5.15.

See the following shell dump for the problems in action:

<https://cpaste.org/?cddf4595181c8c42#8HKBmcBcP8hE2RUhG3wi3QbsQmAXHDtVuMFYfeuYz7Lj>

Wrt 1: this was exported in Namjae's old git repo 
https://github.com/namjaejeon/ksmbd but was lost in the Makefile rewrite.

Wrt 2: it seems the patches included in 
https://github.com/namjaejeon/ksmbd around 
9caf6ed9f0b6c71b0bcc661b316e08172b7a6d55 are missing in 
github.com:smfrench/smb3-kernel.git.

How shall we address this? Is building the module on older kernels (I'm 
on 5.13.14-200.fc34.x86_64) actually supported?

Thanks!
-slow

-- 
Ralph Boehme, Samba Team                 https://samba.org/
SerNet Samba Team Lead      https://sernet.de/en/team-samba


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Building ksmbd as external module on kernel < 5.15
  2021-09-25  9:15 Building ksmbd as external module on kernel < 5.15 Ralph Boehme
@ 2021-09-25  9:51 ` Ralph Boehme
  2021-09-25 10:32   ` Namjae Jeon
  0 siblings, 1 reply; 3+ messages in thread
From: Ralph Boehme @ 2021-09-25  9:51 UTC (permalink / raw)
  To: COMMON INTERNET FILE SYSTEM SERVER; +Cc: Steve French, Namjae Jeon


[-- Attachment #1.1: Type: text/plain, Size: 569 bytes --]

Am 25.09.21 um 11:15 schrieb Ralph Boehme:
> Is building the module on older kernels (I'm 
> on 5.13.14-200.fc34.x86_64) actually supported?

as checkpatch.pl warns about using LINUX_VERSION_CODE, I guess the 
building on older kernels is not supported anymore.

Is there any other way to just compile the module against a running 
distro kernel? Or do I have to bite the bullit and build and install the 
full kernel?

-slow

-- 
Ralph Boehme, Samba Team                 https://samba.org/
SerNet Samba Team Lead      https://sernet.de/en/team-samba


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Building ksmbd as external module on kernel < 5.15
  2021-09-25  9:51 ` Ralph Boehme
@ 2021-09-25 10:32   ` Namjae Jeon
  0 siblings, 0 replies; 3+ messages in thread
From: Namjae Jeon @ 2021-09-25 10:32 UTC (permalink / raw)
  To: Ralph Boehme; +Cc: COMMON INTERNET FILE SYSTEM SERVER, Steve French

2021-09-25 18:51 GMT+09:00, Ralph Boehme <slow@samba.org>:
> Am 25.09.21 um 11:15 schrieb Ralph Boehme:
>> Is building the module on older kernels (I'm
>> on 5.13.14-200.fc34.x86_64) actually supported?
>
> as checkpatch.pl warns about using LINUX_VERSION_CODE, I guess the
> building on older kernels is not supported anymore.
>
> Is there any other way to just compile the module against a running
> distro kernel? Or do I have to bite the bullit and build and install the
> full kernel?
Yes. Install the latest kernel or use QEMU.
>
> -slow
>
> --
> Ralph Boehme, Samba Team                 https://samba.org/
> SerNet Samba Team Lead      https://sernet.de/en/team-samba
>
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-09-25 10:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-25  9:15 Building ksmbd as external module on kernel < 5.15 Ralph Boehme
2021-09-25  9:51 ` Ralph Boehme
2021-09-25 10:32   ` Namjae Jeon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).