[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] qcow crypto changes crash on migration (was: [PULL 10/1
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] qcow crypto changes crash on migration (was: [PULL 10/12] block: convert qcow/qcow2 to use generic cipher API) |
Date: |
Thu, 9 Jul 2015 16:51:39 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On 2015-07-09 12:53, Christian Borntraeger wrote:
> Forgot some CCs (patch author and migration folks)
>
>
> Am 09.07.2015 um 12:17 schrieb Christian Borntraeger:
> > Am 07.07.2015 um 16:12 schrieb Paolo Bonzini:
> >> From: "Daniel P. Berrange" <address@hidden>
> >>
> >> Switch the qcow/qcow2 block driver over to use the generic cipher
> >> API, this allows it to use the pluggable AES implementations,
> >> instead of being hardcoded to use QEMU's built-in impl.
> >>
> >> Signed-off-by: Daniel P. Berrange <address@hidden>
> >> Message-Id: <address@hidden>
> >> Signed-off-by: Paolo Bonzini <address@hidden>
> >
> > For whatever reason this breaks migration(or virsh restore)
> > from guests that were created with an older version of QEMU.
> >
> >
> >
> > Thread 1 (Thread 0x3fffb856bd0 (LWP 32226)):
> > #0 0x0000000080354a54 in qcrypto_cipher_free (cipher=0x0) at
> > /home/cborntra/REPOS/qemu/crypto/cipher-builtin.c:357
> > #1 0x00000000802ca912 in qcow2_close (bs=0x80a0cea0) at
> > /home/cborntra/REPOS/qemu/block/qcow2.c:1477
> > #2 0x00000000802caa32 in qcow2_invalidate_cache (bs=0x80a0cea0,
> > errp=0x81a3fc58) at /home/cborntra/REPOS/qemu/block/qcow2.c:1509
> > #3 0x000000008029bac0 in bdrv_invalidate_cache (bs=0x80a0cea0,
> > errp=0x81a3fd08) at /home/cborntra/REPOS/qemu/block.c:3135
> > #4 0x000000008029bbe6 in bdrv_invalidate_cache_all (errp=0x81a3fdd0) at
> > /home/cborntra/REPOS/qemu/block.c:3160
> > #5 0x000000008021de50 in process_incoming_migration_co (opaque=0x80acaae0)
> > at /home/cborntra/REPOS/qemu/migration/migration.c:160
> > #6 0x00000000802ab96a in coroutine_trampoline (i0=0, i1=-2137149984) at
> > /home/cborntra/REPOS/qemu/coroutine-ucontext.c:80
> > #7 0x000003fffc463ca2 in __makecontext_ret () from /lib64/libc.so.6
> > Backtrace stopped: previous frame identical to this frame (corrupt stack?)
This is the same kind of backtrace I got on a MIPS host (see my other
mail). The reason is that a NULL pointer is dereferenced before testing
it is non NULL in qcrypto_cipher_free.
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
address@hidden http://www.aurel32.net
- Re: [Qemu-devel] [PULL 05/12] crypto: introduce generic cipher API & built-in implementation, (continued)
- [Qemu-devel] [PULL 07/12] crypto: add a nettle cipher implementation, Paolo Bonzini, 2015/07/07
- [Qemu-devel] [PULL 11/12] ui: convert VNC to use generic cipher API, Paolo Bonzini, 2015/07/07
- [Qemu-devel] [PULL 06/12] crypto: add a gcrypt cipher implementation, Paolo Bonzini, 2015/07/07
- [Qemu-devel] [PULL 09/12] ui: convert VNC websockets to use crypto APIs, Paolo Bonzini, 2015/07/07
- [Qemu-devel] [PULL 12/12] ossaudio: fix memory leak, Paolo Bonzini, 2015/07/07
- [Qemu-devel] [PULL 10/12] block: convert qcow/qcow2 to use generic cipher API, Paolo Bonzini, 2015/07/07
- Re: [Qemu-devel] [PULL v2 00/12] Final changes for 2.4-rc0, Peter Maydell, 2015/07/08