On 9/7/15, Paolo Bonzini wrote:
>
>
> On 03/09/2015 19:37, Salva Peiró wrote:
>> The file memory.c directly calls the function pointers provided in
>> the MemoryRegionOps to handle read and write operations for memory
>> regions.
>> The function pointers are called without checking if the function
On Tue, Sep 08, 2015 at 04:08:06PM +1000, Michael Ellerman wrote:
> On Wed, 2015-08-12 at 10:53 +0530, Bharata B Rao wrote:
> > On Tue, Aug 11, 2015 at 03:48:26PM +0200, Andrea Arcangeli wrote:
> > > Hello Bharata,
> > >
> > > On Tue, Aug 11, 2015 at 03:37:29PM +0530, Bharata B Rao wrote:
> > > >
Hi Christian:
"-no-shutdown" works fine for s390. thanks a lot for your suggestion.
I added the parameter of "-no-shutdown" only for s390-ccw-virtio as below,
_make_test_img $IMG_SIZE
+
+case "$QEMU_DEFAULT_MACHINE" in
+ s390-ccw-virtio)
+ platform_parm="-no-shutdown"
+ ;;
+ *
Peter, are you willing to commit this directly as a build fix? Or would
you prefer a pull request?
> On 02/09/2015 07:40, Amit Shah wrote:
> >> The buffer_find_nonzero_offset() will be called to check the zero
> >> page
> >> > during live migration, it's a hot function.
> >> > buffer_find_nonzero_offset() has already been optimized with SSE2
> >> > instructions, for platform that supports AVX2,
On Wed, 2015-08-12 at 10:53 +0530, Bharata B Rao wrote:
> On Tue, Aug 11, 2015 at 03:48:26PM +0200, Andrea Arcangeli wrote:
> > Hello Bharata,
> >
> > On Tue, Aug 11, 2015 at 03:37:29PM +0530, Bharata B Rao wrote:
> > > May be it is a bit late to bring this up, but I needed the following fix
> > >
This was just a resend of V2.
I think this is a good idea to move sd.h and sdhci.h to include/hw/sd/
As these device models like sdhci can be easily instantiated, when we have the
type and its object declaration available in header in includes.
Regards,
Sai
> -Original Message-
> From:
On 07/21/2015 01:45 AM, Max Reitz wrote:
> And a helper function for that, which directly takes a pointer to the
> BDS to be inserted instead of its node-name (which will be used for
> implementing 'change' using blockdev-insert-medium).
Is it OK to insert a medium to more than one BB?
Thanks
Wen
Move sdhci.h to include/hw/sd/. Which makes easy creation of device
using object_initialize.
Signed-off-by: Sai Pavan Boddu
---
Changes for V2:
Create new area in includes for sd. And move sdhci.h to same.
---
hw/sd/sdhci.c | 2 +-
{hw => include/hw}/sd/sdhci.h | 0
2 files ch
Create new folder in includes for sd headers.
Signed-off-by: Sai Pavan Boddu
---
hw/sd/milkymist-memcard.c | 2 +-
hw/sd/omap_mmc.c | 2 +-
hw/sd/pl181.c | 2 +-
hw/sd/pxa2xx_mmci.c | 2 +-
hw/sd/sd.c| 2 +-
hw/sd/sdhci.h | 2 +-
hw/sd/ssi-s
On 08/09/15 07:03, Sam Bobroff wrote:
> On Tue, Sep 01, 2015 at 12:53:26PM +0200, Thomas Huth wrote:
>> On 01/09/15 02:38, David Gibson wrote:
>>> On Mon, Aug 31, 2015 at 08:46:01PM +0200, Thomas Huth wrote:
From: Michael Ellerman
Some powerpc systems have support for a hardware ran
On Mon, Sep 7, 2015 at 5:45 PM, Markus Armbruster wrote:
> A feature new in Python 2.7 crept into commit 77e703b: re.subn()'s
> fifth argument. Avoid that, use re.compile().
>
> Reported-by: Laurent Desnogues
> Signed-off-by: Markus Armbruster
Tested-by: Laurent Desnogues
Works fine here. Th
On 09/07/2015 10:31 AM, Peter Maydell wrote:
-if (cond < 0x0e) { /* continue */
-gen_set_label(label_continue);
+/* If COND was false, force the flags to #nzcv.
+ Note that T1 = (COND ? 0 : -1), T2 = (COND ? -1 : 0). */
+tcg_t1 = tcg_temp_new_i32();
+tcg_t2 = tcg_te
On Tue, Sep 08, 2015 at 03:03:16PM +1000, Sam Bobroff wrote:
> On Tue, Sep 01, 2015 at 12:53:26PM +0200, Thomas Huth wrote:
> > On 01/09/15 02:38, David Gibson wrote:
> > > On Mon, Aug 31, 2015 at 08:46:01PM +0200, Thomas Huth wrote:
> > >> From: Michael Ellerman
> > >>
> > >> Some powerpc systems
On 09/07/2015 09:57 AM, Peter Maydell wrote:
On 2 September 2015 at 18:57, Richard Henderson wrote:
This is a bug fix for aarch64. At present, we have branches using
the 32-bit (translate.c) versions of cpu_[NZCV]F, but we set the flags
using the 64-bit (translate-a64.c) versions of cpu_[NZCV]
On 09/07/2015 10:17 AM, Peter Maydell wrote:
+c64->value = tcg_temp_new_i64();
+c64->cond = c32.cond;
+if (c32.cond == TCG_COND_EQ || c32.cond == TCG_COND_NE) {
+tcg_gen_extu_i32_i64(c64->value, c32.value);
+} else {
+tcg_gen_ext_i32_i64(c64->value, c32.value);
+
On 09/07/2015 10:09 AM, Peter Maydell wrote:
On 2 September 2015 at 18:57, Richard Henderson wrote:
+case 9: /* ls: !C || Z -> !(C && !Z) */
+cond = TCG_COND_NE;
+value = tcg_temp_new_i32();
+global = false;
+tcg_gen_neg_i32(value, cpu_CF);
+tcg_gen_a
On Tue, Sep 01, 2015 at 12:53:26PM +0200, Thomas Huth wrote:
> On 01/09/15 02:38, David Gibson wrote:
> > On Mon, Aug 31, 2015 at 08:46:01PM +0200, Thomas Huth wrote:
> >> From: Michael Ellerman
> >>
> >> Some powerpc systems have support for a hardware random number generator
> >> (hwrng). If suc
On 09/07/2015 04:16 AM, Markus Armbruster wrote:
> The QAPI code generators work with a syntax tree (nested dictionaries)
> plus a few symbol tables (also dictionaries) on the side.
>
> They have clearly outgrown these simple data structures. There's lots
> of rummaging around in dictionaries, an
In some cases, we need to disable copy-on-read, and just
read the data.
Signed-off-by: Wen Congyang
---
block/io.c| 12 +++-
include/block/block.h | 9 ++---
trace-events | 1 +
3 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/block/io.c b/block/
We will copy data in before_write_notifier to do backup.
It is a nested I/O request, so we cannot do copy-on-read.
The steps to reproduce it:
1. -drive copy-on-read=on,... // qemu option
2. drive_backup -f disk0 /path_to_backup.img // monitor command
Signed-off-by: Wen Congyang
Tested-by: Jeff
If we do copy-on-read in before_write_notifier, qemu will be crashed. The
backtrace is:
Program terminated with signal 6, Aborted.
#0 0x7f5d69207b55 in raise () from /lib64/libc.so.6
(gdb) bt
#0 0x7f5d69207b55 in raise () from /lib64/libc.so.6
#1 0x7f5d69209131 in abort () from /lib6
On 2015/9/7 22:23, Leif Lindholm wrote:
> The DBG2 table can be considered a "companion" to SPCR - it points out
> debug consoles available in the system.
>
> Signed-off-by: Leif Lindholm
> ---
> include/hw/acpi/acpi-defs.h | 37 +++--
> 1 file changed, 35 inser
On 2015/9/7 22:23, Leif Lindholm wrote:
> The Debug Port Table 2 (DBG2) is mandated by the ARM Server Base Boot
> Requirements specification. Add the DBG2 table definitions, and set up
> an entry in the ARM virt machine for the pl011 UART.
>
Looking at Documentation/arm64/acpi_object_usage.txt
On 09/07/2015 09:45 AM, Markus Armbruster wrote:
> A feature new in Python 2.7 crept into commit 77e703b: re.subn()'s
> fifth argument. Avoid that, use re.compile().
>
> Reported-by: Laurent Desnogues
> Signed-off-by: Markus Armbruster
> ---
> scripts/qapi.py | 4 +++-
> 1 file changed, 3 inse
On Sun, Sep 06, 2015 at 04:14:25PM +1000, Alexey Kardashevskiy wrote:
> On 09/04/2015 02:48 PM, David Gibson wrote:
> >On Wed, Sep 02, 2015 at 07:29:58PM +1000, Alexey Kardashevskiy wrote:
> >>For setting debug watchpoints, sPAPR guests use H_SET_MODE hypercall.
> >>The existing QEMU H_SET_MODE han
On Mon, Sep 07, 2015 at 11:37:04AM +0530, Bharata B Rao wrote:
> When drmgr is run in the guest to add a device for which device_add
> hasn't been issued in QEMU, configure-connector call fails.
> When configure-connector call fails, the guest would release (*)
> the previously acquired DRC by sett
On Mon, Sep 07, 2015 at 05:05:48PM +0200, Thomas Huth wrote:
> On 01/09/15 02:47, David Gibson wrote:
> > On Mon, Aug 31, 2015 at 08:46:02PM +0200, Thomas Huth wrote:
> >> The PAPR interface provides a hypercall to pass high-quality
> >> hardware generated random numbers to guests. So let's provide
On 09/08/2015 12:56 AM, Dr. David Alan Gilbert wrote:
> * Eric Blake (ebl...@redhat.com) wrote:
>> On 09/02/2015 02:51 AM, Wen Congyang wrote:
>>> If the child is not ready, read/write/getlength/flush will
>>> return -errno. It is not critical error, and can be ignored:
>>> 1. read/write:
>>>Ju
Hello everybody,
the project implements simple CAN SJA1000 interface emulation
with option to connect emulated CAN buses to host system
SocketCAN bus. Project has been started by Jin Yang in frame
of RTEMS provided GSoC 2013 slot.
Code has been cleaned and is maintained to be up-to
date with actu
Hi Jan,
I am interested in participating in next round of Outreachy program
with AMD IOMMU emulation project.
I have worked on BIOS projects which includes coreboot SeaBios etc and
bootloaders like u-boot and grub. I have experience of working with
qemu and feel that this project is the right ma
Hi,
Please find attached a couple of typo fixes, mostly in comments/docs.
(Please cc me directly, I am not on the list.)
Best regards,
--
Lajos Veres
vla...@gmail.com
0792 7460 778From 56d869ab666a51106eefa398ed404f4272b4db8b Mon Sep 17 00:00:00 2001
From: Veres Lajos
Date: Fri, 7 Aug 2015 23:0
On Mon, Sep 7, 2015 at 2:11 PM, Paolo Bonzini wrote:
>
>
> On 07/09/2015 23:02, Michael Marineau wrote:
>> ping http://patchwork.ozlabs.org/patch/510065/
>
> Just back from vacation, this is queued in my next pull request
> (currently at tags/for-upstream on github.com/bonzini/qemu.git, but I
> ha
On 07/09/2015 23:02, Michael Marineau wrote:
> ping http://patchwork.ozlabs.org/patch/510065/
Just back from vacation, this is queued in my next pull request
(currently at tags/for-upstream on github.com/bonzini/qemu.git, but I
haven't finished testing it).
Paolo
> Am 07.09.2015 um 21:14 schrieb Paolo Bonzini :
>
>
>
>> On 07/09/2015 21:11, Peter Lieven wrote:
>> It helps, but I have a small issue when my backport of the patch
>> is applied.
>>
>> I launch qemu witch a cmdline like this to probe for enforcable CPU types.
>>
>> echo quit | qemu -enabl
On Sun, Aug 23, 2015 at 5:57 PM, Michael Marineau
wrote:
> Using ccache with CCACHE_BASEDIR set to $(SRC_PATH) or a parent will
> rewrite all absolute paths to relative paths. This interacts poorly with
> QEMU's two-level build directory scheme. For example, lets say
> BUILD_DIR=$(SRC_PATH)/build
Quoting Paolo Bonzini (2015-09-07 15:39:39)
>
>
> On 07/09/2015 22:37, Michael Roth wrote:
> > This kinda does it, although it introduces a circular dependency warning
> > on posix (since qemu-ga$(EXESUF) == qemu-ga there, yet qemu-ga has
> > a qemu-ga$(EXESUF) dependency.
> >
> > Could be fixed
On 07/09/2015 22:37, Michael Roth wrote:
> This kinda does it, although it introduces a circular dependency warning
> on posix (since qemu-ga$(EXESUF) == qemu-ga there, yet qemu-ga has
> a qemu-ga$(EXESUF) dependency.
>
> Could be fixed by making a new 'qemu-guest-agent' do-the-right-thing targe
Quoting Michael Roth (2015-09-07 14:55:47)
> Quoting Paolo Bonzini (2015-09-07 05:48:27)
> >
> >
> > On 28/08/2015 01:55, Michael Roth wrote:
> > > Now we can build qemu-ga MSI package with:
> > > ./configure ...
> > > make qemu-ga.exe
> > > make msi
> > >
> > > or simply:
> > > ./config
On 19/06/2015 12:56, Alexandre Derumier wrote:
> This adds "--enable-jemalloc" and "--disable-jemalloc" to allow linking
> to jemalloc memory allocator.
>
> We have already tcmalloc support,
> but it seem to not working well with a lot of iothreads/disks.
>
> The main problem is that tcmalloc u
Quoting Paolo Bonzini (2015-09-07 05:48:27)
>
>
> On 28/08/2015 01:55, Michael Roth wrote:
> > Now we can build qemu-ga MSI package with:
> > ./configure ...
> > make qemu-ga.exe
> > make msi
> >
> > or simply:
> > ./configure ...
> > make msi
>
> Shouldn't the latter have always work
On 07.09.2015 15:26, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> tests/qemu-iotests/137 | 145
> +
> tests/qemu-iotests/137.out | 42 +
> tests/qemu-iotests/group | 1 +
> 3 files changed, 188 insertions(+)
> create mode
On 07.09.2015 15:26, Kevin Wolf wrote:
> For updating the cache sizes or disabling lazy refcounts
And updating the cache clean timer, too.
> there is a bit
> more to do than just changing the variables, but otherwise we're all set
> for cha
On 07.09.2015 15:26, Kevin Wolf wrote:
> 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
> ---
> block/qcow2.c | 113
> +-
>
On 07.09.2015 15:26, Kevin Wolf wrote:
> 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
> ---
> block/qcow2.c | 134
> +-
>
On 07.09.2015 15:26, Kevin Wolf wrote:
> 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
> ---
> block/qcow2.c | 57 +++-
On 07/09/2015 21:11, Peter Lieven wrote:
> It helps, but I have a small issue when my backport of the patch
> is applied.
>
> I launch qemu witch a cmdline like this to probe for enforcable CPU types.
>
> echo quit | qemu -enable-kvm -monitor stdio -nodefaults -nographic -cpu
> Haswell,enforce
Am 07.09.2015 um 18:28 schrieb Paolo Bonzini:
>
> On 03/09/2015 15:11, Peter Lieven wrote:
>> edec47c main-loop: fix qemu_notify_event for aio_notify optimization
> Part of the above AioContext series.
So either the whole series or none of them I guess?
>>> It's a separate bug, and the
On 07.09.2015 17:14, Kevin Wolf wrote:
> BDRVQcowState is already used by qcow1, and gdb is always confused which
> one to use. Rename the qcow2 one so they can be distinguished.
>
> Signed-off-by: Kevin Wolf
> ---
> block/qcow2-cache.c| 14 -
> block/qcow2-cluster.c | 48 ++
On 07.09.2015 07:53, Wen Congyang wrote:
> On 09/02/2015 11:02 PM, Eric Blake wrote:
>> On 07/20/2015 11:45 AM, Max Reitz wrote:
>>> First of all: Thank you, Eric and Berto, for reviewing v3! And thank
>>> you, Fam, for at least having a peek at it and being confident enough to
>>> base a series of
On 6 September 2015 at 19:37, Peter Crosthwaite
wrote:
> This could still cause the same issue here though I think. The guest
> can release first (case 5) without a corresponding fifo pop (case 3),
> which actually means that the first patch idea might be the winner as
> it catches this issue too.
On Mon, Sep 7, 2015 at 9:42 AM, Peter Maydell wrote:
> On 7 September 2015 at 17:40, Markus Armbruster wrote:
>> Peter Maydell writes:
>>
>>> Convert the pxa2xx_mmci device to be a sysbus device.
>
>>> +static Property pxa2xx_mmci_properties[] = {
>>> +/* Note: pointer property 'drive' may r
Am 07.09.2015 um 20:04 hat Max Reitz geschrieben:
> On 07.09.2015 20:03, Kevin Wolf wrote:
> > Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
> >> bdrv_is_inserted() should be invoked recursively on the children of
> >> quorum.
> >>
> >> Signed-off-by: Max Reitz
> >> Reviewed-by: Eric Blake
>
Fix compile time warnings, because of type mismatch for unsigned long
long type.
Signed-off-by: Sai Pavan Boddu
Reviewed-by: Peter Crosthwaite
---
Changes for V3:
Same as V2.
Changes for V2:
Fix commit message.
Correct line lenght.
---
hw/sd/sdhci.c | 7 +--
1 file changed, 5 in
Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
> The tray of an FDD is open iff there is no medium inserted (there are
> only two states for an FDD: "medium inserted" or "no medium inserted").
>
> This results in the tray being reported as open if qemu has been started
> with the default floppy
On Mon, Sep 7, 2015 at 2:21 AM, Peter Maydell wrote:
> CCing the net maintainers on this thread seems like it would
> be a good idea...
>
> On 7 September 2015 at 08:47, Richard Purdie
> wrote:
>> On Sun, 2015-09-06 at 17:48 -0700, Peter Crosthwaite wrote:
>>> This doesn't sound right. There are
On 2 September 2015 at 18:57, Richard Henderson wrote:
> Usually, eliminate an operation from the translator by combining
> a shift with an extract.
>
> In the case of gen_set_NZ64, we don't need a boolean value for cpu_ZF,
> merely a non-zero value. Given that we can extract both halves of a
> 6
> -Original Message-
> From: Peter Crosthwaite [mailto:crosthwaitepe...@gmail.com]
> Sent: Monday, September 07, 2015 10:52 PM
> To: Sai Pavan Boddu; qemu-triv...@nongnu.org
> Cc: qemu-devel@nongnu.org Developers; Eric Blake; Peter Maydell; Alistair
> Francis; Edgar Iglesias; Sai Pavan Bo
On Mon, Sep 7, 2015 at 12:09 AM, Richard Purdie
wrote:
> On Sun, 2015-09-06 at 17:48 -0700, Peter Crosthwaite wrote:
>> On Sun, Sep 6, 2015 at 4:26 PM, Richard Purdie
>> wrote:
>> > On Sun, 2015-09-06 at 11:37 -0700, Peter Crosthwaite wrote:
>> > It seems can_receive isn't enough, we'd need to pu
On 2 September 2015 at 18:57, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target-arm/translate-a64.c | 33 +
> 1 file changed, 21 insertions(+), 12 deletions(-)
>
Reviewed-by: Peter Maydell
thanks
-- PMM
Conditional compilation hides few type mismatch warnings, fix it to
compile unconditionally.
Signed-off-by: Sai Pavan Boddu
Suggested-by: Eric Blake
Reviewed-by: Peter Crosthwaite
---
Changes for V4:
fix the commit message mistakes.
---
hw/sd/sdhci.c | 36 ++
On 07.09.2015 20:03, Kevin Wolf wrote:
> Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
>> bdrv_is_inserted() should be invoked recursively on the children of
>> quorum.
>>
>> Signed-off-by: Max Reitz
>> Reviewed-by: Eric Blake
>> Reviewed-by: Alberto Garcia
>
> If you implement patch 6 like
Fix compile time warnings, because of type mismatch for unsigned long
long type.
Signed-off-by: Sai Pavan Boddu
Reviewed-by: Peter Crosthwaite
---
Changes for V3:
Same as V2.
Changes for V2:
Fix commit message.
Correct line lenght.
---
hw/sd/sdhci.c | 7 +--
1 file changed, 5 in
Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
> bdrv_is_inserted() should be invoked recursively on the children of
> quorum.
>
> Signed-off-by: Max Reitz
> Reviewed-by: Eric Blake
> Reviewed-by: Alberto Garcia
If you implement patch 6 like I suggested, this one could be dropped.
Kevin
On 07.09.2015 19:43, Kevin Wolf wrote:
> Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
>> If bdrv_is_inserted() is called on the top level BDS, it should make
>> sure all nodes in the BDS tree are actually inserted.
>>
>> Signed-off-by: Max Reitz
>> Reviewed-by: Eric Blake
>> Reviewed-by: Alb
On 2 September 2015 at 18:57, Richard Henderson wrote:
> For !SF, this initial ext32u can't be optimized away by the
> current TCG code generator. (It would require backward bit
> liveness propagation.)
>
> But since the range of bits for !SF are already constrained by
> unallocated_encoding, we'
On 2 September 2015 at 18:57, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target-arm/translate-a64.c | 17 +
> 1 file changed, 17 insertions(+)
>
> diff --git a/target-arm/translate-a64.c b/target-arm/translate-a64.c
> index 74dd0f8..8c94edf 100644
> --- a
On 2 September 2015 at 18:57, Richard Henderson wrote:
> ... as aliases of SBFM, and special-case them.
>
> Signed-off-by: Richard Henderson
> ---
Reviewed-by: Peter Maydell
(Not a fan of commit messages that start a sentence in the Subject
and continue it in the body.)
thanks
-- PMM
Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
> If bdrv_is_inserted() is called on the top level BDS, it should make
> sure all nodes in the BDS tree are actually inserted.
>
> Signed-off-by: Max Reitz
> Reviewed-by: Eric Blake
> Reviewed-by: Alberto Garcia
> ---
> block.c | 7 +++
> 1
On 2 September 2015 at 18:57, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target-arm/translate-a64.c | 48
> --
> 1 file changed, 21 insertions(+), 27 deletions(-)
>
> diff --git a/target-arm/translate-a64.c b/target-arm/transl
On Thu, 3 Sep 2015, Juergen Gross wrote:
> Add a backend for para-virtualized USB devices for xen domains.
>
> The backend is using host-libusb to forward USB requests from a
> domain via libusb to the real device(s) passed through.
>
> Signed-off-by: Juergen Gross
Aside from a few minor commen
On 2 September 2015 at 18:57, Richard Henderson wrote:
> This can allow much of a ccmp to be elided when particular
> flags are subsequently dead.
>
> Signed-off-by: Richard Henderson
> ---
> target-arm/translate-a64.c | 65
> +++---
> 1 file changed, 44
On Mon, Sep 7, 2015 at 2:54 AM, Paolo Bonzini wrote:
>
>
> On 16/08/2015 01:28, Peter Crosthwaite wrote:
>> Every arch defines ELF_MACHINE in cpu.h to an arch-specific value. This
>> definition is rarely needed by core code (only in a few cases). It
>> conflicts with the multi-arch effort where cp
On Sun, Sep 6, 2015 at 11:45 PM, Sai Pavan Boddu
wrote:
> Conditionaly compilation hides few type mismatch warnings, fix it to
"conditional"
> compile unconditioinal.
>
"unconditionally"
> Signed-off-by: Sai Pavan Boddu
> Suggested-by: Eric Blake
Otherwise:
Reviewed-by: Peter Crosthwaite
On 2 September 2015 at 18:57, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target-arm/translate-a64.c | 87
> +++---
> 1 file changed, 51 insertions(+), 36 deletions(-)
>
> diff --git a/target-arm/translate-a64.c b/target-arm/transl
On 2 September 2015 at 18:57, Richard Henderson wrote:
> Handling this with TCG_COND_ALWAYS will allow these unlikely
> cases to be handled without special cases in the rest of the
> translator. The TCG optimizer ought to be able to reduce
> these ALWAYS conditions completely.
>
> Signed-off-by:
On 2 September 2015 at 18:57, Richard Henderson wrote:
> Split arm_gen_test_cc into 3 functions, so that it can be reused
> for non-branch TCG comparisons.
>
> Signed-off-by: Richard Henderson
> ---
> target-arm/translate.c | 110
> -
> target-arm
Am 07.09.2015 um 14:04 schrieb Markus Armbruster:
> Michael Tokarev writes:
>
>> 07.09.2015 09:34, Markus Armbruster wrote:
>>> Michael Tokarev writes:
>>>
Applied to -trivial-patches, unfortunately not for 2.4 but at least
for 2.5 :)
>>>
>>> Should we add "Cc: qemu-sta...@nongnu.org"
* Eric Blake (ebl...@redhat.com) wrote:
> On 09/02/2015 02:51 AM, Wen Congyang wrote:
> > If the child is not ready, read/write/getlength/flush will
> > return -errno. It is not critical error, and can be ignored:
> > 1. read/write:
> >Just not report the error event.
>
> What happens if all t
Paolo Bonzini writes:
> On 07/09/2015 17:23, Markus Armbruster wrote:
>> > Apart from copy-n-pasting, there is also the problem that you can run
>> > "checkpatch.pl -f" on a whole file ... it would also be ugly to suddenly
>> > have (much) more warnings here.
>>
>> Feature. If you run checkpatc
On 07/09/15 17:23, Markus Armbruster wrote:
> Thomas Huth writes:
>
>> On 07/09/15 16:22, Paolo Bonzini wrote:
>> [...]
>>> Some other interesting data:
>>>
>>> - However, only 605 files have 5 or more lines with 77+ columns, so the
>>> odds of warnings after copy-n-paste are pretty slim.
>>
>> A
Paolo Bonzini writes:
> On 07/09/2015 17:17, Markus Armbruster wrote:
>>> > + } elsif ($length > 76 && !($rawline =~ /^\+ \* /)) {
>>> > + # The BSD license blurb has 80 character lines.
>>> > + # Avoid warning on cut-and-pasted lice
On 2 September 2015 at 18:57, Richard Henderson wrote:
> This is a bug fix for aarch64. At present, we have branches using
> the 32-bit (translate.c) versions of cpu_[NZCV]F, but we set the flags
> using the 64-bit (translate-a64.c) versions of cpu_[NZCV]F. From
> the view of the TCG code genera
Peter Maydell writes:
> On 7 September 2015 at 17:40, Markus Armbruster wrote:
>> Peter Maydell writes:
>>
>>> Convert the pxa2xx_mmci device to be a sysbus device.
>
>>> +static Property pxa2xx_mmci_properties[] = {
>>> +/* Note: pointer property 'drive' may remain NULL, thus no need
>>> +
On 07.09.2015 18:20, Kevin Wolf wrote:
> Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
>> Tests 071 and 081 test giving references in blockdev-add. It is not
>> necessary to create a BlockBackend here, so omit it.
>>
>> Signed-off-by: Max Reitz
>> Reviewed-by: Eric Blake
>> Reviewed-by: Alber
On 7 September 2015 at 17:40, Markus Armbruster wrote:
> Peter Maydell writes:
>
>> Convert the pxa2xx_mmci device to be a sysbus device.
>> +static Property pxa2xx_mmci_properties[] = {
>> +/* Note: pointer property 'drive' may remain NULL, thus no need
>> + * for dc->cannot_instantiate
Peter Maydell writes:
> Convert the pxa2xx_mmci device to be a sysbus device.
>
> Signed-off-by: Peter Maydell
> ---
> hw/sd/pxa2xx_mmci.c | 96
> +++--
> 1 file changed, 79 insertions(+), 17 deletions(-)
>
> diff --git a/hw/sd/pxa2xx_mmci.c b/hw
On 07.09.2015 18:12, Kevin Wolf wrote:
> Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
>> If the "id" field is missing from the options given to blockdev-add,
>> just omit the BlockBackend and create the BlockDriverState tree alone.
>>
>> However, if "id" is missing, "node-name" must be specifi
On 26/08/2015 01:45, John Snow wrote:
> IDEState's io_buffer_offset was originally added to keep track of offsets
> in AHCI rather exclusively, but it was added to IDEState instead of an
> AHCI-specific structure.
Interesting to see something weird turn into feature. :)
Reviewed-by: Paolo Bonzi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
On 02/09/2015 00:22, Eric Blake wrote:
>> Do you think the file should go under docs/, not docs/specs?
>
> I don't know if there is a strong explanation for why we have the
> separation, but given your argument, docs/ (all qemu-related
> documen
On Mon, Sep 07, 2015 at 05:23:01PM +0100, Dr. David Alan Gilbert wrote:
> * Daniel P. Berrange (berra...@redhat.com) wrote:
> > This extends the TCP migration backend so that it can make use
> > of QIOChannelTLS to provide transparent TLS encryption. To
> > trigger enablement the URI on the incomin
Hello all,
For the 2.4 release, we tried a little experiment.
I asked maintainers who had pull reqs merged in the 2.4 release to
give me a few talking points for their subsystem via email. Since the
release also coincided with the QEMU Summit / KVM Forum, I got a
chance to meet a few contributor
On 03/09/2015 15:11, Peter Lieven wrote:
>>
> edec47c main-loop: fix qemu_notify_event for aio_notify optimization
Part of the above AioContext series.
>>> So either the whole series or none of them I guess?
>> It's a separate bug, and theoretically it's there in 2.3.1 as well, but
>> no
On 07.09.2015 17:59, Kevin Wolf wrote:
> Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
>> It has been deprecated as of 2.3, so we can now remove it.
>>
>> Signed-off-by: Max Reitz
>
>> @@ -2241,8 +2188,9 @@ static BlockAIOCB *hdev_aio_ioctl(BlockDriverState *bs,
>> pool = aio_get_thread_
* Daniel P. Berrange (berra...@redhat.com) wrote:
> This extends the TCP migration backend so that it can make use
> of QIOChannelTLS to provide transparent TLS encryption. To
> trigger enablement the URI on the incoming and outgoing sides
> should have 'tls-creds=ID' appended, eg
>
>tcp:$HOST
Am 20.07.2015 um 19:45 hat Max Reitz geschrieben:
> Tests 071 and 081 test giving references in blockdev-add. It is not
> necessary to create a BlockBackend here, so omit it.
>
> Signed-off-by: Max Reitz
> Reviewed-by: Eric Blake
> Reviewed-by: Alberto Garcia
> ---
> tests/qemu-iotests/071
On 26/08/2015 03:52, arei.gong...@huawei.com wrote:
> From: Gonglei
>
> vhost-scsi bootindex does't work because Qemu passes
> wrong fireware path to seabios.
>
> before:
> /pci@i0cf8/scsi@7channel@0/vhost-scsi@0,0
> after applying the patch:
> /pci@i0cf8/scsi@7/channel@0/vhost-scsi@0,0
>
git repository at:
>
> git://github.com/cohuck/qemu tags/s390x-20150907
>
> for you to fetch changes up to bd80a8ad555c2b5f79591b29edcf8196b8a5109b:
>
> s390/sclp: simplify calculation of rnmax (2015-09-07 16:10:44 +0200)
>
> ---
On Mon, Sep 07, 2015 at 05:08:17PM +0100, Dr. David Alan Gilbert wrote:
> * Daniel P. Berrange (berra...@redhat.com) wrote:
> > The test-vmstate.c file is the only remaining user of the
> > qemu_bufopen function. Half of the test cases already use
> > a temporary file, so convert the remaining case
On 07/09/2015 17:53, Alex Bennée wrote:
>> > With this change we make sure that the compiler will not
>> > optimise the read of the sequence number in any way.
> What was it doing? Using atomic_read to work around a compiler bug seems
> a bit heavy handed if true atomicity isn't needed.
This is
1 - 100 of 373 matches
Mail list logo
