Subject: Re: [PATCH 14/18] virtio: add api for delayed callbacks

From: Rusty Russell <rusty-8n+1lVoiYb80n/[email-masked]>
Date: Mon, 16 May 2011 16:43:21 +0930

On Sun, 15 May 2011 15:48:18 +0300, "Michael S. Tsirkin" <[email-masked]> wrote:
> On Mon, May 09, 2011 at 03:27:33PM +0930, Rusty Russell wrote:
> > On Wed, 4 May 2011 23:52:33 +0300, "Michael S. Tsirkin" <[email-masked]> wrote:
> > > Add an API that tells the other side that callbacks
> > > should be delayed until a lot of work has been done.
> > > Implement using the new used_event feature.
> > 
> > Since you're going to add a capacity query anyway, why not add the
> > threshold argument here?
> 
> I thought that if we keep the API kind of generic
> there might be more of a chance that future transports
> will be able to implement it. For example, with an
> old host we can't commit to a specific index.

No, it's always a hint anyway: you can be notified before the threshold
is reached.  But best make it explicit I think.

Cheers,
Rusty.