All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juergen Gross <juergen.gross@ts.fujitsu.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: dario.faggioli@citrix.com, xen-devel@lists.xen.org,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	dgdegra@tycho.nsa.gov, Ian Campbell <ian.campbell@citrix.com>
Subject: Re: SSID and CPU pool parsing in xl / libxl
Date: Mon, 28 Apr 2014 07:52:20 +0200	[thread overview]
Message-ID: <535DEC94.309@ts.fujitsu.com> (raw)
In-Reply-To: <20140425102024.GA4176@zion.uk.xensource.com>

On 25.04.2014 12:20, Wei Liu wrote:
> Hi Daniel, Dario, Juergen and others
>
> I'm trying to overhaul some libxl infrastructure at the moment, so that
> we can preserve domain configurations across save and restore. The first
> step is libxl needs to be able to serialize / deserialize
> configurations.
>
> When I went through the list of xl config file options, most of the
> options can be easily serialized and sent over the wire to the other
> end. But SSID parsing and CPU pool ID parsing look exceptional -- they
> happen in xl (toolstack) level, not libxl (library) level.
>
> Take "init_seclabel" as an example, xl calls libxl_flask_context_to_sid
> to translate a string to an SSID (which is local to host) then passes it
> to libxl. If libxl serialize its data, then the receiving end only sees
> the translated SSID, not the "init_seclabel". The string is actually the
> relavent bit to rebuild a domain, not the SSID (because it's local to
> host). The same situation applies to CPU pool ID parsing.
>
> I plan to push the parsing down to libxl level, but I'm not sure if
> there's any reason it's done in xl in the first place. Please let me
> know if you have any concern.

Just go ahead. CPU pool ID parsing was only used in xl, so I put it there.


Juergen

-- 
Juergen Gross                 Principal Developer Operating Systems
PSO PM&D ES&S SWE OS6                  Telephone: +49 (0) 89 62060 2932
Fujitsu                                   e-mail: juergen.gross@ts.fujitsu.com
Mies-van-der-Rohe-Str. 8                Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html

      parent reply	other threads:[~2014-04-28  5:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-25 10:20 SSID and CPU pool parsing in xl / libxl Wei Liu
2014-04-25 10:36 ` Ian Campbell
2014-04-25 15:07 ` Daniel De Graaf
2014-04-28  5:52 ` Juergen Gross [this message]

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=535DEC94.309@ts.fujitsu.com \
    --to=juergen.gross@ts.fujitsu.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=dario.faggioli@citrix.com \
    --cc=dgdegra@tycho.nsa.gov \
    --cc=ian.campbell@citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /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.