On Thu, Jan 14, 2021 at 13:52:45 +0100, Peter Krempa wrote:
On Thu, Jan 14, 2021 at 13:37:23 +0100, Michal Privoznik wrote:
> This capability tracks whether memory-backend-file has
> "x-use-canonical-path-for-ramblock-id" attribute. Introduced into
> QEMU by commit fa0cb34d2210cc749b9a70db99bb41c56ad20831. While
> "x-" prefix is considered experimental or internal to QEMU, the
> next commit justifies its use.
Since the detection of this feature is not limited to existing qemus, my
requirement that qemu must add acknowledgement that
"x-use-canonical-path-for-ramblock-id" will be treated as a stable
feature from now on and the qemu commit adding that must be mentioned in
this commit.
The only other way is to limit the feature detection to any released
qemu (thus qemu-5.2 and previous), since we still will not get a
guarantee that they will not change the flag in the future.
Let me reiterate the options when this will be acceptable so that we are
in the clear:
1) qemu declares "x-use-canonical-path-for-ramblock-id" stable in code
and docs (please cc me on the commit)
2) qemu drops the "x-", and ...
2a) we use "use-canonical-path-for-ramblock-id" only with new qemus
which have the stable version
(this is what we would usually do)
2b) we also add support for "x-use-canonical-path-for-ramblock-id"
but detection will be limited to existing releases
Supporting "x-use-canonical-path-for-ramblock-id" for any future release
of qemu without a clear declaration that it's considered stable in qemu
is not acceptable for libvirt.