hyperv: Use lapic_{alloc,free}_ipi to allocate private interrupt vector
ClosedPublic
Actions

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

Details

Reviewers

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

Commits

rS297641: hyperv: Use lapic_{alloc,free}_ipi to allocate private interrupt vector

Summary

Suggested by: jhb

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

sepherosa_gmail.com updated this revision to Diff 14912.Apr 6 2016, 5:18 AM

sepherosa_gmail.com retitled this revision from to hyperv: Use lapic_{alloc,free}_ipi to allocate private interrupt vector.

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, 5:24 AM

decui_microsoft.com edited edge metadata.

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

howard0su_gmail.com added inline comments.Apr 6 2016, 8:33 AM

sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c
385	We don't need this macro sine this is needed anyhow.
386	Suggest to move hv_vmbus_callback code from apic_local.S to a local .S file inside vmbus driver.

sepherosa_gmail.com added inline comments.Apr 6 2016, 8:47 AM

sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c
386	I'd do this and above as a separate commit

Closed by commit rS297641: hyperv: Use lapic_{alloc,free}_ipi to allocate private interrupt vector (authored by sephe). · Explain WhyApr 7 2016, 7:13 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

86 lines

hv_vmbus_priv.h

4 lines

Diff 14912

View Options

sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c