Differential D5852

hyperv: Use mb() instead of atomic_thread_fence_seq_cst()
ClosedPublic
Actions

Authored by sepherosa_gmail.com on Apr 6 2016, 6:39 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Sep 21, 10:35 AM

	Unknown Object (File)
	Mon, Sep 21, 6:36 AM

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

	Unknown Object (File)
	Wed, Sep 9, 10:47 PM

	Unknown Object (File)
	Tue, Sep 8, 10:06 AM

	Unknown Object (File)
	Sep 6 2026, 3:03 AM

	Unknown Object (File)
	Sep 5 2026, 10:58 PM

	Unknown Object (File)
	Sep 5 2026, 3:33 PM

Subscribers

None

Details

Reviewers

kib
howard0su_gmail.com
honzhan_microsoft.com
decui_microsoft.com
jhb
royger

Commits

rS297634: hyperv: Use mb() instead of atomic_thread_fence_seq_cst()

Summary

Since atomic_thread_fence_seq_cst() will become compiler fence on UP kernel.

Noticed by: Dexuan Cui <decui microsoft com>

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

sepherosa_gmail.com updated this revision to Diff 14914.Apr 6 2016, 6:39 AM

sepherosa_gmail.com retitled this revision from to hyperv: Use mb() instead of atomic_thread_fence_seq_cst().

sepherosa_gmail.com updated this object.

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

sepherosa_gmail.com added reviewers: jhb, kib, royger, decui_microsoft.com, honzhan_microsoft.com, howard0su_gmail.com.

decui_microsoft.com accepted this revision.Apr 6 2016, 6:59 AM

decui_microsoft.com edited edge metadata.

This revision is now accepted and ready to land.Apr 6 2016, 6:59 AM

kib accepted this revision.Apr 6 2016, 9:33 AM

kib edited edge metadata.

Closed by commit rS297634: hyperv: Use mb() instead of atomic_thread_fence_seq_cst() (authored by sephe). · Explain WhyApr 7 2016, 5:31 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

dev/

hyperv/

vmbus/

hv_vmbus_drv_freebsd.c

12 lines

Diff 14914

sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c

Loading...