[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [Qemu-devel] [PATCH v8 18/26] block: Add sgfnt_runtime_

From: Eric Blake
Subject: Re: [Qemu-block] [Qemu-devel] [PATCH v8 18/26] block: Add sgfnt_runtime_opts to BlockDriver
Date: Tue, 20 Feb 2018 09:15:30 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0

On 02/20/2018 08:51 AM, Max Reitz wrote:
On 2018-02-06 20:43, Eric Blake wrote:
On 02/05/2018 09:18 AM, Max Reitz wrote:
This new field can be set by block drivers to list the runtime options
they accept that may influence the contents of the respective BDS. As of
a follow-up patch, this list will be used by the common
bdrv_refresh_filename() implementation to decide which options to put
into BDS.full_open_options (and consequently whether a JSON filename has
to be created), thus freeing the drivers of having to implement that
logic themselves.

Additionally, this patch adds the field to all of the block drivers that
need it and sets it accordingly.

Signed-off-by: Max Reitz <address@hidden>

+    /* Pointer to a NULL-terminated array of names of significant
options that
+     * can be specified for bdrv_open(). A significant option is one
+     * changes the data of a BDS.
+     * If this pointer is NULL, the array is considered empty.
+     * "filename" and "driver" are always considered significant. */
+    const char *const *sgfnt_runtime_opts;

Warning: Bikeshedding follows:


might read easier (same number of letters, but has some vowels rthr thn
bng crptc bbrvtn).

So, did we reach any verdict on this? :-)

To sum up IRC (as far as I've understood), I don't quite like "vital" or
"needed" because they sound like those options would be required; but
they usually have defaults so they are not.

I preferred "strong", but you said that might mean something else
(although I still don't know what exactly :-)).


major?  apt?

(the thesaurus mentions other words like meaningful or influential, but those don't buy you any length reduction or easier abbreviations)

I don't recall what (if anything) I said against strong, so I'm also fine if that's what you choose.

At the end of the day, it's all bikeshedding - pick a term, and I'll live with it ;)

Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

reply via email to

[Prev in Thread] Current Thread [Next in Thread]