Re: [PATCH v2 3/3] virsh-domain: Drop support for old APIs in cmdSetmem and cmdSetmaxmem

2021-06-02 Thread Michal Prívozník
On 5/19/21 3:03 PM, Peter Krempa wrote: > On Wed, May 19, 2021 at 12:16:57 +0200, Michal Privoznik wrote: >> Some of our really old APIs are missing @flags argument. We >> introduced their variants with "Flags" suffix and wired some >> logic into virsh to call the new variant only if necessary.

Re: [PATCH v2 3/3] virsh-domain: Drop support for old APIs in cmdSetmem and cmdSetmaxmem

2021-05-19 Thread Peter Krempa
On Wed, May 19, 2021 at 12:16:57 +0200, Michal Privoznik wrote: > Some of our really old APIs are missing @flags argument. We > introduced their variants with "Flags" suffix and wired some > logic into virsh to call the new variant only if necessary. This > enables virsh to talk to older daemon

[PATCH v2 3/3] virsh-domain: Drop support for old APIs in cmdSetmem and cmdSetmaxmem

2021-05-19 Thread Michal Privoznik
Some of our really old APIs are missing @flags argument. We introduced their variants with "Flags" suffix and wired some logic into virsh to call the new variant only if necessary. This enables virsh to talk to older daemon which may be lacking new APIs. However, in case of cmdSetmem() we are