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

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

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

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

head/

sys/

dev/

hyperv/

vmbus/

hv_vmbus_drv_freebsd.c

12 lines

Diff 14960

View Options

head/sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c

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