All of lore.kernel.org
 help / color / mirror / Atom feed
From: Amos Kong <akong@redhat.com>
To: Luiz Capitulino <lcapitulino@redhat.com>
Cc: laine@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com,
	"Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v6] net: add support of mac-programming over macvtap in QEMU side
Date: Mon, 17 Jun 2013 23:03:02 +0800	[thread overview]
Message-ID: <20130617150302.GA5484@amosk.info> (raw)
In-Reply-To: <20130617105439.11a90c42@redhat.com>

On Mon, Jun 17, 2013 at 10:54:39AM -0400, Luiz Capitulino wrote:
> On Mon, 17 Jun 2013 17:42:50 +0300
> "Michael S. Tsirkin" <mst@redhat.com> wrote:
> 
> > > The 1000ms I talked about is *not* what the guest will see. If there are
> > > events pending, the throttle API just queues the event and returns right
> > > away. I'd even _guess_ that this is faster then emitting the event.
> > 
> > If the filter is not updated for 1000ms then that is guest visible:
> > it is not getting packets with the new MAC.
> 
> Let me understand this better: the filter is going to be updated by
> libvirt, is that correctly?

Yes.
 
> If this is right, then I can understand where you came from, but how
> can we possibly control how long it will take for libvirt to take
> action?

Quote from my commitlog

|> There maybe exist an uncontrollable delay if we let Libvirt do the
|> real change, guests normally expect rx-filter updates immediately.
|> But it's another separate issue, we can investigate it when the
|> work in Libvirt side is done.


-- 
			Amos.

  reply	other threads:[~2013-06-17 15:03 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-14  7:45 [Qemu-devel] [PATCH v6] net: add support of mac-programming over macvtap in QEMU side Amos Kong
2013-06-17 13:11 ` Luiz Capitulino
2013-06-17 13:21   ` Michael S. Tsirkin
2013-06-17 13:30     ` Luiz Capitulino
2013-06-17 14:20       ` Michael S. Tsirkin
2013-06-17 14:34         ` Luiz Capitulino
2013-06-17 14:42           ` Michael S. Tsirkin
2013-06-17 14:54             ` Luiz Capitulino
2013-06-17 15:03               ` Amos Kong [this message]
2013-06-17 15:10               ` Michael S. Tsirkin
2013-06-17 15:15                 ` Luiz Capitulino
2013-06-17 14:50   ` Amos Kong
2013-06-17 13:52 ` Michael S. Tsirkin
2013-06-18  1:59 ` Wenchao Xia
2013-06-18  3:00   ` Amos Kong

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=20130617150302.GA5484@amosk.info \
    --to=akong@redhat.com \
    --cc=laine@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@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 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.