mirror of
				https://kernel.googlesource.com/pub/scm/linux/kernel/git/torvalds/linux
				synced 2025-11-04 08:34:47 +10:00 
			
		
		
		
	The change of sizeof(struct smc_diag_linkinfo) by commit79d39fc503("net/smc: Add netlink net namespace support") introduced an ABI regression: since struct smc_diag_lgrinfo contains an object of type "struct smc_diag_linkinfo", offset of all subsequent members of struct smc_diag_lgrinfo was changed by that change. As result, applications compiled with the old version of struct smc_diag_linkinfo will receive garbage in struct smc_diag_lgrinfo.role if the kernel implements this new version of struct smc_diag_linkinfo. Fix this regression by reverting the part of commit79d39fc503that changes struct smc_diag_linkinfo. After all, there is SMC_GEN_NETLINK interface which is good enough, so there is probably no need to touch the smc_diag ABI in the first place. Fixes:79d39fc503("net/smc: Add netlink net namespace support") Signed-off-by: Dmitry V. Levin <ldv@altlinux.org> Reviewed-by: Karsten Graul <kgraul@linux.ibm.com> Link: https://lore.kernel.org/r/20220202030904.GA9742@altlinux.org Signed-off-by: Jakub Kicinski <kuba@kernel.org>
		
			
				
	
	
		
			113 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			113 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
 | 
						|
#ifndef _UAPI_SMC_DIAG_H_
 | 
						|
#define _UAPI_SMC_DIAG_H_
 | 
						|
 | 
						|
#include <linux/types.h>
 | 
						|
#include <linux/inet_diag.h>
 | 
						|
#include <rdma/ib_user_verbs.h>
 | 
						|
 | 
						|
/* Request structure */
 | 
						|
struct smc_diag_req {
 | 
						|
	__u8	diag_family;
 | 
						|
	__u8	pad[2];
 | 
						|
	__u8	diag_ext;		/* Query extended information */
 | 
						|
	struct inet_diag_sockid	id;
 | 
						|
};
 | 
						|
 | 
						|
/* Base info structure. It contains socket identity (addrs/ports/cookie) based
 | 
						|
 * on the internal clcsock, and more SMC-related socket data
 | 
						|
 */
 | 
						|
struct smc_diag_msg {
 | 
						|
	__u8		diag_family;
 | 
						|
	__u8		diag_state;
 | 
						|
	union {
 | 
						|
		__u8	diag_mode;
 | 
						|
		__u8	diag_fallback; /* the old name of the field */
 | 
						|
	};
 | 
						|
	__u8		diag_shutdown;
 | 
						|
	struct inet_diag_sockid id;
 | 
						|
 | 
						|
	__u32		diag_uid;
 | 
						|
	__aligned_u64	diag_inode;
 | 
						|
};
 | 
						|
 | 
						|
/* Mode of a connection */
 | 
						|
enum {
 | 
						|
	SMC_DIAG_MODE_SMCR,
 | 
						|
	SMC_DIAG_MODE_FALLBACK_TCP,
 | 
						|
	SMC_DIAG_MODE_SMCD,
 | 
						|
};
 | 
						|
 | 
						|
/* Extensions */
 | 
						|
 | 
						|
enum {
 | 
						|
	SMC_DIAG_NONE,
 | 
						|
	SMC_DIAG_CONNINFO,
 | 
						|
	SMC_DIAG_LGRINFO,
 | 
						|
	SMC_DIAG_SHUTDOWN,
 | 
						|
	SMC_DIAG_DMBINFO,
 | 
						|
	SMC_DIAG_FALLBACK,
 | 
						|
	__SMC_DIAG_MAX,
 | 
						|
};
 | 
						|
 | 
						|
#define SMC_DIAG_MAX (__SMC_DIAG_MAX - 1)
 | 
						|
 | 
						|
/* SMC_DIAG_CONNINFO */
 | 
						|
 | 
						|
struct smc_diag_cursor {
 | 
						|
	__u16	reserved;
 | 
						|
	__u16	wrap;
 | 
						|
	__u32	count;
 | 
						|
};
 | 
						|
 | 
						|
struct smc_diag_conninfo {
 | 
						|
	__u32			token;		/* unique connection id */
 | 
						|
	__u32			sndbuf_size;	/* size of send buffer */
 | 
						|
	__u32			rmbe_size;	/* size of RMB element */
 | 
						|
	__u32			peer_rmbe_size;	/* size of peer RMB element */
 | 
						|
	/* local RMB element cursors */
 | 
						|
	struct smc_diag_cursor	rx_prod;	/* received producer cursor */
 | 
						|
	struct smc_diag_cursor	rx_cons;	/* received consumer cursor */
 | 
						|
	/* peer RMB element cursors */
 | 
						|
	struct smc_diag_cursor	tx_prod;	/* sent producer cursor */
 | 
						|
	struct smc_diag_cursor	tx_cons;	/* sent consumer cursor */
 | 
						|
	__u8			rx_prod_flags;	/* received producer flags */
 | 
						|
	__u8			rx_conn_state_flags; /* recvd connection flags*/
 | 
						|
	__u8			tx_prod_flags;	/* sent producer flags */
 | 
						|
	__u8			tx_conn_state_flags; /* sent connection flags*/
 | 
						|
	/* send buffer cursors */
 | 
						|
	struct smc_diag_cursor	tx_prep;	/* prepared to be sent cursor */
 | 
						|
	struct smc_diag_cursor	tx_sent;	/* sent cursor */
 | 
						|
	struct smc_diag_cursor	tx_fin;		/* confirmed sent cursor */
 | 
						|
};
 | 
						|
 | 
						|
/* SMC_DIAG_LINKINFO */
 | 
						|
 | 
						|
struct smc_diag_linkinfo {
 | 
						|
	__u8 link_id;			/* link identifier */
 | 
						|
	__u8 ibname[IB_DEVICE_NAME_MAX]; /* name of the RDMA device */
 | 
						|
	__u8 ibport;			/* RDMA device port number */
 | 
						|
	__u8 gid[40];			/* local GID */
 | 
						|
	__u8 peer_gid[40];		/* peer GID */
 | 
						|
};
 | 
						|
 | 
						|
struct smc_diag_lgrinfo {
 | 
						|
	struct smc_diag_linkinfo	lnk[1];
 | 
						|
	__u8				role;
 | 
						|
};
 | 
						|
 | 
						|
struct smc_diag_fallback {
 | 
						|
	__u32 reason;
 | 
						|
	__u32 peer_diagnosis;
 | 
						|
};
 | 
						|
 | 
						|
struct smcd_diag_dmbinfo {		/* SMC-D Socket internals */
 | 
						|
	__u32		linkid;		/* Link identifier */
 | 
						|
	__aligned_u64	peer_gid;	/* Peer GID */
 | 
						|
	__aligned_u64	my_gid;		/* My GID */
 | 
						|
	__aligned_u64	token;		/* Token of DMB */
 | 
						|
	__aligned_u64	peer_token;	/* Token of remote DMBE */
 | 
						|
};
 | 
						|
 | 
						|
#endif /* _UAPI_SMC_DIAG_H_ */
 |