[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v14 1/7] Introduce yank feature

From: Markus Armbruster
Subject: Re: [PATCH v14 1/7] Introduce yank feature
Date: Mon, 11 Jan 2021 14:33:55 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)

Marc-André Lureau <marcandre.lureau@gmail.com> writes:

> Hi
> On Mon, Dec 28, 2020 at 7:08 PM Lukas Straub <lukasstraub2@web.de> wrote:
>> The yank feature allows to recover from hanging qemu by "yanking"
>> at various parts. Other qemu systems can register themselves and
>> multiple yank functions. Then all yank functions for selected
>> instances can be called by the 'yank' out-of-band qmp command.
>> Available instances can be queried by a 'query-yank' oob command.
> Looking at the changes and API usage, I wonder if it wouldn't have been
> simpler to associate the yank function directly with the YankInstance
> (removing the need for register/unregister functions - tracking the state
> left to the callback). Have you tried that approach? If not, I could check
> if this idea would work.

Considering we're at v14...  would it make sense to commit the current
approach, then explore the alternative approach on top?

If yes, is v14 committable as is?

reply via email to

[Prev in Thread] Current Thread [Next in Thread]