about naming:
We will need functions for set/unset a subregion of BdrvDirtyBitmap, to
fix migration and mirror (accordingly to my [PATCH v2] block: fix
spoiling all dirty bitmaps by mirror and migration). Having the
function 'bdrv_reset_dirty_bitmap' from this patch, we'll have to add
13.11.2014 05:59, arei.gong...@huawei.com wrote:
Gonglei (2):
acpi-build: adjust indention 8 - 4 spaces
acpi-build: Make DPRINTF working for acpi-build
Applied both to -trivial, thank you!
/mjt
Applied to -trivial as is, for now, since
the fixes are necessary anyway. Good suggestion
by Stefan, to turn it into constant-depending
always-compilable code stands, but it is not
a show-stopper for the actual fix.
Thanks,
/mnt
##
{ 'enum': 'MigrationCapability',
- 'data': ['xbzrle', 'rdma-pin-all', 'auto-converge', 'zero-blocks']
}
+ 'data': ['xbzrle', 'rdma-pin-all', 'auto-converge', 'zero-blocks',
+ 'compress'] }
I'll repeat what I said on v1 (but this time, with some links to back
it up :)
We really
19.11.2014 10:09, Fam Zheng wrote:
On Wed, 11/19 14:57, Jun Li wrote:
When see usb codes, find there are redundant brackets
!((udev-port-speedmask
USB_SPEED_MASK_SUPER)) here. So delete it.
Applied to -trivial, thank you!
/mjt
Applied to -trivial, thank you!
/mjt
Applied to -trivial, thanks!
/mjt
04.12.2014 17:00, Markus Armbruster wrote:
Markus Armbruster (3):
util: Drop superfluous conditionals around g_free()
Fuse g_malloc(); memset() into g_new0()
util: Use g_new() friends where that makes obvious sense
Applied to trivial (fixing 2/3 subject prefix), thank you!
/mjt
04.12.2014 09:18, Fam Zheng wrote:
Using has is more slick because which(1) is not always there.
Applied to -trivial, thank you!
/mjt
On Tue, 12/09 17:26, Peter Lieven wrote:
diff --git a/include/hw/virtio/virtio-blk.h b/include/hw/virtio/virtio-blk.h
index 3f2652f..0ee9582 100644
--- a/include/hw/virtio/virtio-blk.h
+++ b/include/hw/virtio/virtio-blk.h
@@ -134,13 +134,6 @@ typedef struct VirtIOBlock {
struct
This patch series adds the Netduino 2 Machine to QEMU
Information on the board is avalible at:
http://www.netduino.com/netduino2/specs.htm
The git tree can be found at:
https://github.com/alistair23/qemu/tree/netduino2.7
This patch series makes some changes to the armv7m_init function
that
This patch adds the stm32f2xx USART controller
(UART also uses the same controller).
Signed-off-by: Alistair Francis alistai...@gmail.com
---
V6:
- Rename to STM32F2XX
- Fix up unimplemented printing
- Add a qemu_chr_accept()
V3:
- Update debug printing
V2:
- Drop charecters if the device is
This patch adds the stm32f2xx System Configuration
Controller. This is used to configure what memory is mapped
at address 0 (although that is not supported) as well
as configure how the EXTI interrupts work (also not
supported at the moment).
This device is not required for basic examples, but
This patch adds the Netduino 2 Machine.
This is a Cortex-M3 based machine. Information can be found at:
http://www.netduino.com/netduino2/specs.htm
Signed-off-by: Alistair Francis alistai...@gmail.com
---
hw/arm/Makefile.objs | 1 +
hw/arm/netduino2.c | 54
This patch adds the stm32f2xx timers: TIM2, TIM3, TIM4 and TIM5
to QEMU.
Signed-off-by: Alistair Francis alistai...@gmail.com
---
V6:
- Rename to STM32F2XX
- Change the timer calculations to use ns
- Update the value to timer_mod to ensure it is in ns
- Account for reloadable/resetable timer
This patch allows the board to specifiy the number of NVIC interrupt
lines when using armv7m_init.
Signed-off-by: Alistair Francis alistai...@gmail.com
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/arm/armv7m.c | 7 ---
hw/arm/stellaris.c | 5 -
04.12.2014 14:59, Thomas Huth wrote:
Ping?
Ping again
Thomas, we was in freeze trying to get 2.2 out of the door.
During this time I didn't process -trivial patches, to not
pollute incoming changes stream even further.
Your patch for some reason does not apply to current
state of
This patch moves the memory region init code from the
armv7m_init function to the stellaris_init function
Signed-off-by: Alistair Francis alistai...@gmail.com
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
V3:
- Rename the flash_size argument to mem_size
- Remove the sram_size
This patch adds the stm32f205 SoC. This will be used by the
Netduino 2 to create a machine.
Signed-off-by: Alistair Francis alistai...@gmail.com
---
V6:
- Correct the number of USART/UART devices
- Use macros to define how many devices are inited
- Update the memory regions name from
Applied to trivial.
However, I think this patch is too trivial even
for trivial-patches queue... ;) As in, don't
bother fixing things like this unless there's
other code around being changed too.
Thanks,
/mjt
applied to -trivial, thank you!
/mjt
Applied to -trivial, thanks!
/mjt
09.12.2014 10:15, zhanghailiang wrote:
Refactor superio_ioport_writeb to fix the out of bounds write warning.
Is it just a warning, or real oob write?
From the code it looks like it's just a warning...
[]
+
+}
+if (can_write == true) {
09.12.2014 17:08, Paolo Bonzini wrote:
* William Dauchy (will...@gandi.net) wrote:
Hi David,
Thank you for the quick answer.
On Dec09 18:55, Dr. David Alan Gilbert wrote:
You don't tell us what that command line is!
If you're using -M pc or -M q35 that's expected; if you're not
it's probably a bug - please show us the
07.12.2014 21:20, Antony Pavlov wrote:
Signed-off-by: Antony Pavlov antonynpav...@gmail.com
---
hw/block/pflash_cfi02.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/block/pflash_cfi02.c b/hw/block/pflash_cfi02.c
index 8513a17..389b4aa 100644
--- a/hw/block/pflash_cfi02.c
+++
On 10/12/2014 09:57, Michael Tokarev wrote:
09.12.2014 10:15, zhanghailiang wrote:
Refactor superio_ioport_writeb to fix the out of bounds write warning.
Is it just a warning, or real oob write?
From the code it looks like it's just a warning...
It's a bug.
The simpler patch would have
superio_ioport_readb can read the 256th element of the array.
Coverity reports an out-of-bounds write in superio_ioport_writeb,
but it does not show the corresponding out-of-bounds read
because it cannot prove that it can happen. Fix the root
cause of the problem (zhanghailang's patch instead
On 2014/12/10 16:57, Michael Tokarev wrote:
09.12.2014 10:15, zhanghailiang wrote:
Refactor superio_ioport_writeb to fix the out of bounds write warning.
Is it just a warning, or real oob write?
From the code it looks like it's just a warning...
Er, i don't know when input 'data' will be
45e1611de8be0eae55967694dd6e627c2dc354f2:
Update version for v2.2.0 release (2014-12-09 12:13:37 +)
are available in the git repository at:
git://git.kraxel.org/qemu tags/pull-vnc-20141210-1
for you to fetch changes up to 43948386bb109b97b3de0bb48573f317bdcb5008:
keymaps: correct keymaps.c following Qemu coding style
From: Gonglei arei.gong...@huawei.com
Arguments in wrong order (SWAPPED_ARGUMENTS)
The positions of arguments in the call to
tight_fill_palette do not match the ordering of the parameters:
fg is passed to bg
bg is passed to fg
Cc: Gerd Hoffmann kra...@redhat.com
Signed-off-by: Gonglei
From: Gonglei arei.gong...@huawei.com
It's hard to read because of the confused coding
style in this file. Let's correct it following Qemu
coding style.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
ui/keymaps.c | 196
Hi paolo,
Will this change affects the migration?
I noticed that there is a member 'SuperIOConfig superio_conf' in VT82C686BState.
vt82c686 seems only to be used in mips64el target, Do we support migration for
mips target?
Thanks,
zhanghailiang
On 2014/12/10 17:17, Paolo Bonzini wrote:
Hi,
New version of the series after a logn break. No major changes in the
existing patches, added some new patches though: Patch #3 to add the
vnc display name to the acls, and patches 8-10 adding a new monitor
command to query vnc server information.
please review,
Gerd
Gerd Hoffmann
Replace with a vnc_displays list, so we can have multiple vnc server
instances. Add vnc_server_find function to lookup a display by id.
With no id supplied return the first vnc server, for backward
compatibility reasons.
It is not possible (yet) to actually create mutiple vnc server
instances.
Add websockets bool to VncBasicInfo, report websocket server sockets,
flag websocket client connections.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
qapi-schema.json | 5 -
ui/vnc.c | 14 --
2 files changed, 16 insertions(+), 3 deletions(-)
diff --git
This patch switches vnc over to QemuOpts, and it (more or less
as side effect) allows multiple vnc server instances.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
include/ui/console.h | 4 +-
qmp.c| 15 ++-
ui/vnc.c | 271
vnc joins the party ;)
Also some s/head/seat/ to clarify.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
docs/multiseat.txt | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/docs/multiseat.txt b/docs/multiseat.txt
index 67151e0..b963665 100644
---
Also track the number of connections in connecting and shared state
(in addition to the exclusive state). Apply a configurable limit to
these connections.
The logic to apply the limit to connections in shared state is pretty
simple: When the limit is reached no new connections are allowed.
The
Add new query vnc qmp command, for the lack of better ideas just name it
query-vnc2. Changes over query-vnc:
* It returns a list of vnc servers, so multiple vnc server instances
are covered.
* Each vnc server returns a list of server sockets. Followup patch
will use that to also report
This patch adds a display= parameter to the vnc options. This allows to
bind a vnc server instance to a specific display, allowing to create a
multiseat setup with a vnc server for each seat.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
ui/vnc.c | 50
In case the display id is default (which is the one you get if you
don't explicitly assign one) we keep the old name scheme, without
display, for backward compatibility reasons.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
ui/vnc.c | 22 --
1 file changed, 20
so we can reuse it for the new vnc query command.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
ui/vnc.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/ui/vnc.c b/ui/vnc.c
index 8d189e7..d04802c 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@
On 10/12/2014 10:31, zhanghailiang wrote:
Hi paolo,
Will this change affects the migration?
I noticed that there is a member 'SuperIOConfig superio_conf' in
VT82C686BState.
vt82c686 seems only to be used in mips64el target, Do we support
migration for mips target?
No, there is no
On 10/12/2014 09:57, Michael Tokarev wrote:
09.12.2014 10:15, zhanghailiang wrote:
Refactor superio_ioport_writeb to fix the out of bounds write warning.
Is it just a warning, or real oob write?
From the code it looks like it's just a warning...
[]
+
+}
+if (can_write
On 10/12/2014 10:11, Michael Tokarev wrote:
07.12.2014 21:20, Antony Pavlov wrote:
Signed-off-by: Antony Pavlov antonynpav...@gmail.com
---
hw/block/pflash_cfi02.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/block/pflash_cfi02.c b/hw/block/pflash_cfi02.c
index 8513a17..389b4aa
On 10/12/2014 02:41, Ming Lei wrote:
On Wed, Dec 10, 2014 at 1:45 AM, Paolo Bonzini pbonz...@redhat.com wrote:
On 08/12/2014 08:19, Ming Lei wrote:
Alternatively, I'd accept a SCSI patch setting max_ws_blocks and friends
to 2GB - 1 block.
It should be better to not introduce the limit
Can you check if the same error messages appear in /var/log/syslog and
/var/log/libvirt/qemu/machine.log as I described in bug #1392504?
Because I think it is apparmor related.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
Found the following lines in /var/log/libvirt/qemu/machine.log
libusbx: error [_get_usbfs_fd] libusbx couldn't open USB device
/dev/bus/usb/001/004: Permission denied
libusbx: error [_get_usbfs_fd] libusbx requires write access to USB device
nodes.
libusbx: error [_get_usbfs_fd] libusbx couldn't
On (Wed) 10 Dec 2014 [11:55:49], ChenLiang wrote:
On 2014/12/10 11:18, Amit Shah wrote:
On (Mon) 24 Nov 2014 [19:55:50], arei.gong...@huawei.com wrote:
From: ChenLiang chenlian...@huawei.com
The logic of old code is correct. But Checking byte by byte will
consume time after an
DisplayState isn't used anywhere, drop it. Add the vnc server ID as
parameter instead, so it is possible to specify the server instance.
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
include/ui/console.h | 16
ui/vnc.c | 29 ++---
vl.c
The QEMU block layer has a limit of INT_MAX bytes per transfer.
Expose it in the block limits VPD page for both regular transfers
and WRITE SAME.
Reported-by: Ming Lei ming@canonical.com
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
hw/scsi/scsi-disk.c | 27
From: Max Reitz mre...@redhat.com
Add a command line option for adding a QMP monitor using pretty JSON
formatting.
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
Signed-off-by: Kevin Wolf kw...@redhat.com
---
qemu-options.hx | 8
vl.c|
From: Fam Zheng f...@redhat.com
Similar to bdrv_next, this traverses through graph_bdrv_states. Will be
useful to enumerate all the named nodes.
Signed-off-by: Fam Zheng f...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
Signed-off-by: Max Reitz
From: Stefan Hajnoczi stefa...@redhat.com
Add dataplane support to the blockdev-snapshot-delete-internal-sync QMP
command. By acquiring the AioContext we avoid race conditions with the
dataplane thread which may also be accessing the BlockDriverState.
Signed-off-by: Stefan Hajnoczi
From: Max Reitz mre...@redhat.com
_filter_qmp should be able to correctly filter out the QMP version
object for pretty JSON output.
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
Signed-off-by: Kevin Wolf kw...@redhat.com
---
This allows printing infos of BlockDriverStates that aren't at the root
of the graph (and logically implementing a BlockBackend).
Signed-off-by: Kevin Wolf kw...@redhat.com
---
hmp.c | 34 +-
1 file changed, 25 insertions(+), 9 deletions(-)
diff --git a/hmp.c
From: Fam Zheng f...@redhat.com
This returns the node name of a BDS. Remove the TODO comment and expect
the callers to be explicit.
Signed-off-by: Fam Zheng f...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by: Max Reitz
From: Fam Zheng f...@redhat.com
Node name is a better identifier of BDS.
We will want to query statistics of a BDS node buried in the BDS graph,
so reporting the node's name if there is one will do the trick.
Signed-off-by: Fam Zheng f...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
From: Stefan Hajnoczi stefa...@redhat.com
By acquiring the AioContext we avoid race conditions with the dataplane
thread which may also be accessing the BlockDriverState.
Fix up eject, change, and block_passwd in a single patch because
qmp_eject() and qmp_change_blockdev() both call
From: Stefan Hajnoczi stefa...@redhat.com
The BLOCK_OP_TYPE_INTERNAL_SNAPSHOT_DELETE op blocker exists but was
never used! Let's fix that so snapshot delete can be blocked.
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by: Max Reitz
Signed-off-by: Kevin Wolf kw...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
block/qapi.c | 7 +++
hmp.c | 8
qapi/block-core.json | 20 +++-
tests/qemu-iotests/051.out | 1 +
tests/qemu-iotests/067.out | 25
Commands with multiple boolean flag options (like 'info block') didn't
provide correct completion because only the first one was skipped.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
monitor.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/monitor.c b/monitor.c
index
From: Max Reitz mre...@redhat.com
Because all BlockDriverStates behind a single BlockBackend reside in a
single AioContext, it is fine to just pass these functions
(blk_add_aio_context_notifier() and blk_remove_aio_context_notifier())
through to the root BlockDriverState.
Signed-off-by: Max
The new function prints the info for a single BlockDriverState.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
hmp.c | 192 +-
1 file changed, 97 insertions(+), 95 deletions(-)
diff --git a/hmp.c b/hmp.c
index 05c3730..2e2b91b
The optional parameter specifying a block device allows now to use a
node-name instead of a drive name (and therefore to inspect any node in
the graph). The new -n options allows listing all named nodes instead of
BlockBackends.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
hmp.c | 31
From: Max Reitz mre...@redhat.com
Adding something like a delete notifier to a BlockBackend would not
make much sense, because whoever is interested in registering there will
probably hold a reference to that BlockBackend; therefore, the notifier
will never be called (or only when the notifiee
From: Stefan Hajnoczi stefa...@redhat.com
SATA 3.0 10.3.1 FIS Type values defines the constants used to
differentiate between FIS types.
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by: John Snow js...@redhat.com
Message-id: 1415874281-7371-3-git-send-email-stefa...@redhat.com
From: Max Reitz mre...@redhat.com
There are already some blk_aio_* functions, so we might as well have
blk_co_* functions (as far as we need them). This patch adds
blk_co_flush(), blk_co_discard(), and also blk_invalidate_cache() (which
is not a blk_co_* function but is needed nonetheless).
From: Stefan Hajnoczi stefa...@redhat.com
Debug code using #ifdef is susceptible to bitrot because the compiler
never checks the debug code.
This is easy to avoid, change the DPRINTF() macro to use if (DEBUG_AHCI)
and always give it a 0 or 1 value.
This also allows us to drop an #ifdef
From: Max Reitz mre...@redhat.com
Add one test whether blkdebug is able to generate a plain filename if
given a configuration file and a file to be tested only; and add another
test whether blkdebug is able to do the same without being given a
configuration file.
Signed-off-by: Max Reitz
From: Fam Zheng f...@redhat.com
When which(1) is not installed, we would complain perl not found
because it's the first set_prog_path check. The error message is
wrong.
Fix it by using command -v, a native way to query the existence of a
command.
Suggested-by: Eric Blake ebl...@redhat.com
From: Max Reitz mre...@redhat.com
Substitute BlockDriverState by BlockBackend in every globally visible
function provided by nbd.
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Paolo Bonzini pbonz...@redhat.com
Message-id: 1416309679-333-5-git-send-email-mre...@redhat.com
Signed-off-by:
Signed-off-by: Kevin Wolf kw...@redhat.com
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
Message-id: 1416497234-29880-4-git-send-email-kw...@redhat.com
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
Signed-off-by: Kevin Wolf kw...@redhat.com
---
This patch changes $QEMU_IO so that all tests by default pass a format
argument to qemu-io.
There are a few cases where -f $IMGFMT is not wanted because it selects
the wrong driver or json: filenames including a driver are used. They
are changed to use $QEMU_IO_PROG, which doesn't include any
From: Max Reitz mre...@redhat.com
With all externally visible functions changed to use BlockBackend, this
patch makes nbd use BlockBackend for everything internally as well.
While touching them, substitute 512 by BDRV_SECTOR_SIZE in the calls to
blk_read(), blk_write() and blk_co_discard().
This adds a -f option to qemu-io which allows to explicitly specify the
block driver to use for the given image.
Signed-off-by: Kevin Wolf kw...@redhat.com
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
From: Max Reitz mre...@redhat.com
Because qemu-nbd creates the BlockBackend by itself, it should create
the according BlockDriverState tree by itself as well; that means, it
has call bdrv_open() on its own. This is one of the places where
qemu-nbd still needs to use a BlockDriverState directly
Signed-off-by: Kevin Wolf kw...@redhat.com
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
Message-id: 1416497234-29880-5-git-send-email-kw...@redhat.com
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
Signed-off-by: Kevin Wolf kw...@redhat.com
---
The original intention was to pipe stderr of qemu into $fifo_out.
However, the redirections were specified in the wrong order for this.
This patch fixes it.
Now qemu's output on stderr can be retrieved with _send_qemu_cmd, which
applies several useful filters on the output that were missing
From: Stefan Hajnoczi stefa...@redhat.com
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
The only image format driver that even potentially accesses anything
after 512 bytes in its bdrv_probe() implementation is VMDK, which reads
a plain-text descriptor file. In practice, the field it's looking for
seems to come first and will be well within the first 512 bytes, too.
Signed-off-by:
If the user neglects to specify the image format, QEMU probes the
image to guess it automatically, for convenience.
Relying on format probing is insecure for raw images (CVE-2008-2004).
If the guest writes a suitable header to the device, the next probe
will recognize a format chosen by the
This is forbidden if the raw driver was probed.
Signed-off-by: Kevin Wolf kw...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
Message-id: 1416497234-29880-10-git-send-email-kw...@redhat.com
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
Signed-off-by: Kevin Wolf kw...@redhat.com
---
From: Stefan Hajnoczi stefa...@redhat.com
The transaction QMP command performs operations atomically on a group of
drives. This command needs to acquire AioContext in order to work
safely when virtio-blk dataplane IOThreads are accessing drives.
The transactional nature of the command means
From: Stefan Hajnoczi stefa...@redhat.com
drive_backup_prepare() assigns DriveBackupState fields to NULL in the
error path. This is unnecessary because the DriveBackupState is
allocated using g_malloc0() and other functions like
external_snapshot_prepare() already rely on this.
Do not
Our IDE emulation can't handle logical block sizes other than 512. Check
for it.
The original assumption was that other values would silently be ignored
(which is bad enough), but it's not quite true: The physical block size
is exposed in IDENTIFY DEVICE as a multiple of the logical block size.
From: Stefan Hajnoczi stefa...@redhat.com
The BLOCK_OP_TYPE_INTERNAL_SNAPSHOT op blocker exists but was never
used! Let's fix that so internal snapshots can be blocked.
[Fixed s/external/internal/ typo as pointed out by Paolo Bonzini and Max
Reitz.
--Stefan]
Signed-off-by: Stefan Hajnoczi
After reading the extension header, offset is incremented, but not
checked against end_offset any more. This way an integer overflow could
happen when checking whether the extension end is within the allowed
range, effectively disabling the check.
This patch adds the missing check and a test case
If a qcow2 image specifies a backing file format that doesn't correspond
to any format driver that qemu knows, we shouldn't fall back to probing,
but simply error out.
Not looking up the backing file driver in bdrv_open_backing_file(), but
just filling in the driver option if it isn't there moves
From: Max Reitz mre...@redhat.com
There are some block drivers which are essential to QEMU and may not be
removed: These are raw, file and qcow2 (as the default non-raw format).
Make their BlockDriver objects public so they can be directly referenced
throughout the block layer without needing to
From: Paolo Bonzini pbonz...@redhat.com
Use the external qemu-timer API instead.
No one else should be calling cpu_get_clock(), get_clock() and
get_clock_realtime() directly; they are internal functions and they
should be confined to qemu-timer.c and cpus.c (where the icount
implementation
From: Max Reitz mre...@redhat.com
If a driver supports image creation, it needs to set the .create_opts
field. We can use that to make sure .create_opts for both drivers
involved is not NULL in bdrv_img_create(), which is important so that
the create_opts pointer in that function is not NULL
The real on-disk size of an image depends on things like the host
filesystem. _img_info already filters it out, use the function in 060.
Signed-off-by: Kevin Wolf kw...@redhat.com
Reviewed-by: Michael Mueller m...@linux.vnet.ibm.com
Reviewed-by: Max Reitz mre...@redhat.com
---
From: Anton Blanchard an...@samba.org
Initialise our maximum page size capability to 64kB and increase
the page_size variable from 16 to 32 bits.
Signed-off-by: Anton Blanchard an...@samba.org
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
Signed-off-by: Kevin Wolf kw...@redhat.com
---
From: Max Reitz mre...@redhat.com
We can always assume raw, file and qcow2 being available; so do not use
bdrv_find_format() to locate their BlockDriver objects but statically
reference the respective objects.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Max Reitz mre...@redhat.com
Signed-off-by:
From: Max Reitz mre...@redhat.com
Although virtually impossible right now, bdrv_find_format(qcow) may
fail. The vvfat block driver should heed that case.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Kevin Wolf kw...@redhat.com
Signed-off-by: Kevin Wolf
From: Michael Mueller m...@linux.vnet.ibm.com
The real on-disk size of an image depends on things like the host
filesystem. _img_info already filters it out, use the function in 082.
Signed-off-by: Michael Mueller m...@linux.vnet.ibm.com
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by:
From: Max Reitz mre...@redhat.com
If a driver supports image creation, it needs to set the .create_opts
field. We can use that to make sure .create_opts for both drivers
involved is not NULL for the target image in qemu-img convert, which is
important so that the create_opts pointer in
From: Max Reitz mre...@redhat.com
The nfs protocol driver is capable of creating images, but did not
specify any creation options. Fix it.
A way to test this issue is the following:
$ qemu-img create -f nfs nfs://127.0.0.1/foo.qcow2 64M
Without this patch, it segfaults. With this patch, it
From: Max Reitz mre...@redhat.com
There may be NBD tests which do not create a sample image and simply
test whether wrong usage of the protocol is rejected as expected. In
this case, there will be no NBD server and trying to kill it during
clean-up will fail.
Cc: qemu-sta...@nongnu.org
1 - 100 of 343 matches
Mail list logo