From: Daniel De Graaf <dgdegra@tycho.nsa.gov>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: [PATCH 11/21] mini-os: make frontends and xenbus optional
Date: Mon, 23 Jan 2012 11:21:48 -0500 [thread overview]
Message-ID: <4F1D891C.4050508@tycho.nsa.gov> (raw)
In-Reply-To: <1327325368.24561.130.camel@zakaz.uk.xensource.com>
On 01/23/2012 08:29 AM, Ian Campbell wrote:
> On Fri, 2012-01-20 at 20:47 +0000, Daniel De Graaf wrote:
>> This adds compile-time logic to disable certain frontends in mini-os:
>> - pcifront is disabled by default, enabled for ioemu
>> - blkfront, netfront, fbfront, and kbdfront are enabled by default
>> - xenbus is required for any frontend, and is enabled by default
>>
>> If all frontends and xenbus are disabled, mini-os will run without
>> needing to communicate with xenstore, making it suitable to run the
>> xenstore daemon.
>
> I should've read this properly first time, then it wouldn't have taken
> me until 17/21 to figure it out.
>
> I think would be worthwhile to refactor the xenstore driver "extra"
> consoles from the single console provided via start info and to make the
> former a configurable option in line with the other front end drivers.
> That would, I think, tidy up the changes to xencons_ring. I think
> free_consfront and init_consfront only apply to the extra console case
> so the ifdefs you add within them would instead surround the whole
> functions (or even consfront.o if you decide that works).
>
> Ian.
>
Ah, I hadn't noticed that minios supported multiple consoles; that
explains a lot of the xenstore dependencies here. In that case, I think
it'd be useful to add CONFIG_CONSFRONT to allow dropping that support
(possibly refactoring into multiple files).
--
Daniel De Graaf
National Security Agency
next prev parent reply other threads:[~2012-01-23 16:21 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-20 20:47 [PATCH v3 00/21] Xenstore stub domain Daniel De Graaf
2012-01-20 20:47 ` [PATCH 01/21] xen: reinstate previously unused XENMEM_remove_from_physmap hypercall Daniel De Graaf
2012-01-20 20:47 ` [PATCH 02/21] xen: allow global VIRQ handlers to be delegated to other domains Daniel De Graaf
2012-01-20 20:47 ` [PATCH 03/21] xen: change virq parameters from int to uint32_t Daniel De Graaf
2012-01-20 20:47 ` [PATCH 04/21] xen: use XSM instead of IS_PRIV for getdomaininfo Daniel De Graaf
2012-01-20 20:47 ` [PATCH 05/21] xen: Preserve reserved grant entries when switching versions Daniel De Graaf
2012-01-20 20:47 ` [PATCH 06/21] tools/libxl: pull xenstore/console domids from xenstore Daniel De Graaf
2012-01-20 20:47 ` [PATCH 07/21] lib{xc, xl}: Seed grant tables with xenstore and console grants Daniel De Graaf
2012-01-23 12:26 ` Ian Campbell
2012-01-20 20:47 ` [PATCH 08/21] mini-os: avoid crash if no console is provided Daniel De Graaf
2012-01-20 20:47 ` [PATCH 09/21] mini-os: remove per-fd evtchn limit Daniel De Graaf
2012-01-20 20:47 ` [PATCH 10/21] mini-os: create app-specific configuration Daniel De Graaf
2012-01-23 12:41 ` Ian Campbell
2012-01-23 16:05 ` Daniel De Graaf
2012-01-23 16:23 ` Ian Campbell
2012-01-20 20:47 ` [PATCH 11/21] mini-os: make frontends and xenbus optional Daniel De Graaf
2012-01-23 12:51 ` Ian Campbell
2012-01-23 16:21 ` Daniel De Graaf
2012-01-23 16:24 ` Ian Campbell
2012-01-23 13:29 ` Ian Campbell
2012-01-23 16:21 ` Daniel De Graaf [this message]
2012-01-20 20:47 ` [PATCH 12/21] mini-os: fix list.h include guard name Daniel De Graaf
2012-01-20 20:47 ` [PATCH 13/21] xenstored: use grant references instead of map_foreign_range Daniel De Graaf
2012-01-23 13:06 ` Ian Campbell
2012-01-20 20:47 ` [PATCH 14/21] xenstored: add NO_SOCKETS compilation option Daniel De Graaf
2012-01-23 10:33 ` Stefano Stabellini
2012-01-23 13:12 ` Ian Campbell
2012-01-20 20:47 ` [PATCH 15/21] xenstored: support for tdb_copy with TDB_INTERNAL Daniel De Graaf
2012-01-23 13:14 ` Ian Campbell
2012-01-20 20:47 ` [PATCH 16/21] xenstored: support running in minios stubdom Daniel De Graaf
2012-01-23 10:39 ` Stefano Stabellini
2012-01-20 20:47 ` [PATCH 17/21] stubdom: enable xenstored build Daniel De Graaf
2012-01-23 13:17 ` Ian Campbell
2012-01-20 20:47 ` [PATCH 18/21] xenstored: add --event parameter for bootstrapping Daniel De Graaf
2012-01-20 20:47 ` [PATCH 19/21] xenstored: use domain_is_unprivileged instead of checking conn->id Daniel De Graaf
2012-01-20 20:47 ` [PATCH 20/21] xenstored: add --priv-domid parameter Daniel De Graaf
2012-01-20 20:47 ` [PATCH 21/21] xenstored: Add stub domain builder Daniel De Graaf
2012-01-23 13:52 ` Ian Campbell
2012-01-23 14:26 ` Stefano Stabellini
2012-01-23 14:31 ` Ian Campbell
2012-01-20 20:47 ` [PATCH] xenbus: Add support for xenbus backend in stub domain Daniel De Graaf
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=4F1D891C.4050508@tycho.nsa.gov \
--to=dgdegra@tycho.nsa.gov \
--cc=Ian.Campbell@citrix.com \
--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.