Page MenuHomeFreeBSD
Differential D6636

hyperv/vmbus: Redefine SynIC message.
ClosedPublic
Actions

Authored by sepherosa_gmail.com on May 30 2016, 1:39 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 25, 12:56 AM
Unknown Object (File)
Thu, May 23, 2:45 PM
Unknown Object (File)
Apr 24 2024, 9:25 PM
Unknown Object (File)
Feb 25 2024, 12:28 AM
Unknown Object (File)
Feb 22 2024, 2:03 PM
Unknown Object (File)
Dec 20 2023, 1:24 AM
Unknown Object (File)
Nov 16 2023, 5:33 AM
Unknown Object (File)
Nov 14 2023, 5:42 AM
View All 45 Files
Subscribers
None

Details

Reviewers
howard0su_gmail.com
honzhan_microsoft.com
decui_microsoft.com
delphij
royger
Commits
rS301019: hyperv/vmbus: Redefine SynIC message.
Summary
  • Avoid unnecessary indirection.
  • Avoid bit fields.
  • Use __packed.

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 17085.May 30 2016, 1:39 AM
sepherosa_gmail.com retitled this revision from to hyperv/vmbus: Redefine SynIC message..
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.
howard0su_gmail.com added inline comments.May 30 2016, 12:34 PM
sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c
131 ↗(On Diff #17085)

volatile is not needed here since msg pointer will not change out of this context.

sepherosa_gmail.com added inline comments.May 30 2016, 12:53 PM
sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c
131 ↗(On Diff #17085)

It will be updated by the host, so we need to make sure that compiler don't over-optimize the msg accessing.

Closed by commit rS301019: hyperv/vmbus: Redefine SynIC message. (authored by sephe). · Explain WhyMay 31 2016, 5:19 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
hyperv/
vmbus/
45 lines
35 lines
56 lines

Diff 17133

View Options

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

Loading...
View Options

head/sys/dev/hyperv/vmbus/hv_vmbus_priv.h

Loading...
View Options

head/sys/dev/hyperv/vmbus/vmbus_reg.h

Loading...