All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory Haskins <ghaskins@novell.com>
To: Avi Kivity <avi@redhat.com>
Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
	davidel@xmailserver.org
Subject: Re: [KVM PATCH v2 2/2] kvm: add support for irqfd via	eventfd-notification interface
Date: Tue, 28 Apr 2009 07:04:27 -0400	[thread overview]
Message-ID: <49F6E2BB.9070604@novell.com> (raw)
In-Reply-To: <49F6E1CA.1010106@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 1052 bytes --]

Avi Kivity wrote:
> Gregory Haskins wrote:
>
>  
>
>>> But eventfd_signal basically marries us to eventfd.
>>>     
>>
>> Well, only if we expect the fd to have eventfd semantics.  There are
>> advantages to doing so, as we have discussed, because things like AIO
>> can polymorhpically signal an interrupt without even knowing whats
>> behind the eventfd.  But this isn't a strict requirement to support
>> AIO.  Really all we need is a way for both kernel and userspace to
>> signal.  Perhaps I should export an "irqfd_signal()" function from kvm,
>> which today will map to eventfd_signal(), and tomorrow to ??.  I don't
>> think using f_ops->write() is an option for in-kernel signaling, so we
>> need some kind of interface here.
>>
>> Does that sound reasonable?
>>   
>
> irqfd_signal() ties the user of irqfd to kvm.  I want this user to be
> independent of kvm; it should work with eventfd, kvm's eventfd
> lookalike (if we move away from eventfd) or pipes.

So what is your proposal for such interface?

-Greg



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 266 bytes --]

  reply	other threads:[~2009-04-28 11:05 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-24  4:25 [KVM PATCH v2 0/2] irqfd Gregory Haskins
2009-04-24  4:25 ` [KVM PATCH v2 1/2] eventfd: export fget and signal interfaces for module use Gregory Haskins
2009-04-24  4:25 ` [KVM PATCH v2 2/2] kvm: add support for irqfd via eventfd-notification interface Gregory Haskins
2009-04-24 17:07   ` Gregory Haskins
2009-04-24 17:47     ` Davide Libenzi
2009-04-27  8:55   ` Avi Kivity
2009-04-27 10:35     ` Gregory Haskins
2009-04-27 10:48       ` Avi Kivity
2009-04-27 13:27         ` Gregory Haskins
2009-04-28  9:35           ` Avi Kivity
2009-04-28 10:34             ` Gregory Haskins
2009-04-28 11:00               ` Avi Kivity
2009-04-28 11:04                 ` Gregory Haskins [this message]
2009-04-28 11:05                   ` Avi Kivity
2009-04-28 11:08                     ` Avi Kivity
2009-04-28 11:38                       ` Gregory Haskins
2009-04-28 11:48                         ` Avi Kivity
2009-04-28 12:07                           ` Gregory Haskins
2009-04-27 10:58       ` Gregory Haskins
2009-04-27 11:23         ` Avi Kivity

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=49F6E2BB.9070604@novell.com \
    --to=ghaskins@novell.com \
    --cc=avi@redhat.com \
    --cc=davidel@xmailserver.org \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /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 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.