Re: [PATCH net-next 3/4] tcp: use EXPORT_IPV6_MOD[_GPL]()

[Sabrina Dubroca <sd@queasysnail.net>]

2025-02-10, 10:56:22 +0100, Eric Dumazet wrote:
> On Mon, Feb 10, 2025 at 10:44 AM Eric Dumazet <edumazet@google.com> wrote:
> >
> > On Mon, Feb 10, 2025 at 10:30 AM Sabrina Dubroca <sd@queasysnail.net> wrote:
> > >
> > > 2025-02-10, 08:28:04 +0000, Eric Dumazet wrote:
> > > > @@ -613,7 +613,7 @@ __poll_t tcp_poll(struct file *file, struct socket *sock, poll_table *wait)
> > > >
> > > >       return mask;
> > > >  }
> > > > -EXPORT_SYMBOL(tcp_poll);
> > > > +EXPORT_IPV6_MOD(tcp_poll);
> > >
> > > ktls uses it directly (net/tls/tls_main.c):
> >
> >
> > Oh, right.
> >
> > >
> > > static __poll_t tls_sk_poll(struct file *file, struct socket *sock,
> > >                             struct poll_table_struct *wait)
> > > {
> > >         struct tls_sw_context_rx *ctx;
> > >         struct tls_context *tls_ctx;
> > >         struct sock *sk = sock->sk;
> > >         struct sk_psock *psock;
> > >         __poll_t mask = 0;
> > >         u8 shutdown;
> > >         int state;
> > >
> > >         mask = tcp_poll(file, sock, wait);
> > > [...]
> > > }
> > >
> > > If you want to un-export tcp_poll, I guess we'll need to add the same
> > > thing as for ->sk_data_ready (save the old ->poll to tls_context).
> >
> > No need, I simply missed tls was using tcp_poll() can could be a
> > module, I will fix in V2
> >
> 
> TLS also calls tcp_under_memory_pressure() via tcp_epollin_ready(),
> so tcp_memory_pressure needs to be exported as well.

Ah, yes, didn't think about inline helpers.

I ran a quick build since I was concerned about what the Chelsio TOE
driver might be using, but spotted this instead:

ERROR: modpost: "secure_tcpv6_seq" [drivers/infiniband/hw/irdma/irdma.ko] undefined!

-- 
Sabrina