We are hitting this bug as well. With Ceph storage back end running in
OpenStack environment.
Ubuntu 14.04
Kernel - 3.13.0-52-generic
Qemu: QEMU emulator version 2.0.0 (Debian 2.0.0+dfsg-2ubuntu1.11)
Please provide a workaround or solution or if a patch is being worked
on.
Thank you,
Arghya
-
We have a few fields that exist mainly to hold information from
__init__() until check() (matching the fact that parsing is
two-pass; the first to find type names, the second to associate
types together while honoring forward references), or which should
only be used through accessor methods. We s
On Fri, Sep 4, 2015 at 12:59 PM, John Snow wrote:
>
>
> On 09/04/2015 09:42 AM, Peter Maydell wrote:
>> On 31 August 2015 at 22:28, Alistair Francis
>> wrote:
>>> On Mon, Aug 31, 2015 at 1:54 PM, John Snow wrote:
For some reason, this patch never made it to the qemu-devel archives for
On Fri, Sep 4, 2015 at 3:50 PM, Frederic Konrad
wrote:
> On 01/09/2015 22:58, Alistair Francis wrote:
>>
>> On Tue, Jul 21, 2015 at 10:17 AM, wrote:
>>>
>>> From: KONRAD Frederic
>>>
>>> Seems this field is not needed.
>>
>> The commit message should be updated to represent the patch.
>
> Hmmm
On 08/21/2015 09:37 AM, Kővágó, Zoltán wrote:
> The current OptsVisitor flattens the whole structure, if there are same
> named fields under different paths (like `in' and `out' in `Audiodev'),
> the current visitor can't cope with them (for example setting
> `frequency=44100' will set the in's fre
On 08/21/2015 09:37 AM, Kővágó, Zoltán wrote:
> Probably more logical if NetdevBase comes before Netdev.
> No schematic changes.
>
> Signed-off-by: Kővágó, Zoltán
> ---
> qapi-schema.json | 46 +++---
> 1 file changed, 23 insertions(+), 23 deletions(-)
Ju
On Fri, Sep 04, 2015 at 05:22:56PM +0100, Peter Maydell wrote:
> This patchset enables the TZ support in the GIC for the systems
> where we enable TZ support in the CPU. In practice that means
> just the "virt" and "vexpress" boards, since all the others
> disable the CPU TZ support.
>
> There are
On 08/21/2015 09:37 AM, Kővágó, Zoltán wrote:
> Except qapi-schema.json, this patch was generated by:
>
> find . -name .git -prune -o -type f \! -name '*~' -print0 | \
> xargs -0 sed -i \
> -e 's/NetClientOptionsKind/NetClientDriver/g' \
> -e 's/NET_CLIENT_OPTIONS_KIND_/NET_CLIENT_DRIVER
On 02/09/2015 01:19, Alistair Francis wrote:
On Tue, Jul 21, 2015 at 10:17 AM, wrote:
From: KONRAD Frederic
This introduces a DPCD module. It wires on a aux-bus and can be accessed by
driver to get lane-speed, etc.
"the driver"
Also, the commit titles are a little messy. The capitalisation
On 02/09/2015 02:00, Alistair Francis wrote:
On Tue, Sep 1, 2015 at 2:48 PM, Alistair Francis wrote:
On Tue, Jul 21, 2015 at 10:17 AM, wrote:
From: KONRAD Frederic
This introduces a new bus: aux-bus.
It contains an address space for aux slaves devices and a bridge to an I2C bus
for I2C th
On 01/09/2015 22:58, Alistair Francis wrote:
On Tue, Jul 21, 2015 at 10:17 AM, wrote:
From: KONRAD Frederic
Seems this field is not needed.
The commit message should be updated to represent the patch.
Hmmm what do you mean?
It's not precise?
Thanks,
Fred
Signed-off-by: KONRAD Frederic
On 4 September 2015 at 20:53, Markus Armbruster wrote:
> RHEL-6 and SLES-11 provide Python 2.6. It'll also work on OS X back
> to 10.6.
>
> Signed-off-by: Markus Armbruster
Reviewed-by: Peter Maydell
I wonder if we're ever going to need to add that Python 3
support the error message alludes t
On 4 September 2015 at 20:09, Serge Hallyn wrote:
> Because it has deprecated it with commit
> 3be6ed60aa58095691bd697344765e715a327fc1. This prevents the warning
>
> (process:3535): GLib-WARNING **:
> /build/glib2.0-3tdlHy/glib2.0-2.45.6/./glib/gmem.c:482: custom memory
> allocation vtable not
Le 04/09/2015 15:35, Peter Maydell a écrit :
> On 3 September 2015 at 00:58, Laurent Vivier wrote:
>> This patch introduces a system very similar to the one used in the kernel
>> to attach specific functions to a given file descriptor.
>>
>> In this case, we attach a specific "host_to_target()"
On Fri, Sep 4, 2015 at 2:42 PM, Marc-André Lureau
wrote:
> Hi
>
> On Fri, Sep 4, 2015 at 11:25 PM, Peter Crosthwaite
> wrote:
>> On Fri, Sep 4, 2015 at 2:01 PM, wrote:
>>> From: Marc-André Lureau
>>>
>>> It looks like this documentation is obsolete: a child object may lookup
>>> its parent sto
On 09/04/2015 03:36 PM, Eric Blake wrote:
> On 08/21/2015 09:37 AM, Kővágó, Zoltán wrote:
>> This way we no longer need NetClientOptions and can convert Netdev
>> into a flat union.
>>
>> Signed-off-by: Kővágó, Zoltán
>> ---
>> net/clients.h| 20 ++--
>> net/dump.c | 6
Hi
On Fri, Sep 4, 2015 at 11:25 PM, Peter Crosthwaite
wrote:
> On Fri, Sep 4, 2015 at 2:01 PM, wrote:
>> From: Marc-André Lureau
>>
>> It looks like this documentation is obsolete: a child object may lookup
>> its parent stored in the Object struct.
>>
>
> Using a function wrapper or direct ac
Hi
On Wed, Aug 26, 2015 at 2:03 PM, Daniel P. Berrange wrote:
> +ObjectProperty *
> +object_class_property_add(ObjectClass *klass,
> + const char *name,
> + const char *type,
> + ObjectPropertyAccessor *get,
> +
On 08/21/2015 09:37 AM, Kővágó, Zoltán wrote:
> This way we no longer need NetClientOptions and can convert Netdev
> into a flat union.
>
> Signed-off-by: Kővágó, Zoltán
> ---
> net/clients.h| 20 ++--
> net/dump.c | 6 +++---
> net/hub.c| 6 +++---
> net/l2tp
On Fri, Sep 4, 2015 at 2:01 PM, wrote:
> From: Marc-André Lureau
>
> It looks like this documentation is obsolete: a child object may lookup
> its parent stored in the Object struct.
>
Using a function wrapper or direct access to Object structure field?
The latter would indicate they are worka
On 08/21/2015 09:37 AM, Kővágó, Zoltán wrote:
> NetLegacy is just Netdev with some extra fields (name, vlan) and an
> optional id. This patch merges the two structs, and net_client_init1
> got some extra checks to make sure only accept valid -netdev command
> lines. This is some extra work, but a
On 08/26/2015 09:31 AM, Eduardo Habkost wrote:
> As long as somebody who understands QAPI says the changes make sense and
> should work, I am OK with them if the following is changed:
That would be me, right? See my other mail for my tentative R-b;
although if you start renaming things due to this
On Fri, Sep 04, 2015 at 07:12:23PM +0100, Peter Maydell wrote:
> On 4 September 2015 at 19:00, Alistair Francis
> wrote:
> > On Thu, Sep 3, 2015 at 3:56 PM, Edgar E. Iglesias
> > wrote:
> >> On Thu, Sep 03, 2015 at 03:10:52PM -0700, Alistair Francis wrote:
> >>> Bit 15 of the PHY Specific Status
On 08/21/2015 09:36 AM, Kővágó, Zoltán wrote:
> Signed-off-by: Kővágó, Zoltán
The subject line is a one-line "what", and the commit body should be the
"why" - all but the most trivial "what" deserve a non-empty commit body.
In particular, I think you NEED to mention in the commit body that the
c
From: Marc-André Lureau
It looks like this documentation is obsolete: a child object may lookup
its parent stored in the Object struct.
Signed-off-by: Marc-André Lureau
---
include/qom/object.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/qom/object.h b/include/qom/object.h
in
On 08/21/2015 09:36 AM, Kővágó, Zoltán wrote:
> They are required for flat unions (you still have to allocate the
> structs).
>
> Signed-off-by: Kővágó, Zoltán
> ---
> qapi/opts-visitor.c | 15 +++
> 1 file changed, 15 insertions(+)
>
Reviewed-by: Eric Blake
However, note that I
From: Alberto Garcia
This adds a new 'cache-clean-interval' option that cleans all qcow2
cache entries that haven't been used in a certain interval, given in
seconds.
This allows setting a large L2 cache size so it can handle scenarios
with lots of I/O and at the same time use little memory duri
From: Alberto Garcia
After having emptied the cache, the data in the cache tables is no
longer useful, so we can tell the kernel that we are done with it. In
Linux this frees the resources associated with it.
The effect of this can be seen in the HMP commit operation: it moves
data from the top
On 08/21/2015 09:36 AM, Kővágó, Zoltán wrote:
> This will let us print options in a format that the user would actually
> write it on the command line (foo=bar,baz=asd,etc=def), without
> prepending a spurious comma at the beginning of the list, or quoting
> values unnecessarily. This patch provid
From: Alberto Garcia
QEMU has options to configure the size of the L2 and refcount
caches for the qcow2 format. However, choosing the right sizes for
a particular disk image is not a straightforward operation since
the ratio between the cache size and the allocated disk space is
not obvious and d
On 09/04/2015 01:30 PM, Laurent Vivier wrote:
> As '-help' output is 400 lines long it is not easy
> to find information, but generally we know from
> which area we want the information.
>
> As sections already exist in the help description,
> add some options to only display the wanted section.
>
From: Max Reitz
Currently, if a subprocess of a python test (i.e. qemu-io, qemu-img, or
qemu) receives a signal and is subsequently aborted, this is not logged.
This patch makes python tests always check the exit code of these
subprocesses, and emit a message if they have been killed.
Signed-of
From: Max Reitz
This patch allows specifying the interface to be used for the drive, and
makes specifying a path optional (if the path is None, the "file" option
will be omitted, thus creating an empty drive).
Signed-off-by: Max Reitz
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
tes
From: Wen Congyang
We need to use threshold to check if too many write operation fails.
If threshold is larger than num children, we always get write error
event even if all write operations success.
Signed-off-by: Wen Congyang
Message-id: 55962f72.3060...@cn.fujitsu.com
Reviewed-by: Alberto Ga
From: Max Reitz
Currently, if a qemu/qemu-io/qemu-img/qemu-nbd invocation receives a
segmentation fault, that message is invisible in most cases since the
output is generally filtered and bash suppresses the segmentation fault
notice for any but the last element of a pipe.
Most of the time, the
From: Max Reitz
The filename given to qemu_open() in block/raw-posix.c should generally
have been processed by raw_normalize_devicepath(); unless we are only
probing (in which case the caller often checks whether the file is a
block device or not, and this property will be changed by
raw_normaliz
From: Alberto Garcia
Changing the current ordering saves 8 bytes per cache entry in x86_64.
Signed-off-by: Alberto Garcia
Reviewed-by: Max Reitz
Reviewed-by: Eric Blake
Reviewed-by: Kevin Wolf
Message-id:
0bd55291211df3dfb514d0e7d2031dd5c4f9f807.1438690126.git.be...@igalia.com
Signed-off-by
From: Max Reitz
While -nodefaults is set in $QEMU_OPTIONS, this is currently (wrongly)
ignored for Python iotests. In order to be prepared for when this is
fixed, we should explicitly add an IDE CD-ROM drive instead of relying
on it being created automatically.
Signed-off-by: Max Reitz
Reviewed
From: Max Reitz
Example:
$ ./qemu-img create -f qcow2 /tmp/t.qcow2 64M
$ ./qemu-img amend -f qcow2 -o backing_file=/tmp/t.qcow2, -o help \
/tmp/t.qcow2
This should not crash. This actually is tested by iotest 082, but not
caught due to the segmentation fault being silent (which is something
From: Bo Tu
There is no 'ide-cd' device defined on non-pc platform, so
test_medium_not_found() test should be skipped.
Reviewed-by: Max Reitz
Reviewed-by: Michael Mueller
Reviewed-by: Sascha Silbe
Signed-off-by: Xiao Guang Chen
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/041 | 6 +
From: Bo Tu
The default device id of hard disk on the s390 platform is "virtio0"
which differs to the "ide0-hd0" for the x86 platform. Setting id in
the drive definition, ie:"qemu -drive id=testdisk", will be the same
on all platforms.
Reviewed-by: Max Reitz
Signed-off-by: Bo Tu
Signed-off-by:
From: Bo Tu
when creating an image qemu-img enable us specifying the size of the
image using -o size=xx options. But when we specify an invalid size
such as a negtive size then different platform gives different result.
parse_option_size() function in util/qemu-option.c will be called to
parse t
From: Bo Tu
This patch adds qemu machine type support to the io test suite.
Based on the qemu default machine type and alias of the default machine
type the reference output file can now vary from the default to a
machine specific output file if necessary. When using a machine specific
reference
From: Bo Tu
This patch fixes an io test suite issue that was introduced with the
commit c88930a6866e74953e931ae749781e98e486e5c8 'qemu-char: Permit only
a single "stdio" character device'. The option supresses the creation of
default devices such as the floopy and cdrom. Output files for test cas
The following changes since commit b597aa037dbd98014c8dec3d69a5e2240f432533:
Merge remote-tracking branch 'remotes/armbru/tags/pull-monitor-2015-09-04'
into staging (2015-09-04 17:37:50 +0100)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git tags/for-upstream
for yo
On 09/04/2015 01:55 PM, Stefan Weil wrote:
> Am 04.09.2015 um 21:42 schrieb Eric Blake:
>> On 09/04/2015 01:30 PM, Laurent Vivier wrote:
>>> The goal is to be able to use '-help' alone, or with
>>> a sub-section, i.e. '-help network,usb'.
>> Uggh. I hate reinventing the wheel. We aren't using geto
On 09/04/2015 09:42 AM, Peter Maydell wrote:
> On 31 August 2015 at 22:28, Alistair Francis
> wrote:
>> On Mon, Aug 31, 2015 at 1:54 PM, John Snow wrote:
>>> For some reason, this patch never made it to the qemu-devel archives for
>>> either v5 or v6. I only received a direct copy to my inbox,
Am 04.09.2015 um 21:42 schrieb Eric Blake:
> On 09/04/2015 01:30 PM, Laurent Vivier wrote:
>> The goal is to be able to use '-help' alone, or with
>> a sub-section, i.e. '-help network,usb'.
> Uggh. I hate reinventing the wheel. We aren't using getopt_long_only();
> but if we were, the ONLY way to
On 09/04/2015 01:42 PM, Eric Blake wrote:
> On 09/04/2015 01:30 PM, Laurent Vivier wrote:
>> The goal is to be able to use '-help' alone, or with
>> a sub-section, i.e. '-help network,usb'.
>
> Uggh. I hate reinventing the wheel. We aren't using getopt_long_only();
> but if we were, the ONLY way
RHEL-6 and SLES-11 provide Python 2.6. It'll also work on OS X back
to 10.6.
Signed-off-by: Markus Armbruster
---
configure | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/configure b/configure
index 21c4089..bce1769 100755
--- a/configure
+++ b/configure
@@ -11
On 04.09.2015 19:18, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> tests/qemu-iotests/137 | 143
> +
> tests/qemu-iotests/137.out | 43 ++
> tests/qemu-iotests/group | 1 +
> 3 files changed, 187 insertions(+)
> create mod
On Fri, Sep 4, 2015 at 8:17 AM, Alistair Francis
wrote:
> Connect the Sysbus AHCI device to ZynqMP.
>
> Signed-off-by: Alistair Francis
> Reviewed-by: Sai Pavan Boddu
Reviewed-by: Peter Crosthwaite
> ---
> V6:
> - Fix up Macros
> V2:
> - Marcoify the number of SATA prts
> - Change the erro
On Fri, Sep 4, 2015 at 8:17 AM, Alistair Francis
wrote:
> The AHCIState struct can either have AHCIPCIState or SysbusAHCIState
> as a parent. The ahci_irq_lower() and ahci_irq_raise() functions
> assume that it is always AHCIPCIState, which is not always the
> case, which causes a seg fault. Verif
On 09/04/2015 01:41 PM, marcandre.lur...@redhat.com wrote:
> From: Marc-André Lureau
>
> The legacy<> type is no longer used since 7ce7ffe02.
>
> Signed-off-by: Marc-André Lureau
> ---
> qapi-schema.json | 15 ++-
> 1 file changed, 6 insertions(+), 9 deletions(-)
> @@ -1689,10 +16
On 09/04/2015 01:30 PM, Laurent Vivier wrote:
> The goal is to be able to use '-help' alone, or with
> a sub-section, i.e. '-help network,usb'.
Uggh. I hate reinventing the wheel. We aren't using getopt_long_only();
but if we were, the ONLY way to specify optional arguments to a long
option is by
From: Marc-André Lureau
The legacy<> type is no longer used since 7ce7ffe02.
Signed-off-by: Marc-André Lureau
---
qapi-schema.json | 15 ++-
1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/qapi-schema.json b/qapi-schema.json
index 67fef37..884d476 100644
--- a/qapi-sc
On 09/04/2015 01:30 PM, Laurent Vivier wrote:
> Signed-off-by: Laurent Vivier
> ---
> qemu-options.hx | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Eric Blake
Subject line might have been more obvious as 'help: add missing newline'
>
> diff --git a/qemu-options.hx b/qe
As '-help' output is 400 lines long it is not easy
to find information, but generally we know from
which area we want the information.
As sections already exist in the help description,
add some options to only display the wanted section.
'-help' now can take an optional parameter, which is
a com
Signed-off-by: Laurent Vivier
---
qemu-options.hx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index 77f5853..ab1af02 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -3510,7 +3510,7 @@ DEF("dump-vmstate", HAS_ARG, QEMU_OPTION_dump_vms
This series allows to only display a given list of help sections
(and fix a typo).
v4: as proposed by Eric, provide a list of sections, restore default -help
v3: add a 2nd patch to add an help section, -help displays only this section
v2: simplify the dance of #define/#undef, thanks to Marc-André.
The goal is to be able to use '-help' alone, or with
a sub-section, i.e. '-help network,usb'.
Signed-off-by: Laurent Vivier
---
vl.c | 20 +++-
1 file changed, 15 insertions(+), 5 deletions(-)
diff --git a/vl.c b/vl.c
index 584ca88..a4fa7e5 100644
--- a/vl.c
+++ b/vl.c
@@ -1932,
On 04.09.2015 19:18, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> tests/qemu-iotests/039 | 27 +++
> tests/qemu-iotests/039.out | 18 ++
> 2 files changed, 45 insertions(+)
>
> diff --git a/tests/qemu-iotests/039 b/tests/qemu-iotests/039
> ind
On 04.09.2015 19:18, Kevin Wolf wrote:
> For updating the cache sizes or disabling lazy refcounts there is a bit
> more to do than just changing the variables, but otherwise we're all set
> for changing options during bdrv_reopen().
>
> Just implement the missing pieces and hook the functions up i
Because it has deprecated it with commit
3be6ed60aa58095691bd697344765e715a327fc1. This prevents the warning
(process:3535): GLib-WARNING **:
/build/glib2.0-3tdlHy/glib2.0-2.45.6/./glib/gmem.c:482: custom memory
allocation vtable not supported
Changelog (v2): consolidate #ifdefs
Signed-off-by
Convert all machines to use DEFINE_MACHINE instead of QEMUMachine
automatically using a script.
Cc: Richard Henderson
Cc: Peter Maydell
Cc: Li Guang
Cc: Antony Pavlov
Cc: Evgeny Voevodin
Cc: Maksim Kozlov
Cc: Igor Mitsyanko
Cc: Dmitry Solodkiy
Cc: Rob Herring
Cc: Peter Chubb
Cc: Jan Kisz
The struct is not used anymore and can be eliminated.
Signed-off-by: Eduardo Habkost
---
include/hw/boards.h | 27 ---
include/qemu/typedefs.h | 1 -
include/sysemu/blockdev.h | 2 +-
include/sysemu/sysemu.h | 2 +-
vl.c | 36
Coding style change only.
Signed-off-by: Eduardo Habkost
---
hw/ppc/mac_oldworld.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/ppc/mac_oldworld.c b/hw/ppc/mac_oldworld.c
index 06fdbaf..b72d67d 100644
--- a/hw/ppc/mac_oldworld.c
+++ b/hw/ppc/mac_oldworld.c
@@ -365,7 +
This will make the code follow the same pattern used for other machines,
and will make it easier to automatically convert the code to be
QOM-based.
Cc: Peter Maydell
Cc: Evgeny Voevodin
Cc: Maksim Kozlov
Cc: Igor Mitsyanko
Cc: Dmitry Solodkiy
Signed-off-by: Eduardo Habkost
---
hw/arm/exynos
The macro will allow easy registration of a TYPE_MACHINE subclass, using
only the machine name and a MachineClass initialization function as
parameter.
Signed-off-by: Eduardo Habkost
---
Changes series v1 -> series v2: none
Changes series v2 -> series v3:
* Don't set MachineClass::name explicitl
We don't need a QEMUMachine array to query max_cpus, if we can get the
corresponding MachineClass.
Cc: Peter Maydell
Cc: Evgeny Voevodin
Cc: Maksim Kozlov
Cc: Igor Mitsyanko
Cc: Dmitry Solodkiy
Signed-off-by: Eduardo Habkost
---
hw/arm/exynos4_boards.c | 4 +++-
1 file changed, 3 insertions
The code is checking smp_cpus against EXYNOS4210_NCPUS, not against
max_cpus, so use EXYNOS4210_NCPUS in the error message for consistency.
Cc: Peter Maydell
Cc: Evgeny Voevodin
Cc: Maksim Kozlov
Cc: Igor Mitsyanko
Cc: Dmitry Solodkiy
Signed-off-by: Eduardo Habkost
---
hw/arm/exynos4_boards
This converts all code using QEMUMachine to register a QOM class directly,
using a new macro: DEFINE_MACHINE.
Python script used to convert all machines automatically can be seen at:
https://gist.github.com/ehabkost/582fe4969205975febb2#file-qemumachine-eliminate-py
Git branch:
git://github.
On 04.09.2015 19:18, Kevin Wolf wrote:
> Eric says that "any" sounds better than "either", and my non-native
> feeling says the same, so let's change it.
>
> Suggested-by: Eric Blake
> Signed-off-by: Kevin Wolf
> ---
> block/qcow2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Revie
On 04.09.2015 19:18, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> qemu-io-cmds.c | 90
> ++
> 1 file changed, 90 insertions(+)
Reviewed-by: Max Reitz
signature.asc
Description: OpenPGP digital signature
On Fri, Sep 4, 2015 at 11:12 AM, Peter Maydell wrote:
> On 4 September 2015 at 19:00, Alistair Francis
> wrote:
>> On Thu, Sep 3, 2015 at 3:56 PM, Edgar E. Iglesias
>> wrote:
>>> On Thu, Sep 03, 2015 at 03:10:52PM -0700, Alistair Francis wrote:
Bit 15 of the PHY Specific Status Register is
> -Original Message-
> From: Alistair Francis [mailto:alistai...@gmail.com]
> Sent: Friday, September 04, 2015 11:51 PM
> To: Sai Pavan Boddu
> Cc: qemu-devel@nongnu.org Developers; Sai Pavan Boddu; Peter Maydell;
> Peter Crosthwaite; Alistair Francis; Edgar Iglesias
> Subject: Re: [Qemu-
On Fri, Sep 4, 2015 at 2:01 AM, Frederic Konrad
wrote:
> On 04/09/2015 01:34, Alistair Francis wrote:
>>
>> On Thu, Sep 3, 2015 at 12:28 AM, Frederic Konrad
>> wrote:
>>>
>>> On 02/09/2015 23:39, Alistair Francis wrote:
On Tue, Jul 21, 2015 at 10:17 AM, wrote:
>
> From: KONRAD
On Fri, Sep 4, 2015 at 12:12 AM, Sai Pavan Boddu
wrote:
> For a easy creation of device directly form sdhci, without building a wrapper
> around.
Can you fix the commit message?
>
> Signed-off-by: Sai Pavan Boddu
> ---
> hw/sd/sdhci.c | 2 +-
> {hw/sd => include/hw}/sdhci.h |
On 4 September 2015 at 19:00, Alistair Francis
wrote:
> On Thu, Sep 3, 2015 at 3:56 PM, Edgar E. Iglesias
> wrote:
>> On Thu, Sep 03, 2015 at 03:10:52PM -0700, Alistair Francis wrote:
>>> Bit 15 of the PHY Specific Status Register is reserved and
>>> should remain 0. Fix the reset value to ensure
On 4 September 2015 at 18:58, Serge Hallyn wrote:
> Because it has deprecated it with commit
> 3be6ed60aa58095691bd697344765e715a327fc1. This prevents the warning
>
> (process:3535): GLib-WARNING **:
> /build/glib2.0-3tdlHy/glib2.0-2.45.6/./glib/gmem.c:482: custom memory
> allocation vtable not
On 04.09.2015 19:18, Kevin Wolf wrote:
> qemu_opts_parse_noisily() already prints an error message with the exact
> reason why the parsing failed. No need to add another less specific one.
>
> Signed-off-by: Kevin Wolf
Reviewed-by: Max Reitz
signature.asc
Description: OpenPGP digital signatu
On Thu, Sep 3, 2015 at 3:56 PM, Edgar E. Iglesias
wrote:
> On Thu, Sep 03, 2015 at 03:10:52PM -0700, Alistair Francis wrote:
>> Bit 15 of the PHY Specific Status Register is reserved and
>> should remain 0. Fix the reset value to ensure that the 15th
>> bit is not set.
>>
>> Signed-off-by: Alistai
Because it has deprecated it with commit
3be6ed60aa58095691bd697344765e715a327fc1. This prevents the warning
(process:3535): GLib-WARNING **:
/build/glib2.0-3tdlHy/glib2.0-2.45.6/./glib/gmem.c:482: custom memory
allocation vtable not supported
as reported at https://bugs.launchpad.net/ubuntu/+
On 4 September 2015 at 15:24, Markus Armbruster wrote:
> The following changes since commit b041066421e8dcc7d080dfcfd83551c9c9f24ade:
>
> Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-request'
> into staging (2015-09-03 16:17:28 +0100)
>
> are available in the git repository
Hi Pavel,
On 09/04/2015 10:26 AM, Pavel Fedin wrote:
It is more convenient for ITS implementation. I already have the code for it,
and for ITS device it is useful to have two separate sysbus-mmio regions, one
for control and one for translation.
Do you plan posting the ITS patches on the l
03.09.2015 19:35, Peter Maydell пишет:
On 3 September 2015 at 15:31, Sergey Smolov wrote:
Do you think it is possible to implement another QEMU logger which will
make a record for every executed block,
Yes (this would just need to disable the TB linking optimisation,
which we've discussed p
On 4 September 2015 at 18:20, Richard Purdie
wrote:
> On Fri, 2015-09-04 at 13:43 +0100, Richard Purdie wrote:
>> On Fri, 2015-09-04 at 12:31 +0100, Peter Maydell wrote:
>> > On 4 September 2015 at 12:24, Richard Purdie
>> > wrote:
>> > > So just based on that, yes, seems that the rx_fifo looks t
For updating the cache sizes or disabling lazy refcounts there is a bit
more to do than just changing the variables, but otherwise we're all set
for changing options during bdrv_reopen().
Just implement the missing pieces and hook the functions up in
bdrv_reopen().
Signed-off-by: Kevin Wolf
---
On Fri, 2015-09-04 at 13:43 +0100, Richard Purdie wrote:
> On Fri, 2015-09-04 at 12:31 +0100, Peter Maydell wrote:
> > On 4 September 2015 at 12:24, Richard Purdie
> > wrote:
> > > So just based on that, yes, seems that the rx_fifo looks to be
> > > overrunning. I can add the asserts but I think i
On return, either all new options should be applied to BDRVQcowState (on
success), or all of the old settings should be preserved (on failure).
Signed-off-by: Kevin Wolf
Reviewed-by: Eric Blake
Reviewed-by: Max Reitz
---
block/qcow2.c | 52
qcow2_update_options() only updates some variables in BDRVQcowState and
doesn't really depend on other parts of it being initialised yet, so it
can be moved so that it immediately follows the other half of option
handling code in qcow2_open().
Signed-off-by: Kevin Wolf
Reviewed-by: Eric Blake
Re
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/137 | 143 +
tests/qemu-iotests/137.out | 43 ++
tests/qemu-iotests/group | 1 +
3 files changed, 187 insertions(+)
create mode 100755 tests/qemu-iotests/137
create mode 100644 test
Signed-off-by: Kevin Wolf
---
qemu-io-cmds.c | 90 ++
1 file changed, 90 insertions(+)
diff --git a/qemu-io-cmds.c b/qemu-io-cmds.c
index 53477e1..d6572a8 100644
--- a/qemu-io-cmds.c
+++ b/qemu-io-cmds.c
@@ -1979,6 +1979,95 @@ static const
Before we can allow updating options at runtime with bdrv_reopen(), we
need to split the function into prepare/commit/abort parts.
Signed-off-by: Kevin Wolf
Reviewed-by: Eric Blake
Reviewed-by: Max Reitz
---
block/qcow2.c | 101 ++
1 file
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/039 | 27 +++
tests/qemu-iotests/039.out | 18 ++
2 files changed, 45 insertions(+)
diff --git a/tests/qemu-iotests/039 b/tests/qemu-iotests/039
index 617f397..9e9b379 100755
--- a/tests/qemu-iotests/039
Signed-off-by: Kevin Wolf
Reviewed-by: Eric Blake
Reviewed-by: Max Reitz
---
block/qcow2.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/block/qcow2.c b/block/qcow2.c
index 9b67334..24acefd 100644
--- a/block/qcow2.c
+++ b/block/qcow2.c
@@ -549,8 +549,8 @@ st
With this commit, the handling of driver-specific options in
qcow2_open() is completely separated out into qcow2_update_options().
Signed-off-by: Kevin Wolf
Reviewed-by: Eric Blake
Reviewed-by: Max Reitz
---
block/qcow2.c | 111 +-
1 file
This is part two of what I had sent earlier as "[PATCH 00/34] block:
Cache mode for children, reopen overhaul and more". Most of the patches
were actually already reviewed in v1.
Apart from a few addressed review comments, there are no functional
changes compared to v1. Some rebasing was necessary
Signed-off-by: Kevin Wolf
Reviewed-by: Max Reitz
---
block.c | 42 +++---
block/commit.c| 4 ++--
include/block/block.h | 4 +++-
3 files changed, 44 insertions(+), 6 deletions(-)
diff --git a/block.c b/block.c
index 090923c..a2cd9b8 1
Eric says that "any" sounds better than "either", and my non-native
feeling says the same, so let's change it.
Suggested-by: Eric Blake
Signed-off-by: Kevin Wolf
---
block/qcow2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block/qcow2.c b/block/qcow2.c
index 76c331b..e1
1 - 100 of 327 matches
Mail list logo