* [PATCH] cifsd: Select SG_POOL for SMB_SERVER
@ 2021-04-07 3:45 ` Zhang Xiaoxu
2021-04-07 4:02 ` Namjae Jeon
0 siblings, 1 reply; 2+ messages in thread
From: Zhang Xiaoxu @ 2021-04-07 3:45 UTC (permalink / raw)
To: namjae.jeon, zhangxiaoxu5, sfrench, sergey.senozhatsky, hyc.lee,
linux-cifs, linux-cifsd-devel, yukuai3
hulk-robot following build error:
fs/cifsd/transport_rdma.c: In function ‘read_write_done’:
fs/cifsd/transport_rdma.c:1297:2: error: implicit declaration of function ‘sg_free_table_chained’ [-Werror=implicit-function-declaration]
1297 | sg_free_table_chained(&msg->sgt, SG_CHUNK_SIZE);
The reason is CONFIG_SG_POOL is not enabled in the config, to
avoid such failure, select SG_POOL in Kconfig for SMB_SERVER.
Fixes: 75b8988dfe83 ("cifsd: add server handler for central processing and tranport layers")
Signed-off-by: Zhang Xiaoxu <zhangxiaoxu5@huawei.com>
---
fs/cifsd/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/fs/cifsd/Kconfig b/fs/cifsd/Kconfig
index d1ac53c83125..fb57672424be 100644
--- a/fs/cifsd/Kconfig
+++ b/fs/cifsd/Kconfig
@@ -17,6 +17,7 @@ config SMB_SERVER
select CRYPTO_AEAD2
select CRYPTO_CCM
select CRYPTO_GCM
+ select SG_POOL
default n
help
Choose Y here if you want to allow SMB3 compliant clients
--
2.25.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* RE: [PATCH] cifsd: Select SG_POOL for SMB_SERVER
2021-04-07 3:45 ` [PATCH] cifsd: Select SG_POOL for SMB_SERVER Zhang Xiaoxu
@ 2021-04-07 4:02 ` Namjae Jeon
0 siblings, 0 replies; 2+ messages in thread
From: Namjae Jeon @ 2021-04-07 4:02 UTC (permalink / raw)
To: 'Zhang Xiaoxu'
Cc: sfrench, sergey.senozhatsky, hyc.lee, linux-cifs,
linux-cifsd-devel, yukuai3
> hulk-robot following build error:
> fs/cifsd/transport_rdma.c: In function ‘read_write_done’:
> fs/cifsd/transport_rdma.c:1297:2: error: implicit declaration of function ‘sg_free_table_chained’ [-
> Werror=implicit-function-declaration]
> 1297 | sg_free_table_chained(&msg->sgt, SG_CHUNK_SIZE);
>
> The reason is CONFIG_SG_POOL is not enabled in the config, to avoid such failure, select SG_POOL in
> Kconfig for SMB_SERVER.
>
> Fixes: 75b8988dfe83 ("cifsd: add server handler for central processing and tranport layers")
> Signed-off-by: Zhang Xiaoxu <zhangxiaoxu5@huawei.com>
> ---
> fs/cifsd/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/fs/cifsd/Kconfig b/fs/cifsd/Kconfig index d1ac53c83125..fb57672424be 100644
> --- a/fs/cifsd/Kconfig
> +++ b/fs/cifsd/Kconfig
> @@ -17,6 +17,7 @@ config SMB_SERVER
> select CRYPTO_AEAD2
> select CRYPTO_CCM
> select CRYPTO_GCM
> + select SG_POOL
> default n
> help
> Choose Y here if you want to allow SMB3 compliant clients
transport_rdma.c is built when SMB_SERVER_SMBDIRECT is set.
So we need to move it to config SMB_SERVER_SMBDIRECT.
diff --git a/fs/cifsd/Kconfig b/fs/cifsd/Kconfig
index d1ac53c83125..b94cf1158182 100644
--- a/fs/cifsd/Kconfig
+++ b/fs/cifsd/Kconfig
@@ -43,6 +43,7 @@ config SMB_SERVER
config SMB_SERVER_SMBDIRECT
bool "Support for SMB Direct protocol"
depends on SMB_SERVER=m && INFINIBAND && INFINIBAND_ADDR_TRANS || SMB_SERVER=y && INFINIBAND=y && INFINIBAND_ADDR_TRANS=y
+ select SG_POOL
default n
help
I will directly update your patch.
Thanks for your patch!
> --
> 2.25.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-04-07 4:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20210407034026epcas1p30694eae719a68b92f759dc4e5a92b542@epcas1p3.samsung.com>
2021-04-07 3:45 ` [PATCH] cifsd: Select SG_POOL for SMB_SERVER Zhang Xiaoxu
2021-04-07 4:02 ` 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).