* [PATCH 0/4] Enable MSI support for KVM VT-d @ 2008-09-26 5:17 Sheng Yang 2008-10-19 13:16 ` Avi Kivity 0 siblings, 1 reply; 4+ messages in thread From: Sheng Yang @ 2008-09-26 5:17 UTC (permalink / raw) To: Avi Kivity; +Cc: kvm, Weidong", Allen M" Hi, Avi This patchset enable MSI support for KVM VT-d. And here are only kernel space ones. The third patch would go to also goto x86 upstream. The userspace code would looks like this: assigned_irq_data.guest_msi_addr = *(uint32_t *)(d->msi_cap + 4); assigned_irq_data.guest_msi_data = *(uint16_t *)(d->msi_cap + 8); assigned_irq_data.flags |= KVM_DEV_IRQ_ASSIGN_ENABLE_MSI; r = kvm_assign_irq(kvm_context, &assigned_irq_data); I've test the patchset with some userspace hack, it works well. Thanks! -- regards Yang, Sheng ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 0/4] Enable MSI support for KVM VT-d 2008-09-26 5:17 [PATCH 0/4] Enable MSI support for KVM VT-d Sheng Yang @ 2008-10-19 13:16 ` Avi Kivity 2008-10-20 4:09 ` Zhang, Xiantao 0 siblings, 1 reply; 4+ messages in thread From: Avi Kivity @ 2008-10-19 13:16 UTC (permalink / raw) To: Sheng Yang; +Cc: kvm, weidong.han, allen.m.kay Sheng Yang wrote: > Hi, Avi > > This patchset enable MSI support for KVM VT-d. > > And here are only kernel space ones. The third patch would go to also goto x86 > upstream. > > The userspace code would looks like this: > > assigned_irq_data.guest_msi_addr = *(uint32_t *)(d->msi_cap + 4); > assigned_irq_data.guest_msi_data = *(uint16_t *)(d->msi_cap + 8); > assigned_irq_data.flags |= KVM_DEV_IRQ_ASSIGN_ENABLE_MSI; > r = kvm_assign_irq(kvm_context, &assigned_irq_data); > > I've test the patchset with some userspace hack, it works well. > > Can you resend this patch with all the updates, as well as the userspace changes? -- error compiling committee.c: too many arguments to function ^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH 0/4] Enable MSI support for KVM VT-d 2008-10-19 13:16 ` Avi Kivity @ 2008-10-20 4:09 ` Zhang, Xiantao 2008-10-20 5:39 ` Yang, Sheng 0 siblings, 1 reply; 4+ messages in thread From: Zhang, Xiantao @ 2008-10-20 4:09 UTC (permalink / raw) To: Avi Kivity, Yang, Sheng; +Cc: kvm, Han, Weidong, Kay, Allen M Avi Kivity wrote: > Sheng Yang wrote: >> Hi, Avi >> >> This patchset enable MSI support for KVM VT-d. >> >> And here are only kernel space ones. The third patch would go to >> also goto x86 upstream. >> >> The userspace code would looks like this: >> >> assigned_irq_data.guest_msi_addr = *(uint32_t *)(d->msi_cap + 4); >> assigned_irq_data.guest_msi_data = *(uint16_t *)(d->msi_cap + 8); >> assigned_irq_data.flags |= KVM_DEV_IRQ_ASSIGN_ENABLE_MSI; >> r = kvm_assign_irq(kvm_context, &assigned_irq_data); >> >> I've test the patchset with some userspace hack, it works well. >> >> > > Can you resend this patch with all the updates, as well as the > userspace changes? Maybe Sheng need to make it work on kvm/ia64, and at least the changes can't break ia64 side. Xiantao > -- > error compiling committee.c: too many arguments to function ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 0/4] Enable MSI support for KVM VT-d 2008-10-20 4:09 ` Zhang, Xiantao @ 2008-10-20 5:39 ` Yang, Sheng 0 siblings, 0 replies; 4+ messages in thread From: Yang, Sheng @ 2008-10-20 5:39 UTC (permalink / raw) To: Zhang, Xiantao Cc: Avi Kivity, kvm@vger.kernel.org, Han, Weidong, Kay, Allen M On Monday 20 October 2008 12:09:20 Zhang, Xiantao wrote: > Avi Kivity wrote: > > Sheng Yang wrote: > >> Hi, Avi > >> > >> This patchset enable MSI support for KVM VT-d. > >> > >> And here are only kernel space ones. The third patch would go to > >> also goto x86 upstream. > >> > >> The userspace code would looks like this: > >> > >> assigned_irq_data.guest_msi_addr = *(uint32_t *)(d->msi_cap + 4); > >> assigned_irq_data.guest_msi_data = *(uint16_t *)(d->msi_cap + 8); > >> assigned_irq_data.flags |= KVM_DEV_IRQ_ASSIGN_ENABLE_MSI; > >> r = kvm_assign_irq(kvm_context, &assigned_irq_data); > >> > >> I've test the patchset with some userspace hack, it works well. > > > > Can you resend this patch with all the updates, as well as the > > userspace changes? > > Maybe Sheng need to make it work on kvm/ia64, and at least the changes > can't break ia64 side. Xiantao > Yes, I would ensure that won't break ia64. I would repost the patchset soon. But Avi, for the userspace, the Amit's patch still not checked in, so I haven't written a complete version because of lacking code base. I only got a experiment patch by hand, expose MSI cap to guest and enable MSI when guest wrote MSI enable bit. Well, I prefer to give you a complete version after Amit's patch is there. (I will conclude the change of userspace in the first mail). -- regards Yang, Sheng ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-10-20 5:41 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2008-09-26 5:17 [PATCH 0/4] Enable MSI support for KVM VT-d Sheng Yang 2008-10-19 13:16 ` Avi Kivity 2008-10-20 4:09 ` Zhang, Xiantao 2008-10-20 5:39 ` Yang, Sheng
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.