> -----Original Message----- > From: Vladimir Sementsov-Ogievskiy <[email protected]> > Sent: Friday, April 14, 2023 5:51 PM > To: Zhang, Chen <[email protected]>; [email protected] > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; Zhang, > Hailiang <[email protected]>; [email protected]; > [email protected]; [email protected]; den- > [email protected] > Subject: Re: [PATCH] replication: compile out some staff when replication is > not configured > > On 14.04.23 04:24, Zhang, Chen wrote: > >> So, if I want to have an option to disable all COLO modules, do you > >> mean it should be additional --disable-colo option? Or better keep > >> one option -- disable-replication (and, maybe just rename to to --disable- > colo)? > > I think keep the option --disable-replication is enough. > > Generally speaking, these three modules do not belong to COLO, It has > been decoupled at the time of design. > > Instead, COLO is formed when these three modules are used in > combination. > > But it's not enough to me, I want to have a possibility to not build the > subsystem I don't need.
As I said, COLO not a specific subsystem, It is a usage of three general subsystems. Let's back to this patch, it try to not build block replication when not configured. It's OK. Although COLO may be the only user of replication, but can't assume all the COLO used subsystem not needed, even have a --disable-colo. For example in this patch disabled the net/filter-mirror/redirector.... Qemu network filter is a general framework with many submodules: filter-buffer/replay/mirror/rewriter..... Logically speaking, It is completely irrelevant with COLO. Thanks Chen > > -- > Best regards, > Vladimir
