Re: git: 349d1366edd2 - main - socket: Export the FIB number of sockets

From: Mark Johnston <markj_at_freebsd.org>
Date: Tue, 12 Nov 2024 02:07:53 UTC
On Mon, Nov 11, 2024 at 04:56:44PM -0800, Gleb Smirnoff wrote:
> On Mon, Nov 04, 2024 at 02:56:33PM +0000, Mark Johnston wrote:
> M> diff --git a/sys/sys/socketvar.h b/sys/sys/socketvar.h
> M> index 40464f555816..fda8d23f5649 100644
> M> --- a/sys/sys/socketvar.h
> M> +++ b/sys/sys/socketvar.h
> M> @@ -615,7 +615,8 @@ struct xsocket {
> M>  	uint32_t	so_qlimit;
> M>  	pid_t		so_pgid;
> M>  	uid_t		so_uid;
> M> -	int32_t		so_spare32[8];
> M> +	int32_t		so_fibnum;
> M> +	int32_t		so_spare32[7];
> M>  	int16_t		so_type;
> M>  	int16_t		so_options;
> M>  	int16_t		so_linger;
> 
> I would suggest not to spend the spares in the main branch.
> So this commit should add the field, keeping 8 spares. But
> the merge to stable/14 should use a spare.

This would break consumers running in a 14.x jail on a 15.x host, no?
With VNET jails, this compatibility is very useful to have in my
experience.

> Otherwise we eventually will run out of spares in main
> and also nearly run out of spares in some stable/22 branch.
> 
> I'd suggest to follow the policy of not spending spares
> in the main branch for all structures, not just xsocket.
> 
> -- 
> Gleb Smirnoff