public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Marcelo Tosatti <mtosatti@redhat.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: kvm-devel@lists.sourceforge.net, Avi Kivity <avi@qumranet.com>
Subject: Re: [PATCH][RFC] Use pipe() to simulate signalfd()
Date: Tue, 29 Apr 2008 21:38:01 -0300	[thread overview]
Message-ID: <20080430003801.GB18682@dmt> (raw)
In-Reply-To: <4817BBDD.4020505@us.ibm.com>

On Tue, Apr 29, 2008 at 07:22:53PM -0500, Anthony Liguori wrote:
> Marcelo Tosatti wrote:
> >True.
> >
> >Either way, this starvation due to signals can't happen with the current
> >scheme because signals are blocked. It seems a significant drawback.
> >  
> 
> Practically speaking, I don't see signal starving being a problem.  Part 
> of the benefit of this approach is that we're reducing the overall 
> number of signals received.  With the in-kernel PIT, the number of 
> userspace signals is even further reduced.
> 
> >Moving the signal handling + pipe write to a separate thread should get
> >rid of it.
> >  
> 
> Yeah, but then you just introduce buffering problems since if you're 
> getting that many signals, the pipe will get full.

It is OK to lose signals if you have at least one queued in the pipe.

> No point in designing for something that isn't likely to happen in practice.

You should not design something making the assumption that this scenario
won't happen.

For example this could happen in high throughput guests using POSIX AIO, 
actually pretty likely to happen if data is cached in hosts pagecache.

Its somewhat similar to what happens with NAPI and interrupt mitigation.


-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone

  reply	other threads:[~2008-04-30  0:38 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-29 14:28 [PATCH][RFC] Use pipe() to simulate signalfd() Anthony Liguori
2008-04-29 22:37 ` Marcelo Tosatti
2008-04-29 22:42   ` Anthony Liguori
2008-04-29 23:13     ` Marcelo Tosatti
2008-04-29 23:15       ` Anthony Liguori
2008-04-29 23:37         ` Marcelo Tosatti
2008-04-29 23:44           ` Anthony Liguori
2008-04-30  0:08             ` Marcelo Tosatti
2008-04-30  0:22               ` Anthony Liguori
2008-04-30  0:38                 ` Marcelo Tosatti [this message]
2008-04-30  0:47                   ` Anthony Liguori
2008-04-30  2:16                     ` Marcelo Tosatti

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=20080430003801.GB18682@dmt \
    --to=mtosatti@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=avi@qumranet.com \
    --cc=kvm-devel@lists.sourceforge.net \
    /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