qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] "Internal" bdrv-dirty-bitmaps
@ 2019-07-08 21:53 John Snow
  0 siblings, 0 replies; only message in thread
From: John Snow @ 2019-07-08 21:53 UTC (permalink / raw)
  To: Markus Armbruster
  Cc: Vladimir Sementsov-Ogievskiy, qemu-devel, Qemu-block, Max Reitz

Hi Markus;

recently it's come up a few times that it might be nice to hide bitmaps
that have no names from the query results, because these are "internal"
bitmaps used by QEMU that the user cannot manipulate.

Actually, all of the bitmap-finding functions will refuse to touch
bitmaps with no name.

We added names to bitmaps with 0db6e54a8a2, but bitmaps existed prior to
this and already had a "query" mechanism -- meaning that the "internal"
use bitmaps created by mirror have always been user-visible.

commit 21b56835 added the bdrv_query_dirty_bitmaps function, replacing
and older one-per-node bitmap querying utility, which... leads me to
believe that we've always explicitly exposed these bitmaps...

Further back in time (2012), we see b9a9b3a4626, which explicitly
exposed this information -- intentionally showing the bitmap dirty count
for... either migration or mirror, it could have been both.

I guess it's back from
https://lists.gnu.org/archive/html/qemu-devel/2012-09/msg04545.html --
and just assumed we'd want to see this count during operations that used it.

I guess at this point I definitely can't sweep the last 7 years under a
mat. The problem I have with it is that without a name, these bitmaps
are meaningless to an outside viewer. There could be many no-name
bitmaps at once, each correlating to different operations that could
finish and delete them at any time.

Are these useful to still have? If not, is there a reasonable way to
announce their farewell party by the deprecation policy?

--js


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2019-07-08 21:54 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-08 21:53 [Qemu-devel] "Internal" bdrv-dirty-bitmaps John Snow

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).