src/sys/ofed/drivers, branch main FreeBSD source tree ofed: jiffies is unsigned long 2025-04-29T13:53:40+00:00 Konstantin Belousov kib@FreeBSD.org 2025-01-31T19:20:01+00:00 87e57632bf88b270f3f9a09f579372d3437aeb17 Sponsored by: NVidia networking Differential revision: https://reviews.freebsd.org/D48878 
Sponsored by:	NVidia networking
Differential revision:	https://reviews.freebsd.org/D48878
LinuxKPI: Remove owner argument from class_create function on KBI layer 2024-07-21T13:09:58+00:00 Vladimir Kondratyev wulf@FreeBSD.org 2024-07-21T13:09:58+00:00 1fe7cd02434257692100c484cd345566c9dba528 To chase Linux 6.4 Sponsored by: Serenity Cyber Security, LLC Differential Revision: https://reviews.freebsd.org/D45849 
To chase Linux 6.4

Sponsored by:   Serenity Cyber Security, LLC
Differential Revision:	https://reviews.freebsd.org/D45849
net: Remove unneeded NULL check for the allocated ifnet 2024-06-28T10:16:29+00:00 Zhenlei Huang zlei@FreeBSD.org 2024-06-28T10:16:29+00:00 aa3860851b9f6a6002d135b1cac7736e0995eedc Change 4787572d0580 made if_alloc_domain() never fail, then also do the wrappers if_alloc(), if_alloc_dev(), and if_gethandle(). No functional change intended. Reviewed by: kp, imp, glebius, stevek MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D45740 
Change 4787572d0580 made if_alloc_domain() never fail, then also do the
wrappers if_alloc(), if_alloc_dev(), and if_gethandle().

No functional change intended.

Reviewed by:	kp, imp, glebius, stevek
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D45740
ibcore: Mark write-only variables 2024-06-12T13:04:45+00:00 Andrew Turner andrew@FreeBSD.org 2024-06-12T13:04:45+00:00 19782e5bef3403a6ed65d46653f2e70f81dced37 Some LinuxKPI lock macros pass need a flags field passed in. This is written to but never read from so gcc complains. Fix this by marking the flags variables as unused to quieten the compiler. Reviewed by: brooks (earlier version), kib Sponsored by: Arm Ltd Differential Revision: https://reviews.freebsd.org/D45303 
Some LinuxKPI lock macros pass need a flags field passed in. This is
written to but never read from so gcc complains.

Fix this by marking the flags variables as unused to quieten the
compiler.

Reviewed by:	brooks (earlier version), kib
Sponsored by:	Arm Ltd
Differential Revision:	https://reviews.freebsd.org/D45303
ibcore: Remove the use of NULL_IB_OBJECT 2024-04-12T17:04:23+00:00 Ka Ho Ng khng@FreeBSD.org 2024-04-12T16:57:35+00:00 cb722b67796ddc700c73919183e80049c3ec17f3 LinuxKPI's XArray implementation accepts NULL as an input as of the following commit: - linuxkpi: Accept NULL as a value in linux_xarray (3102ea3b15b6) Sponsored by: Juniper Networks, Inc. MFC after: 1 week Reviewed by: kib Differential Revision: https://reviews.freebsd.org/D44533 
LinuxKPI's XArray implementation accepts NULL as an input as of the
following commit:
 - linuxkpi: Accept NULL as a value in linux_xarray (3102ea3b15b6)

Sponsored by:	Juniper Networks, Inc.
MFC after:	1 week
Reviewed by:	kib
Differential Revision:	https://reviews.freebsd.org/D44533
sockets: provide correct pr_shutdown for keysock and SDP 2024-01-16T20:02:59+00:00 Gleb Smirnoff glebius@FreeBSD.org 2024-01-16T20:00:36+00:00 296a4cb5c5b18f82da7a365d9f209cb9fc09003b My failure to run all kinds of kernel builds lead to missing the keysock and incorrectly assuming SDP as not having a shutdown method. Fixes: 5bba2728079ed4da33f727dbc2b6ae1de02ba897 
My failure to run all kinds of kernel builds lead to missing the keysock
and incorrectly assuming SDP as not having a shutdown method.

