提交 · 28c1202ba63370d44c003fd35257584b1e68de76 · openeuler / qemu

15 12月, 2011 9 次提交

block/qcow2.c: call qcow2_free_snapshots in the function of qcow2_close · 28c1202b

由 Li Zhi Hui 提交于 12月 07, 2011

Signed-off-by: NLi Zhi Hui <zhihuili@linux.vnet.ibm.com>
Reviewed-by: NStefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>

28c1202b

block: avoid useless checks on acb->bh · 3f3aace8

由 Paolo Bonzini 提交于 11月 14, 2011

Coverity is confused by this "if" and reports leaks on acb->bh.
The bottom half is always deleted before releasing the AIOCB,
in either bdrv_aio_cancel_em or bdrv_aio_bh_cb.
Signed-off-by: NPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>

3f3aace8

block: dma_bdrv_* does not return NULL · b9b2008b

由 Paolo Bonzini 提交于 11月 14, 2011

Initially attempted with the following semantic patch:

@ rule1 @
expression E;
statement S;
@@
  E =
(
   dma_bdrv_io
|  dma_bdrv_read
|  dma_bdrv_write
)
     (...);
(
- if (E == NULL) { ... }
|
- if (E)
    { <... S ...> }
)

which however did not match anything.
Signed-off-by: NPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>

b9b2008b

dma: the passed io_func does not return NULL · 6bee44ea

由 Paolo Bonzini 提交于 11月 14, 2011

Initially found with the following semantic patch:

@ type @
BlockDriverAIOCB *x;
expression E;
@@
  x = E;
- if (x == NULL) { ... }

@ acb1 @
expression E, E1;
@@
  E1->acb = E;
- if (E1->acb == NULL) { ... }

@ aiocb1 @
expression E, E1;
@@
  E1->aiocb = E;
- if (E1->aiocb == NULL) { ... }

@ acb @
expression E, E1;
@@
  E1.acb = E;
- if (E1.acb == NULL) { ... }

@ aiocb @
expression E, E1;
@@
  E1.aiocb = E;
- if (E1.aiocb == NULL) { ... }

but changed manually to include an assert.
Signed-off-by: NPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>

6bee44ea

block: qemu_aio_get does not return NULL · 91977c2e

由 Paolo Bonzini 提交于 11月 14, 2011

Initially done with the following semantic patch:

@ rule1 @
expression E;
statement S;
@@
  E = qemu_aio_get (...);
(
- if (E == NULL) { ... }
|
- if (E)
    { <... S ...> }
)

which however missed occurrences in linux-aio.c and posix-aio-compat.c.
Those were done by hand.

The change in vdi_aio_setup's caller was also done by hand.
Signed-off-by: NPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>

91977c2e

block: simplify failure handling for bdrv_aio_multiwrite · df9309fb

由 Paolo Bonzini 提交于 11月 14, 2011

Now that early failure of bdrv_aio_writev is not possible anymore,
mcb->num_requests can be set before the loop starts.
Signed-off-by: NPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>

df9309fb

block: bdrv_aio_* do not return NULL · ad54ae80

由 Paolo Bonzini 提交于 11月 30, 2011

Initially done with the following semantic patch:

@ rule1 @
expression E;
statement S;
@@
  E =
(
   bdrv_aio_readv
|  bdrv_aio_writev
|  bdrv_aio_flush
|  bdrv_aio_discard
|  bdrv_aio_ioctl
)
     (...);
(
- if (E == NULL) { ... }
|
- if (E)
    { <... S ...> }
)

which however missed the occurrence in block/blkverify.c
(as it should have done), and left behind some unused
variables.
Signed-off-by: NPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>

ad54ae80

tcg/arm: remove fixed map code buffer restriction · 222f23f5

由 Dr. David Alan Gilbert 提交于 12月 12, 2011

On ARM, don't map the code buffer at a fixed location, and fix up the
call/goto tcg routines to let it do long jumps.

Mapping the code buffer at a fixed address could sometimes result in it being
mapped over the top of the heap with pretty random results.
Signed-off-by: NDr. David Alan Gilbert <david.gilbert@linaro.org>
Signed-off-by: NAndrzej Zaborowski <andrew.zaborowski@intel.com>

222f23f5

A

Merge branch 'target-arm.for-upstream' of git://git.linaro.org/people/pmaydell/qemu-arm · 23ce84b1
由 Andrzej Zaborowski 提交于 12月 14, 2011

23ce84b1

14 12月, 2011 19 次提交

A

Merge remote-tracking branch 'stefanha/trivial-patches-next' into staging · 9423a2e8
由 Anthony Liguori 提交于 12月 14, 2011

9423a2e8

doc: Remove Symbian Virtual Platform · 126c7913

由 Stefan Weil 提交于 12月 14, 2011

Commit d396a657 removed the code
for SVP, so the documentation needs this update.
Signed-off-by: NStefan Weil <sw@weilnetz.de>
Signed-off-by: NStefan Hajnoczi <stefanha@linux.vnet.ibm.com>

126c7913

migration.h: remove incoming_expected declarations · 3060eb75

由 Isaku Yamahata 提交于 12月 13, 2011

The variable is deleted by 1bcef683
So remove its declaration.
Acked-by: NLuiz Capitulino <lcapitulino@redhat.com>
Signed-off-by: NIsaku Yamahata <yamahata@valinux.co.jp>
Signed-off-by: NStefan Hajnoczi <stefanha@linux.vnet.ibm.com>

3060eb75

tcg: make tcg_const_ptr actually accept a pointer argument · 73f5e313

由 Peter Maydell 提交于 12月 10, 2011

Make tcg_const_ptr() include a cast so that you can pass it a
pointer. This allows us to drop the casts we had in all the places
that use this macro.
Acked-by: NAndreas Färber <andreas.faerber@web.de>
Signed-off-by: NPeter Maydell <peter.maydell@linaro.org>
Signed-off-by: NStefan Hajnoczi <stefanha@linux.vnet.ibm.com>

73f5e313

Fix spelling in comments, documentation and messages · e7d81004