[libvirt] [PATCH v2 0/6] Add support for setting the emulator scheduler parameters

Martin Kletzander mkletzan at redhat.com
Tue Apr 16 11:24:51 UTC 2019


On Tue, Apr 16, 2019 at 12:59:34PM +0200, Ján Tomko wrote:
>On Mon, Apr 15, 2019 at 04:59:30PM +0200, Ján Tomko wrote:
>>On Mon, Apr 15, 2019 at 04:23:18PM +0200, Martin Kletzander wrote:
>>>
>>>I'll fix other things you mentioned and send a v3 if you want, just let me know.
>>>
>>
>>Depending on your opinion on the refactor:
>>https://www.redhat.com/archives/libvir-list/2019-April/msg00996.html
>>
>
>Given Andrea's comments on that series I withdraw my virXMLFormatElement
>suggestion.
>

Based on the fact virDomainSchedulerParse() callers utilize the return value
(which would not be used in the new one) nicely, I'll leave that function as is
in this series so that the refactor discussion takes place in a separate thread
(if anyone cares enough to do that).  I think this way it actually is better as
one function serves one purpose and not two slightly different ones based on
variable parameter.

Thanks.

>Jano


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190416/847daaf9/attachment-0001.sig>


More information about the libvir-list mailing list