Fixes:	5bba2728079ed4da33f727dbc2b6ae1de02ba897
ofed: garbage collect now unused sdp_sockaddr() 2023-12-01T05:50:16+00:00 Gleb Smirnoff glebius@FreeBSD.org 2023-12-01T05:50:16+00:00 cd996b848944ec0d5302fea3fbe36f253470fbcc Submitted by: zlei 
Submitted by:	zlei
sockets: don't malloc/free sockaddr memory on getpeername/getsockname 2023-11-30T16:31:10+00:00 Gleb Smirnoff glebius@FreeBSD.org 2023-11-30T16:30:55+00:00 0fac350c54d0a72f5341e15021efcde63eb58a4b Just like it was done for accept(2) in cfb1e92912b4, use same approach for two simplier syscalls that return socket addresses. Although, these two syscalls aren't performance critical, this change generalizes some code between 3 syscalls trimming code size. Following example of accept(2), provide VNET-aware and INVARIANT-checking wrappers sopeeraddr() and sosockaddr() around protosw methods. Reviewed by: tuexen Differential Revision: https://reviews.freebsd.org/D42694 
Just like it was done for accept(2) in cfb1e92912b4, use same approach
for two simplier syscalls that return socket addresses.  Although,
these two syscalls aren't performance critical, this change generalizes
some code between 3 syscalls trimming code size.

Following example of accept(2), provide VNET-aware and INVARIANT-checking
wrappers sopeeraddr() and sosockaddr() around protosw methods.

Reviewed by:		tuexen
Differential Revision:	https://reviews.freebsd.org/D42694
sockets: don't malloc/free sockaddr memory on accept(2) 2023-11-30T16:30:55+00:00 Gleb Smirnoff glebius@FreeBSD.org 2023-11-30T16:30:55+00:00 cfb1e92912b4cf75360b7fbe86197cc29bc212c1 Let the accept functions provide stack memory for protocols to fill it in. Generic code should provide sockaddr_storage, specialized code may provide smaller structure. While rewriting accept(2) make 'addrlen' a true in/out parameter, reporting required length in case if provided length was insufficient. Our manual page accept(2) and POSIX don't explicitly require that, but one can read the text as they do. Linux also does that. Update tests accordingly. Reviewed by: rscheff, tuexen, zlei, dchagin Differential Revision: https://reviews.freebsd.org/D42635 
Let the accept functions provide stack memory for protocols to fill it in.
Generic code should provide sockaddr_storage, specialized code may provide
smaller structure.

While rewriting accept(2) make 'addrlen' a true in/out parameter, reporting
required length in case if provided length was insufficient.  Our manual
page accept(2) and POSIX don't explicitly require that, but one can read
the text as they do.  Linux also does that. Update tests accordingly.

Reviewed by:		rscheff, tuexen, zlei, dchagin
Differential Revision:	https://reviews.freebsd.org/D42635
sys: Automated cleanup of cdefs and other formatting 2023-11-27T05:24:00+00:00 Warner Losh imp@FreeBSD.org 2023-11-24T20:12:57+00:00 fdafd315ad0d0f28a11b9fb4476a9ab059c62b92 Apply the following automated changes to try to eliminate no-longer-needed sys/cdefs.h includes as well as now-empty blank lines in a row. Remove /^#if.*\n#endif.*\n#include\s+<sys/cdefs.h>.*\n/ Remove /\n+#include\s+<sys/cdefs.h>.*\n+#if.*\n#endif.*\n+/ Remove /\n+#if.*\n#endif.*\n+/ Remove /^#if.*\n#endif.*\n/ Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/types.h>/ Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/param.h>/ Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/capsicum.h>/ Sponsored by: Netflix 
Apply the following automated changes to try to eliminate
no-longer-needed sys/cdefs.h includes as well as now-empty
blank lines in a row.

Remove /^#if.*\n#endif.*\n#include\s+<sys/cdefs.h>.*\n/
Remove /\n+#include\s+<sys/cdefs.h>.*\n+#if.*\n#endif.*\n+/
Remove /\n+#if.*\n#endif.*\n+/
Remove /^#if.*\n#endif.*\n/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/types.h>/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/param.h>/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/capsicum.h>/

Sponsored by:		Netflix