sendfile: remove COMPAT_FREEBSD4 from kern_sendfile
Needs ReviewPublic
Actions

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

Tags

None

Referenced Files

	F172317569: D59198.diff
	Thu, Sep 17, 4:57 PM

	F172279146: D59198.id.diff
	Thu, Sep 17, 10:46 AM

	Unknown Object (File)
	Mon, Sep 14, 12:27 AM

	Unknown Object (File)
	Fri, Sep 11, 8:49 PM

	Unknown Object (File)
	Thu, Sep 10, 2:51 PM

	Unknown Object (File)
	Tue, Sep 8, 3:30 PM

	Unknown Object (File)
	Sun, Sep 6, 8:48 PM

	Unknown Object (File)
	Sun, Sep 6, 7:56 PM

Subscribers

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.