Re: [PATCH v2 6/8] Convert colo main documentation to restructuredText

[Lukas Straub <lukasstraub2@web.de>]

[-- Attachment #1: Type: text/plain, Size: 1846 bytes --]

On Tue, 20 Jan 2026 12:26:17 -0500
Peter Xu <peterx@redhat.com> wrote:

> On Sat, Jan 17, 2026 at 03:09:13PM +0100, Lukas Straub wrote:
> > Signed-off-by: Lukas Straub <lukasstraub2@web.de>
> > ---
> >  MAINTAINERS               |   2 +-
> >  docs/COLO-FT.txt          | 334 ------------------------------------------
> >  docs/system/index.rst     |   1 +
> >  docs/system/qemu-colo.rst | 361 ++++++++++++++++++++++++++++++++++++++++++++++
> >  4 files changed, 363 insertions(+), 335 deletions(-)  
> 
> Thank you.
> 
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 92ca20c9d4186a08519d15bfe8cbd583ab061a8b..4c30dc50d15c74b317443e43920e01b4560b03a5 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -3841,7 +3841,7 @@ F: migration/multifd-colo.*
> >  F: include/migration/colo.h
> >  F: include/migration/failover.h
> >  F: tests/qtest/migration/colo-tests.c
> > -F: docs/COLO-FT.txt
> > +F: docs/devel/qemu-colo.rst  
> 
> Should we still put it under docs/devel/migration?
> 
> COLO framework is under migration/.  COLO tests will be under
> tests/qtest/migration/.  I still think we should keep doc under migration/
> too, IOW when someone touches that we want to get copied too.

Whoops, this should be docs/system/qemu-colo.rst so user documentation.

I think this document is more for users, so they can test it just with
standalone qemu.
Also I want to replace the wiki page with this so they don't go
out of sync.

> 
> I also raised some other requests while discussing with you on the COLO
> details.  If you want, feel free to attach one more patch to add those
> contents into the doc (after this conversion patch), on either doubled mem
> consumption of SVM, or RAM mgmt details.

Will do.
I think I will make a extra doc for devel/migration.

> 
> Thanks,
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]