mountd: Convert mountlist to SLIST
ClosedPublic
Actions

Authored by manu on Sep 26 2017, 8:42 AM.

Tags

None

Referenced Files

	F152646682: D12504.diff
	Thu, Apr 16, 6:16 AM

	F152638524: D12504.id.diff
	Thu, Apr 16, 4:50 AM

	F152548537: D12504.diff
	Wed, Apr 15, 3:02 PM

	Unknown Object (File)
	Wed, Apr 15, 5:58 AM

	Unknown Object (File)
	Mon, Apr 6, 9:48 PM

	Unknown Object (File)
	Mon, Apr 6, 3:59 AM

	Unknown Object (File)
	Sun, Apr 5, 4:03 PM

	Unknown Object (File)
	Fri, Apr 3, 11:50 AM

View All Files

Subscribers

imp

Details

Reviewers

bapt
rmacklem

Commits

rS324258: mountd: Convert mountlist to SLIST

Summary

Use SLIST from sys/queue.h instead of homebrew linked list for mountlist.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

manu created this revision.Sep 26 2017, 8:42 AM

Herald added a subscriber: imp. · View Herald TranscriptSep 26 2017, 8:42 AM

LGTM

usr.sbin/mountd/mountd.c
229 ↗	(On Diff #33437)	static SLIST...

This revision is now accepted and ready to land.Sep 26 2017, 8:59 AM

Make the SLIST static.

This revision now requires review to proceed.Sep 26 2017, 9:02 AM

bapt accepted this revision.Sep 26 2017, 9:03 AM

This revision is now accepted and ready to land.Sep 26 2017, 9:03 AM

Looks ok to me. I think the only effect is a change in the order of entries in the list
and I don't believe that matters.

Closed by commit rS324258: mountd: Convert mountlist to SLIST (authored by manu). · Explain WhyOct 4 2017, 8:48 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

mountd/

mountd.c

49 lines

Diff 33677

View Options

head/usr.sbin/mountd/mountd.c

mountd: Convert mountlist to SLISTClosedPublicActions