On 06-12-22, 17:53, Oleksandr Tyshchenko wrote:
On 05.12.22 11:11, Viresh Kumar wrote:
Maybe we can add these fields and a config param, once someone wants to reuse this stuff for QEMU ?
I don't know what to suggest here, sorry.
On the one hand, it is an extra work for you trying to add functionality you don't need at the moment. On the other hand if we add "backendtype" config param right now with default to STANDALONE it might simplify work for someone who ends up adding other type (in particular, the QEMU). Let's see what the maintainers will say.
The extra work is minimal and that isn't something that worries me. What worries me, based on past experience, is adding code that _may_ be required in future, and is never used eventually. And for that I always vouch for not adding something unless we are sure we need it and there is some code which makes use of that configuration right away.