Currently, devices are plugged before features are negotiated.
If the backend doesn't support VIRTIO_F_VERSION_1, the transport
need to rewind some settings.
This is the case for both PCI and CCW.
For CCW, a post_plugged callback had been introduced, where
max_rev field is just updated if VIRTIO_F
Pranith Kumar writes:
> Signed-off-by: Pranith Kumar
> ---
> tests/tcg/arm/Makefile| 45
> +++
> tests/tcg/{ => arm}/hello-arm.c | 0
> tests/tcg/{ => arm}/test-arm-iwmmxt.s | 0
> 3 files changed, 45 insertions(+)
> create mode 100644
On Sat, Sep 03, 2016 at 09:39:41PM +0100, Richard Henderson wrote:
> --- a/tcg/tcg.h
> +++ b/tcg/tcg.h
> @@ -1175,6 +1175,59 @@ uint64_t helper_be_ldq_cmmu(CPUArchState *env,
> target_ulong addr,
> # define helper_ret_ldq_cmmu helper_le_ldq_cmmu
> #endif
>
> +uint32_t helper_atomic_cmpxchgb_m
On 09/09/2016 12:02, Cao jin wrote:
> kvm_setup_guest_memory() also just does "madivse to QEMU_MADV_DONTFORK".
>
> Signed-off-by: Cao jin
> ---
>
> Though I don't understand these code well, tt seems is duplicate code to me.
> But I am not sure whether I am doing the right thing, so, RFC.
>
>
Alex Bennée writes:
> Nikunj A Dadhania writes:
>
> I think we need a little more detail here. In fact when you post the
> next version of the series could you please include a cover letter to
> cover what the series is trying to achieve?
Sure will do that.
>
>> Signed-off-by: Nikunj A Dadhani
Pranith Kumar writes:
> Signed-off-by: Pranith Kumar
> ---
> tests/tcg/misc/Makefile | 75
> +++
> tests/tcg/{ => misc}/linux-test.c | 3 ++
> tests/tcg/{ => misc}/sha1.c | 0
> tests/tcg/{ => misc}/test-mmap.c | 0
> tests/tcg/{ => misc
Hello Eric, Kevin, Paolo,
Thank you very much for the very fast help.
All works fine now!!!
Best regards and have nice weekend
Holger
Am 08.09.2016 um 19:34 schrieb Holger Schranz:
Hi Eric,
Am 08.09.2016 um 17:25 schrieb Eric Blake:
On 09/08/2016 09:27 AM, Holger Schranz wrote:
Hi Eric,
Hi Prem,
On 22/08/2016 18:17, Prem Mallappa wrote:
> Added ACPI IORT tables, was needed for internal project purpose, but
> posting here for anyone looking for testing ACPI on ARM platforms.
> (P.S: Linux side IORT patches are WIP)
I am also interested in IORT ITS group and currently prototyping
s
On Fri, 9 Sep 2016 05:10:27 -0400
Pradeep Jagadeesh wrote:
> Uses throttling APIs to limit I/O bandwidth and number of operations on the
> devices which use 9p-local driver.
>
> Signed-off-by: Pradeep Jagadeesh
> ---
Hi Pradeep,
Please find some remarks below. I haven't dived deep enough to
Hi Prem,
Missing commit message
> Signed-off-by: Prem Mallappa
> ---
> include/qemu/log.h | 1 +
> util/log.c | 2 ++
> 2 files changed, 3 insertions(+)
>
> diff --git a/include/qemu/log.h b/include/qemu/log.h
> index 234fa81..3dd2131 100644
> --- a/include/qemu/log.h
> +++ b/include/q
On Fri, Sep 09, 2016 at 04:14:59PM +0200, Maxime Coquelin wrote:
> Currently, devices are plugged before features are negotiated.
> If the backend doesn't support VIRTIO_F_VERSION_1, the transport
> need to rewind some settings.
>
> This is the case for both PCI and CCW.
> For CCW, a post_plugged
On Fri 09 Sep 2016 05:29:16 PM CEST, Greg Kurz wrote:
> On Fri, 9 Sep 2016 05:10:27 -0400
> Pradeep Jagadeesh wrote:
>
>> Uses throttling APIs to limit I/O bandwidth and number of operations on the
>> devices which use 9p-local driver.
>>
>> Signed-off-by: Pradeep Jagadeesh
>> ---
>
> Hi Prade
On Fri, 9 Sep 2016 17:29:16 +0200
Greg Kurz wrote:
> > +bool throttle9p_get_io_limits_state(FsThrottle *fst)
>
> The name looks a bit strange, since this helper simply returns a boolean flag.
> I guess throttle9p_enabled() is enough.
>
> > +{
> > +
> > +return fst->io_limits_enabled;
> > +
Hi Prem,
> SMMUv3 needs device tree entry like below
To me the commit message should be more explicit and mention appendprop
functionality
>
> interrupt-names = "gerror", "priq", "eventq", "cmdq-sync";
>
> This patch introduces helper function to add entries like above
>
> Signed-off-by: P
Pranith Kumar writes:
> Signed-off-by: Pranith Kumar
> ---
> tests/tcg/misc/Makefile | 75
> +++
> tests/tcg/{ => misc}/linux-test.c | 3 ++
> tests/tcg/{ => misc}/sha1.c | 0
> tests/tcg/{ => misc}/test-mmap.c | 0
> tests/tcg/{ => misc
On 09/09/2016 05:39 PM, Michael S. Tsirkin wrote:
On Fri, Sep 09, 2016 at 04:14:59PM +0200, Maxime Coquelin wrote:
> Currently, devices are plugged before features are negotiated.
> If the backend doesn't support VIRTIO_F_VERSION_1, the transport
> need to rewind some settings.
>
> This is the
"Dr. David Alan Gilbert" writes:
> * Daniel P. Berrange (berra...@redhat.com) wrote:
>> On Tue, Sep 06, 2016 at 11:18:06AM +0100, Dr. David Alan Gilbert (git) wrote:
>> > From: "Dr. David Alan Gilbert"
>> >
>> > This started off as Andreas Färber's implementation from
>> > March 2015, but after
Let's prepare for new Aspeed SoCs and rename the ast2400 file to a
more generic one. There are no changes in the code apart from the
header file include.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
hw/arm/Makefile.objs | 2 +-
The AST2500 SoC being very close to the AST2400 SoC, the goal of the
changes below is to modify the existing platform palmetto-bmc and the
AST2400 SoC to take into account the small differences and avoid code
duplication. This is mostly inspired by the realview and the rpi2
platforms.
First patche
This is a name replacement to prepare ground for other SoCs.
Let's also remove the AST2400_SMC_BASE definition from the address
space mappings, as it is not used. This controller was removed from
the Aspeed SoC AST2500, so this provides us a better common base for
the address space mapping on both
The ast2500 eval board has a hardware strapping register value of
0xF100C2E6 which we use for a definition of AST2500_EVB_HW_STRAP1
below.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
Changes since v3:
- fixed trailing comma.
hw/arm/aspeed.c |
This is mostly a name replacement to prepare ground for other SoCs
specificities. It also adds a TypeInfo struct for the palmetto-bmc
board with a custom initialization for the same reason.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
hw/arm/aspeed
aspeed_board_init() now uses a board identifier to customize some values
specific to the board.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
Changes since v3:
- added trailing comma to enum
Changes since v2:
- removed silicon-rev and cpu-mod
If the RAM size is invalid, the memory controller will use a default
value.
Signed-off-by: Cédric Le Goater
---
hw/arm/aspeed.c | 18 +-
1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
index 562bbb253391..6b18c7f1727c 100644
--- a/
This gives some explanation behind the magic number 0x120CE416.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
Changes since v3:
- added brackets around macro parameters
Changes since v2:
- more precise definitions of the hw-strap1 register
-
Based on previous work done by Andrew Jeffery .
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
Changes since v3:
- added brackets around macro parameters
- fixed trailing comma in enum
Changes since v2:
- more precise definitions of the hw-st
We plan to add more Aspeed boards to this file. There are no changes
in the code.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
hw/arm/Makefile.objs| 2 +-
hw/arm/{palmetto-bmc.c => aspeed.c} | 0
2 files changed, 1 insertion(+), 1 d
There is no need to do this at each reset as the RAM size will not
change.
Signed-off-by: Cédric Le Goater
---
hw/misc/aspeed_sdmc.c | 16 ++--
include/hw/misc/aspeed_sdmc.h | 1 +
2 files changed, 15 insertions(+), 2 deletions(-)
diff --git a/hw/misc/aspeed_sdmc.c b/hw/mis
On Fri, Sep 09, 2016 at 10:03:27AM -0400, Theodore Ts'o wrote:
> On Fri, Sep 09, 2016 at 11:19:25AM +0200, Jan Kara wrote:
> > > > > > 1. make the guest kernel based on your tree, the top commit is
> > > > > >10d7902fa0e82b (dax: unmap/truncate on device shutdown) and
> > > > > >the config
Also change the default value used in case of an error. The minimum
size is a bit severe, so let's just use an average RAM size.
Signed-off-by: Cédric Le Goater
---
hw/misc/aspeed_sdmc.c | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/hw/misc/aspeed_sdmc.c b
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
Reviewed-by: Peter Maydell
---
hw/arm/aspeed.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
index 2af9fe934441..9013d35a674c 100644
--- a/hw/arm/aspeed.c
+++ b/hw/arm/aspeed.c
@@ -145,7 +145,6 @@
Hi Prem,
> Default virt platform now creates SMMU device.
> Default config to build SMMU device along is in previous patches.
>
> Signed-off-by: Prem Mallappa
> ---
> hw/arm/virt.c | 62
> +++
> include/hw/arm/smmu.h | 33
On 09/09/2016 07:46 AM, Leon Alrae wrote:
Wouldn't it be useful if tcg.h provided also aliases for _le/_be atomic
helpers (equivalent to helper_ret_X_mmu) so that in target-* code we wouldn't
need to care about the endianness (specifically I'm thinking about SC in MIPS
where I need to select betw
Configure the size of the RAM of the SOC using a property to propagate
the value down to the memory controller from the board level.
Signed-off-by: Cédric Le Goater
---
hw/arm/aspeed.c | 2 ++
hw/arm/aspeed_soc.c | 2 ++
hw/misc/aspeed_sdmc.c | 23 +-
On Thu, 8 Sep 2016 21:00:07 +0200
Laurent Vivier wrote:
> Add a first test to validate the protocol:
>
> - rtas/get-time-of-day compares the time
> from the guest with the time from the host.
>
> Signed-off-by: Laurent Vivier
> ---
> v6:
> - rebase
>
> v5:
> - use qtest_spapr_boot() instea
Let's define an object class for each Aspeed SoC we support. A
AspeedSoCInfo struct gathers the SoC specifications which can later be
used by an instance of the class or by a board using the SoC.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
---
Changes since v4:
- fixed Aspeed
On 09/09/2016 04:10 PM, Maxime Coquelin wrote:
This patch adds virtio_test_backend_virtio_1() function to
check whether backend supports VIRTIO_F_VERSION_1 before the
negociation takes place.
Cc: Cornelia Huck
Cc: Marcel Apfelbaum
Cc: Michael S. Tsirkin
Cc: qemu-sta...@nongnu.org
Signed-off-b
On 09/09/2016 04:10 PM, Maxime Coquelin wrote:
This patch makes pci devices plugging more robust, by not confusing
guest with modern interface when the backend doesn't support
VIRTIO_F_VERSION_1.
Cc: Cornelia Huck
Cc: Marcel Apfelbaum
Cc: Michael S. Tsirkin
Cc: qemu-sta...@nongnu.org
Signed-o
Peter Xu writes:
> v4 changes:
> - remove two standard headers since they are included in osdep.h
> already [Fam]
> - make sure it passes build on all platforms (no --target-list
> specified during configure)
>
> v3 changes:
> - implement error_report_fatal using function [Markus]
> - provide
Richard Henderson writes:
> Signed-off-by: Richard Henderson
Reviewed-by: Alex Bennée
> ---
> exec.c| 4 ++--
> include/qemu/int128.h | 10 ++
> 2 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/exec.c b/exec.c
> index 8ffde75..373313d 100644
> ---
Richard Henderson writes:
> Signed-off-by: Richard Henderson
> ---
> include/qemu/int128.h | 135
> +-
> tests/test-int128.c | 22
> 2 files changed, 145 insertions(+), 12 deletions(-)
>
> diff --git a/include/qemu/int128.h b/include
On Fri, Sep 09, 2016 at 07:05:04PM +0200, Markus Armbruster wrote:
> Peter Xu writes:
>
> > v4 changes:
> > - remove two standard headers since they are included in osdep.h
> > already [Fam]
> > - make sure it passes build on all platforms (no --target-list
> > specified during configure)
> >
From: Pavel Fedin
If supported by the configuration, ITS will be added automatically.
This patch also renames v2m_phandle to msi_phandle because it's now used
by both MSI implementations.
Signed-off-by: Pavel Fedin
Signed-off-by: Eric Auger
Reviewed-by: Peter Maydell
--
v3 -> v4:
- added P
From: Eric Auger
Advertise gsi routing and set up irqchip routing entries for
GIC SPIs.
This is not mandated as long as MSI routing is not used
(because the kernel sets a default irqchip routing table).
However once MSI routing gets used (for VIRTIO-PCI vhost for
example), the first call to KVM_
Machine.c contains code related to migration. Let's move
gicv3_class_name to kvm_arm.h instead.
Signed-off-by: Eric Auger
Suggested-by: Peter Maydell
Reviewed-by: Peter Maydell
---
v5 -> v6:
- add Peter's R-b
v4 -> v5:
- add #include "qemu/error-report.h"
- rebased on target-arm: Fix unreach
This series introduces support for in-kernel GICv3 ITS emulation.
On dt guest the functionality is complete and was tested on Cavium ThunderX
with virtio-net-pci and vhost-net.
On ACPI guest the series was tested with virtio-net-pci only. For vhost-net,
using MSIX we currently miss the ACPI IORT
From: Pavel Fedin
Introduce global kvm_msi_use_devid flag and pass the device ID,
if needed, while building the MSI route entry. Device IDs are
required by the ARM GICv3 ITS (IRQ remapping function is based on
this information).
Signed-off-by: Pavel Fedin
Signed-off-by: Eric Auger
---
v5 -> v
From: Shannon Zhao
If GIC ITS is supported, add description in ACPI MADT table, then guest
could use ITS when booting with ACPI.
Signed-off-by: Shannon Zhao
---
hw/arm/virt-acpi-build.c | 12
1 file changed, 12 insertions(+)
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acp
From: Pavel Fedin
This is the basic skeleton for both KVM and software-emulated ITS.
Since we already prepare status structure, we also introduce complete
VMState description. But, because we currently have no migratable
implementations, we also set unmigratable flag.
Signed-off-by: Pavel Fedin
On Fri, Sep 09, 2016 at 06:21:15PM +0200, Markus Armbruster wrote:
> "Dr. David Alan Gilbert" writes:
>
> > * Daniel P. Berrange (berra...@redhat.com) wrote:
> >> IIUC, you switched because string-output-visitor could not handle
> >> complex types.
> >>
> >> I have previously written a text-outp
From: Shannon Zhao
ACPI Spec 6.0 introduces GIC Interrupt Translation Service Structure.
Here we add the definition of the Structure.
Signed-off-by: Shannon Zhao
---
include/hw/acpi/acpi-defs.h | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/include/hw/acpi/ac
From: Pavel Fedin
The ITS control frame is in-kernel emulated while accesses to the
GITS_TRANSLATER are mediated through the KVM_SIGNAL_MSI ioctl (MSI
direct MSI injection advertised by the CAP_SIGNAL_MSI capability)
the kvm_gsi_direct_mapping is explicitly set to false to emphasize the
differen
Introduce qemu_sztostr which takes an int and turns it
into a sized string. Variants are added for both signed
and unsigned integers.
Signed-off-by: Daniel P. Berrange
---
include/qemu/cutils.h | 12 +++
tests/test-cutils.c | 96 +++
util/cut
This is an update of the bits of this previous
series which were not merged
https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg01723.html
The problem addressed in this series is that the
'qemu-img info' command does not have a stable
output format for the image specific info objects.
The
On 09/09/2016 10:19 AM, Alex Bennée wrote:
Richard Henderson writes:
Signed-off-by: Richard Henderson
---
include/qemu/int128.h | 135 +-
tests/test-int128.c | 22
2 files changed, 145 insertions(+), 12 deletions(-)
diff --git a/i
The current approach for pretty-printing QAPI types is to
convert them to JSON using the QMP output visitor and then
pretty-print the JSON document. This has an unfixable problem
that structs get their keys printed out in random order, since
JSON dicts do not contain any key ordering information.
Not all visitor implementations supply the full set of
visitor callback functions. For example, the string
output visitor does not provide 'start_struct' and
friends. If you don't know this and feed it an object
that uses structs, you'll get a crash:
Segmentation fault (core dumped)
Crashing is
When 'qemu-img info' prints out format specific information,
it first converts the QAPI object into a JSON based QObject
data structure. Unfortunately structs have to be turned into
dicts, which looses all information about field ordering,
so the data printed appears in a semi-random order.
Conver
Instead inlining code for formatting sized integers,
call out to the new qemu_szutostr function.
Signed-off-by: Daniel P. Berrange
---
qapi/string-output-visitor.c | 20 +---
tests/test-string-output-visitor.c | 22 ++
2 files changed, 27 insertions(+),
On Fri, Sep 09, 2016 at 03:10:07PM +0200, Maxime Coquelin wrote:
> This patch makes pci devices plugging more robust, by not confusing
> guest with modern interface when the backend doesn't support
> VIRTIO_F_VERSION_1.
>
> Cc: Cornelia Huck
> Cc: Marcel Apfelbaum
> Cc: Michael S. Tsirkin
> Cc:
On Fri, Sep 09, 2016 at 08:00:31PM +0300, Marcel Apfelbaum wrote:
> On 09/09/2016 04:10 PM, Maxime Coquelin wrote:
> > This patch adds virtio_test_backend_virtio_1() function to
> > check whether backend supports VIRTIO_F_VERSION_1 before the
> > negociation takes place.
> >
> > Cc: Cornelia Huck
Remove the get_human_readable_size() method in favour of using
the new common qemu_szutostr() method. There are two slight
differences in output format
- the new code will also deal with petabyte and exabyte suffixes,
where as old code stopped at the terrabyte level.
- if the value has a trai
On 9/8/2016 1:39 PM, Jike Song wrote:
> On 08/25/2016 11:53 AM, Kirti Wankhede wrote:
>> +---+
>> | |
>> | +---+ | mdev_register_driver() +--+
>> | | | +<+ __init() |
>> | | mdev | |
On 09/09/2016 07:47 PM, Michael S. Tsirkin wrote:
On Fri, Sep 09, 2016 at 08:00:31PM +0300, Marcel Apfelbaum wrote:
On 09/09/2016 04:10 PM, Maxime Coquelin wrote:
This patch adds virtio_test_backend_virtio_1() function to
check whether backend supports VIRTIO_F_VERSION_1 before the
negociatio
On 09/09/2016 07:48 PM, Michael S. Tsirkin wrote:
On Fri, Sep 09, 2016 at 03:10:07PM +0200, Maxime Coquelin wrote:
This patch makes pci devices plugging more robust, by not confusing
guest with modern interface when the backend doesn't support
VIRTIO_F_VERSION_1.
Cc: Cornelia Huck
Cc: Marcel
!legacy && !modern is shorter than !(legacy || modern).
I also perfer this (less ()s) as a matter of taste.
Cc: Greg Kurz
Signed-off-by: Michael S. Tsirkin
---
hw/virtio/virtio-pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pc
On Thu, Sep 08, 2016 at 09:17:17AM +0200, Gerd Hoffmann wrote:
> We can't hotplug display adapters in qemu, tag virtio-gpu-pci
> accordingly (virtio-vga already has this).
>
> Signed-off-by: Gerd Hoffmann
Acked-by: Michael S. Tsirkin
I assume you merged this?
> ---
> hw/display/virtio-gpu-pc
On Wed, Aug 31, 2016 at 07:17:39PM +0300, David Kiarie wrote:
> Hi all,
>
> This patchset adds basic AMD IOMMU emulation support to Qemu.
Jan, Igor, any comments on this one?
I suspect it's time we merged this and followed-up upstream.
> Changes since v16 - this is mainly supposed to come as a
On Wed, Aug 31, 2016 at 07:25:11PM +0300, David Kiarie wrote:
> Hello all,
>
> Changes since V2
> -formating fixes.
> -fixed an issue where the right IOAPIC id was not being correctly set when
> using kernel_irqchip=off
>
> The following patchset implements AMD-Vi interrupt remapping logic a
Richard Henderson writes:
> Changes since v2:
> * Fix build for 32-bit host without 64-bit atomics.
> Tested with --extra-cflags='-march=-i486'.
> This is patch 15, which might ought to get folded back into
> patch 13 for bisection, but is ugly for review.
Hmm I wonder what's brea
struct target_ucontext_v2 is not at the begining of the signal frame,
therefore do_sigaltstack was being passed bogus arguments.
As the offset depends on the type of signal frame fixed by passing in the
beginning of the context from do_sigreturn_v2 and do_rt_sigreturn_v2.
Suggested-by: Peter Mayd
On Fri, 9 Sep 2016 23:18:45 +0530
Kirti Wankhede wrote:
> On 9/8/2016 1:39 PM, Jike Song wrote:
> > On 08/25/2016 11:53 AM, Kirti Wankhede wrote:
>
> >> +---+
> >> | |
> >> | +---+ | mdev_register_driver() +--+
> >> | | | +<-
On Fri, Sep 09, 2016 at 06:16:27PM +0200, Maxime Coquelin wrote:
>
>
> On 09/09/2016 05:39 PM, Michael S. Tsirkin wrote:
> > On Fri, Sep 09, 2016 at 04:14:59PM +0200, Maxime Coquelin wrote:
> > > > Currently, devices are plugged before features are negotiated.
> > > > If the backend doesn't suppo
On Tue, Sep 06, 2016 at 10:22:26PM +0200, Paolo Bonzini wrote:
>
>
> On 06/09/2016 21:21, Michael S. Tsirkin wrote:
> > On Tue, Sep 06, 2016 at 09:11:16PM +0200, Paolo Bonzini wrote:
> >>
> >>
> >> On 06/09/2016 17:51, Michael S. Tsirkin wrote:
> Just use "-machine accel=kvm:tcg" and let QEM
I am running make test on a box without kvm, and I am seeing these warnings:
GTESTER check-qtest-ppc64
"kvm" accelerator not found.
"kvm" accelerator not found.
GTESTER check-qtest-x86_64
Could not access KVM kernel module: No such file or directory
failed to initialize KVM: No such file or direct
Mark old-commands for speed and downtime as deprecated.
Move max-bandwidth and downtime-limit into migrate-set-parameters for
setting maximum migration speed and expected downtime limit parameters
respectively.
Change downtime units to milliseconds (only for new-command) and update
the query part i
On 09/09/2016 11:33 AM, Alex Bennée wrote:
Richard Henderson writes:
Changes since v2:
* Fix build for 32-bit host without 64-bit atomics.
Tested with --extra-cflags='-march=-i486'.
This is patch 15, which might ought to get folded back into
patch 13 for bisection, but is ugly f
This patch fixes the out-of-bounds check migration parameters in
qmp_migrate_set_parameters() for cpu-throttle-initial and
cpu-throttle-increment by adding a return statement for both.
Due to the missing return statements, parmaters were getting set to
out-of-bounds values despite the error.
Signe
On Wed, Sep 07, 2016 at 09:52:59AM +0800, Dou Liyang wrote:
[...]
> >
> > The option:
> > -smp MODEL,+FOO,+BAR
>
> I guess you may mean "-cpu", not "-smp"
Oops, yes. Thanks. :)
>
> > is internally translated to:
> > -global MODEL.FOO=on
> > -global MODEL.BAR=on
> > in addition to setting
On 09.09.2016 21:03, Michael S. Tsirkin wrote:
> I am running make test on a box without kvm, and I am seeing these warnings:
>
> GTESTER check-qtest-ppc64
> "kvm" accelerator not found.
> "kvm" accelerator not found.
> GTESTER check-qtest-x86_64
> Could not access KVM kernel module: No such file
Richard Henderson writes:
> On 09/09/2016 11:33 AM, Alex Bennée wrote:
>>
>> Richard Henderson writes:
>>
>>> Changes since v2:
>>> * Fix build for 32-bit host without 64-bit atomics.
>>> Tested with --extra-cflags='-march=-i486'.
>>> This is patch 15, which might ought to get folded
On 9/10/2016 12:12 AM, Alex Williamson wrote:
> On Fri, 9 Sep 2016 23:18:45 +0530
> Kirti Wankhede wrote:
>
>> On 9/8/2016 1:39 PM, Jike Song wrote:
>>> On 08/25/2016 11:53 AM, Kirti Wankhede wrote:
>>
+---+
| |
| +---+ | mdev_register_driv
On 9 September 2016 at 19:35, Timothy E Baldwin
wrote:
> struct target_ucontext_v2 is not at the begining of the signal frame,
> therefore do_sigaltstack was being passed bogus arguments.
>
> As the offset depends on the type of signal frame fixed by passing in the
> beginning of the context from
On 09/09/2016 12:29 PM, Alex Bennée wrote:
config-temp/qemu-conf.c: In function 'main':
config-temp/qemu-conf.c:12:8: error: unused variable 'is_host64'
[-Werror=unused-variable]
cc1: all warnings being treated as errors
Ah hah. All I needed was a compiler without __atomic support.
I thought
On 09/09/2016 12:29 PM, Alex Bennée wrote:
config-temp/qemu-conf.c:12:8: error: unused variable 'is_host64'
[-Werror=unused-variable]
cc1: all warnings being treated as errors
Try this.
r~
diff --git a/configure b/configure
index 519de5d..bd59cdd 100755
--- a/configure
+++ b/configure
@@ -
On 09/09/2016 02:14 PM, Ashijeet Acharya wrote:
> This patch fixes the out-of-bounds check migration parameters in
> qmp_migrate_set_parameters() for cpu-throttle-initial and
> cpu-throttle-increment by adding a return statement for both.
> Due to the missing return statements, parmaters were getti
On 09/09/2016 06:01 AM, Leon Alrae wrote:
This causes build failures for me on CentOS6.5:
/user/lea/dev/qemu/include/qemu/int128.h:7: error: expected ‘=’, ‘,’, ‘;’,
‘asm’ or ‘__attribute__’ before ‘Int128’
/user/lea/dev/qemu/include/qemu/int128.h:9: error: expected ‘=’, ‘,’, ‘;’,
‘asm’ or ‘__at
From: Thomas Huth
event_notifier_init() can fail in real life, for example when there
are not enough open file handles available (EMFILE) when using a lot
of devices. So instead of leaving the average user with a cryptic
error number only, print out a proper error message with strerror()
instead,
From: "Longpeng(Mike)"
This will used by the next patch.
Signed-off-by: Longpeng(Mike)
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h | 3 +++
hw/i386/pc_piix.c| 16 +---
hw/i386/pc_q35.c | 13 +++--
3 files changed, 27
From: Marcel Apfelbaum
Use kvm acceleration if available.
Disable kernel-irqchip and use qemu64 cpu
for both kvm and tcg cases.
Using kvm acceleration saves about a second
and disabling kernel-irqchip has no visible
performance impact.
Acked-by: Michael S. Tsirkin
Signed-off-by: Marcel Apfelba
The following changes since commit c2a57aae9a1c3dd7de77daf5478df10379aeeebf:
Merge remote-tracking branch 'remotes/famz/tags/docker-pull-request' into
staging (2016-09-09 12:49:41 +0100)
are available in the git repository at:
git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upst
PIO MR registration should use size from the correct notify struct.
Doesn't affect any visible behaviour because the field values are the
same (both are 4).
Signed-off-by: Michael S. Tsirkin
---
hw/virtio/virtio-pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/virtio/
This patch fixes the out-of-bounds check of migration parameters in
qmp_migrate_set_parameters() for cpu-throttle-initial and
cpu-throttle-increment by adding a return statement for both as they
were broken since their introduction in 2.5 via commit 1626fee.
Due to the missing return statements, pa
From: Marcel Apfelbaum
Currently each VQ Notification Virtio Capability is allocated
on a different page. The idea is to enable split drivers within
guests, however there are no known plans to do that.
The allocation will result in a 8MB BAR, more than various
guest firmwares pre-allocates for PC
From: "Longpeng(Mike)"
Some software algorithms are based on the hardware's cache info, for example,
for x86 linux kernel, when cpu1 want to wakeup a task on cpu2, cpu1 will trigger
a resched IPI and told cpu2 to do the wakeup if they don't share low level
cache. Oppositely, cpu1 will access cpu2
From: Jason Wang
We used to set vring call fd unconditionally even if guest driver does
not use MSIX for this vritqueue at all. This will cause lots of
unnecessary userspace access and other checks for drivers does not use
interrupt at all (e.g virtio-net pmd). So check and clean vring call
fd if
From: Stefan Hajnoczi
vq->inuse must be zeroed upon device reset like most other virtqueue
fields.
In theory, virtio_reset() just needs assert(vq->inuse == 0) since
devices must clean up in-flight requests during reset (requests cannot
not be leaked!).
In practice, it is difficult to achieve vq
On Sat, Sep 10, 2016 at 2:03 AM, Ashijeet Acharya
wrote:
> This patch fixes the out-of-bounds check of migration parameters in
> qmp_migrate_set_parameters() for cpu-throttle-initial and
> cpu-throttle-increment by adding a return statement for both as they
> were broken since their introduction
From: Ladi Prosek
The one pending element is being freed but not discarded on device
reset, which causes svq->inuse to creep up, eventually hitting the
"Virtqueue size exceeded" error.
Properly discarding the element on device reset makes sure that its
buffers are unmapped and the inuse counter
This patch fixes the out-of-bounds check of migration parameters in
qmp_migrate_set_parameters() for cpu-throttle-initial and
cpu-throttle-increment by adding a return statement for both as they
were broken since their introduction in 2.5 via commit 1626fee.
Due to the missing return statements, p
101 - 200 of 231 matches
Mail list logo
