From: Juergen Gross <email@example.com>
To: George Dunlap <firstname.lastname@example.org>, Wei Liu <email@example.com>
Cc: Stefano Stabellini <firstname.lastname@example.org>,
Konrad Rzeszutek Wilk <email@example.com>,
George Dunlap <George.Dunlap@eu.citrix.com>,
Andrew Cooper <firstname.lastname@example.org>,
IanJackson <email@example.com>, Tim Deegan <firstname.lastname@example.org>,
Julien Grall <email@example.com>,
Jan Beulich <firstname.lastname@example.org>,
Daniel De Graaf <email@example.com>
Subject: Re: [Xen-devel] [PATCH v2 1/2] xen: add interface for obtaining .config from hypervisor
Date: Mon, 5 Aug 2019 17:12:56 +0200 [thread overview]
Message-ID: <firstname.lastname@example.org> (raw)
On 05.08.19 17:07, George Dunlap wrote:
> On 4/4/19 2:35 PM, Juergen Gross wrote:
>> On 04/04/2019 15:27, Wei Liu wrote:
>>> On Fri, Mar 15, 2019 at 05:29:28PM +0100, Juergen Gross wrote:
>>>> On 15/03/2019 16:55, Andrew Cooper wrote:
>>>>> On 14/03/2019 11:59, Juergen Gross wrote:
>>>>>> @@ -1100,6 +1100,20 @@ typedef struct xen_sysctl_cpu_policy xen_sysctl_cpu_policy_t;
>>>>>> + * XEN_SYSCTL_get_config
>>>>>> + *
>>>>>> + * Return gzip-ed .config file
>>>>> I don't think it is wise to bake gzip into the API. These days, there
>>>>> really are better options than gzip both in terms of compression ratio
>>>>> and decompress speed.
>>>>> I'd suggest just saying "file may be compressed" and leave it up to the
>>>>> packagers build system and `xl` choices to agree on which compression
>>>>> (if any) to use.
>>>> I am not convinced this is a good idea. I think the data format should
>>>> be specified.
>>>> And TBH: the difference in size for a rather "normal" .config is about
>>>> 50 bytes (I tried gzip, xz and bzip2 with gzip producing the smallest
>>> Has this been resolved?
>>> I don't really have an opinion on this.
>> I got no reply for that answer.
> You don't need a reply; if a suggestion is made, and you explain why you
> don't like it, then it's up to the other person to unabiguously Nack it
> if they don't like it; silence is interpreted as consent.
> To move this forward, I'd suggest re-posting it, adding a comment about
> the choice of compression algorithm in the commit message.
At the summit we had a discussion about a simple name-value store in the
hypervisor for e.g. communicating mitigation settings or runtime param
settings to at least dom0. The .config used for building Xen was agreed
to be another candidate for that interface.
I wanted to have a look at that when I'm finished with other urgent work
I'm actually doing.
Xen-devel mailing list
next prev parent reply other threads:[~2019-08-05 15:13 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-14 11:59 [PATCH v2 0/2] add xl command to get hypervisor .config Juergen Gross
2019-03-14 11:59 ` [PATCH v2 1/2] xen: add interface for obtaining .config from hypervisor Juergen Gross
2019-03-14 12:27 ` Wei Liu
2019-03-15 13:57 ` Jan Beulich
[not found] ` <5C8BAF5A020000780021F53D@suse.com>
2019-03-15 14:01 ` Juergen Gross
2019-03-15 14:24 ` Jan Beulich
2019-03-15 15:55 ` Andrew Cooper
2019-03-15 16:29 ` Juergen Gross
2019-04-04 13:27 ` Wei Liu
2019-04-04 13:35 ` Juergen Gross
2019-08-05 15:07 ` [Xen-devel] " George Dunlap
2019-08-05 15:12 ` Juergen Gross [this message]
2019-03-15 18:57 ` Daniel De Graaf
2019-03-14 11:59 ` [PATCH v2 2/2] tools: add new xl command get-hypervisor-config Juergen Gross
2019-03-14 12:27 ` Wei Liu
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:
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
* 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.