Only clear TDP_NERRNO when needed
ClosedPublic
Actions

Authored by trasz on Sep 30 2020, 7:07 PM.

Tags

None

Referenced Files

	F169106406: D26612.id77695.diff
	Mon, Aug 31, 10:37 AM

	F169083863: D26612.id77746.diff
	Mon, Aug 31, 9:05 AM

	F169078400: D26612.id.diff
	Mon, Aug 31, 8:44 AM

	F169054911: D26612.diff
	Mon, Aug 31, 6:47 AM

	Unknown Object (File)
	Sat, Aug 29, 2:04 AM

	Unknown Object (File)
	Sun, Aug 23, 1:16 AM

	Unknown Object (File)
	Sat, Aug 22, 11:01 AM

	Unknown Object (File)
	Tue, Aug 18, 3:58 PM

View All Files

Subscribers

imp

mjg

Details

Reviewers

kib

Commits

rS366336: Only clear TDP_NERRNO when needed, ie when it's previously been set.

Summary

Only clear TDP_NERRNO when needed, ie when it's previously been set.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

trasz created this revision.Sep 30 2020, 7:07 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 30 2020, 7:07 PM

trasz requested review of this revision.Sep 30 2020, 7:07 PM

Harbormaster completed remote builds in B33904: Diff 77695.Sep 30 2020, 7:07 PM

kib accepted this revision.Sep 30 2020, 7:13 PM

This revision is now accepted and ready to land.Sep 30 2020, 7:13 PM

this adds a branch which likely pessimizes the code

In D26612#592862, @mjg wrote:

this adds a branch which likely pessimizes the code

The branch is already there, you had to check td_pflags anyway. Not to mention that previously the branch was statically mispredicted (the common case is for (td_pflags & TDP_NERRNO) to be 0.

Closed by commit rS366336: Only clear TDP_NERRNO when needed, ie when it's previously been set. (authored by trasz). · Explain WhyOct 1 2020, 6:45 PM

This revision was automatically updated to reflect the committed changes.

trasz added a commit: rS366336: Only clear TDP_NERRNO when needed, ie when it's previously been set..

Revision Contents
Changeset List

Path

Size

head/

sys/

kern/

subr_syscall.c

11 lines

Diff 77746

View Options

head/sys/kern/subr_syscall.c

Only clear TDP_NERRNO when neededClosedPublicActions