Switch from FD_SETSIZE to getdtablesize().
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Nov 27, 4:48 AM

	Unknown Object (File)
	Fri, Nov 22, 12:44 PM

	Unknown Object (File)
	Nov 17 2024, 10:08 AM

	Unknown Object (File)
	Nov 17 2024, 3:01 AM

	Unknown Object (File)
	Nov 16 2024, 3:45 PM

	Unknown Object (File)
	Nov 12 2024, 12:45 PM

	Unknown Object (File)
	Nov 12 2024, 5:31 AM

	Unknown Object (File)
	Nov 5 2024, 5:19 PM

View All 78 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