From: Roman Kagan <rkagan@virtuozzo.com>
To: Vitaly Kuznetsov <vkuznets@redhat.com>
Cc: kvm@vger.kernel.org, "Paolo Bonzini" <pbonzini@redhat.com>,
"Radim Krčmář" <rkrcmar@redhat.com>,
"Denis V. Lunev" <den@openvz.org>
Subject: Re: [PATCH 2/2] x86:kvm:hyperv: guest->host event signaling via eventfd
Date: Wed, 6 Dec 2017 20:32:07 +0300 [thread overview]
Message-ID: <20171206173207.GI2265@rkaganb.sw.ru> (raw)
In-Reply-To: <87d13rlr4s.fsf@vitty.brq.redhat.com>
On Wed, Dec 06, 2017 at 06:09:55PM +0100, Vitaly Kuznetsov wrote:
> Roman Kagan <rkagan@virtuozzo.com> writes:
> > On Wed, Dec 06, 2017 at 04:19:22PM +0100, Vitaly Kuznetsov wrote:
> >> Or,
> >> alternatively, we can probably add both VMBUS_MESSAGE_CONNECTION_ID and
> >> VMBUS_MONITOR_CONNECTION_ID to the mechanism...
> >
> > These two are not used with the SIGNAL_EVENT hypercall. Or are you
> > suggesting to also handle the POST_MESSAGE hypercall in KVM? I don't
> > see a compelling reason to do so, since this is a slow control mechanism
> > and only used at setup/teardown, so handling it in userspace is good
> > enough.
>
> Yes, it is good enough but the new mechanism's name look generic enough:
> KVM_HYPERV_EVENTFD and it is unclear why only SIGNAL_EVENT is handled.
Because SIGNAL_EVENT matches the eventfd semantics while POST_MESSAGE
doesn't.
Because POST_MESSAGE is, well, about posting messages. It bears up to
256 bytes of data which need to be copyied aside before returning to the
guest and then delivered somehow to the userspace for processing.
Roman.
next prev parent reply other threads:[~2017-12-06 17:32 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-04 19:00 [PATCH 0/2] x86:kvm:hyperv: guest->host event signaling via eventfd Roman Kagan
2017-12-04 19:00 ` [PATCH 1/2] x86:kvm:hyperv: factor out kvm.arch.hyperv (de)init Roman Kagan
2017-12-04 19:00 ` [PATCH 2/2] x86:kvm:hyperv: guest->host event signaling via eventfd Roman Kagan
2017-12-06 15:19 ` Vitaly Kuznetsov
2017-12-06 17:00 ` Roman Kagan
2017-12-06 17:09 ` Vitaly Kuznetsov
2017-12-06 17:32 ` Roman Kagan [this message]
2017-12-06 15:28 ` Konrad Rzeszutek Wilk
2017-12-06 16:37 ` Roman Kagan
2017-12-07 4:31 ` kbuild test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171206173207.GI2265@rkaganb.sw.ru \
--to=rkagan@virtuozzo.com \
--cc=den@openvz.org \
--cc=kvm@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=rkrcmar@redhat.com \
--cc=vkuznets@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox