From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Andrew Jones <drjones@redhat.com>, waldi@debian.org
Cc: konrad@darnok.org, jeremy@goop.org,
xen-devel@lists.xensource.com,
stefano stabellini <stefano.stabellini@eu.citrix.com>,
virtualization@lists.linux-foundation.org
Subject: Re: [Xen-devel] [PATCH 1/4] xen kconfig: keep XEN_XENBUS_FRONTEND builtin
Date: Thu, 12 Jan 2012 09:37:45 -0500 [thread overview]
Message-ID: <20120112143745.GE7685@phenom.dumpdata.com> (raw)
In-Reply-To: <e37e8bec-f4ec-4420-81f0-da0ef90a6120@zmail13.collab.prod.int.phx2.redhat.com>
On Thu, Jan 12, 2012 at 05:49:57AM -0500, Andrew Jones wrote:
>
>
> ----- Original Message -----
> > On Wed, Jan 11, 2012 at 05:36:38PM +0100, Andrew Jones wrote:
> > > When XEN_XENBUS_FRONTEND gets selected as a module it can lead to
> > > unbootable configs. If we need it, then we should just build it in.
> >
> > Hm, don't the frontends by themsevles load this module? So if you
> > do 'modprobe xen-pcifront' it would load this automatically?
> >
>
> The problem is on boot, getting it there before we need xen-blkfront.
> I think it might solvable if you make sure your tooling pushes the
> xenbus module into your initrd. However we've had problems with
So it seems we also need patches for dracut and initramfs here?
Or is it possible to make the modules (fb) try to load xenbus frontend
automatically? Preferrably one would do this:
modprobe xen_fbfront
which would then automatically load xenbus_module, xen_kbdfront
Better yet if udev/kudzu figured out this automtically and loaded
the modules.
Is there a mechanism to automatically have udev do all of this loading?
Let CC the Debian maintainer on this converstation as he might have
some ideas in this.
> platform_pci being a module in the past, and if I'm not mistaken
Right, but that driver is not tied in with the frontends. It just
pokes at the ioport to disable QEMU HVM drivers.
> that was one of the motivators for building it in (5fbdc10395cd). I
> see this as a similar story.
>
> Drew
>
> > >
> > > Signed-off-by: Andrew Jones <drjones@redhat.com>
> > > ---
> > > drivers/xen/Kconfig | 2 +-
> > > 1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/drivers/xen/Kconfig b/drivers/xen/Kconfig
> > > index 8795480..1d24061 100644
> > > --- a/drivers/xen/Kconfig
> > > +++ b/drivers/xen/Kconfig
> > > @@ -118,7 +118,7 @@ config XEN_SYS_HYPERVISOR
> > > but will have no xen contents.
> > >
> > > config XEN_XENBUS_FRONTEND
> > > - tristate
> > > + bool
> > >
> > > config XEN_GNTDEV
> > > tristate "userspace grant access device driver"
> > > --
> > > 1.7.7.5
> >
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xensource.com
> > http://lists.xensource.com/xen-devel
> >
next prev parent reply other threads:[~2012-01-12 14:37 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-06 16:39 [PATCH] xen: remove CONFIG_XEN_DOM0 compile option Andrew Jones
2012-01-06 16:59 ` Stefano Stabellini
2012-01-09 10:37 ` [Xen-devel] " Andrew Jones
2012-01-09 11:39 ` Stefano Stabellini
2012-01-09 14:56 ` Konrad Rzeszutek Wilk
2012-01-09 16:12 ` Stefano Stabellini
2012-01-09 17:04 ` Konrad Rzeszutek Wilk
2012-01-09 17:38 ` Andrew Jones
2012-01-09 18:44 ` Stefano Stabellini
2012-01-10 0:57 ` Jeremy Fitzhardinge
2012-01-11 15:37 ` [Xen-devel] " Andrew Jones
2012-01-11 16:19 ` Konrad Rzeszutek Wilk
2012-01-11 16:36 ` [PATCH 1/4] xen kconfig: keep XEN_XENBUS_FRONTEND builtin Andrew Jones
2012-01-11 16:36 ` [PATCH 2/4 v2] xen kconfig: relax INPUT_XEN_KBDDEV_FRONTEND deps Andrew Jones
2012-01-11 17:30 ` Konrad Rzeszutek Wilk
2012-01-12 10:59 ` Andrew Jones
2012-01-11 16:36 ` [PATCH 3/4 v2] xen kconfig: add dom0 support help text Andrew Jones
2012-01-11 16:36 ` [PATCH 4/4] xen kconfig: describe xen tmem in the config menu Andrew Jones
2012-01-11 17:35 ` Konrad Rzeszutek Wilk
2012-01-12 10:54 ` Andrew Jones
2012-01-11 17:28 ` [PATCH 1/4] xen kconfig: keep XEN_XENBUS_FRONTEND builtin Konrad Rzeszutek Wilk
2012-01-12 10:49 ` [Xen-devel] " Andrew Jones
2012-01-12 14:37 ` Konrad Rzeszutek Wilk [this message]
2012-01-12 15:42 ` Bastian Blank
2012-01-12 17:46 ` [Xen-devel] " Andrew Jones
2012-01-11 17:27 ` [PATCH] xen: remove CONFIG_XEN_DOM0 compile option Konrad Rzeszutek Wilk
2012-01-12 10:53 ` [Xen-devel] " Andrew Jones
2012-01-09 18:12 ` Stefano Stabellini
2012-01-09 17:26 ` Andrew Jones
2012-01-06 17:16 ` Ian Campbell
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=20120112143745.GE7685@phenom.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=drjones@redhat.com \
--cc=jeremy@goop.org \
--cc=konrad@darnok.org \
--cc=stefano.stabellini@eu.citrix.com \
--cc=virtualization@lists.linux-foundation.org \
--cc=waldi@debian.org \
--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 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).