Re: [PATCH net-next v3 03/15] quic: provide common utilities and data structures

Xin Long
2025-09-23T15:49:13Z

On Tue, Sep 23, 2025 at 5:06 AM Simon Horman <horms@kernel.org> wrote:
>
> On Thu, Sep 18, 2025 at 06:34:52PM -0400, Xin Long wrote:
>
> > index f79f43f0c17f..b54532916aa2 100644
> > --- a/net/quic/protocol.c
> > +++ b/net/quic/protocol.c
> > @@ -336,6 +336,9 @@ static __init int quic_init(void)
> > if (err)
> > goto err_percpu_counter;
> >
> > + if (quic_hash_tables_init())
>
> Hi Xin,
>
> If we reach here then the function will return err, which is 0.
> So it seems that err should be set to a negative error value instead.
> Perhaps the return value of quic_hash_tables_init.
Good catch!