Differential D36361

linuxolator: add netlink support
ClosedPublic
Actions

Authored by melifaro on Aug 26 2022, 2:59 PM.

Tags

None

Referenced Files

	F156936250: D36361.id110448.diff
	Sun, May 17, 11:52 AM

	F156868203: D36361.id.diff
	Sun, May 17, 12:02 AM

	F156866618: D36361.id110129.diff
	Sat, May 16, 11:51 PM

	F156855626: D36361.id109874.diff
	Sat, May 16, 10:36 PM

	F156827180: D36361.id111299.diff
	Sat, May 16, 5:56 PM

	Unknown Object (File)
	Sat, May 16, 1:56 AM

	Unknown Object (File)
	Fri, May 15, 5:52 AM

	Unknown Object (File)
	Wed, May 13, 8:18 PM

Subscribers

Details

Reviewers

Group Reviewers

Linux Emulation

Commits

rGc1c40c8627a0: linuxolator: add netlink support
rG7c40e2d5f685: linuxolator: add netlink support

Summary

Compat part for running Linux binaries with Netlink ( D36002 ).

Test Plan

Please see testing section in D36002.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 47138
Build 44025: arc lint + arc unit

Event Timeline

melifaro created this revision.Aug 26 2022, 2:59 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 26 2022, 2:59 PM

melifaro requested review of this revision.Aug 26 2022, 2:59 PM

Harbormaster completed remote builds in B47138: Diff 109874.Aug 26 2022, 2:59 PM

melifaro added a parent revision: D36002: netlink: add netlink support.Aug 26 2022, 3:00 PM

melifaro edited the summary of this revision. (Show Details)

melifaro edited the test plan for this revision. (Show Details)

melifaro added a reviewer: Linux Emulation.

emaste added a subscriber: emaste.Aug 28 2022, 11:40 PM

Update to recent -HEAD.

Harbormaster completed remote builds in B47241: Diff 110129.Sep 4 2022, 8:22 PM

mostly style, thank you! tested with 2.35 glibc tests suite, inet tests completed!

sys/compat/linux/linux_netlink.c
32	I suggest to add an empty line here
61	style, blank line, and in many functions below
121	unnecessary parentheses
131	the same here
167	may be use return (netlink_family != NETLINK_ROUTE ? hdr : rtnl_from_linux(hdr, npt)); ?
184	blank line
186	if (out_hdr != NULL) memcpy(out_hdr, hdr, hdr->nlmsg_len); return (out_hdr != NULL); ?
219	the same as above
235	unnecessary parentheses
386	unnecessary parentheses

dchagin accepted this revision.Sep 8 2022, 8:35 AM

This revision is now accepted and ready to land.Sep 8 2022, 8:35 AM

Update to reflect netlink KPI changes.

This revision now requires review to proceed.Sep 11 2022, 6:58 PM

Harbormaster completed remote builds in B47328: Diff 110448.Sep 11 2022, 6:58 PM

This revision was not accepted when it landed; it landed in state Needs Review.Oct 1 2022, 2:20 PM

Closed by commit rG7c40e2d5f685: linuxolator: add netlink support (authored by melifaro). · Explain Why

This revision was automatically updated to reflect the committed changes.

melifaro added a commit: rG7c40e2d5f685: linuxolator: add netlink support.

melifaro added a commit: rGc1c40c8627a0: linuxolator: add netlink support.Jan 26 2023, 3:42 PM

Revision Contents
Changeset List

Path

Size

sys/

compat/

linux/

3 lines

13 lines

3 lines

linux_netlink.c

567 lines

6 lines

modules/

linux_common/

2 lines

Diff 109874

sys/compat/linux/linux.h

Loading...

sys/compat/linux/linux.c

Loading...

sys/compat/linux/linux_common.c

Loading...

sys/compat/linux/linux_netlink.c

Loading...

sys/compat/linux/linux_socket.c

Loading...

sys/modules/linux_common/Makefile

Loading...