* [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n
@ 2020-11-16 2:31 Tom Seewald
2020-11-17 22:25 ` Jakub Kicinski
0 siblings, 1 reply; 8+ messages in thread
From: Tom Seewald @ 2020-11-16 2:31 UTC (permalink / raw)
To: netdev
Cc: linux-kernel, Tom Seewald, Vishal Kulkarni, David S. Miller,
Jakub Kicinski, Rohit Maheshwari
After commit 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough")
building the kernel with CHELSIO_T4=y and CHELSIO_TLS_DEVICE=n results
in the following error:
ld: drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.o: in function
`cxgb_select_queue':
cxgb4_main.c:(.text+0x2dac): undefined reference to `tls_validate_xmit_skb'
This is caused by cxgb_select_queue() calling cxgb4_is_ktls_skb() without
checking if CHELSIO_TLS_DEVICE=y. Fix this by calling cxgb4_is_ktls_skb()
only when this config option is enabled.
Fixes: 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough")
Signed-off-by: Tom Seewald <tseewald@gmail.com>
---
drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
index 7fd264a6d085..8e8783afd6df 100644
--- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
+++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
@@ -1176,7 +1176,9 @@ static u16 cxgb_select_queue(struct net_device *dev, struct sk_buff *skb,
txq = netdev_pick_tx(dev, skb, sb_dev);
if (xfrm_offload(skb) || is_ptp_enabled(skb, dev) ||
skb->encapsulation ||
- cxgb4_is_ktls_skb(skb) ||
+#if IS_ENABLED(CONFIG_CHELSIO_TLS_DEVICE)
+ cxgb4_is_ktls_skb(skb) ||
+#endif /* CHELSIO_TLS_DEVICE */
(proto != IPPROTO_TCP && proto != IPPROTO_UDP))
txq = txq % pi->nqsets;
--
2.20.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n 2020-11-16 2:31 [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n Tom Seewald @ 2020-11-17 22:25 ` Jakub Kicinski 2020-11-19 5:40 ` Tom Seewald 0 siblings, 1 reply; 8+ messages in thread From: Jakub Kicinski @ 2020-11-17 22:25 UTC (permalink / raw) To: Tom Seewald Cc: netdev, linux-kernel, Vishal Kulkarni, David S. Miller, Rohit Maheshwari On Sun, 15 Nov 2020 20:31:40 -0600 Tom Seewald wrote: > After commit 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > building the kernel with CHELSIO_T4=y and CHELSIO_TLS_DEVICE=n results > in the following error: > > ld: drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.o: in function > `cxgb_select_queue': > cxgb4_main.c:(.text+0x2dac): undefined reference to `tls_validate_xmit_skb' > > This is caused by cxgb_select_queue() calling cxgb4_is_ktls_skb() without > checking if CHELSIO_TLS_DEVICE=y. Fix this by calling cxgb4_is_ktls_skb() > only when this config option is enabled. > > Fixes: 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > Signed-off-by: Tom Seewald <tseewald@gmail.com> > --- > drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > index 7fd264a6d085..8e8783afd6df 100644 > --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > @@ -1176,7 +1176,9 @@ static u16 cxgb_select_queue(struct net_device *dev, struct sk_buff *skb, > txq = netdev_pick_tx(dev, skb, sb_dev); > if (xfrm_offload(skb) || is_ptp_enabled(skb, dev) || > skb->encapsulation || > - cxgb4_is_ktls_skb(skb) || > +#if IS_ENABLED(CONFIG_CHELSIO_TLS_DEVICE) > + cxgb4_is_ktls_skb(skb) || > +#endif /* CHELSIO_TLS_DEVICE */ > (proto != IPPROTO_TCP && proto != IPPROTO_UDP)) > txq = txq % pi->nqsets; > The tls header already tries to solve this issue, it just does it poorly. This is a better fix: diff --git a/include/net/tls.h b/include/net/tls.h index baf1e99d8193..2ff3f4f7954a 100644 --- a/include/net/tls.h +++ b/include/net/tls.h @@ -441,11 +441,11 @@ struct sk_buff * tls_validate_xmit_skb(struct sock *sk, struct net_device *dev, struct sk_buff *skb); static inline bool tls_is_sk_tx_device_offloaded(struct sock *sk) { -#ifdef CONFIG_SOCK_VALIDATE_XMIT +#ifdef CONFIG_TLS_DEVICE return sk_fullsock(sk) && (smp_load_acquire(&sk->sk_validate_xmit_skb) == &tls_validate_xmit_skb); #else return false; Please test this and submit if it indeed solves the problem. Thanks! ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n 2020-11-17 22:25 ` Jakub Kicinski @ 2020-11-19 5:40 ` Tom Seewald 2020-11-19 17:37 ` Jakub Kicinski 0 siblings, 1 reply; 8+ messages in thread From: Tom Seewald @ 2020-11-19 5:40 UTC (permalink / raw) To: Jakub Kicinski; +Cc: netdev, David S. Miller On Tue, Nov 17, 2020 at 4:26 PM Jakub Kicinski <kuba@kernel.org> wrote: > > On Sun, 15 Nov 2020 20:31:40 -0600 Tom Seewald wrote: > > After commit 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > > building the kernel with CHELSIO_T4=y and CHELSIO_TLS_DEVICE=n results > > in the following error: > > > > ld: drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.o: in function > > `cxgb_select_queue': > > cxgb4_main.c:(.text+0x2dac): undefined reference to `tls_validate_xmit_skb' > > > > This is caused by cxgb_select_queue() calling cxgb4_is_ktls_skb() without > > checking if CHELSIO_TLS_DEVICE=y. Fix this by calling cxgb4_is_ktls_skb() > > only when this config option is enabled. > > > > Fixes: 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > > Signed-off-by: Tom Seewald <tseewald@gmail.com> > > --- > > drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > index 7fd264a6d085..8e8783afd6df 100644 > > --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > @@ -1176,7 +1176,9 @@ static u16 cxgb_select_queue(struct net_device *dev, struct sk_buff *skb, > > txq = netdev_pick_tx(dev, skb, sb_dev); > > if (xfrm_offload(skb) || is_ptp_enabled(skb, dev) || > > skb->encapsulation || > > - cxgb4_is_ktls_skb(skb) || > > +#if IS_ENABLED(CONFIG_CHELSIO_TLS_DEVICE) > > + cxgb4_is_ktls_skb(skb) || > > +#endif /* CHELSIO_TLS_DEVICE */ > > (proto != IPPROTO_TCP && proto != IPPROTO_UDP)) > > txq = txq % pi->nqsets; > > > > The tls header already tries to solve this issue, it just does it > poorly. This is a better fix: > > diff --git a/include/net/tls.h b/include/net/tls.h > index baf1e99d8193..2ff3f4f7954a 100644 > --- a/include/net/tls.h > +++ b/include/net/tls.h > @@ -441,11 +441,11 @@ struct sk_buff * > tls_validate_xmit_skb(struct sock *sk, struct net_device *dev, > struct sk_buff *skb); > > static inline bool tls_is_sk_tx_device_offloaded(struct sock *sk) > { > -#ifdef CONFIG_SOCK_VALIDATE_XMIT > +#ifdef CONFIG_TLS_DEVICE > return sk_fullsock(sk) && > (smp_load_acquire(&sk->sk_validate_xmit_skb) == > &tls_validate_xmit_skb); > #else > return false; > > > Please test this and submit if it indeed solves the problem. > > Thanks! Hi Jakub, Thanks for the reply, unfortunately that patch does not resolve the issue, I still get the same error as before. After looking into this a bit further, the issue seems to be with CONFIG_TLS=m as everything works when CONFIG_TLS=y. I also see that there was a similar issue [1] reported by Intel's kbuild test robot where the cxgb4 driver isn't able to see the TLS symbols when CONFIG_TLS=m. [1] https://lkml.org/lkml/2020/8/7/601 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n 2020-11-19 5:40 ` Tom Seewald @ 2020-11-19 17:37 ` Jakub Kicinski 2020-11-20 6:54 ` Tom Seewald 0 siblings, 1 reply; 8+ messages in thread From: Jakub Kicinski @ 2020-11-19 17:37 UTC (permalink / raw) To: Tom Seewald; +Cc: netdev, David S. Miller On Wed, 18 Nov 2020 23:40:40 -0600 Tom Seewald wrote: > On Tue, Nov 17, 2020 at 4:26 PM Jakub Kicinski <kuba@kernel.org> wrote: > > > > On Sun, 15 Nov 2020 20:31:40 -0600 Tom Seewald wrote: > > > After commit 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > > > building the kernel with CHELSIO_T4=y and CHELSIO_TLS_DEVICE=n results > > > in the following error: > > > > > > ld: drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.o: in function > > > `cxgb_select_queue': > > > cxgb4_main.c:(.text+0x2dac): undefined reference to `tls_validate_xmit_skb' > > > > > > This is caused by cxgb_select_queue() calling cxgb4_is_ktls_skb() without > > > checking if CHELSIO_TLS_DEVICE=y. Fix this by calling cxgb4_is_ktls_skb() > > > only when this config option is enabled. > > > > > > Fixes: 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > > > Signed-off-by: Tom Seewald <tseewald@gmail.com> > > > --- > > > drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 4 +++- > > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > > index 7fd264a6d085..8e8783afd6df 100644 > > > --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > > +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > > @@ -1176,7 +1176,9 @@ static u16 cxgb_select_queue(struct net_device *dev, struct sk_buff *skb, > > > txq = netdev_pick_tx(dev, skb, sb_dev); > > > if (xfrm_offload(skb) || is_ptp_enabled(skb, dev) || > > > skb->encapsulation || > > > - cxgb4_is_ktls_skb(skb) || > > > +#if IS_ENABLED(CONFIG_CHELSIO_TLS_DEVICE) > > > + cxgb4_is_ktls_skb(skb) || > > > +#endif /* CHELSIO_TLS_DEVICE */ > > > (proto != IPPROTO_TCP && proto != IPPROTO_UDP)) > > > txq = txq % pi->nqsets; > > > > > > > The tls header already tries to solve this issue, it just does it > > poorly. This is a better fix: > > > > diff --git a/include/net/tls.h b/include/net/tls.h > > index baf1e99d8193..2ff3f4f7954a 100644 > > --- a/include/net/tls.h > > +++ b/include/net/tls.h > > @@ -441,11 +441,11 @@ struct sk_buff * > > tls_validate_xmit_skb(struct sock *sk, struct net_device *dev, > > struct sk_buff *skb); > > > > static inline bool tls_is_sk_tx_device_offloaded(struct sock *sk) > > { > > -#ifdef CONFIG_SOCK_VALIDATE_XMIT > > +#ifdef CONFIG_TLS_DEVICE > > return sk_fullsock(sk) && > > (smp_load_acquire(&sk->sk_validate_xmit_skb) == > > &tls_validate_xmit_skb); > > #else > > return false; > > > > > > Please test this and submit if it indeed solves the problem. > > > > Thanks! > > Hi Jakub, > > Thanks for the reply, unfortunately that patch does not resolve the > issue, I still get the same error as before. After looking into this a > bit further, the issue seems to be with CONFIG_TLS=m as everything > works when CONFIG_TLS=y. > > I also see that there was a similar issue [1] reported by Intel's > kbuild test robot where the cxgb4 driver isn't able to see the TLS > symbols when CONFIG_TLS=m. Interesting. Does your original patch solve the allyesconfig + TLS=m problem? Seems to me that CHELSIO_T4 should depend on (TLS || TLS=n), the CONFIG_CHELSIO_TLS_DEVICE has the dependency but AFAICT nothing prevents CONFIG_CHELSIO_TLS_DEVICE=m and CHELSIO_T4=y and cxgb4_main.c is under the latter. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n 2020-11-19 17:37 ` Jakub Kicinski @ 2020-11-20 6:54 ` Tom Seewald 2020-11-20 15:35 ` Jakub Kicinski 0 siblings, 1 reply; 8+ messages in thread From: Tom Seewald @ 2020-11-20 6:54 UTC (permalink / raw) To: Jakub Kicinski; +Cc: netdev, David S. Miller On Thu, Nov 19, 2020 at 11:37 AM Jakub Kicinski <kuba@kernel.org> wrote: > > On Wed, 18 Nov 2020 23:40:40 -0600 Tom Seewald wrote: > > On Tue, Nov 17, 2020 at 4:26 PM Jakub Kicinski <kuba@kernel.org> wrote: > > > > > > On Sun, 15 Nov 2020 20:31:40 -0600 Tom Seewald wrote: > > > > After commit 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > > > > building the kernel with CHELSIO_T4=y and CHELSIO_TLS_DEVICE=n results > > > > in the following error: > > > > > > > > ld: drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.o: in function > > > > `cxgb_select_queue': > > > > cxgb4_main.c:(.text+0x2dac): undefined reference to `tls_validate_xmit_skb' > > > > > > > > This is caused by cxgb_select_queue() calling cxgb4_is_ktls_skb() without > > > > checking if CHELSIO_TLS_DEVICE=y. Fix this by calling cxgb4_is_ktls_skb() > > > > only when this config option is enabled. > > > > > > > > Fixes: 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > > > > Signed-off-by: Tom Seewald <tseewald@gmail.com> > > > > --- > > > > drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 4 +++- > > > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > > > index 7fd264a6d085..8e8783afd6df 100644 > > > > --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > > > +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c > > > > @@ -1176,7 +1176,9 @@ static u16 cxgb_select_queue(struct net_device *dev, struct sk_buff *skb, > > > > txq = netdev_pick_tx(dev, skb, sb_dev); > > > > if (xfrm_offload(skb) || is_ptp_enabled(skb, dev) || > > > > skb->encapsulation || > > > > - cxgb4_is_ktls_skb(skb) || > > > > +#if IS_ENABLED(CONFIG_CHELSIO_TLS_DEVICE) > > > > + cxgb4_is_ktls_skb(skb) || > > > > +#endif /* CHELSIO_TLS_DEVICE */ > > > > (proto != IPPROTO_TCP && proto != IPPROTO_UDP)) > > > > txq = txq % pi->nqsets; > > > > > > > > > > The tls header already tries to solve this issue, it just does it > > > poorly. This is a better fix: > > > > > > diff --git a/include/net/tls.h b/include/net/tls.h > > > index baf1e99d8193..2ff3f4f7954a 100644 > > > --- a/include/net/tls.h > > > +++ b/include/net/tls.h > > > @@ -441,11 +441,11 @@ struct sk_buff * > > > tls_validate_xmit_skb(struct sock *sk, struct net_device *dev, > > > struct sk_buff *skb); > > > > > > static inline bool tls_is_sk_tx_device_offloaded(struct sock *sk) > > > { > > > -#ifdef CONFIG_SOCK_VALIDATE_XMIT > > > +#ifdef CONFIG_TLS_DEVICE > > > return sk_fullsock(sk) && > > > (smp_load_acquire(&sk->sk_validate_xmit_skb) == > > > &tls_validate_xmit_skb); > > > #else > > > return false; > > > > > > > > > Please test this and submit if it indeed solves the problem. > > > > > > Thanks! > > > > Hi Jakub, > > > > Thanks for the reply, unfortunately that patch does not resolve the > > issue, I still get the same error as before. After looking into this a > > bit further, the issue seems to be with CONFIG_TLS=m as everything > > works when CONFIG_TLS=y. > > > > I also see that there was a similar issue [1] reported by Intel's > > kbuild test robot where the cxgb4 driver isn't able to see the TLS > > symbols when CONFIG_TLS=m. > > Interesting. Does your original patch solve the allyesconfig + TLS=m > problem? No it does not solve it, my original patch was incorrect and should not be applied. It only masks the issue when using my specific kernel config. > Seems to me that CHELSIO_T4 should depend on (TLS || TLS=n), the > CONFIG_CHELSIO_TLS_DEVICE has the dependency but AFAICT nothing prevents > CONFIG_CHELSIO_TLS_DEVICE=m and CHELSIO_T4=y and cxgb4_main.c is under > the latter. You are right, adding (TLS || TLS=n) for CHELSIO_T4 resolves the build error I am encountering. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n 2020-11-20 6:54 ` Tom Seewald @ 2020-11-20 15:35 ` Jakub Kicinski 2020-11-20 19:25 ` [PATCH] cxgb4: Fix build failure when CONFIG_TLS=m Tom Seewald 0 siblings, 1 reply; 8+ messages in thread From: Jakub Kicinski @ 2020-11-20 15:35 UTC (permalink / raw) To: Tom Seewald; +Cc: netdev, David S. Miller On Fri, 20 Nov 2020 00:54:00 -0600 Tom Seewald wrote: > > Seems to me that CHELSIO_T4 should depend on (TLS || TLS=n), the > > CONFIG_CHELSIO_TLS_DEVICE has the dependency but AFAICT nothing prevents > > CONFIG_CHELSIO_TLS_DEVICE=m and CHELSIO_T4=y and cxgb4_main.c is under > > the latter. > > You are right, adding (TLS || TLS=n) for CHELSIO_T4 resolves the build > error I am encountering. Great, please submit a patch. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] cxgb4: Fix build failure when CONFIG_TLS=m 2020-11-20 15:35 ` Jakub Kicinski @ 2020-11-20 19:25 ` Tom Seewald 2020-11-21 21:12 ` Jakub Kicinski 0 siblings, 1 reply; 8+ messages in thread From: Tom Seewald @ 2020-11-20 19:25 UTC (permalink / raw) To: netdev; +Cc: linux-kernel, davem, kuba, ayush.sawal, rajur, Tom Seewald After commit 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") whenever CONFIG_TLS=m and CONFIG_CHELSIO_T4=y, the following build failure occurs: ld: drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.o: in function `cxgb_select_queue': cxgb4_main.c:(.text+0x2dac): undefined reference to `tls_validate_xmit_skb' Fix this by ensuring that if TLS is set to be a module, CHELSIO_T4 will also be compiled as a module. As otherwise the cxgb4 driver will not be able to access TLS' symbols. Fixes: 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") Signed-off-by: Tom Seewald <tseewald@gmail.com> --- drivers/net/ethernet/chelsio/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/chelsio/Kconfig b/drivers/net/ethernet/chelsio/Kconfig index 87cc0ef68b31..8ba0e08e5e64 100644 --- a/drivers/net/ethernet/chelsio/Kconfig +++ b/drivers/net/ethernet/chelsio/Kconfig @@ -68,7 +68,7 @@ config CHELSIO_T3 config CHELSIO_T4 tristate "Chelsio Communications T4/T5/T6 Ethernet support" - depends on PCI && (IPV6 || IPV6=n) + depends on PCI && (IPV6 || IPV6=n) && (TLS || TLS=n) select FW_LOADER select MDIO select ZLIB_DEFLATE -- 2.20.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] cxgb4: Fix build failure when CONFIG_TLS=m 2020-11-20 19:25 ` [PATCH] cxgb4: Fix build failure when CONFIG_TLS=m Tom Seewald @ 2020-11-21 21:12 ` Jakub Kicinski 0 siblings, 0 replies; 8+ messages in thread From: Jakub Kicinski @ 2020-11-21 21:12 UTC (permalink / raw) To: Tom Seewald; +Cc: netdev, linux-kernel, davem, ayush.sawal, rajur On Fri, 20 Nov 2020 13:25:28 -0600 Tom Seewald wrote: > After commit 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > whenever CONFIG_TLS=m and CONFIG_CHELSIO_T4=y, the following build > failure occurs: > > ld: drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.o: in function > `cxgb_select_queue': > cxgb4_main.c:(.text+0x2dac): undefined reference to `tls_validate_xmit_skb' > > Fix this by ensuring that if TLS is set to be a module, CHELSIO_T4 will > also be compiled as a module. As otherwise the cxgb4 driver will not be > able to access TLS' symbols. > > Fixes: 9d2e5e9eeb59 ("cxgb4/ch_ktls: decrypted bit is not enough") > Signed-off-by: Tom Seewald <tseewald@gmail.com> Applied, thanks! ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-11-21 21:12 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-11-16 2:31 [PATCH] cxbgb4: Fix build failure when CHELSIO_TLS_DEVICE=n Tom Seewald 2020-11-17 22:25 ` Jakub Kicinski 2020-11-19 5:40 ` Tom Seewald 2020-11-19 17:37 ` Jakub Kicinski 2020-11-20 6:54 ` Tom Seewald 2020-11-20 15:35 ` Jakub Kicinski 2020-11-20 19:25 ` [PATCH] cxgb4: Fix build failure when CONFIG_TLS=m Tom Seewald 2020-11-21 21:12 ` Jakub Kicinski
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.