On Thu, 07/18 10:28, Wenchao Xia wrote:
> 于 2013-7-17 23:21, Kevin Wolf 写道:
> >Am 17.07.2013 um 16:23 hat Eric Blake geschrieben:
> >>On 07/17/2013 08:03 AM, Wenchao Xia wrote:
> >>>This series allow user to read internal snapshot's contents without
> >>>qemu-img
> >>>convert. Another purpose is t
On Tue, 07/16 18:29, Paolo Bonzini wrote:
> This command dumps the metadata of an entire chain, in either tabular or JSON
> format.
>
> Signed-off-by: Paolo Bonzini
> ---
> v1->v2: mention encrypted clusters, use PRId64
>
> qemu-img-cmds.hx | 6 ++
> qemu-img.c | 185
>
On Tue, 2013-07-16 at 12:48 -0400, mrhi...@linux.vnet.ibm.com wrote:
> From: "Michael R. Hines"
>
> Code that does need to be visible is kept
> well contained inside this file and this is the only
> new additional file to the entire patch.
>
> This file includes the entire protocol and interface
ping
On 28/06/13 14:20, Leon Alrae wrote:
> ping
>
> http://patchwork.ozlabs.org/patch/251250/
> http://patchwork.ozlabs.org/patch/251254/
> http://patchwork.ozlabs.org/patch/251253/
> http://patchwork.ozlabs.org/patch/251251/
> http://patchwork.ozlabs.org/patch/251252/
> http://patchwork.ozlabs.
Am 17.07.2013 um 22:12 hat Mark Cave-Ayland geschrieben:
> On 17/07/13 14:35, Kevin Wolf wrote:
>
> >Okay, so I've had a quick look at that DMA controller, and it seems that
> >for a complete emulation, there's no way around using a bounce buffer
> >(and calling directly into the block layer inste
this patch adds a efficient encoding for zero blocks by
adding a new flag indicating a block is completely zero.
additionally bdrv_write_zeros() is used at the destination
to efficiently write these zeroes. depending on the implementation
this avoids that the destination target gets fully provisio
On 16.07.2013 18:29, Paolo Bonzini wrote:
This series adds a subcommand to "map" that can dump file metadata.
Metadata that is dumped includes:
- whether blocks are allocated in bs->file and, if so, where
- whether blocks are zero
- whether data is read from bs or bs->backing_hd
Metadata is d
Signed-off-by: Peter Lieven
---
block/iscsi.c | 77 +
1 file changed, 77 insertions(+)
diff --git a/block/iscsi.c b/block/iscsi.c
index 0bbf0b1..ab42f1e 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -49,6 +49,10 @@ typedef struct Iscs
this patch adds a coroutine for .bdrv_co_is_allocated as well as
a generic framework that can be used to build coroutines in block/iscsi.
Signed-off-by: Peter Lieven
---
block/iscsi.c | 130 +
1 file changed, 130 insertions(+)
diff --git
this series adds logical block provisioning functions to the iscsi layer.
it also is the first step to the change of migration to coroutines in
block/iscsi.
v3->v4:
- this series collapsed into 2 patches not yet merged. as per discussion
write_zero optimization will be a different approach h
On 17.07.2013 17:05, Paolo Bonzini wrote:
Il 11/07/2013 14:16, Peter Lieven ha scritto:
this series adds logical block provisioning functions to the iscsi layer.
it also is the first step to the change of migration to coroutines in
block/iscsi.
the changes to qemu-img and block migration have b
Signed-off-by: Peter Lieven
---
block.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block.c b/block.c
index b560241..c7f0197 100644
--- a/block.c
+++ b/block.c
@@ -2250,7 +2250,7 @@ int bdrv_read_unthrottled(BlockDriverState *bs, int64_t
sector_num,
enabled = bs
Il 18/07/2013 10:17, Peter Lieven ha scritto:
>
> maybe i have missed it in your patches, but would it be also possible to issue
> DISCARDZEROES ioctl on linux to return zero status for block devices also?
I think that would belong in your patches. Once we have discard_zeroes,
we can use it as y
Il 18/07/2013 09:25, Fam Zheng ha scritto:
>> > +/* Probe up to 1 G at a time. */
>> > +sector_num = (curr.start + curr.length) >> BDRV_SECTOR_BITS;
>> > +nsectors_left = DIV_ROUND_UP(length, BDRV_SECTOR_SIZE) -
>> > sector_num;
>> > +n = MAX(1 << (30 - BDRV_SECTOR
Am 18.07.2013 um 10:19 hat Peter Lieven geschrieben:
> this series adds logical block provisioning functions to the iscsi layer.
> it also is the first step to the change of migration to coroutines in
> block/iscsi.
>
> v3->v4:
> - this series collapsed into 2 patches not yet merged. as per disc
On Thu, Jul 18, 2013 at 09:48:50AM +0200, Peter Lieven wrote:
> this patch adds a efficient encoding for zero blocks by
> adding a new flag indicating a block is completely zero.
>
> additionally bdrv_write_zeros() is used at the destination
> to efficiently write these zeroes. depending on the im
Running qemu with "-device ?" option returns ~145 lines.
It is hard to manage understanding the output.
Theses patches aim to partially solve the problem by dividing the devices
into logical categories like "Network/Display/..." and sorting them by it.
Marcel Apfelbaum (2):
qemu-help: Sort devi
The category will be used to sort the devices displayed in
the command line help.
Signed-off-by: Marcel Apfelbaum
---
Note that these are not all the needed changes, the only purpose of
this patch is to be a proof of concept.
hw/audio/ac97.c | 1 +
hw/display/cirrus_vga.c | 1 +
hw/net/
Categorize devices that appear as output to "-device ?" command
by logical functionality. Sort the devices by logical categories
before showing them to user.
Signed-off-by: Marcel Apfelbaum
Reviewed-by: Kevin Wolf
---
include/hw/qdev-core.h | 7 +++
qdev-monitor.c | 23
Am 17.07.2013 um 17:54 hat ronnie sahlberg geschrieben:
> On Wed, Jul 17, 2013 at 3:25 AM, Kevin Wolf wrote:
> > Am 17.07.2013 um 11:58 hat Paolo Bonzini geschrieben:
> >> BTW, Peter and Ronnie: we were assuming that UNMAP with LBPRZ=1 always
> >> zeroes blocks, but is that true for unaligned oper
Am 17.07.2013 um 19:48 hat Peter Lieven geschrieben:
>
> Am 17.07.2013 um 19:04 schrieb Paolo Bonzini :
>
> > Il 17/07/2013 19:02, Peter Lieven ha scritto:
> >> For Disks we always use read/write16 so i think we Should also use
> >> writesame16. Or not?
> >
> > Yes.
> >
> > Remember you can st
On 07/17/2013 11:02 PM, Scott Wood wrote:
> On 07/17/2013 05:17:06 AM, Fabien Chouteau wrote:
>> On 07/16/2013 07:50 PM, Scott Wood wrote:
>> > On 07/16/2013 10:28:28 AM, Fabien Chouteau wrote:
>> >> On 07/16/2013 04:06 AM, Scott Wood wrote:
>> >> > On 07/10/2013 12:10:02 PM, Fabien Chouteau wrote:
Ping?
thanks
-- PMM
On 6 July 2013 17:39, Peter Maydell wrote:
> SPARC is one of the CPUs which has a funny syscall ABI for the
> pipe syscall; add it to the set of special cases in do_pipe().
>
> Signed-off-by: Peter Maydell
> ---
> bash is much more useful with this patch -- without it, it wi
Ping?
thanks
-- PMM
On 6 July 2013 21:44, Peter Maydell wrote:
> OpenRISC uses the asm-generic versions of target_stat and
> target_stat64, but it was incorrectly using the x86/ARM/etc version
> due to a misplaced defined(TARGET_OPENRISC). The previously unused
> OpenRISC section of the ifdef l
Il 18/07/2013 11:23, Kevin Wolf ha scritto:
> Am 17.07.2013 um 19:48 hat Peter Lieven geschrieben:
>>
>> Am 17.07.2013 um 19:04 schrieb Paolo Bonzini :
>>
>>> Il 17/07/2013 19:02, Peter Lieven ha scritto:
For Disks we always use read/write16 so i think we Should also use
writesame16. Or
Am 15.07.2013 17:29, schrieb Paolo Bonzini:
> Il 15/07/2013 17:20, Andreas Färber ha scritto:
>> We have some ugly include chains - yes, it shouldn't be here forever.
>> Just like the qemu/log.h situation is pretty unsatisfactory (I wouldve
>> liked to place log_cpu_state() into qom/cpu.h but it de
Il 18/07/2013 10:19, Peter Lieven ha scritto:
> this patch adds a coroutine for .bdrv_co_is_allocated as well as
> a generic framework that can be used to build coroutines in block/iscsi.
This conflicts with my bdrv_get_block_status patches.
Paolo
> Signed-off-by: Peter Lieven
> ---
> block/is
Il 18/07/2013 10:19, Peter Lieven ha scritto:
> Signed-off-by: Peter Lieven
> ---
> block/iscsi.c | 77
> +
> 1 file changed, 77 insertions(+)
>
> diff --git a/block/iscsi.c b/block/iscsi.c
> index 0bbf0b1..ab42f1e 100644
> --- a/block/i
Am 18.07.2013 um 12:24 hat Paolo Bonzini geschrieben:
> Il 18/07/2013 11:23, Kevin Wolf ha scritto:
> > Am 17.07.2013 um 19:48 hat Peter Lieven geschrieben:
> >>
> >> Am 17.07.2013 um 19:04 schrieb Paolo Bonzini :
> >>
> >>> Il 17/07/2013 19:02, Peter Lieven ha scritto:
> For Disks we always u
On 18.07.2013 12:24, Paolo Bonzini wrote:
Il 18/07/2013 11:23, Kevin Wolf ha scritto:
Am 17.07.2013 um 19:48 hat Peter Lieven geschrieben:
Am 17.07.2013 um 19:04 schrieb Paolo Bonzini :
Il 17/07/2013 19:02, Peter Lieven ha scritto:
For Disks we always use read/write16 so i think we Should al
Am 15.07.2013 08:43, schrieb Stefan Hajnoczi:
> On Mon, Jul 15, 2013 at 07:31:05AM +0200, Christian Berendt wrote:
>> Think we should clean up the registered build slaves. Here's a list
>> of offline slaves. Can they be removed from the bot?
>>
>> default_s390
>
> This slave goes offline relativel
On 18.07.2013 12:36, Paolo Bonzini wrote:
Il 18/07/2013 10:19, Peter Lieven ha scritto:
this patch adds a coroutine for .bdrv_co_is_allocated as well as
a generic framework that can be used to build coroutines in block/iscsi.
This conflicts with my bdrv_get_block_status patches.
I know, but th
Il 18/07/2013 12:44, Peter Lieven ha scritto:
> On 18.07.2013 12:24, Paolo Bonzini wrote:
>> Il 18/07/2013 11:23, Kevin Wolf ha scritto:
>>> Am 17.07.2013 um 19:48 hat Peter Lieven geschrieben:
Am 17.07.2013 um 19:04 schrieb Paolo Bonzini :
> Il 17/07/2013 19:02, Peter Lieven ha scrit
Il 18/07/2013 10:37, Peter Lieven ha scritto:
> Signed-off-by: Peter Lieven
> ---
> block.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/block.c b/block.c
> index b560241..c7f0197 100644
> --- a/block.c
> +++ b/block.c
> @@ -2250,7 +2250,7 @@ int bdrv_read_unthrott
On 18.07.2013 12:56, Paolo Bonzini wrote:
Il 18/07/2013 12:44, Peter Lieven ha scritto:
On 18.07.2013 12:24, Paolo Bonzini wrote:
Il 18/07/2013 11:23, Kevin Wolf ha scritto:
Am 17.07.2013 um 19:48 hat Peter Lieven geschrieben:
Am 17.07.2013 um 19:04 schrieb Paolo Bonzini :
Il 17/07/2013 19:
On 07/15/2013 03:21 PM, Oleksii Shevchuk wrote:
I tried do run QEMU with OVMF UEFI bios (QEMU and EDK2 trunk). It hangs
while booting:
qemu-kvm -L . -bios OVMF.fd --enable-kvm -debugcon file:/tmp/debug
-global isa-debugcon.iobase=0x402 -global PIIX4_PM.disable_s3=0 -global
PIIX4_PM.disable_
On Thu, Jul 11, 2013 at 01:46:58PM +0800, Wenchao Xia wrote:
> diff --git a/include/qemu-common.h b/include/qemu-common.h
> index f439738..06c777f 100644
> --- a/include/qemu-common.h
> +++ b/include/qemu-common.h
> @@ -191,6 +191,9 @@ int64_t strtosz_suffix(const char *nptr, char **end,
> const c
Fabio Fantoni writes ("Re: [PATCH v3] libxl: usb2 and usb3 controller support
for upstream qemu"):
> Il 12/07/2013 17:33, George Dunlap ha scritto:
> > On 12/07/13 13:36, Fabio Fantoni wrote:
[someone wrote:]
> >>> I'm just curious, why is this so complicated? Is this likely to be
> >>> fragile
On Thu, Jul 11, 2013 at 01:46:56PM +0800, Wenchao Xia wrote:
> This series brings internal snapshot support at block devices level, now we
> have two three methods to do block snapshot lively: 1) backing chain,
> 2) internal one and 3) drive-back up approach.
>
> Comparation:
> Adva
On 18/07/13 12:09, Ian Jackson wrote:
Fabio Fantoni writes ("Re: [PATCH v3] libxl: usb2 and usb3 controller support for
upstream qemu"):
Il 12/07/2013 17:33, George Dunlap ha scritto:
On 12/07/13 13:36, Fabio Fantoni wrote:
[someone wrote:]
I'm just curious, why is this so complicated? Is t
On Thu, Jul 18, 2013 at 6:09 AM, Ian Jackson wrote:
> Fabio Fantoni writes ("Re: [PATCH v3] libxl: usb2 and usb3 controller support
> for upstream qemu"):
>> Il 12/07/2013 17:33, George Dunlap ha scritto:
>> > On 12/07/13 13:36, Fabio Fantoni wrote:
> [someone wrote:]
>> >>> I'm just curious, why
Am 11.07.2013 um 07:46 hat Wenchao Xia geschrieben:
> Snapshot creation actually already distinguish id and name since it take
> a structured parameter *sn, but delete can't. Later an accurate delete
> is needed in qmp_transaction abort and blockdev-snapshot-delete-sync,
> so change its prototype.
On 07/18/2013 03:30 AM, Marcel Apfelbaum wrote:
On Tue, 2013-07-16 at 12:48 -0400, mrhi...@linux.vnet.ibm.com wrote:
From: "Michael R. Hines"
Code that does need to be visible is kept
well contained inside this file and this is the only
new additional file to the entire patch.
This file inclu
The category will be used to sort the devices displayed in
the command line help.
Signed-off-by: Marcel Apfelbaum
---
Note that these are not all the needed changes, the only purpose of
this patch is to be a proof of concept.
hw/audio/ac97.c | 1 +
hw/display/cirrus_vga.c | 1 +
hw/net/
Running qemu with "-device ?" option returns ~145 lines.
It is hard to manage understanding the output.
Theses patches aim to partially solve the problem by dividing the devices
into logical categories like "Network/Display/..." and sorting them by it.
Marcel Apfelbaum (2):
qemu-help: Sort devi
Categorize devices that appear as output to "-device ?" command
by logical functionality. Sort the devices by logical categories
before showing them to user.
Signed-off-by: Marcel Apfelbaum
Reviewed-by: Kevin Wolf
---
include/hw/qdev-core.h | 7 +++
qdev-monitor.c | 23
Anthony Liguori writes ("Re: [Qemu-devel] [PATCH v3] libxl: usb2 and usb3
controller support for upstream qemu"):
> The current way of creating each function will always be supported though.
Thanks for that reply. I'm reassured.
Ian.
Am 11.07.2013 um 07:46 hat Wenchao Xia geschrieben:
> Unlike savevm, the qmp_transaction interface will not generate
> snapshot name automatically, saving trouble to return information
> of the new created snapshot.
>
> Although qcow2 support storing multiple snapshots with same name
> but differe
Il 18/07/2013 14:07, Michael R. Hines ha scritto:
> On 07/18/2013 03:30 AM, Marcel Apfelbaum wrote:
>> On Tue, 2013-07-16 at 12:48 -0400, mrhi...@linux.vnet.ibm.com wrote:
>>> From: "Michael R. Hines"
>>>
>>> Code that does need to be visible is kept
>>> well contained inside this file and this is
Am 12.07.2013 13:06, schrieb George Dunlap:
> On 12/07/13 11:22, Fabio Fantoni wrote:
>> Usage: usbversion=1|2|3 (default=2)
>> Specifies the type of an emulated USB bus in the guest. 1 for usb1,
>> 2 for usb2 and 3 for usb3, it is available only with upstream qemu.
>> Default is 2.
>>
>> Signed-of
Il 18/07/2013 13:04, Peter Lieven ha scritto:
>> But if you set BDRV_DISCARD_WRITE_ZEROES, then you always need a
>> fallback to bdrv_write_zeroes. Why not just call bdrv_write_zeroes to
>> begin with? That's why extending bdrv_write_zeroes is preferable.
> In this case wo do not need a flag to t
Am 11.07.2013 um 07:46 hat Wenchao Xia geschrieben:
> This series brings internal snapshot support at block devices level, now we
> have two three methods to do block snapshot lively: 1) backing chain,
> 2) internal one and 3) drive-back up approach.
>
> Comparation:
> Advantages:
Il 18/07/2013 14:31, Andreas Färber ha scritto:
>> > I'm just curious, why is this so complicated? Is this likely to be
>> > fragile and break in the future?
> As pointed out previously, the bus=pci.0 bit will break with different
> PCI host bridges, such as the q35 machine existing today (-M q35
On Tue, Jul 16, 2013 at 03:01:58PM +0300, Gleb Natapov wrote:
> On Tue, Jul 16, 2013 at 07:56:25PM +0800, Arthur Chunqi Li wrote:
> > On Tue, Jul 16, 2013 at 7:42 PM, Gleb Natapov wrote:
> > > On Sun, Jul 07, 2013 at 11:13:37PM +0800, Arthur Chunqi Li wrote:
> > >> The recent KVM patch adds IA32_F
Even if the VM is already stopped, we cannot assume that all data has
already been successfully flushed to disk. The flush during the previous
vm_stop() could have failed.
Run bdrv_flush_all() unconditionally so that we get an error each time
if the block device isn't really flushed.
Signed-off-b
Am 18.07.2013 um 14:52 hat Kevin Wolf geschrieben:
> Even if the VM is already stopped, we cannot assume that all data has
> already been successfully flushed to disk. The flush during the previous
> vm_stop() could have failed.
>
> Run bdrv_flush_all() unconditionally so that we get an error each
On 07/18/2013 06:55 AM, Kevin Wolf wrote:
> Am 18.07.2013 um 14:52 hat Kevin Wolf geschrieben:
>> Even if the VM is already stopped, we cannot assume that all data has
>> already been successfully flushed to disk. The flush during the previous
>> vm_stop() could have failed.
>>
>> Run bdrv_flush_al
Il 18/07/2013 08:39, Fam Zheng ha scritto:
> On Wed, 07/17 13:04, Ian Main wrote:
>> This patch adds the original source drive as a backing drive to our target
>> image so that the target image will appear complete during backup. This
>> is especially useful for SYNC_MODE_NONE as it allows export
On 18.07.2013 14:31, Paolo Bonzini wrote:
Il 18/07/2013 13:04, Peter Lieven ha scritto:
But if you set BDRV_DISCARD_WRITE_ZEROES, then you always need a
fallback to bdrv_write_zeroes. Why not just call bdrv_write_zeroes to
begin with? That's why extending bdrv_write_zeroes is preferable.
In t
On 1 July 2013 20:07, Peter Maydell wrote:
> On 1 July 2013 18:34, Peter Maydell wrote:
>> This patchset is v5 of the "preparation patchset" that started
>> off with Alex, was passed to John Rigby and now to me.
>
> Also available via git:
>
> git://git.linaro.org/people/pmaydell/qemu-arm.git aar
On 18.07.2013, at 09:41, Kevin Wolf wrote:
> Am 17.07.2013 um 22:12 hat Mark Cave-Ayland geschrieben:
>> On 17/07/13 14:35, Kevin Wolf wrote:
>>
>>> Okay, so I've had a quick look at that DMA controller, and it seems that
>>> for a complete emulation, there's no way around using a bounce buffer
Il 18/07/2013 15:29, Peter Lieven ha scritto:
>>> If the driver would have a better method of writing zeroes than
>>> discard it simply should not set bdi->write_zeroes_w_discard = 1.
>> If the driver had a better method of writing zeroes than discard, it
>> simply should ignore the BDRV_MAY_UNMAP
Il 18/07/2013 10:27, Marcel Apfelbaum ha scritto:
> Running qemu with "-device ?" option returns ~145 lines.
> It is hard to manage understanding the output.
>
> Theses patches aim to partially solve the problem by dividing the devices
> into logical categories like "Network/Display/..." and sorti
It appears that even a very simple /etc/qemu-ifup configuration can
require the arch_prctl() syscall, see the example below:
#!/bin/sh
/sbin/ifconfig $1 0.0.0.0 up
/usr/sbin/brctl addif $1
Signed-off-by: Paul Moore
---
qemu-seccomp.c |3 ++-
1 file changed, 2 insert
The new modes are equal-rank, exclusive sub-modes of LM_IN_PROGRESS. Teach
opts_next_list(), opts_type_int() and opts_type_uint64() to handle them.
Also enumerate explicitly what functions are valid to call in what modes:
- opts_next_list() is valid to call while flattening a range,
- opts_end_lis
Simplify the code in preparation for the next patch.
Signed-off-by: Laszlo Ersek
---
qapi/opts-visitor.c | 23 +--
1 files changed, 5 insertions(+), 18 deletions(-)
diff --git a/qapi/opts-visitor.c b/qapi/opts-visitor.c
index 287aa06..ce6c290 100644
--- a/qapi/opts-visitor
On Thu, Jul 18, 2013 at 6:29 AM, Peter Lieven wrote:
> On 18.07.2013 14:31, Paolo Bonzini wrote:
>>
>> Il 18/07/2013 13:04, Peter Lieven ha scritto:
But if you set BDRV_DISCARD_WRITE_ZEROES, then you always need a
fallback to bdrv_write_zeroes. Why not just call bdrv_write_zeroes t
We're going to need more state while processing a list of repeated
options. This change eliminates "repeated_opts_first" and adds a new state
variable:
list_mode repeated_opts repeated_opts_first
-- - ---
LM_NONE NULL false
Prevent mistyped command line options from incurring high memory and CPU
usage at startup. 64K elements in a range should be enough for everyone
(TM).
The OPTS_VISITOR_RANGE_MAX macro is public so that unit tests can
construct corner cases with it.
Signed-off-by: Laszlo Ersek
---
include/qapi/o
Am 18.07.2013 um 15:44 hat Alexander Graf geschrieben:
>
> On 18.07.2013, at 09:41, Kevin Wolf wrote:
>
> > Am 17.07.2013 um 22:12 hat Mark Cave-Ayland geschrieben:
> >> On 17/07/13 14:35, Kevin Wolf wrote:
> >>
> >>> Okay, so I've had a quick look at that DMA controller, and it seems that
> >>>
On Thu, 2013-07-18 at 15:56 +0200, Paolo Bonzini wrote:
> Il 18/07/2013 10:27, Marcel Apfelbaum ha scritto:
> > Running qemu with "-device ?" option returns ~145 lines.
> > It is hard to manage understanding the output.
> >
> > Theses patches aim to partially solve the problem by dividing the devi
When a well-formed range value, bounded by signed integers, is encountered
while processing a repeated option, enter LM_SIGNED_INTERVAL and return
the low bound.
Signed-off-by: Laszlo Ersek
---
qapi/opts-visitor.c | 34 --
1 files changed, 28 insertions(+), 6 de
Consider the following QAPI schema fragment, for the purpose of command
line parsing with OptsVisitor:
{ 'type': 'UInt16', 'data': { 'u16': 'uint16' }}
{ 'union': 'NumaOptions',
'data': {
'node': 'NumaNodeOptions',
'mem' : 'NumaMemOptions' }}
{ 'type': 'NumaNodeOptions',
Basing the alarm off the rtc_clock was silly. It leads to horrible
spinning in the guest after being suspended and resumed, as it tries
to catch up with lost ticks.
This requires adding an accessor for reading the vm_clock too.
Signed-off-by: Richard Henderson
---
hw/alpha/typhoon.c|
On 18.07.2013 15:52, Paolo Bonzini wrote:
Il 18/07/2013 15:29, Peter Lieven ha scritto:
If the driver would have a better method of writing zeroes than
discard it simply should not set bdi->write_zeroes_w_discard = 1.
If the driver had a better method of writing zeroes than discard, it
simply s
For reasons that are probably obvious in retrospect, the paravirtual
alarm clocks should run on VM time, not wall clock time. Resume the
guest and it'll spin like mad trying to make up for lost ticks.
Please pull.
r~
The following changes since commit 6453a3a69488196f26d12654c6b148446abdf3d6:
On Thu, Jul 18, 2013 at 11:27:42AM +0300, Marcel Apfelbaum wrote:
> Categorize devices that appear as output to "-device ?" command
> by logical functionality. Sort the devices by logical categories
> before showing them to user.
>
> Signed-off-by: Marcel Apfelbaum
> Reviewed-by: Kevin Wolf
> --
When a well-formed range value, bounded by unsigned integers, is
encountered while processing a repeated option, enter LM_UNSIGNED_INTERVAL
and return the low bound.
Signed-off-by: Laszlo Ersek
---
qapi/opts-visitor.c | 32 +++-
1 files changed, 27 insertions(+), 5
Il 18/07/2013 15:55, ronnie sahlberg ha scritto:
>> > bdrv->write_zeroes will use writesame16 and set the unmap flag only if
>> > BDRV_MAY_DISCARD == 1 and BDRV_O_UNMAP == 1 and lbprz == 1.
> When you use WRITESAME16 you can ignore the lbprz flag.
> Just send a WRITESAME16 command with one block of
On 18 July 2013 15:02, Paolo Bonzini wrote:
> Il 18/07/2013 15:50, Richard Henderson ha scritto:
>> Update image to c87a92639b28ac42bc8f6c67443543b405dc479b,
>> incorporating changes for vm_time.
>>
>> Signed-off-by: Richard Henderson
>> ---
>> pc-bios/README | 2 +-
>> pc-bios/palcod
Il 18/07/2013 16:09, Peter Lieven ha scritto:
> On 18.07.2013 15:52, Paolo Bonzini wrote:
>> Il 18/07/2013 15:29, Peter Lieven ha scritto:
> If the driver would have a better method of writing zeroes than
> discard it simply should not set bdi->write_zeroes_w_discard = 1.
If the driver
Marcel Apfelbaum writes:
> Categorize devices that appear as output to "-device ?" command
> by logical functionality. Sort the devices by logical categories
> before showing them to user.
>
> Signed-off-by: Marcel Apfelbaum
> Reviewed-by: Kevin Wolf
> ---
> include/hw/qdev-core.h | 7 +++
On 18.07.2013 16:20, Paolo Bonzini wrote:
Il 18/07/2013 16:09, Peter Lieven ha scritto:
On 18.07.2013 15:52, Paolo Bonzini wrote:
Il 18/07/2013 15:29, Peter Lieven ha scritto:
If the driver would have a better method of writing zeroes than
discard it simply should not set bdi->write_zeroes_w_d
Il 18/07/2013 16:32, Peter Lieven ha scritto:
>>>
>> (Mis)alignment and granularity can be handled later. We can ignore them
>> for now. Later, if we decide the best way to support them is a flag,
>> we'll add it. Let's not put the cart before the horse.
>>
>> BTW, I expect alignment!=0 to be re
On Thu, 2013-07-18 at 09:28 -0500, Anthony Liguori wrote:
> Marcel Apfelbaum writes:
>
> > Categorize devices that appear as output to "-device ?" command
> > by logical functionality. Sort the devices by logical categories
> > before showing them to user.
> >
> > Signed-off-by: Marcel Apfelbaum
Il 18/07/2013 16:42, Marcel Apfelbaum ha scritto:
> On Thu, 2013-07-18 at 09:28 -0500, Anthony Liguori wrote:
>> Marcel Apfelbaum writes:
>>
>>> Categorize devices that appear as output to "-device ?" command
>>> by logical functionality. Sort the devices by logical categories
>>> before showing t
Il 18/07/2013 15:59, Laszlo Ersek ha scritto:
> We're going to need more state while processing a list of repeated
> options. This change eliminates "repeated_opts_first" and adds a new state
> variable:
>
> list_mode repeated_opts repeated_opts_first
> -- - --
Il 18/07/2013 15:59, Laszlo Ersek ha scritto:
> The new modes are equal-rank, exclusive sub-modes of LM_IN_PROGRESS. Teach
> opts_next_list(), opts_type_int() and opts_type_uint64() to handle them.
Perhaps you could use a bitmap then:
LM_NONE = 0
LM_STARTED = 1
LM_IN_PROGRESS = 2
Paolo Bonzini writes:
> Il 18/07/2013 16:42, Marcel Apfelbaum ha scritto:
>> On Thu, 2013-07-18 at 09:28 -0500, Anthony Liguori wrote:
>>> Marcel Apfelbaum writes:
>>>
Categorize devices that appear as output to "-device ?" command
by logical functionality. Sort the devices by logical
From: KONRAD Frederic
This bases rt_clock on icount, as vm_clock.
So vm_clock = rt_clock.
Signed-off-by: KONRAD Frederic
---
qemu-timer.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/qemu-timer.c b/qemu-timer.c
index b2d95e2..6c607e5 100644
--- a/qemu-timer.c
+++ b/
From: KONRAD Frederic
Hi everybody,
As I said in the last email, we have issues with determinism with icount.
We are wondering if determinism is really ensured with icount?
We saw that the rt_clock is used at multiple place which is a pain for
replaying the simulation the same way. So we simply
On Thu, 18 Jul 2013 09:55:51 +0800
Wenchao Xia wrote:
> 于 2013-7-18 3:34, Luiz Capitulino 写道:
> > On Thu, 11 Jul 2013 11:13:41 +0800
> > Wenchao Xia wrote:
> >
> >> Now all completion functions do not use *cur_mon any more, instead
> >> they use rs->mon. In short, structure ReadLineState decide
From: KONRAD Frederic
We don't want vm_clock to be synchronized with rt_clock as it is
not deterministic for replay.
Signed-off-by: KONRAD Frederic
---
cpus.c | 11 +--
main-loop.c | 5 +
2 files changed, 14 insertions(+), 2 deletions(-)
diff --git a/cpus.c b/cpus.c
index 46
On 07/18/2013 07:56 AM, Paolo Bonzini wrote:
> Il 18/07/2013 10:27, Marcel Apfelbaum ha scritto:
>> Running qemu with "-device ?" option returns ~145 lines.
>> It is hard to manage understanding the output.
>>
>> Theses patches aim to partially solve the problem by dividing the devices
>> into logi
From: KONRAD Frederic
This creates a new icount based timer, with no bias.
It moves only with the instruction counter.
Signed-off-by: KONRAD Frederic
---
cpus.c | 10 --
include/qemu/timer.h | 4
qemu-timer.c | 6 ++
3 files changed, 18 insertions(+),
On 10/07/2013 15:45, Paolo Bonzini wrote:
Il 08/07/2013 19:32, Frederic Konrad ha scritto:
Hi everybody,
We get some issues with reverse execution caused by indeterminism.
Something catched our attention:
static void icount_warp_rt(void *opaque), cpus.c:276
We have the feeling that icount is
On 18 July 2013 16:02, wrote:
> As I said in the last email, we have issues with determinism with icount.
> We are wondering if determinism is really ensured with icount?
My opinion is that it *should* be deterministic but it would
be unsurprising if the determinism had got broken along the way.
On 18 July 2013 16:02, wrote:
> @@ -156,6 +156,12 @@ void restore_icount(CPUArchState *env, int save)
> /* Return the virtual CPU time, based on the instruction counter. */
> int64_t cpu_get_icount(void)
> {
> +return qemu_icount_bias + cpu_get_icount_wo_bias();
> +}
> +
> +/* Return the
On 18/07/2013 17:06, Peter Maydell wrote:
On 18 July 2013 16:02, wrote:
As I said in the last email, we have issues with determinism with icount.
We are wondering if determinism is really ensured with icount?
My opinion is that it *should* be deterministic but it would
be unsurprising if the
On 07/18/2013 07:16 AM, Peter Maydell wrote:
> And can we have it and the existing qemu-palcode.git
> live on git.qemu.org like all our other rom submodules?
That I can't help you with. I did at least move it to github...
r~
1 - 100 of 214 matches
Mail list logo
