sendfile: remove COMPAT_FREEBSD4 from kern_sendfile
Needs ReviewPublic
Actions

Authored by brooks on Wed, Aug 26, 12:34 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Aug 26, 7:03 PM

	Unknown Object (File)
	Wed, Aug 26, 3:57 PM

	Unknown Object (File)
	Wed, Aug 26, 3:55 PM

	Unknown Object (File)
	Wed, Aug 26, 3:50 PM

	Unknown Object (File)
	Wed, Aug 26, 2:14 PM

	Unknown Object (File)
	Wed, Aug 26, 2:06 PM

	Unknown Object (File)
	Wed, Aug 26, 1:46 PM

	Unknown Object (File)
	Wed, Aug 26, 1:40 PM

Subscribers

glebius

imp

olce

Details

Reviewers

markj

Summary

Move compat bits into a wrapper around the copyin of the header's
iovec.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 76188
Build 73071: arc lint + arc unit

Event Timeline

brooks created this revision.Wed, Aug 26, 12:34 PM

Herald added subscribers: olce, glebius, imp. · View Herald TranscriptWed, Aug 26, 12:34 PM

brooks requested review of this revision.Wed, Aug 26, 12:34 PM

Harbormaster completed remote builds in B76188: Diff 185151.Wed, Aug 26, 12:34 PM

brooks mentioned this in D59034: sendfile(2): rework compat implementation.Wed, Aug 26, 12:39 PM

This is a bit more complex than I was hoping. I do wonder if we really need all of these separate callbacks, vs. one callback for the initial copyin plus the header and trailer copyins. That would require a bit more code in the callbacks but seems simpler overall.

Anyway, it's a minor point. I'm okay with the original change as it is.