Re: [PATCH v3 22/22] qga: centralize logic for disabling/enabling commands

2024-07-19 Thread Konstantin Kostiuk
Reviewed-by: Konstantin Kostiuk On Fri, Jul 12, 2024 at 4:26 PM Daniel P. Berrangé wrote: > It is confusing having many different pieces of code enabling and > disabling commands, and it is not clear that they all have the same > semantics, especially wrt prioritization of the block/allow

[PATCH v3 22/22] qga: centralize logic for disabling/enabling commands

2024-07-12 Thread Daniel P . Berrangé
It is confusing having many different pieces of code enabling and disabling commands, and it is not clear that they all have the same semantics, especially wrt prioritization of the block/allow lists. The code attempted to prevent the user from setting both the block and allow lists concurrently,