|
From: | Max Reitz |
Subject: | Re: [Qemu-devel] [PATCH 1/4] blockdev: update outdated qmp_transaction() comments |
Date: | Fri, 21 Nov 2014 14:30:18 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 2014-11-21 at 14:22, Max Reitz wrote:
On 2014-11-21 at 11:48, Stefan Hajnoczi wrote:Originally the transaction QMP command was just for taking snapshots. The command became more general when drive-backup and abort were added. It is more accurate to say the command is about performing operations on an atomic group than to say it is about snapshots. Signed-off-by: Stefan Hajnoczi <address@hidden> --- blockdev.c | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-)Reviewed-by: Max Reitz <address@hidden>
Oh, I forgot to note: The patch is fine and it applies cleanly to master. But there is a contextual conflict when applying it on top of your previous series.
Max
[Prev in Thread] | Current Thread | [Next in Thread] |