All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Chris Wright <chrisw@redhat.com>,
	kvm@vger.kernel.org, qemu-devel@nongnu.org
Subject: Re: KVM call agenda for Apr 27
Date: Tue, 27 Apr 2010 11:14:59 +0300	[thread overview]
Message-ID: <4BD69D03.2050502@redhat.com> (raw)
In-Reply-To: <4BD61584.9080208@codemonkey.ws>

On 04/27/2010 01:36 AM, Anthony Liguori wrote:
>
> A few comments:
>
> 1) The problem was not block watermark itself but generating a 
> notification on the watermark threshold.  It's a heuristic and should 
> be implemented based on polling block stats. 

Polling for an event that never happens is bad engineering.  What 
frequency do you poll?  you're forcing the user to make a lose-lose 
tradeoff.

> Otherwise, we'll be adding tons of events to qemu that we'll struggle 
> to maintain.

That's not a valid reason to reject a user requirement.  We may argue 
the requirement is bogus, or that the suggested implementation is wrong 
and point in a different direction, but saying that we may have to add 
more code in the future due to other requirements is ... well I can't 
find a word for it.

>
> 2) A block plugin doesn't solve the problem if it's just at the 
> BlockDriverState level because it can't interact with qcow2.

Why not?  We have a layered model.  guest -> qcow2 -> plugin (sends 
event) -> raw-posix.  Just need to insert the plugin at the appropriate 
layer.

>
> 3) For general block plugins, it's probably better to tackle userspace 
> block devices.  We have CUSE and FUSE already, a BUSE is a logical 
> conclusion.

We also have an nbd client.

Here's another option:  an nbd-like protocol that remotes all 
BlockDriver operations except read and write over a unix domain socket.  
The open operation returns an fd (SCM_RIGHTS strikes again) that is used 
for read and write.  This can be used to implement snapshots over LVM, 
for example.

-- 
Do not meddle in the internals of kernels, for they are subtle and quick to panic.


WARNING: multiple messages have this Message-ID (diff)
From: Avi Kivity <avi@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Chris Wright <chrisw@redhat.com>,
	qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: [Qemu-devel] Re: KVM call agenda for Apr 27
Date: Tue, 27 Apr 2010 11:14:59 +0300	[thread overview]
Message-ID: <4BD69D03.2050502@redhat.com> (raw)
In-Reply-To: <4BD61584.9080208@codemonkey.ws>

On 04/27/2010 01:36 AM, Anthony Liguori wrote:
>
> A few comments:
>
> 1) The problem was not block watermark itself but generating a 
> notification on the watermark threshold.  It's a heuristic and should 
> be implemented based on polling block stats. 

Polling for an event that never happens is bad engineering.  What 
frequency do you poll?  you're forcing the user to make a lose-lose 
tradeoff.

> Otherwise, we'll be adding tons of events to qemu that we'll struggle 
> to maintain.

That's not a valid reason to reject a user requirement.  We may argue 
the requirement is bogus, or that the suggested implementation is wrong 
and point in a different direction, but saying that we may have to add 
more code in the future due to other requirements is ... well I can't 
find a word for it.

>
> 2) A block plugin doesn't solve the problem if it's just at the 
> BlockDriverState level because it can't interact with qcow2.

Why not?  We have a layered model.  guest -> qcow2 -> plugin (sends 
event) -> raw-posix.  Just need to insert the plugin at the appropriate 
layer.

>
> 3) For general block plugins, it's probably better to tackle userspace 
> block devices.  We have CUSE and FUSE already, a BUSE is a logical 
> conclusion.

We also have an nbd client.

Here's another option:  an nbd-like protocol that remotes all 
BlockDriver operations except read and write over a unix domain socket.  
The open operation returns an fd (SCM_RIGHTS strikes again) that is used 
for read and write.  This can be used to implement snapshots over LVM, 
for example.

