Switch from FD_SETSIZE to getdtablesize().
ClosedPublic
Actions

Authored by araujo on Jan 18 2016, 2:20 AM.

Tags

None

Referenced Files

	F84023922: D4970.id.diff
	Sat, May 18, 6:43 AM

	Unknown Object (File)
	Sun, May 5, 10:30 AM

	Unknown Object (File)
	Sat, May 4, 10:53 PM

	Unknown Object (File)
	Apr 16 2024, 11:20 PM

	Unknown Object (File)
	Mar 22 2024, 5:27 PM

	Unknown Object (File)
	Mar 12 2024, 7:27 PM

	Unknown Object (File)
	Jan 5 2024, 12:26 PM

	Unknown Object (File)
	Dec 25 2023, 8:46 AM

View All 45 Files

Subscribers

imp

• rodrigc

Details

Reviewers

• rodrigc
bapt

Commits

rS294543: Switch from FD_SETSIZE to getdtablesize(2) as it can make the FD to be

Summary

@rodrigc requested to switch to getdtablesize() as it will be more close with
what was implemented on OpenBSD. I have no objection with it as it has a good
side and the FD can be tunable.

However, it will be slower than FD_SETSIZE as it needs to calloc more than 1024.
Not a big deal to be honest.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

araujo updated this revision to Diff 12396.Jan 18 2016, 2:20 AM

araujo retitled this revision from to Switch from FD_SETSIZE to getdtablesize()..

araujo updated this object.

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

araujo added reviewers: • rodrigc, bapt.

araujo added a subscriber: • rodrigc.

Herald added a subscriber: imp. · View Herald TranscriptJan 18 2016, 2:20 AM

• rodrigc accepted this revision.Jan 22 2016, 2:33 AM

• rodrigc edited edge metadata.

This revision is now accepted and ready to land.Jan 22 2016, 2:33 AM

Closed by commit rS294543: Switch from FD_SETSIZE to getdtablesize(2) as it can make the FD to be (authored by araujo). · Explain WhyJan 22 2016, 3:02 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

ypldap/

yp.c

15 lines

Diff 12583

View Options

Switch from FD_SETSIZE to getdtablesize().ClosedPublicActions