Differential D6573

hyperv/et: Fix STIMER0 operations.
ClosedPublic
Actions

Authored by sepherosa_gmail.com on May 26 2016, 7:19 AM.

Tags

None

Referenced Files

	F167132767: D6573.id17098.diff
	Wed, Aug 19, 9:02 AM

	F167132730: D6573.id16897.diff
	Wed, Aug 19, 9:02 AM

	F167132719: D6573.id16963.diff
	Wed, Aug 19, 9:02 AM

	F167132120: D6573.diff
	Wed, Aug 19, 8:58 AM

	Unknown Object (File)
	Sun, Aug 9, 6:53 AM

	Unknown Object (File)
	Sun, Aug 9, 3:59 AM

	Unknown Object (File)
	Sun, Aug 9, 2:55 AM

	Unknown Object (File)
	Sun, Aug 9, 2:10 AM

Subscribers

None

Details

Reviewers

howard0su_gmail.com
honzhan_microsoft.com
decui_microsoft.com
delphij
royger

Commits

rS300987: hyperv/et: Fix STIMER0 operations.

Summary

Make sure that STIMER0 is disabled before writting to it, since writing to an enabled STIMER will result in undefined behaviour.
It is unnecessary to reconfigure STIMER0 upon each et_start().
Make sure that MSR_HV_REF_TIME_COUNT will not return 0, since writing 0 to STIMER_COUNT will disable the target STIMER.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

sepherosa_gmail.com updated this revision to Diff 16897.May 26 2016, 7:19 AM

sepherosa_gmail.com retitled this revision from to hyperv/et: Fix STIMER0 operations..

sepherosa_gmail.com updated this object.

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

sepherosa_gmail.com added reviewers: delphij, royger, decui_microsoft.com, honzhan_microsoft.com, howard0su_gmail.com.

sepherosa_gmail.com updated this revision to Diff 16963.May 27 2016, 2:50 AM

Closed by commit rS300987: hyperv/et: Fix STIMER0 operations. (authored by sephe). · Explain WhyMay 30 2016, 8:25 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

dev/

hyperv/

vmbus/

53 lines

Diff 16963

sys/dev/hyperv/vmbus/hv_et.c

Loading...