ifnet(9): split ifc_alloc_unit() (should make code a bit cleaner).
ClosedPublic
Actions

Authored by avos on Sep 30 2017, 9:27 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, May 13, 3:46 PM

	Unknown Object (File)
	Fri, May 9, 2:36 PM

	Unknown Object (File)
	Mon, Apr 21, 2:55 AM

	Unknown Object (File)
	Apr 17 2025, 9:42 AM

	Unknown Object (File)
	Apr 17 2025, 7:43 AM

	Unknown Object (File)
	Apr 17 2025, 4:37 AM

	Unknown Object (File)
	Apr 14 2025, 4:35 AM

	Unknown Object (File)
	Apr 11 2025, 8:25 PM

View All 67 Files

Subscribers

imp

Details

Reviewers

glebius

Group Reviewers

network

Commits

rS324813: MFC r324672:
rS324812: MFC r324672:
rS324672: ifnet(9): split ifc_alloc_unit() (should simplify code flow)

Summary

Allocate smallest unit number from pool from ifc_alloc_unit_next() and exact unit number (if available) from ifc_alloc_unit_specific().

While here, fix potential endless loop (*unit < 0 -> alloc_unr() succeeds but ifunit() fails -> (*unit)++ and then alloc_unr_specific() fails ->(*unit) is overridden with return result (-1) and then used instead of previous unit number).

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

avos created this revision.Sep 30 2017, 9:27 PM

Herald added subscribers: ae, imp. · View Herald TranscriptSep 30 2017, 9:27 PM

Does this fix this bug? https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217401

Yes (but without reassigning *unit - it will not change after alloc_unr_specific() anyway).

Can you please grab the PR? :)

Closed by commit rS324672: ifnet(9): split ifc_alloc_unit() (should simplify code flow) (authored by avos). · Explain WhyOct 16 2017, 9:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

net/

if_clone.c

60 lines

Diff 34037

View Options

ifnet(9): split ifc_alloc_unit() (should make code a bit cleaner).ClosedPublicActions