-- 
Do not meddle in the internals of kernels, for they are subtle and quick to panic.

  reply	other threads:[~2010-04-27  8:15 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-26 17:26 KVM call agenda for Apr 27 Chris Wright
2010-04-26 17:26 ` [Qemu-devel] " Chris Wright
2010-04-26 17:51 ` Anthony Liguori
2010-04-26 17:51   ` [Qemu-devel] " Anthony Liguori
2010-04-26 22:12   ` Chris Wright
2010-04-26 22:12     ` [Qemu-devel] " Chris Wright
2010-04-26 22:36     ` Anthony Liguori
2010-04-26 22:36       ` [Qemu-devel] " Anthony Liguori
2010-04-27  8:14       ` Avi Kivity [this message]
2010-04-27  8:14         ` Avi Kivity
2010-04-27  8:48         ` Dor Laor
2010-04-27  8:48           ` [Qemu-devel] " Dor Laor
2010-04-27  8:56           ` Avi Kivity
2010-04-27  8:56             ` [Qemu-devel] " Avi Kivity
2010-04-27  9:08             ` Dor Laor
2010-04-27  9:08               ` [Qemu-devel] " Dor Laor
2010-04-27  9:22               ` Avi Kivity
2010-04-27  9:22                 ` [Qemu-devel] " Avi Kivity
2010-04-27  9:32                 ` Dor Laor
2010-04-27  9:32                   ` [Qemu-devel] " Dor Laor
2010-04-27  9:41                   ` Kevin Wolf
2010-04-27  9:41                     ` [Qemu-devel] " Kevin Wolf
2010-04-27 13:15                     ` Anthony Liguori
2010-04-27 13:15                       ` [Qemu-devel] " Anthony Liguori
2010-04-27  9:16             ` Kevin Wolf
2010-04-27  9:16               ` [Qemu-devel] " Kevin Wolf
2010-04-27  9:28               ` Avi Kivity
2010-04-27  9:28                 ` [Qemu-devel] " Avi Kivity
2010-04-27 13:03         ` Anthony Liguori
2010-04-27 13:03           ` [Qemu-devel] " Anthony Liguori
2010-04-27 13:08           ` Avi Kivity
2010-04-27 13:08             ` [Qemu-devel] " Avi Kivity
2010-04-27 13:11           ` Daniel P. Berrange
2010-04-27 13:11             ` [Qemu-devel] " Daniel P. Berrange
2010-04-27 13:15             ` Gleb Natapov
2010-04-27 13:15               ` [Qemu-devel] " Gleb Natapov
2010-04-27 13:38               ` Daniel P. Berrange
2010-04-27 13:38                 ` [Qemu-devel] " Daniel P. Berrange
2010-04-27 14:10                 ` Gleb Natapov
2010-04-27 14:10                   ` [Qemu-devel] " Gleb Natapov
2010-04-27  8:53       ` Kevin Wolf
2010-04-27 13:10         ` Anthony Liguori
2010-04-27 13:18           ` Kevin Wolf
2010-04-27 13:21             ` Anthony Liguori
2010-04-27 13:42               ` Kevin Wolf
2010-04-27 13:48                 ` Anthony Liguori
2010-04-27 13:58                   ` Kevin Wolf
2010-04-27 14:01                     ` Anthony Liguori
2010-04-27 11:11       ` Gleb Natapov
2010-04-27 11:11         ` [Qemu-devel] " Gleb Natapov
2010-04-27 13:00         ` Anthony Liguori
2010-04-27 13:00           ` [Qemu-devel] " Anthony Liguori
2010-04-27 13:05           ` Gleb Natapov
2010-04-27 13:05             ` [Qemu-devel] " Gleb Natapov
2010-04-27 13:19             ` Anthony Liguori
2010-04-27 13:19               ` [Qemu-devel] " Anthony Liguori
2010-04-27 13:29               ` Gleb Natapov
2010-04-27 13:29                 ` [Qemu-devel] " Gleb Natapov
2010-04-27  1:15   ` Luiz Capitulino
2010-04-27  1:15     ` Luiz Capitulino
2010-04-27  3:39     ` Aurelien Jarno
2010-04-27  3:39       ` Aurelien Jarno

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=4BD69D03.2050502@redhat.com \
    --to=avi@redhat.com \
    --cc=anthony@codemonkey.ws \
    --cc=chrisw@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=qemu-devel@nongnu.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.