Differential D38715

linux(4): Fixup the interface name translation in netlink
ClosedPublic
Actions

Authored by dchagin on Feb 21 2023, 10:30 PM.

Tags

None

Referenced Files

	F157428734: D38715.id117764.diff
	Thu, May 21, 8:41 AM

	F157422399: D38715.id117778.diff
	Thu, May 21, 6:39 AM

	F157422394: D38715.id117778.diff
	Thu, May 21, 6:39 AM

	Unknown Object (File)
	Wed, May 20, 3:44 AM

	Unknown Object (File)
	Wed, May 20, 3:43 AM

	Unknown Object (File)
	Wed, May 20, 2:58 AM

	Unknown Object (File)
	Wed, May 20, 12:48 AM

	Unknown Object (File)
	Sun, May 17, 6:07 AM

Subscribers

Details

Reviewers

melifaro
jhibbits

Commits

rG9006cd0fd543: linux(4): Fixup the interface name translation in netlink
rG123ca9ab1189: linux(4): Fixup the interface name translation in netlink
rGe55e4a6ba3f4: linux(4): Fixup the interface name translation in netlink

Summary

Netlink should translate a FreeBSD interface name to a Linux interface name.

MFC after: 3 days

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dchagin created this revision.Feb 21 2023, 10:30 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 21 2023, 10:30 PM

dchagin requested review of this revision.Feb 21 2023, 10:30 PM

Harbormaster completed remote builds in B49937: Diff 117719.Feb 21 2023, 10:30 PM

dchagin added reviewers: melifaro, jhibbits.Feb 21 2023, 10:35 PM

melifaro added inline comments.Feb 22 2023, 8:35 AM

sys/compat/linux/linux_netlink.c
283	You need to check the message type as well, as this value is only valud for RTM_<NEW\|DEL\|GET>LINK. Also - for readability I’d consider moving attribute rewriting to a separate function. This will help keep copy-all_nla function readable when more translations are added

done

Harbormaster completed remote builds in B49946: Diff 117743.Feb 22 2023, 10:09 AM

Check nlmsg_type

Harbormaster completed remote builds in B49963: Diff 117764.Feb 22 2023, 2:40 PM

LGTM

sys/compat/linux/linux_netlink.c
241	Nit: worth describing the return codes here.
252	I'd explicitly cast it to int, or make it a branch, to help reader understand the return codes.
283–287

This revision is now accepted and ready to land.Feb 22 2023, 2:52 PM

done

This revision now requires review to proceed.Feb 22 2023, 6:21 PM

Harbormaster completed remote builds in B49967: Diff 117778.Feb 22 2023, 6:21 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 23 2023, 8:01 AM

Closed by commit rGe55e4a6ba3f4: linux(4): Fixup the interface name translation in netlink (authored by dchagin). · Explain Why

This revision was automatically updated to reflect the committed changes.

dchagin added a commit: rGe55e4a6ba3f4: linux(4): Fixup the interface name translation in netlink.

dchagin added a commit: rG123ca9ab1189: linux(4): Fixup the interface name translation in netlink.Feb 26 2023, 8:13 AM

dchagin added a commit: rG9006cd0fd543: linux(4): Fixup the interface name translation in netlink.Feb 27 2023, 6:17 PM

Revision Contents
Changeset List

Path

Size

sys/

compat/

linux/

linux_netlink.c

48 lines

Diff 117804

sys/compat/linux/linux_netlink.c

Loading...