All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Vrable <mvrable@cs.ucsd.edu>
To: xen-devel@lists.xensource.com
Subject: Re: Grant Table Network Issues
Date: Sun, 14 Aug 2005 09:27:12 -0700	[thread overview]
Message-ID: <20050814162712.GA6161@vrable.net> (raw)
In-Reply-To: <A95E2296287EAD4EB592B5DEEFCE0E9D282B13@liverpoolst.ad.cl.cam.ac.uk>

On Sun, Aug 14, 2005 at 04:59:53PM +0100, Ian Pratt wrote:
> > On Sun, Aug 14, 2005 at 08:39:53AM -0700, Michael Vrable wrote:
> > > Allocating the arrays statically fixes the problem for me.  
> > Steve Hand 
> > > says he'll likely be committing a fix soon.
> > 
> > Never mind; catching up on the changes to xen-unstable.hg, I 
> > see that you've already committed a fix.  Thanks.
> 
> Michael, if this fix is working fine for you, I minded to enable them in
> the default config... 

I haven't tested this particular commit, since I'm currently using a
testing a version of Xen that is patched to run unprivileged domains in
shadow translated mode, with grant-table based networking.  However, the
equivalent patch in my tree has fixed my networking stability issues.

There are still a few robustness issues to work on.  For example, the
BUG() when grant table donate operations fail in net_rx_action, which
can legitimately happen if the unprivileged domain hasn't set up its
grant tables properly.  I haven't seen this come up in normal use, but
does need to be fixed before the final release, since it will let an
unprivileged domain crash domain-0.  (I have seen it in my work, but
then I'm not doing normal things with unprivileged domains.)

I'd either vote for enabling grant tables now to get some wider testing,
but with a warning that some corner cases still need some work, or do a
check of the places that netback calls BUG() first.

--Michael Vrable

  parent reply	other threads:[~2005-08-14 16:27 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-13 18:59 Grant Table Network Issues Michael Vrable
2005-08-14  8:29 ` Keir Fraser
2005-08-14 15:39   ` Michael Vrable
2005-08-14 15:46     ` Michael Vrable
2005-08-14 15:59       ` Ian Pratt
2005-08-14 16:15         ` Steven Hand
2005-08-14 16:27         ` Michael Vrable [this message]
2005-08-14 16:43           ` Michael Vrable
2005-08-14 16:53     ` David Hopwood

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=20050814162712.GA6161@vrable.net \
    --to=mvrable@cs.ucsd.edu \
    --cc=xen-devel@lists.xensource.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.