xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Brendan Cully <brendan@cs.ubc.ca>
To: Dulloor <dulloor@gmail.com>
Cc: Jeremy Fitzhardinge <jeremy@goop.org>,
	"Xen-devel@lists.xensource.com" <Xen-devel@lists.xensource.com>,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	Ian Campbell <Ian.Campbell@citrix.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: Re: [PATCH] Remus breaks the build
Date: Tue, 17 Aug 2010 10:38:28 -0700	[thread overview]
Message-ID: <20100817173828.GA2898@kremvax.cs.ubc.ca> (raw)
In-Reply-To: <AANLkTiktNSnHfF_v+L7Ad+6BcaYwuQVdWv1WKm_r7wFy@mail.gmail.com>

On Friday, 13 August 2010 at 21:25, Dulloor wrote:
> On Fri, Aug 13, 2010 at 2:14 PM, Jeremy Fitzhardinge <jeremy@goop.org> wrote:
> >  On 08/13/2010 12:42 PM, Brendan Cully wrote:
> >> I assume you're talking about this snippet of tools/remus/kmod/Makefile:
> >>
> >> $(MAKE) -C $(KERNELDIR) SUBDIRS=`pwd` modules
> >>
> >> which expects to find a Makefile in $KERNELDIR but does the actual
> >> building in place, in the tools/remus/kmod directory (unless the
> >> kernel build system has changed recently?). I thought this was a
> >> pretty standard way to build out-of-tree kernel modules.
> >
> > I don't ever build the kernel out of the Xen tree.  In general, it
> > assumes the kernel tree has already been configured and built, which may
> > not be true if you're doing a parallel build, or if you're building the
> > Xen tree piecewise.
> >
> >> I'm not sure why this is causing you problems (is it?), but if you're
> >> willing to carry sch_queue in the pvops tree, I'd be happy to drop
> >> tools/remus/kmod in the unstable tree.
> >
> > Yes, I'm happy to include it.  Do you have a git reference I can merge from?
> 
> My understanding is that we don't need any out-of-tree modules, if
> linux is built with CONFIG_IFB.
> Also, that IFB isn't something available on 2.6.18 (and maybe
> 2.6.27?), where we will need to build these modules.
> 
> Is that right ? And, if thats the case, isn't it better to fold these
> drivers into 2.6.18 (2.6.27 ?) and
> support Remus conditional on IFB for pvops tree ?

Remus actually uses two modules. One is sch_queue, a Linux queueing
discipline that queues outbound guest traffic until the machine state
that produced it has been committed at the backup. This is the one
that we're talking about moving into the pvops tree (after I've tested
it -- I'm having the customary day-long fight getting Xen running
smoothly after having not updated for a while).

We need a second module (IFB or IMQ, depending on the kernel version)
because Linux queueing disciplines only operate on a device's outbound
traffic. Since Remus runs in dom0, it sees the guest's outbound
traffic as _inbound_ traffic on a VIF device. So IMQ/IFB is used to
redirect that incoming VIF traffic to a virtual intermediate device
with the sch_queue queueing discipline installed on it.

  reply	other threads:[~2010-08-17 17:38 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-13  0:38 [PATCH] Remus breaks the build Jeremy Fitzhardinge
2010-08-13 10:38 ` Ian Campbell
2010-08-13 11:55   ` Stefano Stabellini
2010-08-13 12:52     ` Ian Jackson
2010-08-13 19:44       ` Brendan Cully
2010-08-18 20:09         ` Brendan Cully
2010-08-13 19:42 ` Brendan Cully
2010-08-13 21:14   ` Jeremy Fitzhardinge
2010-08-14  4:25     ` Dulloor
2010-08-17 17:38       ` Brendan Cully [this message]
2010-08-18 13:40         ` Ian Jackson
2010-08-18 17:58           ` Brendan Cully
2010-08-19 14:38             ` Ian Jackson
2010-08-18 20:26     ` Brendan Cully
2010-08-18 20:34       ` Jed Smith
2010-08-18 20:39         ` Brendan Cully
2010-08-18 23:54       ` Jeremy Fitzhardinge
2010-08-19  0:03         ` Brendan Cully
2010-08-19  6:03           ` Pasi Kärkkäinen

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=20100817173828.GA2898@kremvax.cs.ubc.ca \
    --to=brendan@cs.ubc.ca \
    --cc=Ian.Campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=Xen-devel@lists.xensource.com \
    --cc=dulloor@gmail.com \
    --cc=jeremy@goop.org \
    --cc=stefano.stabellini@eu.citrix.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).