As before, using Debian SID compilers.
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index 65e3dee741..59dd8e314d 100644
--- a/tests/docker/Makefile.include
+++ b/tests
We only have compilers for the (default) little endian variants.
Signed-off-by: Alex Bennée
Reviewed-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
diff --git a/MAINTAINERS b/MAINTAINERS
index a6d233a43a..de1ae8423e 100644
--- a/MAINTAINERS
+++ b/M
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
diff --git a/tests/tcg/mips/Makefile.target b/tests/tcg/mips/Makefile.target
new file mode 100644
index 00..71f0c2dd53
--- /dev/null
+++ b/tests/tcg/mips/Makefile.target
@@ -0,0 +1,19 @@
+#
This allows us to use the docker cross compiler image to build these
tests.
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
diff --git a/tests/tcg/arm/Makefile.include b/tests/tcg/arm/Makefile.include
new file mode 100644
index 00..8e7ea
While you can construct a compile command that does work using the
x86_64 host compiler that most people use this is flakey. Different
distros handle this is different ways so we default to using a known
good i386 compiler via docker.
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Now we can check the age of a docker image we can be a little more
intelligent about re-building Sid images and only force NOCACHE if
it is "old".
Signed-off-by: Alex Bennée
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index c0f09505bb..91d9665517 100644
--- a/tests
This got broken in commit 4319db7 but generally only shows up when you
try and do massive parallel builds on fresh machines.
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.inclu
From: Fam Zheng
We have -Werror=missing-prototype, add a dummy prototype to avoid that
warning.
Signed-off-by: Fam Zheng
Reviewed-by: Thomas Huth
Reviewed-by: Richard Henderson
Signed-off-by: Alex Bennée
diff --git a/tests/tcg/i386/hello-i386.c b/tests/tcg/i386/hello-i386.c
index fa00380de2
From: Fam Zheng
To keep the compiler happy, and to fit in our buildsys flags:
- Make local functions "static"
- #ifdef out unused functions
- drop cutils/osdep dependencies
Signed-off-by: Fam Zheng
[AJB: drop cutils/osdep dependencies]
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Revi
We need to rename the source file to a .S so we can do a single-line
assemble and link invocation. We also specify the additional CFLAGS
for the compile as it's a non-standard ARM binary.
Signed-off-by: Alex Bennée
[rth: force fpu configuration]
Signed-off-by: Richard Henderson
Reviewed-by: Phil
This doesn't add any additional tests but enables building the
multiarch tests for MIPS using docker cross compilers. We don't have a
cross compiler for mips64 big endian though.
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
diff --git a/tests
The default test run outputs to stdout so it can be re-directed.
Errors are still reported to stderr.
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
Reviewed-by: Thomas Huth
diff --git a/tests/tcg/multiarch/test
The original Jessie based cross builder hasn't worked for a while. The
state of the libraries is still perilous for cross-building QEMU but
we can use it for building TCG tests.
The debian-apt-fake.sh script can also be dropped as it is no longer
used.
Signed-off-by: Alex Bennée
Reviewed-by: Phi
We don't include anything from qemu itself for the build.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
diff --git a/tests/tcg/i386/test-i386.c b/tests/tcg/i386/test-i386.c
index 959920
On Wed, 20 Jun 2018, David Gibson wrote:
On Tue, Jun 19, 2018 at 10:52:15AM +0200, BALATON Zoltan wrote:
Rewrite to make it closer to how real device works so that guest OS
drivers can access I2C devices. Previously this was only a hack to
allow U-Boot to get past accessing SPD EEPROMs but to su
Marc-André Lureau writes:
> Add helpers to wrap generated code with #if/#endif lines.
>
> Add QAPIGenCSnippet class to write C snippet code, make QAPIGenC
> inherit from it, for full C files with copyright headers etc.
>
> Add a 'with' statement context manager that will be used to wrap
> generat
These only need to be built for ARM guests.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
diff --git a/MAINTAINERS b/MAINTAINERS
index 096b5bbf61..a6d233a43a 100644
--- a/MAINTAINERS
++
Hello,
On Wed, 20 Jun 2018, David Gibson wrote:
On Tue, Jun 19, 2018 at 10:52:15AM +0200, BALATON Zoltan wrote:
Next spin of sam460ex pathces with changes according to review and
some new patches.
So, the sm501 patches I don't really have the technical knowledge to
review, and I'm a little un
We can't use our normal Debian based compilers as Alpha isn't an
officially supported architecture. However it is available as a port
and fortunately cross compilers for all these targets are included in
Debian Sid, the perpetual rolling/unstable/testing version of Debian.
Signed-off-by: Alex Benn
On 06/18/2018 08:36 AM, David Gibson wrote:
> KVM HV has some limitations (deriving from the hardware) that mean not all
> host-cpu supported pagesizes may be usable in the guest. At present this
> means that KVM guests and TCG guests may see different available page sizes
> even if they notionall
On Wed, 20 Jun 2018, David Gibson wrote:
On Tue, Jun 19, 2018 at 10:52:15AM +0200, BALATON Zoltan wrote:
As well as being able to generate its own i2c transactions, the ppc4xx
i2c controller has a DIRECTCNTL register which allows explicit control
of the i2c lines.
Using this register an OS can
Signed-off-by: Alex Bennée
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index 8c24f8807e..65e3dee741 100644
--- a/tests/docker/Makefile.include
+++ b/tests/docker/Makefile.include
@@ -71,6 +71,7
The sources for x86_64 are shared in the i386 directory which will be
included thanks to TARGET_BASE_ARCH. However not all sources build so
we need to filter out the ones we can't build in the 64 bit world and
those that can't be built for 32 bit.
Signed-off-by: Alex Bennée
Reviewed-by: Philippe
One problem with satisfying your docker dependencies in a sub-make it
you might end up trying to satisfy the dependency multiple times. This
is especially a problem with debian-sid based cross compilers and CI
setups. We solve this by doing a docker build pass at the top level
before any sub-makes
On Wed, Jun 20, 2018 at 06:33:51PM +0100, Peter Maydell wrote:
> On 18 June 2018 at 18:18, Michael S. Tsirkin wrote:
> > On Mon, Jun 18, 2018 at 02:14:31PM -0300, Eduardo Habkost wrote:
> >> > Sure if someone does that, we'll have no choice, but as long as 'pc' is
> >> > shipped we shouldn't gratu
On Wed, Jun 20, 2018 at 02:28:24PM -0300, Eduardo Habkost wrote:
> On Mon, Jun 18, 2018 at 08:18:16PM +0300, Michael S. Tsirkin wrote:
> > On Mon, Jun 18, 2018 at 02:14:31PM -0300, Eduardo Habkost wrote:
> > > > Sure if someone does that, we'll have no choice, but as long as 'pc' is
> > > > shipped
Hi Peter,
On 06/20/2018 06:07 PM, Peter Maydell wrote:
> On 12 June 2018 at 09:08, Eric Auger wrote:
>> We emulate a TLB cache of size SMMU_IOTLB_MAX_SIZE=256.
>> It is implemented as a hash table whose key is a combination
>> of the 16b asid and 48b IOVA (Jenkins hash).
>>
>> Entries are invalid
Hi Peter,
On 06/20/2018 05:56 PM, Peter Maydell wrote:
> On 12 June 2018 at 09:08, Eric Auger wrote:
>> Let's cache config data to avoid fetching and parsing STE/CD
>> structures on each translation. We invalidate them on data structure
>> invalidation commands.
>>
>> We put in place a per-smmu m
On 06/18/2018 08:36 AM, David Gibson wrote:
> Currently during KVM initialization on POWER, kvm_fixup_page_sizes()
> rewrites a bunch of information in the cpu state to reflect the
> capabilities of the host MMU and KVM. This overwrites the information
> that's already there reflecting how the TCG
Hi Peter,
On 06/20/2018 06:10 PM, Peter Maydell wrote:
> On 20 June 2018 at 16:56, Peter Maydell wrote:
>> On 12 June 2018 at 09:08, Eric Auger wrote:
>>> +smmuv3_config_cache_hit(uint32_t sid, uint32_t hits, uint32_t misses,
>>> float perc) "Config cache HIT for sid %d (hits=%d, misses=%d, hit
Marc-André Lureau writes:
> Modify the test visitor to check correct passing of values.
>
> Signed-off-by: Marc-André Lureau
> Reviewed-by: Markus Armbruster
> ---
[...]
> diff --git a/tests/qapi-schema/test-qapi.py b/tests/qapi-schema/test-qapi.py
> index 10e68b01d9..6d25a37dda 100644
> --- a/
Emulate the i2c part of SM501 which is used to access the EDID info
from a monitor.
The vmstate structure is changed and its version is increased but
SM501 is only used on SH and PPC sam460ex machines that don't support
cross-version migration.
Signed-off-by: BALATON Zoltan
---
default-configs/
These are fixes to sm501 emulation that were found while testing with
AmigaOS 4.1FE (AmigaOS 4 is known to work on real hardware but we have
no way to verify it against hardware). While this is mainly to improve
sam460ex emulation this is now a separate series with the sm501
specific patches becaus
From: Sebastian Bauer
Changing the palette of a color index has as an immediate effect on
all pixels with the corresponding index on real hardware. Performing a
full update after a palette change is a simple way to emulate this
effect.
Signed-off-by: Sebastian Bauer
Signed-off-by: BALATON Zolta
From: Sebastian Bauer
Before, crt_h_total was used for src_width and dst_width. This is a
property of the current display setting and not relevant for the 2d
operation that also can be done off-screen. The pitch register's purpose
is to describe line pitch relevant of the 2d operation.
Signed-of
Display updates and drawing hardware cursor did not work when frame
buffer address was non-zero. Fix this by taking the frame buffer
address into account in these cases. This fixes screen dragging on
AmigaOS. Based on patch by Sebastian Bauer.
Signed-off-by: Sebastian Bauer
Signed-off-by: BALATON
Set the changed memory region dirty after performed a 2D operation to
ensure that the screen is updated properly.
Signed-off-by: BALATON Zoltan
---
hw/display/sm501.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/hw/display/sm501.c b/hw/display/sm501.c
index a6232ed..e426d2f 100644
--
ferable. We'd also want the corresponding
> > patches for pc-bios/s390-ccw reflected in the 2.11.x tree.
>
> Er, sorry was a bit confused. I suppose that part is covered already if
> there's no additional patches needed in the rebuild other than what's in
> 2.11.
Fix the following issues:
common.py:873:13: E129 visually indented line with same indent as next
logical line
common.py:1766:5: E741 ambiguous variable name 'l'
common.py:1784:1: E305 expected 2 blank lines after class or function
definition, found 1
common.py:1833:1: E305 expect
On Wed, 20 Jun 2018 16:39:41 +0100
"Dr. David Alan Gilbert (git)" wrote:
> From: "Dr. David Alan Gilbert"
>
> Add a flag to command definitions to allow them to be used in preconfig
> and check it.
> If users try to use commands that aren't available, tell them to use
> the exit_preconfig coman
On 21 June 2018 at 07:25, Alex Bennée wrote:
> Although the docker.py is nominally python2 we actually invoke it with
> the configured python from the configure script.
>
> Signed-off-by: Alex Bennée
>
> diff --git a/tests/docker/docker.py b/tests/docker/docker.py
> index 306e14cf69..e4095270eb 1
On Wed, 20 Jun 2018 16:39:45 +0100
"Dr. David Alan Gilbert (git)" wrote:
> From: "Dr. David Alan Gilbert"
>
> Allow a bunch of the info commands to be used in preconfig.
>
> version, chardev, name, uuid,memdev, iothreads
> Were enabled in QMP in the previous patch from Igor
>
> status, hotp
On 21 June 2018 at 04:19, Eric Blake wrote:
> The following changes since commit 46012db666990ff2eed1d3dc199ab8006439a93b:
>
> Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20180619' into
> staging (2018-06-20 09:51:30 +0100)
>
> are available in the Git repository at:
>
> git://rep
On Thu, Jun 21, 2018 at 09:49:55AM +0100, Peter Maydell wrote:
> On 21 June 2018 at 07:25, Alex Bennée wrote:
> > Although the docker.py is nominally python2 we actually invoke it with
> > the configured python from the configure script.
> >
> > Signed-off-by: Alex Bennée
> >
> > diff --git a/tes
On Thu, Jun 21, 2018 at 10:35:51AM +0200, Markus Armbruster wrote:
> Fix the following issues:
>
> common.py:873:13: E129 visually indented line with same indent as next
> logical line
> common.py:1766:5: E741 ambiguous variable name 'l'
> common.py:1784:1: E305 expected 2 blank lines
The following changes since commit 840d3ddc259b70a7b4e1ba367945cc31a45c93c6:
pseries: Update SLOF firmware image to qemu-slof-20180621 (2018-06-21
19:16:26 +1000)
are available in the git repository at:
g...@github.com:aik/qemu.git tags/qemu-slof-20180621
for you to fetch changes up to
On Mon, 18 Jun 2018 16:36:02 +1000
David Gibson wrote:
> The way the POWER Hash Page Table (HPT) MMU is virtualized by KVM HV means
> that every page that the guest puts in the pagetables must be truly
> physically contiguous, not just GPA-contiguous. In effect this means that
> an HPT guest can
Hi,
This series failed build test on s390x host. Please find the details below.
Type: series
Message-id: 20180621032539.134944-1-ebl...@redhat.com
Subject: [Qemu-devel] [PATCH] nbd/client: add x-block-status hack for testing
server
=== TEST SCRIPT BEGIN ===
#!/bin/bash
# Testing script will be
On Tue, 15 May 2018 14:14:30 +0200
Marc-André Lureau wrote:
> From: Stefan Berger
>
> Implement a virtual memory device for the TPM Physical Presence interface.
> The memory is located at 0xfffef000 and used by ACPI to send messages to the
> firmware (BIOS) and by the firmware to provide parame
* Juan Quintela (quint...@redhat.com) wrote:
> We synchronize all threads each RAM_SAVE_FLAG_EOS. Bitmap
> synchronizations don't happen inside a ram section, so we are safe
> about two channels trying to overwrite the same memory.
>
> Signed-off-by: Juan Quintela
Note the testing didn't like
On Tue, 15 May 2018 14:14:31 +0200
Marc-André Lureau wrote:
> From: Stefan Berger
>
> To avoid having to hard code the base address of the PPI virtual
> memory device we introduce a fw_cfg file etc/tpm/config that holds the
> base address of the PPI device, the version of the PPI interface and
On 08/06/2018 22:07, Daniel Henrique Barboza wrote:
> The previous patches implemented a way to deliver an emulated
> Block Limits (BL) response for the guest in case the underlying
> hardware does not support this page.
>
> However, the approach used is crude. We're executing the logic for
> all
On 08/06/2018 22:07, Daniel Henrique Barboza wrote:
> +unsigned int unmap_sectors = s->conf.discard_granularity / s->blocksize;
> +unsigned int min_io_size = s->conf.min_io_size / s->blocksize;
> +unsigned int opt_io_size = s->conf.opt_io_size / s->blocksize;
> +unsigned int max_unm
On 11/06/2018 21:56, Eduardo Habkost wrote:
> The newline was removed by commit c97d6d2c, and broke -help output:
>
> Before this patch:
>
> $ qemu-system-x86_64 -help | grep smp
> thread=single|multi (enable multi-threaded TCG)-smp [...]
>
> After this patch:
>
> $ qemu-s
Hi David,
On 06/20/2018 07:15 PM, Dr. David Alan Gilbert wrote:
> * Eric Auger (eric.au...@redhat.com) wrote:
>> The kvm-type property currently is used to pass
>> a user parameter to KVM_CREATE_VM. This matches
>> the way KVM/ARM expects to pass the max_vm_phys_shift
>> parameter.
>>
>> This patc
20.06.2018 21:09, Eric Blake wrote:
On 06/20/2018 12:04 PM, Vladimir Sementsov-Ogievskiy wrote:
20.06.2018 19:27, Eric Blake wrote:
On 06/09/2018 10:17 AM, Vladimir Sementsov-Ogievskiy wrote:
Handle new NBD meta namespace: "qemu", and corresponding queries:
"qemu:dirty-bitmap:".
With new meta
Hi
On Thu, Jun 21, 2018 at 12:00 PM, Igor Mammedov wrote:
> On Tue, 15 May 2018 14:14:31 +0200
> Marc-André Lureau wrote:
>
>> From: Stefan Berger
>>
>> To avoid having to hard code the base address of the PPI virtual
>> memory device we introduce a fw_cfg file etc/tpm/config that holds the
>>
20.06.2018 21:14, Eric Blake wrote:
On 06/20/2018 09:13 AM, Vladimir Sementsov-Ogievskiy wrote:
20.06.2018 14:26, Eric Blake wrote:
On 06/09/2018 10:17 AM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
qapi/block.json | 23 +++
bloc
21.06.2018 06:19, Eric Blake wrote:
From: Vladimir Sementsov-Ogievskiy
Handle a new NBD meta namespace: "qemu", and corresponding queries:
"qemu:dirty-bitmap:".
With the new metadata context negotiated, BLOCK_STATUS query will reply
with dirty-bitmap data, converted to extents. The new public
To be able to add new suspend mechanisms we need to detach
the existing QMP functions from the current implementation
specifics.
At this moment we have functions such as qmp_guest_suspend_ram
calling bios_suspend_mode and guest_suspend passing the
pmutils command and arguments as parameters. This
In bios_supports_mode there is a verification to assert if
the chosen suspend mode is supported by the pmutils tools and,
if not, we see if the Linux sys state files supports it.
This verification is done in the same function, one after
the other, and it works for now. But, when adding a new
suspe
changes in v2 from Marc-Andre Lureau review:
- use error_free() accordingly
- use g_spawn_sync() instead of fork() in run_process_child()
- previous version link:
https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg05499.html
This series adds systemd suspend support for QGA. Some newer
guest
On 20.06.2018 17:13, Vladimir Sementsov-Ogievskiy wrote:
> 20.06.2018 14:26, Eric Blake wrote:
>> On 06/09/2018 10:17 AM, Vladimir Sementsov-Ogievskiy wrote:
>>> Signed-off-by: Vladimir Sementsov-Ogievskiy
>>> ---
>>> qapi/block.json | 23 +++
>>> blockdev-nbd.c | 23 +++
This is a cleanup of the resulting code after detaching
pmutils and Linux sys state file logic:
- remove the SUSPEND_MODE_* macros and use an enumeration
instead. At the same time, drop the switch statements
at the start of each function and use the enumeration
index to get the right binary/argume
Following the same logic of the previous patch, let's also
decouple the suspend logic from guest_suspend into specialized
functions, one for each strategy we support at this moment.
Signed-off-by: Daniel Henrique Barboza
---
qga/commands-posix.c | 170 +++
pmutils isn't being supported by newer OSes like Fedora 27
or Mint. This means that the only suspend option QGA offers
for these guests are writing directly into the Linux sys state
file. This also means that QGA also loses the ability to do
hybrid suspend in those guests - this suspend mode is onl
21.06.2018 02:29, John Snow wrote:
On 06/20/2018 09:04 AM, Vladimir Sementsov-Ogievskiy wrote:
13.06.2018 05:06, John Snow wrote:
We don't need to re-read this list every time, exactly. We can keep it
cached
and delete our copy when we flush to disk.
Because we don't try to flush bitmaps on c
bios_support_mode verifies if the guest has support for a certain
suspend mode but it doesn't inform back which suspend tool
provides it. The caller, guest_suspend, executes all suspend
strategies in order again.
After adding systemd suspend support, bios_support_mode now will
verify for support f
On Mon, 18 Jun 2018 16:36:03 +1000
David Gibson wrote:
> The way we used to handle KVM allowable guest pagesizes for PAPR guests
> required some convoluted checking of memory attached to the guest.
>
> The allowable pagesizes advertised to the guest cpus depended on the memory
> which was attach
* David Hildenbrand (da...@redhat.com) wrote:
> Not needed. Don't expose last_ram_page().
>
> Signed-off-by: David Hildenbrand
Ah nice find; that disappeared part way through the review of my
postcopy series.
Reviewed-by: Dr. David Alan Gilbert
> ---
>
> v1 -> v2:
> - Make "last_ram_page" s
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180618063606.2513-1-da...@gibson.dropbear.id.au
Subject: [Qemu-devel] [PATCH 0/9] spapr: Clean up pagesize handling
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
tot
On 20/06/2018 22:28, John Snow wrote:
>
> ...Oh. So the PIO Setup FIS ... gets generated before the data is sent,
> but we don't copy it to the HBA memory buffers and notify the client
> until afterwards, but this is per-DRQ, I think, and not per-IDE command.
No, the copy happens before: "When a
Hi
On Thu, Jun 21, 2018 at 11:49 AM, Igor Mammedov wrote:
> On Tue, 15 May 2018 14:14:30 +0200
> Marc-André Lureau wrote:
>
>> From: Stefan Berger
>>
>> Implement a virtual memory device for the TPM Physical Presence interface.
>> The memory is located at 0xfffef000 and used by ACPI to send mes
On Mon, Jun 18, 2018 at 08:40:24AM -1000, Richard Henderson wrote:
> This is almost a grab-bag of little improvements to the port.
>
> patches 1-3:
> Fix singlestepping for gdbstub. This has apparently never
> worked, as the first commit has the same bug of not advancing
> the pc when stepp
@pmaydell Is there anything I can do to help speed this up? I can run
tests or make reproduction easier if that helps. But I need direction
what actually helps you.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpa
* Igor Mammedov (imamm...@redhat.com) wrote:
> On Wed, 20 Jun 2018 16:39:45 +0100
> "Dr. David Alan Gilbert (git)" wrote:
>
> > From: "Dr. David Alan Gilbert"
> >
> > Allow a bunch of the info commands to be used in preconfig.
> >
> > version, chardev, name, uuid,memdev, iothreads
> > Were e
Hi Markus,
On 06/08/2018 03:27 AM, Markus Armbruster wrote:
> Philippe Mathieu-Daudé writes:
>
>> Use error_report() + exit() instead of error_setg(&error_fatal),
>> as suggested by the "qapi/error.h" documentation:
>>
>>Please don't error_setg(&error_fatal, ...), use error_report() and
>>
On 20.06.2018 12:08, David Hildenbrand wrote:
> Let's treat this like a separate device. TCG will have to store the
> actual state/time later on.
>
> Include cpu-qom.h in kvm_s390x.h (due to S390CPU) to compile tod-kvm.c.
>
> Signed-off-by: David Hildenbrand
> ---
[...]
> +static void s390_tod_c
We emulate a TLB cache of size SMMU_IOTLB_MAX_SIZE=256.
It is implemented as a hash table whose key is a combination
of the 16b asid and 48b IOVA (Jenkins hash).
Entries are invalidated on TLB invalidation commands, either
globally, or per asid, or per asid/iova.
Signed-off-by: Eric Auger
---
v
This series brings translation configuration caching and IOTLB
emulation. The last patch implements VHOST integration and
allows to run VSMMUv3 along with VHOST emulated end points.
The first patch fixes the passthrough mode bug reported by Jia.
It reworks the translate function and this series ne
Let's cache config data to avoid fetching and parsing STE/CD
structures on each translation. We invalidate them on data structure
invalidation commands.
We put in place a per-smmu mutex to protect the config cache. This
will be useful too to protect the IOTLB cache. The caches can be
accessed with
From: Jia He
In case the STE's config is "Bypass" we currently don't set the
IOMMUTLBEntry perm flags and the access does not succeed. Also
if the config is 0b0xx (Aborted/Reserved), decode_ste and
smmuv3_decode_config currently returns -EINVAL and we don't enter
the expected code path: we record
On TLB invalidation commands, let's call registered
IOMMU notifiers. Those can only be UNMAP notifiers.
SMMUv3 does not support notification on MAP (VFIO).
This patch allows vhost use case where IOTLB API is notified
on each guest IOTLB invalidation.
Signed-off-by: Eric Auger
Reviewed-by: Peter
* Collin Walling (wall...@linux.ibm.com) wrote:
> When a user incorrectly provides an hmp command, an error response will be
> printed that prompts the user to try "help ". However, when
> the command contains multiple parts e.g. "info uuid xyz", only the last
> whitespace delimited string will be
Hi David,
On 06/21/2018 02:37 AM, David Gibson wrote:
> On Wed, Jun 20, 2018 at 03:07:30PM +0200, Eric Auger wrote:
>> Add the kvm_get_max_vm_phys_shift() helper that returns the
>> log of the maximum IPA size supported by KVM. This capability
>> needs to be known to create the VM with a correct I
On 20/06/2018 20:15, Thomas Huth wrote:
> On 20.06.2018 13:07, Gerd Hoffmann wrote:
>> Hi,
>>
>> $subject says all. Noticed while testing the upcoming seabios update.
>> Reproducer:
>>
>> qemu-system-x86_64 -M q35 -m 4G -cdrom
>> Fedora-Workstation-Live-x86_64-28-1.1.iso
>
> Once this has be
Daniel P. Berrangé writes:
> On Thu, Jun 21, 2018 at 10:35:51AM +0200, Markus Armbruster wrote:
>> Fix the following issues:
>>
>> common.py:873:13: E129 visually indented line with same indent as next
>> logical line
>> common.py:1766:5: E741 ambiguous variable name 'l'
>> common.p
On 06/21/2018 01:00 AM, Stafford Horne wrote:
> Thanks for these, I think there are a few white space issues throughout. Do
> you
> mind if I take these and clean them up (indent with space) and work on the DSX
> and Interrupt issue I mentioned earlier?
>
> I can submit all during the next merge
Forgot tests/qapi-schema/test-qapi.py, as usual. To be squashed in:
diff --git a/tests/qapi-schema/test-qapi.py b/tests/qapi-schema/test-qapi.py
index 4512a41504..abbc9c7830 100644
--- a/tests/qapi-schema/test-qapi.py
+++ b/tests/qapi-schema/test-qapi.py
@@ -33,8 +33,8 @@ class QAPISchemaTestVisi
On Thu, Jun 21, 2018 at 08:29:36AM +0200, Cédric Le Goater wrote:
> On 06/18/2018 08:36 AM, David Gibson wrote:
> > The way we used to handle KVM allowable guest pagesizes for PAPR guests
> > required some convoluted checking of memory attached to the guest.
> >
> > The allowable pagesizes adverti
On Thu, Jun 21, 2018 at 08:22:15AM +0200, Cédric Le Goater wrote:
> On 06/18/2018 08:36 AM, David Gibson wrote:
> > The way the POWER Hash Page Table (HPT) MMU is virtualized by KVM HV means
> > that every page that the guest puts in the pagetables must be truly
> > physically contiguous, not just
On Thu, Jun 21, 2018 at 12:29:14PM +0200, Greg Kurz wrote:
> On Mon, 18 Jun 2018 16:36:03 +1000
> David Gibson wrote:
[snip]
> > @@ -304,6 +305,23 @@ static void
> > cap_safe_indirect_branch_apply(sPAPRMachineState *spapr,
> >
> > #define VALUE_DESC_TRISTATE " (broken, workaround, fixed)"
On Thu, Jun 21, 2018 at 11:19:41AM +0200, Greg Kurz wrote:
> On Mon, 18 Jun 2018 16:36:02 +1000
> David Gibson wrote:
>
> > The way the POWER Hash Page Table (HPT) MMU is virtualized by KVM HV means
> > that every page that the guest puts in the pagetables must be truly
> > physically contiguous,
On Thu, Jun 21, 2018 at 07:18:52PM +1000, Alexey Kardashevskiy wrote:
> The following changes since commit 840d3ddc259b70a7b4e1ba367945cc31a45c93c6:
>
> pseries: Update SLOF firmware image to qemu-slof-20180621 (2018-06-21
> 19:16:26 +1000)
>
> are available in the git repos
On 20.06.2018 12:08, David Hildenbrand wrote:
> Never set to anything but 0.
>
> Signed-off-by: David Hildenbrand
> ---
> target/s390x/cpu.c | 1 -
> target/s390x/cpu.h | 1 -
> target/s390x/misc_helper.c | 4 ++--
> 3 files changed, 2 insertions(+), 4 deletions(-)
Reviewed-by:
Use assert() instead of error_setg(&error_abort),
as suggested by the "qapi/error.h" documentation:
Please don't error_setg(&error_fatal, ...), use error_report() and
exit(), because that's more obvious.
Likewise, don't error_setg(&error_abort, ...), use assert().
Signed-off-by: Phili
Use error_report() + exit() instead of error_setg(&error_fatal),
as suggested by the "qapi/error.h" documentation:
Please don't error_setg(&error_fatal, ...), use error_report() and
exit(), because that's more obvious.
This fixes CID 1352173:
"Passing null pointer dt_name to qemu_fdt_no
Hi,
This series converts error_setg(&error_fatal) to error_report() + exit() as
suggested by the "qapi/error.h" documentation.
This reduce Coverity and Clang static analyzer positive falses.
See http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg07585.html:
On 07/24/2017 04:52 PM, E
Use error_report() + exit() instead of error_setg(&error_fatal),
as suggested by the "qapi/error.h" documentation:
Please don't error_setg(&error_fatal, ...), use error_report() and
exit(), because that's more obvious.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Eric Auger
Reviewed
On 06/21/2018 05:35 AM, Markus Armbruster wrote:
> Fix the following issues:
>
> common.py:873:13: E129 visually indented line with same indent as next
> logical line
> common.py:1766:5: E741 ambiguous variable name 'l'
> common.py:1784:1: E305 expected 2 blank lines after class or fu
1 - 100 of 394 matches
Mail list logo