On Thu, 10 Nov 2016 17:54:35 +0200
"Michael S. Tsirkin" wrote:
> On Thu, Nov 10, 2016 at 08:30:21AM -0700, Alex Williamson wrote:
> > On Thu, 10 Nov 2016 17:14:24 +0200
> > "Michael S. Tsirkin" wrote:
> >
> > > On Tue, Nov 08, 2016 at 01:04:21PM +0200, Aviv B.D wrote:
> > > > From: "Aviv Be
mrs is always 64bit, so we should always use a 64bit register.
Sometimes we'll only want to return the lower 32, but not for
MPIDR, as that does define fields in the upper 32.
Reviewed-by: Alex Bennée
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
lib/arm64/asm/processor.h | 15 ++
Make implementation equivalent to Linux's include/linux/stringify.h
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
lib/libcflat.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/libcflat.h b/lib/libcflat.h
index 72b1bf9668ef..82005f5d014f 100644
--- a/lib/li
Allow a thread to wait some specified amount of time. Can
specify in cycles, usecs, and msecs.
Reviewed-by: Alex Bennée
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
lib/arm/asm/processor.h | 19 +++
lib/arm/processor.c | 15 +++
lib/arm64/asm/proc
By adding support for launching with gicv3 we can break the 8 vcpu
limit. This patch adds support to smp code and also selects the
vgic model corresponding to the host. The vgic model may also be
manually selected by adding e.g. -machine gic-version=3 to
extra_params.
Reviewed-by: Alex Bennée
Sig
The following changes since commit 6bbcb76301a72dc80c8d29af13d40bb9a759c9c6:
MAINTAINERS: Remove obsolete stable branches (2016-11-10 15:29:59 +)
are available in the git repository at:
git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
for you to fetch changes up to 8
From: Gonglei
Currently the virtio-crypto device hasn't supported
hotpluggable and live migration well. Let's tag it
as not hotpluggable and migration actively and reopen
them once we support them well.
Suggested-by: Michael S. Tsirkin
Signed-off-by: Gonglei
Reviewed-by: Michael S. Tsirkin
Si
From: Peter Xu
Signed-off-by: Peter Xu
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/intel_iommu.h | 9 -
hw/i386/intel_iommu.c | 2 +-
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/include/hw/i386/intel_iommu.h b/includ
From: Peter Xu
Reported-by: Michael S. Tsirkin
Signed-off-by: Peter Xu
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
hw/i386/intel_iommu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
index 20c4d2c..
From: Marc-André Lureau
The call fd is not watched
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
tests/vhost-user-bridge.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
index
Allow user to select who sends ipis and with which irq,
rather than just always sending irq=0 from cpu0.
Signed-off-by: Andrew Jones
---
v4: improve structure and make sure spurious checking is
done even when the sender isn't cpu0
v2: actually check that the irq received was the irq sent,
Legacy features are those that transitional devices only
expose on the legacy interface.
Allow different ones per device class.
Cc: qemu-sta...@nongnu.org # dependency for the next patch
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Cornelia Huck
---
include/hw/virtio/virtio.h | 5 +
hw/s3
v4:
- Eric's r-b's
- Andre's suggestion to only take defines we need
- several other changes listed in individual patches
v3:
- Rebased on latest master
- Added Alex's r-b's
v2:
Rebased on latest master + my "populate argv[0]" series (will
send a REPOST for that shortly. Additionally a few
From: Rafael David Tinoco
Commit 31190ed7 added a migration blocker in vhost_dev_init() to
check if memfd would succeed. It is better if this blocker first
checks if vhost backend requires shared log. This will avoid a
situation where a blocker is added inappropriately (e.g. shared
log allocation
Reviewed-by: Alex Bennée
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
lib/arm/asm/processor.h | 10 ++
lib/arm64/asm/processor.h | 10 ++
2 files changed, 20 insertions(+)
diff --git a/lib/arm/asm/processor.h b/lib/arm/asm/processor.h
index afc903ca7d4a..75a8d08b89
virtio 1.0 spec says this is a legacy feature bit,
hide it from guests in modern mode.
Note: for cross-version migration compatibility,
we keep the bit set in host_features.
The result will be that a guest migrating cross-version
will see host features change under it.
As guests only seem to read
Signed-off-by: Andrew Jones
---
v4:
- heavily comment gicv3_ipi_send_tlist() [Eric]
- changes needed for gicv2 iar/irqstat fix to other patch
v2:
- use IRM for gicv3 broadcast
---
arm/gic.c | 195 ++---
arm/unittests.cfg | 6
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
tests/vhost-user-bridge.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
index 97e45d8..5b618f6 100644
---
Add some gicv2 support. This just adds init and enable
functions, allowing unit tests to start messing with it.
Signed-off-by: Andrew Jones
---
v4:
- only take defines from kernel we need now [Andre]
- moved defines to asm/gic.h so they'll be shared with v3 [drew]
- simplify enable by not car
From: Felipe Franciosi
Recent changes on vhost_dev_enable/disable_notifiers() produced a
VirtioBusState vbus variable which can be used instead of the
VIRTIO_BUS() macro. This commit just makes the code a little bit cleaner
and more consistent.
Signed-off-by: Felipe Franciosi
Reviewed-by: Micha
From: Xiao Guangrong
as there is a global lock to protect vm-exit handlers and
QMP/monitor, this lock can be dropped
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mam
From: Felipe Franciosi
Following the recent refactoring of virtio notifiers [1], more specifically
the patch
ed08a2a0b ("virtio: use virtio_bus_set_host_notifier to start/stop ioeventfd")
that uses virtio_bus_set_host_notifier [2] by default, core
virtio code requires 'ioeventfd_started' to be se
From: Marc-André Lureau
Add a library to help implementing vhost-user backend (or slave).
Dealing with vhost-user as an application developer isn't so easy: you
have all the trouble with any protocol: validation, unix ancillary data,
shared memory, eventfd, logging, and on top of that you need t
From: Marc-André Lureau
Use the libvhost-user library.
This ended up being a rather large patch that cannot be easily splitted,
due to massive code move and API changes.
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
tests/vhost-user-b
From: Jason Wang
Using uint8_t for source id will lose bus num and get the
wrong/invalid IOTLB entry. Fixing by using uint16_t instead and
enlarge level shift.
Cc: Paolo Bonzini
Cc: Richard Henderson
Cc: Eduardo Habkost
Cc: Michael S. Tsirkin
Signed-off-by: Jason Wang
Reviewed-by: Michael S
From: Cao jin
When user specify invalid property aer_log_max, device should fail to
create, and report appropriate message.
Signed-off-by: Cao jin
Reviewed-by: Marcel Apfelbaum
Acked-by: Dmitry Fleytman
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/pci/pc
From: Xiao Guangrong
as it is never called when nvdimm hotplug happens
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
hw/i386/pc.c | 6 --
1 file ch
From: Xiao Guangrong
To make the code more clearer, we
1) check ram_slots first, and build ssdt & nfit only when it is available
2) use nvdimm_get_plugged_device_list() to check if there is nvdimm device
plugged
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael
From: Marc-André Lureau
dispatcher_remove() is in use.
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
tests/vhost-user-bridge.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
i
From: Xiao Guangrong
as nvdimm acpi is okay to build fit when the nvdimm device
has not been 'realized'
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
i
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
v4: properly mask irqnr in ipi_handler
v2: add more details in the output if a test fails,
report spurious interrupts if we get them
---
arm/Makefile.common | 6 +-
arm/gic.c | 195 +++
From: Ladi Prosek
The function undoes the effect of virtqueue_pop and doesn't do anything
destructive or irreversible so virtqueue_unpop is a more fitting name.
Signed-off-by: Ladi Prosek
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
incl
Signed-off-by: Andrew Jones
---
v4:
- only take defines from kernel we need now [Andre]
- simplify enable by not caring if we reinit the distributor [drew]
v2:
- configure irqs as NS GRP1
---
lib/arm/asm/arch_gicv3.h | 42 +
lib/arm/asm/gic-v3.h | 92 ++
From: Xiao Guangrong
Its behavior has been changed as the nvdimm device which is being
realized also will be handled in this function, so rename it to
reflect the fact
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
From: Xiao Guangrong
Improve the description and clearly document the length field
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
docs/specs/acpi_nvdimm
From: Xiao Guangrong
and use it to replace the raw number
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
hw/acpi/nvdimm.c | 30 +
From: Peter Xu
We should not use cpu_to_le16() here, instead each of device/function
value is stored in a 8 byte field.
Signed-off-by: Peter Xu
Reviewed-by: Igor Mammedov
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/acpi/acpi-defs.h | 5 -
hw/i386/acp
From: Ladi Prosek
The function does not fully initialize the returned VirtQueueElement and should
be used only internally from the virtio module.
Signed-off-by: Ladi Prosek
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/virtio/v
From: Xiao Guangrong
and use these codes to refine the code
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
hw/acpi/nvdimm.c | 46 +++
From: Xiao Guangrong
inline buf_size to refine the code a bit
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
hw/acpi/nvdimm.c | 4 +---
1 file changed,
From: Cao jin
On migration target, msix_vector_use() will be called in vmxnet3_post_load()
in second time, without a matching second call to msi_vector_unuse(),
which results in vector reference leak.
CC: Dmitry Fleytman
CC: Jason Wang
CC: Markus Armbruster
CC: Michael S. Tsirkin
Reviewed-b
From: Xiao Guangrong
as they use completely different way to handle hotplug event
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
include/hw/mem/nvdimm.h
From: Xiao Guangrong
fixed the English issue and code-style issue
Suggested-by: Stefan Hajnoczi
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
hw/acpi/nvdimm.c | 4 ++--
1 file cha
From: Cao jin
msix_init() reports errors with error_report(), which is wrong when
it's used in realize(). The same issue was fixed for msi_init() in
commit 1108b2f.
For some devices(like e1000e, vmxnet3) who won't fail because of
msix_init's failure, suppress the error report by passing NULL er
From: Greg Kurz
With virtio 1, the vring layout is split in 3 separate regions of
contiguous memory for the descriptor table, the available ring and the
used ring, as opposed with legacy virtio which uses a single region.
In case of memory re-mapping, the code ensures it doesn't affect the
vring
From: Xiao Guangrong
Rename it to nvdimm_plug()
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
include/hw/mem/nvdimm.h | 2 +-
hw/acpi/nvdimm.c|
From: Xiao Guangrong
Rename it to nvdimm_dsm_handle_reserved_root_method
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
hw/acpi/nvdimm.c | 7 ---
1
From: Peter Xu
These macros will be useful to do page alignment checks.
Signed-off-by: Peter Xu
[drew: also added SZ_64K]
Signed-off-by: Andrew Jones
---
lib/libcflat.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/lib/libcflat.h b/lib/libcflat.h
index 82005f5d014f..143fc53061fe 1
From: Cao jin
Internal flag msix_used is unnecessary, it has the same effect as
msix_enabled().
The corresponding msi flag is already dropped in commit 1070048e.
CC: Dmitry Fleytman
CC: Jason Wang
CC: Markus Armbruster
CC: Michael S. Tsirkin
Reviewed-by: Markus Armbruster
Reviewed-by: Dmi
From: Greg Kurz
The legacy vring layout is not used anymore as we use the separate
mappings even for legacy devices.
This patch simply removes it.
This also fixes a bug with virtio 1 devices when the vring descriptor table
is mapped at a higher address than the used vring because the following
f
From: Xiao Guangrong
to make the code more clearer
Suggested-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Igor Mammedov
---
hw/acpi/nvdimm.c | 6 ++
1 file changed, 2 inserti
From: Cao jin
CC: Markus Armbruster
CC: Marcel Apfelbaum
CC: Michael S. Tsirkin
Reviewed-by: Markus Armbruster
Signed-off-by: Cao jin
Acked-by: Marcel Apfelbaum
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
hw/pci/msix.c | 8 ++--
1 file changed, 6 insertions
From: Greg Kurz
These are not used anymore.
Signed-off-by: Greg Kurz
Reviewed-by: Cornelia Huck
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/virtio/virtio.h | 2 --
hw/virtio/virtio.c | 11 ---
2 files changed, 13 deletions(-)
diff --git
From: Cao jin
usb_xhci_realize() corrects invalid values of property "intrs"
automatically, but the uncorrected value is passed to msi_init(),
which chokes on invalid values. Delay that until after the
correction.
Resources allocated by usb_xhci_init() are leaked when msi_init()
fails. Fix by
From: Cao jin
Resolve the TODO, msix=auto means msix on; if user specify msix=on,
then device creation fail on msix_init failure.
CC: Gerd Hoffmann
CC: Michael S. Tsirkin
CC: Markus Armbruster
CC: Marcel Apfelbaum
Reviewed-by: Gerd Hoffmann
Reviewed-by: Markus Armbruster
Signed-off-by: Ca
From: Marcel Apfelbaum
Proposes best practices on how to use PCI Express/PCI device
in PCI Express based machines and explain the reasoning behind them.
Reviewed-by: Laszlo Ersek
Signed-off-by: Marcel Apfelbaum
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
docs/pcie.
From: Cao jin
Resolve the TODO, msix=auto means msix on; if user specify msix=on,
then device creation fail on msix_init failure.
Also undo the overwrites of user configuration of msix.
CC: Michael S. Tsirkin
CC: Hannes Reinecke
CC: Paolo Bonzini
CC: Markus Armbruster
CC: Marcel Apfelbaum
On 11/10/16 16:45, Stefan Hajnoczi wrote:
> On Thu, Nov 10, 2016 at 2:10 PM, Laszlo Ersek wrote:
>> On 11/10/16 11:48, Stefan Hajnoczi wrote:
>>> On Wed, Nov 09, 2016 at 10:09:24AM +0100, Gerd Hoffmann wrote:
Hi,
This rebases ipxe to latest master. ipxe qemu builds will not use
>
Pranith Kumar writes:
> Hi Alex,
>
> This patch is causing some build errors on a 32-bit box:
>
> In file included from /home/pranith/qemu/include/exec/exec-all.h:44:0,
> from /home/pranith/qemu/cputlb.c:23:
> /home/pranith/qemu/cputlb.c: In function
> ‘tlb_flush_page_by_mmuidx
From: Cao jin
Commit afea4e14 seems forgetting to undo the overwrites, which is
unsuitable.
CC: Hannes Reinecke
CC: Paolo Bonzini
CC: Markus Armbruster
CC: Marcel Apfelbaum
CC: Michael S. Tsirkin
Reviewed-by: Markus Armbruster
Signed-off-by: Cao jin
Acked-by: Marcel Apfelbaum
Reviewed-b
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
tests/vhost-user-bridge.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
index 19b0e94..97e45d8 100644
--- a
Ping,
Kevin: Look ok?
--js
On 11/07/2016 04:13 PM, John Snow wrote:
Requires patches in my IDE branch, for context.
This series changes blk_eject (used for a software-initiated eject request)
to always trigger a QMP tray event, in contrast to the current behavior where
a tray event only occur
From: Cao jin
Also move certain hunk above, to place msix init related code together.
CC: Hannes Reinecke
CC: Paolo Bonzini
CC: Markus Armbruster
CC: Marcel Apfelbaum
CC: Michael S. Tsirkin
Reviewed-by: Markus Armbruster
Signed-off-by: Cao jin
Acked-by: Marcel Apfelbaum
Reviewed-by: Mic
Marc-André Lureau writes:
> As the name suggests, the qapi2texi script converts JSON QAPI
> description into a texi file suitable for different target
> formats (info/man/txt/pdf/html...).
>
> It parses the following kind of blocks:
>
> Free-form:
>
> ##
> # = Section
> # == Subsection
>
From: Cao jin
According to the disscussion:
http://lists.nongnu.org/archive/html/qemu-devel/2016-09/msg08215.html
Let leaf function returns reasonable -errno, let caller decide how to
handle the return value.
Suggested-by: Markus Armbruster
CC: Markus Armbruster
CC: Michael S. Tsirkin
CC: Ma
On 11/09/2016 03:57 PM, Alex Bennée wrote:
+void tlb_flush_page_all(target_ulong addr)
It's a nit, but when I read this I think all pages, not all cpus.
Can we rename this tlb_fluch_page_all_cpus?
Otherwise,
Reviewed-by: Richard Henderson
r~
On 11/09/2016 03:57 PM, Alex Bennée wrote:
For SoftMMU the TLB flushes are an example of a task that can be
triggered on one vCPU by another. To deal with this properly we need to
use safe work to ensure these changes are done safely. The new assert
can be enabled while debugging to catch these c
On Thu, Nov 10, 2016 at 05:47:54PM +0100, Halil Pasic wrote:
> the typos
That's something that's avoidable btw, please use
a spell and grammar checker. I hear http://www.languagetool.org/
is useful to some people.
--
MST
Fixed in:
commit 423f7cf233fe262c777db7f87db3e9fac29e02d1
Author: Gerd Hoffmann
Date: Wed Nov 9 09:48:44 2016 +0100
ipxe: update to 20161108 snapshot
** Changed in: qemu
Status: In Progress => Fix Committed
--
You received this bug notification because you are a member of qemu-
On Thu, Nov 10, 2016 at 06:12:03PM +0200, Michael S. Tsirkin wrote:
> From: Marc-André Lureau
>
> Add a library to help implementing vhost-user backend (or slave).
>
> Dealing with vhost-user as an application developer isn't so easy: you
> have all the trouble with any protocol: validation, uni
Richard Henderson writes:
> On 11/09/2016 03:57 PM, Alex Bennée wrote:
>> +if (qemu_tcg_mttcg_enabled() || !single_tcg_cpu_thread) {
>> +parallel_cpus = true;
>
> Why are we setting this here,
>
>> cpu->thread = g_malloc0(sizeof(QemuThread));
>> cpu->halt_cond = g_m
On 11/09/2016 03:57 PM, Alex Bennée wrote:
+if (qemu_tcg_mttcg_enabled() || !single_tcg_cpu_thread) {
+parallel_cpus = true;
Why are we setting this here,
cpu->thread = g_malloc0(sizeof(QemuThread));
cpu->halt_cond = g_malloc0(sizeof(QemuCond));
qemu_con
This is part of a series that I'm working on and that aims to remove the
bdrv_aio_*() emulation in io.c. blkdebug and blkverify were easy, but for
quorum I need a few more patches, so I'm sending this out as an RFC while I
continue work on the rest (QED, and then possibly some polishing).
After th
On 11/09/2016 03:57 PM, Alex Bennée wrote:
This moves the helper function closer to where it is called and updates
the error message to report via error_report instead of the deprecated
fprintf.
Signed-off-by: Alex Bennée
---
cputlb.c | 24
1 file changed, 12 in
Revi
On 11/09/2016 03:57 PM, Alex Bennée wrote:
From: Pranith Kumar
The patch enables handling atomic code in the guest. This should be
preferably done in cpu_handle_exception(), but the current assumptions
regarding when we can execute atomic sections cause a deadlock.
Signed-off-by: Pranith Kumar
In the context of asynchronous work, if we have a worker coroutine that
didn't yield, the parent coroutine cannot be reentered because it hasn't
yielded yet. In this case we don't even have to reenter the parent
because it will see that the work is already done and won't even yield.
Signed-off-by:
On Sat, Aug 20, 2016 at 05:07:42PM +0900, Namhyung Kim wrote:
> The virtio pstore driver provides interface to the pstore subsystem so
> that the guest kernel's log/dump message can be saved on the host
> machine. Users can access the log file directly on the host, or on the
> guest at the next bo
Instead of calling quorum_aio_finalize() deeply nested in what used
to be an AIO callback, do it in the same functions that allocated the
AIOCB.
Signed-off-by: Kevin Wolf
---
block/quorum.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/block/quorum.c b/block
Reviewed-by: Alex Bennée
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
lib/arm/asm/processor.h | 10 ++
lib/arm64/asm/processor.h | 10 ++
2 files changed, 20 insertions(+)
diff --git a/lib/arm/asm/processor.h b/lib/arm/asm/processor.h
index bc46d1f980ee..959ecda5dc
Inlining the function removes some boilerplace code and replaces
recursion by a simple loop, so the code becomes somewhat easier to
understand.
Signed-off-by: Kevin Wolf
---
block/quorum.c | 42 +-
1 file changed, 13 insertions(+), 29 deletions(-)
diff --
This enables byte granularity requests on quorum nodes.
Note that the QMP events emitted by the driver are an external API that
we were careless enough to define as sector based. The offset and length
of requests reported in events are rounded down therefore.
Signed-off-by: Kevin Wolf
---
block
On 11/10/16 16:09, Michael S. Tsirkin wrote:
> On Tue, Sep 06, 2016 at 04:28:33PM +0800, Lin Ma wrote:
>> If user specifies binary file on command line to load smbios entries, then
>> will get error messages while decoding them in guest.
>>
>> Reproducer:
>> 1. dump a smbios table to a binary file
On 11/10/2016 02:15 PM, Michael S. Tsirkin wrote:
> On Thu, Nov 10, 2016 at 09:37:40AM +, Gonglei (Arei) wrote:
>> > Hi,
>> >
>> > I attach a diff for next version in order to review more convenient, with
>> >
>> > - Drop the all gap stuff;
>> > - Drop all structures undefined in virtio_c
Add some gicv2 support. This just adds init and enable
functions, allowing unit tests to start messing with it.
Signed-off-by: Andrew Jones
---
v5: share/use only the modern register names [Andre]
v4:
- only take defines from kernel we need now [Andre]
- moved defines to asm/gic.h so they'll b
On Thu, Nov 10, 2016 at 04:52:30PM +, Andre Przywara wrote:
> Hi,
>
> so is this actually v4 just resent?
> Or is this is a new version with s/5/4/?
> I can't spot any of the key changes quickly ...
Argh... Forgot to edit my bash history after pulling it up to send.
OK, the real v5 is coming.
Hi,
so is this actually v4 just resent?
Or is this is a new version with s/5/4/?
I can't spot any of the key changes quickly ...
Cheers,
Andre.
On 10/11/16 16:07, Andrew Jones wrote:
> v4:
> - Eric's r-b's
> - Andre's suggestion to only take defines we need
> - several other changes listed in
On Thu, Nov 10, 2016 at 05:05:03PM +, Daniel P. Berrange wrote:
> On Thu, Nov 10, 2016 at 06:12:03PM +0200, Michael S. Tsirkin wrote:
> > From: Marc-André Lureau
> >
> > Add a library to help implementing vhost-user backend (or slave).
> >
> > Dealing with vhost-user as an application develo
There is no point in passing the value of bs->opaque in order to
overwrite it with itself.
Signed-off-by: Kevin Wolf
---
block/quorum.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/block/quorum.c b/block/quorum.c
index d122299..dfa9fd3 100644
--- a/block/quorum.c
Make implementation equivalent to Linux's include/linux/stringify.h
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
lib/libcflat.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/libcflat.h b/lib/libcflat.h
index 72b1bf9668ef..82005f5d014f 100644
--- a/lib/li
The following changes since commit 207faf24c58859f5240f66bf6decc33b87a1776e:
Merge remote-tracking branch 'pm215/tags/pull-target-arm-20161107' into
staging (2016-11-07 14:02:15 +)
are available in the git repository at:
git://github.com/bonzini/qemu.git tags/for-upstream
for you to f
Signed-off-by: Kevin Wolf
---
block/quorum.c | 194 ++---
1 file changed, 117 insertions(+), 77 deletions(-)
diff --git a/block/quorum.c b/block/quorum.c
index dfa9fd3..3cb579e 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@ -97,7 +97,7 @@
v5:
- fix arm32/gicv3 compile [drew]
- use modern register names [Andre]
- one Andre r-b
v4:
- Eric's r-b's
- Andre's suggestion to only take defines we need
- several other changes listed in individual patches
v3:
- Rebased on latest master
- Added Alex's r-b's
v2:
Rebased on latest ma
This is a conversion to a more natural coroutine style and improves the
readability of the driver.
Signed-off-by: Kevin Wolf
---
block/quorum.c | 112 ++---
1 file changed, 51 insertions(+), 61 deletions(-)
diff --git a/block/quorum.c b/block/
Replacing it with bdrv_co_pwritev() prepares us for byte granularity
requests and gets us rid of the last bdrv_aio_*() user in quorum.
Signed-off-by: Kevin Wolf
---
block/quorum.c | 52 +---
1 file changed, 33 insertions(+), 19 deletions(-)
diff -
Signed-off-by: Andrew Jones
---
v5: use modern register names [Andre]
v4:
- only take defines from kernel we need now [Andre]
- simplify enable by not caring if we reinit the distributor [drew]
v2:
- configure irqs as NS GRP1
---
lib/arm/asm/arch_gicv3.h | 42 +
lib/arm/
mrs is always 64bit, so we should always use a 64bit register.
Sometimes we'll only want to return the lower 32, but not for
MPIDR, as that does define fields in the upper 32.
Reviewed-by: Alex Bennée
Reviewed-by: Eric Auger
Signed-off-by: Andrew Jones
---
v5: switch arm32's get_mpidr to 'unsi
From: Michael Tokarev
With current code, pid file is open after various
sockets, chardevs, fsdevs and the like. This causes
interesting effects, for example when monitor is a
unix-socket, and another qemu instance is already
running, new qemu first "damages" the socket and
next complain that it
By adding support for launching with gicv3 we can break the 8 vcpu
limit. This patch adds support to smp code and also selects the
vgic model corresponding to the host. The vgic model may also be
manually selected by adding e.g. -machine gic-version=3 to
extra_params.
Reviewed-by: Alex Bennée
Sig
Allow user to select who sends ipis and with which irq,
rather than just always sending irq=0 from cpu0.
Signed-off-by: Andrew Jones
---
v4: improve structure and make sure spurious checking is
done even when the sender isn't cpu0
v2: actually check that the irq received was the irq sent,
OK, thanks a lot for your response ... so let's close this bug now.
** Changed in: qemu
Status: Incomplete => Invalid
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/616769
Title:
interrupt p
101 - 200 of 298 matches
Mail list logo
